--- loncom/interface/courseprefs.pm	2009/05/17 00:54:38	1.4
+++ loncom/interface/courseprefs.pm	2009/09/04 21:42:38	1.7
@@ -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.7 2009/09/04 21:42:38 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' =>
@@ -655,7 +655,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 +1424,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">
 
@@ -1758,7 +1758,7 @@ sub print_discussion {
         'plc.roles.denied' => {
                    text => '<b>'.&mt($itemtext->{'plc.roles.denied'}).'</b>'.
                            &Apache::loncommon::help_open_topic("Course_Disable_Discussion").
-                           &mt('(roles-based)'),
+                           &mt('(role-based)'),
                    input => 'checkbox',
                  },
 
@@ -2076,7 +2076,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 +2411,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 +2433,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>';