[LON-CAPA-cvs] cvs: loncom /interface lonhelper.pm

[bowersj2]

bowersj2		Mon Jun 23 16:39:18 2003 EDT

  Modified files:              
    /loncom/interface	lonhelper.pm 
  Log:
  Should fix 1867.
  
  
Index: loncom/interface/lonhelper.pm
diff -u loncom/interface/lonhelper.pm:1.39 loncom/interface/lonhelper.pm:1.40
--- loncom/interface/lonhelper.pm:1.39	Thu Jun 19 15:28:52 2003
+++ loncom/interface/lonhelper.pm	Mon Jun 23 16:39:18 2003
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # .helper XML handler to implement the LON-CAPA helper
 #
-# $Id: lonhelper.pm,v 1.39 2003/06/19 19:28:52 bowersj2 Exp $
+# $Id: lonhelper.pm,v 1.40 2003/06/23 20:39:18 bowersj2 Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -2654,17 +2654,19 @@
         }
     }
 
-    if (scalar(@results) == 0) {
-        return '';
-    }
-
-    my $result = "<ul>\n";
-    for my $re (@results) {
-        $result .= '    <li>' . $re . "</li>\n";
-    }
+    my $result;
 
-    if (!@results) {
-        $result .= '    <li>No changes were made to current settings.</li>';
+    if (scalar(@results) != 0) {
+	$result .= "<ul>\n";
+	for my $re (@results) {
+	    $result .= '    <li>' . $re . "</li>\n";
+	}
+	
+	if (!@results) {
+	    $result .= '    <li>No changes were made to current settings.</li>';
+	}
+	
+	$result .= '</ul>';
     }
 
     if ($self->{'restartCourse'}) {
@@ -2678,7 +2680,7 @@
             "</form></center>";
     }
 
-    return $result . '</ul>';
+    return $result;
 }
 
 sub overrideForm {