--- loncom/interface/lonpickcode.pm	2007/10/12 20:08:38	1.13
+++ loncom/interface/lonpickcode.pm	2019/01/31 16:08:06	1.17
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Pick a CODE from the list of possible CODEs
 #
-# $Id: lonpickcode.pm,v 1.13 2007/10/12 20:08:38 banghart Exp $
+# $Id: lonpickcode.pm,v 1.17 2019/01/31 16:08:06 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -39,7 +39,7 @@ sub get_code_freq {
     my ($r)=@_;
     my %codes;
     my %scantron_config=
-	&Apache::grades::get_scantron_config($env{'form.scantron_format'});
+	&Apache::lonnet::get_scantron_config($env{'form.scantron_format'});
     $r->rflush();
     my ($scanlines,$scan_data)=&Apache::grades::scantron_getfile();
     for (my $i=0;$i<=$scanlines->{'count'};$i++) {
@@ -69,7 +69,7 @@ sub handler {
     if  (!($env{'request.course.id'}) && 
 	 (&Apache::lonnet::allowed('usc',$env{'request.course.id'}))) {
 	$r->print(&Apache::loncommon::start_page().
-		  &mt('Access not allowed.').
+		  '<p class="LC_error">'.&mt('Access not allowed.').'</p>'.
 		  &Apache::loncommon::end_page());
         return OK;
     }
@@ -92,37 +92,37 @@ sub code_list {
     # $context = 1 No page header, print enclosing table
     # $context = 2 No page header, no enclosing table
     my $table_head;
-    my $enc_table_start;
-    my $enc_table_close;
-    
+    my $extra_css;
     if (!$context) {
 	$r->print(&Apache::loncommon::start_page("View CODEs",undef,
 						 {'no_nav_bar' => 1}));
-	$table_head = ('<b>'.&mt('Select a set of saved CODEs to view.')."</b>");
+	$table_head = &mt('Select a set of saved CODEs to view.');
     } elsif ($context eq 1) {
-	$table_head = ('<b>'.&mt('Select another set of saved CODEs to view.')."</b>");
-	$enc_table_start = '<table width="100%"><tr><td bgcolor="#777777">';
-	$enc_table_close = '</tr></td></table>';
+	$table_head = &mt('Select another set of saved CODEs to view.');
     } elsif ($context eq 2) {
-        $table_head = ('<b>'.&mt('Select a set of saved CODEs to view.')."</b>");
+        $table_head = &mt('Select a set of saved CODEs to view.');
+	$extra_css = 'LC_scantron_action';
     }
-    $r->print("<form method='POST' action='/adm/pickcode' name='pickcode'>");
-    $r->print($enc_table_start);
-    $r->print('<table width="100%">');
-    $r->print('<tr><td bgcolor="#e6ffff">');
+    $r->print("<form method='post' action='/adm/pickcode' name='pickcode'>");
+    $r->print('
+     '.&Apache::loncommon::start_data_table($extra_css).'
+       '.&Apache::loncommon::start_data_table_header_row());
+    $r->print('<th>');
     $r->print($table_head);
-    $r->print('</td></tr>');
-    $r->print('<tr><td bgcolor="#ffffe6">');
+    $r->print('</th>');
+    $r->print('
+       '.&Apache::loncommon::end_data_table_header_row().'
+       '.&Apache::loncommon::start_data_table_row());
+    $r->print('<td>');
+    $r->print("<input type='submit' name='submit' value='".&mt("View:")."' /> ");
     $r->print(&Apache::grades::scantron_CODElist());
-    $r->print('</td></tr>');
+    $r->print('</td>');
+    $r->print('
+       '.&Apache::loncommon::end_data_table_row().'
+       '.&Apache::loncommon::end_data_table());
     $r->print("<input type='hidden' name='command' value='showcodes' />");
     $r->print("<input type='hidden' name='symb' value='".$env{'form.symb'}."' />");
     $r->print("<input type='hidden' name='url' value='".$env{'form.url'}."' />");
-    $r->print('<tr><td bgcolor="#ffffe6">');
-    $r->print("<input type='submit' name='submit' value='".&mt("View")."' />");
-    $r->print('</td></tr>');
-    $r->print('</table>');
-    $r->print($enc_table_close); # conditional
     $r->print("</form>");
     
 }
@@ -145,10 +145,12 @@ sub show_codes {
 
 sub picking_a_code {
     my ($r)=@_;
-    $r->print(&Apache::loncommon::start_page("Selecting a CODE",undef,
+    my $title = 'Selecting a CODE';
+    $r->print(&Apache::loncommon::start_page($title,undef,
 					     {'no_nav_bar' => 1}));
-    $r->print(<<ENDSCRIPT);
-<script>
+    $r->print('<h1>'.&mt($title).'</h1>');
+
+    $r->print(&Apache::lonhtmlcommon::scripttag("
 function gochoose(newcode) {
     opener.document.$env{'form.form'}.scantron_CODE_selectedvalue.value=newcode;
     var slct=opener.document.$env{'form.form'}.scantron_CODE_resolution;
@@ -158,18 +160,27 @@ function gochoose(newcode) {
     }
     self.close();
 }
-</script>
-ENDSCRIPT
+"));
 
-
-    $r->print("<p>The CODE on the paper is <tt><b>".$env{'form.curCODE'}.
-	      "</b></tt>. Please Select a new one.</p>\n".'<form>');
+    $r->print(
+        '<p>'
+       .&mt('The CODE on the paper is [_1]. Please select a new one.',
+                '<tt><b>'.$env{'form.curCODE'}.'</b></tt>')
+       ."</p>\n".'<form action="">'
+    );
     my %codes=&Apache::grades::get_codes();
     my %code_freq=&get_code_freq($r);
     my $num_matches=length($env{'form.curCODE'});
     for (my $i=$num_matches;$i>=0;$i--) {
-	my $to_print="<p>".&mt("CODEs with [_1] matching letters",$i)."</p>";
-	$to_print.='<table border="1"><tr><td></td><td>CODE</td><td>exams using this CODE</td>';
+	my $to_print =
+            '<p>'.&mt('CODEs with [_1] matching letters:',$i)."</p>\n"
+           .&Apache::loncommon::start_data_table()
+           .&Apache::loncommon::start_data_table_header_row()
+           .'<th>&nbsp;</th>'
+           .'<th>'.&mt('CODE').'</th>'
+           .'<th>'.&mt('Bubblesheet Exams using this CODE').'</th>'
+           .&Apache::loncommon::end_data_table_header_row();
+
 	my $print;
 	foreach my $code (sort(keys(%codes))) {
 	    if (&Apache::grades::num_matches($env{'form.curCODE'},$code) != $i) { next; }
@@ -181,15 +192,17 @@ ENDSCRIPT
 		$count=scalar(@{$code_freq{$code}});
 		$list=' - '.join(', ',@{$code_freq{$code}});
 	    }
-	    $to_print.='<tr><td>'.
-		      '<input type="button" value="'.&mt('Select').
-		      '" onClick="gochoose(\''.$code.'\')" /></td>'.
-		      '<td><tt>'.$code.'</tt></td><td>'.$count.
-		      $list.'</td></tr>';
-	    delete($codes{$code});
-	}
-	$to_print.='</table>';
-	if ($print) { $r->print($to_print); }
+        $to_print .=
+            &Apache::loncommon::start_data_table_row()
+           .'<td><input type="button" value="'.&mt('Select')
+           .'" onclick="gochoose(\''.$code.'\')" /></td>'
+           .'<td><tt>'.$code.'</tt></td>'
+           .'<td>'.$count.$list.'</td>'
+           .&Apache::loncommon::end_data_table_row();
+            delete($codes{$code});
+        }
+        $to_print .= &Apache::loncommon::end_data_table();
+        if ($print) { $r->print($to_print); }
     }
     $r->print('</form>');
 }