From 96524089740b9ede71722c2a5e1b37dc1304a7b2 Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Sat, 25 Oct 2003 02:53:02 +0000
Subject: [PATCH] Now uses choose_from_menu() for the format display, and the
 items are now sorted.

The other items could be easily done with choose_from_menu() too  ;-)
---
 mod/glossary/mod.html | 27 +++++++--------------------
 1 file changed, 7 insertions(+), 20 deletions(-)

diff --git a/mod/glossary/mod.html b/mod/glossary/mod.html
index b9da7ff092..922210052e 100644
--- a/mod/glossary/mod.html
+++ b/mod/glossary/mod.html
@@ -195,20 +195,10 @@ if (!$mainglossary or $mainglossary->id == $form->instance ) {
 <TR valign=top>
     <TD align=right><P><B><?php echo get_string("displayformat", "glossary") ?>:</B></P></TD>
     <TD>
-  <select size="1" name="displayformat">
-  <option value="0" <?php
-   if ( $form->displayformat == 0) {
-      echo "selected";
-   }
-  ?>><?php echo get_string("displayformatdefault", "glossary") ?></option>
-  <option value="1" <?php
-   if ( $form->displayformat == 1) {
-      echo "selected";
-   }
-  ?>><?php echo get_string("displayformatcontinuous", "glossary") ?></option>
-
-<?php
-    global $CFG;
+    <?php
+    $formats = array();
+    $formats[0] = get_string("displayformatdefault", "glossary");
+    $formats[0] = get_string("displayformatcontinuous", "glossary");
 
     $basedir = opendir("$CFG->dirroot/mod/glossary/formats");
     while ($dir = readdir($basedir)) {
@@ -222,15 +212,12 @@ if (!$mainglossary or $mainglossary->id == $form->instance ) {
         if ( $pos = strpos($dir, ".") ) {
             $dir = substr($dir, 0, $pos );
             if ($dir != 0 and $dir != 1) {  // excluding basic formats
-                echo "<option value=\"$dir\" ";
-                if ( $form->displayformat == $dir) {
-                    echo "selected ";
-                }
-                echo ">";
-                echo get_string("displayformat$dir", "glossary")."</option>";	
+                $formats[$dir] = get_string("displayformat$dir", "glossary");
             }
         }
     }
+    asort($formats);
+    choose_from_menu ($formats, "displayformat", $form->displayformat, "");
 
 ?>
 
-- 
2.39.5