r2144 - fixed rack properties editor
authorDenis Ovsienko <infrastation@yandex.ru>
Tue, 19 Aug 2008 12:14:32 +0000 (12:14 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Tue, 19 Aug 2008 12:14:32 +0000 (12:14 +0000)
inc/interface.php
inc/navigation.php
inc/ophandlers.php

index 1ef0451..d77e686 100644 (file)
@@ -518,7 +518,8 @@ function renderEditObjectForm ($object_id)
 // This is a clone of renderEditObjectForm().
 function renderEditRackForm ($rack_id)
 {
-       global $pageno, $tabno;
+       showMessageOrError();
+       global $pageno;
        $rack = getRackData ($rack_id);
        if ($rack == NULL)
        {
@@ -526,7 +527,6 @@ function renderEditRackForm ($rack_id)
                return;
        }
 
-       // Render a form for the next.
        startPortlet ('Rack attributes');
        printOpFormIntro ('updateRack');
        echo '<table border=0 align=center>';
@@ -1293,6 +1293,7 @@ function printLog ($log)
                                65 => array ('code' => 'success', 'format' => "Added new rack '%s'"),
                                66 => array ('code' => 'success', 'format' => "File sent Ok via handler '%s'"),
                                67 => array ('code' => 'success', 'format' => "Tag rolling done, %u objects involved"),
+                               68 => array ('code' => 'success', 'format' => "Updated rack '%s'"),
 
                                100 => array ('code' => 'error', 'format' => 'Generic error: %s'),
                                101 => array ('code' => 'error', 'format' => 'Port name cannot be empty'),
@@ -1371,6 +1372,7 @@ function printLog ($log)
                                174 => array ('code' => 'error', 'format' => 'Bad IPv4 address'),
                                175 => array ('code' => 'error', 'format' => 'Invalid netmask'),
                                176 => array ('code' => 'error', 'format' => 'This network already exists'),
+                               177 => array ('code' => 'error', 'format' => 'commitUpdateRack() failed'),
 
                                200 => array ('code' => 'warning', 'format' => 'generic warning: %s'),
                                201 => array ('code' => 'warning', 'format' => 'nothing happened...'),
index ef35547..b9818ab 100644 (file)
@@ -72,6 +72,8 @@ $tabhandler['rack']['tags'] = 'renderRackTags';
 $trigger['rack']['tags'] = 'trigger_tags';
 $ophandler['rack']['edit']['updateRack'] = 'updateRack';
 $ophandler['rack']['tags']['saveTags'] = 'saveRackTags';
+$msgcode['rack']['edit']['updateRack']['OK'] = 68;
+$msgcode['rack']['edit']['updateRack']['ERR'] = 177;
 
 $page['objgroup']['title_handler'] = 'dynamic_title_objgroup';
 $page['objgroup']['handler'] = 'renderObjectGroup';
index faef648..0e14aa6 100644 (file)
@@ -1390,26 +1390,21 @@ function addRack ()
        }
        else
                return buildRedirectURL ('ERR2');
-       printLog ($log);
 }
 
-// FIXME: switch to message log v2
 function updateRack ()
 {
-       $log = array ('v' => 1);
        assertUIntArg ('rack_id', __FUNCTION__);
        assertUIntArg ('rack_row_id', __FUNCTION__);
        assertUIntArg ('rack_height', __FUNCTION__);
        assertStringArg ('rack_name', __FUNCTION__);
        assertStringArg ('rack_comment', __FUNCTION__, TRUE);
-       $name = $_REQUEST['rack_name'];
 
-       if (commitUpdateRack ($_REQUEST['rack_id'], $name, $_REQUEST['rack_height'], $_REQUEST['rack_row_id'], $_REQUEST['rack_comment']) === TRUE)
-               $log[] = array ('code' => 'success', 'message' => "Updated rack '${name}'");
-       else
-               $log[] = array ('code' => 'error', 'message' => __FUNCTION__ . ': commitUpdateRack() failed');
        resetThumbCache ($_REQUEST['rack_id']);
-       return buildWideRedirectURL ($log);
+       if (TRUE === commitUpdateRack ($_REQUEST['rack_id'], $_REQUEST['rack_name'], $_REQUEST['rack_height'], $_REQUEST['rack_row_id'], $_REQUEST['rack_comment']))
+               return buildRedirectURL ('OK', array ($_REQUEST['rack_name']));
+       else
+               return buildRedirectURL ('ERR');
 }
 
 function querySNMPData ()