Home/ Questions/Q 8927467
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-06-15T08:13:35+00:00

In the below script I added or operator to check the condition, Like if

  • 0

In the below script I added or operator to check the condition, Like if it is “ALL TCP” or “ALL UDP” item must get disabled, Please suggest me to modify the code. 

<script type="text/javascript">
    function chgJob(pThis, pRow)
    {
        if ((pThis.value != 'All TCP') || (pThis.value != 'All UDP'))
        {
            $x_disableItem('f02_' + pRow, false);
        }
        else
        {
            $x_disableItem('f02_' + pRow, true);
        }
    }
</script>
Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    A string can never be equal to 'All TCP' and equal to 'All UDP' at the same time, so the condition will always be true.

    You want to use the && operator instead of the || operator:

    if ((pThis.value != 'All TCP') && (pThis.value != 'All UDP'))

    Edit:

    As Cerbrus pointed out, you don’t need the if statement at all, you can use the boolean expression directly by inverting it (x==y || x==z is the same as !(x!=y && x!=z)):

    <script type="text/javascript">
function chgJob(pThis, pRow) {
    $x_disableItem('f02_' + pRow, pThis.value == 'All TCP' || pThis.value == 'All UDP');
}
</script>
    • 0