--- loncom/interface/courseprefs.pm	2015/03/11 00:28:28	1.49.2.17
+++ loncom/interface/courseprefs.pm	2016/11/13 16:20:54	1.49.2.22
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to set configuration settings for a course
 #
-# $Id: courseprefs.pm,v 1.49.2.17 2015/03/11 00:28:28 raeburn Exp $
+# $Id: courseprefs.pm,v 1.49.2.22 2016/11/13 16:20:54 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -137,7 +137,7 @@ whether Domain coordinators who are curr
 advanced course user .db file for a course are to be hidden (e.g.,
 in syllabus, or from course user lists).
 
-Inputs: $cdom,$item,$settings,$crstype
+Inputs: $cdom,$item,$settings,$crstype,$noedit
 where $cdom is course domain, item is nothideprivileged, $settings is
 HASH ref of the current values for nothideprivileged, $crstype is 
 course type (Course or Community). 
@@ -217,6 +217,7 @@ use Apache::lonhtmlcommon();
 use Apache::lonconfigsettings;
 use Apache::lonrelrequtils;
 use Apache::lonparmset;
+use Apache::courseclassifier;
 use Apache::lonlocal;
 use LONCAPA qw(:DEFAULT :match);
 
@@ -234,9 +235,9 @@ sub handler {
     my $cid = $env{'request.course.id'};
     my ($cnum,$cdom,$chome) = &get_course($cid);
     my $crstype = &Apache::loncommon::course_type();
-    my $parm_permission = &Apache::lonnet::allowed('opa',$cid);
+    my ($parm_permission,$allowed) = &get_permission($cid);
     my $navmap = Apache::lonnavmaps::navmap->new();
-    if ($parm_permission && $navmap) {
+    if ($allowed && $navmap) {
         &Apache::loncommon::content_type($r,'text/html');
         $r->send_http_header;
     } else {
@@ -343,12 +344,12 @@ sub handler {
         $phase = $env{'form.phase'};
     }
 
-    if ($phase eq 'categorizecourse') {
+    if (($phase eq 'categorizecourse') && ($parm_permission->{'categorizecourse'})) {
         &assign_course_categories($r,$crstype);
         return OK;
     }
 
-    if ($phase eq 'releaseinfo') {
+    if (($phase eq 'releaseinfo') && ($parm_permission->{'releaseinfo'})) {
         my $loncaparev = $env{'course.'.$cid.'.internal.releaserequired'};
         if ($loncaparev) { 
             if (&display_loncaparev_constraints($r,$navmap,$loncaparev,$crstype)) {
@@ -534,23 +535,52 @@ sub handler {
                               }],
                   },
     );
-    if ($phase eq 'process') {
+    if (($phase eq 'process') && ($parm_permission->{'process'})) {
         my @allitems = &get_allitems(%prefs);
         &Apache::lonconfigsettings::make_changes($r,$cdom,$phase,$context,
                                                  \@prefs_order,\%prefs,\%values,
-                                                  $cnum,undef,\@allitems,'coursepref');
-    } elsif ($phase eq 'display') {
-        my $jscript = &get_jscript($cid,$cdom,$phase,$crstype);
+                                                  $cnum,undef,\@allitems,
+                                                  'coursepref',$parm_permission);
+    } elsif (($phase eq 'display') && ($parm_permission->{'display'})) {
+        my $jscript = &get_jscript($cid,$cdom,$phase,$crstype,\%values);
         my @allitems = &get_allitems(%prefs);
         &Apache::lonconfigsettings::display_settings($r,$cdom,$phase,$context,
-            \@prefs_order,\%prefs,\%values,undef,$jscript,\@allitems,$crstype,'coursepref');
+            \@prefs_order,\%prefs,\%values,undef,$jscript,\@allitems,$crstype,
+            'coursepref',$parm_permission);
     } else {
         &Apache::lonconfigsettings::display_choices($r,$phase,$context,
-                                                    \@prefs_order,\%prefs,'coursepref');
+                                                    \@prefs_order,\%prefs,
+                                                    'coursepref',$parm_permission);
     }
     return OK;
 }
 
+sub get_permission {
+    my ($cid) = @_;
+    my %permission;
+    my $allowed = 0;
+    return (\%permission,$allowed) unless ($cid);
+    if (&Apache::lonnet::allowed('opa',$cid)) {
+        %permission= (
+                       'pickactions'      => 1,
+                       'categorizecourse' => 1,
+                       'releaseinfo'      => 1,
+                       'process'          => 1,
+                       'display'          => 1,
+         );
+    } elsif (&Apache::lonnet::allowed('vpa',$env{'request.course.id'})) {
+        %permission = (
+                       'pickactions'      => 1,
+                       'releaseinfo'      => 1,
+                       'display'          => 1,
+                      );
+    }
+    foreach my $perm (values(%permission)) {
+        if ($perm) { $allowed=1; last; }
+    }
+    return (\%permission,$allowed);
+}
+
 sub get_allitems {
     my (%prefs) = @_;
     my @allitems;
@@ -568,9 +598,15 @@ sub get_allitems {
 }
 
 sub print_config_box {
-    my ($r,$cdom,$phase,$action,$item,$settings,$allitems,$crstype) = @_;
+    my ($r,$cdom,$phase,$action,$item,$settings,$allitems,$crstype,$parm_permission) = @_;
     my $ordered = $item->{'ordered'};
     my $itemtext = $item->{'itemtext'};
+    my $noedit;
+    if (ref($parm_permission) eq 'HASH') {
+        unless ($parm_permission->{'process'}) {
+            $noedit = 1;
+        }
+    }
     my $rowtotal = 0;
     my $output =
          '<table class="LC_nested_outer">
@@ -605,9 +641,9 @@ sub print_config_box {
         }
         $rowtotal ++;
         if ($action eq 'feedback') {
-            $output .= &print_feedback('top',$cdom,$settings,$ordered,$itemtext,\$rowtotal);
+            $output .= &print_feedback('top',$cdom,$settings,$ordered,$itemtext,\$rowtotal,$noedit);
         } elsif ($action eq 'classlists') {
-            $output .= &print_classlists('top',$cdom,$settings,$itemtext,\$rowtotal,$crstype);
+            $output .= &print_classlists('top',$cdom,$settings,$itemtext,\$rowtotal,$crstype,$noedit);
         }
         $output .= '
            </table>
@@ -628,7 +664,7 @@ sub print_config_box {
         $output .= &mt($item->{'header'}->[1]->{'col2'}).'</td>
              </tr>';
         if ($action eq 'classlists') {
-            $output .= &print_classlists('middle',$cdom,$settings,$itemtext,\$rowtotal,$crstype).
+            $output .= &print_classlists('middle',$cdom,$settings,$itemtext,\$rowtotal,$crstype,$noedit).
                        '
             </table>
            </td>
@@ -667,27 +703,27 @@ sub print_config_box {
     }
     $rowtotal ++;
     if ($action eq 'courseinfo') {
-        $output .= &print_courseinfo($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype);
+        $output .= &print_courseinfo($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype,$noedit);
     } elsif ($action eq 'localization') {
-        $output .= &print_localization($cdom,$settings,$ordered,$itemtext,\$rowtotal);
+        $output .= &print_localization($cdom,$settings,$ordered,$itemtext,\$rowtotal,$noedit);
     } elsif ($action eq 'feedback') {
-        $output .= &print_feedback('bottom',$cdom,$settings,$ordered,$itemtext,\$rowtotal);
+        $output .= &print_feedback('bottom',$cdom,$settings,$ordered,$itemtext,\$rowtotal,$noedit);
     } elsif ($action eq 'discussion') {
-        $output .= &print_discussion($cdom,$settings,$ordered,$itemtext,\$rowtotal);
+        $output .= &print_discussion($cdom,$settings,$ordered,$itemtext,\$rowtotal,$noedit);
     } elsif (($action eq 'classlists') || ($action eq 'viewableroster')) {
-        $output .= &print_classlists('bottom',$cdom,$settings,$itemtext,\$rowtotal,$crstype);
+        $output .= &print_classlists('bottom',$cdom,$settings,$itemtext,\$rowtotal,$crstype,$noedit);
     } elsif ($action eq 'appearance') {
-        $output .= &print_appearance($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype);
+        $output .= &print_appearance($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype,$noedit);
     } elsif ($action eq 'grading') {
-        $output .= &print_grading($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype);
+        $output .= &print_grading($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype,$noedit);
     } elsif ($action eq 'printouts') {
-        $output .= &print_printouts($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype);
+        $output .= &print_printouts($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype,$noedit);
     } elsif ($action eq 'spreadsheet') {
-        $output .= &print_spreadsheet($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype);
+        $output .= &print_spreadsheet($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype,$noedit);
     } elsif ($action eq 'bridgetasks') {
-        $output .= &print_bridgetasks($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype);
+        $output .= &print_bridgetasks($cdom,$settings,$ordered,$itemtext,\$rowtotal,$crstype,$noedit);
     } elsif ($action eq 'other') {
-        $output .= &print_other($cdom,$settings,$allitems,\$rowtotal,$crstype);
+        $output .= &print_other($cdom,$settings,$allitems,\$rowtotal,$crstype,$noedit);
     }
     $output .= '
    </table>
@@ -786,7 +822,7 @@ sub process_changes {
                                 my @clonedoms;
                                 if (exists($env{'form.cloners_activate'})) {
                                     my $actnum = $env{'form.cloners_activate'};
-                                    if ($actnum ne '') {
+                                    if ($actnum ne '-1') {
                                         if ($env{'form.cloners_dom_'.$actnum} ne '') {
                                             my $clonedom = $env{'form.cloners_dom_'.$actnum};
                                             if (&check_clone($clonedom,$disallowed) eq 'ok') {
@@ -804,7 +840,7 @@ sub process_changes {
                                             my $clonedom = $env{'form.cloners_dom_'.$i};
                                             if (&check_clone($clonedom,$disallowed) eq 'ok') {
                                                 if (!grep(/^\*:\Q$clonedom\E$/,@clonedoms)) {
-                                                    push (@clonedoms,'*:'.$clonedom);
+                                                    push(@clonedoms,'*:'.$clonedom);
                                                 }
                                             }
                                         }
@@ -850,6 +886,37 @@ sub process_changes {
                                         }
                                     }
                                 }
+                                if (ref($values) eq 'HASH') {
+                                    my (@code_order,%codedefaults);
+                                    &Apache::lonnet::auto_instcode_defaults($cdom,\%codedefaults,
+                                                                            \@code_order);
+                                    my $clonebycode;
+                                    if ($env{'form.cloners_instcode'}) {
+                                        if (@code_order > 0) {
+                                            my @standardnames = &Apache::loncommon::get_standard_codeitems();
+                                            my %local_to_standard;
+                                            for (my $i=0; $i<@code_order; $i++) {
+                                                $local_to_standard{$code_order[$i]} = $standardnames[$i];
+                                            }
+                                            foreach my $item (@code_order) {
+                                                my $key = $local_to_standard{$item};
+                                                if ($key ne '') {
+                                                    if ($env{'form.'.$key}) {
+                                                        $clonebycode .= $key.'='.&escape($env{'form.'.$key}).'&';
+                                                    }
+                                                }
+                                            }
+                                            $clonebycode =~ s/\&$//;
+                                        }
+                                    }
+                                    if ($clonebycode) {
+                                        if ($newvalues{$entry}) {
+                                            $newvalues{$entry} .= ','.$clonebycode;
+                                        } else {
+                                            $newvalues{$entry} = $clonebycode;
+                                        }
+                                    }
+                                }
                             }
                             if (ref($disallowed) eq 'HASH') {
                                 if (ref($disallowed->{'cloners'}) eq 'HASH') {
@@ -1684,12 +1751,93 @@ sub get_course {
 }
 
 sub get_jscript {
-    my ($cid,$cdom,$phase,$crstype) = @_;
+    my ($cid,$cdom,$phase,$crstype,$settings) = @_;
     my ($can_toggle_cat,$can_categorize) = &can_modify_catsettings($cdom,$crstype);
-    my ($jscript,$categorize_js,$loncaparev_js);
+    my ($jscript,$categorize_js,$loncaparev_js,$instcode_js);
     my $stubrowse_js = &Apache::loncommon::studentbrowser_javascript();
     my $browse_js = &Apache::loncommon::browser_and_searcher_javascript('parmset');
     my $cloners_js = &cloners_javascript($phase);
+    my @code_order;
+    if ($crstype ne 'Community') {
+        if (ref($settings) eq 'HASH') {
+            my %codedefaults;
+            &Apache::lonnet::auto_instcode_defaults($cdom,\%codedefaults,
+                                                    \@code_order);
+            if (@code_order > 0) {
+                if ($phase eq 'display') {
+                    my @actions = &Apache::loncommon::get_env_multiple('form.actions');
+                    if (grep(/^courseinfo$/,@actions)) {
+                        if ($settings->{'cloners'} ne '') {
+                            unless ($settings->{'cloners'} eq '*') {
+                                my @cloners = split(/,/,$settings->{'cloners'});
+                                my @standardnames = &Apache::loncommon::get_standard_codeitems();
+                                my %local_to_standard;
+                                for (my $i=0; $i<@code_order; $i++) {
+                                    $local_to_standard{$code_order[$i]} = $standardnames[$i];
+                                }
+                                foreach my $cloner (@cloners) {
+                                    if (($cloner !~ /^\Q*:\E$match_domain$/) &&
+                                        ($cloner !~ /^$match_username\:$match_domain$/)) {
+                                        foreach my $item (split(/\&/,$cloner)) {
+                                            my ($key,$val) = split(/\=/,$item);
+                                            $val = &unescape($val);
+                                            foreach my $code (@code_order) {
+                                                if ($key eq $local_to_standard{$code}) {
+                                                    $env{'form.'.$key} = $val;
+                                                }
+                                            }
+                                        }
+                                    }
+                                }
+                            }
+                        }
+                    }
+                }
+                my @codetitles;
+                (undef,$instcode_js) =
+                    &Apache::courseclassifier::instcode_selectors($cdom,'display',undef,
+                                                                  \@codetitles);
+                $instcode_js .= <<ENDSCRIPT;
+function toggleCloners(callerradio) {
+    if (document.getElementById('LC_cloners_instcode')) {
+        if (callerradio.value == 1) {
+            document.getElementById('LC_cloners_instcode').style.display = "block";
+        } else {
+            uncheckOfficialLists();
+            document.getElementById('LC_cloners_instcode').style.display = "none";
+        }
+    }
+}
+
+if (!document.getElementsByClassName) {
+    function getElementsByClassName(node, classname) {
+        var a = [];
+        var re = new RegExp('(^| )'+classname+'( |$)');
+        var els = node.getElementsByTagName("*");
+        for(var i=0,j=els.length; i<j; i++)
+            if(re.test(els[i].className))a.push(els[i]);
+        return a;
+    }
+}
+
+function uncheckOfficialLists() {
+    var codepicks;
+    if (document.getElementsByClassName) {
+        codepicks = document.getElementsByClassName('LC_cloners_instcodes');
+    } else {
+        codepicks = getElementsByClassName(document.body,'LC_cloners_instcodes');
+    }
+    if (codepicks.length) {
+        for (var i=0; i<codepicks.length; i++) {
+            codepicks[i].selectedIndex = 0;
+        }
+    }
+}
+
+ENDSCRIPT
+            }
+        }
+    }
     if ($can_categorize) {
         $categorize_js = <<ENDSCRIPT;
 function catsbrowser() {
@@ -1732,7 +1880,7 @@ ENDSCRIPT
     $jscript = '<script type="text/javascript" language="Javascript">'."\n".
                '// <![CDATA['."\n".
                $browse_js."\n".$categorize_js."\n".$loncaparev_js."\n".
-               $cloners_js."\n".
+               $cloners_js."\n".$instcode_js.
                $syllabus_js."\n".'//]]>'."\n".
                '</script>'."\n".$stubrowse_js."\n";
     return $jscript;
@@ -1768,7 +1916,7 @@ function update_cloners(caller,num) {
                         if (document.$formname.cloners_activate[i].value == '0') {
                             document.$formname.cloners_activate[i].checked = false;
                         }
-                        if (document.$formname.cloners_activate[i].value == '') {
+                        if (document.$formname.cloners_activate[i].value == '-1') {
                             document.$formname.cloners_activate[i].checked = true;
                         }
                     }
@@ -1821,28 +1969,80 @@ ENDSCRIPT
 
 
 sub print_courseinfo {
-    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype) = @_;
+    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype,$noedit) = @_;
     unless ((ref($settings) eq 'HASH') && (ref($ordered) eq 'ARRAY') && (ref($itemtext) eq 'HASH')) {
         return;
     }
-    my ($cathash,$categoriesform,$autocoowner);
+    my ($cathash,$categoriesform,$autocoowner,$clonedefaults,$disabled);
+    if ($noedit) {
+        $disabled = ' disabled="disabled"';
+    }
     my %domconf = 
-        &Apache::lonnet::get_dom('configuration',['coursecategories','autoenroll'],$cdom);
+        &Apache::lonnet::get_dom('configuration',
+                                 ['coursecategories','autoenroll','coursedefaults'],$cdom);
     if (ref($domconf{'coursecategories'}) eq 'HASH') {
         $cathash = $domconf{'coursecategories'}{'cats'};
         if (ref($cathash) eq 'HASH') {
             $categoriesform = 
                 &Apache::loncommon::assign_categories_table($cathash,
-                                                $settings->{'categories'},$crstype)."\n";
+                                                $settings->{'categories'},$crstype,$disabled)."\n";
         }
     }
     if (ref($domconf{'autoenroll'}) eq 'HASH') {
         $autocoowner = $domconf{'autoenroll'}{'co-owners'};
     }
+    my ($currcanclone,@code_order,$cloner_instcode);
+    my %codedefaults;
+    &Apache::lonnet::auto_instcode_defaults($cdom,\%codedefaults,
+                                            \@code_order);
+    if ($settings->{'cloners'}) {
+        unless ($settings->{'cloners'} eq '*') {
+            my @currclone = split(/,/,$settings->{'cloners'});
+            foreach my $item (@currclone) {
+                unless ($item eq '*') {
+                    if (($item !~ /\:/) && ($item =~ /=/)) {
+                        $cloner_instcode = 1;
+                    }
+                }
+            }
+        }
+    }
+    if (ref($domconf{'coursedefaults'}) eq 'HASH') {
+        my $canclone = $domconf{'coursedefaults'}{'canclone'};
+        if (ref($canclone) eq 'HASH') {
+            if (ref($canclone->{'instcode'}) eq 'ARRAY') {
+                if ($settings->{'internal.coursecode'}) {
+                    my @posscodes;
+                    if (@code_order > 0) {
+                        $currcanclone = 'instcode';
+                        foreach my $item (@{$canclone->{'instcode'}}) {
+                            if (grep(/^\Q$item\E$/,@code_order)) {
+                                push(@posscodes,$item);
+                            }
+                        }
+                        my $codestr = join(' + ',@posscodes);
+                        $clonedefaults = &mt('Default for official courses is to also allow cloning if [_1] match in cloner and cloned.',
+                                             '"<span class="LC_nobreak" style="font-style:italic">'.$codestr.'</span>"').'<br />';
+                    }
+                }
+            }
+        } elsif ($canclone eq 'domain') {
+            $clonedefaults = &mt('Default is for any course requester in [_1] domain to be able to clone.',
+                                 '<span class="LC_nobreak" style="font-style:italic">'.$cdom.'</span>').'<br />';
+            $currcanclone = 'domain';
+        }
+        if ($clonedefaults) {
+            if ($settings->{'cloners'} ne '') {
+                $clonedefaults .= '<br />'.&mt('Default does [_1]not[_2] currently apply because values have been set in the course.','<b>','</b>');
+            } else {
+                $clonedefaults .= &mt('Default is disregarded if [_1]any[_2] values are set here in the course.','<i>','</i>');
+            }
+            $clonedefaults = '<br /><br />'.$clonedefaults;
+        }
+    }
     if (!defined($categoriesform)) {
         $categoriesform = &mt('No categories defined in this domain.');
     }
-
     my ($can_toggle_cat,$can_categorize) = &can_modify_catsettings($cdom,$crstype);
     my $cnum = $env{'course.'.$env{'request.course.id'}.'.num'};
 
@@ -1889,7 +2089,8 @@ sub print_courseinfo {
                           },
         'cloners'      => { 
                    text => '<b>'.&mt($itemtext->{'cloners'}).'</b><br />'.
-                           &mt('Owner and Coordinators included automatically'),
+                           &mt('Owner and Coordinators included automatically').
+                           $clonedefaults,
                    input => 'textbox',
                    size  => '40'
                          },
@@ -1945,7 +2146,7 @@ sub print_courseinfo {
         	$datatable .= &item_table_row_start($items{$item}{text},$count,undef,$colspan);
         }
         if ($items{$item}{input} eq 'radio') {
-            $datatable .= &yesno_radio($item,$settings);
+            $datatable .= &yesno_radio($item,$settings,undef,undef,undef,$noedit);
         } elsif ($item eq 'cloners') {
             my $includeempty = 1;
             my $num = 0;
@@ -1959,13 +2160,14 @@ sub print_courseinfo {
                 $datatable .= ' checked="checked" ';
             }
             $datatable .= 'onchange="javascript:update_cloners('.
-                          "'cloners_all'".');" />'.&mt('Yes').'</label>'.
-                          ('&nbsp;'x2).'<input type="radio" name="cloners_all" value="0" ';
+                          "'cloners_all'".');"'.$disabled.' />'.&mt('Yes').'</label>'.
+                          ('&nbsp;'x2).'<label>'.
+                          '<input type="radio" name="cloners_all" value="0" ';
             if ($settings->{$item} ne '*') {
                 $datatable .= ' checked="checked" ';
             }
             $datatable .= ' onchange="javascript:update_cloners('.
-                          "'cloners_all'".');"/>'.&mt('No').'</label></td>'.
+                          "'cloners_all'".');"'.$disabled.' />'.&mt('No').'</label></td>'.
                           &Apache::loncommon::end_data_table_row().
                           &Apache::loncommon::end_data_table().
                           '<table><tr><td align="left"><b>'.&mt('Or').
@@ -1973,7 +2175,13 @@ sub print_courseinfo {
                           &Apache::loncommon::start_data_table();
             my @cloners;
             if ($settings->{$item} eq '') {
-                $datatable .= &new_cloners_dom_row($cdom,'0');
+                unless ($noedit) {
+                    my $default;
+                    if ($currcanclone eq 'domain') {
+                        $default = '0';
+                    }
+                    $datatable .= &new_cloners_dom_row($cdom,'0',$default);
+                } 
             } elsif ($settings->{$item} ne '*') {
                 my @entries = split(/,/,$settings->{$item});
                 if (@entries > 0) {
@@ -1994,7 +2202,7 @@ sub print_courseinfo {
                                 '</b><input type="hidden" name="cloners_dom_'.$num.
                                 '" value="'.$udom.'" /></span><br />'.
                                 '<span class="LC_nobreak"><label><input type="checkbox" '.
-                                'name="cloners_delete" value="'.$num.'" onchange="javascript:update_cloners('."'cloners_delete','$num'".');" />'.
+                                'name="cloners_delete" value="'.$num.'" onchange="javascript:update_cloners('."'cloners_delete','$num'".');"'.$disabled.' />'.
                                 &mt('Delete').'</label></span></td>'.
                                 &Apache::loncommon::end_data_table_row();
                             $num ++;
@@ -2017,7 +2225,8 @@ sub print_courseinfo {
                           '<td align="left"><span class="LC_nobreak">'.
                           $add_domtitle.'</span><br />'.
                           &Apache::loncommon::select_dom_form('','cloners_newdom',
-                                                              $includeempty).
+                                                              $includeempty,undef,undef,
+                                                              undef,undef,$noedit).
                           '<input type="hidden" name="cloners_total" value="'.$num.'" />'.
                           '</td>'.&Apache::loncommon::end_data_table_row().
                           &Apache::loncommon::end_data_table().
@@ -2029,9 +2238,40 @@ sub print_courseinfo {
                           &mt('Specific users').'&nbsp;(<tt>'.
                           &mt('user:domain,user:domain').'</tt>)<br />'.
                           &Apache::lonhtmlcommon::textbox($item,$cloners_str,
-                                                          $items{$item}{'size'}).
+                                                          $items{$item}{'size'},$disabled).
                           '</td>'.&Apache::loncommon::end_data_table_row().
                           &Apache::loncommon::end_data_table();
+            if (@code_order > 0) {
+                my (%cat_items,@codetitles,%cat_titles,%cat_order);
+                my ($jscript,$totcodes,$numtitles,$lasttitle) =
+                    &Apache::courseclassifier::instcode_selectors_data($cdom,'display',
+                                                                       \%cat_items,\@codetitles,
+                                                                       \%cat_titles,\%cat_order);
+                my $showncodes = 'off';
+                my $checkedoff = ' checked="checked"';
+                my $checkedon = '';
+                if ($cloner_instcode) {
+                    $checkedon = $checkedoff;
+                    $checkedoff = '';
+                    $showncodes = 'on';
+                }
+                $datatable .= '<table><tr><td align="left"><b>'.&mt('And').
+                              '</b></td></tr></table>'.
+                              &Apache::loncommon::start_data_table().
+                              &Apache::loncommon::start_data_table_row().
+                              '<td align="left"><span class="LC_nobreak">'.
+                              &mt('Cloning by official course(s) based on course category').('&nbsp;'x2).
+                              '<label>'.
+                              '<input type="radio" name="cloners_instcode" value="1" onclick="toggleCloners(this);"'.$checkedon.$disabled.' />'.&mt('Yes').'</label>'.
+                              ('&nbsp;').
+                              '<label>'.
+                              '<input type="radio" name="cloners_instcode" value="0" onclick="toggleCloners(this);"'.$checkedoff.$disabled.' />'.&mt('No').'</label>'.
+                              &Apache::courseclassifier::build_instcode_selectors($numtitles,
+                                  $lasttitle,\%cat_items,\@codetitles,\%cat_titles,\%cat_order,
+                                  $showncodes,'LC_cloners_instcodes','LC_cloners_instcode',$noedit).
+                              '</td>'.&Apache::loncommon::end_data_table_row().
+                              &Apache::loncommon::end_data_table();
+            }
         } elsif ($item eq 'rolenames') {
             $datatable .= '</td><td align="right">'.
                           &Apache::loncommon::start_data_table();
@@ -2049,12 +2289,17 @@ sub print_courseinfo {
                               '</span></td><td align="left">'.
                               &Apache::lonhtmlcommon::textbox('rolenames_'.$role,
                                                  $settings->{$role.'.plaintext'},
-                                                 $items{$item}{size}).'</td>'.
+                                                 $items{$item}{size},$disabled).'</td>'.
                               &Apache::loncommon::end_data_table_row();
             }
             $datatable .= &Apache::loncommon::end_data_table().'</td>';
         } elsif ($item eq 'categories') {
-            my $launcher = 'onfocus="this.blur();javascript:catsbrowser();"';
+            my $launcher;
+            if ($noedit) {
+                $launcher = $disabled;
+            } else {
+                $launcher = 'onfocus="this.blur();javascript:catsbrowser();"';
+            }
             $datatable .= '<input type="hidden" name="categories" value="'.$settings->{$item}.'" />'.
                           &Apache::lonhtmlcommon::textbox($item.'_display',$settings->{$item},
                                                           $items{$item}{size},$launcher);
@@ -2094,13 +2339,13 @@ sub print_courseinfo {
                 if (($crstype eq 'Course') && ($env{'course.'.$env{'request.course.id'}.'.internal.coursecode'}) && ($autocoowner)) {
                     $datatable .= &show_autocoowners(@currcoown);
                 } else {
-                    $datatable .= &coowner_invitations($cnum,$cdom,@currcoown);
+                    $datatable .= &coowner_invitations($cnum,$cdom,\@currcoown,$noedit);
                 }
             } else {
                 if (($crstype eq 'Course') && ($env{'course.'.$env{'request.course.id'}.'.internal.coursecode'}) && ($autocoowner)) {
                     $datatable .= &show_autocoowners(@currcoown);
                 } else {
-                    $datatable .= &manage_coownership($cnum,$cdom,@currcoown);
+                    $datatable .= &manage_coownership($cnum,$cdom,\@currcoown,$noedit);
                 }
             }
         } elsif ($item eq 'externalsyllabus') {
@@ -2144,7 +2389,7 @@ sub print_courseinfo {
             }
             $datatable .= $showreqd;
         } else {
-            $datatable .= &Apache::lonhtmlcommon::textbox($item,$settings->{$item},$items{$item}{size});
+            $datatable .= &Apache::lonhtmlcommon::textbox($item,$settings->{$item},$items{$item}{size},$disabled);
         }
         $datatable .= &item_table_row_end();
     }
@@ -2153,19 +2398,24 @@ sub print_courseinfo {
 }
 
 sub new_cloners_dom_row {
-    my ($newdom,$num) = @_;
-    my $output;
+    my ($newdom,$num,$default) = @_;
+    my ($output,$checkedon,$checkedoff);
     if ($newdom ne '') {
+        if ($num eq $default) {
+           $checkedon = 'checked="checked" ';
+        } else {
+           $checkedoff = 'checked="checked" ';
+        }
         $output .= &Apache::loncommon::start_data_table_row().
                    '<td valign="top"><span class="LC_nobreak">'.
                    &mt('Any user in domain:').'&nbsp;<b>'.$newdom.'</b>'.
                    ('&nbsp;'x2).'<label><input type="radio" '.
-                   'name="cloners_activate" value="'.$num.'" '.
+                   'name="cloners_activate" value="'.$num.'" '.$checkedon.
                    'onchange="javascript:update_cloners('.
                    "'cloners_activate','$num'".');" />'.
                    &mt('Yes').'</label>'.('&nbsp;'x2).
                    '<label><input type="radio" '.
-                   'name="cloners_activate" value="" checked="checked" '.
+                   'name="cloners_activate" value="-1" '.$checkedoff.
                    'onchange="javascript:update_cloners('.
                    "'cloners_activate','$num'".');" />'.
                    &mt('No').'</label><input type="hidden" name="cloners_dom_'.
@@ -2226,6 +2476,7 @@ sub assign_course_categories {
         } else {
             $alert = &mt("Use 'Save' in the main window to save course categories");
         }
+        &js_escape(\$alert);
         $catwin_js = <<ENDSCRIPT;
 <script type="text/javascript">
 
@@ -2821,8 +3072,13 @@ sub show_contents_view {
 sub releases_by_map {
     my ($r,$bymap,$url,$scopeorder,$lt) = @_;
     return unless ((ref($bymap) eq 'HASH') && (ref($scopeorder) eq 'ARRAY'));
+    my $newrow = 0;
     if (ref($bymap->{$url}) eq 'HASH') {
         foreach my $rev (sort(keys(%{$bymap->{$url}}))) {
+            if ($newrow) {
+                $r->print(&Apache::loncommon::end_data_table_row().
+                          &Apache::loncommon::continue_data_table_row());
+            }
             $r->print('<td valign="middle">'.$rev.'</td><td>');
             if (ref($bymap->{$url}{$rev}) eq 'HASH') {
                 $r->print('<table border="0">');
@@ -2844,6 +3100,7 @@ sub releases_by_map {
                 $r->print('</table>');
             }
             $r->print('</td>');
+            $newrow = 1;
         }
     } else {
         $r->print('<td colspan="2">&nbsp;</td>');
@@ -2900,13 +3157,20 @@ sub show_autocoowners {
 }
 
 sub coowner_invitations {
-    my ($cnum,$cdom,@currcoown) = @_;
-    my ($output,@pendingcoown,@othercoords);
+    my ($cnum,$cdom,$currcoownref,$noedit) = @_;
+    my ($output,@pendingcoown,@othercoords,@currcoown);
     my $pendingcoowners =
         $env{'course.'.$env{'request.course.id'}.'.internal.pendingco-owners'};
     if ($pendingcoowners) {
         @pendingcoown = split(',',$pendingcoowners);
     }
+    if (ref($currcoownref) eq 'ARRAY') {
+        @currcoown == @{$currcoownref};
+    }
+    my $disabled;
+    if ($noedit) {
+        $disabled = ' disabled="disabled"';
+    }
     my $ccrole = 'cc';
     my %ccroles = &Apache::lonnet::get_my_roles($cnum,$cdom,undef,undef,[$ccrole]);
     foreach my $key (sort(keys(%ccroles))) {
@@ -2924,7 +3188,7 @@ sub coowner_invitations {
                           '<td><i>'.&mt('Current co-owners').'</i></td><td>';
             foreach my $person (@currcoown) {
                 my ($co_uname,$co_dom) = split(':',$person);
-                $output .= '<span class="LC_nobreak"><label><input type="checkbox" name="coowners" checked="checked" value="'.$person.'" />'.&Apache::loncommon::plainname($co_uname,$co_dom).'</label></span>'.('&nbsp;'x2).' ';
+                $output .= '<span class="LC_nobreak"><label><input type="checkbox" name="coowners" checked="checked" value="'.$person.'"'.$disabled.' />'.&Apache::loncommon::plainname($co_uname,$co_dom).'</label></span>'.('&nbsp;'x2).' ';
             }
             $output .= '</td>'.
                           &Apache::loncommon::end_data_table_row();
@@ -2934,7 +3198,7 @@ sub coowner_invitations {
                           '<td><i>'.&mt('Invited as co-owners [_1](agreement pending)','<br />').'</i></td><td>';
             foreach my $person (@pendingcoown) {
                 my ($co_uname,$co_dom) = split(':',$person);
-                $output .= '<span class="LC_nobreak"><label><input type="checkbox" name="pendingcoowners" checked="checked" value="'.$person.'" />'.&Apache::loncommon::plainname($co_uname,$co_dom).'</label></span>'.('&nbsp;'x2).' ';
+                $output .= '<span class="LC_nobreak"><label><input type="checkbox" name="pendingcoowners" checked="checked" value="'.$person.'"'.$disabled.' />'.&Apache::loncommon::plainname($co_uname,$co_dom).'</label></span>'.('&nbsp;'x2).' ';
             }
             $output .= '</td>'.
                        &Apache::loncommon::end_data_table_row();
@@ -2944,7 +3208,7 @@ sub coowner_invitations {
                           '<td><i>'.&mt('Invite other Coordinators [_1]to become co-owners','<br />').'</i></td><td>';
             foreach my $person (@othercoords) {
                 my ($co_uname,$co_dom) = split(':',$person);
-                $output .= '<span class="LC_nobreak"><label><input type="checkbox" name="invitecoowners" value="'.$person.'" />'.&Apache::loncommon::plainname($co_uname,$co_dom).'</label></span>'.('&nbsp;'x2).' ';
+                $output .= '<span class="LC_nobreak"><label><input type="checkbox" name="invitecoowners" value="'.$person.'"'.$disabled.' />'.&Apache::loncommon::plainname($co_uname,$co_dom).'</label></span>'.('&nbsp;'x2).' ';
             }
             $output .= '</td>'.
                           &Apache::loncommon::end_data_table_row();
@@ -2957,13 +3221,20 @@ sub coowner_invitations {
 }
 
 sub manage_coownership  {
-    my ($cnum,$cdom,@currcoown) = @_;
-    my (@pendingcoown);
+    my ($cnum,$cdom,$currcoownref,$noedit) = @_;
+    my (@pendingcoown,@currcoown);
     my $pendingcoowners =
         $env{'course.'.$env{'request.course.id'}.'.internal.pendingco-owners'};
     if ($pendingcoowners) {
         @pendingcoown = split(',',$pendingcoowners);
     }
+    if (ref($currcoownref) eq 'ARRAY') {
+        @currcoown == @{$currcoownref};
+    }
+    my $disabled;
+    if ($noedit) {
+        $disabled = ' disabled="disabled"';
+    }
     my ($is_coowner,$is_pending,$output);
     my $uname = $env{'user.name'};
     my $udom = $env{'user.domain'};
@@ -2995,8 +3266,8 @@ sub manage_coownership  {
         if ($is_coowner) {
             $output .= &mt('You are currently a co-owner:').'&nbsp;<label><input type="checkbox" name="remove_coowoner" value="'.$uname.':'.$udom.'" />'.&mt('Discontinue?').'</label>';
         } else {
-            $output .= &mt('The course owner has invited you to become a co-owner:').'&nbsp;<label><input type="radio" name="pending_coowoner" value="accept" />'.&mt('Accept?').'</label>'.('&nbsp;'x2).
-                       '<label><input type="radio" name=pending_coowoner" value="decline" />'.&mt('Decline?').'</label>';
+            $output .= &mt('The course owner has invited you to become a co-owner:').'&nbsp;<label><input type="radio" name="pending_coowoner" value="accept"'.$disabled.' />'.&mt('Accept?').'</label>'.('&nbsp;'x2).
+                       '<label><input type="radio" name=pending_coowoner" value="decline"'.$disabled.' />'.&mt('Decline?').'</label>';
         }
         $output .= '</span>';
         if (@currcoown) {
@@ -3010,7 +3281,7 @@ sub manage_coownership  {
 }
 
 sub print_localization {
-    my ($cdom,$settings,$ordered,$itemtext,$rowtotal) = @_;
+    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$noedit) = @_;
     unless ((ref($settings) eq 'HASH') && (ref($ordered) eq 'ARRAY') && (ref($itemtext) eq 'HASH')) {
         return;
     }
@@ -3029,8 +3300,11 @@ sub print_localization {
                          input => 'selectbox',
                        },
     );
-    my $datatable;
+    my ($datatable,$disabled);
     my $count = 0;
+    if ($noedit) {
+        $disabled = ' disabled="disabled"';
+    }
     foreach my $item (@{$ordered}) {
         $count ++;
         my $colspan;
@@ -3043,7 +3317,7 @@ sub print_localization {
             my $timezone = &Apache::lonlocal::gettimezone();
             $datatable .= 
                 &Apache::loncommon::select_timezone($item,$timezone,undef,
-                                                    $includeempty);
+                                                    $includeempty,$disabled);
         } elsif ($item eq 'datelocale') {
             my $includeempty = 1;
             my $locale_obj = &Apache::lonlocal::getdatelocale();
@@ -3053,11 +3327,13 @@ sub print_localization {
             }
             $datatable .= 
                 &Apache::loncommon::select_datelocale($item,$currdatelocale,
-                                                      undef,$includeempty);
+                                                      undef,$includeempty,$disabled);
         } else {
             if ($settings->{$item} eq '') {
-                $datatable .= '</td><td align="right">'.
-                    &Apache::loncommon::select_language('languages_0','',1);
+                unless ($noedit) {
+                    $datatable .= '</td><td align="right">'.
+                        &Apache::loncommon::select_language('languages_0','',1);
+                }
             } else {
                 my $num = 0;
                 my @languages = split(/\s*[,;:]\s*/,$settings->{$item});
@@ -3077,19 +3353,22 @@ sub print_localization {
                             '</b><input type="hidden" name="languages_'.$num.
                             '" value="'.$lang.'" /></span><br />'.
                             '<span class="LC_nobreak"><label><input type="checkbox" '.
-                            'name="languages_delete" value="'.$num.'" />'.
+                            'name="languages_delete" value="'.$num.'"'.$disabled.' />'.
                             &mt('Delete').'</label></span></td>'.
                             &Apache::loncommon::end_data_table_row();
                             $num ++;
                     }
                 }
-                $datatable .= &Apache::loncommon::start_data_table_row().
+                unless ($noedit) {
+                    $datatable .= 
+                              &Apache::loncommon::start_data_table_row().
                               '<td align="left"><span class="LC_nobreak">'.
                               &mt('Additional language:'). '</span><br />'.
                               &Apache::loncommon::select_language('languages_'.$num,'',1).
                               '<input type="hidden" name="languages_total" value="'.$num.'" />'.
-                              '</td>'.&Apache::loncommon::end_data_table_row().
-                              &Apache::loncommon::end_data_table().'<br />';
+                              '</td>'.&Apache::loncommon::end_data_table_row();
+                }
+                $datatable .= &Apache::loncommon::end_data_table().'<br />';
             }
         }
         $datatable .= &item_table_row_end();
@@ -3110,7 +3389,7 @@ sub get_lang_choices {
 }
 
 sub print_feedback {
-    my ($position,$cdom,$settings,$ordered,$itemtext,$rowtotal) = @_;
+    my ($position,$cdom,$settings,$ordered,$itemtext,$rowtotal,$noedit) = @_;
     unless ((ref($settings) eq 'HASH') && (ref($ordered) eq 'ARRAY') && (ref($itemtext) eq 'HASH')) {
         return;
     }
@@ -3145,7 +3424,10 @@ sub print_feedback {
                                            del      => 'Delete?',
                                            sec      => 'Sections:',
                                          );
-
+    my $disabled;
+    if ($noedit) {
+        $disabled = ' disabled="disabled"';
+    }
     foreach my $item (@{$ordered}) {
         $count ++;
         if ($position eq 'top') {
@@ -3157,10 +3439,10 @@ sub print_feedback {
             my $includeempty = 0;
             $datatable .= '</td><td align="right">'.
                           &user_table($cdom,$item,\@sections,
-                                      $settings->{$item},\%lt);
+                                      $settings->{$item},\%lt,$noedit);
         } else {
             $datatable .= &Apache::lonhtmlcommon::textbox($item.'.text',
-                              $settings->{$item.'.text'},$items{$item}{size});  
+                              $settings->{$item.'.text'},$items{$item}{size},$disabled);  
         }
         $datatable .= &item_table_row_end();
     }
@@ -3169,15 +3451,21 @@ sub print_feedback {
 }
 
 sub user_table {
-    my ($cdom,$item,$sections,$currvalue,$lt) = @_;
+    my ($cdom,$item,$sections,$currvalue,$lt,$noedit) = @_;
     my $output;
     if ($currvalue eq '') {
-        $output .= &select_recipient($item,'0',$cdom,$sections);
+        unless ($noedit) {
+            $output .= &select_recipient($item,'0',$cdom,$sections);
+        }  
     } else {
         my $num = 0;
         my @curr = split(/,/,$currvalue);
         $output .= '<table class="LC_data_table">';
         my ($currusers);
+        my $disabled;
+        if ($noedit) {
+            $disabled = ' disabled="disabled"';
+        }
         foreach my $val (@curr) {
             next if ($val eq '');
             my ($uname,$udom,$seclist) = ($val =~ /^($match_username):($match_domain)(\(?[^\)]*\)?)$/);
@@ -3189,7 +3477,7 @@ sub user_table {
             $currusers .= '<tr>'.
                         '<td valign="top"><span class="LC_nobreak">'.
                         '<label><input type="checkbox" '.
-                        'name="'.$item.'_delete" value="'.$num.'" />'.
+                        'name="'.$item.'_delete" value="'.$num.'"'.$disabled.' />'.
                         $lt->{'del'}.'</label>'.
                         '<input type="hidden" name="'.$item.'_user_'.
                         $num.'" value="'.$uname.':'.$udom.'" />'.('&nbsp;'x2).
@@ -3200,7 +3488,7 @@ sub user_table {
                 if (@{$sections}) {
                     $currusers.= ('&nbsp;'x3).$lt->{'sec'}.'&nbsp;'.
                                   &select_sections($item,$num,$sections,
-                                  \@selsec);
+                                  \@selsec,$noedit);
                 }
             }
             $currusers .= '</span></td></tr>';
@@ -3218,12 +3506,16 @@ sub user_table {
                           '<table>'.$currusers.'</table></td>'.
                           '</tr>';
         }
-        $output .= '<tr>'.
+        unless ($noedit) {
+            $output .= 
+                  '<tr>'.
                   '<td align="left"><span class="LC_nobreak"><i>'.
                   $lt->{'add'}.'</i></span><br />'.
                   &select_recipient($item,$num,$cdom,$sections).
                   '<input type="hidden" name="'.$item.'_total" value="'.$num.'" />'.
-                  '</td></tr></table>';
+                  '</td></tr>';
+        }
+        $output .= '</table>';
     }
     return $output;
 }
@@ -3250,14 +3542,17 @@ sub select_recipient {
 }
 
 sub select_sections {
-    my ($item,$num,$sections,$selected) = @_;
-    my ($output,@currsecs,$allsec);
+    my ($item,$num,$sections,$selected,$noedit) = @_;
+    my ($output,@currsecs,$allsec,$disabled);
     if (ref($selected) eq 'ARRAY') {
         @currsecs = @{$selected};
     }
     if (!@currsecs) {
         $allsec = ' selected="selected"';
     }
+    if ($noedit) {
+        $disabled = ' disabled="disabled"';
+    }
     if (ref($sections) eq 'ARRAY') {
         if (@{$sections}) {
             my $mult;
@@ -3267,7 +3562,7 @@ sub select_sections {
                     $mult .= ' size="4"';
                 }
             }
-            $output = '<select name="'.$item.'_sections_'.$num.'"'.$mult.'>'.
+            $output = '<select name="'.$item.'_sections_'.$num.'"'.$mult.$disabled.'>'.
                       ' <option value=""'.$allsec.'>'.&mt('All').'</option>';
             foreach my $sec (@{$sections}) {
                 my $is_sel;
@@ -3283,7 +3578,7 @@ sub select_sections {
 }
 
 sub print_discussion {
-    my ($cdom,$settings,$ordered,$itemtext,$rowtotal) = @_;
+    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$noedit) = @_;
     unless ((ref($settings) eq 'HASH') && (ref($ordered) eq 'ARRAY') && (ref($itemtext) eq 'HASH')) {
         return;
     }
@@ -3349,22 +3644,22 @@ sub print_discussion {
         $datatable .= &item_table_row_start($items{$item}{text},$count,undef,$colspan);
         if ($item eq 'plc.roles.denied') {
             $datatable .= '</td><td align="right">'.
-                          '<table>'.&role_checkboxes($cdom,$cnum,$item,$settings).
+                          '<table>'.&role_checkboxes($cdom,$cnum,$item,$settings,undef,undef,$noedit).
                           '</table>';
         } elsif ($item eq 'plc.users.denied') {
             $datatable .= '</td><td align="right">'.
                           &user_table($cdom,$item,undef,
-                                      $settings->{$item},\%lt);
+                                      $settings->{$item},\%lt,$noedit);
         } elsif ($item eq 'pch.roles.denied') {
             $datatable .= '</td><td align="right">'.
-                          '<table>'.&role_checkboxes($cdom,$cnum,$item,$settings).
+                          '<table>'.&role_checkboxes($cdom,$cnum,$item,$settings,undef,undef,$noedit).
                           '</table>';
         } elsif ($item eq 'pch.users.denied') {
             $datatable .= '</td><td align="right">'.
                           &user_table($cdom,$item,undef,
-                                      $settings->{$item},\%lt);
+                                      $settings->{$item},\%lt,$noedit);
         } elsif ($item eq 'allow_limited_html_in_feedback') {
-            $datatable .= &yesno_radio($item,$settings);
+            $datatable .= &yesno_radio($item,$settings,undef,undef,undef,$noedit);
         } elsif ($item eq 'allow_discussion_post_editing') {
             $datatable .= '</td><td align="right"><br />'.
                           &Apache::loncommon::start_data_table().
@@ -3372,7 +3667,7 @@ sub print_discussion {
                           '<th align="left">'.&mt('Role').'</th><th>'.
                           &mt('Sections').'</th>'.
                           &Apache::loncommon::end_data_table_row().
-                          &role_checkboxes($cdom,$cnum,$item,$settings,1).
+                          &role_checkboxes($cdom,$cnum,$item,$settings,1,undef,$noedit).
                           &Apache::loncommon::end_data_table().'<br />';
         } elsif ($item eq 'discussion_post_fonts') {
             $datatable .= '</td><td align="right"><br />'.
@@ -3384,7 +3679,7 @@ sub print_discussion {
                           '<th align="center">'.&mt('font-style').'</th>'.
                           '<th align="center">'.&mt('Other css').'</th>'.
                           &Apache::loncommon::end_data_table_row().
-                          &set_discussion_fonts($cdom,$cnum,$item,$settings).
+                          &set_discussion_fonts($cdom,$cnum,$item,$settings,$noedit).
                           &Apache::loncommon::end_data_table().'<br />';
         }
         $datatable .= &item_table_row_end();
@@ -3394,7 +3689,7 @@ sub print_discussion {
 }
 
 sub role_checkboxes {
-    my ($cdom,$cnum,$item,$settings,$showsections,$crstype) = @_;
+    my ($cdom,$cnum,$item,$settings,$showsections,$crstype,$noedit) = @_;
     my @roles = ('st','ad','ta','ep','in');
     if ($crstype eq 'Community') {
         push(@roles,'co');
@@ -3432,6 +3727,10 @@ sub role_checkboxes {
     }
     my $numinrow = 3;
     my $count = 0;
+    my $disabled;
+    if ($noedit) {
+        $disabled = ' disabled="disabled"';
+    }
     foreach my $role (@roles) {
         my $checked = '';
         if (grep(/^\Q$role\E$/,@current)) {
@@ -3450,11 +3749,11 @@ sub role_checkboxes {
             }
         }
         $output .= '<td align="left"><span class="LC_nobreak"><label><input type="checkbox" name="'.
-                   $item.'" value="'.$role.'"'.$checked.'/>&nbsp;'.
+                   $item.'" value="'.$role.'"'.$checked.$disabled.'/>&nbsp;'.
                    $plrole.'</label></span></td>';
         if ($showsections) {
             $output .= '<td align="left">'.
-                       &select_sections($item,$role,\@sections,$currsec{$role}).
+                       &select_sections($item,$role,\@sections,$currsec{$role},$noedit).
                        '</td></tr>';
         }
         $count ++;
@@ -3482,11 +3781,11 @@ sub role_checkboxes {
                 }
             }
             $output .= '<td><span class="LC_nobreak"><label><input type="checkbox" name="'.
-                       $item.'" value="'.$value.'"'.$checked.' />&nbsp;'.$rolename.
+                       $item.'" value="'.$value.'"'.$checked.$disabled.' />&nbsp;'.$rolename.
                        '</label></span></td>';
             if ($showsections) {
                 $output .= '<td>'.
-                           &select_sections($item,$role,\@sections,$currsec{$role}).
+                           &select_sections($item,$role,\@sections,$currsec{$role},$noedit).
                            '</td>'.&Apache::loncommon::end_data_table_row();
             }
             $total ++;
@@ -3508,11 +3807,15 @@ sub role_checkboxes {
 }
 
 sub set_discussion_fonts {
-    my ($cdom,$cnum,$item,$settings) = @_;
+    my ($cdom,$cnum,$item,$settings,$noedit) = @_;
     my ($classorder,$classtitles) = &discussion_vote_classes();
     my ($styleorder,$styleitems) = &discussion_vote_styles();
     my $output;
     if (ref($classorder) eq 'ARRAY') {
+        my $disabled;
+        if ($noedit) {
+            $disabled = ' disabled="disabled"';
+        }
         foreach my $cat (@{$classorder}) {
             my %curr = ();
             ($curr{'size'},$curr{'weight'},$curr{'style'},$curr{'other'}) =
@@ -3529,12 +3832,12 @@ sub set_discussion_fonts {
                             ($currsize,$currunit) = ($1,$2);
                             $curr{'size'} = $currunit;
                         }
-                        $output .= '<input type="textbox" name="discussion_post_fonts_'.$cat.'_'.$item.'" value="'.$currsize.'" size="5">'.
+                        $output .= '<input type="textbox" name="discussion_post_fonts_'.$cat.'_'.$item.'" value="'.$currsize.'" size="5"'.$disabled.' />'.
                                    '&nbsp;';
                         $selitem = 'unit';
                     }
                     if ((ref($styleitems) eq 'HASH') && (ref($styleitems->{$item}) eq 'ARRAY')) {
-                        $output .= '<select name="discussion_post_fonts_'.$cat.'_'.$selitem.'">';
+                        $output .= '<select name="discussion_post_fonts_'.$cat.'_'.$selitem.'"'.$disabled.'>';
                         foreach my $val (@{$styleitems->{$item}}) {
                             my $sel;
                             if ($val eq $curr{$item}) {
@@ -3545,7 +3848,7 @@ sub set_discussion_fonts {
                     }
                 }
             }
-            $output .= '</span></td><td><input type="textbox" size="10" name="discussion_post_fonts_'.$cat.'_other" value="'.$curr{'other'}.'" /></td>';
+            $output .= '</span></td><td><input type="textbox" size="10" name="discussion_post_fonts_'.$cat.'_other" value="'.$curr{'other'}.'"'.$disabled.' /></td>';
        }
        $output .= &Apache::loncommon::end_data_table_row();
     }
@@ -3577,7 +3880,7 @@ sub discussion_vote_styles {
 }
 
 sub print_classlists {
-    my ($position,$cdom,$settings,$itemtext,$rowtotal,$crstype) = @_;
+    my ($position,$cdom,$settings,$itemtext,$rowtotal,$crstype,$noedit) = @_;
     my @ordered;
     if ($position eq 'top') {
         @ordered = ('default_enrollment_start_date',
@@ -3651,11 +3954,11 @@ sub print_classlists {
             ($settings->{'student_classlist_view'} eq 'section')) { 
         $settings->{'student_classlist_view'} = 'disabled';
     }
-    return &make_item_rows($cdom,\%items,\@ordered,$settings,$rowtotal,$crstype,'classlists');
+    return &make_item_rows($cdom,\%items,\@ordered,$settings,$rowtotal,$crstype,'classlists',$noedit);
 }
 
 sub print_appearance {
-    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype) = @_;
+    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype,$noedit) = @_;
     unless ((ref($settings) eq 'HASH') && (ref($ordered) eq 'ARRAY') && (ref($itemtext) eq 'HASH')) {
         return;
     }
@@ -3688,11 +3991,10 @@ sub print_appearance {
                    input => 'selectbox',
                    options => {
                                 MathJax  => 'MathJax',
-                                jsMath   => 'jsMath',
                                 mimetex => &mt('Convert to Images'),
                                 tth      => &mt('TeX to HTML'),
                               },
-                   order  => ['MathJax','jsMath','mimetex','tth'],
+                   order  => ['MathJax','mimetex','tth'],
                    nullval => $mathdef,
                  },
         'tthoptions' => {
@@ -3709,11 +4011,11 @@ sub print_appearance {
                    input => 'radio',
                  },
     );
-    return &make_item_rows($cdom,\%items,$ordered,$settings,$rowtotal,$crstype,'appearance');
+    return &make_item_rows($cdom,\%items,$ordered,$settings,$rowtotal,$crstype,'appearance',$noedit);
 }
 
 sub print_grading {
-    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype) = @_;
+    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype,$noedit) = @_;
     unless ((ref($settings) eq 'HASH') && (ref($ordered) eq 'ARRAY') && (ref($itemtext) eq 'HASH')) {
         return;
     }
@@ -3764,11 +4066,11 @@ sub print_grading {
                    input => 'radio',
                  },
     );
-    return &make_item_rows($cdom,\%items,$ordered,$settings,$rowtotal,$crstype,'grading');
+    return &make_item_rows($cdom,\%items,$ordered,$settings,$rowtotal,$crstype,'grading',$noedit);
 }
 
 sub print_printouts {
-    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype) = @_;
+    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype,$noedit) = @_;
     unless ((ref($settings) eq 'HASH') && (ref($ordered) eq 'ARRAY') && (ref($itemtext) eq 'HASH')) {
         return;
     }
@@ -3817,11 +4119,11 @@ sub print_printouts {
             nullval => &mt('None specified - use domain default'),
                     }
     );
-    return &make_item_rows($cdom,\%items,$ordered,$settings,$rowtotal,$crstype,'printouts');
+    return &make_item_rows($cdom,\%items,$ordered,$settings,$rowtotal,$crstype,'printouts',$noedit);
 }
 
 sub print_spreadsheet {
-    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype) = @_;
+    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype,$noedit) = @_;
     unless ((ref($settings) eq 'HASH') && (ref($ordered) eq 'ARRAY') && (ref($itemtext) eq 'HASH')) {
         return;
     }
@@ -3853,11 +4155,11 @@ sub print_spreadsheet {
             input => 'radio',
                          },
                 );
-    return &make_item_rows($cdom,\%items,$ordered,$settings,$rowtotal,$crstype,'spreadsheet');
+    return &make_item_rows($cdom,\%items,$ordered,$settings,$rowtotal,$crstype,'spreadsheet',$noedit);
 }
 
 sub print_bridgetasks {
-    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype) = @_;
+    my ($cdom,$settings,$ordered,$itemtext,$rowtotal,$crstype,$noedit) = @_;
     unless ((ref($settings) eq 'HASH') && (ref($ordered) eq 'ARRAY') && (ref($itemtext) eq 'HASH')) {
         return;
     }
@@ -3895,11 +4197,11 @@ sub print_bridgetasks {
              input => 'radio',
                                   },
                 );
-    return &make_item_rows($cdom,\%items,$ordered,$settings,$rowtotal,$crstype,'bridgetasks');
+    return &make_item_rows($cdom,\%items,$ordered,$settings,$rowtotal,$crstype,'bridgetasks',$noedit);
 }
 
 sub print_other {
-    my ($cdom,$settings,$allitems,$rowtotal,$crstype) = @_;
+    my ($cdom,$settings,$allitems,$rowtotal,$crstype,$noedit) = @_;
     unless ((ref($settings) eq 'HASH') && (ref($allitems) eq 'ARRAY')) {
         return;
     }
@@ -3920,7 +4222,7 @@ sub print_other {
                             input => 'textbox',
                             size  => '30',
                            };
-    my $output = &make_item_rows($cdom,\%items,\@ordered,$settings,$rowtotal,$crstype,'other');
+    my $output = &make_item_rows($cdom,\%items,\@ordered,$settings,$rowtotal,$crstype,'other',$noedit);
 }
 
 sub get_other_items {
@@ -3969,7 +4271,7 @@ sub item_table_row_end {
 }
 
 sub yesno_radio {
-    my ($item,$settings,$unsetdefault,$valueyes,$valueno) = @_;
+    my ($item,$settings,$unsetdefault,$valueyes,$valueno,$noedit) = @_;
     my $itemon = ' ';
     my $itemoff = ' checked="checked" ';
     if (($valueyes eq '') && ($valueno eq '')) {
@@ -3988,16 +4290,24 @@ sub yesno_radio {
             }
         }
     }
+    my $disabled;
+    if ($noedit) {
+        $disabled = ' disabled="disabled"';
+    }
     return '<span class="LC_nobreak"><label>'.
            '<input type="radio" name="'.$item.'"'.
-           $itemon.' value="'.$valueyes.'" />'.&mt('Yes').'</label>&nbsp;'.
+           $itemon.' value="'.$valueyes.'"'.$disabled.' />'.&mt('Yes').'</label>&nbsp;'.
            '<label><input type="radio" name="'.$item.'"'.
-           $itemoff.' value="'.$valueno.'" />'.&mt('No').'</label></span>';
+           $itemoff.' value="'.$valueno.'"'.$disabled.' />'.&mt('No').'</label></span>';
 }
 
 sub select_from_options {
-    my ($item,$order,$options,$curr,$nullval,$multiple,$maxsize,$onchange) = @_;
+    my ($item,$order,$options,$curr,$nullval,$multiple,$maxsize,$onchange,$noedit) = @_;
     my $output;
+    my $disabled;
+    if ($noedit) {
+        $disabled = ' disabled=disabled"';
+    }
     if ((ref($order) eq 'ARRAY') && (ref($options) eq 'HASH')) {
         $output='<select name="'.$item.'" '.$onchange;
         if ($multiple) {
@@ -4008,7 +4318,7 @@ sub select_from_options {
                 $output .= ' size="'.$maxsize.'"';
             }
         }
-        $output .= '>'."\n";
+        $output .= $disabled.'>'."\n";
         if ($nullval ne '') {
             $output .= '<option value=""';
             if (ref($curr) eq 'ARRAY') {
@@ -4041,7 +4351,7 @@ sub select_from_options {
 }
 
 sub make_item_rows {
-    my ($cdom,$items,$ordered,$settings,$rowtotal,$crstype,$caller) = @_;
+    my ($cdom,$items,$ordered,$settings,$rowtotal,$crstype,$caller,$noedit) = @_;
     my $datatable;
     if ((ref($items) eq 'HASH') && (ref($ordered) eq 'ARRAY')) {
         my $count = 0;
@@ -4067,15 +4377,19 @@ sub make_item_rows {
                 }
                 $datatable .= $showcredits;
             } elsif ($item eq 'nothideprivileged') {
-                $datatable .= &nothidepriv_row($cdom,$item,$settings,$crstype);
+                $datatable .= &nothidepriv_row($cdom,$item,$settings,$crstype,$noedit);
             } elsif ($item eq 'checkforpriv') {
-                $datatable .= &checkforpriv_row($cdom,$item,$settings,$crstype);
+                $datatable .= &checkforpriv_row($cdom,$item,$settings,$crstype,$noedit);
             } elsif ($item eq 'print_header_format') {
-                $datatable .= &print_hdrfmt_row($item,$settings);
+                $datatable .= &print_hdrfmt_row($item,$settings,$noedit);
             } elsif ($items->{$item}{input} eq 'dates') {
+               my $disabled;
+               if ($noedit) {
+                  $disabled = ' disabled="disabled"';
+               }
                $datatable .=
                    &Apache::lonhtmlcommon::date_setter('display',$item,
-                                                       $settings->{$item});
+                                                       $settings->{$item},$disabled);
             } elsif ($items->{$item}{input} eq 'radio') {
                 my ($unsetdefault,$valueyes,$valueno);
                 if (($item eq 'usejsme') || ($item eq 'uselcmath')) {
@@ -4086,17 +4400,23 @@ sub make_item_rows {
                      $valueyes = "1";
                      $valueno = "0";
                 }
-                $datatable .= &yesno_radio($item,$settings,$unsetdefault,$valueyes,$valueno);
+                $datatable .= &yesno_radio($item,$settings,$unsetdefault,$valueyes,$valueno,$noedit);
             } elsif ($items->{$item}{input} eq 'selectbox') {
                 my $curr = $settings->{$item};
                 $datatable .=
                     &select_from_options($item,$items->{$item}{'order'},
                                          $items->{$item}{'options'},$curr,
-                                         $items->{$item}{'nullval'});
+                                         $items->{$item}{'nullval'},
+                                         undef,undef,undef,$noedit);
             } elsif ($items->{$item}{input} eq 'textbox') {
+                my $disabled;
+                if ($noedit) {
+                    $disabled = ' disabled=disabled"';
+                }
                 $datatable .= 
                     &Apache::lonhtmlcommon::textbox($item,$settings->{$item},
-                                                    $items->{$item}{size});
+                                                    $items->{$item}{size},
+                                                    $disabled);
             }
             $datatable .= &item_table_row_end();
         }
@@ -4108,7 +4428,7 @@ sub make_item_rows {
 }
 
 sub nothidepriv_row {
-    my ($cdom,$item,$settings,$crstype) = @_;
+    my ($cdom,$item,$settings,$crstype,$noedit) = @_;
     my ($cnum) = &get_course();
     my ($datatable,%nothide,@checkdoms);
     if (ref($settings) eq 'HASH') {
@@ -4145,6 +4465,10 @@ sub nothidepriv_row {
         }
     }
     if (@privusers > 0) {
+        my $disabled;
+        if ($noedit) {
+           $disabled = ' disabled="disabled"';
+        }
         $datatable .= '<table align="right">';
         foreach my $user (sort(@privusers)) {
             my $hideon = ' checked="checked" ';
@@ -4161,9 +4485,9 @@ sub nothidepriv_row {
                           '</td><td align="left">'.
                           '<span class="LC_nobreak"><label>'.
                           '<input type="radio" name="'.$item.'_'.$user.'"'.
-                          $hideon.' value="" />'.&mt('Hidden').'</label>&nbsp;'.
+                          $hideon.' value=""'.$disabled.' />'.&mt('Hidden').'</label>&nbsp;'.
                           '<label><input type="radio" name="'.$item.'_'.$user.'"'.
-                          $hideoff.' value="yes" />'.&mt('Shown').'</label></span></td>'.
+                          $hideoff.' value="yes"'.$disabled.' />'.&mt('Shown').'</label></span></td>'.
                           '</tr>';
         }
         $datatable .= '</table>';
@@ -4178,7 +4502,7 @@ sub nothidepriv_row {
 }
 
 sub checkforpriv_row {
-    my ($cdom,$item,$settings,$crstype) = @_;
+    my ($cdom,$item,$settings,$crstype,$noedit) = @_;
     my $datatable;
     my %domains=map { $_ => 1; } &Apache::lonnet::all_domains();
     if (keys(%domains) == 1) {
@@ -4191,12 +4515,17 @@ sub checkforpriv_row {
     if ($settings->{$item} eq '') {
         $datatable = '</td><td align="right">'.
                      &Apache::loncommon::select_dom_form('','checkforpriv_'.$num,
-                                                         1,1,undef,undef,\@excdoms);
+                                                         1,1,undef,undef,\@excdoms,
+                                                         $noedit);
     } else {
         my @privdoms = split(/,/,$settings->{$item});
         my %domains=map { $_ => 1; } &Apache::lonnet::all_domains();
         $datatable = '</td><td align="right"><br />'.
                      &Apache::loncommon::start_data_table();
+        my $disabled;
+        if ($noedit) {
+            $disabled = ' disabled="disabled"';
+        }
         if (@privdoms > 0) {
             foreach my $currdom (@privdoms) {
                 next unless ($domains{$currdom});
@@ -4211,7 +4540,7 @@ sub checkforpriv_row {
                     '</b><input type="hidden" name="checkforpriv_'.$num.
                     '" value="'.$currdom.'" /></span><br />'.
                     '<span class="LC_nobreak"><label><input type="checkbox" '.
-                    'name="checkforpriv_delete" value="'.$num.'" />'.
+                    'name="checkforpriv_delete" value="'.$num.'"'.$disabled.' />'.
                     &mt('Delete').'</label></span></td>'.
                     &Apache::loncommon::end_data_table_row();
                 $num ++;
@@ -4226,7 +4555,7 @@ sub checkforpriv_row {
                 '<td align="left"><span class="LC_nobreak">'.
                 &mt('Additional domain:'). '</span><br />'.
                 &Apache::loncommon::select_dom_form('','checkforpriv_'.$num,1,
-                                                    1,undef,undef,\@excdoms).
+                                                    1,undef,undef,\@excdoms,$noedit).
                 '</td>'.&Apache::loncommon::end_data_table_row();
         }
         $datatable .= &Apache::loncommon::end_data_table().
@@ -4237,7 +4566,7 @@ sub checkforpriv_row {
 }
 
 sub print_hdrfmt_row {
-    my ($item,$settings) = @_;
+    my ($item,$settings,$noedit) = @_;
     my @curr;
     my $currnum = 0;
     my $maxnum = 2;
@@ -4310,36 +4639,43 @@ function getIndexByName(item) {
 ENDJS
     $output .= '</td><td align="right"><br />'.
                $currstr.'<table class="LC_data_table">';
+    my $disabled;
+    if ($noedit) {
+        $disabled = ' disabled="disabled"';
+    }
     if (@curr > 0) {
         for (my $i=0; $i<@curr; $i++) {
             my $pos = $i+1;
             $output .= '<tr>'.
                        '<td align="left"><span class="LC_nobreak">'.
-                       &position_selector($pos,$i,$maxnum).&mt('Delete:').
+                       &position_selector($pos,$i,$maxnum,$disabled).&mt('Delete:').
                        '<input type="checkbox" name="printfmthdr_del_'.$i.
-                       '" /></span></td>';
+                       '"'.$disabled.' /></span></td>';
             if ($curr[$i] =~ /^%\d*[nca]$/) {
                 my ($limit,$subst) = ($curr[$i] =~ /^%(\d*)([nca])$/);
                 $output .= '<td align="left">'.
-                           &substitution_selector($i,$subst,$limit).'</td>';
+                           &substitution_selector($i,$subst,$limit,$disabled).'</td>';
             } else {
                 $output .= '<td colspan="2" align="left">'.&mt('Text').'<br />'.
                            '<input type="textbox" name="printfmthdr_text_'.$i.'"'.
-                           ' value="'.$curr[$i].'" size="25" /></td>';
+                           ' value="'.$curr[$i].'" size="25"'.$disabled.' /></td>';
             }
             $output .= '</tr>';
         }
     }
     my $pos = $currnum+1; 
-    $output .= '<tr>'.
+    unless ($noedit) {
+        $output .= 
+               '<tr>'.
                '<td align="left"><span class="LC_nobreak">'.
                &position_selector($pos,$currnum,$maxnum).
                '<b>'.&mt('New').'</b></span></td><td align="left">'.
                &substitution_selector($currnum).'</td>'.
                '</tr>'; 
-    $pos ++;
-    $currnum ++;
-    $output .= '<tr>'.
+        $pos ++;
+        $currnum ++;
+        $output .= 
+               '<tr>'.
                '<td align="left"><span class="LC_nobreak">'.
                &position_selector($pos,$currnum,$maxnum).
                '<b>'.&mt('New').'</b></span></td>'.
@@ -4348,14 +4684,15 @@ ENDJS
                '" value="" size ="25" />'.
                '<input type="hidden" name="printfmthdr_maxnum" value="'.
                 $maxnum.'" /></td>'.
-               '</tr>'.
-               '</table><br />';
+               '</tr>';
+    }
+    $output .= '</table><br />';
     return $output;
 }
 
 sub position_selector {
-    my ($pos,$num,$maxnum) = @_;
-    my $output = '<select name="printfmthdr_pos_'.$num.'" onchange="reOrder('."'$num'".');">';
+    my ($pos,$num,$maxnum,$disabled) = @_;
+    my $output = '<select name="printfmthdr_pos_'.$num.'" onchange="reOrder('."'$num'".');"'.$disabled.'>';
     for (my $j=1; $j<=$maxnum; $j++) {
         my $sel = '';
         if ($pos == $j) {
@@ -4369,7 +4706,7 @@ sub position_selector {
 }
 
 sub substitution_selector {
-    my ($num,$subst,$limit,$crstype) = @_;
+    my ($num,$subst,$limit,$crstype,$disabled) = @_;
     my ($stunametxt,$crsidtxt);
     if ($crstype eq 'Community') {
         $stunametxt = 'member name';
@@ -4384,7 +4721,7 @@ sub substitution_selector {
                     a => 'assignment note',
              );
     my $output .= &mt('Substitution').'<br />'.
-                  '<select name="printfmthdr_sub_'.$num.'">';
+                  '<select name="printfmthdr_sub_'.$num.'"'.$disabled.'>';
     if ($subst eq '') {
         $output .= '<option value="" selected="selected"> </option>';
     }
@@ -4398,7 +4735,7 @@ sub substitution_selector {
     }
     $output .= '</select></td><td align="left">'.&mt('Size limit').'<br />'.
                '<input type="textbox" name="printfmthdr_limit_'.$num.
-               '" value="'.$limit.'" size="5" /></span>';
+               '" value="'.$limit.'" size="5"'.$disabled.' /></span>';
     return $output;
 }
 
@@ -4416,17 +4753,19 @@ sub change_clone {
         foreach my $currclone (@allowclone) {
             if (!grep(/^$currclone$/,@$oldcloner)) {
                 if ($currclone ne '*') {
-                    my ($uname,$udom) = split(/:/,$currclone);
-                    if ($uname && $udom && $uname ne '*') {
-                        if (&Apache::lonnet::homeserver($uname,$udom) ne 'no_host') {
-                            my %currclonecrs = &Apache::lonnet::dump('environment',$udom,$uname,'cloneable');
-                            if ($currclonecrs{'cloneable'} !~ /\Q$clone_crs\E/) {
-                                if ($currclonecrs{'cloneable'} eq '') {
-                                    $currclonecrs{'cloneable'} = $clone_crs;
-                                } else {
-                                    $currclonecrs{'cloneable'} .= ','.$clone_crs;
+                    if ($currclone =~ /:/) {
+                        my ($uname,$udom) = split(/:/,$currclone);
+                        if ($uname && $udom && $uname ne '*') {
+                            if (&Apache::lonnet::homeserver($uname,$udom) ne 'no_host') {
+                                my %currclonecrs = &Apache::lonnet::dump('environment',$udom,$uname,'cloneable');
+                                if ($currclonecrs{'cloneable'} !~ /\Q$clone_crs\E/) {
+                                    if ($currclonecrs{'cloneable'} eq '') {
+                                        $currclonecrs{'cloneable'} = $clone_crs;
+                                    } else {
+                                        $currclonecrs{'cloneable'} .= ','.$clone_crs;
+                                    }
+                                    &Apache::lonnet::put('environment',\%currclonecrs,$udom,$uname);
                                 }
-                                &Apache::lonnet::put('environment',\%currclonecrs,$udom,$uname);
                             }
                         }
                     }
@@ -4436,24 +4775,26 @@ sub change_clone {
         foreach my $oldclone (@$oldcloner) {
             if (!grep(/^\Q$oldclone\E$/,@allowclone)) {
                 if ($oldclone ne '*') {
-                    my ($uname,$udom) = split(/:/,$oldclone);
-                    if ($uname && $udom && $uname ne '*' ) {
-                        if (&Apache::lonnet::homeserver($uname,$udom) ne 'no_host') {
-                            my %currclonecrs = &Apache::lonnet::dump('environment',$udom,$uname,'cloneable');
-                            my %newclonecrs = ();
-                            if ($currclonecrs{'cloneable'} =~ /\Q$clone_crs\E/) {
-                                if ($currclonecrs{'cloneable'} =~ /,/) {
-                                    my @currclonecrs = split/,/,$currclonecrs{'cloneable'};
-                                    foreach my $crs (@currclonecrs) {
-                                        if ($crs ne $clone_crs) {
-                                            $newclonecrs{'cloneable'} .= $crs.',';
+                    if ($oldclone =~ /:/) {
+                        my ($uname,$udom) = split(/:/,$oldclone);
+                        if ($uname && $udom && $uname ne '*' ) {
+                            if (&Apache::lonnet::homeserver($uname,$udom) ne 'no_host') {
+                                my %currclonecrs = &Apache::lonnet::dump('environment',$udom,$uname,'cloneable');
+                                my %newclonecrs = ();
+                                if ($currclonecrs{'cloneable'} =~ /\Q$clone_crs\E/) {
+                                    if ($currclonecrs{'cloneable'} =~ /,/) {
+                                        my @currclonecrs = split/,/,$currclonecrs{'cloneable'};
+                                        foreach my $crs (@currclonecrs) {
+                                            if ($crs ne $clone_crs) {
+                                                $newclonecrs{'cloneable'} .= $crs.',';
+                                            }
                                         }
+                                        $newclonecrs{'cloneable'} =~ s/,$//;
+                                    } else {
+                                        $newclonecrs{'cloneable'} = '';
                                     }
-                                    $newclonecrs{'cloneable'} =~ s/,$//;
-                                } else {
-                                    $newclonecrs{'cloneable'} = '';
+                                    &Apache::lonnet::put('environment',\%newclonecrs,$udom,$uname);
                                 }
-                                &Apache::lonnet::put('environment',\%newclonecrs,$udom,$uname);
                             }
                         }
                     }