use "&&" and "||" instead of "and" and "or"
authorDenis Ovsienko <denis@ovsienko.info>
Wed, 23 Mar 2016 21:56:32 +0000 (21:56 +0000)
committerDenis Ovsienko <denis@ovsienko.info>
Wed, 23 Mar 2016 21:58:04 +0000 (21:58 +0000)
This change prevents unexpected side effects, that, for instance, used
to require two workarounds in iptree_markup_collapsion() and one in
transformRequestData(). The root cause of the problem is that "and" and
"or" are standalone operators: though their associativity is the same as
that of "&&" and "||", the order of precedence is not just different but
is lower than that of "=". The code style article now explains this.

* syncdomain.php
* getNetUsageAJAX()
* authenticate()
* permitted()
* gotClearanceForTagChain()
* authenticated_via_ldap()
* queryLDAPServer()
* RackCodeParser::accept()
* RackCodeParser::un_expr()
* checkAutotagName()
* findCtxModWarnings()
* listCells()
* fillNetKids()
* amplifyCell()
* commitLinkEntities()
* commitUpdateEntityLink()
* updateAddress()
* generateEntityAutoTags()
* createIPv4Prefix()
* newPortForwarding()
* getDomainVLANs()
* getDomainVLANList()
* upd8021QPort()
* setConfigVar()
* setUserConfigVar()
* xos12ReadLLDPStatus()
* xos12ReadInterfaceStatus()
* vrpReadLLDPStatus()
* ftos8ReadLLDPStatus()
* eos4ReadLLDPStatus()
* ros11ReadLLDPStatus()
* jun10TranslatePushQueue()
* jun10Read8021QConfig()
* ftos8ReadInterfaceStatus()
* eos4ReadInterfaceStatus()
* ucsReadInventory()
* isInnoDBSupported()
* platform_is_ok()
* stringTrace()
* assertUIntArg()
* assertStringArg()
* assertBoolArg()
* genericAssertion()
* rectHeight()
* highlightObject()
* ip4_mask()
* ip6_mask()
* l2addressForDatabase()
* sortTokenize()
* pokeNode()
* peekNode()
* addTraceToNodes()
* redirectIfNecessary()
* getObjectiveTagTree()
* getCellFilter()
* IPNetworkCmp()
* iptree_fill()
* ip_next()
* ip_prev()
* treeApplyFunc()
* getTerminalNetworks()
* iptree_markup_collapsion()
* getPortListPrefs()
* getUnlinkedPortTypeOptions()
* groupIntsToRanges()
* array_values_same()
* buildVLANFilter()
* matchVLANFilter()
* generate8021QDeployOps()
* filter8021QChangeRequests()
* getEmployedVlans()
* get8021QSyncOptions()
* exec8021QDeploy()
* saveDownlinksReverb()
* initiateUplinksReverb()
* recalc8021QPorts()
* authorize8021QChangeRequests()
* compareDecomposedPortNames()
* searchEntitiesByText()
* buildSearchRedirectURL()
* setMessage()
* isEthernetPort()
* getConfigVar()
* getObjectTypeChangeOptions()
* apply8021qChangeRequest()
* fillIPNetsCorrelation()
* fillIPSpareListBstr()
* isIPNetworkEmpty()
* listConstraint()
* checkTypeAndAttribute()
* init.php
* install.php
* not_already_installed()
* init_config()
* get_process_owner()
* check_config_access()
* init_database_dynamic()
* render8021QOrderForm()
* renderVLANDomainListEditor()
* renderVLANDomainVLANList()
* renderObject8021QPorts()
* getAccessPortControlCode()
* renderTrunkPortControls()
* renderVLANInfo()
* renderObject8021QSync()
* renderObject8021QSyncPreview()
* renderConfigMainpage()
* renderTagRowForEditor()
* printNiftySelect()
* transformRequestData()
* getRenderedIPv4NetCapacity()
* serializeTags()
* printTagTRs()
* renderEntitySummary()
* niftyString()
* showLogoutURL()
* renderRackspace()
* renderRow()
* printObjectDetailsForRenderRack()
* renderEditRackForm()
* renderObject()
* showMessageOrError()
* renderRackSpaceForObject()
* renderIPSpaceRecords()
* renderIPSpace()
* renderIPNetwork()
* renderIPAddress()
* renderIPAddressProperties()
* renderNATv4ForObject()
* renderLocationPage()
* renderEditLocationForm()
* renderLivePTR()
* buildTagCheckboxRows()
* renderCellFilterPortlet()
* renderFile()
* renderFilesPortlet()
* printIPNetInfoTDs()
* renderCell()
* getFilePreviewCode()
* renderDiscoveredNeighbors()
* switchportInfoJS()
* addIPAllocation()
* processGridForm()
* updateObjectAttributes()
* addLotOfObjects()
* save8021QPorts()
* updVSTRule()
* importDPData()
* cleanupUCS()
* getOpspec()
* buildOpspecColumns()
* findSparePorts()
* handlePopupPortLink()
* detectDeviceBreed()
* prepareRealServersTable()
* SLBTriplet::getTriplets()
* MacroParser::addMacro()
* addRStoRSPool()
* commitUpdateRS()
* doSwitchSNMPmining()
* createTrueColorOrThrow()
* renderProgressBar4Image()
* renderImagePreview()
* proxyStaticURI()
* trigger_liveports()
* trigger_object_8021qorder()
* trigger_8021q_configured()
* trigger_anyDP()
* triggerCactiGraphs()
* triggerMuninGraphs()
* getDBUpgradePath()
* renderUpgraderHTML()
* index.php

25 files changed:
scripts/syncdomain.php
wwwroot/inc/ajax-interface.php
wwwroot/inc/auth.php
wwwroot/inc/code.php
wwwroot/inc/database.php
wwwroot/inc/deviceconfig.php
wwwroot/inc/dictionary.php
wwwroot/inc/exceptions.php
wwwroot/inc/functions.php
wwwroot/inc/init.php
wwwroot/inc/install.php
wwwroot/inc/interface-8021q.php
wwwroot/inc/interface-config.php
wwwroot/inc/interface-lib.php
wwwroot/inc/interface.php
wwwroot/inc/ophandlers.php
wwwroot/inc/popup.php
wwwroot/inc/remote.php
wwwroot/inc/slb-interface.php
wwwroot/inc/slb.php
wwwroot/inc/snmp.php
wwwroot/inc/solutions.php
wwwroot/inc/triggers.php
wwwroot/inc/upgrade.php
wwwroot/index.php

index fcee79b..c209ef5 100755 (executable)
@@ -153,7 +153,7 @@ foreach ($switch_queue as $object)
                }
                $i_am_child = (0 === $fork_res = pcntl_fork());
        }
