r2694 - renderRackspace(): don't use getRackspace()
authorDenis Ovsienko <infrastation@yandex.ru>
Thu, 7 May 2009 16:06:51 +0000 (16:06 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Thu, 7 May 2009 16:06:51 +0000 (16:06 +0000)
 - getRackspace(): phased out
 - renderRackspaceRowEditor(): fix message text

inc/database.php
inc/interface.php

index 822cfc186d4abfce19be571657c4f02a25ca7422..80bcb5f0165c7951a88d7f94c3a4d94dee1e38df 100644 (file)
@@ -33,26 +33,6 @@ function escapeString ($value, $do_db_escape = TRUE)
        return $ret;
 }
 
-function getRackspace ($tagfilter = array())
-{
-       $whereclause = getWhereClause ($tagfilter);
-       $query =
-               "select RackRow.id as row_id, RackRow.name as row_name, count(Rack.id) as count " .
-               "from RackRow left join Rack on Rack.row_id = RackRow.id " .
-               "left join TagStorage on Rack.id = TagStorage.entity_id and entity_realm = 'rack' " .
-               "where 1=1 " .
-               $whereclause .
-               " group by RackRow.id order by RackRow.name";
-       $result = useSelectBlade ($query, __FUNCTION__);
-       $ret = array();
-       $clist = array ('row_id', 'row_name', 'count');
-       while ($row = $result->fetch (PDO::FETCH_ASSOC))
-               foreach ($clist as $cname)
-                       $ret[$row['row_id']][$cname] = $row[$cname];
-       $result->closeCursor();
-       return $ret;
-}
-
 // Return detailed information about one rack row.
 function getRackRowInfo ($rackrow_id)
 {
index ba6d6a02b60f38a11d3af37849c1e0f42c3fe47c..8fadbe8b656a11c27306c15790a2ee5b70e117b6 100644 (file)
@@ -281,15 +281,16 @@ function renderRackspace ()
        echo '</td><td class=pcright>';
        echo '<table border=0 cellpadding=10 cellpadding=1>';
        // generate thumb gallery
-       $rackrowList = getRackspace ($tagfilter);
        global $nextorder;
        $rackwidth = getRackImageWidth();
        // Zero value effectively disables the limit.
        $maxPerRow = getConfigVar ('RACKS_PER_ROW');
        $order = 'odd';
-       foreach ($rackrowList as $rackrow)
+       foreach (getRackRows() as $row_id => $row_name)
        {
-               $rackList = getRacksForRow ($rackrow['row_id'], $tagfilter);
+               $rackList = getRacksForRow ($row_id, $tagfilter);
+               if (!count ($rackList) and count ($tagfilter))
+                       continue;
                $rackListIdx = 0;
                foreach ($rackList as $rack)
                {
@@ -298,8 +299,8 @@ function renderRackspace ()
                                if ($rackListIdx > 0)
                                        echo '</tr></table></tr>';
                                echo "<tr class=row_${order}><th class=tdleft>";
-                               echo "<a href='".makeHref(array('page'=>'row', 'row_id'=>$rackrow['row_id']))."${tagfilter_str}'>";
-                               echo "${rackrow['row_name']}</a>";
+                               echo "<a href='".makeHref(array('page'=>'row', 'row_id'=>$row_id))."${tagfilter_str}'>";
+                               echo "${row_name}</a>";
                                if ($rackListIdx > 0)
                                        echo ' (continued)';
                                echo "</th><td><table border=0 cellspacing=5><tr>";
@@ -339,8 +340,8 @@ function renderRackspaceRowEditor ()
        foreach (getRackRows() as $row_id => $row_name)
        {
                echo "<tr><td>";
-               if (count (getRacksForRow ($row_id)))
-                       printImageHREF ('nodestroy', $rackrow['count'] . ' racks');
+               if ($rc = count (getRacksForRow ($row_id)))
+                       printImageHREF ('nodestroy', "${rc} rack(s) here");
                else
                {
                        echo "<a href=\"".makeHrefProcess(array('op'=>'delete', 'row_id'=>$row_id))."\">";