r2699 - updateObjectAllocation(): cleanup
authorDenis Ovsienko <infrastation@yandex.ru>
Fri, 8 May 2009 11:57:09 +0000 (11:57 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Fri, 8 May 2009 11:57:09 +0000 (11:57 +0000)
 - renderRackSpaceForObject(): idem
 - deleteObject(): discard thumbnail cache of related racks

inc/interface.php
inc/ophandlers.php

index fb5e66b..9fd17f9 100644 (file)
@@ -1685,7 +1685,6 @@ and either delete them before unmounting or refuse to unmount the object.
 */
 
 // We extensively use $_REQUEST in the function.
-// FIXME: move related code into ophandler
 function renderRackSpaceForObject ($object_id = 0)
 {
        if ($object_id <= 0)
@@ -1693,7 +1692,6 @@ function renderRackSpaceForObject ($object_id = 0)
                showError ('Invalid object_id', __FUNCTION__);
                return;
        }
-       $is_submit = isset ($_REQUEST['got_atoms']);
        $is_update = isset ($_REQUEST['rackmulti'][0]);
        $info = getObjectInfo ($object_id);
        if ($info == NULL)
@@ -1703,13 +1701,8 @@ function renderRackSpaceForObject ($object_id = 0)
        }
        // Always process occupied racks plus racks chosen by user. First get racks with
        // already allocated rackspace...
-       $workingRacksData = getResidentRacksData ($object_id);
-       if ($workingRacksData === NULL)
-       {
-               print_r ($workingRacksData);
-               showError ('getResidentRacksData() failed', __FUNCTION__);
-               return;
-       }
+       if (NULL === ($workingRacksData = getResidentRacksData ($object_id)))
+               die; // some error already shown
 
        // ...and then add those chosen by user (if any).
        if (isset($_REQUEST['rackmulti']))
@@ -1792,7 +1785,7 @@ function renderRackSpaceForObject ($object_id = 0)
                markupAtomGrid ($rackData, 'T');
                // If we have a form processed, discard user input and show new database
                // contents.
-               if (!$is_submit and $is_update)
+               if ($is_update)
                        mergeGridFormToRack ($rackData);
                echo "<td valign=top>";
                echo "<center>\n<h2>${rackData['name']}</h2>\n";
index 841ec21..c3c9334 100644 (file)
@@ -691,18 +691,10 @@ function updateObjectAllocation ()
        assertUIntArg ('object_id', __FUNCTION__);
 
        $is_submit = isset ($_REQUEST['got_atoms']);
-       $is_update = isset ($_REQUEST['rackmulti'][0]);
 
        if ($is_submit)
        {
                $object_id = $_REQUEST['object_id'];
-               $workingRacksData = getResidentRacksData ($object_id);
-               if ($workingRacksData === NULL)
-               {
-                       print_r ($workingRacksData);
-                       showError ('getResidentRacksData() failed', __FUNCTION__);
-                       return;
-               }
                $workingRacksData = array();
                foreach ($_REQUEST['rackmulti'] as $cand_id)
                {
@@ -730,7 +722,6 @@ function updateObjectAllocation ()
                        $log[] = $logrecord;
                }
                return buildWideRedirectURL($log);
-               
        }
        else
        {
@@ -742,7 +733,6 @@ function updateObjectAllocation ()
                unset($_POST['op']);
                return buildWideRedirectURL(array(), NULL, NULL, array_merge($_GET, $_POST));
        }
-
 }
 
 $msgcode['updateObject']['OK'] = 16;
@@ -902,7 +892,10 @@ function deleteObject ()
        if (NULL === ($oinfo = getObjectInfo ($_REQUEST['object_id'])))
                return buildRedirectURL (__FUNCTION__, 'ERR', array ('object not found'));
 
+       $racklist = getResidentRacksData ($_REQUEST['object_id'], FALSE);
        $error = commitDeleteObject ($_REQUEST['object_id']);
+       foreach ($racklist as $rack_id)
+               resetThumbCache ($rack_id);
 
        if ($error != '')
                return buildRedirectURL (__FUNCTION__, 'ERR', array ($error));