[LON-CAPA-cvs] cvs: loncom /auth lonroles.pm

raeburn raeburn@source.lon-capa.org
Wed, 02 Jun 2010 16:26:21 -0000


raeburn		Wed Jun  2 16:26:21 2010 EDT

  Modified files:              
    /loncom/auth	lonroles.pm 
  Log:
  - More of bug 6294.
   - Eliminate <tr></tr>
  
  
Index: loncom/auth/lonroles.pm
diff -u loncom/auth/lonroles.pm:1.250 loncom/auth/lonroles.pm:1.251
--- loncom/auth/lonroles.pm:1.250	Wed Jun  2 15:41:50 2010
+++ loncom/auth/lonroles.pm	Wed Jun  2 16:26:21 2010
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # User Roles Screen
 #
-# $Id: lonroles.pm,v 1.250 2010/06/02 15:41:50 raeburn Exp $
+# $Id: lonroles.pm,v 1.251 2010/06/02 16:26:21 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -1120,10 +1120,12 @@
                         if (ref($roletext->{$sortrole->{$which}}) eq 'ARRAY') {
                             $output.= &Apache::loncommon::start_data_table_row().
                                       $roletext->{$sortrole->{$which}}->[0].
-                                      &Apache::loncommon::end_data_table_row().
-                                      &Apache::loncommon::continue_data_table_row().
-                                      $roletext->{$sortrole->{$which}}->[1].
                                       &Apache::loncommon::end_data_table_row();
+                            if ($roletext->{$sortrole->{$which}}->[1] ne '') {
+                                $output .= &Apache::loncommon::continue_data_table_row().
+                                           $roletext->{$sortrole->{$which}}->[1].
+                                           &Apache::loncommon::end_data_table_row();
+                            }
                         }
                         if ($sortrole->{$which} =~ m-dc\./($match_domain)/-) {
                             if (ref($dcroles) eq 'HASH') {