r2783 - getAttrMap(): also compute AttributeValue refc for each attr_id-objtype_id...

 - renderEditAttributesForm(): keep user from deleting records referenced by AttributeMap (ticket:47)
 - renderEditAttrMapForm(): idem

diff --git a/inc/database.php b/inc/database.php
index 236eb5441eff83f5729ed243771f1a7dc3eb82a2..32f17855b30eeededb3768519373cff0932f96bc 100644 (file)
--- a/inc/database.php
+++ b/inc/database.php
@@ -2088,41 +2088,43 @@ function readChapter ($chapter_name = '')
        return $chapter;
 }
 
+// Return a list of all stickers with sticker map applied. Each sticker records will
+// list all its ways on the map with refcnt set.
 function getAttrMap ()
 {
        $query =
-               "select a.id as attr_id, a.type as attr_type, a.name as attr_name, am.objtype_id, " .
-               "d.dict_value as objtype_name, am.chapter_id, c2.name as chapter_name from " .
-               "Attribute as a left join AttributeMap as am on a.id = am.attr_id " .
-               "left join Dictionary as d on am.objtype_id = d.dict_key " .
-               "left join Chapter as c1 on d.chapter_id = c1.id " .
-               "left join Chapter as c2 on am.chapter_id = c2.id " .
-               "where c1.name = 'RackObjectType' or c1.name is null " .
-               "order by a.name";
+               'SELECT id, type, name, chapter_id, (SELECT name FROM Chapter WHERE id = chapter_id) ' .
+               'AS chapter_name, objtype_id, (SELECT dict_value FROM Dictionary WHERE dict_key = objtype_id) ' .
+               'AS objtype_name, (SELECT COUNT(object_id) FROM AttributeValue AS av INNER JOIN RackObject AS ro ' .
+               'ON av.object_id = ro.id WHERE av.attr_id = Attribute.id AND ro.objtype_id = AttributeMap.objtype_id) ' .
+               'AS refcnt FROM Attribute LEFT JOIN AttributeMap ON id = attr_id ORDER BY Attribute.name, objtype_id';
        $result = useSelectBlade ($query, __FUNCTION__);
        $ret = array();
        while ($row = $result->fetch (PDO::FETCH_ASSOC))
        {
-               $attr_id = $row['attr_id'];
-               if (!isset ($ret[$attr_id]))
-               {
-                       $ret[$attr_id]['id'] = $attr_id;
-                       $ret[$attr_id]['type'] = $row['attr_type'];
-                       $ret[$attr_id]['name'] = $row['attr_name'];
-                       $ret[$attr_id]['application'] = array();
-               }
+               if (!isset ($ret[$row['id']]))
+                       $ret[$row['id']] = array
+                       (
+                               'id' => $row['id'],
+                               'type' => $row['type'],
+                               'name' => $row['name'],
+                               'application' => array(),
+                       );
                if ($row['objtype_id'] == '')
                        continue;
-               $application['objtype_id'] = $row['objtype_id'];
-               $application['objtype_name'] = $row['objtype_name'];
-               if ($row['attr_type'] == 'dict')
+               $application = array
+               (
+                       'objtype_id' => $row['objtype_id'],
+                       'objtype_name' => $row['objtype_name'],
+                       'refcnt' => $row['refcnt'],
+               );
+               if ($row['type'] == 'dict')
                {
                        $application['chapter_no'] = $row['chapter_id'];
                        $application['chapter_name'] = $row['chapter_name'];
                }
-               $ret[$attr_id]['application'][] = $application;
+               $ret[$row['id']]['application'][] = $application;
        }
-       $result->closeCursor();
        return $ret;
 }
 

diff --git a/inc/interface.php b/inc/interface.php
index 11154f904edec7a1f594121023e677108aec6907..d6da8c8afc3330371af156ab5ed8f3f15e343a86 100644 (file)
--- a/inc/interface.php
+++ b/inc/interface.php
@@ -3557,11 +3557,16 @@ function renderEditAttributesForm ()
        foreach ($attrMap as $attr)
        {
                printOpFormIntro ('upd', array ('attr_id' => $attr['id']));
-               echo '<tr>';
-               echo "<td><a href='".makeHrefProcess(array('op'=>'del', 'attr_id'=>$attr['id']))."'>";
-               printImageHREF ('delete', 'Remove attribute');
-               echo '</a></td>';
-               echo "<td><input type=text name=attr_name value='${attr['name']}'></td>";
+               echo '<tr><td>';
+               if (count ($attr['application']))
+                       printImageHREF ('nodelete', count ($attr['application']) . ' reference(s) in attribute map');
+               else
+               {
+                       echo "<a href='".makeHrefProcess(array('op'=>'del', 'attr_id'=>$attr['id']))."'>";
+                       printImageHREF ('delete', 'Remove attribute');
+                       echo '</a>';
+               }
+               echo "</td><td><input type=text name=attr_name value='${attr['name']}'></td>";
                echo "<td class=tdleft>${attr['type']}</td><td>";
                printImageHREF ('save', 'Save changes', TRUE);
                echo '</td></tr>';
@@ -3607,18 +3612,18 @@ function renderEditAttrMapForm ()
        if (getConfigVar ('ADDNEW_AT_TOP') == 'yes')
                printNewItemTR ($attrMap);
        foreach ($attrMap as $attr)
-       {
-               if (count ($attr['application']) == 0)
-                       continue;
                foreach ($attr['application'] as $app)
                {
-                       echo '<tr>';
-                       echo '<td>';
-                       echo "<a href='".makeHrefProcess(array('op'=>'del', 'attr_id'=>$attr['id'], 'objtype_id'=>$app['objtype_id']))."'>";
-                       printImageHREF ('delete', 'Remove mapping');
-                       echo "</a>";
-                       echo '</td>';
-                       echo "<td>${attr['name']}</td>";
+                       echo '<tr><td>';
+                       if ($app['refcnt'])
+                               printImageHREF ('nodelete', $app['refcnt'] . ' value(s) stored for objects');
+                       else
+                       {
+                               echo "<a href='".makeHrefProcess(array('op'=>'del', 'attr_id'=>$attr['id'], 'objtype_id'=>$app['objtype_id']))."'>";
+                               printImageHREF ('delete', 'Remove mapping');
+                               echo "</a>";
+                       }
+                       echo "</td><td>${attr['name']}</td>";
                        echo "<td>${app['objtype_name']}</td>";
                        echo "<td>";
                        if ($attr['type'] == 'dict')
@@ -3627,7 +3632,6 @@ function renderEditAttrMapForm ()
                                echo '&nbsp;';
                        echo "</td></tr>\n";
                }
-       }
        if (getConfigVar ('ADDNEW_AT_TOP') != 'yes')
                printNewItemTR ($attrMap);
        echo "</table>\n";