- Solving bug 995:

   Blank aliases caused the autolinking feature to mess the whole thing...

diff --git a/mod/glossary/dynalink.php b/mod/glossary/dynalink.php
index f41df57033149584d48a81635793abc50c447375..b42571a4bd75169209650bac0d9a85aa11fb3a5a 100644 (file)
--- a/mod/glossary/dynalink.php
+++ b/mod/glossary/dynalink.php
@@ -78,7 +78,9 @@
                             if ( $aliases = get_records("glossary_alias","entryid",$concept->id, "alias") ) {
                                 foreach ($aliases as $alias) {
                                     $currentalias = glossary_addslashes($replace,$alias->alias);
-                                    $currentconcept .= "|" . trim($currentalias);
+                                    if ( trim( $currentalias ) != '' ) {
+                                        $currentconcept .= "|" . trim($currentalias);
+                                    }
                                 }
                             }
                         }

diff --git a/mod/glossary/lib.php b/mod/glossary/lib.php
index ee26b9db8632db838d99dba6c941f1c51df93566..6308183a2e6d6fec590f598d45f13ea570a1623a 100644 (file)
--- a/mod/glossary/lib.php
+++ b/mod/glossary/lib.php
@@ -515,7 +515,7 @@ function  glossary_print_entry_aliases($course, $cm, $glossary, $entry,$mode="",
     $return = '';
     if ( $aliases = get_records("glossary_alias","entryid",$entry->id) ) {
         foreach ($aliases as $alias) {
-            if ($alias->alias) {
+            if (trim($alias->alias)) {
                 if ($return == '') {
                     $return = '<select style="font-size:8pt">';
                 }