r1449 + and even more showError() cleanups
authorDenis Ovsienko <infrastation@yandex.ru>
Fri, 28 Dec 2007 16:18:49 +0000 (16:18 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Fri, 28 Dec 2007 16:18:49 +0000 (16:18 +0000)
inc/config.php
inc/functions.php
inc/gateways.php
inc/ophandlers.php
inc/pagehandlers.php
inc/pagetitles.php

index 49fc984ad80dcc7a5b8165bb1b3059abe9566ff6..9222ae51a75aa287b2f75ebbec97f694e7742e1f 100644 (file)
@@ -24,12 +24,12 @@ function getConfigVar ($varname = '')
        // has failed, we don't retry loading.
        if (!isset ($configCache))
        {
-               showError ("Configuration cache is unavailable in getConfigVar()");
+               showError ("Configuration cache is unavailable", __FUNCTION__);
                die;
        }
        if ($varname == '')
        {
-               showError ("Missing argument to getConfigVar()");
+               showError ("Missing argument", __FUNCTION__);
                die;
        }
        if (isset ($configCache[$varname]))
@@ -50,18 +50,18 @@ function setConfigVar ($varname = '', $varvalue = '', $softfail = FALSE)
        global $configCache;
        if (!isset ($configCache))
        {
-               showError ('Configuration cache is unavailable in setConfigVar()');
+               showError ('Configuration cache is unavailable', __FUNCTION__);
                die;
        }
        if (empty ($varname))
        {
-               showError ("Empty argument to setConfigVar()");
+               showError ("Empty argument", __FUNCTION__);
                die;
        }
        // We don't operate on unknown data.
        if (!isset ($configCache[$varname]))
        {
-               showError ("setConfigVar() doesn't know how to handle '${varname}'");
+               showError ("don't know how to handle '${varname}'", __FUNCTION__);
                die;
        }
        if (empty ($varvalue) && $configCache[$varname]['emptyok'] != 'yes')
@@ -69,7 +69,7 @@ function setConfigVar ($varname = '', $varvalue = '', $softfail = FALSE)
                $errormsg = "'${varname}' is configured to take non-empty value. Perhaps there was a reason to do so.";
                if ($softfail)
                        return $errormsg;
-               showError ($errormsg);
+               showError ($errormsg, __FUNCTION__);
                die;
        }
        if (!empty ($varvalue) && $configCache[$varname]['vartype'] == 'uint' && (!is_numeric ($varvalue) or $varvalue < 0 ))
@@ -77,7 +77,7 @@ function setConfigVar ($varname = '', $varvalue = '', $softfail = FALSE)
                $errormsg = "'${varname}' can accept UINT values only";
                if ($softfail)
                        return $errormsg;
-               showError ($errormsg);
+               showError ($errormsg, __FUNCTION__);
                die;
        }
        // Update cache only if the changes went into DB.
