--- loncom/interface/courseprefs.pm	2009/05/17 00:54:38	1.4
+++ loncom/interface/courseprefs.pm	2009/12/06 22:17:07	1.8.4.1
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to set configuration settings for a course
 #
-# $Id: courseprefs.pm,v 1.4 2009/05/17 00:54:38 raeburn Exp $
+# $Id: courseprefs.pm,v 1.8.4.1 2009/12/06 22:17:07 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -191,7 +191,7 @@ sub handler {
                           default_xml_style       => 'Default XML Style File',
                           pageseparators          => 'Visibly Separate Items on Pages',
                           disable_receipt_display => 'Disable display of problem receipts',
-                          texengine               => 'Force use of a specific math rendering engine.',
+                          texengine               => 'Force use of a specific math rendering engine',
                           tthoptions              => 'Default set of options to pass to tth/m when converting TeX',
                                   },
                   },
@@ -251,7 +251,7 @@ sub handler {
                     itemtext => {
                         task_messages         => 'Send message to student when clicking Done on Tasks',
                         task_grading          => 'Bridge Task grading by instructors and TAs in sections' ,
-                        suppress_embed_prompt => 'Hi$de upload references prompt if uploading file to portfolio',
+                        suppress_embed_prompt => 'Hide upload references prompt if uploading file to portfolio',
                                 },
                   },
         'other' =>
