r1501 + employ __FUNCTION__ instead of hardcoded function names even more
authorDenis Ovsienko <infrastation@yandex.ru>
Sat, 12 Jan 2008 14:18:48 +0000 (14:18 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Sat, 12 Jan 2008 14:18:48 +0000 (14:18 +0000)
inc/database.php
inc/gateways.php
inc/help.php
inc/interface.php

index ba2ed6d4a2af0657a683de88e7b0c5a89daae7a6..60661137dc66a3e8eaaa84e41b2cd99aa5e2f16f 100644 (file)
@@ -465,7 +465,7 @@ function processGridForm (&$rackData, $unchecked_state, $checked_state, $object_
                                "unit_no = ${unit_no} and atom = '${atom}' limit 1";
                        $r = $dbxlink->query ($query);
                        if ($r == NULL)
-                               return array ('code' => 500, 'message' => "${rack_name}: SQL DELETE query failed in processGridForm()");
+                               return array ('code' => 500, 'message' => __FUNCTION__ . ": ${rack_name}: SQL DELETE query failed");
                        if ($newstate != 'F')
                        {
                                $query =
@@ -473,7 +473,7 @@ function processGridForm (&$rackData, $unchecked_state, $checked_state, $object_
                                        "values(${rack_id}, ${unit_no}, '${atom}', '${newstate}') ";
                                $r = $dbxlink->query ($query);
                                if ($r == NULL)
-                                       return array ('code' => 500, 'message' => "${rack_name}: SQL INSERT query failed in processGridForm()");
+                                       return array ('code' => 500, 'message' => __FUNCTION__ . ": ${rack_name}: SQL INSERT query failed");
                        }
                        if ($newstate == 'T' and $object_id != 0)
                        {
@@ -808,9 +808,9 @@ function commitUpdatePort ($port_id, $port_name, $port_label, $port_l2address, $
 function delObjectPort ($port_id)
 {
        if (unlinkPort ($port_id) != '')
-               return 'unlinkPort() failed in delObjectPort()';
+               return __FUNCTION__ . ': unlinkPort() failed';
        if (useDeleteBlade ('Port', 'id', $port_id) != TRUE)
-               return 'useDeleteBlade() failed in delObjectPort()';
+               return __FUNCTION__ . ': useDeleteBlade() failed';
        return '';
 }
 
@@ -1103,11 +1103,11 @@ function updateAddress ($ip=0, $name='', $reserved='no')
 function commitDeleteRange ($id = 0)
 {
        if ($id <= 0)
-               return 'Invalid range ID in commitDeleteRange()';
+               return __FUNCTION__ . ': Invalid range ID';
        if (useDeleteBlade ('IPRanges', 'id', $id))
                return '';
        else
-               return 'SQL query failed in commitDeleteRange';
+               return __FUNCTION__ . ': SQL query failed';
 }
 
 function updateBond ($ip='', $object_id=0, $name='', $type='')
@@ -1956,13 +1956,13 @@ function getDatabaseVersion ()
                $errorInfo = $dbxlink->errorInfo();
                if ($errorInfo[0] == '42S02') // ER_NO_SUCH_TABLE
                        return '0.14.4';
-               die ('SQL query #1 failed in getDatabaseVersion() with error ' . $errorInfo[2]);
+               die (__FUNCTION__ . ': SQL query #1 failed with error ' . $errorInfo[2]);
        }
        $rows = $result->fetchAll (PDO::FETCH_NUM);
        if (count ($rows) != 1 || empty ($rows[0][0]))
        {
                $result->closeCursor();
-               die ('Cannot guess database version. Config table is present, but DB_VERSION is missing or invalid. Giving up.');
+               die (__FUNCTION__ . ': Cannot guess database version. Config table is present, but DB_VERSION is missing or invalid. Giving up.');
        }
        $ret = $rows[0][0];
        $result->closeCursor();
index 0ebec6f8cd412800b5750a5e8f046c2c905b6b24..e5667ca8be0febef31c0b81ea5f71b425280462c 100644 (file)
@@ -34,7 +34,7 @@ function queryGateway ($gwname, $questions)
        $pipes = array();
        $gateway = proc_open ($execpath, $dspec, $pipes);
        if (!is_resource ($gateway))
-               return array ('ERR proc_open() failed in queryGateway()');
+               return array ('ERR proc_open() failed in ' . __FUNCTION__);
 
 // Dialogue starts. Send all questions.
        foreach ($questions as $q)
@@ -159,7 +159,7 @@ function setSwitchVLANs ($object_id = 0, $setcmd)
        global $remote_username;
        $log = array();
        if ($object_id <= 0)
-               return array (array ('code' => 'error', 'message' => 'Invalid object_id in setSwitchVLANs()'));
+               return array (array ('code' => 'error', 'message' => __FUNCTION__ . ': Invalid object_id'));
        $objectInfo = getObjectInfo ($object_id);
        $endpoints = findAllEndpoints ($object_id, $objectInfo['name']);
        if (count ($endpoints) == 0)
@@ -181,7 +181,7 @@ function setSwitchVLANs ($object_id = 0, $setcmd)
                array ("connect ${endpoint} ${hwtype} ${swtype} ${remote_username}", $setcmd)
        );
        if ($data == NULL)
-               return array (array ('code' => 'error', 'message' => 'Failed to get any response from queryGateway() or the gateway died'));
+               return array (array ('code' => 'error', 'message' => __FUNCTION__ . ': Failed to get any response from queryGateway() or the gateway died'));
        if (strpos ($data[0], 'OK!') !== 0)
                return array (array ('code' => 'error', 'message' => "Gateway failure: returned code ${data[0]}."));
        if (count ($data) != 2)
index 2c49020b571f0855cc4566d4f669e0fd17170957..ac64b698aeeba277ad804b5b667cb080881e52a6 100644 (file)
@@ -228,7 +228,7 @@ BCP go here.
 //------------------------------------------------------------------------
                default:
                        startPortlet ('Oops!');
-                       echo "There was no help text found for help tab '${tabno}' in renderHelpTab().";
+                       echo __FUNCTION__ . ": There was no help text found for help tab '${tabno}'";
                        finishPortlet();
                        break;
 //------------------------------------------------------------------------
index 03fa9851732fb9d585fd23ca0deedb788f9a0784..c8f31615299ed1206703296a455015e9e795bd04 100644 (file)
@@ -258,7 +258,7 @@ function renderNewObjectForm ()
                if (commitAddObject ($name, $label, $barcode, $type_id, $asset_no) === TRUE)
                        $log[] = array ('code' => 'success', 'message' => "Added new object '${name}'");
                else
-                       $log[] = array ('code' => 'error', 'message' => 'commitAddObject() failed in renderNewObjectForm()');
+                       $log[] = array ('code' => 'error', 'message' => __FUNCTION__ . ': commitAddObject() failed');
                printLog ($log);
        }
 
@@ -298,7 +298,7 @@ function renderNewRackForm ($row_id)
                if (commitAddRack ($name, $height, $row_id, $comment) === TRUE)
                        $log[] = array ('code' => 'success', 'message' => "Added new rack '${name}'");
                else
-                       $log[] = array ('code' => 'error', 'message' => 'commitAddRack() failed in renderNewRackForm()');
+                       $log[] = array ('code' => 'error', 'message' => __FUNCTION__ . 'commitAddRack() failed');
                printLog ($log);
        }
 
@@ -347,7 +347,7 @@ function renderEditObjectForm ($object_id)
                if (commitUpdateObject ($object_id, $name, $label, $barcode, $type_id, $has_problems, $asset_no, $comment) === TRUE)
                        $log[] = array ('code' => 'success', 'message' => "Updated object '${name}'");
                else
-                       $log[] = array ('code' => 'error', 'message' => 'commitUpdateObject() failed in renderEditObjectForm()');
+                       $log[] = array ('code' => 'error', 'message' => __FUNCTION__ . ': commitUpdateObject() failed');
                printLog ($log);
        }
 