index aac418f6160912c0f4f3c25b74a2b36503a5d9ea..2ad5fa7c5c768e3b6bb5286b5f46f0b76c6d1258 100644 (file)
@@ -111,7 +111,7 @@ function markAllSpans (&$rackData = NULL)
 {
        if ($rackData == NULL)
        {
-               showError ('Invalid rackData in markAllSpans()');
+               showError ('Invalid rackData', __FUNCTION__);
                return;
        }
        for ($i = $rackData['height']; $i > 0; $i--)
@@ -141,7 +141,7 @@ function delRow ($row_id = 0)
 {
        if ($row_id == 0)
        {
-               showError ('Not all required args to delRow() are present.');
+               showError ('Not all required args are present.', __FUNCTION__);
                return;
        }
        if (!isset ($_REQUEST['confirmed']) || $_REQUEST['confirmed'] != 'true')
@@ -154,7 +154,7 @@ function delRow ($row_id = 0)
        $result = $dbxlink->query ("update RackRow set deleted = 'yes' where id=${row_id} limit 1");
        if ($result->rowCount() != 1)
        {
-               showError ('Marked ' . $result.rowCount() . ' rows as deleted, but expected 1');
+               showError ('Marked ' . $result.rowCount() . ' rows as deleted, but expected 1', __FUNCTION__);
                return;
        }
        echo 'OK<br>';
@@ -166,7 +166,7 @@ function delRack ($rack_id = 0)
 {
        if ($rack_id == 0)
        {
-               showError ('Not all required args to delRack() are present.');
+               showError ('Not all required args are present.', __FUNCTION__);
                return;
        }
        if (!isset ($_REQUEST['confirmed']) || $_REQUEST['confirmed'] != 'true')
@@ -179,7 +179,7 @@ function delRack ($rack_id = 0)
        $result = $dbxlink->query ("update Rack set deleted = 'yes' where id=${rack_id} limit 1");
        if ($result->rowCount() != 1)
        {
-               showError ('Marked ' . $result.rowCount() . ' rows as deleted, but expected 1');
+               showError ('Marked ' . $result.rowCount() . ' rows as deleted, but expected 1', __FUNCTION__);
                return;
        }
        echo 'OK<br>';
@@ -191,7 +191,7 @@ function delObject ($object_id = 0)
 {
        if ($object_id == 0)
        {
-               showError ('Not all required args to delObject() are present.');
+               showError ('Not all required args are present.', __FUNCTION__);
                return;
        }
        if (!isset ($_REQUEST['confirmed']) || $_REQUEST['confirmed'] != 'true')
@@ -204,7 +204,7 @@ function delObject ($object_id = 0)
        $result = $dbxlink->query ("update RackObject set deleted = 'yes' where id=${object_id} limit 1");
        if ($result->rowCount() != 1)
        {
-               showError ('Marked ' . $result.rowCount() . ' rows as deleted, but expected 1');
+               showError ('Marked ' . $result.rowCount() . ' rows as deleted, but expected 1', __FUNCTION__);
                return;
        }
        echo 'OK<br>';
@@ -653,7 +653,7 @@ function mergeSearchResults (&$objects, $terms, $fieldname)
        $result = $dbxlink->query($query);
        if ($result == NULL)
        {
-               showError ("SQL query failed in mergeSearchResults()");
+               showError ("SQL query failed", __FUNCTION__);
                return NULL;
        }
 // FIXME: this dead call was executed 4 times per 1 object search!
@@ -731,7 +731,7 @@ function getPrevIDforRack ($row_id = 0, $rack_id = 0)
 {
        if ($row_id <= 0 or $rack_id <= 0)
        {
-               showError ('Invalid arguments passed to getPrevIDforRack()');
+               showError ('Invalid arguments passed', __FUNCTION__);
                return NULL;
        }
        $rackList = getRacksForRow ($row_id);
@@ -745,7 +745,7 @@ function getNextIDforRack ($row_id = 0, $rack_id = 0)
 {
        if ($row_id <= 0 or $rack_id <= 0)
        {
-               showError ('Invalid arguments passed to getNextIDforRack()');
+               showError ('Invalid arguments passed', __FUNCTION__);
                return NULL;
        }
        $rackList = getRacksForRow ($row_id);
index 62b833d3d7f1c9bbbb14a2530abb0179c24e2eca..0ebec6f8cd412800b5750a5e8f046c2c905b6b24 100644 (file)
@@ -70,19 +70,19 @@ function getSwitchVLANs ($object_id = 0)
        global $remote_username;
        if ($object_id <= 0)
        {
-               showError ('Invalid object_id in getSwitchVLANs()');
+               showError ('Invalid object_id', __FUNCTION__);
                return;
        }
        $objectInfo = getObjectInfo ($object_id);
        $endpoints = findAllEndpoints ($object_id, $objectInfo['name']);
        if (count ($endpoints) == 0)
        {
-               showError ('Can\'t find any mean to reach current object. Please either set FQDN attribute or assign an IP address to the object.');
+               showError ('Can\'t find any mean to reach current object. Please either set FQDN attribute or assign an IP address to the object.', __FUNCTION__);
                return NULL;
        }
        if (count ($endpoints) > 1)
        {
-               showError ('More than one IP address is assigned to this object, please configure FQDN attribute.');
+               showError ('More than one IP address is assigned to this object, please configure FQDN attribute.', __FUNCTION__);
                return NULL;
        }
        $hwtype = $swtype = 'unknown';
@@ -104,24 +104,24 @@ function getSwitchVLANs ($object_id = 0)
        $data = queryGateway ('switchvlans', $commands);
        if ($data == NULL)
        {
-               showError ('Failed to get any response from queryGateway() or the gateway died');
+               showError ('Failed to get any response from queryGateway() or the gateway died', __FUNCTION__);
                return NULL;
        }
        if (strpos ($data[0], 'OK!') !== 0)
        {
-               showError ("Gateway failure: returned code ${data[0]}.");
+               showError ("Gateway failure: returned code ${data[0]}.", __FUNCTION__);
                return NULL;
        }
        if (count ($data) != count ($commands))
        {
-               showError ("Gateway failure: mailformed reply.");
+               showError ("Gateway failure: mailformed reply.", __FUNCTION__);
                return NULL;
        }
        // Now we have VLAN list in $data[1] and port list in $data[2]. Let's sort this out.
        $tmp = array_unique (explode (';', substr ($data[1], strlen ('OK!'))));
        if (count ($tmp) == 0)
        {
-               showError ("Gateway succeeded, but returned no VLAN records.");
+               showError ("Gateway succeeded, but returned no VLAN records.", __FUNCTION__);
                return NULL;
        }
        $vlanlist = array();
@@ -139,7 +139,7 @@ function getSwitchVLANs ($object_id = 0)
        }
        if (count ($portlist) == 0)
        {
-               showError ("Gateway succeeded, but returned no port records.");
+               showError ("Gateway succeeded, but returned no port records.", __FUNCTION__);
                return NULL;
        }
        $maclist = array();
index a95ecf67cb42d1d2061cb30974c513456219c9df..2444bc1ec406a23ff765f25c2d80339b2fca03ad 100644 (file)
@@ -527,7 +527,7 @@ function updateUserAccount ()
        $old_hash = getHashByID ($id);
        if ($old_hash == NULL)
        {
-               showError ('getHashByID() failed in updateUserAccount()');
+               showError ('getHashByID() failed', __FUNCTION__);
                return;
        }
        // Update user password only if provided password is not the same as current password hash.
@@ -643,7 +643,7 @@ function savePortMap ()
                                                        $error_count++;
                                                break;
                                        default:
-                                               showError ('oldCompatTable is invalid in savePortMap()');
+                                               showError ('oldCompatTable is invalid', __FUNCTION__);
                                                break;
                                }
        return
@@ -830,7 +830,7 @@ function updateAttrValues ()
                                $oldvalue = $oldvalues[$attr_id]['key'];
                                break;
                        default:
-                               showError ('Internal structure error in updateAttrValues()');
+                               showError ('Internal structure error', __FUNCTION__);
                                die;
                }
                if ($value == $oldvalue)
index f346a9ad97d12d2014e9de4d3fb6460d7952d1eb..44498aed911f32db16a96bb70a16fb3f0a695613 100644 (file)
@@ -11,22 +11,22 @@ function assertUIntArg ($argname, $allow_zero = FALSE)
 {
        if (!isset ($_REQUEST[$argname]))
        {
-               showError ("Parameter '${argname}' is missing.");
+               showError ("Parameter '${argname}' is missing.", __FUNCTION__);
                die();
        }
        if (!is_numeric ($_REQUEST[$argname]))
        {
-               showError ("Parameter '${argname}' is not a number.");
+               showError ("Parameter '${argname}' is not a number.", __FUNCTION__);
                die();
        }
        if ($_REQUEST[$argname] < 0)
        {
-               showError ("Parameter '${argname}' is less than zero.");
+               showError ("Parameter '${argname}' is less than zero.", __FUNCTION__);
                die();
        }
        if (!$allow_zero and $_REQUEST[$argname] == 0)
        {
-               showError ("Parameter '${argname}' is equal to zero.");
+               showError ("Parameter '${argname}' is equal to zero.", __FUNCTION__);
                die();
        }
 }
@@ -37,17 +37,17 @@ function assertStringArg ($argname, $ok_if_empty = FALSE)
 {
        if (!isset ($_REQUEST[$argname]))
        {
-               showError ("Parameter '${argname}' is missing.");
+               showError ("Parameter '${argname}' is missing.", __FUNCTION__);
                die();
        }
        if (!is_string ($_REQUEST[$argname]))
        {
-               showError ("Parameter '${argname}' is not a string.");
+               showError ("Parameter '${argname}' is not a string.", __FUNCTION__);
                die();
        }
        if (!$ok_if_empty and empty ($_REQUEST[$argname]))
        {
-               showError ("Parameter '${argname}' is an empty string.");
+               showError ("Parameter '${argname}' is an empty string.", __FUNCTION__);
                die();
        }
 }
@@ -56,17 +56,17 @@ function assertBoolArg ($argname, $ok_if_empty = FALSE)
 {
        if (!isset ($_REQUEST[$argname]))
        {
-               showError ("Parameter '${argname}' is missing.");
+               showError ("Parameter '${argname}' is missing.", __FUNCTION__);
                die();
        }
        if (!is_string ($_REQUEST[$argname]) or $_REQUEST[$argname] != 'on')
        {
-               showError ("Parameter '${argname}' is not a string.");
+               showError ("Parameter '${argname}' is not a string.", __FUNCTION__);
                die();
        }
        if (!$ok_if_empty and empty ($_REQUEST[$argname]))
        {
-               showError ("Parameter '${argname}' is an empty string.");
+               showError ("Parameter '${argname}' is an empty string.", __FUNCTION__);
                die();
        }
 }
@@ -90,7 +90,7 @@ function handler_config ($tabno)
                        renderConfigMainpage();
                        break;
                default:
-                       showError ("Invalid tab '${tabno}' requested in handler_config().");
+                       showError ("Invalid tab '${tabno}' requested.", __FUNCTION__);
        }
 }
 
@@ -105,7 +105,7 @@ function handler_portmap ($tabno)
                        renderPortMap (TRUE);
                        break;
                default:
-                       showError ("Invalid tab '${tabno}' requested in handler_portmap().");
+                       showError ("Invalid tab '${tabno}' requested.", __FUNCTION__);
        }
 }
 
