r3740 deleteRow(): justify error detection and reporting
authorDenis Ovsienko <infrastation@yandex.ru>
Sat, 5 Jun 2010 10:39:58 +0000 (10:39 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Sat, 5 Jun 2010 10:39:58 +0000 (10:39 +0000)
inc/interface.php
inc/ophandlers.php

index 5b89622831004efe859ba241670ab1ee12b4591b..99a4b9b5bd61472faca2862f0c074d76bbfbe811 100644 (file)
@@ -1564,7 +1564,7 @@ function showMessageOrError ()
                                74 => array ('code' => 'success', 'format' => 'Row "%s" was added successfully'),
                                75 => array ('code' => 'success', 'format' => 'Row "%s" was updated successfully'),
                                76 => array ('code' => 'success', 'format' => 'Object "%s" was deleted successfully'),
-                               77 => array ('code' => 'success', 'format' => 'Row "%s" was deleted successfully'),
+                               77 => array ('code' => 'success', 'format' => 'Record "%s" was deleted successfully'),
                                78 => array ('code' => 'success', 'format' => 'File "%s" saved Ok'),
                                79 => array ('code' => 'success', 'format' => 'Rack "%s" was deleted successfully'),
                                80 => array ('code' => 'success', 'format' => "Added new object '%s'"),
@@ -1617,7 +1617,7 @@ function showMessageOrError ()
                                143 => array ('code' => 'error', 'format' => 'Tried chaining %u tags, but experienced %u errors.'),
                                144 => array ('code' => 'error', 'format' => "Error deleting tag: '%s'"),
                                145 => array ('code' => 'error', 'format' => "Invalid tag name '%s'"),
-// ...
+                               146 => array ('code' => 'error', 'format' => 'Error deleting record "%s"'),
                                147 => array ('code' => 'error', 'format' => "Could not create tag '%s': %s"),
                                148 => array ('code' => 'error', 'format' => "Could not update tag '%s': %s"),
                                149 => array ('code' => 'error', 'format' => 'Turing test failed'),
index 6d53d424e8e5842c44a537b6dbd6c994c3c3a7ab..dab9ce3ed2efb54cfcefe3b3884d27829c08ec5e 100644 (file)
@@ -1726,16 +1726,12 @@ function updateRow ()
 }
 
 $msgcode['deleteRow']['OK'] = 77;
-$msgcode['deleteRow']['ERR'] = 100;
+$msgcode['deleteRow']['ERR'] = 146;
 function deleteRow ()
 {
        assertUIntArg ('row_id');
        $rowinfo = getRackRowInfo ($_REQUEST['row_id']);
-
-       if (TRUE === commitDeleteRow ($_REQUEST['row_id']))
-               return buildRedirectURL (__FUNCTION__, 'OK', array ($rowinfo['name']));
-       else
-               return buildRedirectURL (__FUNCTION__, 'ERR', array ($rowinfo['name']));
+       return buildRedirectURL (__FUNCTION__, FALSE === commitDeleteRow ($_REQUEST['row_id']) ? 'ERR' : 'OK', array ($rowinfo['name']));
 }
 
 $msgcode['addRack']['OK'] = 65;