[LON-CAPA-cvs] cvs: loncom /interface lonstatistics.pm

[bisitz]

bisitz		Wed Mar  6 11:39:00 2013 EDT

  Modified files:              
    /loncom/interface	lonstatistics.pm 
  Log:
  Improvements to "Select One Student" table:
  - Internationalization: Show custom values as they are but translate "none"
  - Removed unneeded additional blank within each link (fullname and username)
  - Punctuation
  
  
Index: loncom/interface/lonstatistics.pm
diff -u loncom/interface/lonstatistics.pm:1.156 loncom/interface/lonstatistics.pm:1.157
--- loncom/interface/lonstatistics.pm:1.156	Thu May  3 11:21:27 2012
+++ loncom/interface/lonstatistics.pm	Wed Mar  6 11:39:00 2013
@@ -1,6 +1,6 @@
 # The LearningOnline Network with CAPA
 #
-# $Id: lonstatistics.pm,v 1.156 2012/05/03 11:21:27 goltermann Exp $
+# $Id: lonstatistics.pm,v 1.157 2013/03/06 11:39:00 bisitz Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -691,7 +691,7 @@
     }
 
     $Str .= '<h2>'.&mt('Select One Student').'</h2>'
-           .'<p>'.&mt("Click on a student's name or username to view their chart").'</p>'
+           .'<p>'.&mt("Click on a student's name or username to view their chart.").'</p>'
            .&Apache::loncommon::start_data_table()
            .&Apache::loncommon::start_data_table_header_row();
     foreach my $field (@Fields) {
@@ -715,12 +715,16 @@
                 $Str .= '&sort='.&escape($env{'form.sort'});
                 $Str .= '&SelectedStudent=';
                 $Str .= &escape($sname).'">';
-                $Str .= $student->{$field}.' ';
+                $Str .= $student->{$field};
                 $Str .= '</a>';
             } elsif ($field eq 'status') {
                 $Str .= &mt($student->{$field});
             } else {
-                $Str .= $student->{$field};
+                if ($student->{$field} eq 'none') {
+                    $Str .= &mt('none')
+                } else {
+                    $Str .= $student->{$field};
+                }
             }
             $Str .= '</td>';
         }