index d48ce646f4a73fd97b83df9e6a45291d1a0bf461..af4d86d518241cea6d1714af727d39a300582866 100644 (file)
@@ -39,7 +39,7 @@ function dynamic_title_row ()
                        $rack = getRackData ($_REQUEST['rack_id']);
                        if ($rack == NULL)
                        {
-                               showError ('getRackData() failed in dynamic_title_row()');
+                               showError ('getRackData() failed', __FUNCTION__);
                                return NULL;
                        }
                        $ret['name'] = $rack['row_name'];
@@ -50,7 +50,7 @@ function dynamic_title_row ()
                        $rowInfo = getRackRowInfo ($_REQUEST['row_id']);
                        if ($rowInfo == NULL)
                        {
-                               showError ('getRackRowInfo() failed in dynamic_title_row()');
+                               showError ('getRackRowInfo() failed', __FUNCTION__);
                                return NULL;
                        }
                        $ret['name'] = $rowInfo['dict_value'];
@@ -79,7 +79,7 @@ function dynamic_title_object ()
                        $object = getObjectInfo ($_REQUEST['object_id']);
                        if ($object == NULL)
                        {
-                               showError ('getObjectInfo() failed in dynamic_title_object()');
+                               showError ('getObjectInfo() failed', __FUNCTION__);
                                return NULL;
                        }
                        $ret['name'] = $object['dname'];
@@ -134,7 +134,7 @@ function dynamic_title_objgroup ()
                        $groupInfo = getObjectGroupInfo ($_REQUEST['group_id']);
                        if ($groupInfo == NULL)
                        {
-                               showError ('getObjectGroupInfo() failed in dynamic_title_objgroup()');
+                               showError ('getObjectGroupInfo() failed', __FUNCTION__);
                                return NULL;
                        }
                        $ret['name'] = $groupInfo['name'];
@@ -145,7 +145,7 @@ function dynamic_title_objgroup ()
                        $objectInfo = getObjectInfo ($_REQUEST['object_id']);
                        if ($objectInfo == NULL)
                        {
-                               showError ('getObjectInfo() failed in dynamic_title_objgroup()');
+                               showError ('getObjectInfo() failed', __FUNCTION__);
                                return NULL;
                        }
                        $ret['name'] = $objectInfo['objtype_name'];