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

[raeburn]

raeburn		Thu Nov 30 01:49:20 2017 EDT

  Modified files:              
    /loncom/interface	domainprefs.pm 
  Log:
  - Remove unused scalar.
  - Add a space before list of custom LTI fields for an external tool.
  - Replace erroneous display="" attribute with correct style="" attribute
    in a div tag.
  
  
Index: loncom/interface/domainprefs.pm
diff -u loncom/interface/domainprefs.pm:1.316 loncom/interface/domainprefs.pm:1.317
--- loncom/interface/domainprefs.pm:1.316	Sun Nov 12 19:46:30 2017
+++ loncom/interface/domainprefs.pm	Thu Nov 30 01:49:19 2017
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to set domain-wide configuration settings
 #
-# $Id: domainprefs.pm,v 1.316 2017/11/12 19:46:30 raeburn Exp $
+# $Id: domainprefs.pm,v 1.317 2017/11/30 01:49:19 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -3885,7 +3885,7 @@
         for (my $i=0; $i<@items; $i++) {
             $css_class = $itemcount%2?' class="LC_odd_row"':'';
             my $item = $ordered{$items[$i]};
-            my ($title,$key,$secret,$url,$imgsrc,$version);
+            my ($title,$key,$secret,$url,$imgsrc);
             if (ref($settings->{$item}) eq 'HASH') {
                 $title = $settings->{$item}->{'title'};
                 $url = $settings->{$item}->{'url'};
@@ -4330,7 +4330,7 @@
                     if ($checked) {
                         $show = 'block';
                     }
-                    $additional = '<div id="cloneinstcode" style="display:'.$show.'" />'.
+                    $additional = '<div id="cloneinstcode" style="display:'.$show.';" />'.
                                   &mt('Institutional codes for new and cloned course have identical:').
                                   '<br />';
                     foreach my $item (@code_order) {
@@ -5645,7 +5645,7 @@
     }
     my $space;
     if ($islast && $num == 1) {
-        $space = '<div display="inline-block"> </div>';
+        $space = '<div style="display:inline-block;"> </div>';
     }
     my $output =
         '<tr class="'.$css_class.'" id="balanceruletr_'.$balnum.'_'.$num.'"><td style="vertical-align: top">'.$space.
@@ -10529,7 +10529,7 @@
                             } 
                         }
                         if ($customlist) {
-                            $resulttext .= '<li>'.&mt('Custom items').':'.$customlist.'</li>';
+                            $resulttext .= '<li>'.&mt('Custom items').': '.$customlist.'</li>';
                         }
                     } 
                     $resulttext .= '</ul></li>';