]> git.mjollnir.org Git - moodle.git/commitdiff
MDL-19787 Upgraded calls to choose_from_menu
authornicolasconnault <nicolasconnault>
Mon, 17 Aug 2009 15:15:20 +0000 (15:15 +0000)
committernicolasconnault <nicolasconnault>
Mon, 17 Aug 2009 15:15:20 +0000 (15:15 +0000)
17 files changed:
admin/auth_config.php
admin/enrol.php
admin/mnet/access_control.php
admin/register.php
admin/report/capability/index.php
admin/report/courseoverview/index.php
admin/report/questioninstances/index.php
admin/roles/assign.php
admin/roles/lib.php
admin/timezone.php
admin/xmldb/actions/edit_field/edit_field.class.php
admin/xmldb/actions/edit_index/edit_index.class.php
admin/xmldb/actions/edit_key/edit_key.class.php
admin/xmldb/actions/new_statement/new_statement.class.php
admin/xmldb/actions/new_table_from_mysql/new_table_from_mysql.class.php
admin/xmldb/actions/view_structure_php/view_structure_php.class.php
admin/xmldb/actions/view_table_php/view_table_php.class.php

index 57cad02b9353b7e071984d3cc0b064eaf2e23c33..4b1d9dea5780826d9424900d8456974fac6d2119 100644 (file)
@@ -86,7 +86,7 @@ exit;
 // other options
 // Note: lockconfig_ fields have special handling.
 function print_auth_lock_options ($auth, $user_fields, $helptext, $retrieveopts, $updateopts) {
-
+    global $OUTPUT;
     echo '<tr><td colspan="3">';
     if ($retrieveopts) {
         echo $OUTPUT->heading(get_string('auth_data_mapping', 'auth'));
@@ -147,23 +147,23 @@ function print_auth_lock_options ($auth, $user_fields, $helptext, $retrieveopts,
             echo "<input id=\"lockconfig_{$varname}\" name=\"lockconfig_{$varname}\" type=\"text\" size=\"30\" value=\"{$pluginconfig->$varname}\" />";
             echo '<div style="text-align: right">';
             echo '<label for="menulockconfig_field_updatelocal_'.$field.'">'.get_string('auth_updatelocal', 'auth') . '</label>&nbsp;';
-            choose_from_menu($updatelocaloptions, "lockconfig_field_updatelocal_{$field}", $pluginconfig->{"field_updatelocal_$field"}, "");
+            echo $OUTPUT->select(html_select::make($updatelocaloptions, "lockconfig_field_updatelocal_{$field}", $pluginconfig->{"field_updatelocal_$field"}, false));
             echo '<br />';
             if ($updateopts) {
                 echo '<label for="menulockconfig_field_updateremote_'.$field.'">'.get_string('auth_updateremote', 'auth') . '</label>&nbsp;';
-                choose_from_menu($updateextoptions, "lockconfig_field_updateremote_{$field}", $pluginconfig->{"field_updateremote_$field"}, "");
+                echo $OUTPUT->select(html_select::make($updateextoptions, "lockconfig_field_updateremote_{$field}", $pluginconfig->{"field_updateremote_$field"}, false));
                 echo '<br />';
 
 
             }
             echo '<label for="menulockconfig_field_lock_'.$field.'">'.get_string('auth_fieldlock', 'auth') . '</label>&nbsp;';
-            choose_from_menu($lockoptions, "lockconfig_field_lock_{$field}", $pluginconfig->{"field_lock_$field"}, "");
+            echo $OUTPUT->select(html_select::make($lockoptions, "lockconfig_field_lock_{$field}", $pluginconfig->{"field_lock_$field"}, false));
             echo '</div>';
         } else {
             echo '<tr valign="top"><td align="right">';
             echo '<label for="menulockconfig_field_lock_'.$field.'">'.$fieldname.'</label>';
             echo '</td><td>';
-            choose_from_menu($lockoptions, "lockconfig_field_lock_{$field}", $pluginconfig->{"field_lock_$field"}, "");
+            echo $OUTPUT->select(html_select::make($lockoptions, "lockconfig_field_lock_{$field}", $pluginconfig->{"field_lock_$field"}, false));
         }
         echo '</td>';
         if (!empty($helptext)) {
index ee8b9e5aacb0a7fd7beca219a545e185aeb5e83c..3c20d6c361260f6a29286c6df6de9037ccd8d7ec 100644 (file)
     echo '<span class="form-shortname">sendcoursewelcomemessage</span>';
     echo '</label></div>';
     echo '<div class="form-setting"><div class="form-checkbox defaultsnext">';
-    choose_from_menu($yesnooptions, 'sendcoursewelcomemessage', $CFG->sendcoursewelcomemessage, '');
+    echo $OUTPUT->select(html_select::make($yesnooptions, 'sendcoursewelcomemessage', $CFG->sendcoursewelcomemessage, false));
     echo '</div><div class="form-defaultinfo">'.get_string('defaultsettinginfo', 'admin', get_string('yes')).'</div></div>';
     echo '<div class="form-description">' . get_string('configsendcoursewelcomemessage', 'admin') . '</div>';
     echo '</div>';
index 34799f2b93905a69a55c42b163667af09a209c67..242e0a79a01e1b4487268071bc3dfa858b3f604a 100644 (file)
@@ -213,7 +213,7 @@ echo " " . get_string('remotehost', 'mnet') . ":\n";
 if (!empty($formerror['mnet_host_id'])) {
     echo '<span class="error"> * </span>';
 }
-choose_from_menu($mnethosts, 'mnet_host_id');
+echo $OUTPUT->select(html_select::make($mnethosts, 'mnet_host_id'));
 
 // choose an access level
 echo " " . get_string('accesslevel', 'mnet') . ":\n";
@@ -222,7 +222,7 @@ if (!empty($formerror['accessctrl'])) {
 }
 $accessmenu['allow'] = get_string('allow', 'mnet');
 $accessmenu['deny'] = get_string('deny', 'mnet');
-choose_from_menu($accessmenu, 'accessctrl');
+echo $OUTPUT->select(html_select::make($accessmenu, 'accessctrl'));
 
 // submit button
 echo '<input type="submit" value="' . get_string('addtoacl', 'mnet') . '" />';
index ba93a2f7ab39b4fdc8f698a11be6d76f783595a2..a712d375527cdcb8eda0e557636bc5fad21e5a80 100644 (file)
@@ -78,7 +78,9 @@
     echo '<div class="fitem">';
     echo '<div class="fitemtitle"><label for="menucountry">'.get_string("country").'</label></div>';
     echo '<div class="felement ftext">';
-    choose_from_menu (get_list_of_countries(), "country", $admin->country, get_string("selectacountry")."...", "", "");
+    $select = html_select::make(get_list_of_countries(), "country", $admin->country, get_string("selectacountry")."...");
+    $select->nothingvalue = '';
+    echo $OUTPUT->select($select);
     echo '</div>';
     echo '</div>';
 
@@ -88,7 +90,9 @@
     $options[0] = get_string("publicdirectory0");
     $options[1] = get_string("publicdirectory1");
     $options[2] = get_string("publicdirectory2");
-    choose_from_menu ($options, "public", "2", "", "", "");
+    $select = html_select::make($options, "public", "2", false);
+    $select->nothingvalue = '';
+    echo $OUTPUT->select($select);
     unset($options);
     echo '</div>';
     echo '</div>';
     echo '<div class="felement ftext">';
     $options[0] = get_string("registrationcontactno");
     $options[1] = get_string("registrationcontactyes");
-    choose_from_menu ($options, "contact", "1", "", "", "");
+    $select = html_select::make($options, "contact", "1", false);
+    $select->nothingvalue = '';
+    echo $OUTPUT->select($select);
     unset($options);
     echo '</div>';
     echo '</div>';
     echo '<div class="felement ftext">';
     $options[0] = get_string("registrationno");
     $options[1] = get_string("registrationyes");
-    choose_from_menu ($options, "mailme", "1", "", "", "");
+    $select = html_select::make($options, "mailme", "1", false);
+    $select->nothingvalue = '';
+    echo $OUTPUT->select($select);
     unset($options);
     echo '</div>';
     echo '</div>';
index 0c2f33ca61c48e0d89c543283f1cbf5f4f5b560f..a4cc958a13ab86deb8606d018cb850acb5ca1fd9 100644 (file)
@@ -74,9 +74,18 @@ echo '<form method="get" action="." id="settingsform">';
 echo $OUTPUT->heading(get_string('reportsettings', 'report_capability'));
 echo '<p id="intro">', get_string('intro', 'report_capability') , '</p>';
 echo '<p><label for="menucapability"> ' . get_string('capabilitylabel', 'report_capability') . '</label></p>';
-choose_from_menu($capabilitychoices, 'capability', $capability, 'choose', '', '', false, false, 0, '', true);
+$select = html_select::make($capabilitychoices, 'capability', $capability);
+$select->nothingvalue = '';
+$select->listbox = true;
+$select->tabindex = 0;
+echo $OUTPUT->select($select);
 echo '<p><label for="menuroles"> ' . get_string('roleslabel', 'report_capability') . '</label></p>';
-choose_from_menu($rolechoices, 'roles[]', $selectedroleids, '', '', '', false, false, 0, '', true, true);
+$select = html_select::make($rolechoices, 'roles[]', $selectedroleids, false);
+$select->nothingvalue = '';
+$select->listbox = true;
+$select->multiple = true;
+$select->tabindex = 0;
+echo $OUTPUT->select($select);
 echo '<p><input type="submit" id="settingssubmit" value="' . get_string('getreport', 'report_capability') . '" /></p>';
 echo '</form>';
 echo $OUTPUT->box_end();
index 1d54fba7b5ef8419426283595f3f4e723a6fa93f..3c95477c57bde13cb4541af4335fc63d159e80de 100644 (file)
 
     $table->width = '*';
     $table->align = array('left','left','left','left','left','left');
-    $table->data[] = array(get_string('statsreporttype'),choose_from_menu($reportoptions,'report',$report,'','','',true),
-                           get_string('statstimeperiod'),choose_from_menu($timeoptions,'time',$time,'','','',true),
+    
+    $select = html_select::make($reportoptions,'report',$report, false);
+    $select->nothingvalue = '';
+    $reporttypemenu = $OUTPUT->select($select);
+    
+    $select = html_select::make($timeoptions,'time',$time, false);
+    $select->nothingvalue = '';
+    $timeoptionsmenu = $OUTPUT->select($select);
+    
+    $table->data[] = array(get_string('statsreporttype'),$reporttypemenu,
+                           get_string('statstimeperiod'),$timeoptionsmenu,
                            '<input type="text" name="numcourses" size="3" maxlength="2" value="'.$numcourses.'" />',
                            '<input type="submit" value="'.get_string('view').'" />') ;
 
index 32ac9997b71872eb27132bcc8010f924de86e7ab..d097cbabc6a7301a2250b58c4f3dedac502e3885 100644 (file)
@@ -33,7 +33,9 @@ echo '<form method="get" action="." id="settingsform"><div>';
 echo $OUTPUT->heading(get_string('reportsettings', 'report_questioninstances'));
 echo '<p id="intro">', get_string('intro', 'report_questioninstances') , '</p>';
 echo '<p><label for="menuqtype"> ' . get_string('questiontype', 'admin') . '</label> ';
-choose_from_menu($qtypechoices, 'qtype', $requestedqtype, get_string('all'), '', '_all_');
+$select = html_select::make($qtypechoices, 'qtype', $requestedqtype, get_string('all'));
+$select->nothingvalue = '_all_';
+echo $OUTPUT->select($select);
 echo '</p>';
 echo '<p><input type="submit" id="settingssubmit" value="' .
         get_string('getreport', 'report_questioninstances') . '" /></p>';
index c22c8f176b42f03ee7ec64b47741a0ffcc3e8fe1..e449c95a68d0b2056cc9b0d5bb2b40deb6b5593a 100755 (executable)
               </label></p>
 
               <p><label for="extendperiod"><?php print_string('enrolperiod') ?></label><br />
-              <?php choose_from_menu($periodmenu, 'extendperiod', $defaultperiod, $unlimitedperiod); ?></p>
+              <?php echo $OUTPUT->select(html_select::make($periodmenu, 'extendperiod', $defaultperiod, $unlimitedperiod)); ?></p>
 
               <p><label for="extendbase"><?php print_string('startingfrom') ?></label><br />
-              <?php choose_from_menu($basemenu, 'extendbase', $extendbase, ''); ?></p>
+              <?php echo $OUTPUT->select(html_select::make($basemenu, 'extendbase', $extendbase, false)); ?></p>
               <?php print_collapsible_region_end(); ?>
           </div>
 
index 792be3633e756cf69e814ebee0b857da0b9ea5cd..19740f25aa47b49a30d42237f12ebf3494b9161b 100644 (file)
@@ -630,12 +630,13 @@ class define_role_table_advanced extends capability_table_with_risks {
     }
 
     protected function get_legacy_type_field($id) {
+        global $OUTPUT;
         $options = array();
         $options[''] = get_string('none');
         foreach($this->legacyroles as $type => $cap) {
             $options[$type] = get_string('legacy:'.$type, 'role');
         }
-        return choose_from_menu($options, 'legacytype', $this->role->legacytype, '', '', 0, true);
+        return $OUTPUT->select(html_select::make($options, 'legacytype', $this->role->legacytype, false));
     }
 
     protected function get_assignable_levels_control() {
@@ -1496,4 +1497,4 @@ function roles_get_potential_user_selector($context, $name, $options) {
     return $potentialuserselector;
 }
 
-?>
\ No newline at end of file
+?>
index 5b04706da233c02f8402d190fe9f0b694119b2cb..57f68a2e72dba0ee9fa1eaab221252cd7b48b81b 100644 (file)
@@ -39,7 +39,9 @@
 
     echo '<center><form action="timezone.php" method="post">';
     echo "$strusers ($strall): ";
-    choose_from_menu ($timezones, "zone", $current, get_string("serverlocaltime"), "", "99");
+    $select = html_select::make($timezones, "zone", $current, get_string("serverlocaltime"));
+    $select->nothingvalue ="99";
+    echo $OUTPUT->select($select);
     echo "<input type=\"hidden\" name=\"sesskey\" value=\"".sesskey()."\" />";
     echo '<input type="submit" value="'.s($strsavechanges).'" />';
     echo "</form></center>";
index 24791b0d86d5d2920d66703e461668d8fc054378..1711946239e8ba774506684dda7405da78bb22bb 100644 (file)
@@ -62,7 +62,7 @@ class edit_field extends XMLDBAction {
         $this->does_generate = ACTION_GENERATE_HTML;
 
     /// These are always here
-        global $CFG, $XMLDB;
+        global $CFG, $XMLDB, $OUTPUT;
 
     /// Do the job, setting result as needed
     /// Get the dir containing the file
@@ -143,8 +143,10 @@ class edit_field extends XMLDBAction {
         if ($field->getType() != XMLDB_TYPE_DATETIME) {
             unset ($typeoptions[XMLDB_TYPE_DATETIME]);
         }
+        $select = html_select::make($typeoptions, 'type', $field->getType(), false);
+        $select->nothingvalue = '';
         $o.= '      <tr valign="top"><td><label for="menutype" accesskey="t">Type:</label></td>';
-        $o.= '        <td colspan="2">' . choose_from_menu($typeoptions, 'type', $field->getType(), '', '', '', true) . '</td></tr>';
+        $o.= '        <td colspan="2">' . $OUTPUT->select($select) . '</td></tr>';
     /// xmldb_field Length
         $o.= '      <tr valign="top"><td><label for="length" accesskey="l">Length:</label></td>';
         $o.= '        <td colspan="2"><input name="length" type="text" size="6" maxlength="6" id="length" value="' . s($field->getLength()) . '" /><span id="lengthtip"></span></td></tr>';
@@ -153,16 +155,22 @@ class edit_field extends XMLDBAction {
         $o.= '        <td colspan="2"><input name="decimals" type="text" size="6" maxlength="6" id="decimals" value="' . s($field->getDecimals()) . '" /><span id="decimalstip"></span></td></tr>';
     /// xmldb_field Unsigned
         $unsignedoptions = array (0 => 'signed', 1 => 'unsigned');
+        $select = html_select::make($unsignedoptions, 'unsigned', $field->getUnsigned(), false);
+        $select->nothingvalue = '';
         $o.= '      <tr valign="top"><td><label for="menuunsigned" accesskey="u">Unsigned:</label></td>';
-        $o.= '        <td colspan="2">' . choose_from_menu($unsignedoptions, 'unsigned', $field->getUnsigned(), '', '', '', true) . '</td></tr>';
+        $o.= '        <td colspan="2">' . $OUTPUT->select($select) . '</td></tr>';
     /// xmldb_field NotNull
         $notnulloptions = array (0 => 'null', 'not null');
+        $select = html_select::make($notnulloptions, 'notnull', $field->getNotNull(), false);
+        $select->nothingvalue = '';
         $o.= '      <tr valign="top"><td><label for="menunotnull" accesskey="n">Not Null:</label></td>';
-        $o.= '        <td colspan="2">' . choose_from_menu($notnulloptions, 'notnull', $field->getNotNull(), '', '', '', true) . '</td></tr>';
+        $o.= '        <td colspan="2">' . $OUTPUT->select($select) . '</td></tr>';
     /// xmldb_field Sequence
+        $select = html_select::make($sequenceoptions, 'sequence', $field->getSequence(), false);
+        $select->nothingvalue = '';
         $sequenceoptions = array (0 => $this->str['no'], 1 => 'auto-numbered');
         $o.= '      <tr valign="top"><td><label for="menusequence" accesskey="s">Sequence:</label></td>';
-        $o.= '        <td colspan="2">' . choose_from_menu($sequenceoptions, 'sequence', $field->getSequence(), '', '', '', true) . '</td></tr>';
+        $o.= '        <td colspan="2">' . $OUTPUT->select($select) . '</td></tr>';
     /// xmldb_field Default
         $o.= '      <tr valign="top"><td><label for="default" accesskey="d">Default:</label></td>';
         $o.= '        <td colspan="2"><input type="text" name="default" size="30" maxlength="80" id="default" value="' . s($field->getDefault()) . '" /></td></tr>';
index 999765db5ae26c3a067f368bf472f060bd97b65b..01f0d17a35f6b401a453cb7863f42ad680f8428c 100644 (file)
@@ -61,7 +61,7 @@ class edit_index extends XMLDBAction {
         $this->does_generate = ACTION_GENERATE_HTML;
 
     /// These are always here
-        global $CFG, $XMLDB;
+        global $CFG, $XMLDB, $OUTPUT;
 
     /// Do the job, setting result as needed
     /// Get the dir containing the file
@@ -119,7 +119,9 @@ class edit_index extends XMLDBAction {
         $typeoptions = array (0 => 'not unique',
                               1 => 'unique');
         $o.= '      <tr valign="top"><td><label for="menuunique" accesskey="t">Type:</label></td>';
-        $o.= '        <td colspan="2">' . choose_from_menu($typeoptions, 'unique', $index->getUnique(), '', '', '', true) . '</td></tr>';
+        $select = html_select::make($typeoptions, 'unique', $index->getUnique(), false);
+        $select->nothingvalue = '';
+        $o.= '        <td colspan="2">' . $OUTPUT->select($select) . '</td></tr>';
     /// xmldb_index Fields
         $o.= '      <tr valign="top"><td><label for="fields" accesskey="f">Fields:</label></td>';
         $o.= '        <td colspan="2"><input name="fields" type="text" size="40" maxlength="80" id="fields" value="' . s(implode(', ', $index->getFields())) . '" /></td></tr>';
index c87ddaeb459d1a71cffb54c4da2745e9729164af..2e60c755f01eccc155ef545070b32ff27c62c279 100644 (file)
@@ -61,7 +61,7 @@ class edit_key extends XMLDBAction {
         $this->does_generate = ACTION_GENERATE_HTML;
 
     /// These are always here
-        global $CFG, $XMLDB;
+        global $CFG, $XMLDB, $OUTPUT;
 
     /// Do the job, setting result as needed
     /// Get the dir containing the file
@@ -124,8 +124,11 @@ class edit_key extends XMLDBAction {
     /// if ($key->getType() != XMLDB_KEY_FOREIGN_UNIQUE) {
     ///     unset ($typeoptions[XMLDB_KEY_FOREIGN_UNIQUE);
     /// }
+        $select = html_select::make($typeoptions, 'type', $key->getType(), false);
+        $select->nothingvalue = '';
+
         $o.= '      <tr valign="top"><td><label for="menutype" accesskey="t">Type:</label></td>';
-        $o.= '        <td colspan="2">' . choose_from_menu($typeoptions, 'type', $key->getType(), '', '', '', true) . '</td></tr>';
+        $o.= '        <td colspan="2">' . $OUTPUT->select($select) . '</td></tr>';
     /// xmldb_key Fields
         $o.= '      <tr valign="top"><td><label for="fields" accesskey="f">Fields:</label></td>';
         $o.= '        <td colspan="2"><input name="fields" type="text" size="40" maxlength="80" id="fields" value="' . s(implode(', ', $key->getFields())) . '" /></td></tr>';
index 982d55e0a913c088ad59f3e486e88dad34a14cfc..2cf683859c28770cb81e2f77d0ff3cf23cbd76d1 100644 (file)
@@ -60,7 +60,7 @@ class new_statement extends XMLDBAction {
         $this->does_generate = ACTION_GENERATE_HTML;
 
     /// These are always here
-        global $CFG, $XMLDB, $DB;
+        global $CFG, $XMLDB, $DB, $OUTPUT;
 
     /// Do the job, setting result as needed
     /// Get the dir containing the file
@@ -103,7 +103,7 @@ class new_statement extends XMLDBAction {
             $o.= '    <input type="hidden" name ="action" value="new_statement" />';
             $o.= '    <input type="hidden" name ="postaction" value="edit_statement" />';
             $o.= '    <table id="formelements" class="boxaligncenter" cellpadding="5">';
-            $o.= '      <tr><td><label for="type" accesskey="t">' . $this->str['statementtype'] .' </label>' . choose_from_menu($typeoptions, 'type', '', 'choose', '', 0, true) . '<label for="table" accesskey="a">' . $this->str['statementtable'] . ' </label>' .choose_from_menu($selecttables, 'table', '', 'choose', '', 0, true) . '</td></tr>';
+            $o.= '      <tr><td><label for="type" accesskey="t">' . $this->str['statementtype'] .' </label>' . $OUTPUT->select(html_select::make($typeoptions, 'type')) . '<label for="table" accesskey="a">' . $this->str['statementtable'] . ' </label>' .$OUTPUT->select(html_select::make($selecttables, 'table')) . '</td></tr>';
             $o.= '      <tr><td colspan="2" align="center"><input type="submit" value="' .$this->str['create'] . '" /></td></tr>';
             $o.= '      <tr><td colspan="2" align="center"><a href="index.php?action=edit_xml_file&amp;dir=' . urlencode(str_replace($CFG->dirroot, '', $dirpath)) . '">[' . $this->str['back'] . ']</a></td></tr>';
             $o.= '    </table>';
index ca25a8b3eef5a2e00d40f836df2e29884eab7452..79a69b0b946f601a98f18df203ab86d425a2fa72 100644 (file)
@@ -60,7 +60,7 @@ class new_table_from_mysql extends XMLDBAction {
         $this->does_generate = ACTION_GENERATE_HTML;
 
     /// These are always here
-        global $CFG, $XMLDB, $DB;
+        global $CFG, $XMLDB, $DB, $OUTPUT;
 
     /// Do the job, setting result as needed
     /// Get the dir containing the file
@@ -111,7 +111,7 @@ class new_table_from_mysql extends XMLDBAction {
             $o.= '    <input type="hidden" name ="action" value="new_table_from_mysql" />';
             $o.= '    <input type="hidden" name ="postaction" value="edit_table" />';
             $o.= '    <table id="formelements" class="boxaligncenter" cellpadding="5">';
-            $o.= '      <tr><td><label for="table" accesskey="t">' . $this->str['createtable'] .' </label>' . choose_from_menu($selecttables, 'table', '', 'choose', '', 0, true) . '<label for="after" accesskey="a">' . $this->str['aftertable'] . ' </label>' .choose_from_menu($aftertables, 'after', '', 'choose', '', 0, true) . '</td></tr>';
+            $o.= '      <tr><td><label for="table" accesskey="t">' . $this->str['createtable'] .' </label>' . $OUTPUT->select(html_select::make($selecttables, 'table')) . '<label for="after" accesskey="a">' . $this->str['aftertable'] . ' </label>' .$OUTPUT->select(html_select::make($aftertables, 'after')) . '</td></tr>';
             $o.= '      <tr><td colspan="2" align="center"><input type="submit" value="' .$this->str['create'] . '" /></td></tr>';
             $o.= '      <tr><td colspan="2" align="center"><a href="index.php?action=edit_xml_file&amp;dir=' . urlencode(str_replace($CFG->dirroot, '', $dirpath)) . '">[' . $this->str['back'] . ']</a></td></tr>';
             $o.= '    </table>';
index e04e9ab64ed2d87d2070f2ee3e0a5842f9ea1a4c..5f4c5b0af24f42911456bc871ec03669d0f92e42 100644 (file)
@@ -62,7 +62,7 @@ class view_structure_php extends XMLDBAction {
         $this->does_generate = ACTION_GENERATE_HTML;
 
     /// These are always here
-        global $CFG, $XMLDB;
+        global $CFG, $XMLDB, $OUTPUT;
 
     /// Do the job, setting result as needed
     /// Get the dir containing the file
@@ -115,7 +115,7 @@ class view_structure_php extends XMLDBAction {
         $o.= '    <input type="hidden" name ="dir" value="' . str_replace($CFG->dirroot, '', $dirpath) . '" />';
         $o.= '    <input type="hidden" name ="action" value="view_structure_php" />';
         $o.= '    <table id="formelements" class="boxaligncenter" cellpadding="5">';
-        $o.= '      <tr><td><label for="action" accesskey="c">' . $this->str['selectaction'] .' </label>' . choose_from_menu($popcommands, 'command', $commandparam, '', '', 0, true) . '&nbsp;<label for="table" accesskey="t">' . $this->str['selecttable'] . ' </label>' .choose_from_menu($poptables, 'table', $tableparam, '', '', 0, true) . '</td></tr>';
+        $o.= '      <tr><td><label for="action" accesskey="c">' . $this->str['selectaction'] .' </label>' . $OUTPUT->select(html_select::make($popcommands, 'command', $commandparam, false)) . '&nbsp;<label for="table" accesskey="t">' . $this->str['selecttable'] . ' </label>' .$OUTPUT->select(html_select::make($poptables, 'table', $tableparam, false)) . '</td></tr>';
         $o.= '      <tr><td colspan="2" align="center"><input type="submit" value="' .$this->str['view'] . '" /></td></tr>';
         $o.= '    </table>';
         $o.= '</div></form>';
index e519228805d2e3f28aba734e859eb4da6b881839..b11a7c284d5a9c491d45c5d6c7b5c258e4d55349 100644 (file)
@@ -68,7 +68,7 @@ class view_table_php extends XMLDBAction {
         $this->does_generate = ACTION_GENERATE_HTML;
 
     /// These are always here
-        global $CFG, $XMLDB;
+        global $CFG, $XMLDB, $OUTPUT;
 
     /// Do the job, setting result as needed
     /// Get the dir containing the file
@@ -166,7 +166,7 @@ class view_table_php extends XMLDBAction {
         $o.= '    <input type="hidden" name ="table" value="' . s($tableparam) . '" />';
         $o.= '    <input type="hidden" name ="action" value="view_table_php" />';
         $o.= '    <table id="formelements" class="boxaligncenter" cellpadding="5">';
-        $o.= '      <tr><td><label for="action" accesskey="c">' . $this->str['selectaction'] .' </label>' . choose_from_menu($popcommands, 'command', $commandparam, '', '', 0, true) . '&nbsp;<label for="fieldkeyindex" accesskey="f">' . $this->str['selectfieldkeyindex'] . ' </label>' .choose_from_menu($popfields, 'fieldkeyindex', $origfieldkeyindexparam, '', '', 0, true) . '</td></tr>';
+        $o.= '      <tr><td><label for="action" accesskey="c">' . $this->str['selectaction'] .' </label>' . $OUTPUT->select(html_select::make($popcommands, 'command', $commandparam, false)) . '&nbsp;<label for="fieldkeyindex" accesskey="f">' . $this->str['selectfieldkeyindex'] . ' </label>' .$OUTPUT->select(html_select::make($popfields, 'fieldkeyindex', $origfieldkeyindexparam, false)) . '</td></tr>';
         $o.= '      <tr><td colspan="2" align="center"><input type="submit" value="' .$this->str['view'] . '" /></td></tr>';
         $o.= '    </table>';
         $o.= '</div></form>';