--- loncom/interface/lonhelper.pm	2006/07/17 16:26:09	1.157
+++ loncom/interface/lonhelper.pm	2006/08/01 15:12:32	1.158
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # .helper XML handler to implement the LON-CAPA helper
 #
-# $Id: lonhelper.pm,v 1.157 2006/07/17 16:26:09 raeburn Exp $
+# $Id: lonhelper.pm,v 1.158 2006/08/01 15:12:32 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -2430,7 +2430,7 @@ sub render {
 
     #   Current personel
 
-    $result .= '<h4>Select Currently Enrolled Students and Active Course Personnel</h4>';
+    $result .= '<h4>'.&mt('Select Currently Enrolled Students and Active Course Personnel').'</h4>';
     $result .= &Apache::lonselstudent::render_student_list( $current_members,
 							    "helpform",
 							    "current", 
@@ -2442,11 +2442,11 @@ sub render {
 
     # If activeonly is not set then we can also give the expired students:
     #
-    if (!$self->{'activeonly'} && ((scalar @$expired_members) > 0)) {
+    if (!$self->{'activeonly'} && ((scalar(@$future_members)) > 0)) {
 
 	# And future.
 
-	$result .= '<h4>Select Future Enrolled Students and Future Course Personnel</h4>';
+	$result .= '<h4>'.&mt('Select Future Enrolled Students and Future Course Personnel').'</h4>';
        
 	$result .= &Apache::lonselstudent::render_student_list( $future_members,
 								"helpform",
@@ -2455,9 +2455,11 @@ sub render {
 								$self->{'multichoice'},
 								$self->{'variable'},
 								0);
+    }
+    if (!$self->{'activeonly'} && ((scalar(@$expired_members)) > 0)) {
 	# Past 
 
-	$result .= '<h4>Select Previously Enrolled Students and Inactive Course Personnel</h4>';
+	$result .= '<h4>'.&mt('Select Previously Enrolled Students and Inactive Course Personnel').'</h4>';
 	$result .= &Apache::lonselstudent::render_student_list($expired_members,
 							       "helpform",
 							       "past",