r2809 - pull bugfix into trunk
authorDenis Ovsienko <infrastation@yandex.ru>
Mon, 1 Jun 2009 15:34:29 +0000 (15:34 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Mon, 1 Jun 2009 15:34:29 +0000 (15:34 +0000)
12 files changed:
ChangeLog
download.php
inc/auth.php
inc/config.php
inc/database.php
inc/functions.php
inc/gateways.php
inc/init.php
inc/interface.php
inc/ophandlers.php
inc/snmp.php
process.php

index c29be0d3cc5c672d4820ac3af9bf9080852a457d..07cc41c31e74cc4eb386621ad811dadd175be92f 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -4,6 +4,7 @@
        bugfix: user's displayed name was sometimes lost with LDAP auth
        bugfix: rackspace rows highlightling was broken in previous release
        bugfix: IPv4 address picker window was broken
+       bugfix: string value "0" was incorrectly handled, when displaying or entering data
 0.17.0 2009-05-20
        new feature: file attachments (by Aaron)
        new feature: ability to delete objects (by Aaron)
index b6b70358a08102461aac4e4acfbf5510ec4c9347..12f321fca84c92325992409116514dabca442140 100644 (file)
@@ -1,12 +1,7 @@
 <?php
 require 'inc/init.php';
 
-if (empty($_REQUEST['file_id']))
-{
-       showError ("Invalid file specified", __FILE__);
-       die();
-}
-
+assertUIntArg ('file_id', __FILE__);
 $pageno = 'file';
 $tabno = 'default';
 fixContext();
index faa3dcda1568d4b85e448020f7a7127368151c80..51238f26c3c9c04840b03bfd80cf247cece2c2a7 100644 (file)
@@ -106,7 +106,7 @@ function permitted ($p = NULL, $t = NULL, $o = NULL, $annex = array())
        $my_auto_tags = $auto_tags;
        $my_auto_tags[] = array ('tag' => '$page_' . $p);
        $my_auto_tags[] = array ('tag' => '$tab_' . $t);
-       if ($o === NULL and !empty ($op)) // $op can be set to empty string
+       if ($o === NULL and strlen ($op)) // $op can be set to empty string
        {
                $my_auto_tags[] = array ('tag' => '$op_' . $op);
                $my_auto_tags[] = array ('tag' => '$any_op');
@@ -237,14 +237,14 @@ function queryLDAPServer ($username, $password)
                return array ('result' => 'CAN');
 
        // Decide on the username we will actually authenticate for.
-       if (isset ($LDAP_options['domain']) and !empty ($LDAP_options['domain']))
+       if (isset ($LDAP_options['domain']) and strlen ($LDAP_options['domain']))
                $auth_user_name = $username . "@" . $LDAP_options['domain'];
        elseif
        (
                isset ($LDAP_options['search_dn']) and
-               !empty ($LDAP_options['search_dn']) and
+               strlen ($LDAP_options['search_dn']) and
                isset ($LDAP_options['search_attr']) and
-               !empty ($LDAP_options['search_attr'])
+               strlen ($LDAP_options['search_attr'])
        )
        {
                $results = @ldap_search ($connect, $LDAP_options['search_dn'], '(' . $LDAP_options['search_attr'] . "=${username})", array("dn"));
@@ -279,9 +279,12 @@ function queryLDAPServer ($username, $password)
        // Displayed name only makes sense for authenticated users anyway.
        if
        (
-               !empty ($LDAP_options['displayname_attrs']) and
-               !empty ($LDAP_options['search_dn']) and
-               !empty ($LDAP_options['search_attr'])
+               isset ($LDAP_options['displayname_attrs']) and
+               strlen ($LDAP_options['displayname_attrs']) and
+               isset ($LDAP_options['search_dn']) and
+               strlen ($LDAP_options['search_dn']) and
+               isset ($LDAP_options['search_attr']) and
+               strlen ($LDAP_options['search_attr'])
        )
        {
                $results = @ldap_search
index ac16917c74a1fdbf9997e610acd64ea19c22a263..a3c4a9635c2ea4c7c611a47abaf850e4f3b17a70 100644 (file)
@@ -61,7 +61,7 @@ function setConfigVar ($varname = '', $varvalue = '', $softfail = FALSE)
                showError ('Configuration cache is unavailable', __FUNCTION__);
                die;
        }
-       if (empty ($varname))
+       if (!strlen ($varname))
        {
                showError ("Empty argument", __FUNCTION__);
                die;
@@ -80,7 +80,7 @@ function setConfigVar ($varname = '', $varvalue = '', $softfail = FALSE)
                showError ($errormsg, __FUNCTION__);
                die;
        }
-       if (empty ($varvalue) && $configCache[$varname]['emptyok'] != 'yes')
+       if (!strlen ($varvalue) && $configCache[$varname]['emptyok'] != 'yes')
        {
                $errormsg = "'${varname}' is configured to take non-empty value. Perhaps there was a reason to do so.";
                if ($softfail)
@@ -88,7 +88,7 @@ function setConfigVar ($varname = '', $varvalue = '', $softfail = FALSE)
                showError ($errormsg, __FUNCTION__);
                die;
        }
-       if (!empty ($varvalue) && $configCache[$varname]['vartype'] == 'uint' && (!is_numeric ($varvalue) or $varvalue < 0 ))
+       if (strlen ($varvalue) && $configCache[$varname]['vartype'] == 'uint' && (!is_numeric ($varvalue) or $varvalue < 0 ))
        {
                $errormsg = "'${varname}' can accept UINT values only";
                if ($softfail)
index 32f17855b30eeededb3768519373cff0932f96bc..f57af4aff52d20f4d907e1120146c1aa63bee4c8 100644 (file)
@@ -576,7 +576,7 @@ function getObjectPortsAndLinks ($object_id)
                        $ret[$tmpkey]['remote_id'] = $remote_id;
                        unset ($result);
                        // only call displayedName() when necessary
-                       if (empty ($ret[$tmpkey]['remote_object_name']) and !empty ($ret[$tmpkey]['remote_object_id']))
+                       if (!strlen ($ret[$tmpkey]['remote_object_name']) and strlen ($ret[$tmpkey]['remote_object_id']))
                        {
                                $oi = spotEntity ('object', $ret[$tmpkey]['remote_object_id']);
                                $ret[$tmpkey]['remote_object_name'] = $oi['dname'];
@@ -588,7 +588,7 @@ function getObjectPortsAndLinks ($object_id)
 
 function commitAddRack ($name, $height = 0, $row_id = 0, $comment, $taglist)
 {
-       if ($row_id <= 0 or $height <= 0 or empty ($name))
+       if ($row_id <= 0 or $height <= 0 or !strlen ($name))
                return FALSE;
        $result = useInsertBlade
        (
@@ -620,11 +620,11 @@ function commitAddObject ($new_name, $new_label, $new_barcode, $new_type_id, $ne
                'RackObject',
                array
                (
-                       'name' => empty ($new_name) ? 'NULL' : "'${new_name}'",
+                       'name' => !strlen ($new_name) ? 'NULL' : "'${new_name}'",
                        'label' => "'${new_label}'",
-                       'barcode' => empty ($new_barcode) ? 'NULL' : "'${new_barcode}'",
+                       'barcode' => !strlen ($new_barcode) ? 'NULL' : "'${new_barcode}'",
                        'objtype_id' => $new_type_id,
-                       'asset_no' => empty ($new_asset_no) ? 'NULL' : "'${new_asset_no}'"
+                       'asset_no' => !strlen ($new_asset_no) ? 'NULL' : "'${new_asset_no}'"
                )
        );
        if ($result1 == NULL)
@@ -653,9 +653,9 @@ function commitUpdateObject ($object_id = 0, $new_name = '', $new_label = '', $n
                return FALSE;
        }
        global $dbxlink;
-       $new_asset_no = empty ($new_asset_no) ? 'NULL' : "'${new_asset_no}'";
-       $new_barcode = empty ($new_barcode) ? 'NULL' : "'${new_barcode}'";
-       $new_name = empty ($new_name) ? 'NULL' : "'${new_name}'";
+       $new_asset_no = !strlen ($new_asset_no) ? 'NULL' : "'${new_asset_no}'";
+       $new_barcode = !strlen ($new_barcode) ? 'NULL' : "'${new_barcode}'";
+       $new_name = !strlen ($new_name) ? 'NULL' : "'${new_name}'";
        $query = "update RackObject set name=${new_name}, label='${new_label}', barcode=${new_barcode}, objtype_id='${new_type_id}', " .
                "has_problems='${new_has_problems}', asset_no=${new_asset_no}, comment='${new_comment}' " .
                "where id='${object_id}' limit 1";
@@ -719,7 +719,7 @@ function commitDeleteRack($rack_id)
 
 function commitUpdateRack ($rack_id, $new_name, $new_height, $new_row_id, $new_comment)
 {
-       if (empty ($rack_id) || empty ($new_name) || empty ($new_height))
+       if (!strlen ($rack_id) || !strlen ($new_name) || !strlen ($new_height))
        {
                showError ('Not all required args are present.', __FUNCTION__);
                return FALSE;
@@ -1891,7 +1891,7 @@ function getRackspaceStats ()
        {
                $result = useSelectBlade ($item['q'], __FUNCTION__);
                $row = $result->fetch (PDO::FETCH_NUM);
-               $ret[$item['txt']] = empty ($row[0]) ? 0 : $row[0];
+               $ret[$item['txt']] = !strlen ($row[0]) ? 0 : $row[0];
                $result->closeCursor();
                unset ($result);
        }
@@ -1961,7 +1961,7 @@ mysql> select tag_id from TagStorage left join TagTree on tag_id = id where id i
 
 function commitUpdateDictionary ($chapter_no = 0, $dict_key = 0, $dict_value = '')
 {
-       if ($chapter_no <= 0 or $dict_key <= 0 or empty ($dict_value))
+       if ($chapter_no <= 0 or $dict_key <= 0 or !strlen ($dict_value))
        {
                showError ('Invalid args', __FUNCTION__);
                die;
@@ -1981,7 +1981,7 @@ function commitUpdateDictionary ($chapter_no = 0, $dict_key = 0, $dict_value = '
 
 function commitSupplementDictionary ($chapter_no = 0, $dict_value = '')
 {
-       if ($chapter_no <= 0 or empty ($dict_value))
+       if ($chapter_no <= 0 or !strlen ($dict_value))
        {
                showError ('Invalid args', __FUNCTION__);
                die;
@@ -2015,7 +2015,7 @@ function commitReduceDictionary ($chapter_no = 0, $dict_key = 0)
 
 function commitAddChapter ($chapter_name = '')
 {
-       if (empty ($chapter_name))
+       if (!strlen ($chapter_name))
        {
                showError ('Invalid args', __FUNCTION__);
                die;
@@ -2029,7 +2029,7 @@ function commitAddChapter ($chapter_name = '')
 
 function commitUpdateChapter ($chapter_no = 0, $chapter_name = '')
 {
-       if ($chapter_no <= 0 or empty ($chapter_name))
+       if ($chapter_no <= 0 or !strlen ($chapter_name))
        {
                showError ('Invalid args', __FUNCTION__);
                die;
@@ -2070,7 +2070,7 @@ function commitDeleteChapter ($chapter_no = 0)
 // nice <select> drop-downs.
 function readChapter ($chapter_name = '')
 {
-       if (empty ($chapter_name))
+       if (!strlen ($chapter_name))
        {
                showError ('invalid argument', __FUNCTION__);
                return NULL;
@@ -2130,7 +2130,7 @@ function getAttrMap ()
 
 function commitUpdateAttribute ($attr_id = 0, $attr_name = '')
 {
-       if ($attr_id <= 0 or empty ($attr_name))
+       if ($attr_id <= 0 or !strlen ($attr_name))
        {
                showError ('Invalid args', __FUNCTION__);
                die;
@@ -2150,7 +2150,7 @@ function commitUpdateAttribute ($attr_id = 0, $attr_name = '')
 
 function commitAddAttribute ($attr_name = '', $attr_type = '')
 {
-       if (empty ($attr_name))
+       if (!strlen ($attr_name))
        {
                showError ('Invalid args', __FUNCTION__);
                die;
@@ -2300,7 +2300,7 @@ function commitUpdateAttrValue ($object_id = 0, $attr_id = 0, $value = '')
                showError ('Invalid arguments', __FUNCTION__);
                die;
        }
-       if (empty ($value))
+       if (!strlen ($value))
                return commitResetAttrValue ($object_id, $attr_id);
        global $dbxlink;
        $query1 = "select type as attr_type from Attribute where id = ${attr_id}";
@@ -2430,7 +2430,7 @@ function loadConfigCache ()
 function storeConfigVar ($varname = NULL, $varvalue = NULL)
 {
        global $dbxlink;
-       if (empty ($varname) || $varvalue === NULL)
+       if (!strlen ($varname) || $varvalue === NULL)
        {
                showError ('Invalid arguments', __FUNCTION__);
                return FALSE;
@@ -2465,7 +2465,7 @@ function getDatabaseVersion ()
                die (__FUNCTION__ . ': SQL query #1 failed with error ' . $errorInfo[2]);
        }
        $rows = $result->fetchAll (PDO::FETCH_NUM);
-       if (count ($rows) != 1 || empty ($rows[0][0]))
+       if (count ($rows) != 1 || !strlen ($rows[0][0]))
        {
                $result->closeCursor();
                die (__FUNCTION__ . ': Cannot guess database version. Config table is present, but DB_VERSION is missing or invalid. Giving up.');
@@ -2517,7 +2517,7 @@ function addRStoRSPool ($pool_id = 0, $rsip = '', $rsport = 0, $inservice = 'no'
                showError ('Invalid arguments', __FUNCTION__);
                die;
        }
-       if (empty ($rsport) or $rsport == 0)
+       if (!strlen ($rsport) or $rsport === 0)
                $rsport = 'NULL';
        return useInsertBlade
        (
@@ -2528,14 +2528,14 @@ function addRStoRSPool ($pool_id = 0, $rsip = '', $rsport = 0, $inservice = 'no'
                        'rsport' => $rsport,
                        'rspool_id' => $pool_id,
                        'inservice' => ($inservice == 'yes' ? "'yes'" : "'no'"),
-                       'rsconfig' => (empty ($rsconfig) ? 'NULL' : "'${rsconfig}'")
+                       'rsconfig' => (!strlen ($rsconfig) ? 'NULL' : "'${rsconfig}'")
                )
        );
 }
 
 function commitCreateVS ($vip = '', $vport = 0, $proto = '', $name = '', $vsconfig, $rsconfig, $taglist = array())
 {
-       if (empty ($vip) or $vport <= 0 or empty ($proto))
+       if (!strlen ($vip) or $vport <= 0 or !strlen ($proto))
                return __FUNCTION__ . ': invalid arguments';
        if (!useInsertBlade
        (
@@ -2545,9 +2545,9 @@ function commitCreateVS ($vip = '', $vport = 0, $proto = '', $name = '', $vsconf
                        'vip' => "inet_aton('${vip}')",
                        'vport' => $vport,
                        'proto' => "'${proto}'",
-                       'name' => (empty ($name) ? 'NULL' : "'${name}'"),
-                       'vsconfig' => (empty ($vsconfig) ? 'NULL' : "'${vsconfig}'"),
-                       'rsconfig' => (empty ($rsconfig) ? 'NULL' : "'${rsconfig}'")
+                       'name' => (!strlen ($name) ? 'NULL' : "'${name}'"),
+                       'vsconfig' => (!strlen ($vsconfig) ? 'NULL' : "'${vsconfig}'"),
+                       'rsconfig' => (!strlen ($rsconfig) ? 'NULL' : "'${rsconfig}'")
                )
        ))
                return __FUNCTION__ . ': SQL insertion failed';
@@ -2569,8 +2569,8 @@ function addLBtoRSPool ($pool_id = 0, $object_id = 0, $vs_id = 0, $vsconfig = ''
                        'object_id' => $object_id,
                        'rspool_id' => $pool_id,
                        'vs_id' => $vs_id,
-                       'vsconfig' => (empty ($vsconfig) ? 'NULL' : "'${vsconfig}'"),
-                       'rsconfig' => (empty ($rsconfig) ? 'NULL' : "'${rsconfig}'")
+                       'vsconfig' => (!strlen ($vsconfig) ? 'NULL' : "'${vsconfig}'"),
+                       'rsconfig' => (!strlen ($rsconfig) ? 'NULL' : "'${rsconfig}'")
                )
        );
 }
@@ -2618,12 +2618,12 @@ function commitUpdateRS ($rsid = 0, $rsip = '', $rsport = 0, $rsconfig = '')
                showError ("Invalid IP address '${rsip}'", __FUNCTION__);
                die;
        }
-       if (empty ($rsport) or $rsport == 0)
+       if (!strlen ($rsport) or $rsport === 0)
                $rsport = 'NULL';
        global $dbxlink;
        $query =
                "update IPv4RS set rsip = inet_aton('${rsip}'), rsport = ${rsport}, rsconfig = " .
-               (empty ($rsconfig) ? 'NULL' : "'${rsconfig}'") .
+               (!strlen ($rsconfig) ? 'NULL' : "'${rsconfig}'") .
                " where id = ${rsid} limit 1";
        $result = $dbxlink->query ($query);
        if ($result == NULL)
@@ -2644,9 +2644,9 @@ function commitUpdateLB ($object_id = 0, $pool_id = 0, $vs_id = 0, $vsconfig = '
        global $dbxlink;
        $query =
                "update IPv4LB set vsconfig = " .
-               (empty ($vsconfig) ? 'NULL' : "'${vsconfig}'") .
+               (!strlen ($vsconfig) ? 'NULL' : "'${vsconfig}'") .
                ', rsconfig = ' .
-               (empty ($rsconfig) ? 'NULL' : "'${rsconfig}'") .
+               (!strlen ($rsconfig) ? 'NULL' : "'${rsconfig}'") .
                " where object_id = ${object_id} and rspool_id = ${pool_id} " .
                "and vs_id = ${vs_id} limit 1";
        $result = $dbxlink->exec ($query);
@@ -2658,7 +2658,7 @@ function commitUpdateLB ($object_id = 0, $pool_id = 0, $vs_id = 0, $vsconfig = '
 
 function commitUpdateVS ($vsid = 0, $vip = '', $vport = 0, $proto = '', $name = '', $vsconfig = '', $rsconfig = '')
 {
-       if ($vsid <= 0 or empty ($vip) or $vport <= 0 or empty ($proto))
+       if ($vsid <= 0 or !strlen ($vip) or $vport <= 0 or !strlen ($proto))
        {
                showError ('Invalid args', __FUNCTION__);
                die;
@@ -2668,9 +2668,9 @@ function commitUpdateVS ($vsid = 0, $vip = '', $vport = 0, $proto = '', $name =
                "vip = inet_aton('${vip}'), " .
                "vport = ${vport}, " .
                "proto = '${proto}', " .
-               'name = ' . (empty ($name) ? 'NULL,' : "'${name}', ") .
-               'vsconfig = ' . (empty ($vsconfig) ? 'NULL,' : "'${vsconfig}', ") .
-               'rsconfig = ' . (empty ($rsconfig) ? 'NULL' : "'${rsconfig}'") .
+               'name = ' . (!strlen ($name) ? 'NULL,' : "'${name}', ") .
+               'vsconfig = ' . (!strlen ($vsconfig) ? 'NULL,' : "'${vsconfig}', ") .
+               'rsconfig = ' . (!strlen ($rsconfig) ? 'NULL' : "'${rsconfig}'") .
                " where id = ${vsid} limit 1";
        $result = $dbxlink->exec ($query);
        if ($result === NULL)
@@ -2745,16 +2745,16 @@ function getRSPoolsForObject ($object_id = 0)
 
 function commitCreateRSPool ($name = '', $vsconfig = '', $rsconfig = '', $taglist = array())
 {
-       if (empty ($name))
+       if (!strlen ($name))
                return __FUNCTION__ . ': invalid arguments';
        if (!useInsertBlade
        (
                'IPv4RSPool',
                array
                (
-                       'name' => (empty ($name) ? 'NULL' : "'${name}'"),
-                       'vsconfig' => (empty ($vsconfig) ? 'NULL' : "'${vsconfig}'"),
-                       'rsconfig' => (empty ($rsconfig) ? 'NULL' : "'${rsconfig}'")
+                       'name' => (!strlen ($name) ? 'NULL' : "'${name}'"),
+                       'vsconfig' => (!strlen ($vsconfig) ? 'NULL' : "'${vsconfig}'"),
+                       'rsconfig' => (!strlen ($rsconfig) ? 'NULL' : "'${rsconfig}'")
                )
        ))
                return __FUNCTION__ . ': SQL insertion failed';
@@ -2779,9 +2779,9 @@ function commitUpdateRSPool ($pool_id = 0, $name = '', $vsconfig = '', $rsconfig
        }
        global $dbxlink;
        $query = "update IPv4RSPool set " .
-               'name = ' . (empty ($name) ? 'NULL,' : "'${name}', ") .
-               'vsconfig = ' . (empty ($vsconfig) ? 'NULL,' : "'${vsconfig}', ") .
-               'rsconfig = ' . (empty ($rsconfig) ? 'NULL' : "'${rsconfig}'") .
+               'name = ' . (!strlen ($name) ? 'NULL,' : "'${name}', ") .
+               'vsconfig = ' . (!strlen ($vsconfig) ? 'NULL,' : "'${vsconfig}', ") .
+               'rsconfig = ' . (!strlen ($rsconfig) ? 'NULL' : "'${rsconfig}'") .
                " where id = ${pool_id} limit 1";
        $result = $dbxlink->exec ($query);
        if ($result === NULL)
@@ -2859,7 +2859,7 @@ function getSLBConfig ($object_id)
 
 function commitSetInService ($rs_id = 0, $inservice = '')
 {
-       if ($rs_id <= 0 or empty ($inservice))
+       if ($rs_id <= 0 or !strlen ($inservice))
        {
                showError ('Invalid args', __FUNCTION__);
                return NULL;
@@ -3010,7 +3010,7 @@ function deleteTagForEntity ($entity_realm, $entity_id, $tag_id)
 // Push a record into TagStorage unconditionally.
 function addTagForEntity ($realm = '', $entity_id, $tag_id)
 {
-       if (empty ($realm))
+       if (!strlen ($realm))
                return FALSE;
        return useInsertBlade
        (
@@ -3077,7 +3077,7 @@ function createIPv4Prefix ($range = '', $name = '', $is_bcast = FALSE, $taglist
        $ip = $rangeArray[0];
        $mask = $rangeArray[1];
 
-       if (empty ($ip) or empty ($mask))
+       if (!strlen ($ip) or !strlen ($mask))
                return "Invalid IPv4 prefix '${range}'";
        $ipL = ip2long($ip);
        $maskL = ip2long($mask);
@@ -3155,7 +3155,7 @@ function loadScript ($name)
 
 function saveScript ($name, $text)
 {
-       if (empty ($name))
+       if (!strlen ($name))
        {
                showError ('Invalid argument');
                return FALSE;
@@ -3481,7 +3481,7 @@ function getFileLinks ($file_id = 0)
                }
 
                // name needs to have some value for hrefs to work
-        if (empty($name))
+        if (!strlen ($name))
                        $name = sprintf("[Unnamed %s]", formatEntityName($row['entity_type']));
 
                $ret[$row['id']] = array(
@@ -3604,7 +3604,7 @@ function commitReplaceFile ($file_id = 0, $contents)
 
 function commitUpdateFile ($file_id = 0, $new_name = '', $new_type = '', $new_comment = '')
 {
-       if ($file_id <= 0 or empty ($new_name) or empty ($new_type))
+       if ($file_id <= 0 or !strlen ($new_name) or !strlen ($new_type))
        {
                showError ('Not all required args are present.', __FUNCTION__);
                return FALSE;
index c4470e831cdfae9f71796a9577ce48e559995199..963b41c1ec626fc2925377dbe549e13ca3628e92 100644 (file)
@@ -587,13 +587,13 @@ function findAllEndpoints ($object_id, $fallback = '')
 {
        $values = getAttrValues ($object_id);
        foreach ($values as $record)
-               if ($record['name'] == 'FQDN' && !empty ($record['value']))
+               if ($record['name'] == 'FQDN' && strlen ($record['value']))
                        return array ($record['value']);
        $regular = array();
        foreach (getObjectIPv4Allocations ($object_id) as $dottedquad => $alloc)
                if ($alloc['type'] == 'regular')
                        $regular[] = $dottedquad;
-       if (!count ($regular) && !empty ($fallback))
+       if (!count ($regular) && strlen ($fallback))
                return array ($fallback);
        return $regular;
 }
@@ -682,7 +682,7 @@ function getRSUforRackRow ($rowData = NULL)
 function lf_wrap ($str)
 {
        $ret = trim ($str, "\r\n");
-       if (!empty ($ret))
+       if (strlen ($ret))
                $ret .= "\n";
        return $ret;
 }
@@ -1383,8 +1383,8 @@ function buildLVSConfig ($object_id = 0)
        foreach ($lbconfig as $vs_id => $vsinfo)
        {
                $newconfig .=  "########################################################\n" .
-                       "# VS (id == ${vs_id}): " . (empty ($vsinfo['vs_name']) ? 'NO NAME' : $vsinfo['vs_name']) . "\n" .
-                       "# RS pool (id == ${vsinfo['pool_id']}): " . (empty ($vsinfo['pool_name']) ? 'ANONYMOUS' : $vsinfo['pool_name']) . "\n" .
+                       "# VS (id == ${vs_id}): " . (!strlen ($vsinfo['vs_name']) ? 'NO NAME' : $vsinfo['vs_name']) . "\n" .
+                       "# RS pool (id == ${vsinfo['pool_id']}): " . (!strlen ($vsinfo['pool_name']) ? 'ANONYMOUS' : $vsinfo['pool_name']) . "\n" .
                        "########################################################\n";
                # The order of inheritance is: VS -> LB -> pool [ -> RS ]
                $macros = array
@@ -1406,7 +1406,7 @@ function buildLVSConfig ($object_id = 0)
                );
                foreach ($vsinfo['rslist'] as $rs)
                {
-                       if (empty ($rs['rsport']))
+                       if (!strlen ($rs['rsport']))
                                $rs['rsport'] = $vsinfo['vport'];
                        $macros['%RSIP%'] = $rs['rsip'];
                        $macros['%RSPORT%'] = $rs['rsport'];
@@ -1517,7 +1517,7 @@ function IPv4NetworkCmp ($netA, $netB)
 // Modify the given tag tree so, that each level's items are sorted alphabetically.
 function sortTree (&$tree, $sortfunc = '')
 {
-       if (empty ($sortfunc))
+       if (!strlen ($sortfunc))
                return;
        $self = __FUNCTION__;
        usort ($tree, $sortfunc);
@@ -1529,7 +1529,7 @@ function sortTree (&$tree, $sortfunc = '')
 
 function iptree_fill (&$netdata)
 {
-       if (!isset ($netdata['kids']) or empty ($netdata['kids']))
+       if (!isset ($netdata['kids']) or !count ($netdata['kids']))
                return;
        // If we really have nested prefixes, they must fit into the tree.
        $worktree = array
@@ -1607,13 +1607,13 @@ function iptree_embed (&$node, $pfx)
 
 function treeApplyFunc (&$tree, $func = '', $stopfunc = '')
 {
-       if (empty ($func))
+       if (!strlen ($func))
                return;
        $self = __FUNCTION__;
        foreach (array_keys ($tree) as $key)
        {
                $func ($tree[$key]);
-               if (!empty ($stopfunc) and $stopfunc ($tree[$key]))
+               if (strlen ($stopfunc) and $stopfunc ($tree[$key]))
                        continue;
                $self ($tree[$key]['kids'], $func);
        }
@@ -1633,7 +1633,7 @@ function countOwnIPv4Addresses (&$node)
        $node['mask_bin_inv'] = binInvMaskFromDec ($node['mask']);
        $node['db_first'] = sprintf ('%u', 0x00000000 + $node['ip_bin'] & $node['mask_bin']);
        $node['db_last'] = sprintf ('%u', 0x00000000 + $node['ip_bin'] | ($node['mask_bin_inv']));
-       if (empty ($node['kids']))
+       if (!count ($node['kids']))
        {
                $toscan[] = array ('i32_first' => $node['db_first'], 'i32_last' => $node['db_last']);
                $node['addrt'] = binInvMaskFromDec ($node['mask']) + 1;
@@ -1658,7 +1658,7 @@ function nodeIsCollapsed ($node)
 function loadOwnIPv4Addresses (&$node)
 {
        $toscan = array();
-       if (empty ($node['kids']))
+       if (!count ($node['kids']))
                $toscan[] = array ('i32_first' => $node['db_first'], 'i32_last' => $node['db_last']);
        else
                foreach ($node['kids'] as $nested)
@@ -2027,7 +2027,7 @@ function getIPv4VSOptions ()
 {
        $ret = array();
        foreach (listCells ('ipv4vs') as $vsid => $vsinfo)
-               $ret[$vsid] = $vsinfo['dname'] . (empty ($vsinfo['name']) ? '' : " (${vsinfo['name']})");
+               $ret[$vsid] = $vsinfo['dname'] . (!strlen ($vsinfo['name']) ? '' : " (${vsinfo['name']})");
        return $ret;
 }
 
index 43efd2dc448ec16533293a59b24dac4ebf18f819..48ec836d4f82fa9022cb14ae2579b8426db521a7 100644 (file)
@@ -46,7 +46,7 @@ function queryGateway ($gwname, $questions)
        while (!feof($pipes[1]))
        {
                $a = fgets ($pipes[1]);
-               if (empty ($a))
+               if (!strlen ($a))
                        continue;
                // Somehow I got a space appended at the end. Kick it.
                $answers[] = trim ($a);
@@ -88,9 +88,9 @@ function getSwitchVLANs ($object_id = 0)
        $hwtype = $swtype = 'unknown';
        foreach (getAttrValues ($object_id, TRUE) as $record)
        {
-               if ($record['name'] == 'SW type' && !empty ($record['value']))
+               if ($record['name'] == 'SW type' && strlen ($record['value']))
                        $swtype = str_replace (' ', '+', $record['value']);
-               if ($record['name'] == 'HW type' && !empty ($record['value']))
+               if ($record['name'] == 'HW type' && strlen ($record['value']))
                        $hwtype = str_replace (' ', '+', $record['value']);
        }
        $endpoint = str_replace (' ', '+', $endpoints[0]);
@@ -146,7 +146,7 @@ function getSwitchVLANs ($object_id = 0)
        foreach (explode (';', substr ($data[3], strlen ('OK!'))) as $pair)
        {
                list ($macaddr, $pair2) = explode ('=', $pair);
-               if (empty ($pair2))
+               if (!strlen ($pair2))
                        continue;
                list ($vlanid, $ifname) = explode ('@', $pair2);
                $maclist[$ifname][$vlanid][] = $macaddr;
@@ -168,9 +168,9 @@ function setSwitchVLANs ($object_id = 0, $setcmd)
        $hwtype = $swtype = 'unknown';
        foreach (getAttrValues ($object_id, TRUE) as $record)
        {
-               if ($record['name'] == 'SW type' && !empty ($record['value']))
+               if ($record['name'] == 'SW type' && strlen ($record['value']))
                        $swtype = strtr ($record['value'], ' ', '+');
-               if ($record['name'] == 'HW type' && !empty ($record['value']))
+               if ($record['name'] == 'HW type' && strlen ($record['value']))
                        $hwtype = strtr ($record['value'], ' ', '+');
        }
        $endpoint = str_replace (' ', '+', $endpoints[0]);
@@ -266,7 +266,7 @@ function gwRecvFile ($endpoint, $handlername, &$output)
 function gwSendFileToObject ($object_id = 0, $handlername, $filetext = '')
 {
        global $remote_username;
-       if ($object_id <= 0 or empty ($handlername))
+       if ($object_id <= 0 or !strlen ($handlername))
                return oneLiner (160); // invalid arguments
        $objectInfo = spotEntity ('object', $object_id);
        $endpoints = findAllEndpoints ($object_id, $objectInfo['name']);
@@ -281,7 +281,7 @@ function gwSendFileToObject ($object_id = 0, $handlername, $filetext = '')
 function gwRecvFileFromObject ($object_id = 0, $handlername, &$output)
 {
        global $remote_username;
-       if ($object_id <= 0 or empty ($handlername))
+       if ($object_id <= 0 or !strlen ($handlername))
                return oneLiner (160); // invalid arguments
        $objectInfo = spotEntity ('object', $object_id);
        $endpoints = findAllEndpoints ($object_id, $objectInfo['name']);
index 74b8165fbaffa492fd76889141dd5dc58d721ff3..a6ceaf76bac2ba70f2ee847850a466f4b0a442b4 100644 (file)
@@ -112,7 +112,7 @@ if (!count ($configCache))
 
 require_once 'inc/code.php'; // for getRackCode()
 $rackCodeCache = loadScript ('RackCodeCache');
-if ($rackCodeCache == NULL or empty ($rackCodeCache))
+if ($rackCodeCache == NULL or !strlen ($rackCodeCache))
 {
        $rackCode = getRackCode (loadScript ('RackCode'));
        saveScript ('RackCodeCache', base64_encode (serialize ($rackCode)));
index 0be92b8d37eecf1e4c285e1bd4192b94440867cc..08d42db78f9168237d65db72cbf922946a5f37ce 100644 (file)
@@ -420,7 +420,7 @@ function showError ($info = '', $location = 'N/A')
        elseif ($location != 'N/A')
                $location = $location . '()';
        echo "<div class=msg_error>An error has occured in [${location}]. ";
-       if (empty ($info))
+       if (!strlen ($info))
                echo 'No additional information is available.';
        else
                echo "Additional information:<br><p>\n<pre>\n${info}\n</pre></p>";
@@ -446,7 +446,7 @@ function assertUIntArg ($argname, $caller = 'N/A', $allow_zero = FALSE)
                showError ("Parameter '${argname}' is less than zero (calling function is [${caller}]).", __FUNCTION__);
                die();
        }
-       if (!$allow_zero and $_REQUEST[$argname] == 0)
+       if (!$allow_zero and $_REQUEST[$argname] === 0)
        {
                showError ("Parameter '${argname}' is equal to zero (calling function is [${caller}]).", __FUNCTION__);
                die();
@@ -467,7 +467,7 @@ function assertStringArg ($argname, $caller = 'N/A', $ok_if_empty = FALSE)
                showError ("Parameter '${argname}' is not a string (calling function is [${caller}]).", __FUNCTION__);
                die();
        }
-       if (!$ok_if_empty and empty ($_REQUEST[$argname]))
+       if (!$ok_if_empty and !strlen ($_REQUEST[$argname]))
        {
                showError ("Parameter '${argname}' is an empty string (calling function is [${caller}]).", __FUNCTION__);
                die();
@@ -486,7 +486,7 @@ function assertBoolArg ($argname, $caller = 'N/A', $ok_if_empty = FALSE)
                showError ("Parameter '${argname}' is not a string (calling function is [${caller}]).", __FUNCTION__);
                die();
        }
-       if (!$ok_if_empty and empty ($_REQUEST[$argname]))
+       if (!$ok_if_empty and !strlen ($_REQUEST[$argname]))
        {
                showError ("Parameter '${argname}' is an empty string (calling function is [${caller}]).", __FUNCTION__);
                die();
@@ -496,7 +496,7 @@ function assertBoolArg ($argname, $caller = 'N/A', $ok_if_empty = FALSE)
 function assertIPv4Arg ($argname, $caller = 'N/A', $ok_if_empty = FALSE)
 {
        assertStringArg ($argname, $caller, $ok_if_empty);
-       if (!empty ($_REQUEST[$argname]) and long2ip (ip2long ($_REQUEST[$argname])) !== $_REQUEST[$argname])
+       if (strlen ($_REQUEST[$argname]) and long2ip (ip2long ($_REQUEST[$argname])) !== $_REQUEST[$argname])
        {
                showError ("IPv4 address validation failed for value '" . $_REQUEST[$argname] . "' (calling function is [${caller}]).", __FUNCTION__);
                die();
@@ -554,12 +554,12 @@ function renderRack ($rack_id, $hl_obj_id = 0)
                        {
                                case 'T':
                                        $objectData = spotEntity ('object', $rackData[$i][$locidx]['object_id']);
-                                       if (!empty ($objectData['asset_no']))
+                                       if (strlen ($objectData['asset_no']))
                                                $prefix = "<div title='${objectData['asset_no']}";
                                        else
                                                $prefix = "<div title='no asset tag";
                                        // Don't tell about label, if it matches common name.
-                                       if ($objectData['name'] != $objectData['label'] and !empty ($objectData['label']))
+                                       if ($objectData['name'] != $objectData['label'] and strlen ($objectData['label']))
                                                $suffix = ", visible label is \"${objectData['label']}\"'>";
                                        else
                                                $suffix = "'>";
@@ -655,7 +655,7 @@ function renderEditObjectForm ($object_id)
                {
                        echo "<input type=hidden name=${i}_attr_id value=${record['id']}>";
                        echo '<tr><td>';
-                       if (!empty ($record['value']))
+                       if (strlen ($record['value']))
                        {
                                echo "<a href='".makeHrefProcess(array('op'=>'clearSticker', 'object_id'=>$object_id, 'attr_id'=>$record['id']))."'>";
                                printImageHREF ('clear', 'Clear value');
@@ -783,7 +783,7 @@ function renderRackInfoPortlet ($rackData)
        echo count ($rackData['mountedObjects']);
        echo "</td></tr>\n";
        printTagTRs ($rackData, makeHref(array('page'=>'rackspace', 'tab'=>'default'))."&");
-       if (!empty ($rackData['comment']))
+       if (strlen ($rackData['comment']))
                echo "<tr><th width='50%' class=tdright>Comment:</th><td class=tdleft>${rackData['comment']}</td></tr>\n";
        echo '</table>';
        finishPortlet();
@@ -864,7 +864,7 @@ function renderRackObject ($object_id)
        echo "<tr><td class=pcleft>";
        startPortlet ('summary');
        echo "<table border=0 cellspacing=0 cellpadding=3 width='100%'>\n";
-       if (!empty ($info['name']))
+       if (strlen ($info['name']))
                echo "<tr><th width='50%' class=tdright>Common name:</th><td class=tdleft>${info['name']}</td></tr>\n";
        // FIXME: don't call spotEntity() each time, do it once in the beginning.
        elseif (considerConfiguredConstraint (spotEntity ('object', $object_id), 'NAMEWARN_LISTSRC'))
@@ -876,19 +876,19 @@ function renderRackObject ($object_id)
                'cfe' => '{$typeid_' . $info['objtype_id'] . '}'
        ));
        echo "'>${info['objtype_name']}</a></td></tr>\n";
-       if (!empty ($info['asset_no']))
+       if (strlen ($info['asset_no']))
                echo "<tr><th width='50%' class=tdright>Asset tag:</th><td class=tdleft>${info['asset_no']}</td></tr>\n";
        // FIXME: ditto
        elseif (considerConfiguredConstraint (spotEntity ('object', $object_id), 'ASSETWARN_LISTSRC'))
                echo "<tr><td colspan=2 class=msg_error>Asset tag is missing.</td></tr>\n";
-       if (!empty ($info['label']))
+       if (strlen ($info['label']))
                echo "<tr><th width='50%' class=tdright>Visible label:</th><td class=tdleft>${info['label']}</td></tr>\n";
-       if (!empty ($info['barcode']))
+       if (strlen ($info['barcode']))
                echo "<tr><th width='50%' class=tdright>Barcode:</th><td class=tdleft>${info['barcode']}</td></tr>\n";
        if ($info['has_problems'] == 'yes')
                echo "<tr><td colspan=2 class=msg_error>Has problems</td></tr>\n";
        foreach (getAttrValues ($object_id, TRUE) as $record)
-               if (!empty ($record['value']))
+               if (strlen ($record['value']))
                        echo "<tr><th width='50%' class=sticker>${record['name']}:</th><td class=sticker>${record['a_value']}</td></tr>\n";
        printTagTRs
        (
@@ -907,7 +907,7 @@ function renderRackObject ($object_id)
        echo "</table><br>\n";
        finishPortlet();
 
-       if (!empty ($info['comment']))
+       if (strlen ($info['comment']))
        {
                startPortlet ('Comment');
                echo '<div class=commentblock>' . string_insert_hrefs ($info['comment']) . '</div>';
@@ -944,7 +944,7 @@ function renderRackObject ($object_id)
                                        echo "<td><a href='".makeHref(array('page'=>'object', 'object_id'=>$port['remote_object_id'], 'hl_port_id'=>$port['remote_id']))."'>${port['remote_object_name']}</a></td>";
                                        echo "<td>${port['remote_name']}</td>";
                                }
-                               elseif (!empty ($port['reservation_comment']))
+                               elseif (strlen ($port['reservation_comment']))
                                {
                                        echo "<td><b>Reserved;</b></td>";
                                        echo "<td>${port['reservation_comment']}</td>";
@@ -992,7 +992,7 @@ function renderRackObject ($object_id)
                        if (getConfigVar ('EXT_IPV4_VIEW') != 'yes')
                                echo '<small>/' . (NULL === $netid ? '??' : $netinfo['mask']) . '</small>';
                        echo '&nbsp;' . $aac[$alloc['type']];
-                       if (!empty ($alloc['addrinfo']['name']))
+                       if (strlen ($alloc['addrinfo']['name']))
                                echo ' (' . niftyString ($alloc['addrinfo']['name']) . ')';
                        echo '</td>';
                        if (getConfigVar ('EXT_IPV4_VIEW') == 'yes')
@@ -1028,7 +1028,7 @@ function renderRackObject ($object_id)
                                if ($allocpeer['object_id'] == $object_id)
                                        continue;
                                echo $prefix . "<a href='".makeHref(array('page'=>'object', 'object_id'=>$allocpeer['object_id']))."'>";
-                               if (!empty ($allocpeer['osif']))
+                               if (strlen ($allocpeer['osif']))
                                        echo $allocpeer['osif'] . '@';
                                echo $allocpeer['object_name'] . '</a>';
                                $prefix = '; ';
@@ -1069,7 +1069,7 @@ function renderRackObject ($object_id)
                                if (count ($address['allocs']))
                                        foreach($address['allocs'] as $bond)
                                                echo "<a href='".makeHref(array('page'=>'object', 'tab'=>'default', 'object_id'=>$bond['object_id']))."'>${bond['object_name']}(${bond['name']})</a> ";
-                               elseif (!empty ($pf['remote_addr_name']))
+                               elseif (strlen ($pf['remote_addr_name']))
                                        echo '(' . $pf['remote_addr_name'] . ')';
                                echo "</td><td class='description'>${pf['description']}</td></tr>";
                        }
@@ -1222,7 +1222,7 @@ function renderPortsForObject ($object_id)
                        printImageHREF ('cut', 'Unlink this port');
                        echo "</a></td>";
                }
-               elseif (!empty ($port['reservation_comment']))
+               elseif (strlen ($port['reservation_comment']))
                {
                        echo "<td><b>Reserved;</b></td>";
                        echo "<td><input type=text name=reservation_comment value='${port['reservation_comment']}'></td>";
@@ -1317,7 +1317,7 @@ function renderIPv4ForObject ($object_id)
                        echo $dottedquad;
                if (getConfigVar ('EXT_IPV4_VIEW') != 'yes')
                        echo '<small>/' . (NULL === $netid ? '??' : $netinfo['mask']) . '</small>';
-               if (!empty ($alloc['addrinfo']['name']))
+               if (strlen ($alloc['addrinfo']['name']))
                        echo ' (' . niftyString ($alloc['addrinfo']['name']) . ')';
                echo '</td>';
                // FIXME: this a copy-and-paste from renderRackObject()
@@ -1355,7 +1355,7 @@ function renderIPv4ForObject ($object_id)
                        if ($allocpeer['object_id'] == $object_id)
                                continue;
                        echo $prefix . "<a href='".makeHref(array('page'=>'object', 'object_id'=>$allocpeer['object_id']))."'>";
-                       if (!empty ($allocpeer['osif']))
+                       if (strlen ($allocpeer['osif']))
                                echo $allocpeer['osif'] . '@';
                        echo $allocpeer['object_name'] . '</a>';
                        $prefix = '; ';
@@ -2431,7 +2431,7 @@ function renderIPv4Network ($id)
                {
                        echo $delim . $aac2[$ref['type']];
                        echo "<a href='".makeHref(array('page'=>'object', 'object_id'=>$ref['object_id'], 'hl_ipv4_addr'=>$addr['ip']))."'>";
-                       echo $ref['name'] . (empty ($ref['name']) ? '' : '@');
+                       echo $ref['name'] . (!strlen ($ref['name']) ? '' : '@');
                        echo "${ref['object_name']}</a>";
                        $delim = '; ';
                }
@@ -2489,7 +2489,7 @@ function renderIPv4Address ($dottedquad)
        $address = getIPv4Address ($dottedquad);
        echo "<table border=0 class=objectview cellspacing=0 cellpadding=0>";
        echo "<tr><td colspan=2 align=center><h1>${dottedquad}</h1></td></tr>\n";
-       if (!empty ($address['name']))
+       if (strlen ($address['name']))
                echo "<tr><td colspan=2 align=center><h2>${address['name']}</h2></td></tr>\n";
 
        echo "<tr><td class=pcleft>";
@@ -2605,7 +2605,7 @@ function renderIPv4AddressProperties ($dottedquad)
        echo "></tr><tr><td class=tdleft>";
        printImageHREF ('SAVE', 'Save changes', TRUE);
        echo "</td></form><td class=tdright>";
-       if (empty ($address['name']) and $address['reserved'] == 'no')
+       if (!strlen ($address['name']) and $address['reserved'] == 'no')
                printImageHREF ('CLEAR gray');
        else
        {
@@ -2681,8 +2681,8 @@ function renderNATv4ForObject ($object_id)
 
                foreach ($alloclist as $dottedquad => $alloc)
                {
-                       $name = empty ($alloc['addrinfo']['name']) ? '' : (' (' . niftyString ($alloc['addrinfo']['name']) . ')');
-                       $osif = empty ($alloc['osif']) ? '' : ($alloc['osif'] . ': ');
+                       $name = !strlen ($alloc['addrinfo']['name']) ? '' : (' (' . niftyString ($alloc['addrinfo']['name']) . ')');
+                       $osif = !strlen ($alloc['osif']) ? '' : ($alloc['osif'] . ': ');
                        echo "<option value='${dottedquad}'>${osif}${dottedquad}${name}</option>";
                }
 
@@ -2731,7 +2731,7 @@ function renderNATv4ForObject ($object_id)
                printImageHREF ('delete', 'Delete NAT rule');
                echo "</a></td>";
                echo "<td>${pf['proto']}/${osif}<a href='".makeHref(array('page'=>'ipaddress', 'tab'=>'default', 'ip'=>$pf['localip']))."'>${pf['localip']}</a>:${pf['localport']}";
-               if (!empty ($pf['local_addr_name']))
+               if (strlen ($pf['local_addr_name']))
                        echo ' (' . $pf['local_addr_name'] . ')';
                echo "</td>";
                echo "<td><a href='".makeHref(array('page'=>'ipaddress', 'tab'=>'default', 'ip'=>$pf['remoteip']))."'>${pf['remoteip']}</a>:${pf['remoteport']}</td>";
@@ -2742,7 +2742,7 @@ function renderNATv4ForObject ($object_id)
                if (count ($address['allocs']))
                        foreach ($address['allocs'] as $bond)
                                echo "<a href='".makeHref(array('page'=>'object', 'tab'=>'default', 'object_id'=>$bond['object_id']))."'>${bond['object_name']}(${bond['name']})</a> ";
-               elseif (!empty ($pf['remote_addr_name']))
+               elseif (strlen ($pf['remote_addr_name']))
                        echo '(' . $pf['remote_addr_name'] . ')';
                printOpFormIntro
                (
@@ -2857,7 +2857,7 @@ function renderSearchResults ()
 {
        global $root;
        $terms = trim ($_REQUEST['q']);
-       if (empty ($terms))
+       if (!strlen ($terms))
        {
                showError ('Search string cannot be empty.', __FUNCTION__);
                return;
@@ -3655,7 +3655,7 @@ function getImageHREF ($tag, $title = '', $do_input = FALSE, $tabindex = 0)
                        "src='${root}${img['path']}' " .
                        "border=0 " .
                        ($tabindex ? "tabindex=${tabindex}" : '') .
-                       (empty ($title) ? '' : " title='${title}'") . // JT: Add title to input hrefs too
+                       (!strlen ($title) ? '' : " title='${title}'") . // JT: Add title to input hrefs too
                        ">";
        else
                return
@@ -3664,7 +3664,7 @@ function getImageHREF ($tag, $title = '', $do_input = FALSE, $tabindex = 0)
                        "width=${img['width']} " .
                        "height=${img['height']} " .
                        "border=0 " .
-                       (empty ($title) ? '' : "title='${title}'") .
+                       (!strlen ($title) ? '' : "title='${title}'") .
                        ">";
 }
 
@@ -4031,25 +4031,25 @@ function renderVirtualService ($vsid)
        $vsinfo = spotEntity ('ipv4vs', $vsid);
        amplifyCell ($vsinfo);
        echo '<table border=0 class=objectview cellspacing=0 cellpadding=0>';
-       if (!empty ($vsinfo['name']))
+       if (strlen ($vsinfo['name']))
                echo "<tr><td colspan=2 align=center><h1>${vsinfo['name']}</h1></td></tr>\n";
        echo '<tr>';
 
        echo '<td class=pcleft>';
        startPortlet ('Frontend');
        echo "<table border=0 cellspacing=0 cellpadding=3 width='100%'>\n";
-       if (!empty ($vsinfo['name']))
+       if (strlen ($vsinfo['name']))
                echo "<tr><th width='50%' class=tdright>Name:</th><td class=tdleft>${vsinfo['name']}</td></tr>\n";
        echo "<tr><th width='50%' class=tdright>Protocol:</th><td class=tdleft>${vsinfo['proto']}</td></tr>\n";
        echo "<tr><th width='50%' class=tdright>Virtual IP address:</th><td class=tdleft><a href='".makeHref(array('page'=>'ipaddress', 'tab'=>'default', 'ip'=>$vsinfo['vip']))."'>${vsinfo['vip']}</a></td></tr>\n";
        echo "<tr><th width='50%' class=tdright>Virtual port:</th><td class=tdleft>${vsinfo['vport']}</td></tr>\n";
        printTagTRs ($vsinfo, makeHref(array('page'=>'ipv4vslist', 'tab'=>'default'))."&");
-       if (!empty ($vsinfo['vsconfig']))
+       if (strlen ($vsinfo['vsconfig']))
        {
                echo "<tr><th class=slbconf>VS configuration:</th><td>&nbsp;</td></tr>";
                echo "<tr><td colspan=2 class='dashed slbconf'>${vsinfo['vsconfig']}</td></tr>\n";
        }
-       if (!empty ($vsinfo['rsconfig']))
+       if (strlen ($vsinfo['rsconfig']))
        {
                echo "<tr><th class=slbconf>RS configuration:</th><td class=tdleft>&nbsp;</td></tr>\n";
                echo "<tr><td colspan=2 class='dashed slbconf'>${vsinfo['rsconfig']}</td></tr>\n";
@@ -4071,9 +4071,9 @@ function renderVirtualService ($vsid)
                echo "<tr><td colspan=2>";
                renderCell (spotEntity ('ipv4rspool', $pool_id));
                echo "</td></tr>";
-               if (!empty ($poolInfo['vsconfig']))
+               if (strlen ($poolInfo['vsconfig']))
                        echo "<tr><th>VS config</th><td class='dashed slbconf'>${poolInfo['vsconfig']}</td></tr>";
-               if (!empty ($poolInfo['rsconfig']))
+               if (strlen ($poolInfo['rsconfig']))
                        echo "<tr><th>RS config</th><td class='dashed slbconf'>${poolInfo['rsconfig']}</td></tr>";
                echo '</table>';
                echo '</td><td>';
@@ -4088,9 +4088,9 @@ function renderVirtualService ($vsid)
                                echo "<tr><td colspan=2>";
                                renderLBCell ($object_id);
                                echo '</td></tr>';
-                               if (!empty ($lbInfo['vsconfig']))
+                               if (strlen ($lbInfo['vsconfig']))
                                        echo "<tr><th>VS config</th><td class='dashed slbconf'>${lbInfo['vsconfig']}</td></tr>";
-                               if (!empty ($lbInfo['rsconfig']))
+                               if (strlen ($lbInfo['rsconfig']))
                                        echo "<tr><th>RS config</th><td class='dashed slbconf'>${lbInfo['rsconfig']}</td></tr>";
                        }
                        echo '</table>';
@@ -4110,7 +4110,7 @@ function renderProgressBar ($percentage = 0, $theme = '')
        global $root;
        $done = ((int) ($percentage * 100));
        echo "<img width=100 height=10 border=0 title='${done}%' src='${root}render_image.php?img=progressbar&done=${done}";
-       echo (empty ($theme) ? '' : "&theme=${theme}") . "'>";
+       echo (!strlen ($theme) ? '' : "&theme=${theme}") . "'>";
 }
 
 function renderRSPoolServerForm ($pool_id)
@@ -4295,23 +4295,23 @@ function renderRSPool ($pool_id)
        amplifyCell ($poolInfo);
 
        echo "<table border=0 class=objectview cellspacing=0 cellpadding=0>";
-       if (!empty ($poolInfo['name']))
+       if (strlen ($poolInfo['name']))
                echo "<tr><td colspan=2 align=center><h1>{$poolInfo['name']}</h1></td></tr>";
        echo "<tr><td class=pcleft>\n";
 
        startPortlet ('Summary');
        echo "<table border=0 cellspacing=0 cellpadding=3 width='100%'>\n";
-       if (!empty ($poolInfo['name']))
+       if (strlen ($poolInfo['name']))
                echo "<tr><th width='50%' class=tdright>Pool name:</th><td class=tdleft>${poolInfo['name']}</td></tr>\n";
        echo "<tr><th width='50%' class=tdright>Real servers:</th><td class=tdleft>" . count ($poolInfo['rslist']) . "</td></tr>\n";
        echo "<tr><th width='50%' class=tdright>Load balancers:</th><td class=tdleft>" . count ($poolInfo['lblist']) . "</td></tr>\n";
        printTagTRs ($poolInfo, makeHref(array('page'=>'ipv4rsplist', 'tab'=>'default'))."&");
-       if (!empty ($poolInfo['vsconfig']))
+       if (strlen ($poolInfo['vsconfig']))
        {
                echo "<tr><th width='50%' class=tdright>VS configuration:</th><td>&nbsp;</td></tr>\n";
                echo "<tr><td colspan=2 class='dashed slbconf'>${poolInfo['vsconfig']}</td></tr>\n";
        }
-       if (!empty ($poolInfo['rsconfig']))
+       if (strlen ($poolInfo['rsconfig']))
        {
                echo "<tr><th width='50%' class=tdright>RS configuration:</th><td>&nbsp;</td></tr>\n";
                echo "<tr><td colspan=2 class='dashed slbconf'>${poolInfo['rsconfig']}</td></tr>\n";
@@ -4498,7 +4498,7 @@ function renderRealServerList ()
                        $last_pool_id = $rsinfo['rspool_id'];
                }
                echo "<tr valign=top class=row_${order}><td><a href='".makeHref(array('page'=>'ipv4rspool', 'pool_id'=>$rsinfo['rspool_id']))."'>";
-               echo empty ($pool_list[$rsinfo['rspool_id']]['name']) ? 'ANONYMOUS' : $pool_list[$rsinfo['rspool_id']]['name'];
+               echo !strlen ($pool_list[$rsinfo['rspool_id']]['name']) ? 'ANONYMOUS' : $pool_list[$rsinfo['rspool_id']]['name'];
                echo '</a></td><td align=center>';
                if ($rsinfo['inservice'] == 'yes')
                        printImageHREF ('inservice', 'in service');
@@ -4609,13 +4609,13 @@ function renderLivePTR ($id)
                $print_cbox = FALSE;
                if ($addr['name'] == $ptrname)
                {
-                       if (!empty ($ptrname))
+                       if (strlen ($ptrname))
                        {
                                echo ' class=trok';
                                $cnt_match++;
                        }
                }
-               elseif (empty ($addr['name']) or empty ($ptrname))
+               elseif (!strlen ($addr['name']) or !strlen ($ptrname))
                {
                        echo ' class=trwarning';
                        $print_cbox = TRUE;
@@ -4628,7 +4628,7 @@ function renderLivePTR ($id)
                        $cnt_mismatch++;
                }
                echo "><td class='tdleft";
-               if (!empty ($range['addrlist'][$ip]['class']))
+               if (strlen ($range['addrlist'][$ip]['class']))
                        echo ' ' . $range['addrlist'][$ip]['class'];
                echo "'><a href='".makeHref(array('page'=>'ipaddress', 'ip'=>$straddr))."'>${straddr}</a></td>";
                echo "<td class=tdleft>${addr['name']}</td><td class=tdleft>${ptrname}</td><td>";
@@ -5255,7 +5255,7 @@ function renderFile ($file_id)
        printf("<td class=tdleft>%s</td></tr>", formatTimestamp($file['atime']));
 
        printTagTRs ($file, makeHref(array('page'=>'files', 'tab'=>'default'))."&");
-       if (!empty ($file['comment']))
+       if (strlen ($file['comment']))
        {
                echo '<tr><th class=slbconf>Comment:</th><td>&nbsp;</td></tr>';
                echo '<tr><td colspan=2 class="dashed slbconf">' . string_insert_hrefs (htmlspecialchars ($file['comment'])) . '</td></tr>';
@@ -5497,7 +5497,7 @@ function printOpFormIntro ($opname, $extra = array(), $upload = FALSE)
 function niftyString ($string, $maxlen = 30)
 {
        $cutind = '&hellip;'; // length is 1
-       if (empty ($string))
+       if (!strlen ($string))
                return '&nbsp;';
        if (mb_strlen ($string) > $maxlen)
                return "<span title='" . htmlspecialchars ($string, ENT_QUOTES, 'UTF-8') . "'>" .
@@ -5534,12 +5534,12 @@ function printIPv4NetInfoTDs ($netinfo, $tdclass = 'tdleft', $indent = 0, $symbo
                $symbol = '';
        }
        echo "<td class='${tdclass}' style='padding-left: " . ($indent * 16) . "px;'>";
-       if (!empty ($symbol))
+       if (strlen ($symbol))
        {
-               if (!empty ($symbolurl))
+               if (strlen ($symbolurl))
                        echo "<a href='${symbolurl}'>";
                printImageHREF ($symbol, $symbolurl);
-               if (!empty ($symbolurl))
+               if (strlen ($symbolurl))
                        echo '</a>';
        }
        if (isset ($netinfo['id']))
@@ -5621,7 +5621,7 @@ function renderCell ($cell)
        case 'ipv4rspool':
                echo "<table class=slbcell><tr><td>";
                echo "<a href='${root}?page=ipv4rspool&pool_id=${cell['id']}'>";
-               echo empty ($cell['name']) ? "ANONYMOUS pool [${cell['id']}]" : niftyString ($cell['name']);
+               echo !strlen ($cell['name']) ? "ANONYMOUS pool [${cell['id']}]" : niftyString ($cell['name']);
                echo "</a></td></tr><tr><td>";
                printImageHREF ('RS pool');
                echo "</td></tr><tr><td>";
@@ -5679,7 +5679,7 @@ function renderRouterCell ($dottedquad, $ifname, $object_id, $object_dname)
 {
        global $root;
        echo "<table class=slbcell><tr><td rowspan=3>${dottedquad}";
-       if (!empty ($ifname))
+       if (strlen ($ifname))
                echo '@' . $ifname;
        echo "</td>";
        echo "<td><a href='${root}?page=object&object_id=${object_id}&hl_ipv4_addr=${dottedquad}'><strong>${object_dname}</strong></a></td>";
@@ -5879,7 +5879,7 @@ function dynamic_title_decoder ($path_position)
                $poolInfo = spotEntity ('ipv4rspool', $_REQUEST['pool_id']);
                return array
                (
-                       'name' => empty ($poolInfo['name']) ? 'ANONYMOUS' : $poolInfo['name'],
+                       'name' => !strlen ($poolInfo['name']) ? 'ANONYMOUS' : $poolInfo['name'],
                        'params' => array ('pool_id' => $_REQUEST['pool_id'])
                );
        case 'ipv4vs':
index 2ab4bcace5f31696ca619f2acc7bcf40e1faafd4..e59aca67d427c3ccb21614b450b5cd69f56b45eb 100644 (file)
@@ -59,7 +59,7 @@ function addPortForwarding ()
        assertStringArg ('proto', __FUNCTION__);
        assertStringArg ('description', __FUNCTION__, TRUE);
        $remoteport = isset ($_REQUEST['remoteport']) ? $_REQUEST['remoteport'] : '';
-       if (empty ($remoteport))
+       if (!strlen ($remoteport))
                $remoteport = $_REQUEST['localport'];
 
        $error = newPortForwarding
@@ -140,7 +140,7 @@ function addPortForObject ()
 {
        assertUIntArg ('object_id', __FUNCTION__);
        assertStringArg ('port_name', __FUNCTION__, TRUE);
-       if (empty ($_REQUEST['port_name']))
+       if (!strlen ($_REQUEST['port_name']))
                return buildRedirectURL (__FUNCTION__, 'ERR1');
        $error = commitAddPort ($_REQUEST['object_id'], $_REQUEST['port_name'], $_REQUEST['port_type_id'], $_REQUEST['port_label'], $_REQUEST['port_l2address']);
        if ($error != '')
@@ -158,10 +158,10 @@ function editPortForObject ()
        assertUIntArg ('port_type_id', __FUNCTION__);
        // tolerate empty value now to produce custom informative message later
        assertStringArg ('name', __FUNCTION__, TRUE);
-       if (empty ($_REQUEST['name']))
+       if (!strlen ($_REQUEST['name']))
                return buildRedirectURL (__FUNCTION__, 'ERR1');
 
-       if (isset ($_REQUEST['reservation_comment']) and !empty ($_REQUEST['reservation_comment']))
+       if (isset ($_REQUEST['reservation_comment']) and strlen ($_REQUEST['reservation_comment']))
                $port_rc = '"' . $_REQUEST['reservation_comment'] . '"';
        else
                $port_rc = 'NULL';
@@ -236,7 +236,7 @@ function addMultiPorts ()
        {
                $parts = explode ('\r', $line);
                reset ($parts);
-               if (empty ($parts[0]))
+               if (!strlen ($parts[0]))
                        continue;
                else
                        $lines2[] = rtrim ($parts[0]);
@@ -292,7 +292,7 @@ http://www.cisco.com/en/US/products/hw/routers/ps274/products_tech_note09186a008
                                break;
                        case 'ssv1':
                                $words = explode (' ', $line);
-                               if (empty ($words[0]) or empty ($words[1]))
+                               if (!strlen ($words[0]) or !strlen ($words[1]))
                                        continue;
                                $ports[] = array
                                (
@@ -380,7 +380,7 @@ function addIPv4Allocation ()
        if ($error != '')
                return buildRedirectURL (__FUNCTION__, 'ERR2', array ($error));
        $address = getIPv4Address ($ip);
-       if ($address['reserved'] == 'yes' or !empty ($address['name']))
+       if ($address['reserved'] == 'yes' or strlen ($address['name']))
        {
                $release = getConfigVar ('IPV4_AUTO_RELEASE');
                if ($release >= 1)
@@ -774,7 +774,7 @@ function updateObject ()
                $attr_id = $_REQUEST["${i}_attr_id"];
 
                // Field is empty, delete attribute and move on.
-               if (empty($_REQUEST["${i}_value"]))
+               if (!strlen ($_REQUEST["${i}_value"]))
                {
                        commitResetAttrValue ($_REQUEST['object_id'], $attr_id);
                        continue;
@@ -798,7 +798,7 @@ function updateObject ()
                                showError ('Internal structure error', __FUNCTION__);
                                die;
                }
-               if ($value == $oldvalue)
+               if ($value === $oldvalue) // ('' == 0), but ('' !== 0)
                        continue;
 
                // Note if the queries succeed or not, it determines which page they see.
@@ -860,7 +860,7 @@ function addLotOfObjects()
        assertUIntArg ('global_type_id', __FUNCTION__, TRUE);
        assertStringArg ('namelist', __FUNCTION__, TRUE);
        $global_type_id = $_REQUEST['global_type_id'];
-       if ($global_type_id == 0 or empty ($_REQUEST['namelist']))
+       if ($global_type_id == 0 or !strlen ($_REQUEST['namelist']))
                $log = mergeLogs ($log, oneLiner (186));
        else
        {
@@ -871,7 +871,7 @@ function addLotOfObjects()
                {
                        $parts = explode ('\r', $line);
                        reset ($parts);
-                       if (empty ($parts[0]))
+                       if (!strlen ($parts[0]))
                                continue;
                        else
                                $names2[] = rtrim ($parts[0]);
@@ -920,7 +920,6 @@ $msgcode['updateUI']['ERR'] = 125;
 function updateUI ()
 {
        assertUIntArg ('num_vars', __FUNCTION__);
-       $error = '';
 
        for ($i = 0; $i < $_REQUEST['num_vars']; $i++)
        {
@@ -930,18 +929,14 @@ function updateUI ()
                $varvalue = $_REQUEST["${i}_varvalue"];
 
                // If form value = value in DB, don't bother updating DB
-               if ($varvalue == getConfigVar ($varname))
+               if ($varvalue === getConfigVar ($varname))
                        continue;
 
                // Note if the queries succeed or not, it determines which page they see.
                $error = setConfigVar ($varname, $varvalue, TRUE);
-               if (!empty ($error))
-                       break;
+               if (strlen ($error))
+                       return buildRedirectURL (__FUNCTION__, 'ERR', array ($error));
        }
-
-       if ($error != '')
-               return buildRedirectURL (__FUNCTION__, 'ERR', array ($error));
-
        return buildRedirectURL (__FUNCTION__, 'OK');
 }
 
@@ -1028,7 +1023,7 @@ function addRealServers ()
        // Keep in mind, that the text will have HTML entities (namely '>') escaped.
        foreach (explode ('\n', $rawtext) as $line)
        {
-               if (empty ($line))
+               if (!strlen ($line))
                        continue;
                $match = array ();
                switch ($_REQUEST['format'])
@@ -1639,7 +1634,7 @@ function addRack ()
                {
                        $parts = explode ('\r', $line);
                        reset ($parts);
-                       if (empty ($parts[0]))
+                       if (!strlen ($parts[0]))
                                continue;
                        else
                                $names2[] = rtrim ($parts[0]);
index 92bfeadf051477805b4fdc5353ccaf6513545278..2e493495de9d68bf981653b78de2b795dcf2eea1 100644 (file)
@@ -139,7 +139,7 @@ function doSNMPmining ($object_id, $community)
        // Only fill in attribute values, if they are not set.
        // FIXME: this is hardcoded
 
-       if (empty ($attrs[3]['value']) && !empty ($sysName)) // FQDN
+       if (!strlen ($attrs[3]['value']) && strlen ($sysName)) // FQDN
        {
                $error = commitUpdateAttrValue ($object_id, 3, $sysName);
                if ($error == TRUE)
@@ -148,7 +148,7 @@ function doSNMPmining ($object_id, $community)
                        $log[] = array ('code' => 'error', 'message' => 'Failed settig FQDN: ' . $error);
        }
 
-       if (empty ($attrs[5]['value']) and strlen ($swversion) > 0) // SW version
+       if (!strlen ($attrs[5]['value']) and strlen ($swversion) > 0) // SW version
        {
                $error = commitUpdateAttrValue ($object_id, 5, $swversion);
                if ($error == TRUE)
@@ -157,7 +157,7 @@ function doSNMPmining ($object_id, $community)
                        $log[] = array ('code' => 'error', 'message' => 'Failed settig SW version: ' . $error);
        }
 
-       if (empty ($attrs[1]['value']) and strlen ($sysChassi) > 0) // OEM Serial #1
+       if (!strlen ($attrs[1]['value']) and strlen ($sysChassi) > 0) // OEM Serial #1
        {
                $error = commitUpdateAttrValue ($object_id, 1, $sysChassi);
                if ($error == TRUE)
@@ -166,7 +166,7 @@ function doSNMPmining ($object_id, $community)
                        $log[] = array ('code' => 'error', 'message' => 'Failed settig OEM S/N 1: ' . $error);
        }
 
-       if (empty ($attrs[4]['value']) and $swfamily != 'HP') // switch OS type
+       if (!strlen ($attrs[4]['value']) and $swfamily != 'HP') // switch OS type
        {
                switch ($swfamily . '-' . $swrelease)
                {
@@ -205,7 +205,7 @@ function doSNMPmining ($object_id, $community)
                return $log;
        }
        $log[] = array ('code' => 'success', 'message' => 'HW is ' . $ciscomodel[$sysObjectID]);
-       if (empty ($attrs[2]['value']) and isset ($hwtype[$sysObjectID])) // switch HW type
+       if (!strlen ($attrs[2]['value']) and isset ($hwtype[$sysObjectID])) // switch HW type
        {
                $error = commitUpdateAttrValue ($object_id, 2, $hwtype[$sysObjectID]);
                if ($error == TRUE)
index fab98cdd25774ee65e24a7666cc76012342935d1..0358e107554a3d928bf5fbe266649a6f946cfdda 100644 (file)
@@ -11,7 +11,7 @@ if ($_REQUEST['op'] == 'addFile' && !isset($_FILES['file']['error'])) {
 }
 fixContext();
 
-if (empty ($op) or !isset ($ophandler[$pageno][$tabno][$op]))
+if (!strlen ($op) or !isset ($ophandler[$pageno][$tabno][$op]))
 {
        showError ("Invalid request in operation broker: page '${pageno}', tab '${tabno}', op '${op}'", __FILE__);
        die();
@@ -23,7 +23,7 @@ if (!isset ($delayauth[$pageno][$tabno][$op]) and !permitted())
 else
 {
        $location = $ophandler[$pageno][$tabno][$op]();
-       if (empty ($location))
+       if (!strlen ($location))
        {
                showError ('Operation handler failed to return its status', __FILE__);
        }