--- loncom/interface/loncommon.pm	2005/11/08 02:12:24	1.282
+++ loncom/interface/loncommon.pm	2005/11/08 03:08:15	1.283
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # a pile of common routines
 #
-# $Id: loncommon.pm,v 1.282 2005/11/08 02:12:24 albertel Exp $
+# $Id: loncommon.pm,v 1.283 2005/11/08 03:08:15 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -1168,26 +1168,27 @@ Args:
   $name - name of the <select> element
   $value - sclara or array ref of values that should already be selected
   $size - number of rows long the select element is
-  %hash - the elements should be 'option' => 'shown text'
+  $hash - the elements should be 'option' => 'shown text'
           (shown text should already have been &mt())
+
 =cut
 
 #-------------------------------------------
 sub multiple_select_form {
-    my ($name,$value,$size,%hash)=@_;
+    my ($name,$value,$size,$hash)=@_;
     my %selected = map { $_ => 1 } ref($value)?@{$value}:($value);
     my $output='';
     if (! defined($size)) {
         $size = 4;
-        if (scalar(keys(%hash))<4) {
-            $size = scalar(keys(%hash));
+        if (scalar(keys(%$hash))<4) {
+            $size = scalar(keys(%$hash));
         }
     }
     $output.="\n<select name='$name' size='$size' multiple='1'>";
-    foreach (sort(keys(%hash))) {
+    foreach (sort(keys(%$hash))) {
         $output.='<option value="'.$_.'" ';
         $output.='selected="selected" ' if ($selected{$_});
-        $output.='>'.$hash{$_}."</option>\n";
+        $output.='>'.$hash->{$_}."</option>\n";
     }
     $output.="</select>\n";
     return $output;
@@ -2529,7 +2530,7 @@ sub pgrdlink {
 Inputs: $text $uname $udom $symb $target
 
 Returns: A link to parmset.pm such as to see the PPRM view of a
-student andn resource
+student and a specific resource
 
 =cut