--- loncom/homework/grades.pm	2001/12/07 23:43:46	1.18
+++ loncom/homework/grades.pm	2002/05/03 22:34:25	1.22
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # The LON-CAPA Grading handler
 #
-# $Id: grades.pm,v 1.18 2001/12/07 23:43:46 albertel Exp $
+# $Id: grades.pm,v 1.22 2002/05/03 22:34:25 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -64,7 +64,9 @@ sub listStudents {
   my ($request) = shift;
   my ($cdom,$cnum) = split(/_/,$ENV{'request.course.id'});
   my $chome=$ENV{"course.$ENV{'request.course.id'}.home"};
-  $request->print ("Found $cdom:$cnum:$chome<br />");
+  $request->print ('<h1>Show Student Submissions on Assessment</h1>'.
+ '<table border="1"><tr><th>Username</th><th>Domain</th><th>Name</th><th>&nbsp;</th></tr>'
+ );
   my (%classlist) = &getclasslist($cdom,$cnum,$chome,'0');
   foreach my $student ( sort(@{ $classlist{'allids'} }) ) {
       my ($sname,$sdom) = split(/:/,$student);
@@ -77,22 +79,28 @@ sub listStudents {
 #      my $sfullname = $Apache::lonnet::unescape($nameparts[0]);
 
       if ( $Apache::grades::viewgrades eq 'F' ) {
-	  $request->print('<form action="/adm/grades" method="post">'."\n");
-	  $request->print("$sname @nameparts\n");
+	  $request->print("\n".'<tr>'."<td>$sname</td><td>$sdom</td><td>@nameparts</td><td>".
+			  '<form action="/adm/grades" method="post">');
 	  if ($ENV{'form.url'}) {
-	    $request->print('<input type="hidden" name="url" value="'.$ENV{'form.url'}.'" />'."\n");
+	    $request->print(
+    '<input type="hidden" name="url" value="'.$ENV{'form.url'}.'" />');
 	  }
 	  if ($ENV{'form.symb'}) {
-	    $request->print('<input type="hidden" name="symb" value="'.$ENV{'form.symb'}.'" />'."\n");
+	    $request->print(
+    '<input type="hidden" name="symb" value="'.$ENV{'form.symb'}.'" />');
 	  }
-	  $request->print('<input type="hidden" name="command" value="'.$ENV{'form.command'}.'" />'."\n");
-	  $request->print('<input type="hidden" name="student" value="'.$sname.'" />'."\n");
-	  $request->print('<input type="hidden" name="domain" value="'.$sdom.'" />'."\n");
-	  $request->print('<input type="submit" name="submit" value="View" />'."<br />\n");
-	  $request->print('</form>');
+	  $request->print(
+    '<input type="hidden" name="command" value="'.$ENV{'form.command'}.'" />');
+	  $request->print(
+             '<input type="hidden" name="student" value="'.$sname.'" />');
+	  $request->print(
+             '<input type="hidden" name="domain" value="'.$sdom.'" />');
+	  $request->print(
+                         '<input type="submit" name="submit" value="View" />');
+	  $request->print('</form></td></tr>');
       }
   }
-
+  $request->print('</table>');
 }
 
 
@@ -259,9 +267,9 @@ sub submission {
   my $result="<h2> Submission Record </h2>  $uname:$udom for $url <br />".$answer;
   my $rendered=&Apache::loncommon::get_student_view($symb,$uname,$udom,
 						   $ENV{'request.course.id'});
-  $result.="Student's view of the problem:<br /> $rendered <br /> Correct answer:";
+  $result.="Student's view of the problem:<br /> $rendered <br /> Correct answer:<br />";
 
-  my $answer=&Apache::loncommon::get_student_answers($symb,$uname,$udom,
+  $answer=&Apache::loncommon::get_student_answers($symb,$uname,$udom,
 						  $ENV{'request.course.id'});
   $result.=$answer;
   return $result;
@@ -414,7 +422,7 @@ sub handler {
   } else {
     $Apache::grades::viewgrades=&Apache::lonnet::allowed('vgr',$ENV{'request.course.id'});
     if ($command eq 'submission') {
-      $request->print(&listStudents($request)) if ($ENV{'form.student'} eq '');
+      &listStudents($request) if ($ENV{'form.student'} eq '');
       $request->print(&submission($request)) if ($ENV{'form.student'} ne '');
     } elsif ($command eq 'viewgrades') {
       $request->print(&viewgrades($request));