r2627 - deleteRow(): don't use row name from request
authorDenis Ovsienko <infrastation@yandex.ru>
Thu, 9 Apr 2009 15:11:42 +0000 (15:11 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Thu, 9 Apr 2009 15:11:42 +0000 (15:11 +0000)
 - renderRackspaceRowEditor(): don't put row name into delete request
 - ibid: fix forms table layout
 - use quotes in messages 69~79

inc/interface.php
inc/ophandlers.php

index 86ac1898581da0c1a985f1380d28ab5d39e05ed8..cdcb1c9e38ab1ced6ae0fdb7f74849253d7e72d3 100644 (file)
@@ -311,31 +311,35 @@ function renderRackspaceRowEditor ()
        function printNewItemTR ()
        {
                printOpFormIntro ('addRow');
-               echo "<tr><td><input type=text name=name tabindex=100></td><td>";
+               echo "<tr><td>";
+               printImageHREF ('create', 'Add new row', TRUE);
+               echo "</td><td><input type=text name=name tabindex=100></td><td>";
                printImageHREF ('create', 'Add new row', TRUE, 101);
-               echo "</td><td></td></tr></form>";
+               echo "</td></tr></form>";
        }
        global $pageno, $tabno;
        showMessageOrError();
        startPortlet ('Rows');
        echo "<table border=0 cellspacing=0 cellpadding=5 align=center class=widetable>\n";
-       echo "<tr><th>Name</th></tr>\n";
+       echo "<tr><th>&nbsp;</th><th>Name</th><th>&nbsp;</th></tr>\n";
        if (getConfigVar ('ADDNEW_AT_TOP') == 'yes')
                printNewItemTR();
        $rackrowList = getRackspace ();
        foreach ($rackrowList as $rackrow)
        {
-               printOpFormIntro ('updateRow', array ('row_id' => $rackrow['row_id']));
-               echo "<tr><td><input type=text name=name value='${rackrow['row_name']}'></td><td>";
-               printImageHREF ('save', 'Save changes', TRUE);
-               echo "</td></form><td>";
-               if ($rackrow['count'] == 0)
+               echo "<tr><td>";
+               if ($rackrow['count'])
+                       printImageHREF ('nodestroy', $rackrow['count'] . ' racks');
+               else
                {
-                       echo "<a href=\"".makeHrefProcess(array('op'=>'delete', 'row_id'=>$rackrow['row_id'], 'name'=>$rackrow['row_name']))."\">";
-                       printImageHREF ('delete', 'Delete row');
+                       echo "<a href=\"".makeHrefProcess(array('op'=>'delete', 'row_id'=>$rackrow['row_id']))."\">";
+                       printImageHREF ('destroy', 'Delete row');
                        echo "</a>";
                }
-               echo "</td></tr>\n";
+               printOpFormIntro ('updateRow', array ('row_id' => $rackrow['row_id']));
+               echo "</td><td><input type=text name=name value='${rackrow['row_name']}'></td><td>";
+               printImageHREF ('save', 'Save changes', TRUE);
+               echo "</form></td></tr>\n";
        }
        if (getConfigVar ('ADDNEW_AT_TOP') != 'yes')
                printNewItemTR();
@@ -1418,17 +1422,17 @@ function printLog ($log)
                                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'"),
-                               69 => array ('code' => 'success', 'format' => 'File %s was added successfully'),
-                               70 => array ('code' => 'success', 'format' => 'File %s was updated successfully'),
-                               71 => array ('code' => 'success', 'format' => 'File %s was linked successfully'),
-                               72 => array ('code' => 'success', 'format' => 'File %s was unlinked successfully'),
-                               73 => array ('code' => 'success', 'format' => 'File %s was deleted successfully'),
-                               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'),
-                               78 => array ('code' => 'success', 'format' => 'File %s saved Ok'),
-                               79 => array ('code' => 'success', 'format' => 'Rack %s was deleted successfully'),
+                               69 => array ('code' => 'success', 'format' => 'File "%s" was added successfully'),
+                               70 => array ('code' => 'success', 'format' => 'File "%s" was updated successfully'),
+                               71 => array ('code' => 'success', 'format' => 'File "%s" was linked successfully'),
+                               72 => array ('code' => 'success', 'format' => 'File "%s" was unlinked successfully'),
+                               73 => array ('code' => 'success', 'format' => 'File "%s" was deleted successfully'),
+                               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'),
+                               78 => array ('code' => 'success', 'format' => 'File "%s" saved Ok'),
+                               79 => array ('code' => 'success', 'format' => 'Rack "%s" was deleted successfully'),
 
 // records 100~199 with fatal error messages
                                100 => array ('code' => 'error', 'format' => '%s'),
index b7fdb69239775bffd14014cc41a0c3ed5474f62f..9e6a158f21b2edebf7a2e8c7ff968621b7567312 100644 (file)
@@ -1496,15 +1496,14 @@ function updateRow ()
 function deleteRow ()
 {
        assertUIntArg ('row_id', __FUNCTION__);
+       $rowinfo = getRackRowInfo ($_REQUEST['row_id']);
 
        if (TRUE === commitDeleteRow ($_REQUEST['row_id']))
-               return buildRedirectURL (__FUNCTION__, 'OK', array ($_REQUEST['name']));
+               return buildRedirectURL (__FUNCTION__, 'OK', array ($rowinfo['name']));
        else
-               return buildRedirectURL (__FUNCTION__, 'ERR', array ($_REQUEST['name']));
+               return buildRedirectURL (__FUNCTION__, 'ERR', array ($rowinfo['name']));
 }
 
-
-
 function addRack ()
 {
        assertUIntArg ('row_id', __FUNCTION__);