@@ -461,7 +461,7 @@ function renderEditRackForm ($rack_id)
                if (commitUpdateRack ($rack_id, $name, $height, $row_id, $comment) === TRUE)
                        $log[] = array ('code' => 'success', 'message' => "Updated rack '${name}'");
                else
-                       $log[] = array ('code' => 'error', 'message' => 'commitUpdateRack() failed in renderEditRackForm()');
+                       $log[] = array ('code' => 'error', 'message' => __FUNCTION__ . ': commitUpdateRack() failed');
                printLog ($log);
        }
 
@@ -1209,7 +1209,7 @@ function renderRackSpaceForObject ($object_id = 0)
                                // Reload our working copy after form processing.
                                $rackData = getRackData ($rack_id);
                                if ($rackData == NULL)
-                                       $log[] = array ('code' => 500, 'message' => 'Working copy update failed in renderRackSpaceForObject()');
+                                       $log[] = array ('code' => 500, 'message' => 'Working copy update failed in ', __FUNCTION__);
                                applyObjectMountMask ($rackData, $object_id);
                                $workingRacksData[$rack_id] = $rackData;
                        }
@@ -2136,7 +2136,7 @@ function renderAddMultipleObjectsForm ()
                        if (commitAddObject ($name[$i], $label[$i], $barcode[$i], $type_id[$i], $asset_no[$i]) === TRUE)
                                $log[] = array ('code' => 'success', 'message' => "Added new object '${name[$i]}'");
                        else
-                               $log[] = array ('code' => 'error', 'message' => 'commitAddObject() failed in renderAddMultipleObjectsForm()');
+                               $log[] = array ('code' => 'error', 'message' => __FUNCTION__ . ': commitAddObject() failed');
                }
        }
        elseif (isset ($_REQUEST['got_very_fast_data']))