@@ -554,7 +554,6 @@ sub process_changes {
                                 if ($newvalues{$ext_entry} ne $values->{$stdrole.'.plaintext'}) {
                                     my $dupname = 0;
                                     if ($newvalues{$ext_entry} ne '') {
-                                        my $dupname = 0;
                                         if (grep(/^\Q$newvalues{$ext_entry}\E$/,@replacements)) {
                                             $dupname = 1;
                                             push(@{$disallowed->{'rolenames'}{'replacements'}},$newvalues{$ext_entry});
@@ -655,7 +654,7 @@ sub process_changes {
                             }
                             foreach my $key (keys(%env)) {
                                 if ($key =~ /^form\.\Q$entry\E_($match_username:$match_domain)$/) {
-                                    if ($env{'form.'.$key}) {
+                                    if ($env{$key}) {
                                        my $nothide = $1;
                                        if (!grep(/^\Q$nothide\E$/,@new_nothide)) {
                                            push(@new_nothide,$nothide); 
@@ -1424,7 +1423,7 @@ sub assign_course_categories {
     }
     my $catwin_js;
     if ($hascats) {
-        my $alert = &mt('Use \"Save\" in the main window to save course categories');
+        my $alert = &mt("Use 'Save' in the main window to save course categories");
         $catwin_js = <<ENDSCRIPT;
 <script type="text/javascript">
 
@@ -1641,7 +1640,7 @@ sub user_table {
     } else {
         my $num = 0;
         my @curr = split(/,/,$currvalue);
-        $output .= &Apache::loncommon::start_data_table();
+        $output .= '<table class="LC_nested_outer">';
         my ($currusers);
         foreach my $val (@curr) {
             next if ($val eq '');
@@ -1672,7 +1671,7 @@ sub user_table {
             $num ++;
         }
         if ($num) {
-            $output .= &Apache::loncommon::start_data_table_row().
+            $output .= '<tr>'.
                        '<td align="left"><i>';
             if ($num > 1) {
                 $output .= $lt->{'currone'};
@@ -1681,15 +1680,14 @@ sub user_table {
             }
             $output .= '</i><br />'.
                           '<table>'.$currusers.'</table></td>'.
-                          &Apache::loncommon::end_data_table_row();
+                          '</tr>';
         }
-        $output .= &Apache::loncommon::start_data_table_row().
+        $output .= '<tr>'. 
                   '<td align="left"><span class="LC_nobreak"><i>'.
                   $lt->{'add'}.'</i></span><br />'.
                   &select_recipient($item,$num,$cdom,$sections).
                   '<input type="hidden" name="'.$item.'_total" value="'.$num.'" />'.
-                  '</td>'.&Apache::loncommon::end_data_table_row().
-                  &Apache::loncommon::end_data_table();
+                  '</td></tr></table>';
     }
     return $output;
 }
@@ -1700,9 +1698,8 @@ sub select_recipient {
     my $selectlink =
         &Apache::loncommon::selectstudent_link('display',$item.'_uname_'.$num,
                                                $item.'_udom_'.$num,1);
-    my $css_class = ($num % 2)?'LC_even_row':'LC_odd_row';
     my $output = 
-        '<table><tr class="'.$css_class.'"><td align="center">'.&mt('Username').'<br />'.
+        '<table><tr><td align="center">'.&mt('Username').'<br />'.
         '<input type="text" name="'.$item.'_uname_'.$num.'" value="" /></td>'.
         '<td align="center">'.&mt('Domain').'<br />'.$domform.'</td>';
     if (ref($sections) eq 'ARRAY') {
@@ -1756,9 +1753,10 @@ sub print_discussion {
     }
     my %items = (
         'plc.roles.denied' => {
-                   text => '<b>'.&mt($itemtext->{'plc.roles.denied'}).'</b>'.
-                           &Apache::loncommon::help_open_topic("Course_Disable_Discussion").
-                           &mt('(roles-based)'),
+                   text => '<span class="LC_nobreak"><b>'.
+                           &mt($itemtext->{'plc.roles.denied'}).'</b>'.
+                           &Apache::loncommon::help_open_topic("Course_Disable_Discussion").'</span><br />'.
+                           &mt('(role-based)'),
                    input => 'checkbox',
                  },
 
@@ -1769,8 +1767,9 @@ sub print_discussion {
                  },
 
         'pch.roles.denied'   => {
-                   text => '<b>'.&mt($itemtext->{'pch.roles.denied'}).'</b>'.
-                           &Apache::loncommon::help_open_topic("Course_Disable_Discussion").'<br />'.
+                   text => '<span class="LC_nobreak"><b>'.
+                           &mt($itemtext->{'pch.roles.denied'}).'</b>'.
+                           &Apache::loncommon::help_open_topic("Course_Disable_Discussion").'</span><br />'.
                            &mt('(role-based)'),
                    input => 'checkbox',
                  },
@@ -2076,7 +2075,7 @@ sub print_grading {
                  },
         'receiptalg'  => {
                    text => '<b>'.&mt($itemtext->{'receiptalg'}).'</b><br />'.
-                           &mt('This controls how receipt numbers are generated.'),
+                           &mt('This controls how receipt numbers are generated'),
                    input => 'selectbox',
                    options => {
                                 receipt  => 'receipt',
@@ -2411,7 +2410,9 @@ sub nothidepriv_row {
             }
         }
         if (exists($privileged{$udom}{$uname})) {
-            push(@privusers,$user);
+            unless (grep(/^\Q$user\E$/,@privusers)) {
+                push(@privusers,$user);
+            }
         }
     }
     if (@privusers) {
@@ -2431,8 +2432,8 @@ sub nothidepriv_row {
                           '</td><td align="left">'.
                           '<span class="LC_nobreak"><label>'.
                           '<input type="radio" name="'.$item.'_'.$user.'"'.
-                          $hideon.' value="yes" />'.&mt('Hidden').'</label>&nbsp;'.
-                          '<label><input type="radio" name="'.$item.'_'.$user.'"'.                          $hideoff.' value="" />'.&mt('Shown').'</label></span></td>'.
+                          $hideon.' value="" />'.&mt('Hidden').'</label>&nbsp;'.
+                          '<label><input type="radio" name="'.$item.'_'.$user.'"'.                          $hideoff.' value="yes" />'.&mt('Shown').'</label></span></td>'.
                           '</tr>';
         }
         $datatable .= '</table>';
@@ -2514,11 +2515,11 @@ function getIndexByName(item) {
 </script>
 
 ENDJS
-    $output .= $currstr.&Apache::loncommon::start_data_table();
+    $output .= $currstr.'<table class="LC_nested_outer">';
     if (@curr > 0) {
         for (my $i=0; $i<@curr; $i++) {
             my $pos = $i+1;
-            $output .= &Apache::loncommon::start_data_table_row().
+            $output .= '<tr>'.
                        '<td align="left"><span class="LC_nobreak">'.
                        &position_selector($pos,$i,$maxnum).&mt('Delete:').
                        '<input type="checkbox" name="printfmthdr_del_'.$i.
@@ -2532,19 +2533,19 @@ ENDJS
                            '<input type="textbox" name="printfmthdr_text_'.$i.'"'.
                            ' value="'.$curr[$i].'" size="25" /></td>';
             }
-            $output .= &Apache::loncommon::end_data_table_row();
+            $output .= '</tr>';
         }
     }
     my $pos = $currnum+1; 
-    $output .= &Apache::loncommon::start_data_table_row().
+    $output .= '<tr>'.
                '<td align="left"><span class="LC_nobreak">'.
                &position_selector($pos,$currnum,$maxnum).
                '<b>'.&mt('New').'</b></span></td><td align="left">'.
                &substitution_selector($currnum).'</td>'.
-               &Apache::loncommon::end_data_table_row();
+               '</tr>';
     $pos ++;
     $currnum ++;
-    $output .= &Apache::loncommon::start_data_table_row().
+    $output .= '<tr>'.
                '<td align="left"><span class="LC_nobreak">'.
                &position_selector($pos,$currnum,$maxnum).
                '<b>'.&mt('New').'</b></span></td>'.
@@ -2553,8 +2554,8 @@ ENDJS
                '" value="" size ="25" />'.
                '<input type="hidden" name="printfmthdr_maxnum" value="'.
                 $maxnum.'" /></td>'.
-               &Apache::loncommon::end_data_table_row().
-               &Apache::loncommon::end_data_table(); 
+               '</tr>'.
+               '</table>';
     return $output;
 }