r1376 + eliminating dict_key in readChapter() return value
authorDenis Ovsienko <infrastation@yandex.ru>
Fri, 7 Dec 2007 14:53:41 +0000 (14:53 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Fri, 7 Dec 2007 14:53:41 +0000 (14:53 +0000)
inc/database.php
inc/interface.php

index 47546791da94cefda25cff4417f11d2a277fa1e7..3773caac8c836a0f5629c5213b68640a69d0f8f1 100644 (file)
@@ -1516,7 +1516,8 @@ function commitDeleteChapter ($chapter_no = 0)
        return TRUE;
 }
 
-// This is a dictionary accessor.
+// This is a dictionary accessor. We perform link rendering, so the user sees
+// nice <select> drop-downs.
 function readChapter ($chapter_name = '')
 {
        if (empty ($chapter_name))
@@ -1537,12 +1538,7 @@ function readChapter ($chapter_name = '')
        }
        $chapter = array();
        while ($row = $result->fetch (PDO::FETCH_ASSOC))
-       {
-               $word = array();
-               $word['dict_key'] = $row['dict_key'];
-               $word['dict_value'] = parseWikiLink ($row['dict_value'], 'o');
-               $chapter[] = $word;
-       }
+               $chapter[$row['dict_key']] = parseWikiLink ($row['dict_value'], 'o');
        $result->closeCursor();
        return $chapter;
 }
index 16b5145133cbfd64b3454f9a5d2fc79e7a77ff48..38b4e22781f9657bc50ae163514b0330bc964813 100644 (file)
@@ -411,7 +411,7 @@ function renderEditObjectForm ($object_id)
                                break;
                        case 'dict':
                                $chapter = readChapter ($record['chapter_name']);
-                               $chapter[] = array ('dict_key' => 0, 'dict_value' => '-- NOT SET --');
+                               $chapter[0] = '-- NOT SET --';
                                printSelect ($chapter, "${i}_value", $record['key']);
                                break;
                }
@@ -474,7 +474,7 @@ function renderEditRackForm ($rack_id)
        echo "<input type=hidden name=rack_id value=${rack_id}>";
        echo '<table border=0 align=center>';
        echo "<tr><th class=tdright>Rack row:</th><td class=tdleft>";
-       printSelect (getRackRowInfo(), 'rack_row_id', $rack['row_id']);
+       printSelect (readChapter ('RackRow'), 'rack_row_id', $rack['row_id']);
        echo "</td></tr>\n";
        echo "<tr><th class=tdright>Name (required):</th><td class=tdleft><input type=text name=rack_name value='${rack['name']}'></td></tr>\n";
        echo "<tr><th class=tdright>Height (required):</th><td class=tdleft><input type=text name=rack_height value='${rack['height']}'></td></tr>\n";
@@ -492,12 +492,12 @@ function renderEditRackForm ($rack_id)
 function printSelect ($rowList, $select_name, $selected_id = 1)
 {
        echo "<select name=${select_name}>";
-       foreach ($rowList as $dummy => $data)
+       foreach ($rowList as $dict_key => $dict_value)
        {
-               echo "<option value=${data['dict_key']}";
-               if ($data['dict_key'] == $selected_id)
+               echo "<option value=${dict_key}";
+               if ($dict_key == $selected_id)
                        echo ' selected';
-               echo ">${data['dict_value']}</option>";
+               echo ">${dict_value}</option>";
        }
        echo "</select>";
 }
@@ -2100,8 +2100,7 @@ function renderAddMultipleObjectsForm ()
 
        // Render a form for the next.
        $typelist = getObjectTypeList();
-       $typelist[0]['dict_key'] = 0;
-       $typelist[0]['dict_value'] = 'select type...';
+       $typelist[0] = 'select type...';
 
        startPortlet ('Fast way');
        echo '<form>';