-       if (! $do_fork or $i_am_child)
+       if (! $do_fork || $i_am_child)
        {
                try
                {
index 073d6c4..87974f4 100644 (file)
@@ -299,7 +299,7 @@ function getNetUsageAJAX()
        list ($ip, $mask) = explode ('/', $_REQUEST['net_id']);
        $ip_bin = ip_parse ($ip);
        $net = spotNetworkByIP ($ip_bin, $mask + 1);
-       if (! isset ($net) or $net['mask'] != $mask)
+       if (! isset ($net) || $net['mask'] != $mask)
                $net = constructIPRange ($ip_bin, $mask);
        loadIPAddrList ($net);
        echo getRenderedIPNetCapacity ($net);
index 1838187..118e645 100644 (file)
@@ -27,11 +27,11 @@ function authenticate ()
                $script_mode,
                $require_local_account;
        // Phase 1. Assert basic pre-requisites, short-circuit the logout request.
-       if (!isset ($user_auth_src) or !isset ($require_local_account))
+       if (! isset ($user_auth_src) || ! isset ($require_local_account))
                throw new RackTablesError ('secret.php: either user_auth_src or require_local_account are missing', RackTablesError::MISCONFIGURED);
        if (isset ($_REQUEST['logout']))
        {
-               if (isset ($user_auth_src) and 'saml' == $user_auth_src)
+               if (isset ($user_auth_src) && 'saml' == $user_auth_src)
                        saml_logout ();
                throw new RackTablesError ('', RackTablesError::NOT_AUTHENTICATED); // Reset browser credentials cache.
        }
@@ -44,9 +44,9 @@ function authenticate ()
                case 'ldap' == $user_auth_src:
                        if
                        (
-                               ! isset ($_SERVER['PHP_AUTH_USER']) or
-                               $_SERVER['PHP_AUTH_USER'] == '' or
-                               ! isset ($_SERVER['PHP_AUTH_PW']) or
+                               ! isset ($_SERVER['PHP_AUTH_USER']) ||
+                               $_SERVER['PHP_AUTH_USER'] == '' ||
+                               ! isset ($_SERVER['PHP_AUTH_PW']) ||
                                $_SERVER['PHP_AUTH_PW'] == ''
                        )
                                throw new RackTablesError ('', RackTablesError::NOT_AUTHENTICATED);
@@ -73,7 +73,7 @@ function authenticate ()
        }
        // Phase 3. Handle local account requirement, pull user tags into security context.
        $userinfo = constructUserCell ($remote_username);
-       if ($require_local_account and !isset ($userinfo['user_id']))
+       if ($require_local_account && ! isset ($userinfo['user_id']))
                throw new RackTablesError ('', RackTablesError::NOT_AUTHENTICATED);
        $user_given_tags = $userinfo['etags'];
        $auto_tags = array_merge ($auto_tags, $userinfo['atags']);
@@ -89,7 +89,7 @@ function authenticate ()
                                $remote_username;
                        return; // success
                // When using LDAP, leave a mean to fix things. Admin user is always authenticated locally.
-               case array_key_exists ('user_id', $userinfo) and $userinfo['user_id'] == 1:
+               case array_key_exists ('user_id', $userinfo) && $userinfo['user_id'] == 1:
                case 'database' == $user_auth_src:
                        $remote_displayname = $userinfo['user_realname'] != '' ?
                                $userinfo['user_realname'] :
@@ -126,7 +126,7 @@ function permitted ($p = NULL, $t = NULL, $o = NULL, $annex = array())
                $p = $pageno;
        if ($t === NULL)
                $t = $tabno;
-       if ($o === NULL and $op != '') // $op can be set to empty string
+       if ($o === NULL && $op != '') // $op can be set to empty string
                $o = $op;
        $my_auto_tags = $auto_tags;
        $my_auto_tags[] = array ('tag' => '$page_' . $p);
@@ -201,7 +201,7 @@ function gotClearanceForTagChain ($const_base)
                        case 'SYNT_ADJUSTMENT':
                                if
                                (
-                                       eval_expression ($sentence['condition'], $context) and
+                                       eval_expression ($sentence['condition'], $context) &&
                                        processAdjustmentSentence ($sentence['modlist'], $expl_tags)
                                ) // recalculate implicit chain only after actual change, not just on matched condition
                                {
@@ -330,7 +330,7 @@ function authenticated_via_ldap ($username, $password, &$ldap_displayname)
 
                if
                (
-                       $LDAP_options['cache_retry'] > $LDAP_options['cache_refresh'] or
+                       $LDAP_options['cache_retry'] > $LDAP_options['cache_refresh'] ||
                        $LDAP_options['cache_refresh'] > $LDAP_options['cache_expiry']
                )
                        throw new RackTablesError ('LDAP misconfiguration: refresh/retry/expiry mismatch', RackTablesError::MISCONFIGURED);
@@ -522,18 +522,18 @@ function queryLDAPServer ($username, $password)
                $last_successful_server !== $success_server)
                saveScript ('LDAPLastSuccessfulServer', $success_server);
 
-       if (array_key_exists ('options', $LDAP_options) and is_array ($LDAP_options['options']))
+       if (array_key_exists ('options', $LDAP_options) && is_array ($LDAP_options['options']))
                foreach ($LDAP_options['options'] as $opt_code => $opt_value)
                        ldap_set_option ($connect, $opt_code, $opt_value);
 
        // Decide on the username we will actually authenticate for.
-       if (isset ($LDAP_options['domain']) and $LDAP_options['domain'] != '')
+       if (isset ($LDAP_options['domain']) && $LDAP_options['domain'] != '')
                $auth_user_name = $username . "@" . $LDAP_options['domain'];
        elseif
        (
-               isset ($LDAP_options['search_dn']) and
-               $LDAP_options['search_dn'] != '' and
-               isset ($LDAP_options['search_attr']) and
+               isset ($LDAP_options['search_dn']) &&
+               $LDAP_options['search_dn'] != '' &&
+               isset ($LDAP_options['search_attr']) &&
                $LDAP_options['search_attr'] != ''
        )
        {
@@ -587,11 +587,11 @@ function queryLDAPServer ($username, $password)
        // Displayed name only makes sense for authenticated users anyway.
        if
        (
-               isset ($LDAP_options['displayname_attrs']) and
-               $LDAP_options['displayname_attrs'] != '' and
-               isset ($LDAP_options['search_dn']) and
-               $LDAP_options['search_dn'] != '' and
-               isset ($LDAP_options['search_attr']) and
+               isset ($LDAP_options['displayname_attrs']) &&
+               $LDAP_options['displayname_attrs'] != '' &&
+               isset ($LDAP_options['search_dn']) &&
+               $LDAP_options['search_dn'] != '' &&
+               isset ($LDAP_options['search_attr']) &&
                $LDAP_options['search_attr'] != ''
        )
        {
@@ -621,8 +621,8 @@ function queryLDAPServer ($username, $password)
                        for ($i = 0; $i < $info[0][$LDAP_options['group_attr']]['count']; $i++)
                                if
                                (
-                                       preg_match ($LDAP_options['group_filter'], $info[0][$LDAP_options['group_attr']][$i], $matches)
-                                       and validTagName ('$lgcn_' . $matches[1], TRUE)
+                                       preg_match ($LDAP_options['group_filter'], $info[0][$LDAP_options['group_attr']][$i], $matches) &&
+                                       validTagName ('$lgcn_' . $matches[1], TRUE)
                                )
                                        $ret['memberof'][] = '$lgcn_' . $matches[1];
        }
index 7f8e0be..9a38765 100644 (file)
@@ -78,10 +78,10 @@ class RackCodeParser
                $value = $this->lex_value;
                if
                (
-                       $curr !== NULL and (
-                               !isset ($sym) or
-                               is_array ($sym) and in_array ($curr, $sym) or
-                               ! is_array ($sym) and $sym == $curr
+                       $curr !== NULL && (
+                               ! isset ($sym) ||
+                               is_array ($sym) && in_array ($curr, $sym) ||
+                               ! is_array ($sym) && $sym == $curr
                        )
                )
                {
@@ -432,7 +432,7 @@ class RackCodeParser
                        );
                elseif ($k = $this->accept ('LEX_PREDICATE'))
                {
-                       if ($this->prog_mode and ! isset ($this->defined_preds[$k]))
+                       if ($this->prog_mode && ! isset ($this->defined_preds[$k]))
                                throw new RCParserError ("Undefined predicate [$k] refered");
                        return array(
                                'type' => 'LEX_PREDICATE',
@@ -520,7 +520,7 @@ function checkAutotagName ($atag_name)
                case (preg_match ('/^\$username_/', $atag_name)):
                        $recid = preg_replace ('/^\$username_/', '', $atag_name);
                        global $require_local_account;
-                       if ($require_local_account and NULL === getUserIDByUsername ($recid))
+                       if ($require_local_account && NULL === getUserIDByUsername ($recid))
                                return "Local user account '${recid}' does not exist.";
                        break;
                case (preg_match ('/^\$page_([\p{L}0-9]+)$/u', $atag_name, $m)):
@@ -617,7 +617,7 @@ function findCtxModWarnings ($modlist)
 {
        $ret = array();
        foreach ($modlist as $mod)
-               if (($mod['op'] == 'insert' or $mod['op'] == 'remove') and NULL === getTagByName ($mod['tag']))
+               if (($mod['op'] == 'insert' || $mod['op'] == 'remove') && NULL === getTagByName ($mod['tag']))
                        $ret[] = array
                        (
                                'header' => refRCLineno ($mod['lineno']),
index d8317c8..ac90375 100644 (file)
@@ -450,7 +450,7 @@ function listCells ($realm, $parent_id = 0)
                $query .= ($alias == $expression ? "${SQLinfo['table']}.${alias}" : "${expression} as ${alias}") . ', ';
        $query = trim($query, ', ');
        $query .= " FROM ${SQLinfo['table']}";
-       if (isset ($SQLinfo['pidcolumn']) and $parent_id)
+       if (isset ($SQLinfo['pidcolumn']) && $parent_id)
        {
                $query .= " WHERE ${SQLinfo['table']}.${SQLinfo['pidcolumn']} = ?";
                $qparams[] = $parent_id;
@@ -543,7 +543,7 @@ function listCells ($realm, $parent_id = 0)
                        break;
                }
        }
-       if ($realm == 'ipv4net' or $realm == 'ipv6net')
+       if ($realm == 'ipv4net' || $realm == 'ipv6net')
                fillIPNetsCorrelation ($ret);
 
        foreach (array_keys ($ret) as $entity_id)
@@ -707,7 +707,7 @@ ORDER BY ip, mask
        unset ($result);
 
        fillIPNetsCorrelation ($nets, 1);
-       if (is_array ($nets[0]) and $nets[0]['id'] == $net_cell['id'])
+       if (is_array ($nets[0]) && $nets[0]['id'] == $net_cell['id'])
        {
                $net_cell['spare_ranges'] = $nets[0]['spare_ranges'];
                $net_cell['kidc'] = $nets[0]['kidc'];
@@ -761,7 +761,7 @@ function amplifyCell (&$record, $dummy = NULL)
                        $record[$row['unit_no']][$loclist[$row['atom']]]['state'] = $row['state'];
                        $record[$row['unit_no']][$loclist[$row['atom']]]['object_id'] = $row['object_id'];
                        $record[$row['unit_no']][$loclist[$row['atom']]]['hl'] = $row['has_problems'] == 'yes' ? 'w' : '';
-                       if ($row['state'] == 'T' and $row['object_id'] != NULL)
+                       if ($row['state'] == 'T' && $row['object_id'] != NULL)
                                $mounted_objects[$row['object_id']] = TRUE;
                }
 
@@ -1124,8 +1124,8 @@ function commitLinkEntities ($parent_entity_type, $parent_entity_id, $child_enti
        // a location's parent may not be one of its children
        if
        (
-               $parent_entity_type == 'location' and
-               $child_entity_type == 'location' and
+               $parent_entity_type == 'location' &&
+               $child_entity_type == 'location' &&
                in_array ($parent_entity_id, getLocationChildrenList ($child_entity_id))
        )
                throw new RackTablesError ("Circular reference for location ${parent_entity_id}", RackTablesError::INTERNAL);
@@ -1133,8 +1133,8 @@ function commitLinkEntities ($parent_entity_type, $parent_entity_id, $child_enti
        // an object's container may not be one of its contained objects
        if
        (
-               $parent_entity_type == 'object' and
-               $child_entity_type == 'object' and
+               $parent_entity_type == 'object' &&
+               $child_entity_type == 'object' &&
                in_array ($parent_entity_id, getObjectContentsList ($child_entity_id))
        )
                throw new RackTablesError ("Circular reference for object ${parent_entity_id}", RackTablesError::INTERNAL);
@@ -1161,8 +1161,8 @@ function commitUpdateEntityLink
        // a location's parent may not be one of its children
        if
        (
-               $new_parent_entity_type == 'location' and
-               $new_child_entity_type == 'location' and
+               $new_parent_entity_type == 'location' &&
+               $new_child_entity_type == 'location' &&
                in_array ($new_parent_entity_id, getLocationChildrenList ($new_child_entity_id))
        )
                throw new RackTablesError ("Circular reference for location ${new_parent_entity_id}", RackTablesError::INTERNAL);
@@ -1170,8 +1170,8 @@ function commitUpdateEntityLink
        // an object's container may not be one of its contained objects
        if
        (
-               $new_parent_entity_type == 'object' and
-               $new_child_entity_type == 'object' and
+               $new_parent_entity_type == 'object' &&
+               $new_child_entity_type == 'object' &&
                in_array ($new_parent_entity_id, getObjectContentsList ($new_child_entity_id))
        )
                throw new RackTablesError ("Circular reference for object ${new_parent_entity_id}", RackTablesError::INTERNAL);
@@ -2626,7 +2626,7 @@ function updateAddress ($ip_bin, $name = '', $reserved = 'no', $comment)
        // If not, retain the old value.
        $comment = (func_num_args () == 4 ) ? $comment : $old_comment;
        $new_row = array ('name' => $name, 'comment' => $comment, 'reserved' => $reserved);
-       $new_row_empty = !($name != '' or $comment != '' or $reserved != 'no');
+       $new_row_empty = ! ($name != '' || $comment != '' || $reserved != 'no');
 
        unset ($result);
        $messages = array ();
@@ -4139,7 +4139,7 @@ function generateEntityAutoTags ($cell)
                                $ret[] = array ('tag' => '$nameless');
                        if (validTagName ('$cn_' . $cell['name'], TRUE))
                                $ret[] = array ('tag' => '$cn_' . $cell['name']);
-                       if ($cell['rack_id'] == '' and $cell['container_id'] == '')
+                       if ($cell['rack_id'] == '' && $cell['container_id'] == '')
                                $ret[] = array ('tag' => '$unmounted');
                        if (!$cell['nports'])
                                $ret[] = array ('tag' => '$portless');
@@ -4434,7 +4434,7 @@ function createIPv4Prefix ($range = '', $name = '', $is_connected = FALSE, $tagl
        $network_id = lastInsertID();
        lastCreated ('ipv4net', $network_id);
 
-       if ($is_connected and $mask < 31)
+       if ($is_connected && $mask < 31)
        {
                updateV4Address ($net['ip_bin'], 'network', 'yes');
                updateV4Address (ip_last ($net), 'broadcast', 'yes');
@@ -4533,9 +4533,9 @@ function newPortForwarding ($object_id, $localip_bin, $localport, $remoteip_bin,
        }
        else
        {
-               if ( $localport <= 0 or $localport >= 65536 )
+               if ($localport <= 0 || $localport >= 65536)
                        throw new InvalidArgException ('localport', $localport, 'Invaild port');
-               if ( $remoteport <= 0 or $remoteport >= 65536 )
+               if ($remoteport <= 0 || $remoteport >= 65536)
                        throw new InvalidArgException ('remoteport', $remoteport, 'Invaild port');
        }
 
@@ -5134,7 +5134,7 @@ function getDomainGroupMembers ($vdom_group_id)
 function getDomainVLANs ($vdom_id, $strict = FALSE)
 {
        $self = __FUNCTION__;
-       if (! $strict and $members = getDomainGroupMembers ($vdom_id))
+       if (! $strict && $members = getDomainGroupMembers ($vdom_id))
        {
                $ret = $self ($vdom_id, TRUE);
                foreach ($members as $member_vdom_id)
@@ -5186,7 +5186,7 @@ END
 // Otherwise the vlans of group subdomains are not returned.
 function getDomainVLANList ($vdom_id, $strict = FALSE)
 {
-       if (! $strict and $members = getDomainGroupMembers ($vdom_id))
+       if (! $strict && $members = getDomainGroupMembers ($vdom_id))
        {
                $self = __FUNCTION__;
                $ret = $self ($vdom_id, TRUE);
@@ -5505,10 +5505,10 @@ function upd8021QPort ($instance = 'desired', $object_id, $port_name, $port, $be
        // This function indicates an error, but doesn't revert it, so it is
        // assummed, that the calling function performs necessary transaction wrapping.
        // A record on a port with none VLANs allowed makes no sense regardless of port mode.
-       if ($port['mode'] != 'trunk' and !count ($port['allowed']))
+       if ($port['mode'] != 'trunk' && ! count ($port['allowed']))
                return 0;
        $changed = 0;
-       if (! isset ($before) or $before['mode'] != $port['mode'])
+       if (! isset ($before) || $before['mode'] != $port['mode'])
                $changed += usePreparedUpdateBlade
                (
                        $tablemap_8021q[$instance]['pvm'],
@@ -5549,13 +5549,13 @@ function upd8021QPort ($instance = 'desired', $object_id, $port_name, $port, $be
                        array_merge (array ($object_id, $port_name), $where[1])
                );
 
-       if (! $port['native'] and (! isset ($before) or in_array ($before['native'], $port['allowed'])))
+       if (! $port['native'] && (! isset ($before) || in_array ($before['native'], $port['allowed'])))
                $changed += usePreparedDeleteBlade
                (
                        $tablemap_8021q[$instance]['pnv'],
                        array ('object_id' => $object_id, 'port_name' => $port_name)
                );
-       elseif ($port['native'] and (! isset ($before) or $before['native'] != $port['native']))
+       elseif ($port['native'] && (! isset ($before) || $before['native'] != $port['native']))
                $changed += usePreparedExecuteBlade
                (
                        'REPLACE INTO ' . $tablemap_8021q[$instance]['pnv'] .
@@ -5563,7 +5563,7 @@ function upd8021QPort ($instance = 'desired', $object_id, $port_name, $port, $be
                        array ($object_id, $port_name, $port['native'])
                );
 
-       if ($instance == 'desired' and $changed)
+       if ($instance == 'desired' && $changed)
                callHook ('portConfChanged', $object_id, $port_name, $port);
        return $changed ? 1 : 0;
 }
@@ -5647,7 +5647,7 @@ function setConfigVar ($varname, $varvalue)
                throw new InvalidArgException ('varname', $varname, 'a hidden variable cannot be changed');
        if ($varvalue == '' && $var['emptyok'] != 'yes')
                throw new InvalidArgException ('varvalue', $varvalue, "'${varname}' must have a non-empty value");
-       if ($varvalue != '' && $var['vartype'] == 'uint' && (!is_numeric ($varvalue) or $varvalue < 0 ))
+       if ($varvalue != '' && $var['vartype'] == 'uint' && (! is_numeric ($varvalue) || $varvalue < 0 ))
                throw new InvalidArgException ('varvalue', $varvalue, "'${varname}' must be an unsigned integer");
        // Update cache only if the changes went into DB.
        usePreparedUpdateBlade ('Config', array ('varvalue' => $varvalue), array ('varname' => $varname));
@@ -5668,7 +5668,7 @@ function setUserConfigVar ($varname, $varvalue)
                throw new InvalidArgException ('varname', $varname, 'a hidden variable cannot be changed');
        if ($varvalue == '' && $var['emptyok'] != 'yes')
                throw new InvalidArgException ('varvalue', $varvalue, "'${varname}' must have a non-empty value");
-       if ($varvalue != '' && $var['vartype'] == 'uint' && (!is_numeric ($varvalue) or $varvalue < 0 ))
+       if ($varvalue != '' && $var['vartype'] == 'uint' && (! is_numeric ($varvalue) || $varvalue < 0 ))
                throw new InvalidArgException ('varvalue', $varvalue, "'${varname}' must be an unsigned integer");
        // Update cache only if the changes went into DB.
        usePreparedExecuteBlade
index 41311e0..2801655 100644 (file)
@@ -95,8 +95,8 @@ function xos12ReadLLDPStatus ($input)
                case preg_match ('/^    - System Name: "(.+)"$/', $line, $matches):
                        if
                        (
-                               array_key_exists ('current', $ret) and
-                               array_key_exists ('local_port', $ret['current']) and
+                               array_key_exists ('current', $ret) &&
+                               array_key_exists ('local_port', $ret['current']) &&
                                array_key_exists ('remote_port', $ret['current'])
                        )
                                $ret[$ret['current']['local_port']][] = array
@@ -119,9 +119,9 @@ function xos12ReadInterfaceStatus ($input)
                if (preg_match('/^(\d+|\d:\d+)\s+.*\s+([ED])\s+([AR])\s+/', $line, $m))
                {
                        $portname = $m[1];
-                        if ($m[2] == 'E' and $m[3] == 'A')
+                       if ($m[2] == 'E' && $m[3] == 'A')
                                $status = 'up';
-                       elseif ($m[2] == 'E' and $m[3] == 'R' )
+                       elseif ($m[2] == 'E' && $m[3] == 'R')
                                $status = 'down';
                        elseif ($m[2] == 'D')
                                $status = 'disabled';
@@ -183,8 +183,8 @@ function vrpReadLLDPStatus ($input)
                case preg_match ('/^Sys(?:tem)? ?name\s*:\s*(.+)$/i', $line, $matches):
                        if
                        (
-                               array_key_exists ('current', $ret) and
-                               array_key_exists ('PortIdSubtype', $ret['current']) and
+                               array_key_exists ('current', $ret) &&
+                               array_key_exists ('PortIdSubtype', $ret['current']) &&
                                array_key_exists ('local_port', $ret['current'])
                        )
                        {
@@ -235,10 +235,10 @@ function ftos8ReadLLDPStatus ($input)
                case preg_match ('/^    Remote System Name:  (.+)$/', $line, $matches):
                        if
                        (
-                               array_key_exists ('current', $ret) and
-                               array_key_exists ('remote_subtype', $ret['current']) and
-                               in_array ($ret['current']['remote_subtype'], $valid_subtypes) and
-                               array_key_exists ('remote_port', $ret['current']) and
+                               array_key_exists ('current', $ret) &&
+                               array_key_exists ('remote_subtype', $ret['current']) &&
+                               in_array ($ret['current']['remote_subtype'], $valid_subtypes) &&
+                               array_key_exists ('remote_port', $ret['current']) &&
                                array_key_exists ('local_port', $ret['current'])
                        )
                                $ret[$ret['current']['local_port']][] = array
@@ -280,10 +280,10 @@ function eos4ReadLLDPStatus ($input)
                case preg_match ('/^    - System Name: "(.+)"$/', $line, $matches):
                        if
                        (
-                               array_key_exists ('current', $ret) and
-                               array_key_exists ('remote_subtype', $ret['current']) and
-                               in_array ($ret['current']['remote_subtype'], $valid_subtypes) and
-                               array_key_exists ('remote_port', $ret['current']) and
+                               array_key_exists ('current', $ret) &&
+                               array_key_exists ('remote_subtype', $ret['current']) &&
+                               in_array ($ret['current']['remote_subtype'], $valid_subtypes) &&
+                               array_key_exists ('remote_port', $ret['current']) &&
                                array_key_exists ('local_port', $ret['current'])
                        )
                                $ret[$ret['current']['local_port']][] = array
@@ -317,8 +317,8 @@ function ros11ReadLLDPStatus ($input)
                case preg_match ('/^System Name: (.+)$/', $line, $m):
                        if
                        (
-                               array_key_exists ('current', $ret) and
-                               array_key_exists ('remote_port', $ret['current']) and
+                               array_key_exists ('current', $ret) &&
+                               array_key_exists ('remote_port', $ret['current']) &&
                                array_key_exists ('local_port', $ret['current'])
                        )
                                $ret[$ret['current']['local_port']][] = array
@@ -1725,7 +1725,7 @@ function jun10TranslatePushQueue ($dummy_object_id, $queue, $vlan_names)
                                {
                                        $vlan = array_shift ($cmd['vlans']);
                                        $ret .= "$pre $vlan\n";
-                                       if ($del and isset ($vlan_names[$vlan]))
+                                       if ($del && isset ($vlan_names[$vlan]))
                                                $ret .= "$pre ${vlan_names[$vlan]}\n";
                                }
                        break;
@@ -2221,7 +2221,7 @@ function jun10Read8021QConfig ($input)
                        break;
                elseif (preg_match ('/^(\S+)(?:\s+{|;)$/', $line, $m))
                        $current_group = $m[1];
-               elseif (isset ($current_group) and preg_match ('/^\s*family ethernet-switching\b/', $line))
+               elseif (isset ($current_group) && preg_match ('/^\s*family ethernet-switching\b/', $line))
                        throw new RTGatewayError ("Config-group '$current_group' contains switchport commands, which is not supported");
        }
 
@@ -2240,7 +2240,7 @@ function jun10Read8021QConfig ($input)
                $line_class = 'line-other';
                if (preg_match ('/# END OF CONFIG|^(interface-range )?(\S+)\s+{$/', $line, $m)) // line starts with interface name
                { // found interface section opening, or end-of-file
-                       if (isset ($current['name']) and $current['is_ethernet'])
+                       if (isset ($current['name']) && $current['is_ethernet'])
                        {
                                // add previous interface to the results
                                if (! isset ($current['config']['mode']))
@@ -2255,8 +2255,8 @@ function jun10Read8021QConfig ($input)
                                                $current['config']['allowed'] = array();
                                }
                                if (
-                                       $current['config']['mode'] == 'trunk' and
-                                       $current['config']['native'] != 0 and
+                                       $current['config']['mode'] == 'trunk' &&
+                                       $current['config']['native'] != 0 &&
                                        ! in_array ($current['config']['native'], $current['config']['allowed'])
                                )
                                        $current['config']['allowed'][] = $current['config']['native'];
@@ -2287,9 +2287,9 @@ function jun10Read8021QConfig ($input)
                        $current['is_ethernet'] = TRUE;
                        $current['indent'] = $m[1];
                }
-               elseif (isset ($current['indent']) and $line == $current['indent'] . '}')
+               elseif (isset ($current['indent']) && $line == $current['indent'] . '}')
                        $current['indent'] = NULL;
-               elseif ($current['is_ethernet'] and isset ($current['indent']))
+               elseif ($current['is_ethernet'] && isset ($current['indent']))
                {
                        $line_class = 'line-8021q';
                        if (preg_match ('/^\s+port-mode (trunk|access);/', $line, $m))
@@ -2304,7 +2304,7 @@ function jun10Read8021QConfig ($input)
                                        $item = trim ($item);
                                        if (preg_match ('/^(\d+)(?:-(\d+))?$/', $item, $m))
                                        {
-                                               if (isset ($m[2]) and $m[2] > $m[1])
+                                               if (isset ($m[2]) && $m[2] > $m[1])
                                                        $members = array_merge (range ($m[1], $m[2]), $members);
                                                else
                                                        $members[] = $m[1];
@@ -2939,7 +2939,7 @@ function ftos8ReadInterfaceStatus ($text)
                else
                {
                        $fields = explodeTableLine ($line, $table_schema);
-                       if (! empty ($fields['Port']) and ! empty ($fields['Speed']) and ! empty ($fields['Duplex']))
+                       if (! empty ($fields['Port']) && ! empty ($fields['Speed']) && ! empty ($fields['Duplex']))
                        {
                                $status = strtolower ($fields['Status']);
                                if ($status != 'up' && $status != 'down')
@@ -2969,7 +2969,7 @@ function eos4ReadInterfaceStatus ($text)
                else
                {
                        $fields = explodeTableLine ($line, $table_schema);
-                       if (! empty ($fields['Port']) and ! empty ($fields['Speed']) and ! empty ($fields['Duplex']))
+                       if (! empty ($fields['Port']) && ! empty ($fields['Speed']) && ! empty ($fields['Duplex']))
                        {
                                $status = strtolower ($fields['Status']);
                                if ($status == 'connected')
@@ -3330,7 +3330,7 @@ function ucsReadInventory ($text)
                        $tmp = array();
                }
        # validate the array
-       if (count ($replies) != 2 and count ($replies) != 1)
+       if (count ($replies) != 2 && count ($replies) != 1)
                throw new RTGatewayError ('replies count does not match commands count');
        if ($replies[0]['code'] != 'OK')
                throw new RTGatewayError ('UCS login failed');
index f1fb03b..09ae103 100644 (file)
@@ -43,7 +43,7 @@ function isInnoDBSupported ()
        $dbxlink->query("CREATE TRIGGER `trigger_test` BEFORE INSERT ON `innodb_test` FOR EACH ROW BEGIN END");
        $trigger_row = $dbxlink->query("SELECT COUNT(*) AS count FROM information_schema.TRIGGERS WHERE TRIGGER_SCHEMA = SCHEMA() AND TRIGGER_NAME = 'trigger_test'")->fetch(PDO::FETCH_ASSOC);
        $dbxlink->query("DROP TABLE `innodb_test`");
-       return ($innodb_row['Engine'] == 'InnoDB' and $trigger_row['count'] == 1);
+       return $innodb_row['Engine'] == 'InnoDB' && $trigger_row['count'] == 1;
 }
 
 function platform_function_test ($funcname, $extname, $what_if_not = 'not found', $error_class = 'trerror')
@@ -88,7 +88,7 @@ function platform_is_ok ()
        $nerrs += platform_function_test ('bcmul', 'BC Math extension');
        platform_generic_test
        (
-               (!empty($_SERVER['HTTPS']) and $_SERVER['HTTPS'] != 'off'),
+               (! empty ($_SERVER['HTTPS']) && $_SERVER['HTTPS'] != 'off'),
                'accessed over HTTPS',
                'No! (all your passwords will be transmitted in cleartext)',
                'trwarning'
index a3d78ff..381d73b 100644 (file)
@@ -328,7 +328,7 @@ function stringTrace($trace)
                        $ret .= $line['file'].':'.$line['line'].' ';
                $ret .= $line['function'].'(';
                $f = true;
-               if (isset ($line['args']) and is_array ($line['args']))
+               if (isset ($line['args']) && is_array ($line['args']))
                        foreach ($line['args'] as $arg)
                        {
                                if (! $f) $ret .= ', ';
index 25ea757..4f1d90a 100644 (file)
@@ -179,7 +179,7 @@ function assertUIntArg ($argname, $allow_zero = FALSE)
                throw new InvalidRequestArgException($argname, $_REQUEST[$argname], 'parameter is not a number');
        if ($_REQUEST[$argname] < 0)
                throw new InvalidRequestArgException($argname, $_REQUEST[$argname], 'parameter is less than zero');
-       if (!$allow_zero and $_REQUEST[$argname] == 0)
+       if (! $allow_zero && $_REQUEST[$argname] == 0)
                throw new InvalidRequestArgException($argname, $_REQUEST[$argname], 'parameter is zero');
        return $_REQUEST[$argname];
 }
@@ -214,7 +214,7 @@ function assertStringArg ($argname, $ok_if_empty = FALSE)
                throw new InvalidRequestArgException($argname, '', 'parameter is missing');
        if (!is_string ($_REQUEST[$argname]))
                throw new InvalidRequestArgException($argname, $_REQUEST[$argname], 'parameter is not a string');
-       if (!$ok_if_empty and $_REQUEST[$argname] == '')
+       if (! $ok_if_empty && $_REQUEST[$argname] == '')
                throw new InvalidRequestArgException($argname, $_REQUEST[$argname], 'parameter is an empty string');
        return $sic[$argname];
 }
@@ -223,9 +223,9 @@ function assertBoolArg ($argname, $ok_if_empty = FALSE)
 {
        if (!isset ($_REQUEST[$argname]))
                throw new InvalidRequestArgException($argname, '', 'parameter is missing');
-       if (!is_string ($_REQUEST[$argname]) or $_REQUEST[$argname] != 'on')
+       if (! is_string ($_REQUEST[$argname]) || $_REQUEST[$argname] != 'on')
                throw new InvalidRequestArgException($argname, $_REQUEST[$argname], 'parameter is not a string');
-       if (!$ok_if_empty and $_REQUEST[$argname] == '')
+       if (! $ok_if_empty && $_REQUEST[$argname] == '')
                throw new InvalidRequestArgException($argname, $_REQUEST[$argname], 'parameter is an empty string');
        return $_REQUEST[$argname] == TRUE;
 }
@@ -395,18 +395,18 @@ function genericAssertion ($argname, $argtype)
        case 'vlan':
        case 'vlan1':
                assertUIntArg ($argname);
-               if ($argtype == 'vlan' and $sic[$argname] == VLAN_DFL_ID)
+               if ($argtype == 'vlan' && $sic[$argname] == VLAN_DFL_ID)
                        throw new InvalidRequestArgException ($argname, $sic[$argname], 'default VLAN not allowed');
-               if ($sic[$argname] > VLAN_MAX_ID or $sic[$argname] < VLAN_MIN_ID)
+               if ($sic[$argname] > VLAN_MAX_ID || $sic[$argname] < VLAN_MIN_ID)
                        throw new InvalidRequestArgException ($argname, $sic[$argname], 'not a valid VLAN ID');
                return $sic[$argname];
        case 'uint-vlan':
        case 'uint-vlan1':
                if (! preg_match ('/^([1-9][0-9]*)-([1-9][0-9]*)$/', assertStringArg ($argname), $m))
                        throw new InvalidRequestArgException ($argname, $sic[$argname], 'format error');
-               if ($argtype == 'uint-vlan' and $m[2] == VLAN_DFL_ID)
+               if ($argtype == 'uint-vlan' && $m[2] == VLAN_DFL_ID)
                        throw new InvalidRequestArgException ($argname, $sic[$argname], 'default VLAN not allowed');
-               if ($m[2] > VLAN_MAX_ID or $m[2] < VLAN_MIN_ID)
+               if ($m[2] > VLAN_MAX_ID || $m[2] < VLAN_MIN_ID)
                        throw new InvalidRequestArgException ($argname, $sic[$argname], 'not a valid VLAN ID');
                return $sic[$argname];
        case 'rackcode/expr':
@@ -518,7 +518,7 @@ function rectHeight ($rackData, $startRow, $template_idx)
                        }
                }
                // If the first row can't offer anything, bail out.
-               if ($height == 0 and $object_id == 0)
+               if ($height == 0 && $object_id == 0)
                        break;
                $height++;
        }
@@ -661,8 +661,8 @@ function highlightObject (&$rackData, $object_id)
                        $atom = &$rackData[$unit_no][$locidx];
                        if
                        (
-                               $atom['state'] == 'T' and
-                               ($atom['object_id'] == $object_id or isset ($parents[$atom['object_id']]))
+                               $atom['state'] == 'T' &&
+                               ($atom['object_id'] == $object_id || isset ($parents[$atom['object_id']]))
                        )
                                $atom['hl'] = 'h' . $atom['hl'];
                }
@@ -747,7 +747,7 @@ function ip4_mask ($prefix_len)
                "\xFF\xFF\xFF\xFF", // 32
        );
 
-       if ($prefix_len >= 0 and $prefix_len <= 32)
+       if ($prefix_len >= 0 && $prefix_len <= 32)
                return $mask[$prefix_len];
        throw new InvalidArgException ('prefix_len', $prefix_len);
 }
@@ -889,7 +889,7 @@ function ip6_mask ($prefix_len)
                "\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF", // 128
        );
 
-       if ($prefix_len >= 0 and $prefix_len <= 128)
+       if ($prefix_len >= 0 && $prefix_len <= 128)
                return $mask[$prefix_len];
        throw new InvalidArgException ('prefix_len', $prefix_len);
 }
@@ -902,10 +902,10 @@ function l2addressForDatabase ($string)
        $ret = '';
        switch (TRUE)
        {
-               case ($string == '' or preg_match (RE_L2_SOLID, $string) or preg_match (RE_L2_WWN_SOLID, $string)):
+               case ($string == '' || preg_match (RE_L2_SOLID, $string) || preg_match (RE_L2_WWN_SOLID, $string)):
                        $ret = $string;
                        break;
-               case (preg_match (RE_L2_IFCFG, $string) or preg_match (RE_L2_WWN_COLON, $string)):
+               case (preg_match (RE_L2_IFCFG, $string) || preg_match (RE_L2_WWN_COLON, $string)):
                        // reformat output of SunOS ifconfig
                        $ret = '';
                        foreach (explode (':', $string) as $byte)
@@ -918,7 +918,7 @@ function l2addressForDatabase ($string)
                case (preg_match (RE_L2_HUAWEI, $string)):
                        $ret = str_replace ('-', '', $string);
                        break;
-               case (preg_match (RE_L2_IPCFG, $string) or preg_match (RE_L2_WWN_HYPHEN, $string)):
+               case (preg_match (RE_L2_IPCFG, $string) || preg_match (RE_L2_WWN_HYPHEN, $string)):
                        $ret = str_replace ('-', '', $string);
                        break;
                default:
@@ -1014,7 +1014,7 @@ function sortTokenize ($a, $b)
        for ($i=0; $i<count($ar) && $i<count($br); $i++)
        {
                $ret = 0;
-               if (is_numeric($ar[$i]) and is_numeric($br[$i]))
+               if (is_numeric($ar[$i]) && is_numeric($br[$i]))
                        $ret = ($ar[$i]==$br[$i])?0:($ar[$i]<$br[$i]?-1:1);
                else
                        $ret = strcasecmp($ar[$i], $br[$i]);
@@ -1290,7 +1290,7 @@ function pokeNode (&$tree, $trace, $key, $value, $threshold = 0)
                $self ($tree[$myid]['kids'], $trace, $key, $value, $threshold);
        else // just did
        {
-               if (!$threshold or ($threshold and $tree[$myid]['kidc'] + 1 < $threshold))
+               if (! $threshold || ($threshold && $tree[$myid]['kidc'] + 1 < $threshold))
                        $tree[$myid]['kids'][$key] = $value;
                // Reset accumulated records once, when the limit is reached, not each time
                // after that.
@@ -1307,13 +1307,13 @@ function peekNode ($tree, $trace, $target_id)
        if (NULL === ($next = array_shift ($trace))) // warm
        {
                foreach ($tree as $node)
-                       if (array_key_exists ('id', $node) and $node['id'] == $target_id) // hot
+                       if (array_key_exists ('id', $node) && $node['id'] == $target_id) // hot
                                return $node;
        }
        else // cold
        {
                foreach ($tree as $node)
-                       if (array_key_exists ('id', $node) and $node['id'] == $next) // warmer
+                       if (array_key_exists ('id', $node) && $node['id'] == $next) // warmer
                                return $self ($node['kids'], $trace, $target_id);
        }
        throw new RackTablesError ('inconsistent tree data', RackTablesError::INTERNAL);
@@ -1355,7 +1355,7 @@ function addTraceToNodes ($nodelist)
                        }
 
                        // check for cycles every 10 steps
-                       if (0 == (count ($trace) % 10) and in_array ($parentid, $trace))
+                       if (0 == (count ($trace) % 10) && in_array ($parentid, $trace))
                        {
                                // cycle detected
                                $trace = NULL;
@@ -1593,10 +1593,10 @@ function redirectIfNecessary ()
        startSession();
        if
        (
-               ! isset ($_REQUEST['tab']) and
-               isset ($_SESSION['RTLT'][$pageno]) and
-               getConfigVar ('SHOW_LAST_TAB') == 'yes' and
-               permitted ($pageno, $_SESSION['RTLT'][$pageno]['tabname']) and
+               ! isset ($_REQUEST['tab']) &&
+               isset ($_SESSION['RTLT'][$pageno]) &&
+               getConfigVar ('SHOW_LAST_TAB') == 'yes' &&
+               permitted ($pageno, $_SESSION['RTLT'][$pageno]['tabname']) &&
                time() - $_SESSION['RTLT'][$pageno]['time'] <= TAB_REMEMBER_TIMEOUT
        )
                redirectUser (buildRedirectURL ($pageno, $_SESSION['RTLT'][$pageno]['tabname']));
@@ -1605,7 +1605,7 @@ function redirectIfNecessary ()
        // but isn't OK anymore when generating the current page and the tab is the requested tab.
        if
        (
-               isset ($trigger[$pageno][$tabno]) and
+               isset ($trigger[$pageno][$tabno]) &&
                '' == call_user_func ($trigger[$pageno][$tabno])
        )
        {
@@ -1748,8 +1748,8 @@ function getObjectiveTagTree ($tree, $realm, $preselect)
                // regardless of how many sub-nodes it features.
                if
                (
-                       isset ($taginfo['refcnt'][$realm]) or
-                       count ($subsearch) > 1 or
+                       isset ($taginfo['refcnt'][$realm]) ||
+                       count ($subsearch) > 1 ||
                        in_array ($taginfo['id'], $preselect)
                )
                        $ret[] = array
@@ -1882,11 +1882,11 @@ function getCellFilter ()
                unset($_SESSION[$pageno]); // delete saved filter
 
        // otherwise inject saved filter to the $_REQUEST and $sic vars
-       elseif (isset ($_SESSION[$pageno]['filter']) and is_array ($_SESSION[$pageno]['filter']) and getConfigVar ('STATIC_FILTER') == 'yes')
+       elseif (isset ($_SESSION[$pageno]['filter']) && is_array ($_SESSION[$pageno]['filter']) && getConfigVar ('STATIC_FILTER') == 'yes')
                foreach (array('andor', 'cfe', 'cft[]', 'cfp[]', 'nft[]', 'nfp[]') as $param)
                {
                        $param = str_replace ('[]', '', $param, $is_array);
-                       if (! isset ($_REQUEST[$param]) and isset ($_SESSION[$pageno]['filter'][$param]) and (!$is_array or is_array ($_SESSION[$pageno]['filter'][$param])))
+                       if (! isset ($_REQUEST[$param]) && isset ($_SESSION[$pageno]['filter'][$param]) && (! $is_array || is_array ($_SESSION[$pageno]['filter'][$param])))
                        {
                                $_REQUEST[$param] = $_SESSION[$pageno]['filter'][$param];
                                if (! $is_array)
@@ -1926,7 +1926,7 @@ function getCellFilter ()
        // handled somehow. Discard them silently for now.
        global $taglist, $pTable;
        foreach (array ('cft', 'cfp', 'nft', 'nfp') as $param)
-               if (isset ($_REQUEST[$param]) and is_array ($_REQUEST[$param]))
+               if (isset ($_REQUEST[$param]) && is_array ($_REQUEST[$param]))
                {
                        $_SESSION[$pageno]['filter'][$param] = $_REQUEST[$param];
                        foreach ($_REQUEST[$param] as $req_key)
@@ -2166,9 +2166,9 @@ function IPNetworkCmp ($netA, $netB)
        $ret = IPCmp ($netA['ip_bin'], $netB['ip_bin']);
        if ($ret == 0)
                $ret = $netA['mask'] < $netB['mask'] ? -1 : ($netA['mask'] > $netB['mask'] ? 1 : 0);
-       if ($ret == -1 and $netA['ip_bin'] === ($netB['ip_bin'] & $netA['mask_bin']))
+       if ($ret == -1 && $netA['ip_bin'] === ($netB['ip_bin'] & $netA['mask_bin']))
                $ret = -2;
-       if ($ret == 1 and $netB['ip_bin'] === ($netA['ip_bin'] & $netB['mask_bin']))
+       if ($ret == 1 && $netB['ip_bin'] === ($netA['ip_bin'] & $netB['mask_bin']))
                $ret = 2;
        return $ret;
 }
@@ -2208,7 +2208,7 @@ function sortTree (&$tree, $cmpfunc = '')
 
 function iptree_fill (&$netdata)
 {
-       if (!isset ($netdata['kids']) or !count ($netdata['kids']))
+       if (! isset ($netdata['kids']) || ! count ($netdata['kids']))
                return;
 
        foreach ($netdata['spare_ranges'] as $mask => $list)
@@ -2540,7 +2540,7 @@ function ip_next ($ip_bin)
        {
                $oct = $p + ord ($ret[$i]);
                $ret[$i] = chr ($oct & 0xff);
-               if ($oct <= 255 and $oct >= 0)
+               if ($oct <= 255 && $oct >= 0)
                        break;
        }
        return $ret;
@@ -2554,7 +2554,7 @@ function ip_prev ($ip_bin)
        {
                $oct = $p + ord ($ret[$i]);
                $ret[$i] = chr ($oct & 0xff);
-               if ($oct <= 255 and $oct >= 0)
+               if ($oct <= 255 && $oct >= 0)
                        break;
        }
        return $ret;
@@ -2653,7 +2653,7 @@ function treeApplyFunc (&$tree, $func = '', $stopfunc = '')
        foreach (array_keys ($tree) as $key)
        {
                $func ($tree[$key]);
-               if ($stopfunc != '' and $stopfunc ($tree[$key]))
+               if ($stopfunc != '' && $stopfunc ($tree[$key]))
                        continue;
                $self ($tree[$key]['kids'], $func);
        }
@@ -2794,7 +2794,7 @@ function getTerminalNetworks ($tree)
        $self = __FUNCTION__;
        $ret = array();
        foreach ($tree as $node)
-               if ($node['kidc'] == 0 and isset ($node['realm']))
+               if ($node['kidc'] == 0 && isset ($node['realm']))
                        $ret[] = $node;
                else
                        $ret = array_merge ($ret, $self ($node['kids']));
@@ -2810,18 +2810,18 @@ function iptree_markup_collapsion (&$tree, $threshold = 1024, $target = 0)
        $ret = FALSE;
        foreach (array_keys ($tree) as $key)
        {
-               $here = ($target === 'ALL' or ($target > 0 and isset ($tree[$key]['id']) and $tree[$key]['id'] == $target));
+               $here = $target === 'ALL' || ($target > 0 && isset ($tree[$key]['id']) && $tree[$key]['id'] == $target);
                $below = $self ($tree[$key]['kids'], $threshold, $target);
                $expand_enabled = ($target !== 'NONE');
                if (!$tree[$key]['kidc']) // terminal node
                        $tree[$key]['symbol'] = 'spacer';
-               elseif ($expand_enabled and $tree[$key]['kidc'] < $threshold)
+               elseif ($expand_enabled && $tree[$key]['kidc'] < $threshold)
                        $tree[$key]['symbol'] = 'node-expanded-static';
-               elseif ($expand_enabled and ($here or $below))
+               elseif ($expand_enabled && ($here || $below))
                        $tree[$key]['symbol'] = 'node-expanded';
                else
                        $tree[$key]['symbol'] = 'node-collapsed';
-               $ret = ($ret or $here or $below); // parentheses are necessary for this to be computed correctly
+               $ret = $ret || $here || $below;
        }
        return $ret;
 }
@@ -3249,7 +3249,7 @@ function getPortListPrefs()
        foreach (explode (';', getConfigVar ('DEFAULT_PORT_OIF_IDS')) as $tmp)
        {
                $tmp = explode ('=', trim ($tmp));
-               if (count ($tmp) == 2 and $tmp[0] > 0 and $tmp[1] > 0)
+               if (count ($tmp) == 2 && $tmp[0] > 0 && $tmp[1] > 0)
                        $ret['oif_picks'][$tmp[0]] = $tmp[1];
        }
        // enforce default value
@@ -3289,7 +3289,7 @@ function getUnlinkedPortTypeOptions ($port_iif_id)
        {
                if ($row['iif_id'] == $prefs['iif_pick'] || $row['iif_id'] == $port_iif_id)
                        $optgroup = $row['iif_name'];
-               elseif (array_key_exists ($row['iif_id'], $prefs['oif_picks']) and $prefs['oif_picks'][$row['iif_id']] == $row['oif_id'])
+               elseif (array_key_exists ($row['iif_id'], $prefs['oif_picks']) && $prefs['oif_picks'][$row['iif_id']] == $row['oif_id'])
                        $optgroup = 'other';
                else
                        continue;
@@ -3344,7 +3344,7 @@ function groupIntsToRanges ($list, $exclude_value = NULL)
        $id_from = $id_to = 0;
        $list[] = -1;
        foreach ($list as $next_id)
-               if (!isset ($exclude_value) or $next_id != $exclude_value)
+               if (! isset ($exclude_value) || $next_id != $exclude_value)
                        if ($id_to && $next_id == $id_to + 1)
                                $id_to = $next_id; // merge
                        else
@@ -3456,7 +3456,7 @@ function array_values_same ($a1, $a2)
                throw new InvalidArgException ('a1', $a1, 'is not an array');
        if (! is_array ($a2))
                throw new InvalidArgException ('a2', $a2, 'is not an array');
-       return !count (array_diff ($a1, $a2)) and !count (array_diff ($a2, $a1));
+       return ! count (array_diff ($a1, $a2)) && ! count (array_diff ($a2, $a1));
 }
 
 # Reindex provided array of arrays by a column value that is present in
@@ -3558,7 +3558,7 @@ function buildVLANFilter ($role, $string)
        // transform
        $vlanidlist = array();
        foreach (iosParseVLANString ($string) as $vlan_id)
-               if ($min <= $vlan_id and $vlan_id <= $max)
+               if ($min <= $vlan_id && $vlan_id <= $max)
                        $vlanidlist[] = $vlan_id;
        return listToRanges ($vlanidlist);
 }
@@ -3604,7 +3604,7 @@ function listToRanges ($vlanidlist, $limit = 0)
 function matchVLANFilter ($vlan_id, $vfilter)
 {
        foreach ($vfilter as $range)
-               if ($range['from'] <= $vlan_id and $vlan_id <= $range['to'])
+               if ($range['from'] <= $vlan_id && $vlan_id <= $range['to'])
                        return TRUE;
        return FALSE;
 }
@@ -3628,7 +3628,7 @@ function generate8021QDeployOps ($vswitch, $device_vlanlist, $before, $changes)
        foreach ($device_vlanlist as $vlan_id)
                if
                (
-                       !array_key_exists ($vlan_id, $domain_vlanlist) or
+                       ! array_key_exists ($vlan_id, $domain_vlanlist) ||
                        $domain_vlanlist[$vlan_id]['vlan_type'] != 'alien'
                )
                        $old_managed_vlans[] = $vlan_id;
@@ -3692,7 +3692,7 @@ function generate8021QDeployOps ($vswitch, $device_vlanlist, $before, $changes)
                                        continue;
                                if
                                (
-                                       array_key_exists ($vlan_id, $domain_vlanlist) and
+                                       array_key_exists ($vlan_id, $domain_vlanlist) &&
                                        $domain_vlanlist[$vlan_id]['vlan_type'] == 'alien'
                                )
                                        continue;
@@ -3711,8 +3711,8 @@ function generate8021QDeployOps ($vswitch, $device_vlanlist, $before, $changes)
                foreach ($port['allowed'] as $vlan_id)
                        if
                        (
-                               isset ($domain_vlanlist[$vlan_id]) and
-                               $domain_vlanlist[$vlan_id]['vlan_type'] == 'ondemand' and
+                               isset ($domain_vlanlist[$vlan_id]) &&
+                               $domain_vlanlist[$vlan_id]['vlan_type'] == 'ondemand' &&
                                !in_array ($vlan_id, $new_managed_vlans)
                        )
                                $new_managed_vlans[] = $vlan_id;
@@ -3742,7 +3742,7 @@ function generate8021QDeployOps ($vswitch, $device_vlanlist, $before, $changes)
                {
                case 'trunk->trunk':
                        // "old" native is set and differs from the "new" native
-                       if ($port['old_native'] and $port['old_native'] != $port['new_native'])
+                       if ($port['old_native'] && $port['old_native'] != $port['new_native'])
                                $crq[] = array
                                (
                                        'opcode' => 'unset native',
@@ -3767,7 +3767,7 @@ function generate8021QDeployOps ($vswitch, $device_vlanlist, $before, $changes)
                                }
                        break;
                case 'access->access':
-                       if ($port['old_native'] and $port['old_native'] != $port['new_native'])
+                       if ($port['old_native'] && $port['old_native'] != $port['new_native'])
                        {
                                $crq[] = array
                                (
@@ -3865,7 +3865,7 @@ function generate8021QDeployOps ($vswitch, $device_vlanlist, $before, $changes)
                                );
                        // One of the "allowed" VLANs for this port may probably be "native".
                        // "new native" is set and differs from "old native"
-                       if ($port['new_native'] and $port['new_native'] != $port['old_native'])
+                       if ($port['new_native'] && $port['new_native'] != $port['old_native'])
                                $crq[] = array
                                (
                                        'opcode' => 'set native',
@@ -3874,7 +3874,7 @@ function generate8021QDeployOps ($vswitch, $device_vlanlist, $before, $changes)
                                );
                        break;
                case 'access->access':
-                       if ($port['new_native'] and $port['new_native'] != $port['old_native'])
+                       if ($port['new_native'] && $port['new_native'] != $port['old_native'])
                                $crq[] = array
                                (
                                        'opcode' => 'set access',
@@ -3972,7 +3972,7 @@ function filter8021QChangeRequests
                                // discarded the old contents of 'allowed' for current port.
                                if
                                (
-                                       $before[$port_name]['native'] == $immune or
+                                       $before[$port_name]['native'] == $immune ||
                                        $port['native'] == $immune
                                )
                                {
@@ -4021,7 +4021,7 @@ function getEmployedVlans ($object_id, $domain_vlanlist)
        foreach (getObjectIPAllocationList ($object_id) as $ip_bin => $alloc)
                if ($net = spotNetworkByIP ($ip_bin))
                        foreach ($net['8021q'] as $vlan)
-                               if (isset ($domain_vlanlist[$vlan['vlan_id']]) and ! isset ($employed[$vlan['vlan_id']]))
+                               if (isset ($domain_vlanlist[$vlan['vlan_id']]) && ! isset ($employed[$vlan['vlan_id']]))
                                        $employed[$vlan['vlan_id']] = 1;
        $ret = array_keys ($employed);
        $override = callHook ('getEmployedVlans_hook', $ret, $object_id, $domain_vlanlist);
@@ -4153,7 +4153,7 @@ function get8021QSyncOptions
                // catch anomalies early
                if ($port['vst_role'] == 'none')
                {
-                       if ((!array_key_exists ($pn, $R) or $R[$pn]['mode'] == 'none') and !array_key_exists ($pn, $C))
+                       if ((! array_key_exists ($pn, $R) || $R[$pn]['mode'] == 'none') && ! array_key_exists ($pn, $C))
                                $ret[$pn] = array ('status' => 'none');
                        else
                                $ret[$pn] = array
@@ -4164,7 +4164,7 @@ function get8021QSyncOptions
                                );
                        continue;
                }
-               elseif ((!array_key_exists ($pn, $R) or $R[$pn]['mode'] == 'none') and array_key_exists ($pn, $C))
+               elseif ((! array_key_exists ($pn, $R) || $R[$pn]['mode'] == 'none') && array_key_exists ($pn, $C))
                {
                        $ret[$pn] = array
                        (
@@ -4203,7 +4203,7 @@ function get8021QSyncOptions
                $D_eq_C = same8021QConfigs ($D[$pn], $C[$pn]);
                $C_eq_R = same8021QConfigs ($C[$pn], $R[$pn]);
                // (DCR), D = C = R: data in sync
-               if ($D_eq_C and $C_eq_R) // implies D == R
+               if ($D_eq_C && $C_eq_R) // implies D == R
                {
                        $ret[$pn] = array
                        (
@@ -4328,7 +4328,7 @@ function exec8021QDeploy ($object_id, $do_push)
 
        $out_of_sync = FALSE;
        $errno = E_8021Q_NOERROR;
-       if ($nsaved + $nsaved_uplinks or count ($ok_to_push))
+       if ($nsaved + $nsaved_uplinks || count ($ok_to_push))
                $out_of_sync = TRUE;
        if ($conflict)
        {
@@ -4403,7 +4403,7 @@ function saveDownlinksReverb ($object_id, $requested_changes)
        $changes_to_save = array();
        // first filter by wrt_vlans constraint
        foreach ($requested_changes as $pn => $requested)
-               if (array_key_exists ($pn, $before) and $requested['vst_role'] == 'downlink')
+               if (array_key_exists ($pn, $before) && $requested['vst_role'] == 'downlink')
                        $changes_to_save[$pn] = array
                        (
                                'vst_role' => 'downlink',
@@ -4431,7 +4431,7 @@ function initiateUplinksReverb ($object_id, $uplink_ports)
        foreach (getObjectPortsAndLinks ($object_id, FALSE) as $portinfo)
                if
                (
-                       $portinfo['linked'] and
+                       $portinfo['linked'] &&
                        array_key_exists ($portinfo['name'], $uplink_ports)
                )
                        $upstream_config[$portinfo['remote_object_id']][$portinfo['remote_name']] = $uplink_ports[$portinfo['name']];
@@ -4486,7 +4486,7 @@ function recalc8021QPorts ($switch_id)
        foreach ($ports as $portinfo)
                if
                (
-                       isset ($order[$portinfo['name']]) and
+                       isset ($order[$portinfo['name']]) &&
                        $order[$portinfo['name']]['vst_role'] == 'downlink'
                )
                {
@@ -4605,7 +4605,7 @@ function authorize8021QChangeRequests ($before, $changes, $op = NULL)
        if (NULL !== $ret = callHook ('authorize8021QChangeRequests_hook', $before, $changes, $op))
                return $ret;
        global $script_mode;
-       if (isset ($script_mode) and $script_mode)
+       if (isset ($script_mode) && $script_mode)
                return $changes;
        $ret = array();
        foreach ($changes as $pn => $change)
@@ -4707,14 +4707,14 @@ function compareDecomposedPortNames ($porta, $portb)
                $index_diff = -1; // a < b
 
        // compare by portname fields
-       if ($prefix_diff != 0 and ($porta['numidx'] <= 1 or $portb['numidx'] <= 1)) // if index count is lte 1, sort by prefix
+       if ($prefix_diff != 0 && ($porta['numidx'] <= 1 || $portb['numidx'] <= 1)) // if index count is lte 1, sort by prefix
        {
                $ret = $porta['numidx'] - $portb['numidx'];
                if ($ret == 0)
                        $ret = $prefix_diff;
        }
        // if index count > 1 and ports have different prefixes in intersecting index sections, sort by prefix
-       elseif ($prefix_diff != 0 and $a_parent != '' and $a_parent == $b_parent)
+       elseif ($prefix_diff != 0 && $a_parent != '' && $a_parent == $b_parent)
                $ret = $prefix_diff;
        // if indices are not equal, sort by index
        elseif ($index_diff != 0)
@@ -4852,7 +4852,7 @@ function searchEntitiesByText ($terms)
                // search by FQDN has special treatment - if single object found, do not search by other fields
                $object_id_by_fqdn = NULL;
                $domains = preg_split ('/\s*,\s*/', strtolower (getConfigVar ('SEARCH_DOMAINS')));
-               if (! empty ($domains) and $object_id = searchByMgmtHostname ($terms))
+               if (! empty ($domains) && $object_id = searchByMgmtHostname ($terms))
                {
                        // get FQDN
                        $attrs = getAttrValues ($object_id);
@@ -4943,7 +4943,7 @@ function buildSearchRedirectURL ($result_type, $record)
                                return NULL;
                        $next_page = $pageno_by_etype[$result_type];
                        if ($result_type == 'object')
-                               if (isset ($record['by_port']) and 1 == count ($record['by_port']))
+                               if (isset ($record['by_port']) && 1 == count ($record['by_port']))
                                {
                                        $found_ports_ids = array_keys ($record['by_port']);
                                        $params['hl_port_id'] = $found_ports_ids[0];
@@ -4955,7 +4955,7 @@ function buildSearchRedirectURL ($result_type, $record)
        if (! isset ($key) || ! isset ($id))
                return NULL;
        $params[$key] = $id;
-       if (isset ($_REQUEST['last_tab']) and isset ($_REQUEST['last_page']) and $next_page == $_REQUEST['last_page'])
+       if (isset ($_REQUEST['last_tab']) && isset ($_REQUEST['last_page']) && $next_page == $_REQUEST['last_page'])
                $next_tab = assertStringArg('last_tab');
        return buildRedirectURL ($next_page, isset ($next_tab) ? $next_tab : 'default', $params);
 }
@@ -5069,9 +5069,9 @@ function setMessage ($type, $message, $direct_rendering)
        global $script_mode;
        if ($direct_rendering)
                echo '<div class="msg_' . $type . '">' . $message . '</div>';
-       elseif (isset ($script_mode) and $script_mode)
+       elseif (isset ($script_mode) && $script_mode)
        {
-               if ($type == 'warning' or $type == 'error')
+               if ($type == 'warning' || $type == 'error')
                        file_put_contents ('php://stderr', strtoupper ($type) . ': ' . strip_tags ($message) . "\n");
        }
        else
@@ -5146,7 +5146,7 @@ function getMessagesCount ($message_type = 'all')
 
 function isEthernetPort($port)
 {
-       return ($port['iif_id'] != 1 or preg_match('/Base|LACP/i', $port['oif_name']));
+       return ($port['iif_id'] != 1 || preg_match('/Base|LACP/i', $port['oif_name']));
 }
 
 function loadConfigDefaults()
@@ -5200,8 +5200,8 @@ function getConfigVar ($varname = '')
                throw new RackTablesError ('configuration cache is unavailable', RackTablesError::INTERNAL);
        if
        (
-               $varname == ''
-               or ! array_key_exists ($varname, $configCache)
+               $varname == '' ||
+               ! array_key_exists ($varname, $configCache)
        )
                throw new InvalidArgException ('varname', $varname);
        return $configCache[$varname]['varvalue'];
@@ -5269,8 +5269,8 @@ function getObjectTypeChangeOptions ($object_id)
                        $app = $map[$attr['id']]['application'];
                        if
                        (
-                               (NULL === $appidx = scanArrayForItem ($app, 'objtype_id', $test_id)) or
-                               ($attr['type'] == 'dict' and $attr['chapter_id'] != $app[$appidx]['chapter_no'])
+                               (NULL === $appidx = scanArrayForItem ($app, 'objtype_id', $test_id)) ||
+                               ($attr['type'] == 'dict' && $attr['chapter_id'] != $app[$appidx]['chapter_no'])
                        )
                                continue 2; // next type ID
                }
@@ -5400,7 +5400,7 @@ function apply8021qChangeRequest ($switch_id, $changes, $verbose = TRUE, $mutex_
        {
                if (NULL === $vswitch = getVLANSwitchInfo ($switch_id, 'FOR UPDATE'))
                        throw new InvalidArgException ('object_id', $switch_id, 'VLAN domain is not set for this object');
-               if (isset ($mutex_rev) and $vswitch['mutex_rev'] != $mutex_rev)
+               if (isset ($mutex_rev) && $vswitch['mutex_rev'] != $mutex_rev)
                        throw new InvalidRequestArgException ('mutex_rev', $mutex_rev, 'expired form data');
                $after = $before = apply8021QOrder ($vswitch, getStored8021QConfig ($vswitch['object_id'], 'desired'));
                $domain_vlanlist = getDomainVLANList ($vswitch['domain_id']);
@@ -5434,7 +5434,7 @@ function apply8021qChangeRequest ($switch_id, $changes, $verbose = TRUE, $mutex_
        $nsaved_downlinks = initiateUplinksReverb ($vswitch['object_id'], $new_uplinks);
        // instant deploy to that switch if configured
        $done = 0;
-       if ($npulled + $nsaved_uplinks > 0 and getConfigVar ('8021Q_INSTANT_DEPLOY') == 'yes')
+       if ($npulled + $nsaved_uplinks > 0 && getConfigVar ('8021Q_INSTANT_DEPLOY') == 'yes')
        {
                try
                {
@@ -5475,7 +5475,7 @@ function fillIPNetsCorrelation (&$nets, $max_depth = 0)
                        if (IPNetContains ($top, $net))
                        {
                                // skip the network if max_depth exceeded
-                               if ($max_depth and count ($stack) > $max_depth)
+                               if ($max_depth && count ($stack) > $max_depth)
                                        continue 2;
                                $top['kidc']++;
                                break;
@@ -5544,7 +5544,7 @@ function fillIPSpareListBstr (&$net, $a, $b)
                {
                        $bmask = ip_mask ($mask, $len == 128);
                        $last_a = $a | ~ $bmask;
-                       if ($a == ($a & $bmask) and 0 >= strcmp ($last_a, $b))
+                       if ($a == ($a & $bmask) && 0 >= strcmp ($last_a, $b))
                        {
                                $net['spare_ranges'][$mask][] = $a;
                                $a = ip_next ($last_a);
@@ -5596,9 +5596,9 @@ function isIPNetworkEmpty (&$netinfo)
        foreach ($pure_array as $ip => $comment)
                if
                (
-                       array_key_exists ($ip, $netinfo['addrlist']) and
-                       $netinfo['addrlist'][$ip]['name'] == $comment and
-                       $netinfo['addrlist'][$ip]['reserved'] == 'yes' and
+                       array_key_exists ($ip, $netinfo['addrlist']) &&
+                       $netinfo['addrlist'][$ip]['name'] == $comment &&
+                       $netinfo['addrlist'][$ip]['reserved'] == 'yes' &&
                        isIPAddressEmpty ($netinfo['addrlist'][$ip], array ('name', 'reserved'))
                )
                        $pure_auto++;
@@ -5877,7 +5877,7 @@ function mkCellA ($cell, $title = NULL)
 function listConstraint ($realm, $varname = '')
 {
        $wideList = listCells ($realm);
-       if (($varname != '') and ('' != $filter = getConfigVar ($varname)))
+       if ($varname != '' && ('' != $filter = getConfigVar ($varname)))
        {
                $expr = compileExpression ($filter);
                if (! $expr)
@@ -6065,7 +6065,7 @@ function checkTypeAndAttribute ($object_id, $type_id, $attr_id, $values)
        $object = spotEntity ('object', $object_id);
        if ($object['objtype_id'] == $type_id)
                foreach (getAttrValues ($object_id) as $record)
-                       if ($record['id'] == $attr_id and in_array ($record['key'], $values))
+                       if ($record['id'] == $attr_id && in_array ($record['key'], $values))
                                return TRUE;
        return FALSE;
 }
index ce96701..8be1b15 100644 (file)
@@ -57,7 +57,7 @@ if (!mb_internal_encoding ('UTF-8'))
        throw new RackTablesError ('Failed setting multibyte string encoding to UTF-8', RackTablesError::INTERNAL);
 
 $rackCodeCache = loadScript ('RackCodeCache');
-if ($rackCodeCache == NULL or $rackCodeCache == '')
+if ($rackCodeCache == NULL || $rackCodeCache == '')
 {
        $rackCode = getRackCode (loadScript ('RackCode'));
        saveScript ('RackCodeCache', base64_encode (serialize ($rackCode)));
@@ -116,7 +116,7 @@ $pageheaders = array
 );
 addCSS ('css/pi.css');
 
-if (!isset ($script_mode) or $script_mode !== TRUE)
+if (! isset ($script_mode) || $script_mode !== TRUE)
 {
        // A successful call to authenticate() always generates autotags and somethimes
        // even given/implicit tags. It also sets remote_username and remote_displayname.
@@ -151,7 +151,7 @@ if (FALSE !== $plugin_files = glob ("${racktables_plugins_dir}/*.php"))
                 require_once $plugin_file;
 // display plugins output if it contains something but newlines
 $tmp = ob_get_clean();
-if ($tmp != '' and ! preg_match ("/^\n+$/D", $tmp))
+if ($tmp != '' && ! preg_match ("/^\n+$/D", $tmp))
        echo $tmp;
 unset ($tmp);
 
index 0fe79bf..fa098fe 100644 (file)
@@ -21,7 +21,7 @@ else
 
 if ($step > count ($stepfunc))
 {
-       $root = (empty($_SERVER['HTTPS']) or $_SERVER['HTTPS'] == 'off') ? 'http://' : 'https://';
+       $root = (empty ($_SERVER['HTTPS']) || $_SERVER['HTTPS'] == 'off') ? 'http://' : 'https://';
        $root .= isset ($_SERVER['HTTP_HOST']) ? $_SERVER['HTTP_HOST'] : ($_SERVER['SERVER_NAME'].($_SERVER['SERVER_PORT']=='80'?'':$_SERVER['SERVER_PORT']));
        // "Since PHP 4.3.0, you will often get a slash or a dot back from
        // dirname() in situations where the older functionality would have given
@@ -89,7 +89,7 @@ echo "<input type=hidden name=step value='${next_step}'>\n";
 function not_already_installed()
 {
        global $found_secret_file, $pdo_dsn;
-       if ($found_secret_file and isset ($pdo_dsn))
+       if ($found_secret_file && isset ($pdo_dsn))
        {
                echo 'Your configuration file exists and seems to hold necessary data already.<br>';
                return FALSE;
@@ -156,7 +156,7 @@ function init_config ()
                print_form();
                return FALSE;
        }
-       if (empty ($_REQUEST['mysql_db']) or empty ($_REQUEST['mysql_username']))
+       if (empty ($_REQUEST['mysql_db']) || empty ($_REQUEST['mysql_username']))
        {
                print_form
                (
@@ -171,7 +171,7 @@ function init_config ()
                echo '<h2 class=trerror>Missing database/username parameter!</h2>';
                return FALSE;
        }
-       if ($_REQUEST['conn'] == 'conn_tcp' and empty ($_REQUEST['mysql_host']))
+       if ($_REQUEST['conn'] == 'conn_tcp' && empty ($_REQUEST['mysql_host']))
        {
                print_form
                (
@@ -186,7 +186,7 @@ function init_config ()
                echo '<h2 class=trerror>Missing TCP hostname parameter!</h2>';
                return FALSE;
        }
-       if ($_REQUEST['conn'] == 'conn_unix' and empty ($_REQUEST['mysql_socket']))
+       if ($_REQUEST['conn'] == 'conn_unix' && empty ($_REQUEST['mysql_socket']))
        {
                print_form
                (
@@ -207,7 +207,7 @@ function init_config ()
        {
        case 'conn_tcp':
                $pdo_dsn .= 'host=' . $_REQUEST['mysql_host'];
-               if (!empty ($_REQUEST['mysql_port']) and $_REQUEST['mysql_port'] != '3306')
+               if (! empty ($_REQUEST['mysql_port']) && $_REQUEST['mysql_port'] != '3306')
                        $pdo_dsn .= ';port=' . $_REQUEST['mysql_port'];
                break;
        case 'conn_unix':
@@ -312,7 +312,7 @@ ENDOFTEXT
 function get_process_owner()
 {
        // this function requires the posix extention and returns the fallback value otherwise
-       if (is_callable('posix_getpwuid') and is_callable('posix_geteuid'))
+       if (is_callable ('posix_getpwuid') && is_callable ('posix_geteuid'))
        {
                $user = posix_getpwuid(posix_geteuid());
                if (isset ($user['name']))
@@ -324,7 +324,7 @@ function get_process_owner()
 function check_config_access()
 {
        global $path_to_secret_php;
-       if (! is_writable ($path_to_secret_php) and is_readable ($path_to_secret_php))
+       if (! is_writable ($path_to_secret_php) && is_readable ($path_to_secret_php))
        {
                echo 'The configuration file ownership/permissions seem to be OK.<br>';
                return TRUE;
@@ -445,7 +445,7 @@ function init_database_dynamic ()
 {
        connect_to_db_or_die();
        global $dbxlink;
-       if (!isset ($_REQUEST['password']) or empty ($_REQUEST['password']))
+       if (! isset ($_REQUEST['password']) || empty ($_REQUEST['password']))
        {
                $result = $dbxlink->query ('select count(user_id) from UserAccount where user_id = 1');
                $row = $result->fetch (PDO::FETCH_NUM);
index 1a634c5..141af16 100644 (file)
@@ -107,8 +107,8 @@ function render8021QOrderForm ($some_id)
        // in a "plus" row on the form, when there is already a "minus" one
        if
        (
-               getConfigVar ('ADDNEW_AT_TOP') == 'yes' and
-               ($pageno != 'object' or !count ($minuslines))
+               getConfigVar ('ADDNEW_AT_TOP') == 'yes' &&
+               ($pageno != 'object' || ! count ($minuslines))
        )
                printNewItemTR();
        $vdomlist = getVLANDomainOptions();
@@ -150,8 +150,8 @@ function render8021QOrderForm ($some_id)
        }
        if
        (
-               getConfigVar ('ADDNEW_AT_TOP') != 'yes' and
-               ($pageno != 'object' or !count ($minuslines))
+               getConfigVar ('ADDNEW_AT_TOP') != 'yes' &&
+               ($pageno != 'object' || ! count ($minuslines))
        )
                printNewItemTR();
        echo '</table>';
@@ -267,7 +267,7 @@ function renderVLANDomainListEditor ()
        {
                printOpFormIntro ('upd', array ('vdom_id' => $vdom_id));
                echo '<tr><td>';
-               if ($dominfo['switchc'] or $dominfo['vlanc'] > 1)
+               if ($dominfo['switchc'] || $dominfo['vlanc'] > 1)
                        printImageHREF ('nodestroy', 'domain used elsewhere');
                else
                        echo getOpLink (array ('op' => 'del', 'vdom_id' => $vdom_id), '', 'destroy', 'delete domain');
@@ -392,7 +392,7 @@ function renderVLANDomainVLANList ($vdom_id)
        {
                printOpFormIntro ('upd', array ('vlan_id' => $vlan_id));
                echo '<tr><td>';
-               if ($vlan_info['portc'] or $vlan_id == VLAN_DFL_ID)
+               if ($vlan_info['portc'] || $vlan_id == VLAN_DFL_ID)
                        printImageHREF ('nodestroy', $vlan_info['portc'] . ' port(s) configured');
                else
                        echo getOpLink (array ('op' => 'del', 'vlan_id' => $vlan_id), '', 'destroy', 'delete VLAN');
@@ -452,9 +452,9 @@ function renderObject8021QPorts ($object_id)
                $pn = shortenIfName ($port['name'], $breed);
                if (! isset ($indexed_ports[$pn]) || ! $indexed_ports[$pn]['linked'])
                        $indexed_ports[$pn] = $port;
-               if ($port['name'] != '' and array_key_exists ($port['name'], $desired_config))
+               if ($port['name'] != '' && array_key_exists ($port['name'], $desired_config))
                {
-                       if (isset ($hl_port_id) and $hl_port_id == $port['id'])
+                       if (isset ($hl_port_id) && $hl_port_id == $port['id'])
                                $hl_port_name = $port['name'];
                        $socket = array ('interface' => formatPortIIFOIF ($port), 'link' => '&nbsp;');
                        if ($port['remote_object_id'])
@@ -523,7 +523,7 @@ function renderObject8021QPorts ($object_id)
                }
                $anchor = '';
                $tdclass = '';
-               if (isset ($hl_port_name) and $hl_port_name == $port_name)
+               if (isset ($hl_port_name) && $hl_port_name == $port_name)
                {
                        $tdclass .= 'class="border_highlight"';
                        $anchor = "name='port-$hl_port_id'";
@@ -545,7 +545,7 @@ function renderObject8021QPorts ($object_id)
                        }
        }
        echo '<tr><td colspan=5 class=tdcenter><ul class="btns-8021q-sync">';
-       if ($req_port_name == '' and $nports)
+       if ($req_port_name == '' && $nports)
        {
                echo "<input type=hidden name=nports value=${nports}>";
                echo '<li>' . getImageHREF ('SAVE', 'save configuration', TRUE) . '</li>';
@@ -602,7 +602,7 @@ function getAccessPortControlCode ($req_port_name, $vdom, $port_name, $port, &$n
                return '&nbsp;';
        if
        (
-               array_key_exists ($port['native'], $vdom['vlanlist']) and
+               array_key_exists ($port['native'], $vdom['vlanlist']) &&
                $vdom['vlanlist'][$port['native']]['vlan_type'] == 'alien'
        )
                return formatVLANAsLabel ($vdom['vlanlist'][$port['native']]);
@@ -627,9 +627,9 @@ function getAccessPortControlCode ($req_port_name, $vdom, $port_name, $port, &$n
        foreach ($vdom['vlanlist'] as $vlan_id => $vlan_info)
                if
                (
-                       ($vlan_id != $from or $port['mode'] == 'trunk') and
-                       $vlan_info['vlan_type'] != 'alien' and
-                       in_array ($vlan_id, $vlanpermissions[$from]) and
+                       ($vlan_id != $from || $port['mode'] == 'trunk') &&
+                       $vlan_info['vlan_type'] != 'alien' &&
+                       in_array ($vlan_id, $vlanpermissions[$from]) &&
                        matchVLANFilter ($vlan_id, $port['wrt_vlans'])
                )
                        $options[$vlan_id] = formatVLANAsOption ($vlan_info);
@@ -754,7 +754,7 @@ function renderTrunkPortControls ($vswitch, $vdom, $port_name, $vlanport)
                        // that is explicitly protected from it.
                        if
                        (
-                               $native_options[$vlanport['native']]['vlan_type'] == 'alien' or
+                               $native_options[$vlanport['native']]['vlan_type'] == 'alien' ||
                                $option['vlan_type'] == 'alien'
                        )
                                $selected .= ' disabled';
@@ -826,14 +826,14 @@ function renderVLANInfo ($vlan_ck)
                        );
                        if
                        (
-                               $counterpart_vlan['domain_id'] == $vlan['domain_group_id'] or
+                               $counterpart_vlan['domain_id'] == $vlan['domain_group_id'] ||
                                in_array($counterpart_vlan['domain_id'], $group_members)
                        )
                        {
                                $group_ck_list[$counterpart_ck] = $counterpart_vlan;
                                $group_counterparts[] = $counterpart_link;
                        }
-                       elseif ($vlan['domain_group_id'] and $counterpart_vlan['domain_group_id'] == $vlan['domain_group_id'])
+                       elseif ($vlan['domain_group_id'] && $counterpart_vlan['domain_group_id'] == $vlan['domain_group_id'])
                                $group_counterparts[] = $counterpart_link;
                        else
                                $counterparts[] = $counterpart_link;
@@ -1079,9 +1079,9 @@ function renderObject8021QSync ($object_id)
                foreach ($plan as $port)
                        if
                        (
-                               $port['status'] == 'delete_conflict' or
-                               $port['status'] == 'merge_conflict' or
-                               $port['status'] == 'add_conflict' or
+                               $port['status'] == 'delete_conflict' ||
+                               $port['status'] == 'merge_conflict' ||
+                               $port['status'] == 'add_conflict' ||
                                $port['status'] == 'martian_conflict'
                        )
                                $maxdecisions++;
@@ -1284,8 +1284,8 @@ END
                        $radio_attrs = array ('left' => '', 'asis' => ' checked', 'right' => '');
                        if
                        (
-                               !acceptable8021QConfig ($item['right']) or
-                               count (array_diff ($item['right']['allowed'], $domvlans)) or
+                               ! acceptable8021QConfig ($item['right']) ||
+                               count (array_diff ($item['right']['allowed'], $domvlans)) ||
                                !goodModeForVSTRole ($item['right']['mode'], $item['vst_role'])
                        )
                                $radio_attrs['left'] = ' disabled';
@@ -1328,7 +1328,7 @@ END
 
                $anchor = '';
                $td_class = '';
-               if (isset ($hl_port_name) and $hl_port_name == $port_name)
+               if (isset ($hl_port_name) && $hl_port_name == $port_name)
                {
                        $anchor = "name='port-$hl_port_id'";
                        $td_class = ' border_highlight';
index 7b67516..c08a5ed 100644 (file)
@@ -10,7 +10,7 @@ function renderConfigMainpage ()
        global $pageno, $page;
        echo '<ul>';
        foreach ($page as $cpageno => $cpage)
-               if (isset ($cpage['parent']) and $cpage['parent'] == $pageno  && permitted($cpageno))
+               if (isset ($cpage['parent']) && $cpage['parent'] == $pageno && permitted ($cpageno))
                        echo '<li>' . mkA (getTitle ($cpageno), $cpageno) . "</li>\n";
        echo '</ul>';
 }
@@ -881,7 +881,7 @@ function renderTagRowForEditor ($taginfo, $parent_name = NULL, $level = 0)
        echo "<tr${trclass}><td align=left style='padding-left: " . ($level * 16) . "px;'>";
        if ($taginfo['kidc'])
                printImageHREF ('node-expanded-static');
-       if ($taginfo['refcnt']['total'] > 0 or $taginfo['kidc'])
+       if ($taginfo['refcnt']['total'] > 0 || $taginfo['kidc'])
                printImageHREF ('nodestroy', $taginfo['refcnt']['total'] . ' references, ' . $taginfo['kidc'] . ' sub-tags');
        else
                echo getOpLink (array ('op' => 'destroyTag', 'tag_id' => $taginfo['id']), '', 'destroy', 'Delete tag');
index 931912c..f5793fa 100644 (file)
@@ -345,7 +345,7 @@ function printNiftySelect ($groupList, $select_attrs = array(), $selected_id = N
 function getNiftySelect ($groupList, $select_attrs, $selected_id = NULL)
 {
        // special treatment for ungrouped data
-       if (count ($groupList) == 1 and isset ($groupList['other']))
+       if (count ($groupList) == 1 && isset ($groupList['other']))
                return getSelect ($groupList['other'], $select_attrs, $selected_id);
        if (!array_key_exists ('name', $select_attrs))
                return '';
@@ -440,7 +440,7 @@ function transformRequestData()
        global $sic;
        // Magic quotes feature is deprecated, but just in case the local system
        // still has it activated, reverse its effect.
-       $do_magic_quotes = (function_exists ('get_magic_quotes_gpc') and get_magic_quotes_gpc());
+       $do_magic_quotes = function_exists ('get_magic_quotes_gpc') && get_magic_quotes_gpc();
        $seen_keys = array();
 
        // Escape any globals before we ever try to use them, but keep a copy of originals.
@@ -559,7 +559,7 @@ function getRenderedIPv4NetCapacity ($range)
                $total = ip4_range_size ($range);
 
                // compute $a_total: own range size, without subranges
-               if (! isset ($range['kidc']) or $range['kidc'] == 0)
+               if (! isset ($range['kidc']) || $range['kidc'] == 0)
                        $a_total = $total;
                else
                {
@@ -609,7 +609,7 @@ function getRenderedIPv4NetCapacity ($range)
                addJS ('js/net-usage.js');
 
                $free_text = '';
-               if (isset ($range['kidc']) and $range['kidc'] > 0)
+               if (isset ($range['kidc']) && $range['kidc'] > 0)
                {
                        $free_masks = array_keys ($range['spare_ranges']);
                        sort ($free_masks, SORT_NUMERIC);
@@ -743,7 +743,7 @@ function serializeTags ($chain, $baseurl = '')
        foreach ($chain as $taginfo)
        {
                $title = '';
-               if (isset ($taginfo['user']) and isset ($taginfo['time']))
+               if (isset ($taginfo['user']) && isset ($taginfo['time']))
                        $title = htmlspecialchars ($taginfo['user'] . ', ' . formatAge ($taginfo['time']), ENT_QUOTES);
                if (isset($taginfo['parent_id']))
                {
@@ -796,17 +796,17 @@ function getPageName ($page_code)
 
 function printTagTRs ($cell, $baseurl = '')
 {
-       if (getConfigVar ('SHOW_EXPLICIT_TAGS') == 'yes' and count ($cell['etags']))
+       if (getConfigVar ('SHOW_EXPLICIT_TAGS') == 'yes' && count ($cell['etags']))
        {
                echo "<tr><th width='50%' class=tagchain>Explicit tags:</th><td class=tagchain>";
                echo serializeTags ($cell['etags'], $baseurl) . "</td></tr>\n";
        }
-       if (getConfigVar ('SHOW_IMPLICIT_TAGS') == 'yes' and count ($cell['itags']))
+       if (getConfigVar ('SHOW_IMPLICIT_TAGS') == 'yes' && count ($cell['itags']))
        {
                echo "<tr><th width='50%' class=tagchain>Implicit tags:</th><td class=tagchain>";
                echo serializeTags ($cell['itags'], $baseurl) . "</td></tr>\n";
        }
-       if (getConfigVar ('SHOW_AUTOMATIC_TAGS') == 'yes' and count ($cell['atags']))
+       if (getConfigVar ('SHOW_AUTOMATIC_TAGS') == 'yes' && count ($cell['atags']))
        {
                echo "<tr><th width='50%' class=tagchain>Automatic tags:</th><td class=tagchain>";
                echo serializeTags ($cell['atags']) . "</td></tr>\n";
@@ -835,7 +835,7 @@ function renderEntitySummary ($cell, $title, $values = array())
        echo "<table border=0 cellspacing=0 cellpadding=3 width='100%'>\n";
        foreach ($values as $name => $value)
        {
-               if (is_array ($value) and count ($value) == 1)
+               if (is_array ($value) && count ($value) == 1)
                {
                        $value = array_shift ($value);
                        echo $value;
@@ -1032,7 +1032,7 @@ function niftyString ($string, $maxlen = 30, $usetags = TRUE)
                return '&nbsp;';
        // a tab counts for a space
        $string = preg_replace ("/\t/", ' ', $string);
-       if (!$maxlen or mb_strlen ($string) <= $maxlen)
+       if (! $maxlen || mb_strlen ($string) <= $maxlen)
                return htmlspecialchars ($string, ENT_QUOTES, 'UTF-8');
        return
                ($usetags ? ("<span title='" . htmlspecialchars ($string, ENT_QUOTES, 'UTF-8') . "'>") : '') .
index aab087a..61b01e5 100644 (file)
@@ -83,7 +83,7 @@ $attrtypes = array
 
 function showLogoutURL ()
 {
-       $https = (isset ($_SERVER['HTTPS']) and $_SERVER['HTTPS'] == 'on') ? 's' : '';
+       $https = (isset ($_SERVER['HTTPS']) && $_SERVER['HTTPS'] == 'on') ? 's' : '';
        $port = (! in_array ($_SERVER['SERVER_PORT'], array (80, 443))) ? ':' . $_SERVER['SERVER_PORT'] : '';
        $pathinfo = pathinfo ($_SERVER['REQUEST_URI']);
        $dirname = $pathinfo['dirname'];
@@ -510,8 +510,8 @@ function renderRackspace ()
                                        $rackList = $row['racks'];
 
                                        if (
-                                               $location_id != '' and isset ($_SESSION['locationFilter']) and !in_array ($location_id, $_SESSION['locationFilter']) or
-                                               empty ($rackList) and ! $cellfilter['is_empty']
+                                               $location_id != '' && isset ($_SESSION['locationFilter']) && ! in_array ($location_id, $_SESSION['locationFilter']) ||
+                                               empty ($rackList) && ! $cellfilter['is_empty']
                                        )
                                                continue;
                                        $rackListIdx = 0;
@@ -523,7 +523,7 @@ function renderRackspace ()
                                        else
                                                foreach ($rackList as $rack)
                                                {
-                                                       if ($rackListIdx > 0 and $maxPerRow > 0 and $rackListIdx % $maxPerRow == 0)
+                                                       if ($rackListIdx > 0 && $maxPerRow > 0 && $rackListIdx % $maxPerRow == 0)
                                                        {
                                                                echo '</tr></table></th></tr>';
                                                                echo "<tr class=row_${order}><th class=tdleft></th><th class=tdleft>${row['row_name']} (continued)";
@@ -695,7 +695,7 @@ function renderRow ($row_id)
        foreach (getAttrValuesSorted ($row_id) as $record)
                if
                (
-                       $record['value'] != '' and
+                       $record['value'] != '' &&
                        permitted (NULL, NULL, NULL, array (array ('tag' => '$attr_' . $record['id'])))
                )
                        $summary['{sticker}' . $record['name']] = formatAttributeValue ($record);
@@ -813,7 +813,7 @@ function printObjectDetailsForRenderRack ($object_id, $hl_obj_id = 0)
                $prefix = "<div title='no asset tag";
        // Don't tell about label, if it matches common name.
        $body = '';
-       if ($objectData['name'] != $objectData['label'] and $objectData['label'] != '')
+       if ($objectData['name'] != $objectData['label'] && $objectData['label'] != '')
                $body = ", visible label is \"${objectData['label']}\"";
        // Display list of child objects, if any
        $objectChildren = getChildren ($objectData, 'object');
@@ -847,7 +847,7 @@ function printObjectDetailsForRenderRack ($object_id, $hl_obj_id = 0)
                                        $slotTitle[$slot] = "<div title='${childData['asset_no']}";
                                else
                                        $slotTitle[$slot] = "<div title='no asset tag";
-                               if ($childData['label'] != '' and $childData['label'] != $childData['dname'])
+                               if ($childData['label'] != '' && $childData['label'] != $childData['dname'])
                                        $slotTitle[$slot] .= ", visible label is \"${childData['label']}\"";
                                $slotTitle[$slot] .= "'>";
                                $slotClass[$slot] = 'state_T';
@@ -1222,7 +1222,7 @@ function renderEditRackForm ($rack_id)
        {
                // Skip the 'height' attribute as it's already displayed as a required field
                // Also skip the 'sort_order' attribute
-               if ($record['id'] == 27 or $record['id'] == 29)
+               if ($record['id'] == 27 || $record['id'] == 29)
                        continue;
                echo "<input type=hidden name=${i}_attr_id value=${record['id']}>";
                echo '<tr><td>';
@@ -1471,7 +1471,7 @@ function renderObject ($object_id)
        foreach (getAttrValuesSorted ($object_id) as $record)
                if
                (
-                       $record['value'] != '' and
+                       $record['value'] != '' &&
                        permitted (NULL, NULL, NULL, array (array ('tag' => '$attr_' . $record['id'])))
                )
                        $summary['{sticker}' . $record['name']] = formatAttributeValue ($record);
@@ -1588,7 +1588,7 @@ function renderObject ($object_id)
        }
 
        $forwards = $info['nat4'];
-       if (count($forwards['in']) or count($forwards['out']))
+       if (count ($forwards['in']) || count ($forwards['out']))
        {
                startPortlet('NATv4');
 
@@ -2015,7 +2015,7 @@ function showMessageOrError ()
        // Handle the arguments. Is there any better way to do it?
        foreach ($log_messages as $record)
        {
-               if (!isset ($record['c']) or !isset ($msginfo[$record['c']]))
+               if (! isset ($record['c']) || ! isset ($msginfo[$record['c']]))
                {
                        $prefix = isset ($record['c']) ? $record['c'] . ': ' : '';
                        echo "<div class=msg_neutral>(${prefix}this message was lost)</div>";
@@ -2206,7 +2206,7 @@ function renderRackSpaceForObject ($object_id)
        $allRacksData = listCells ('rack');
 
        // filter rack list to match only racks having common tags with the object (reducing $allRacksData)
-       if (! isset ($_REQUEST['show_all_racks']) and getConfigVar ('FILTER_RACKLIST_BY_TAGS') == 'yes')
+       if (! isset ($_REQUEST['show_all_racks']) && getConfigVar ('FILTER_RACKLIST_BY_TAGS') == 'yes')
        {
                $matching_racks = array();
                $matched_tags = array();
@@ -2226,7 +2226,7 @@ function renderRackSpaceForObject ($object_id)
                        if (! isset ($matching_racks[$rack_id]))
                                $matching_racks[$rack_id] = $rack;
                // if matching racks found, and rack list is reduced, show 'show all' link
-               if (count ($matching_racks) and count ($matching_racks) != count ($allRacksData))
+               if (count ($matching_racks) && count ($matching_racks) != count ($allRacksData))
                {
                        $filter_text = '';
                        foreach ($matched_tags as $tag)
@@ -2578,7 +2578,7 @@ function renderIPSpaceRecords ($tree, $baseurl, $target = 0, $level = 1)
                        if ($display_routers)
                                printRoutersTD (findNetRouters ($item), getConfigVar ('IPV4_TREE_RTR_AS_CELL'));
                        echo "</tr>";
-                       if ($item['symbol'] == 'node-expanded' or $item['symbol'] == 'node-expanded-static')
+                       if ($item['symbol'] == 'node-expanded' || $item['symbol'] == 'node-expanded-static')
                                $self ($item['kids'], $baseurl, $target, $level + 1);
                }
                elseif (getConfigVar ('IPV4_TREE_SHOW_UNALLOCATED') == 'yes')
@@ -2616,9 +2616,9 @@ function renderIPSpace()
                $top = NULL;
                foreach (listCells ($realm) as $net)
                {
-                       if (isset ($top) and IPNetContains ($top, $net))
+                       if (isset ($top) && IPNetContains ($top, $net))
                                ;
-                       elseif (! count ($cellfilter['expression']) or judgeCell ($net, $cellfilter['expression']))
+                       elseif (! count ($cellfilter['expression']) || judgeCell ($net, $cellfilter['expression']))
                                $top = $net;
                        else
                                continue;
@@ -2816,7 +2816,7 @@ function renderIPNetwork ($id)
                $domainclass[$domain_id] = $vlan_count == 1 ? '' : ($reuse_domain ? '{trwarning}' : '{trerror}');
        foreach ($range['8021q'] as $item)
                $summary[] = array ($domainclass[$item['domain_id']] . 'VLAN:', formatVLANAsHyperlink (getVlanRow ($item['domain_id'] . '-' . $item['vlan_id'])));
-       if (getConfigVar ('EXT_IPV4_VIEW') == 'yes' and count ($routers = findNetRouters ($range)))
+       if (getConfigVar ('EXT_IPV4_VIEW') == 'yes' && count ($routers = findNetRouters ($range)))
        {
                $summary['Routed by'] = '';
                foreach ($routers as $rtr)
@@ -3251,7 +3251,7 @@ function renderIPAddress ($ip_bin)
        renderEntitySummary ($address, 'summary', $summary);
 
        // render SLB portlet
-       if (! empty ($address['vslist']) or ! empty ($address['vsglist']) or ! empty ($address['rsplist']))
+       if (! empty ($address['vslist']) || ! empty ($address['vsglist']) || ! empty ($address['rsplist']))
        {
                startPortlet ("");
                if (! empty ($address['vsglist']))
@@ -3279,7 +3279,7 @@ function renderIPAddress ($ip_bin)
        echo "</td>\n";
 
        echo "<td class=pcright>";
-       if (isset ($address['class']) and ! empty ($address['allocs']))
+       if (isset ($address['class']) && ! empty ($address['allocs']))
        {
                startPortlet ('allocations');
                echo "<table class='widetable' cellpadding=5 cellspacing=0 border=0 align='center' width='100%'>\n";
@@ -3288,7 +3288,7 @@ function renderIPAddress ($ip_bin)
                foreach ($address['allocs'] as $bond)
                {
                        $tr_class = "${address['class']} tdleft";
-                       if (isset ($_REQUEST['hl_object_id']) and $_REQUEST['hl_object_id'] == $bond['object_id'])
+                       if (isset ($_REQUEST['hl_object_id']) && $_REQUEST['hl_object_id'] == $bond['object_id'])
                                $tr_class .= ' highlight';
                        echo "<tr class='$tr_class'>" .
                                "<td>" . makeIPAllocLink ($ip_bin, $bond) . "</td>" .
@@ -3359,7 +3359,7 @@ function renderIPAddressProperties ($ip_bin)
        echo "></tr><tr><td class=tdleft>";
        printImageHREF ('SAVE', 'Save changes', TRUE);
        echo "</td></form><td class=tdright>";
-       if ($address['name'] == '' and $address['reserved'] == 'no')
+       if ($address['name'] == '' && $address['reserved'] == 'no')
                printImageHREF ('CLEAR gray');
        else
        {
@@ -3434,8 +3434,8 @@ function renderNATv4ForObject ($object_id)
                foreach ($alloclist as $ip_bin => $alloc)
                {
                        $ip = $alloc['addrinfo']['ip'];
-                       $name = (!isset ($alloc['addrinfo']['name']) or $alloc['addrinfo']['name'] == '') ? '' : (' (' . stringForLabel ($alloc['addrinfo']['name']) . ')');
-                       $osif = (!isset ($alloc['osif']) or $alloc['osif'] == '') ? '' : ($alloc['osif'] . ': ');
+                       $name = (! isset ($alloc['addrinfo']['name']) || $alloc['addrinfo']['name'] == '') ? '' : (' (' . stringForLabel ($alloc['addrinfo']['name']) . ')');
+                       $osif = (! isset ($alloc['osif']) || $alloc['osif'] == '') ? '' : ($alloc['osif'] . ': ');
                        $options[$ip] = $osif . $ip . $name;
                }
                printSelect ($options, array ('name' => 'localip'));
@@ -3974,7 +3974,7 @@ function renderLocationPage ($location_id)
        foreach (getAttrValuesSorted ($locationData['id']) as $record)
                if
                (
-                       $record['value'] != '' and
+                       $record['value'] != '' &&
                        permitted (NULL, NULL, NULL, array (array ('tag' => '$attr_' . $record['id'])))
                )
                        $summary['{sticker}' . $record['name']] = formatAttributeValue ($record);
@@ -4065,7 +4065,7 @@ function renderEditLocationForm ($location_id)
                '<th class=tdright><label for=has_problems>Has problems:</label></th>' .
                '<td class=tdleft><input type=checkbox name=has_problems id=has_problems' . ($location['has_problems'] == 'yes' ? ' checked' : '') . '></td>' .
                "</tr>\n";
-       if (count ($location['locations']) == 0 and count ($location['rows']) == 0)
+       if (count ($location['locations']) == 0 && count ($location['rows']) == 0)
        {
                echo "<tr><td>&nbsp;</td><th class=tdright>Actions:</th><td class=tdleft>";
                echo getOpLink (array('op'=>'deleteLocation'), '', 'destroy', 'Delete location', 'need-confirmation');
@@ -4284,7 +4284,7 @@ function renderLivePTR ($id)
                                $cnt_match++;
                        }
                }
-               elseif ($addr['name'] == '' or $ptrname == '')
+               elseif ($addr['name'] == '' || $ptrname == '')
                {
                        echo ' class=trwarning';
                        $print_cbox = TRUE;
@@ -4297,7 +4297,7 @@ function renderLivePTR ($id)
                        $cnt_mismatch++;
                }
                echo "><td class='tdleft";
-               if (isset ($range['addrlist'][$ip_bin]['class']) and $range['addrlist'][$ip_bin]['class'] != '')
+               if (isset ($range['addrlist'][$ip_bin]['class']) && $range['addrlist'][$ip_bin]['class'] != '')
                        echo ' ' . $range['addrlist'][$ip_bin]['class'];
                echo "'>" . mkA ($straddr, 'ipaddress', $straddr) . '</td>';
                echo "<td class=tdleft>${addr['name']}</td><td class=tdleft>${ptrname}</td>";
@@ -4384,12 +4384,12 @@ function buildTagCheckboxRows ($inputname, $preselect, $neg_preselect, $taginfo,
                $ret['td_class'] .= $inverted ? ' selected-inverted' : ' selected';
                $ret['input_extraattrs'] = 'checked';
        }
-       if (array_key_exists ('is_assignable', $taginfo) and $taginfo['is_assignable'] == 'no')
+       if (array_key_exists ('is_assignable', $taginfo) && $taginfo['is_assignable'] == 'no')
        {
                $ret['input_extraattrs'] = 'disabled';
-               $ret['tr_class'] .= (array_key_exists ('kidc', $taginfo) and $taginfo['kidc'] == 0) ? ' trwarning' : ' trnull';
+               $ret['tr_class'] .= (array_key_exists ('kidc', $taginfo) && $taginfo['kidc'] == 0) ? ' trwarning' : ' trnull';
        }
-       if ($refcnt_realm != '' and isset ($taginfo['refcnt'][$refcnt_realm]))
+       if ($refcnt_realm != '' && isset ($taginfo['refcnt'][$refcnt_realm]))
                $ret['text_refcnt'] = $taginfo['refcnt'][$refcnt_realm];
        $ret = array ($ret);
        if (array_key_exists ('kids', $taginfo))
@@ -4496,7 +4496,7 @@ function renderCellFilterPortlet ($preselect, $realm, $cell_list = array(), $byp
        // "apply filter" button only gets active when there are checkbox/textarea inputs on the roster
        $enable_apply = FALSE;
        // and/or block
-       if (getConfigVar ('FILTER_SUGGEST_ANDOR') == 'yes' or $preselect['andor'] != '')
+       if (getConfigVar ('FILTER_SUGGEST_ANDOR') == 'yes' || $preselect['andor'] != '')
        {
                echo $hr;
                $hr = $ruler;
@@ -4515,7 +4515,7 @@ function renderCellFilterPortlet ($preselect, $realm, $cell_list = array(), $byp
        foreach ($preselect['negatedlist'] as $key)
                $negated_chain[] = array ('id' => $key);
        // tags block
-       if (getConfigVar ('FILTER_SUGGEST_TAGS') == 'yes' or count ($preselect['tagidlist']))
+       if (getConfigVar ('FILTER_SUGGEST_TAGS') == 'yes' || count ($preselect['tagidlist']))
        {
                if (count ($preselect['tagidlist']))
                        $enable_reset = TRUE;
@@ -4536,7 +4536,7 @@ function renderCellFilterPortlet ($preselect, $realm, $cell_list = array(), $byp
                        addJS ('tag_cb.enableSubmitOnClick()', TRUE);
        }
        // predicates block
-       if (getConfigVar ('FILTER_SUGGEST_PREDICATES') == 'yes' or count ($preselect['pnamelist']))
+       if (getConfigVar ('FILTER_SUGGEST_PREDICATES') == 'yes' || count ($preselect['pnamelist']))
        {
                if (count ($preselect['pnamelist']))
                        $enable_reset = TRUE;
@@ -4563,7 +4563,7 @@ function renderCellFilterPortlet ($preselect, $realm, $cell_list = array(), $byp
        }
        // extra code
        $enable_textify = FALSE;
-       if (getConfigVar ('FILTER_SUGGEST_EXTRA') == 'yes' or $preselect['extratext'] != '')
+       if (getConfigVar ('FILTER_SUGGEST_EXTRA') == 'yes' || $preselect['extratext'] != '')
        {
                $enable_textify = !empty ($preselect['text']) || !empty($preselect['extratext']);
                $enable_apply = TRUE;
@@ -4731,7 +4731,7 @@ function renderFile ($file_id)
 
        echo "</td>";
 
-       if (isolatedPermission ('file', 'download', $file) and '' != ($pcode = getFilePreviewCode ($file)))
+       if (isolatedPermission ('file', 'download', $file) && '' != ($pcode = getFilePreviewCode ($file)))
        {
                echo "<td class=pcright>";
                callHook ('renderFilePreview', $pcode);
@@ -4853,7 +4853,7 @@ function renderFilesPortlet ($entity_type = NULL, $entity_id = 0)
                        $file = spotEntity ('file', $file['id']);
                        renderCell ($file);
                        echo "</td><td class=tdleft>${file['comment']}</td></tr>";
-                       if (isolatedPermission ('file', 'download', $file) and '' != ($pcode = getFilePreviewCode ($file)))
+                       if (isolatedPermission ('file', 'download', $file) && '' != ($pcode = getFilePreviewCode ($file)))
                                echo "<tr><td colspan=2>${pcode}</td></tr>\n";
                }
                echo "</table><br>\n";
@@ -4966,7 +4966,7 @@ function printIPNetInfoTDs ($netinfo, $decor = array())
        echo $formatted;
        if (isset ($netinfo['id']))
                echo '</a>';
-       if (getConfigVar ('IPV4_TREE_SHOW_VLAN') == 'yes' and ! empty ($netinfo['8021q']))
+       if (getConfigVar ('IPV4_TREE_SHOW_VLAN') == 'yes' && ! empty ($netinfo['8021q']))
        {
                echo '<br>';
                renderNetVLAN ($netinfo);
@@ -5042,7 +5042,7 @@ function renderCell ($cell)
                echo "</td><td>";
                echo mkA ('<strong>' . stringForTD ($cell['name']) . '</strong>', 'file', $cell['id']);
                echo "</td><td rowspan=3 valign=top>";
-               if (isset ($cell['links']) and count ($cell['links']))
+               if (isset ($cell['links']) && count ($cell['links']))
                        printf ("<small>%s</small>", serializeFileLinks ($cell['links']));
                echo "</td></tr><tr><td>";
                echo count ($cell['etags']) ? ("<small>" . serializeTags ($cell['etags']) . "</small>") : '&nbsp;';
@@ -5144,7 +5144,7 @@ function getFilePreviewCode ($file)
                        $image = imagecreatefromstring ($file['contents']);
                        $width = imagesx ($image);
                        $height = imagesy ($image);
-                       if ($width < getConfigVar ('PREVIEW_IMAGE_MAXPXS') and $height < getConfigVar ('PREVIEW_IMAGE_MAXPXS'))
+                       if ($width < getConfigVar ('PREVIEW_IMAGE_MAXPXS') && $height < getConfigVar ('PREVIEW_IMAGE_MAXPXS'))
                                $resampled = FALSE;
                        else
                        {
@@ -5735,8 +5735,8 @@ function renderDiscoveredNeighbors ($object_id)
                                        {
                                                if
                                                (
-                                                       $portinfo_local['remote_object_id'] == $dp_remote_object_id
-                                                       and $portinfo_local['remote_name'] == $dp_neighbor['port']
+                                                       $portinfo_local['remote_object_id'] == $dp_remote_object_id &&
+                                                       $portinfo_local['remote_name'] == $dp_neighbor['port']
                                                )
                                                {
                                                        // set $portinfo_remote to corresponding remote port
@@ -5793,7 +5793,7 @@ function renderDiscoveredNeighbors ($object_id)
                        {
                                $count = count ($remote_list);
                                $td_class = '';
-                               if (isset ($hl_port_id) and $hl_port_id == $portinfo_local['id'])
+                               if (isset ($hl_port_id) && $hl_port_id == $portinfo_local['id'])
                                        $td_class = "class='border_highlight'";
                                echo "<td rowspan=\"$count\" $td_class NOWRAP>" .
                                        ($portinfo_local ?
@@ -6187,7 +6187,7 @@ function switchportInfoJS($object_id)
        foreach ($available_ops as $prefix => $data)
                if
                (
-                       permitted ('object', 'liveports', $data['op']) and
+                       permitted ('object', 'liveports', $data['op']) &&
                        validBreedFunction ($breed, $data['gw'])
                )
                        $allowed_ops[] = $prefix;
index 25c8ccd..552dea6 100644 (file)
@@ -1004,7 +1004,7 @@ function addIPAllocation ()
        if(!empty($address['allocs']) && ( ($address['allocs'][0]['type'] != 'shared') || ($alloc_type != 'shared') ) )
                showWarning("IP ".ip_format($ip_bin)." already in use by ".$address['allocs'][0]['object_name']." - ".$address['allocs'][0]['name']);
 
-       if  (getConfigVar ('IPV4_JAYWALK') != 'yes' and NULL === getIPAddressNetworkId ($ip_bin))
+       if (getConfigVar ('IPV4_JAYWALK') != 'yes' && NULL === getIPAddressNetworkId ($ip_bin))
        {
                showFuncMessage (__FUNCTION__, 'ERR1', array (ip_format ($ip_bin)));
                return;
@@ -1237,7 +1237,7 @@ function processGridForm (&$rackData, $unchecked_state, $checked_state, $object_
                        usePreparedDeleteBlade ('RackSpace', array ('rack_id' => $rack_id, 'unit_no' => $unit_no, 'atom' => $atom));
                        if ($newstate != 'F')
                                usePreparedInsertBlade ('RackSpace', array ('rack_id' => $rack_id, 'unit_no' => $unit_no, 'atom' => $atom, 'state' => $newstate));
-                       if ($newstate == 'T' and $object_id != 0)
+                       if ($newstate == 'T' && $object_id != 0)
                        {
                                // At this point we already have a record in RackSpace.
                                usePreparedUpdateBlade
@@ -1406,7 +1406,7 @@ function updateObjectAttributes ($object_id)
 
                // If the object is a rack, skip certain attributes as they are handled elsewhere
                // (height, sort_order)
-               if ($type_id == 1560 and ($attr_id == 27 or $attr_id == 29))
+               if ($type_id == 1560 && ($attr_id == 27 || $attr_id == 29))
                        continue;
 
                // Delete attribute and move on, when the field is empty or if the field
@@ -1487,7 +1487,7 @@ function addLotOfObjects()
        $taglist = genericAssertion ('taglist', 'array0');
        assertStringArg ('namelist', TRUE);
        $global_type_id = genericAssertion ('global_type_id', 'uint0');
-       if ($global_type_id == 0 or $_REQUEST['namelist'] == '')
+       if ($global_type_id == 0 || $_REQUEST['namelist'] == '')
        {
                showError ('Incomplete form has been ignored. Cheers.');
                return;
@@ -2903,7 +2903,7 @@ function save8021QPorts ()
        global $sic;
        $object_id = getBypassValue();
        $form_mode = genericAssertion ('form_mode', 'string');
-       if ($form_mode != 'save' and $form_mode != 'duplicate')
+       if ($form_mode != 'save' && $form_mode != 'duplicate')
                throw new InvalidRequestArgException ('form_mode', $form_mode);
        $extra = array();
 
@@ -3155,13 +3155,13 @@ function updVSTRule()
                        $rule_no++;
                        if
                        (
-                               ! isInteger (updVSTRule_get_named_param ('rule_no', $rule, $last_field))
-                               or ! isPCRE (updVSTRule_get_named_param ('port_pcre', $rule, $last_field))
-                               or NULL === updVSTRule_get_named_param ('port_role', $rule, $last_field)
-                               or ! array_key_exists (updVSTRule_get_named_param ('port_role', $rule, $last_field), $port_role_options)
-                               or NULL ===  updVSTRule_get_named_param ('wrt_vlans', $rule, $last_field)
-                               or ! preg_match ('/^[ 0-9\-,]*$/',  updVSTRule_get_named_param ('wrt_vlans', $rule, $last_field))
-                               or NULL ===  updVSTRule_get_named_param ('description', $rule, $last_field)
+                               ! isInteger (updVSTRule_get_named_param ('rule_no', $rule, $last_field)) ||
+                               ! isPCRE (updVSTRule_get_named_param ('port_pcre', $rule, $last_field)) ||
+                               NULL === updVSTRule_get_named_param ('port_role', $rule, $last_field) ||
+                               ! array_key_exists (updVSTRule_get_named_param ('port_role', $rule, $last_field), $port_role_options) ||
+                               NULL ===  updVSTRule_get_named_param ('wrt_vlans', $rule, $last_field) ||
+                               ! preg_match ('/^[ 0-9\-,]*$/',  updVSTRule_get_named_param ('wrt_vlans', $rule, $last_field)) ||
+                               NULL ===  updVSTRule_get_named_param ('description', $rule, $last_field)
                        )
                                throw new InvalidRequestArgException ($last_field, $rule[$last_field], "rule #$rule_no");
                }
@@ -3170,7 +3170,7 @@ function updVSTRule()
        catch (Exception $e)
        {
                // Every case that is soft-processed in process.php, will have the working copy available for a retry.
-               if ($e instanceof InvalidRequestArgException or $e instanceof RTDatabaseError)
+               if ($e instanceof InvalidRequestArgException || $e instanceof RTDatabaseError)
                {
                        startSession();
                        $_SESSION['vst_edited'] = $data;
@@ -3209,9 +3209,9 @@ function importDPData()
                        $portb = getPortInfo ($params['b_id']);
                        if
                        (
-                               $porta['linked'] or
-                               $portb['linked'] or
-                               ($porta['object_id'] != $object_id and $portb['object_id'] != $object_id)
+                               $porta['linked'] ||
+                               $portb['linked'] ||
+                               ($porta['object_id'] != $object_id && $portb['object_id'] != $object_id)
                        )
                        {
                                $nignored++;
@@ -3422,7 +3422,7 @@ function cleanupUCS()
                $o = spotEntity ('object', $item_id);
                $attrs = getAttrValues ($item_id);
                # use HW type to decide if the object was produced by autoPopulateUCS()
-               if (! array_key_exists (2, $attrs) or ! in_array ($attrs[2]['key'], $ucsproductmap))
+               if (! array_key_exists (2, $attrs) || ! in_array ($attrs[2]['key'], $ucsproductmap))
                {
                        showWarning ('Contained object ' . mkA ($o['dname'], 'object', $item_id) . ' is not an automatic UCS object');
                        $clear = FALSE;
@@ -3451,8 +3451,8 @@ function getOpspec()
        $ret = $opspec_list[$pageno . '-' . $tabno . '-' . $op];
        if
        (
-               !array_key_exists ('table', $ret)
-               or !array_key_exists ('action', $ret)
+               ! array_key_exists ('table', $ret) ||
+               array_key_exists ('action', $ret)
                // add further checks here
        )
                throw new RackTablesError ('malformed array structure in opspec_list', RackTablesError::INTERNAL);
@@ -3587,8 +3587,8 @@ function buildOpspecColumns ($opspec, $listname)
                        }
                        elseif // FIXME: remove the old declaration style at a later point
                        (
-                               ($argspec['assertion'] == 'uint0' and $arg_value == 0)
-                               or ($argspec['assertion'] == 'string0' and $arg_value == '')
+                               ($argspec['assertion'] == 'uint0' && $arg_value == 0) ||
+                               ($argspec['assertion'] == 'string0' && $arg_value == '')
                        )
                                switch (TRUE)
                                {
index 709591e..3e00f43 100644 (file)
@@ -117,7 +117,7 @@ INNER JOIN (
        while (TRUE)
        {
                $row = $result->fetch (PDO::FETCH_ASSOC);
-               if (isset ($prev_object_id) and (! $row or $row['object_id'] != $prev_object_id))
+               if (isset ($prev_object_id) && (! $row || $row['object_id'] != $prev_object_id))
                {
                        // handle sorted object's portlist
                        foreach (sortPortList ($rows_by_pn) as $ports_subarray)
@@ -222,7 +222,7 @@ function handlePopupPortLink()
        $port_info = getPortInfo ($_REQUEST['port']);
        $remote_port_info = getPortInfo ($_REQUEST['remote_port']);
        $POIFC = getPortOIFCompat();
-       if (isset ($_REQUEST['port_type']) and isset ($_REQUEST['remote_port_type']))
+       if (isset ($_REQUEST['port_type']) && isset ($_REQUEST['remote_port_type']))
        {
                $type_local = $_REQUEST['port_type'];
                $type_remote = $_REQUEST['remote_port_type'];
index a313f02..a7bd15f 100644 (file)
@@ -195,11 +195,11 @@ function detectDeviceBreed ($object_id)
 {
        global $breed_by_swcode, $breed_by_hwcode, $breed_by_mgmtcode;
        foreach (getAttrValues ($object_id) as $record)
-               if ($record['id'] == 4 and array_key_exists ($record['key'], $breed_by_swcode))
+               if ($record['id'] == 4 && array_key_exists ($record['key'], $breed_by_swcode))
                        return $breed_by_swcode[$record['key']];
-               elseif ($record['id'] == 2 and array_key_exists ($record['key'], $breed_by_hwcode))
+               elseif ($record['id'] == 2 && array_key_exists ($record['key'], $breed_by_hwcode))
                        return $breed_by_hwcode[$record['key']];
-               elseif ($record['id'] == 30 and array_key_exists ($record['key'], $breed_by_mgmtcode))
+               elseif ($record['id'] == 30 && array_key_exists ($record['key'], $breed_by_mgmtcode))
                        return $breed_by_mgmtcode[$record['key']];
        return '';
 }
index 0ea816a..ee41abf 100644 (file)
@@ -352,7 +352,7 @@ function prepareRealServersTable ($rs_list)
        $not_seen = $columns;
        foreach ($rs_list as $rs)
                foreach ($rs as $key => $value)
-                       if (! empty ($value) and isset ($not_seen[$key]))
+                       if (! empty ($value) && isset ($not_seen[$key]))
                                unset ($not_seen[$key]);
        foreach (array_keys ($not_seen) as $key)
                if ($key != 'rsip')
index b0e6078..bfca291 100644 (file)
@@ -48,7 +48,7 @@ class SLBTriplet
 
        static public function getTriplets ($cell)
        {
-               if (isset ($cell['ip_bin']) and isset ($cell['vslist']))
+               if (isset ($cell['ip_bin']) && isset ($cell['vslist']))
                        // cell is IPAddress
                        return self::getTripletsByIP ($cell['ip_bin']);
                $ret = array();
@@ -302,7 +302,7 @@ class MacroParser
                                for ($i = 0; $i < strlen ($line); $i++)
                                {
                                        $c = $line[$i];
-                                       if ($c == "'" and 0 == --$macro_deep)
+                                       if ($c == "'" && 0 == --$macro_deep)
                                        {
                                                if ($op === ':=')
                                                        $this->macros[$mname] = $this->expand ($mvalue);
@@ -566,7 +566,7 @@ function addRStoRSPool ($pool_id, $rsip_bin, $rsport = 0, $inservice = 'no', $rs
                (
                        'rspool_id' => $pool_id,
                        'rsip' => $rsip_bin,
-                       'rsport' => ($rsport == '' or $rsport === 0) ? NULL : $rsport,
+                       'rsport' => ($rsport == '' || $rsport === 0) ? NULL : $rsport,
                        'inservice' => $inservice == 'yes' ? 'yes' : 'no',
                        'rsconfig' => nullIfEmptyStr ($rsconfig),
                        'comment' => nullIfEmptyStr ($comment),
@@ -608,7 +608,7 @@ function commitUpdateRS ($rsid, $rsip_bin, $rsport = 0, $inservice = 'yes', $rsc
                array
                (
                        $rsip_bin,
-                       ($rsport == '' or $rsport === 0) ? NULL : $rsport,
+                       ($rsport == '' || $rsport === 0) ? NULL : $rsport,
                        $inservice,
                        nullIfEmptyStr ($rsconfig),
                        nullIfEmptyStr ($comment),
index 941b619..396ef49 100644 (file)
@@ -4122,7 +4122,7 @@ function doSwitchSNMPmining ($objectInfo, $device)
                if (array_key_exists ($major_line, $ios_codes))
                        updateStickerForCell ($objectInfo, 4, $ios_codes[$major_line]);
                $sysChassi = $device->snmpget ('1.3.6.1.4.1.9.3.6.3.0');
-               if ($sysChassi !== FALSE or $sysChassi !== NULL)
+               if ($sysChassi !== FALSE || $sysChassi !== NULL)
                        updateStickerForCell ($objectInfo, 1, str_replace ('"', '', substr ($sysChassi, strlen ('STRING: '))));
                checkPIC ('1-29');
                commitAddPort ($objectInfo['id'], 'con0', '1-29', 'console', ''); // RJ-45 RS-232 console
@@ -4135,7 +4135,7 @@ function doSwitchSNMPmining ($objectInfo, $device)
                        commitAddPort ($objectInfo['id'], 'AC-in-1', '1-16', 'AC1', '');
                        commitAddPort ($objectInfo['id'], 'AC-in-2', '1-16', 'AC2', '');
                }
-               elseif ($sysObjectID != '9.1.749' and $sysObjectID != '9.1.920')
+               elseif ($sysObjectID != '9.1.749' && $sysObjectID != '9.1.920')
                {
                        // assume the rest have one AC input, but exclude blade devices
                        checkPIC ('1-16'); // AC input
@@ -4168,7 +4168,7 @@ function doSwitchSNMPmining ($objectInfo, $device)
                        updateStickerForCell ($objectInfo, 4, $nxos_codes[$major_line]);
                updateStickerForCell ($objectInfo, 5, $exact_release);
                $sysChassi = $device->snmpget ('1.3.6.1.2.1.47.1.1.1.1.11.149');
-               if ($sysChassi !== FALSE or $sysChassi !== NULL)
+               if ($sysChassi !== FALSE || $sysChassi !== NULL)
                        updateStickerForCell ($objectInfo, 1, str_replace ('"', '', substr ($sysChassi, strlen ('STRING: '))));
                checkPIC ('1-29');
                commitAddPort ($objectInfo['id'], 'con0', '1-29', 'console', ''); // RJ-45 RS-232 console
@@ -4241,7 +4241,7 @@ function doSwitchSNMPmining ($objectInfo, $device)
                updateStickerForCell ($objectInfo, 5, $exact_release);
                # FOUNDRY-SN-AGENT-MIB::snChasSerNum.0
                $sysChassi = $device->snmpget ('enterprises.1991.1.1.1.1.2.0');
-               if ($sysChassi !== FALSE or $sysChassi !== NULL)
+               if ($sysChassi !== FALSE || $sysChassi !== NULL)
                        updateStickerForCell ($objectInfo, 1, str_replace ('"', '', substr ($sysChassi, strlen ('STRING: '))));
 
                # Type of uplink module installed.
index 9a449a2..67de9fc 100644 (file)
@@ -132,7 +132,7 @@ function createTrueColorOrThrow ($context, $width, $height)
        // Sometimes GD is missing even though it was available at install time.
        if
        (
-               ! function_exists ('imagecreatetruecolor') or
+               ! function_exists ('imagecreatetruecolor') ||
                FALSE === $img = @imagecreatetruecolor ($width, $height)
        )
                throw new RTImageError ($context);
@@ -272,7 +272,7 @@ function renderProgressBar4Image ($px1, $px2, $px3)
        {
                $off = $offsets[$i];
                $clr = $colors[$i];
-               if ($pos + $off > $width or $off < 0)
+               if ($pos + $off > $width || $off < 0)
                        throw new RTImageError ('pbar_arg_error');
                if ($off > 0)
                        imagefilledrectangle ($img, $pos, 0, $pos + $off, $height, $clr);
@@ -336,7 +336,7 @@ function renderImagePreview ($file_id)
        unset ($file);
        $width = imagesx ($image);
        $height = imagesy ($image);
-       if ($width > getConfigVar ('PREVIEW_IMAGE_MAXPXS') or $height > getConfigVar ('PREVIEW_IMAGE_MAXPXS'))
+       if ($width > getConfigVar ('PREVIEW_IMAGE_MAXPXS') || $height > getConfigVar ('PREVIEW_IMAGE_MAXPXS'))
        {
                $ratio = getConfigVar ('PREVIEW_IMAGE_MAXPXS') / max ($width, $height);
                $newwidth = $width * $ratio;
@@ -385,14 +385,14 @@ function proxyStaticURI ($URI)
        $matches = array();
        if
        (
-               ! preg_match (RE_STATIC_URI, $URI, $matches)
-               or ! array_key_exists (strtolower ($matches[1]), $content_type)
+               ! preg_match (RE_STATIC_URI, $URI, $matches) ||
+               ! array_key_exists (strtolower ($matches[1]), $content_type)
        )
                printStatic404();
        global $local_staticdir, $racktables_staticdir;
        if (isset ($local_staticdir))
                $fh = @fopen ("${local_staticdir}/${URI}", 'r');
-       if (! isset ($fh) or FALSE === $fh)
+       if (! isset ($fh) || FALSE === $fh)
                $fh = @fopen ("${racktables_staticdir}/${URI}", 'r');
        if (FALSE === $fh)
                printStatic404();
index 66e366f..368a308 100644 (file)
@@ -62,7 +62,7 @@ function trigger_liveports ()
        foreach (array ('getportstatus' => 'get_link_status', 'getmaclist' => 'get_mac_list') as $command => $opname)
                if
                (
-                       validBreedFunction ($breed, $command) and
+                       validBreedFunction ($breed, $command) &&
                        permitted (NULL, 'liveports', $opname)
                )
                        return 'std';
@@ -184,7 +184,7 @@ function trigger_object_8021qorder ()
 {
        if (NULL !== getVLANSwitchInfo (getBypassValue()))
                return 'std';
-       if (!count (getVLANDomainOptions()) or ! getEntitiesCount ('vst'))
+       if (! count (getVLANDomainOptions()) || ! getEntitiesCount ('vst'))
                return '';
        if (considerConfiguredConstraint (spotEntity ('object', getBypassValue()), 'VLANSWITCH_LISTSRC'))
                return 'attn';
@@ -193,7 +193,7 @@ function trigger_object_8021qorder ()
 
 function trigger_8021q_configured ()
 {
-       if (!count (getVLANDomainOptions()) or ! getEntitiesCount ('vst'))
+       if (! count (getVLANDomainOptions()) || ! getEntitiesCount ('vst'))
                return '';
        return 'std';
 }
@@ -266,7 +266,7 @@ function trigger_anyDP ($command, $constraint)
 {
        if
        (
-               validBreedFunction (detectDeviceBreed (getBypassValue()), $command) and
+               validBreedFunction (detectDeviceBreed (getBypassValue()), $command) &&
                considerConfiguredConstraint (spotEntity ('object', getBypassValue()), $constraint)
        )
                return 'std';
@@ -304,7 +304,7 @@ function triggerCactiGraphs ()
                return '';
        if
        (
-               count (getCactiGraphsForObject (getBypassValue())) or
+               count (getCactiGraphsForObject (getBypassValue())) ||
                considerConfiguredConstraint (spotEntity ('object', getBypassValue()), 'CACTI_LISTSRC')
        )
                return 'std';
@@ -318,7 +318,7 @@ function triggerMuninGraphs()
                return '';
        if
        (
-               count (getMuninGraphsForObject (getBypassValue())) or
+               count (getMuninGraphsForObject (getBypassValue())) ||
                considerConfiguredConstraint (spotEntity ('object', getBypassValue()), 'MUNIN_LISTSRC')
        )
                return 'std';
index cdcbfde..8f96a3d 100644 (file)
@@ -274,7 +274,7 @@ function getDBUpgradePath ($v1, $v2)
                '0.20.10',
                '0.20.11',
        );
-       if (!in_array ($v1, $versionhistory) or !in_array ($v2, $versionhistory))
+       if (! in_array ($v1, $versionhistory) || ! in_array ($v2, $versionhistory))
                return NULL;
        $skip = TRUE;
        $path = NULL;
@@ -284,7 +284,7 @@ function getDBUpgradePath ($v1, $v2)
        // Now collect all versions > $v1 and <= $v2
        foreach ($versionhistory as $v)
        {
-               if ($skip and $v == $v1)
+               if ($skip && $v == $v1)
                {
                        $skip = FALSE;
                        $path = array();
@@ -2065,10 +2065,10 @@ function renderUpgraderHTML()
 
        if
        (
-               !isset ($_SERVER['PHP_AUTH_USER']) or
-               $_SERVER['PHP_AUTH_USER'] == '' or
-               !isset ($_SERVER['PHP_AUTH_PW']) or
-               $_SERVER['PHP_AUTH_PW'] == '' or
+               ! isset ($_SERVER['PHP_AUTH_USER']) ||
+               $_SERVER['PHP_AUTH_USER'] == '' ||
+               ! isset ($_SERVER['PHP_AUTH_PW']) ||
+               $_SERVER['PHP_AUTH_PW'] == '' ||
                !authenticate_admin ($_SERVER['PHP_AUTH_USER'], $_SERVER['PHP_AUTH_PW'])
        )
        {
index f321343..dbd383d 100644 (file)
@@ -64,7 +64,7 @@ try {
                $file = getFile (getBypassValue());
                header("Content-Type: {$file['type']}");
                header("Content-Length: {$file['size']}");
-               if (! array_key_exists ('asattach', $_REQUEST) or $_REQUEST['asattach'] != 'no')
+               if (! array_key_exists ('asattach', $_REQUEST) || $_REQUEST['asattach'] != 'no')
                        header("Content-Disposition: attachment; filename={$file['name']}");
                echo $file['contents'];
                break;
@@ -203,7 +203,7 @@ try {
                // Include init after ophandlers/snmp, not before, so local.php can redefine things.
                require_once 'inc/ophandlers.php';
                // snmp.php is an exception, it is treated by a special hack
-               if (isset ($_REQUEST['op']) and $_REQUEST['op'] == 'querySNMPData')
+               if (isset ($_REQUEST['op']) && $_REQUEST['op'] == 'querySNMPData')
                        require_once 'inc/snmp.php';
                require_once 'inc/init.php';
                try
@@ -217,7 +217,7 @@ try {
                        fixContext();
                        if
                        (
-                               !isset ($ophandler[$pageno][$tabno][$op]) or
+                               ! isset ($ophandler[$pageno][$tabno][$op]) ||
                                ! is_callable ($ophandler[$pageno][$tabno][$op])
                        )
                                throw new RackTablesError ("Invalid navigation data for '${pageno}-${tabno}-${op}'", RackTablesError::INTERNAL);
@@ -262,7 +262,7 @@ try {
 
                header ('Content-Type: text/html; charset=UTF-8');
                // call the main handler - page or tab handler.
-               if (isset ($popuphandler[$helper]) and is_callable ($popuphandler[$helper]))
+               if (isset ($popuphandler[$helper]) && is_callable ($popuphandler[$helper]))
                        call_user_func ($popuphandler[$helper], $helper);
                else
                        throw new RackTablesError ("Missing handler function for node '${handler}'", RackTablesError::INTERNAL);