--- loncom/interface/domainprefs.pm	2007/04/04 00:10:01	1.7
+++ loncom/interface/domainprefs.pm	2007/05/02 01:33:48	1.13
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to set domain-wide configuration settings
 #
-# $Id: domainprefs.pm,v 1.7 2007/04/04 00:10:01 albertel Exp $
+# $Id: domainprefs.pm,v 1.13 2007/05/02 01:33:48 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -38,6 +38,7 @@ use Apache::lonhtmlcommon();
 use Apache::lonlocal;
 use LONCAPA();
 use LONCAPA::Enrollment;
+use File::Copy;
 
 sub handler {
     my $r=shift;
@@ -111,6 +112,7 @@ sub handler {
     &Apache::lonhtmlcommon::add_breadcrumb
     ({href=>"javascript:changePage(document.$phase,'display')",
       text=>"Domain Configuration"});
+    my $confname = $dom.'-domainconfig';
     if ($phase eq 'process') {
         &Apache::lonhtmlcommon::add_breadcrumb
           ({href=>"javascript:changePage(document.$phase,'$phase')",
@@ -118,7 +120,8 @@ sub handler {
         &print_header($r,$phase);
         foreach my $item (@prefs) {
             $r->print('<h3>'.&mt($item->{'text'}).'</h3>'.
-                   &process_changes($r,$dom,$item->{'action'},\@roles,%domconfig));
+                      &process_changes($r,$dom,$confname,
+                        $item->{'action'},\@roles,%domconfig));
         }
         $r->print('<p>');
         &print_footer($r,$phase,'display','Back to actions menu');
@@ -135,29 +138,26 @@ sub handler {
             if ($item->{'action'} eq 'login') {
                 $r->print('</td><td width="6%">&nbsp;</td><td align="left" valign="top" width="47%">');
             }
-            &print_config_box($r,$dom,$phase,$item->{'action'},
+            &print_config_box($r,$dom,$confname,$phase,$item->{'action'},
                               $item,$domconfig{$item->{'action'}});
         }
         $r->print('
-           </table>
-          </td>
-         </tr>
-        </table>
       </td>
     </tr>
    </table>');
-        &print_footer($r,$phase,'process','Store changes');
+        &print_footer($r,$phase,'process','Save changes');
     }
     return OK;
 }
 
 sub process_changes {
-    my ($r,$dom,$action,$roles,%domconfig) = @_;
+    my ($r,$dom,$confname,$action,$roles,%domconfig) = @_;
     my $output;
     if ($action eq 'login') {
-        $output = &modify_login($r,$dom,%domconfig);
+        $output = &modify_login($r,$dom,$confname,%domconfig);
     } elsif ($action eq 'rolecolors') {
-        $output = &modify_rolecolors($r,$dom,$roles,%domconfig);
+        $output = &modify_rolecolors($r,$dom,$confname,$roles,
+                                     %domconfig);
     } elsif ($action eq 'quotas') {
         $output = &modify_quotas($dom,%domconfig);
     } elsif ($action eq 'autoenroll') {
@@ -169,7 +169,7 @@ sub process_changes {
 }
 
 sub print_config_box {
-    my ($r,$dom,$phase,$action,$item,$settings) = @_;
+    my ($r,$dom,$confname,$phase,$action,$item,$settings) = @_;
     $r->print('
          <table class="LC_nested_outer">
           <tr>
@@ -189,7 +189,7 @@ sub print_config_box {
         if ($action eq 'autoupdate') {
             $r->print(&print_autoupdate('top',$dom,$settings));
         } else {
-            $r->print(&print_rolecolors($phase,'student',$dom,$settings));
+            $r->print(&print_rolecolors($phase,'student',$dom,$confname,$settings));
         }
         $r->print('
            </table>
@@ -205,7 +205,7 @@ sub print_config_box {
         if ($action eq 'autoupdate') {
             $r->print(&print_autoupdate('bottom',$dom,$settings));
         } else {
-            $r->print(&print_rolecolors($phase,'coordinator',$dom,$settings).'
+            $r->print(&print_rolecolors($phase,'coordinator',$dom,$confname,$settings).'
            </table>
           </td>
          </tr>
@@ -216,7 +216,7 @@ sub print_config_box {
               <td class="LC_left_item"'.$colspan.'>'.$item->{'header'}->[2]->{'col1'}.'</td>
               <td class="LC_right_item">'.$item->{'header'}->[2]->{'col2'}.'</td>
              </tr>'.
-            &print_rolecolors($phase,'author',$dom,$settings).'
+            &print_rolecolors($phase,'author',$dom,$confname,$settings).'
            </table>
           </td>
          </tr>
@@ -227,7 +227,7 @@ sub print_config_box {
               <td class="LC_left_item"'.$colspan.'>'.$item->{'header'}->[3]->{'col1'}.'</td>
               <td class="LC_right_item">'.$item->{'header'}->[3]->{'col2'}.'</td>
              </tr>'.
-            &print_rolecolors($phase,'admin',$dom,$settings));
+            &print_rolecolors($phase,'admin',$dom,$confname,$settings));
         }
     } else {
         $r->print('
@@ -246,7 +246,7 @@ sub print_config_box {
               <td class="LC_right_item">'.$item->{'header'}->[0]->{'col2'}.'</td>
              </tr>');
         if ($action eq 'login') {
-           $r->print(&print_login($dom,$phase,$settings));
+           $r->print(&print_login($dom,$confname,$phase,$settings));
         } elsif ($action eq 'quotas') {
            $r->print(&print_quotas($dom,$settings));
         } elsif ($action eq 'autoenroll') {
@@ -277,7 +277,7 @@ function changePage(formname,newphase) {
                                            $js));
     $r->print(&Apache::lonhtmlcommon::breadcrumbs('Domain Settings'));
     $r->print('
-<form name="parmform">
+<form name="parmform" action="">
 <input type="hidden" name="pres_marker" />
 <input type="hidden" name="pres_type" />
 <input type="hidden" name="pres_value" />
@@ -305,7 +305,7 @@ sub print_footer {
 }
 
 sub print_login {
-    my ($dom,$phase,$settings) = @_;
+    my ($dom,$confname,$phase,$settings) = @_;
     my %choices = &login_choices();
     my ($catalogon,$catalogoff,$adminmailon,$adminmailoff);
     $catalogon = ' checked="checked" ';
@@ -360,23 +360,16 @@ sub print_login {
             }
         }
     } else {
-        $designs{'domlogo'} = '';
-        if (-e '/home/httpd/html/adm/lonDomLogos/'.$dom.'.gif') {
-            $designs{'domlogo'} = '/home/httpd/html/adm/lonDomLogos/'.$dom.'.gif';
-            $is_custom{'domlogo'} = 1;
-        }
-        if ($designhash{$dom.'.login.img'} ne '') {
-            $designs{'img'} = $designhash{$dom.'.login.img'};
-            $is_custom{'img'} = 1;
-        }
-        if ($designhash{$dom.'.login.logo'} ne '') {
-            $designs{'logo'} = $designhash{$dom.'.login.logo'};
-            $is_custom{'logo'} = 1;
-        }
         if ($designhash{$dom.'.login.font'} ne '') {
             $designs{'font'} = $designhash{$dom.'.login.font'};
             $is_custom{'font'} = 1;
         }
+        foreach my $item (@images) {
+            if ($designhash{$dom.'.login.'.$item} ne '') {
+                $designs{$item} = $designhash{$dom.'.login.'.$item};
+                $is_custom{$item} = 1;
+            }
+        }
         foreach my $item (@bgs) {
             if ($designhash{$dom.'.login.'.$item} ne '') {
                 $designs{'bgs'}{$item} = $designhash{$dom.'.login.'.$item};
@@ -398,22 +391,22 @@ sub print_login {
     my $datatable = 
         '<tr'.$css_class.'><td colspan="2">'.$choices{'coursecatalog'}.
         '</td><td>'.
-        '<nobr><input type="radio" name="coursecatalog"'.
-        $catalogon.' value="1" />'.&mt('Yes').'&nbsp;'.
-        '<input type="radio" name="coursecatalog"'.
-        $catalogoff.'value="0" />'.&mt('No').'</nobr></td>'.
+        '<span class="LC_nobreak"><label><input type="radio" name="coursecatalog"'.
+        $catalogon.' value="1" />'.&mt('Yes').'</label>&nbsp;'.
+        '<label><input type="radio" name="coursecatalog"'.
+        $catalogoff.'value="0" />'.&mt('No').'</label></span></td>'.
         '</tr>';
     $itemcount ++;
     $css_class = $itemcount%2?' class="LC_odd_row"':'';
     $datatable .= '<tr'.$css_class.'>'.
         '<td colspan="2">'.$choices{'adminmail'}.'</td>'.
-        '<td><nobr>'.
-        '<input type="radio" name="adminmail"'.
-        $adminmailon.' value="1" />'.&mt('Yes').'&nbsp;'.
-        '<input type="radio" name="adminmail"'.
-        $adminmailoff.'value="0" />'.&mt('No').'</nobr></td>';
+        '<td><span class="LC_nobreak">'.
+        '<label><input type="radio" name="adminmail"'.
+        $adminmailon.' value="1" />'.&mt('Yes').'</label>&nbsp;'.
+        '<label><input type="radio" name="adminmail"'.
+        $adminmailoff.'value="0" />'.&mt('No').'</label></span></td></tr>';
     $itemcount ++;
-    $datatable .= &display_color_options($dom,$phase,'login',$itemcount,\%choices,\%is_custom,\%defaults,\%designs,\@images,\@bgs,\@links,\%alt_text);
+    $datatable .= &display_color_options($dom,$confname,$phase,'login',$itemcount,\%choices,\%is_custom,\%defaults,\%designs,\@images,\@bgs,\@links,\%alt_text);
     $datatable .= '</tr></table></td></tr>';
     return $datatable;
 }
@@ -440,7 +433,7 @@ sub login_choices {
 }
 
 sub print_rolecolors {
-    my ($phase,$role,$dom,$settings) = @_;
+    my ($phase,$role,$dom,$confname,$settings) = @_;
     my %choices = &color_font_choices();
     my @bgs = ('pgbg','tabbg','sidebg');
     my @links = ('link','alink','vlink');
@@ -506,15 +499,14 @@ sub print_rolecolors {
         }
     }
     my $itemcount = 1;
-    my $datatable = display_color_options($dom,$phase,$role,$itemcount,\%choices,\%is_custom,\%defaults,\%designs,\@images,\@bgs,\@links,\%alt_text);
+    my $datatable = &display_color_options($dom,$confname,$phase,$role,$itemcount,\%choices,\%is_custom,\%defaults,\%designs,\@images,\@bgs,\@links,\%alt_text);
     $datatable .= '</tr></table></td></tr>';
     return $datatable;
 }
 
 sub display_color_options {
-    my ($dom,$phase,$role,$itemcount,$choices,$is_custom,$defaults,$designs,
+    my ($dom,$confname,$phase,$role,$itemcount,$choices,$is_custom,$defaults,$designs,
         $images,$bgs,$links,$alt_text) = @_;
-    my $configuname = $dom.'-domainconfig';
     my $css_class = $itemcount%2?' class="LC_odd_row"':'';
     my $datatable = '<tr'.$css_class.'>'.
         '<td>'.$choices->{'font'}.'</td>';
@@ -524,14 +516,15 @@ sub display_color_options {
         $datatable .= '<td>&nbsp;</td>';
     }
     my $fontlink = &color_pick($phase,$role,'font',$choices->{'font'},$designs->{'font'});
-    $datatable .= '<td><nobr>'.
+    $datatable .= '<td><span class="LC_nobreak">'.
                   '<input type="text" size="10" name="'.$role.'_font"'.
-                  ' value="'.$designs->{'font'}.'" />&nbsp'.$fontlink.
-                  '</nobr></td></tr>';
+                  ' value="'.$designs->{'font'}.'" />&nbsp;'.$fontlink.
+                  '</span></td></tr>';
+    my $switchserver = &check_switchserver($dom,$confname);
     foreach my $img (@{$images}) {
         $itemcount ++;
         $css_class = $itemcount%2?' class="LC_odd_row"':'';
-        $datatable .= '</tr><tr'.$css_class.'>'.
+        $datatable .= '<tr'.$css_class.'>'.
                       '<td>'.$choices->{$img}.'</td>';
         my $imgfile;
         if ($designs->{$img} ne '') {
@@ -540,8 +533,8 @@ sub display_color_options {
             $imgfile = $defaults->{$img};
         }
         if ($imgfile) {
-            my $showfile;
-            if ($imgfile =~ m-^(/uploaded/\Q$dom\E/\Q$configuname\E/portfolio.*)/([^/]+)$-) {
+            my ($showfile,$fullsize);
+            if ($imgfile =~ m-^(/res/\Q$dom\E/\Q$confname\E/\Q$img\E)/([^/]+)$-) {
                 my $urldir = $1;
                 my $filename = $2;
                 my @info = &Apache::lonnet::stat_file($designs->{$img});
@@ -565,21 +558,27 @@ sub display_color_options {
                     my $input = "/home/httpd/html".$imgfile;
                     my $output = '/home/httpd/html/'.$imgdir.'/tn-'.$filename;
                     if (!-e $output) {
-                        system("convert -sample 200x50 $input $output");
+                        my ($width,$height) = &thumb_dimensions();
+                        my $size = $width.'x'.$height;
+                        system("convert -sample $size $input $output");
                     }
                     $showfile = '/'.$imgdir.'/tn-'.$filename;
                 }
             } 
             if ($showfile) {
+                $showfile = &Apache::loncommon::lonhttpdurl($showfile);
+                $fullsize =  &Apache::loncommon::lonhttpdurl($imgfile);
                 $datatable.= '<td>';
                 if (!$is_custom->{$img}) {
                     $datatable .= &mt('Default in use:').'<br />';
                 }
-                $datatable.= '<img src="'.$showfile.'" alt="'.
-                             $alt_text->{$img}.'" /></td>';
+                $datatable.= '<a href="'.$fullsize.'" target="_blank"><img src="'.
+                             $showfile.'" alt="'.$alt_text->{$img}.
+                             '" border="0" /></a></td>';
                 if ($is_custom->{$img}) {
-                    $datatable.='<td><nobr><input type="checkbox" name="'.$role.'_del_'.$img.'" value="1">'.
-                                &mt('Delete?').'&nbsp;'.&mt('Replace:').'</nobr><br />';
+                    $datatable.='<td><span class="LC_nobreak"><label><input type="checkbox" name="'.
+                                $role.'_del_'.$img.'" value="1" />'.&mt('Delete?').
+                                '</label>&nbsp;'.&mt('Replace:').'</span><br />';
                 } else {
                     $datatable.='<td valign="bottom">'.&mt('Upload:').'<br />';
                 }
@@ -591,7 +590,12 @@ sub display_color_options {
             $datatable .= '<td colspan="2" class="LC_right_item"><br />'.
                           &mt('Upload:');
         }
-        $datatable .= '&nbsp;<input type="file" name="'.$role.'_'.$img.'" /></nobr></td></tr>';
+        if ($switchserver) {
+            $datatable .= &mt('Upload to library server: [_1]',$switchserver);
+        } else {
+            $datatable .='&nbsp;<input type="file" name="'.$role.'_'.$img.'" />';
+        }
+        $datatable .= '</td></tr>';
     }
     $itemcount ++;
     $css_class = $itemcount%2?' class="LC_odd_row"':'';
@@ -604,7 +608,7 @@ sub display_color_options {
         }
     }
     if ($bgs_def) {
-        $datatable .= '<td>'.&mt('Default(s) in use').'<br /><table border="0"><tr>'.$bgs_def.'</tr></table></td>';
+        $datatable .= '<td>'.&mt('Default(s) in use:').'<br /><table border="0"><tr>'.$bgs_def.'</tr></table></td>';
     } else {
         $datatable .= '<td>&nbsp;</td>';
     }
@@ -631,7 +635,7 @@ sub display_color_options {
         }
     }
     if ($links_def) {
-        $datatable .= '<td>'.&mt('Default(s) in use').'<br /><table border="0"><tr>'.$links_def.'</tr></table></td>';
+        $datatable .= '<td>'.&mt('Default(s) in use:').'<br /><table border="0"><tr>'.$links_def.'</tr></table></td>';
     } else {
         $datatable .= '<td>&nbsp;</td>';
     }
@@ -710,10 +714,10 @@ sub print_quotas {
                 $css_class = $typecount%2?' class="LC_odd_row"':'';
                 $datatable .= '<tr'.$css_class.'>'. 
                               '<td>'.$usertypes->{$type}.'</td>'.
-                              '<td class="LC_right_item"><nobr>'.
+                              '<td class="LC_right_item"><span class="LC_nobreak">'.
                               '<input type="text" name="quota_'.$type.
                               '" value="'.$settings->{$type}.
-                              '" size="5" /> Mb</nobr></td></tr>';
+                              '" size="5" /> Mb</span></td></tr>';
             }
         }
         $othertitle = "Other users";
@@ -728,9 +732,9 @@ sub print_quotas {
     $css_class = $typecount%2?' class="LC_odd_row"':'';
     $datatable .= '<tr'.$css_class.'>'.
                   '<td>'.&mt($othertitle).'</td>'.
-                  '<td class="LC_right_item"><nobr>'.
+                  '<td class="LC_right_item"><span class="LC_nobreak">'.
                   '<input type="text" name="default" value="'.
-                  $defaultquota.'" size="5" /> Mb</nobr></td></tr>';
+                  $defaultquota.'" size="5" /> Mb</span></td></tr>';
     return $datatable;
 }
 
@@ -764,19 +768,19 @@ sub print_autoenroll {
     my $domform = &Apache::loncommon::select_dom_form($defdom,'sender_domain',1);
     my $datatable='<tr class="LC_odd_row">'.
                   '<td>'.&mt('Auto-enrollment active?').'</td>'.
-                  '<td class="LC_right_item"><nobr>'.
-                  '<input type="radio" name="autoenroll_run"'.
-                  $runon.' value="1" />'.&mt('Yes').'&nbsp;'.
+                  '<td class="LC_right_item"><span class="LC_nobreak"><label>'.
                   '<input type="radio" name="autoenroll_run"'.
-                  $runoff.'value="0" />'.&mt('No').'</nobr></td>'.
+                  $runon.' value="1" />'.&mt('Yes').'</label>&nbsp;'.
+                  '<label><input type="radio" name="autoenroll_run"'.
+                  $runoff.'value="0" />'.&mt('No').'</label></span></td>'.
                   '</tr><tr>'.
                   '<td>'.&mt('Notification messages - sender').
-                  '</td><td class="LC_right_item"><nobr>'.
+                  '</td><td class="LC_right_item"><span class="LC_nobreak">'.
                   &mt('username').':&nbsp;'.
                   '<input type="text" name="sender_uname" value="'.
                   $settings->{'sender_uname'}.
                   '" size="10" />&nbsp;&nbsp;'.&mt('domain').
-                  ':&nbsp;'.$domform.'</nobr></td></tr>';
+                  ':&nbsp;'.$domform.'</span></td></tr>';
     return $datatable;
 }
 
@@ -804,18 +808,18 @@ sub print_autoupdate {
                     );
         $datatable = '<tr class="LC_odd_row">'. 
                   '<td>'.&mt($title{'run'}).'</td>'.
-                  '<td class="LC_right_item"><nobr>'.
+                  '<td class="LC_right_item"><span class="LC_nobreak"><label>'.
                   '<input type="radio" name="autoupdate_run"'.
-                  $updateon.' value="1" />'.&mt('Yes').'&nbsp;'.
-                  '<input type="radio" name="autoupdate_run"'.
-                  $updateoff.'value="0" />'.&mt('No').'</nobr></td>'.
+                  $updateon.' value="1" />'.&mt('Yes').'</label>&nbsp;'.
+                  '<label><input type="radio" name="autoupdate_run"'.
+                  $updateoff.'value="0" />'.&mt('No').'</label></span></td>'.
                   '</tr><tr>'.
                   '<td>'.&mt($title{'classlists'}).'</td>'.
-                  '<td class="LC_right_item"><nobr>'.
-                  '<input type="radio" name="classlists"'.
-                  $classlistson.' value="1" />'.&mt('Yes').'&nbsp;'.
-                  '<input type="radio" name="classlists"'.
-                  $classlistsoff.'value="0" />'.&mt('No').'</nobr></td>'.
+                  '<td class="LC_right_item"><span class="LC_nobreak">'.
+                  '<label><input type="radio" name="classlists"'.
+                  $classlistson.' value="1" />'.&mt('Yes').'</label>&nbsp;'.
+                  '<label><input type="radio" name="classlists"'.
+                  $classlistsoff.'value="0" />'.&mt('No').'</label></span></td>'.
                   '</tr>';
     } else {
         my ($usertypes,$order) = &Apache::lonnet::retrieve_inst_usertypes($dom);
@@ -892,10 +896,10 @@ sub usertype_update_row {
                 } else {
                     $datatable .= '<td>';
                 }
-                $datatable .= '<nobr><input type="checkbox" name="updateable_'.
-                              $type.'_'.$fields->[$i].'" '.
-                              '" value="1"'.$check.'/>'.
-                              $fieldtitles->{$fields->[$i]}.'</nobr></td>';
+                $datatable .= '<span class="LC_nobreak"><label>'.
+                              '<input type="checkbox" name="updateable_'.$type.
+                              '_'.$fields->[$i].'" value="1"'.$check.'/>'.
+                              $fieldtitles->{$fields->[$i]}.'</label></span></td>';
             }
             $datatable .= '</tr></table></td></tr>';
         }
@@ -904,14 +908,14 @@ sub usertype_update_row {
 }
 
 sub modify_login {
-    my ($r,$dom,%domconfig) = @_;
+    my ($r,$dom,$confname,%domconfig) = @_;
     my ($resulttext,$errors,$colchgtext,%changes,%colchanges);
     my %title = ( coursecatalog => 'Display course catalog',
                   adminmail => 'Display administrator E-mail address');
     my @offon = ('off','on');
     my %loginhash;
-    ($errors,%colchanges) = &modify_colors($r,$dom,['login'],\%domconfig,
-                                          \%loginhash);
+    ($errors,%colchanges) = &modify_colors($r,$dom,$confname,['login'],
+                                           \%domconfig,\%loginhash);
     $loginhash{login}{coursecatalog} = $env{'form.coursecatalog'};
     $loginhash{login}{adminmail} = $env{'form.adminmail'};
     if (ref($colchanges{'login'}) eq 'HASH') {  
@@ -947,10 +951,11 @@ sub modify_login {
             $resulttext = &mt('No changes made to log-in page settings');
         }
     } else {
-        $resulttext = &mt('An error occurred: [_1]',$putresult);
+        $resulttext = '<span class="LC_error">'.
+	    &mt('An error occurred: [_1]',$putresult).'</span>';
     }
     if ($errors) {
-        $resulttext .= &mt('The following errors occurred: ').'<ul>'.
+        $resulttext .= '<br />'.&mt('The following errors occurred: ').'<ul>'.
                        $errors.'</ul>';
     }
     return $resulttext;
@@ -974,10 +979,10 @@ sub color_font_choices {
 }
 
 sub modify_rolecolors {
-    my ($r,$dom,$roles,%domconfig) = @_;
+    my ($r,$dom,$confname,$roles,%domconfig) = @_;
     my ($resulttext,%rolehash);
     $rolehash{'rolecolors'} = {};
-    my ($errors,%changes) = &modify_colors($r,$dom,$roles,
+    my ($errors,%changes) = &modify_colors($r,$dom,$confname,$roles,
                          $domconfig{'rolecolors'},$rolehash{'rolecolors'});
     my $putresult = &Apache::lonnet::put_dom('configuration',\%rolehash,
                                              $dom);
@@ -989,7 +994,8 @@ sub modify_rolecolors {
             $resulttext = &mt('No changes made to default color schemes');
         }
     } else {
-        $resulttext = &mt('An error occurred: [_1]',$putresult);
+        $resulttext = '<span class="LC_error">'.
+	    &mt('An error occurred: [_1]',$putresult).'</span>';
     }
     if ($errors) {
         $resulttext .= &mt('The following errors occurred: ').'<ul>'.
@@ -999,16 +1005,20 @@ sub modify_rolecolors {
 }
 
 sub modify_colors {
-    my ($r,$dom,$roles,$domconfig,$confhash) = @_;
-    my %changes;
+    my ($r,$dom,$confname,$roles,$domconfig,$confhash) = @_;
+    my (%changes,%choices);
     my @bgs = ('pgbg','mainbg','sidebg');
     my @links = ('link','alink','vlink');
     my @images;
-    my $configuname = $dom.'-domainconfig';
     my $servadm = $r->dir_config('lonAdmEMail');
     my $errors;
     foreach my $role (@{$roles}) {
         if ($role eq 'login') {
+            %choices = &login_choices();
+        } else {
+            %choices = &color_font_choices();
+        }
+        if ($role eq 'login') {
             @images = ('img','logo','domlogo');
         } else {
             @images = ('img');
@@ -1017,34 +1027,61 @@ sub modify_colors {
         foreach my $item (@bgs,@links) {
             $confhash->{$role}{$item} = $env{'form.'.$role.'_'.$item};
         }
-        foreach my $img (@images) { 
+        my ($configuserok,$author_ok,$switchserver,%currroles);
+        my $uhome = &Apache::lonnet::homeserver($confname,$dom,1);
+        ($configuserok,%currroles) = &check_configuser($uhome,$dom,
+                                                       $confname,$servadm);
+        if ($configuserok eq 'ok') {
+            $switchserver = &check_switchserver($dom,$confname);
+            if ($switchserver eq '') {
+                $author_ok = &check_authorstatus($dom,$confname,%currroles);
+            }
+        }
+        my ($width,$height) = &thumb_dimensions();
+        foreach my $img (@images) {
             if ($env{'form.'.$role.'_'.$img.'.filename'} ne '') {
-                my $configuserok; 
-                if (&Apache::lonnet::homeserver($configuname,$dom) eq 'no_host') {
-                    srand( time() ^ ($$ + ($$ << 15))  ); # Seed rand.
-                    my $configpass = &LONCAPA::Enrollment::create_password();
-                    $configuserok = &Apache::lonnet::modifyuser($dom,$configuname,'','internal',$configpass,'','','','','',undef,$servadm);
-                } else {
-                    $configuserok = 'ok';
-                }
+                my $error;
                 if ($configuserok eq 'ok') {
-                    my $result = 
-                      &Apache::lonnet::userfileupload($role.'_'.$img,'',
-                        'portfolio/'.$img,'','','',$configuname,$dom,'200','50');
-                    if ($result =~ m|(^/uploaded/.+)/([^/]+)$|) {
-                        my $urldir = $1;
-                        my $filename = $2; 
-                        my $allowresult = &Apache::lonnet::make_public_indefinitely($result);
-                        if ($allowresult eq 'ok') {
-                            &Apache::lonnet::make_public_indefinitely($urldir.'/tn-'.$filename);
-                            $confhash->{$role}{$img} = $result;
-                            $changes{$role}{$img} = 1;
+                    if ($switchserver) {
+                        $error = &mt("Upload of [_1] image for $role page(s) is not permitted to this server: [_2]",$choices{$img},$switchserver);
+                    } else {
+                        if ($author_ok eq 'ok') {
+                            my ($result,$logourl) = 
+                                &publishlogo($r,'upload',$role.'_'.$img,
+                                           $dom,$confname,$img,$width,$height);
+                            if ($result eq 'ok') {
+                                $confhash->{$role}{$img} = $logourl;
+                                $changes{$role}{'images'}{$img} = 1;
+                            } else {
+                                $error = &mt("Upload of [_1] image for $role page(s) failed because an error occurred publishing the file in RES space. Error was: [_2].",$choices{img},$result);
+                            }
+                        } else {
+                            $error = &mt("Upload of [_1] image for $role page(s) failed because an author role could not be assigned to a Domain Configuation user ([_2]) in domain: [_3].  Error was: [_4].",$choices{$img},$confname,$dom,$author_ok);
                         }
                     }
                 } else {
-                    my $error = &mt("Upload of image [_1] for $role page(s) failed because a Domain Configuation user ([_2]) could not be created in domain: [_3].  Error was: [_4].",$img,$configuname,$dom,$configuserok);
-                   &Apache::lonnet::logthis($error);
-                   $errors .= '<li>'.$error.'</li>';
+                    $error = &mt("Upload of [_1] image for $role page(s) failed because a Domain Configuation user ([_2]) could not be created in domain: [_3].  Error was: [_4].",$choices{$img},$confname,$dom,$configuserok);
+                }
+                if ($error) {
+                    &Apache::lonnet::logthis($error);
+                    $errors .= '<li><span class="LC_error">'.$error.'</span></li>';
+                }
+            } elsif ($domconfig->{$role}{$img} ne '') {
+                if ($domconfig->{$role}{$img} !~ m-^(/res/\Q$dom\E/\Q$confname\E/\Q$img\E)/([^/]+)$-) {
+                    my $error;
+                    if ($configuserok eq 'ok') {
+# is confname an author?
+                        if ($switchserver eq '') {
+                            if ($author_ok eq 'ok') {
+                                my ($result,$logourl) = 
+                               &publishlogo($r,'copy',$domconfig->{$role}{$img},
+                                            $dom,$confname,$img,$width,$height);
+                                if ($result eq 'ok') {
+                                    $confhash->{$role}{$img} = $logourl;
+                                }
+                            }
+                        }
+                    }
                 }
             }
         }
@@ -1054,14 +1091,16 @@ sub modify_colors {
                     if ($domconfig->{$role}{$img} ne '') {
                         if ($env{'form.'.$role.'_del_'.$img}) {
                             $confhash->{$role}{$img} = '';
-                            $changes{$role}{$img} = 1;
+                            $changes{$role}{'images'}{$img} = 1;
                         } else {
-                            $confhash->{$role}{$img} = $domconfig->{$role}{$img};
+                            if ($confhash->{$role}{$img} eq '') {
+                                $confhash->{$role}{$img} = $domconfig->{$role}{$img};
+                            }
                         }
                     } else {
                         if ($env{'form.'.$role.'_del_'.$img}) {
                             $confhash->{$role}{$img} = '';
-                            $changes{$role}{$img} = 1;
+                            $changes{$role}{'images'}{$img} = 1;
                         } 
                     }
                 }  
@@ -1123,7 +1162,7 @@ sub default_change_checker {
     foreach my $img (@{$images}) {
         if ($env{'form.'.$role.'_del_'.$img}) {
             $confhash->{$role}{$img} = '';
-            $changes->{$role}{$img} = 1;
+            $changes->{$role}{'images'}{$img} = 1;
         }
     }
     if ($confhash->{$role}{'font'}) {
@@ -1160,7 +1199,11 @@ sub display_colorchgs {
                         if ($confhash->{$role}{$item} eq '') {
                             $resulttext .= '<li>'.&mt("$choices{$item} set to default").'</li>';
                         } else {
-                            $resulttext .= '<li>'.&mt("$choices{$item} set to [_1]",$confhash->{$role}{$item}).'</li>';
+                            my $newitem = $confhash->{$role}{$item};
+                            if ($key eq 'images') {
+                                $newitem = '<img src="'.$confhash->{$role}{$item}.'" alt="'.$choices{$item}.'" valign="bottom" />';
+                            }
+                            $resulttext .= '<li>'.&mt("$choices{$item} set to [_1]",$newitem).'</li>';
                         }
                     }
                     if ($role ne 'login') {
@@ -1182,6 +1225,253 @@ sub display_colorchgs {
     return $resulttext;
 }
 
+sub thumb_dimensions {
+    return ('200','50');
+}
+
+sub check_configuser {
+    my ($uhome,$dom,$confname,$servadm) = @_;
+    my ($configuserok,%currroles);
+    if ($uhome eq 'no_host') {
+        srand( time() ^ ($$ + ($$ << 15))  ); # Seed rand.
+        my $configpass = &LONCAPA::Enrollment::create_password();
+        $configuserok = 
+            &Apache::lonnet::modifyuser($dom,$confname,'','internal',
+                             $configpass,'','','','','',undef,$servadm);
+    } else {
+        $configuserok = 'ok';
+        %currroles = 
+            &Apache::lonnet::get_my_roles($confname,$dom,'userroles');
+    }
+    return ($configuserok,%currroles);
+}
+
+sub check_authorstatus {
+    my ($dom,$confname,%currroles) = @_;
+    my $author_ok;
+    if (!$currroles{':'.$dom.':au'}) {
+        my $start = time;
+        my $end = 0;
+        $author_ok = 
+            &Apache::lonnet::assignrole($dom,$confname,'/'.$dom.'/',
+                                        'au',$end,$start);
+    } else {
+        $author_ok = 'ok';
+    }
+    return $author_ok;
+}
+
+sub publishlogo {
+    my ($r,$action,$formname,$dom,$confname,$subdir,$thumbwidth,$thumbheight) = @_;
+    my ($output,$fname,$logourl);
+    if ($action eq 'upload') {
+        $fname=$env{'form.'.$formname.'.filename'};
+        chop($env{'form.'.$formname});
+    } else {
+        ($fname) = ($formname =~ /([^\/]+)$/);
+    }
+    $fname=&Apache::lonnet::clean_filename($fname);
+# See if there is anything left
+    unless ($fname) { return ('error: no uploaded file'); }
+    $fname="$subdir/$fname";
+    my $filepath='/home/'.$confname.'/public_html';
+    my ($fnamepath,$file,$fetchthumb);
+    $file=$fname;
+    if ($fname=~m|/|) {
+        ($fnamepath,$file) = ($fname =~ m|^(.*)/([^/]+)$|);
+    }
+    my @parts=split(/\//,$filepath.'/'.$fnamepath);
+    my $count;
+    for ($count=4;$count<=$#parts;$count++) {
+        $filepath.="/$parts[$count]";
+        if ((-e $filepath)!=1) {
+            mkdir($filepath,02770);
+        }
+    }
+    # Check for bad extension and disallow upload
+    if ($file=~/\.(\w+)$/ &&
+        (&Apache::loncommon::fileembstyle($1) eq 'hdn')) {
+        $output = 
+            &mt('Invalid file extension ([_1]) - reserved for LONCAPA use.',$1); 
+    } elsif ($file=~/\.(\w+)$/ &&
+        !defined(&Apache::loncommon::fileembstyle($1))) {
+        $output = &mt('Unrecognized file extension ([_1]) - rename the file with a proper extension and re-upload.',$1);
+    } elsif ($file=~/\.(\d+)\.(\w+)$/) {
+        $output = &mt('File name not allowed a rename the file to remove the number immediately before the file extension([_1]) and re-upload.',$2);
+    } elsif (-d "$filepath/$file") {
+        $output = &mt('File name is a directory name - rename the file and re-upload');
+    } else {
+        my $source = $filepath.'/'.$file;
+        my $logfile;
+        if (!open($logfile,">>$source".'.log')) {
+            return (&mt('No write permission to Construction Space'));
+        }
+        print $logfile
+"\n================= Publish ".localtime()." ================\n".
+$env{'user.name'}.':'.$env{'user.domain'}."\n";
+# Save the file
+        if (!open(FH,'>'.$source)) {
+            &Apache::lonnet::logthis('Failed to create '.$source);
+            return (&mt('Failed to create file'));
+        }
+        if ($action eq 'upload') {
+            if (!print FH ($env{'form.'.$formname})) {
+                &Apache::lonnet::logthis('Failed to write to '.$source);
+                return (&mt('Failed to write file'));
+            }
+        } else {
+            my $original = &Apache::lonnet::filelocation('',$formname);
+            if(!copy($original,$source)) {
+                &Apache::lonnet::logthis('Failed to copy '.$original.' to '.$source);
+                return (&mt('Failed to write file'));
+            }
+        }
+        close(FH);
+        chmod(0660, $source); # Permissions to rw-rw---.
+
+        my $docroot=$r->dir_config('lonDocRoot');
+        my $targetdir=$docroot.'/res/'.$dom.'/'.$confname .'/'.$fnamepath;
+        my $copyfile=$targetdir.'/'.$file;
+
+        my @parts=split(/\//,$targetdir);
+        my $path="/$parts[1]/$parts[2]/$parts[3]/$parts[4]";
+        for (my $count=5;$count<=$#parts;$count++) {
+            $path.="/$parts[$count]";
+            if (!-e $path) {
+                print $logfile "\nCreating directory ".$path;
+                mkdir($path,02770);
+            }
+        }
+        my $versionresult;
+        if (-e $copyfile) {
+            $versionresult = &logo_versioning($targetdir,$file,$logfile);
+        } else {
+            $versionresult = 'ok';
+        }
+        if ($versionresult eq 'ok') {
+            if (copy($source,$copyfile)) {
+                print $logfile "\nCopied original source to ".$copyfile."\n";
+                $output = 'ok';
+                &write_metadata($dom,$confname,$formname,$targetdir,$file,$logfile);
+                $logourl = '/res/'.$dom.'/'.$confname.'/'.$fname;
+            } else {
+                print $logfile "\nUnable to write ".$copyfile.':'.$!."\n";
+                $output = &mt('Failed to copy file to RES space').", $!";
+            }
+            if (($thumbwidth =~ /^\d+$/) && ($thumbheight =~ /^\d+$/)) {
+                my $inputfile = $filepath.'/'.$file;
+                my $outfile = $filepath.'/'.'tn-'.$file;
+                my $thumbsize = $thumbwidth.'x'.$thumbheight;
+                system("convert -sample $thumbsize $inputfile $outfile");
+                chmod(0660, $filepath.'/tn-'.$file);
+                if (-e $outfile) {
+                    my $copyfile=$targetdir.'/tn-'.$file;
+                    if (copy($outfile,$copyfile)) {
+                        print $logfile "\nCopied source to ".$copyfile."\n";
+                        &write_metadata($dom,$confname,$formname,$targetdir,
+                                        'tn-'.$file,$logfile);
+                    } else {
+                        print $logfile "\nUnable to write ".$copyfile.':'.$!."\n";
+                    }
+                }
+            }
+        } else {
+            $output = $versionresult;
+        }
+    }
+    return ($output,$logourl);
+}
+
+sub logo_versioning {
+    my ($targetdir,$file,$logfile) = @_;
+    my $target = $targetdir.'/'.$file;
+    my ($maxversion,$fn,$extn,$output);
+    $maxversion = 0;
+    if ($file =~ /^(.+)\.(\w+)$/) {
+        $fn=$1;
+        $extn=$2;
+    }
+    opendir(DIR,$targetdir);
+    while (my $filename=readdir(DIR)) {
+        if ($filename=~/\Q$fn\E\.(\d+)\.\Q$extn\E$/) {
+            $maxversion=($1>$maxversion)?$1:$maxversion;
+        }
+    }
+    $maxversion++;
+    print $logfile "\nCreating old version ".$maxversion."\n";
+    my $copyfile=$targetdir.'/'.$fn.'.'.$maxversion.'.'.$extn;
+    if (copy($target,$copyfile)) {
+        print $logfile "Copied old target to ".$copyfile."\n";
+        $copyfile=$copyfile.'.meta';
+        if (copy($target.'.meta',$copyfile)) {
+            print $logfile "Copied old target metadata to ".$copyfile."\n";
+            $output = 'ok';
+        } else {
+            print $logfile "Unable to write metadata ".$copyfile.':'.$!."\n";
+            $output = &mt('Failed to copy old meta').", $!, ";
+        }
+    } else {
+        print $logfile "Unable to write ".$copyfile.':'.$!."\n";
+        $output = &mt('Failed to copy old target').", $!, ";
+    }
+    return $output;
+}
+
+sub write_metadata {
+    my ($dom,$confname,$formname,$targetdir,$file,$logfile) = @_;
+    my (%metadatafields,%metadatakeys,$output);
+    $metadatafields{'title'}=$formname;
+    $metadatafields{'creationdate'}=time;
+    $metadatafields{'lastrevisiondate'}=time;
+    $metadatafields{'copyright'}='public';
+    $metadatafields{'modifyinguser'}=$env{'user.name'}.':'.
+                                         $env{'user.domain'};
+    $metadatafields{'authorspace'}=$confname.':'.$dom;
+    $metadatafields{'domain'}=$dom;
+    {
+        print $logfile "\nWrite metadata file for ".$targetdir.'/'.$file;
+        my $mfh;
+        unless (open($mfh,'>'.$targetdir.'/'.$file.'.meta')) {
+            $output = &mt('Could not write metadata');
+        }
+        foreach (sort keys %metadatafields) {
+            unless ($_=~/\./) {
+                my $unikey=$_;
+                $unikey=~/^([A-Za-z]+)/;
+                my $tag=$1;
+                $tag=~tr/A-Z/a-z/;
+                print $mfh "\n\<$tag";
+                foreach (split(/\,/,$metadatakeys{$unikey})) {
+                    my $value=$metadatafields{$unikey.'.'.$_};
+                    $value=~s/\"/\'\'/g;
+                    print $mfh ' '.$_.'="'.$value.'"';
+                }
+                print $mfh '>'.
+                    &HTML::Entities::encode($metadatafields{$unikey},'<>&"')
+                        .'</'.$tag.'>';
+            }
+        }
+        $output = 'ok';
+        print $logfile "\nWrote metadata";
+        close($mfh);
+    }
+}
+
+sub check_switchserver {
+    my ($dom,$confname) = @_;
+    my ($allowed,$switchserver);
+    my $home = &Apache::lonnet::homeserver($confname,$dom);
+    if ($home eq 'no_host') {
+        $home = &Apache::lonnet::domain($dom,'primary');
+    }
+    my @ids=&Apache::lonnet::current_machine_ids();
+    foreach my $id (@ids) { if ($id eq $home) { $allowed=1; } }
+    if (!$allowed) {
+	$switchserver='<a href="/adm/switchserver?otherserver='.$home.'&role=dc./'.$dom.'/">'.&mt('Switch Server').'</a>';
+    }
+    return $switchserver;
+}
+
 sub modify_quotas {
     my ($dom,%domconfig) = @_;
     my ($resulttext,%changes);
@@ -1227,7 +1517,8 @@ sub modify_quotas {
             $resulttext = &mt('No changes made to default quotas');
         }
     } else {
-        $resulttext = &mt('An error occurred: [_1]',$putresult);
+        $resulttext = '<span class="LC_error">'.
+	    &mt('An error occurred: [_1]',$putresult).'</span>';
     }
     return $resulttext;
 }
@@ -1291,7 +1582,8 @@ sub modify_autoenroll {
             $resulttext = &mt('No changes made to auto-enrollment settings');
         }
     } else {
-        $resulttext = &mt('An error occurred: [_1]',$putresult);
+        $resulttext = '<span class="LC_error">'.
+	    &mt('An error occurred: [_1]',$putresult).'</span>';
     }
     return $resulttext;
 }
@@ -1410,7 +1702,8 @@ sub modify_autoupdate {
             $resulttext = &mt('No changes made to autoupdates');
         }
     } else {
-        $resulttext = &mt('An error occurred: [_1]',$putresult);
+        $resulttext = '<span class="LC_error">'.
+	    &mt('An error occurred: [_1]',$putresult).'</span>';
     }
     return $resulttext;
 }