--- loncom/interface/loncommon.pm	2020/02/04 21:54:58	1.1075.2.142
+++ loncom/interface/loncommon.pm	2012/12/27 00:00:23	1.1107
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # a pile of common routines
 #
-# $Id: loncommon.pm,v 1.1075.2.142 2020/02/04 21:54:58 raeburn Exp $
+# $Id: loncommon.pm,v 1.1107 2012/12/27 00:00:23 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -67,22 +67,12 @@ use Apache::lonhtmlcommon();
 use Apache::loncoursedata();
 use Apache::lontexconvert();
 use Apache::lonclonecourse();
-use Apache::lonuserutils();
-use Apache::lonuserstate();
-use Apache::courseclassifier();
 use LONCAPA qw(:DEFAULT :match);
-use HTTP::Request;
 use DateTime::TimeZone;
-use DateTime::Locale;
-use Encode();
+use DateTime::Locale::Catalog;
+use Text::Aspell;
 use Authen::Captcha;
 use Captcha::reCAPTCHA;
-use JSON::DWIW;
-use LWP::UserAgent;
-use Crypt::DES;
-use DynaLoader; # for Crypt::DES version
-use File::Copy();
-use File::Path();
 
 # ---------------------------------------------- Designs
 use vars qw(%defaultdesign);
@@ -167,6 +157,7 @@ sub ssi_with_retries {
 # ----------------------------------------------- Filetypes/Languages/Copyright
 my %language;
 my %supported_language;
+my %supported_codes;
 my %latex_language;		# For choosing hyphenation in <transl..>
 my %latex_language_bykey;	# for choosing hyphenation from metadata
 my %cprtag;
@@ -197,18 +188,19 @@ BEGIN {
     {
         my $langtabfile = $Apache::lonnet::perlvar{'lonTabDir'}.
                                    '/language.tab';
-        if ( open(my $fh,'<',$langtabfile) ) {
+        if ( open(my $fh,"<$langtabfile") ) {
             while (my $line = <$fh>) {
                 next if ($line=~/^\#/);
                 chomp($line);
-                my ($key,$two,$country,$three,$enc,$val,$sup,$latex)=(split(/\t/,$line));
+                my ($key,$code,$country,$three,$enc,$val,$sup,$latex)=(split(/\t/,$line));
                 $language{$key}=$val.' - '.$enc;
                 if ($sup) {
                     $supported_language{$key}=$sup;
+		    $supported_codes{$key}   = $code;
                 }
 		if ($latex) {
 		    $latex_language_bykey{$key} = $latex;
-		    $latex_language{$two} = $latex;
+		    $latex_language{$code} = $latex;
 		}
             }
             close($fh);
@@ -218,7 +210,7 @@ BEGIN {
     {
         my $copyrightfile = $Apache::lonnet::perlvar{'lonIncludes'}.
                                   '/copyright.tab';
-        if ( open (my $fh,'<',$copyrightfile) ) {
+        if ( open (my $fh,"<$copyrightfile") ) {
             while (my $line = <$fh>) {
                 next if ($line=~/^\#/);
                 chomp($line);
@@ -232,7 +224,7 @@ BEGIN {
     {
         my $sourcecopyrightfile = $Apache::lonnet::perlvar{'lonIncludes'}.
                                   '/source_copyright.tab';
-        if ( open (my $fh,'<',$sourcecopyrightfile) ) {
+        if ( open (my $fh,"<$sourcecopyrightfile") ) {
             while (my $line = <$fh>) {
                 next if ($line =~ /^\#/);
                 chomp($line);
@@ -246,7 +238,7 @@ BEGIN {
 # -------------------------------------------------------------- default domain designs
     my $designdir=$Apache::lonnet::perlvar{'lonTabDir'}.'/lonDomColors';
     my $designfile = $designdir.'/default.tab';
-    if ( open (my $fh,'<',$designfile) ) {
+    if ( open (my $fh,"<$designfile") ) {
         while (my $line = <$fh>) {
             next if ($line =~ /^\#/);
             chomp($line);
@@ -260,12 +252,12 @@ BEGIN {
     {
         my $categoryfile = $Apache::lonnet::perlvar{'lonTabDir'}.
                                   '/filecategories.tab';
-        if ( open (my $fh,'<',$categoryfile) ) {
+        if ( open (my $fh,"<$categoryfile") ) {
 	    while (my $line = <$fh>) {
 		next if ($line =~ /^\#/);
 		chomp($line);
                 my ($extension,$category)=(split(/\s+/,$line,2));
-                push(@{$category_extensions{lc($category)}},$extension);
+                push @{$category_extensions{lc($category)}},$extension;
             }
             close($fh);
         }
@@ -275,7 +267,7 @@ BEGIN {
     {
         my $typesfile = $Apache::lonnet::perlvar{'lonTabDir'}.
                '/filetypes.tab';
-        if ( open (my $fh,'<',$typesfile) ) {
+        if ( open (my $fh,"<$typesfile") ) {
             while (my $line = <$fh>) {
 		next if ($line =~ /^\#/);
 		chomp($line);
@@ -537,8 +529,7 @@ ENDAUTHORBRW
 }
 
 sub coursebrowser_javascript {
-    my ($domainfilter,$sec_element,$formname,$role_element,$crstype,
-        $credits_element,$instcode) = @_;
+    my ($domainfilter,$sec_element,$formname,$role_element,$crstype) = @_;
     my $wintitle = 'Course_Browser';
     if ($crstype eq 'Community') {
         $wintitle = 'Community_Browser';
@@ -589,12 +580,6 @@ sub coursebrowser_javascript {
             var ownername = document.forms[formid].ccuname.value;
             var ownerdom =  document.forms[formid].ccdomain.options[document.forms[formid].ccdomain.selectedIndex].value;
             url += '&cloner='+ownername+':'+ownerdom;
-            if (type == 'Course') {
-                url += '&crscode='+document.forms[formid].crscode.value;
-            }
-        }
-        if (formname == 'requestcrs') {
-            url += '&crsdom=$domainfilter&crscode=$instcode';
         }
         if (multflag !=null && multflag != '') {
             url += '&multiple='+multflag;
@@ -607,9 +592,8 @@ sub coursebrowser_javascript {
     }
 $id_functions
 ENDSTDBRW
-    if (($sec_element ne '') || ($role_element ne '') || ($credits_element ne '')) {
-        $output .= &setsec_javascript($sec_element,$formname,$role_element,
-                                      $credits_element);
+    if (($sec_element ne '') || ($role_element ne '')) {
+        $output .= &setsec_javascript($sec_element,$formname,$role_element);
     }
     $output .= '
 // ]]>
@@ -678,7 +662,7 @@ if (!Array.prototype.indexOf) {
         var n = 0;
         if (arguments.length > 0) {
             n = Number(arguments[1]);
-            if (n !== n) { // shortcut for verifying if it's NaN
+            if (n !== n) { // shortcut for verifying if it is NaN
                 n = 0;
             } else if (n !== 0 && n !== (1 / 0) && n !== -(1 / 0)) {
                 n = (n > 0 || -1) * Math.floor(Math.abs(n));
@@ -766,7 +750,7 @@ ENDUSERBRW
 }
 
 sub setsec_javascript {
-    my ($sec_element,$formname,$role_element,$credits_element) = @_;
+    my ($sec_element,$formname,$role_element) = @_;
     my (@courserolenames,@communityrolenames,$rolestr,$courserolestr,
         $communityrolestr);
     if ($role_element ne '') {
@@ -861,14 +845,6 @@ function setRole(crstype) {
 }
 |;
     }
-    if ($credits_element) {
-        $setsections .= qq|
-function setCredits(defaultcredits) {
-    document.$formname.$credits_element.value = defaultcredits;
-    return;
-}
-|;
-    }
     return $setsections;
 }
 
@@ -914,12 +890,12 @@ sub check_uncheck_jscript {
 function checkAll(field) {
     if (field.length > 0) {
         for (i = 0; i < field.length; i++) {
-            if (!field[i].disabled) {
+            if (!field[i].disabled) { 
                 field[i].checked = true;
             }
         }
     } else {
-        if (!field.disabled) {
+        if (!field.disabled) { 
             field.checked = true;
         }
     }
@@ -939,8 +915,8 @@ ENDSCRT
 }
 
 sub select_timezone {
-   my ($name,$selected,$onchange,$includeempty,$disabled)=@_;
-   my $output='<select name="'.$name.'" '.$onchange.$disabled.'>'."\n";
+   my ($name,$selected,$onchange,$includeempty)=@_;
+   my $output='<select name="'.$name.'" '.$onchange.'>'."\n";
    if ($includeempty) {
        $output .= '<option value=""';
        if (($selected eq '') || ($selected eq 'local')) {
@@ -961,8 +937,8 @@ sub select_timezone {
 }
 
 sub select_datelocale {
-    my ($name,$selected,$onchange,$includeempty,$disabled)=@_;
-    my $output='<select name="'.$name.'" '.$onchange.$disabled.'>'."\n";
+    my ($name,$selected,$onchange,$includeempty)=@_;
+    my $output='<select name="'.$name.'" '.$onchange.'>'."\n";
     if ($includeempty) {
         $output .= '<option value=""';
         if ($selected eq '') {
@@ -970,16 +946,15 @@ sub select_datelocale {
         }
         $output .= '> </option>';
     }
-    my @languages = &Apache::lonlocal::preferred_languages();
     my (@possibles,%locale_names);
-    my @locales = DateTime::Locale->ids();
-    foreach my $id (@locales) {
-        if ($id ne '') {
-            my ($en_terr,$native_terr);
-            my $loc = DateTime::Locale->load($id);
-            if (ref($loc)) {
-                $en_terr = $loc->name();
-                $native_terr = $loc->native_name();
+    my @locales = DateTime::Locale::Catalog::Locales;
+    foreach my $locale (@locales) {
+        if (ref($locale) eq 'HASH') {
+            my $id = $locale->{'id'};
+            if ($id ne '') {
+                my $en_terr = $locale->{'en_territory'};
+                my $native_terr = $locale->{'native_territory'};
+                my @languages = &Apache::lonlocal::preferred_languages();
                 if (grep(/^en$/,@languages) || !@languages) {
                     if ($en_terr ne '') {
                         $locale_names{$id} = '('.$en_terr.')';
@@ -993,8 +968,7 @@ sub select_datelocale {
                         $locale_names{$id} = '('.$en_terr.')';
                     }
                 }
-                $locale_names{$id} = Encode::encode('UTF-8',$locale_names{$id});
-                push(@possibles,$id);
+                push (@possibles,$id);
             }
         }
     }
@@ -1005,7 +979,7 @@ sub select_datelocale {
         }
         $output.=">$item";
         if ($locale_names{$item} ne '') {
-            $output.='  '.$locale_names{$item};
+            $output.="  $locale_names{$item}</option>\n";
         }
         $output.="</option>\n";
     }
@@ -1014,7 +988,7 @@ sub select_datelocale {
 }
 
 sub select_language {
-    my ($name,$selected,$includeempty,$noedit) = @_;
+    my ($name,$selected,$includeempty) = @_;
     my %langchoices;
     if ($includeempty) {
         %langchoices = ('' => 'No language preference');
@@ -1025,8 +999,34 @@ sub select_language {
             $langchoices{$code} = &plainlanguagedescription($id);
         }
     }
-    %langchoices = &Apache::lonlocal::texthash(%langchoices);
-    return &select_form($selected,$name,\%langchoices,undef,$noedit);
+    return &select_form($selected,$name,\%langchoices);
+}
+
+=pod
+
+
+=item * &list_languages()
+
+Returns an array reference that is suitable for use in language prompters.
+Each array element is itself a two element array.  The first element
+is the language code.  The second element a descsriptiuon of the 
+language itself.  This is suitable for use in e.g.
+&Apache::edit::select_arg (once dereferenced that is).
+
+=cut 
+
+sub list_languages {
+    my @lang_choices;
+
+    foreach my $id (&languageids()) {
+	my $code = &supportedlanguagecode($id);
+	if ($code) {
+	    my $selector    = $supported_codes{$id};
+	    my $description = &plainlanguagedescription($id);
+	    push (@lang_choices, [$selector, $description]);
+	}
+    }
+    return \@lang_choices;
 }
 
 =pod
@@ -1057,12 +1057,6 @@ linked_select_forms takes the following
 
 =item * $menuorder, the order of values in the first menu
 
-=item * $onchangefirst, additional javascript call to execute for an onchange
-        event for the first <select> tag
-
-=item * $onchangesecond, additional javascript call to execute for an onchange
-        event for the second <select> tag
-
 =back 
 
 Below is an example of such a hash.  Only the 'text', 'default', and 
@@ -1116,8 +1110,6 @@ sub linked_select_forms {
         $secondselectname, 
         $hashref,
         $menuorder,
-        $onchangefirst,
-        $onchangesecond
         ) = @_;
     my $second = "document.$formname.$secondselectname";
     my $first = "document.$formname.$firstselectname";
@@ -1141,7 +1133,7 @@ sub linked_select_forms {
         $result.="select2data.d_$s1.texts = new Array(";        
         my @s2texts;
         foreach my $value (@s2values) {
-            push(@s2texts, $hashref->{$s1}->{'select2'}->{$value});
+            push @s2texts, $hashref->{$s1}->{'select2'}->{$value};
         }
         $result.="\"@s2texts\");\n";
     }
@@ -1174,7 +1166,7 @@ function select1_changed() {
 </script>
 END
     # output the initial values for the selection lists
-    $result .= "<select size=\"1\" name=\"$firstselectname\" onchange=\"select1_changed();$onchangefirst\">\n";
+    $result .= "<select size=\"1\" name=\"$firstselectname\" onchange=\"select1_changed()\">\n";
     my @order = sort(keys(%{$hashref}));
     if (ref($menuorder) eq 'ARRAY') {
         @order = @{$menuorder};
@@ -1187,11 +1179,7 @@ END
     $result .= "</select>\n";
     my %select2 = %{$hashref->{$firstdefault}->{'select2'}};
     $result .= $middletext;
-    $result .= "<select size=\"1\" name=\"$secondselectname\"";
-    if ($onchangesecond) {
-        $result .= ' onchange="'.$onchangesecond.'"';
-    }
-    $result .= ">\n";
+    $result .= "<select size=\"1\" name=\"$secondselectname\">\n";
     my $seconddefault = $hashref->{$firstdefault}->{'default'};
     
     my @secondorder = sort(keys(%select2));
@@ -1251,11 +1239,7 @@ sub help_open_topic {
     $topic=~s/\W/\_/g;
 
     if (!$stayOnPage) {
-        if ($env{'browser.mobile'}) {
-	    $link = "javascript:openMyModal('/adm/help/${filename}.hlp',$width,$height,'yes');";
-        } else {
-            $link = "javascript:void(open('/adm/help/${filename}.hlp', 'Help_for_$topic', 'menubar=0,toolbar=1,scrollbars=1,width=$width,height=$height,resizable=yes'))";
-        }
+	$link = "javascript:openMyModal('/adm/help/${filename}.hlp',$width,$height,'yes');";
     } elsif ($stayOnPage eq 'popup') {
         $link = "javascript:void(open('/adm/help/${filename}.hlp', 'Help_for_$topic', 'menubar=0,toolbar=1,scrollbars=1,width=$width,height=$height,resizable=yes'))";
     } else {
@@ -1306,9 +1290,7 @@ sub helpLatexCheatsheet {
     unless ($not_author) {
         $out .= ' <span>'
 	       .&help_open_topic('Authoring_Output_Tags',&mt('Output Tags'),$stayOnPage,undef,600)
-	       .'</span> <span>'
-               .&help_open_topic('Authoring_Multilingual_Problems',&mt('Languages'),$stayOnPage,undef,600)
-               .'</span>';
+	       .'</span>';
     }
     $out .= '</span>'; # End cheatsheet
     return $out;
@@ -1371,40 +1353,32 @@ sub help_open_menu {
 sub top_nav_help {
     my ($text) = @_;
     $text = &mt($text);
-    my $stay_on_page;
-    unless ($env{'environment.remote'} eq 'on') {
-        $stay_on_page = 1;
-    }
-    my ($link,$banner_link);
-    unless ($env{'request.noversionuri'} =~ m{^/adm/helpmenu}) {
-        $link = ($stay_on_page) ? "javascript:helpMenu('display')"
-	                         : "javascript:helpMenu('open')";
-        $banner_link = &update_help_link(undef,undef,undef,undef,$stay_on_page);
-    }
+    my $stay_on_page = 1;
+
+    my $link = ($stay_on_page) ? "javascript:helpMenu('display')"
+	                     : "javascript:helpMenu('open')";
+    my $banner_link = &update_help_link(undef,undef,undef,undef,$stay_on_page);
+
     my $title = &mt('Get help');
-    if ($link) {
-        return <<"END";
+
+    return <<"END";
 $banner_link
-<a href="$link" title="$title">$text</a>
+ <a href="$link" title="$title">$text</a>
 END
-    } else {
-        return '&nbsp;'.$text.'&nbsp;';
-    }
 }
 
 sub help_menu_js {
-    my ($httphost) = @_;
+    my ($text) = @_;
     my $stayOnPage = 1;
     my $width = 620;
     my $height = 600;
     my $helptopic=&general_help();
-    my $details_link = $httphost.'/adm/help/'.$helptopic.'.hlp';
+    my $details_link = '/adm/help/'.$helptopic.'.hlp';
     my $nothing=&Apache::lonhtmlcommon::javascript_nothing();
     my $start_page =
         &Apache::loncommon::start_page('Help Menu', undef,
 				       {'frameset'    => 1,
 					'js_ready'    => 1,
-                                        'use_absolute' => $httphost,
 					'add_entries' => {
 					    'border' => '0',
 					    'rows'   => "110,*",},});
@@ -1436,10 +1410,9 @@ function helpMenu(target) {
     return;
 }
 function writeHelp(caller) {
-    caller.document.writeln('$start_page\\n<frame name="bannerframe" src="'+banner_link+'" marginwidth="0" marginheight="0" frameborder="0">\\n');
-    caller.document.writeln('<frame name="bodyframe" src="$details_link" marginwidth="0" marginheight="0" frameborder="0">\\n$end_page');
-    caller.document.close();
-    caller.focus();
+    caller.document.writeln('$start_page\\n<frame name="bannerframe" src="'+banner_link+'" />\\n<frame name="bodyframe" src="$details_link" />\\n$end_page')
+    caller.document.close()
+    caller.focus()
 }
 // END LON-CAPA Internal -->
 // ]]>
@@ -1747,246 +1720,12 @@ RESIZE
 
 }
 
-sub colorfuleditor_js {
-    return <<"COLORFULEDIT"
-<script type="text/javascript">
-// <![CDATA[>
-    function fold_box(curDepth, lastresource){
-
-    // we need a list because there can be several blocks you need to fold in one tag
-        var block = document.getElementsByName('foldblock_'+curDepth);
-    // but there is only one folding button per tag
-        var foldbutton = document.getElementById('folding_btn_'+curDepth);
-
-        if(block.item(0).style.display == 'none'){
-
-            foldbutton.value = '@{[&mt("Hide")]}';
-            for (i = 0; i < block.length; i++){
-                block.item(i).style.display = '';
-            }
-        }else{
-
-            foldbutton.value = '@{[&mt("Show")]}';
-            for (i = 0; i < block.length; i++){
-                // block.item(i).style.visibility = 'collapse';
-                block.item(i).style.display = 'none';
-            }
-        };
-        saveState(lastresource);
-    }
-
-    function saveState (lastresource) {
-
-        var tag_list = getTagList();
-        if(tag_list != null){
-            var timestamp = new Date().getTime();
-            var key = lastresource;
-
-            // the value pattern is: 'time;key1,value1;key2,value2; ... '
-            // starting with timestamp
-            var value = timestamp+';';
-
-            // building the list of key-value pairs
-            for(var i = 0; i < tag_list.length; i++){
-                value += tag_list[i]+',';
-                value += document.getElementsByName(tag_list[i])[0].style.display+';';
-            }
-
-            // only iterate whole storage if nothing to override
-            if(localStorage.getItem(key) == null){
-
-                // prevent storage from growing large
-                if(localStorage.length > 50){
-                    var regex_getTimestamp = /^(?:\d)+;/;
-                    var oldest_timestamp = regex_getTimestamp.exec(localStorage.key(0));
-                    var oldest_key;
-
-                    for(var i = 1; i < localStorage.length; i++){
-                        if (regex_getTimestamp.exec(localStorage.key(i)) < oldest_timestamp) {
-                            oldest_key = localStorage.key(i);
-                            oldest_timestamp = regex_getTimestamp.exec(oldest_key);
-                        }
-                    }
-                    localStorage.removeItem(oldest_key);
-                }
-            }
-            localStorage.setItem(key,value);
-        }
-    }
-
-    // restore folding status of blocks (on page load)
-    function restoreState (lastresource) {
-        if(localStorage.getItem(lastresource) != null){
-            var key = lastresource;
-            var value = localStorage.getItem(key);
-            var regex_delTimestamp = /^\d+;/;
-
-            value.replace(regex_delTimestamp, '');
-
-            var valueArr = value.split(';');
-            var pairs;
-            var elements;
-            for (var i = 0; i < valueArr.length; i++){
-                pairs = valueArr[i].split(',');
-                elements = document.getElementsByName(pairs[0]);
-
-                for (var j = 0; j < elements.length; j++){
-                    elements[j].style.display = pairs[1];
-                    if (pairs[1] == "none"){
-                        var regex_id = /([_\\d]+)\$/;
-                        regex_id.exec(pairs[0]);
-                        document.getElementById("folding_btn"+RegExp.\$1).value = "Show";
-                    }
-                }
-            }
-        }
-    }
-
-    function getTagList () {
-
-        var stringToSearch = document.lonhomework.innerHTML;
-
-        var ret = new Array();
-        var regex_findBlock = /(foldblock_.*?)"/g;
-        var tag_list = stringToSearch.match(regex_findBlock);
-
-        if(tag_list != null){
-            for(var i = 0; i < tag_list.length; i++){
-                ret.push(tag_list[i].replace(/"/, ''));
-            }
-        }
-        return ret;
-    }
-
-    function saveScrollPosition (resource) {
-        var tag_list = getTagList();
-
-        // we dont always want to jump to the first block
-        // 170 is roughly above the "Problem Editing" header. we just want to save if the user scrolled down further than this
-        if(\$(window).scrollTop() > 170){
-            if(tag_list != null){
-                var result;
-                for(var i = 0; i < tag_list.length; i++){
-                    if(isElementInViewport(tag_list[i])){
-                        result += tag_list[i]+';';
-                    }
-                }
-                sessionStorage.setItem('anchor_'+resource, result);
-            }
-        } else {
-            // we dont need to save zero, just delete the item to leave everything tidy
-            sessionStorage.removeItem('anchor_'+resource);
-        }
-    }
-
-    function restoreScrollPosition(resource){
-
-        var elem = sessionStorage.getItem('anchor_'+resource);
-        if(elem != null){
-            var tag_list = elem.split(';');
-            var elem_list;
-
-            for(var i = 0; i < tag_list.length; i++){
-                elem_list = document.getElementsByName(tag_list[i]);
-
-                if(elem_list.length > 0){
-                    elem = elem_list[0];
-                    break;
-                }
-            }
-            elem.scrollIntoView();
-        }
-    }
-
-    function isElementInViewport(el) {
-
-        // change to last element instead of first
-        var elem = document.getElementsByName(el);
-        var rect = elem[0].getBoundingClientRect();
-
-        return (
-            rect.top >= 0 &&
-            rect.left >= 0 &&
-            rect.bottom <= (window.innerHeight || document.documentElement.clientHeight) && /*or $(window).height() */
-            rect.right <= (window.innerWidth || document.documentElement.clientWidth) /*or $(window).width() */
-        );
-    }
-
-    function autosize(depth){
-        var cmInst = window['cm'+depth];
-        var fitsizeButton = document.getElementById('fitsize'+depth);
-
-        // is fixed size, switching to dynamic
-        if (sessionStorage.getItem("autosized_"+depth) == null) {
-            cmInst.setSize("","auto");
-            fitsizeButton.value = "@{[&mt('Fixed size')]}";
-            sessionStorage.setItem("autosized_"+depth, "yes");
-
-        // is dynamic size, switching to fixed
-        } else {
-            cmInst.setSize("","300px");
-            fitsizeButton.value = "@{[&mt('Dynamic size')]}";
-            sessionStorage.removeItem("autosized_"+depth);
-        }
-    }
-
-
-
-// ]]>
-</script>
-COLORFULEDIT
-}
-
-sub xmleditor_js {
-    return <<XMLEDIT
-<script type="text/javascript" src="/adm/jQuery/addons/jquery-scrolltofixed.js"></script>
-<script type="text/javascript">
-// <![CDATA[>
-
-    function saveScrollPosition (resource) {
-
-        var scrollPos = \$(window).scrollTop();
-        sessionStorage.setItem(resource,scrollPos);
-    }
-
-    function restoreScrollPosition(resource){
-
-        var scrollPos = sessionStorage.getItem(resource);
-        \$(window).scrollTop(scrollPos);
-    }
-
-    // unless internet explorer
-    if (!(window.navigator.appName == "Microsoft Internet Explorer" && (document.documentMode || document.compatMode))){
-
-        \$(document).ready(function() {
-             \$(".LC_edit_actionbar").scrollToFixed(\{zIndex: 100\});
-        });
-    }
-
-    // inserts text at cursor position into codemirror (xml editor only)
-    function insertText(text){
-        cm.focus();
-        var curPos = cm.getCursor();
-        cm.replaceRange(text.replace(/ESCAPEDSCRIPT/g,'script'), {line: curPos.line,ch: curPos.ch});
-    }
-// ]]>
-</script>
-XMLEDIT
-}
-
-sub insert_folding_button {
-    my $curDepth = $Apache::lonxml::curdepth;
-    my $lastresource = $env{'request.ambiguous'};
-
-    return "<input type=\"button\" id=\"folding_btn_$curDepth\"
-            value=\"".&mt('Hide')."\" onclick=\"fold_box('$curDepth','$lastresource')\">";
-}
-
-
 =pod
 
 =head1 Excel and CSV file utility routines
 
+=over 4
+
 =cut
 
 ###############################################################
@@ -1994,8 +1733,6 @@ sub insert_folding_button {
 
 =pod
 
-=over 4
-
 =item * &csv_translate($text) 
 
 Translate $text to allow it to be output as a 'comma separated values' 
@@ -2242,15 +1979,12 @@ sub multiple_select_form {
 
 =pod
 
-=item * &select_form($defdom,$name,$hashref,$onchange,$readonly)
+=item * &select_form($defdom,$name,$hashref,$onchange)
 
 Returns a string containing a <select name='$name' size='1'> form to 
 allow a user to select options from a ref to a hash containing:
 option_name => displayed text. An optional $onchange can include
-a javascript onchange item, e.g., onchange="this.form.submit();".
-An optional arg -- $readonly -- if true will cause the select form
-to be disabled, e.g., for the case where an instructor has a section-
-specific role, and is viewing/modifying parameters.  
+a javascript onchange item, e.g., onchange="this.form.submit();"  
 
 See lonrights.pm for an example invocation and use.
 
@@ -2258,16 +1992,12 @@ See lonrights.pm for an example invocati
 
 #-------------------------------------------
 sub select_form {
-    my ($def,$name,$hashref,$onchange,$readonly) = @_;
+    my ($def,$name,$hashref,$onchange) = @_;
     return unless (ref($hashref) eq 'HASH');
     if ($onchange) {
         $onchange = ' onchange="'.$onchange.'"';
     }
-    my $disabled;
-    if ($readonly) {
-        $disabled = ' disabled="disabled"';
-    }
-    my $selectform = "<select name=\"$name\" size=\"1\"$onchange$disabled>\n";
+    my $selectform = "<select name=\"$name\" size=\"1\"$onchange>\n";
     my @keys;
     if (exists($hashref->{'select_form_order'})) {
 	@keys=@{$hashref->{'select_form_order'}};
@@ -2436,7 +2166,7 @@ sub select_level_form {
 
 =pod
 
-=item * &select_dom_form($defdom,$name,$includeempty,$showdomdesc,$onchange,$incdoms,$excdoms,$disabled)
+=item * &select_dom_form($defdom,$name,$includeempty,$showdomdesc,$onchange,$incdoms)
 
 Returns a string containing a <select name='$name' size='1'> form to 
 allow a user to select the domain to preform an operation in.  
@@ -2449,36 +2179,25 @@ If the $showdomdesc flag is set, the dom
 
 The optional $onchange argument specifies what should occur if the domain selector is changed, e.g., 'this.form.submit()' if the form is to be automatically submitted.
 
-The optional $incdoms is a reference to an array of domains which will be the only available options.
-
-The optional $excdoms is a reference to an array of domains which will be excluded from the available options.
-
-The optional $disabled argument, if true, adds the disabled attribute to the select tag. 
+The optional $incdoms is a reference to an array of domains which will be the only available options. 
 
 =cut
 
 #-------------------------------------------
 sub select_dom_form {
-    my ($defdom,$name,$includeempty,$showdomdesc,$onchange,$incdoms,$excdoms,$disabled) = @_;
+    my ($defdom,$name,$includeempty,$showdomdesc,$onchange,$incdoms) = @_;
     if ($onchange) {
         $onchange = ' onchange="'.$onchange.'"';
     }
-    if ($disabled) {
-        $disabled = ' disabled="disabled"';
-    }
-    my (@domains,%exclude);
+    my @domains;
     if (ref($incdoms) eq 'ARRAY') {
         @domains = sort {lc($a) cmp lc($b)} (@{$incdoms});
     } else {
         @domains = sort {lc($a) cmp lc($b)} (&Apache::lonnet::all_domains());
     }
     if ($includeempty) { @domains=('',@domains); }
-    if (ref($excdoms) eq 'ARRAY') {
-        map { $exclude{$_} = 1; } @{$excdoms};
-    }
-    my $selectdomain = "<select name=\"$name\" size=\"1\"$onchange$disabled>\n";
+    my $selectdomain = "<select name=\"$name\" size=\"1\"$onchange>\n";
     foreach my $dom (@domains) {
-        next if ($exclude{$dom});
         $selectdomain.="<option value=\"$dom\" ".
             ($dom eq $defdom ? 'selected="selected" ' : '').'>'.$dom;
         if ($showdomdesc) {
@@ -2599,12 +2318,6 @@ Outputs:
 
 =item * $clientos
 
-=item * $clientmobile
-
-=item * $clientinfo
-
-=item * $clientosversion
-
 =back
 
 =back 
@@ -2623,10 +2336,8 @@ sub decode_user_agent {
     my $clientversion='0';
     my $clientmathml='';
     my $clientunicode='0';
-    my $clientmobile=0;
-    my $clientosversion='';
     for (my $i=0;$i<=$#browsertype;$i++) {
-        my ($bname,$match,$notmatch,$vreg,$minv,$univ)=split(/\%/,$browsertype[$i]);
+        my ($bname,$match,$notmatch,$vreg,$minv,$univ)=split(/\:/,$browsertype[$i]);
 	if (($httpbrowser=~/$match/i)  && ($httpbrowser!~/$notmatch/i)) {
 	    $clientbrowser=$bname;
             $httpbrowser=~/$vreg/i;
@@ -2636,7 +2347,6 @@ sub decode_user_agent {
 	}
     }
     my $clientos='unknown';
-    my $clientinfo;
     if (($httpbrowser=~/linux/i) ||
         ($httpbrowser=~/unix/i) ||
         ($httpbrowser=~/ux/i) ||
@@ -2646,24 +2356,10 @@ sub decode_user_agent {
     if ($httpbrowser=~/next/i) { $clientos='next'; }
     if (($httpbrowser=~/mac/i) ||
         ($httpbrowser=~/powerpc/i)) { $clientos='mac'; }
-    if ($httpbrowser=~/win/i) {
-        $clientos='win';
-        if ($httpbrowser =~/Windows\s+NT\s+(\d+\.\d+)/i) {
-            $clientosversion = $1;
-        }
-    }
+    if ($httpbrowser=~/win/i) { $clientos='win'; }
     if ($httpbrowser=~/embed/i) { $clientos='pda'; }
-    if ($httpbrowser=~/(Android|iPod|iPad|iPhone|webOS|Blackberry|Windows Phone|Opera m(?:ob|in)|Fennec)/i) {
-        $clientmobile=lc($1);
-    }
-    if ($httpbrowser=~ m{Firefox/(\d+\.\d+)}) {
-        $clientinfo = 'firefox-'.$1;
-    } elsif ($httpbrowser=~ m{chromeframe/(\d+\.\d+)\.}) {
-        $clientinfo = 'chromeframe-'.$1;
-    }
     return ($httpbrowser,$clientbrowser,$clientversion,$clientmathml,
-            $clientunicode,$clientos,$clientmobile,$clientinfo,
-            $clientosversion);
+            $clientunicode,$clientos,);
 }
 
 ###############################################################
@@ -2830,7 +2526,7 @@ sub authform_nochange {
               kerb_def_dom => 'MSU.EDU',
               @_,
           );
-    my ($authnum,%can_assign) = &get_assignable_auth($in{'domain'}); 
+    my ($authnum,%can_assign) = &get_assignable_auth($in{'domain'});
     my $result;
     if (!$authnum) {
         $result = &mt('Under your current role you are not permitted to change login settings for this user');
@@ -2852,16 +2548,13 @@ sub authform_kerberos {
               @_,
               );
     my ($check4,$check5,$krbcheck,$krbarg,$krbver,$result,$authtype,
-        $autharg,$jscall,$disabled);
+        $autharg,$jscall);
     my ($authnum,%can_assign) = &get_assignable_auth($in{'domain'});
     if ($in{'kerb_def_auth'} eq 'krb5') {
        $check5 = ' checked="checked"';
     } else {
        $check4 = ' checked="checked"';
     }
-    if ($in{'readonly'}) {
-        $disabled = ' disabled="disabled"';
-    }
     $krbarg = $in{'kerb_def_dom'};
     if (defined($in{'curr_authtype'})) {
         if ($in{'curr_authtype'} eq 'krb') {
@@ -2906,7 +2599,7 @@ sub authform_kerberos {
         if (defined($in{'mode'})) {
             if ($in{'mode'} eq 'modifycourse') {
                 if ($authnum == 1) {
-                    $authtype = '<input type="radio" name="login" value="krb"'.$disabled.' />';
+                    $authtype = '<input type="radio" name="login" value="krb" />';
                 }
             }
         }
@@ -2915,7 +2608,7 @@ sub authform_kerberos {
     if ($authtype eq '') {
         $authtype = '<input type="radio" name="login" value="krb" '.
                     'onclick="'.$jscall.'" onchange="'.$jscall.'"'.
-                    $krbcheck.$disabled.' />';
+                    $krbcheck.' />';
     }
     if (($can_assign{'krb4'} && $can_assign{'krb5'}) ||
         ($can_assign{'krb4'} && !$can_assign{'krb5'} &&
@@ -2928,9 +2621,9 @@ sub authform_kerberos {
          '<label>'.$authtype,
          '</label><input type="text" size="10" name="krbarg" '.
              'value="'.$krbarg.'" '.
-             'onchange="'.$jscall.'"'.$disabled.' />',
-         '<label><input type="radio" name="krbver" value="4" '.$check4.$disabled.' />',
-         '</label><label><input type="radio" name="krbver" value="5" '.$check5.$disabled.' />',
+             'onchange="'.$jscall.'" />',
+         '<label><input type="radio" name="krbver" value="4" '.$check4.' />',
+         '</label><label><input type="radio" name="krbver" value="5" '.$check5.' />',
 	 '</label>');
     } elsif ($can_assign{'krb4'}) {
         $result .= &mt
@@ -2939,7 +2632,7 @@ sub authform_kerberos {
          '<label>'.$authtype,
          '</label><input type="text" size="10" name="krbarg" '.
              'value="'.$krbarg.'" '.
-             'onchange="'.$jscall.'"'.$disabled.' />',
+             'onchange="'.$jscall.'" />',
          '<label><input type="hidden" name="krbver" value="4" />',
          '</label>');
     } elsif ($can_assign{'krb5'}) {
@@ -2949,7 +2642,7 @@ sub authform_kerberos {
          '<label>'.$authtype,
          '</label><input type="text" size="10" name="krbarg" '.
              'value="'.$krbarg.'" '.
-             'onchange="'.$jscall.'"'.$disabled.' />',
+             'onchange="'.$jscall.'" />',
          '<label><input type="hidden" name="krbver" value="5" />',
          '</label>');
     }
@@ -2962,11 +2655,8 @@ sub authform_internal {
                 kerb_def_dom => 'MSU.EDU',
                 @_,
                 );
-    my ($intcheck,$intarg,$result,$authtype,$autharg,$jscall,$disabled);
+    my ($intcheck,$intarg,$result,$authtype,$autharg,$jscall);
     my ($authnum,%can_assign) = &get_assignable_auth($in{'domain'});
-    if ($in{'readonly'}) {
-        $disabled = ' disabled="disabled"';
-    }
     if (defined($in{'curr_authtype'})) {
         if ($in{'curr_authtype'} eq 'int') {
             if ($can_assign{'int'}) {
@@ -2995,7 +2685,7 @@ sub authform_internal {
         if (defined($in{'mode'})) {
             if ($in{'mode'} eq 'modifycourse') {
                 if ($authnum == 1) {
-                    $authtype = '<input type="radio" name="login" value="int"'.$disabled.' />';
+                    $authtype = '<input type="radio" name="login" value="int" />';
                 }
             }
         }
@@ -3003,14 +2693,14 @@ sub authform_internal {
     $jscall = "javascript:changed_radio('int',$in{'formname'});";
     if ($authtype eq '') {
         $authtype = '<input type="radio" name="login" value="int" '.$intcheck.
-                    ' onchange="'.$jscall.'" onclick="'.$jscall.'"'.$disabled.' />';
+                    ' onchange="'.$jscall.'" onclick="'.$jscall.'" />';
     }
     $autharg = '<input type="password" size="10" name="intarg" value="'.
-               $intarg.'" onchange="'.$jscall.'"'.$disabled.' />';
+               $intarg.'" onchange="'.$jscall.'" />';
     $result = &mt
         ('[_1] Internally authenticated (with initial password [_2])',
          '<label>'.$authtype,'</label>'.$autharg);
-    $result.='<label><input type="checkbox" name="visible" onclick="if (this.checked) { this.form.intarg.type='."'text'".' } else { this.form.intarg.type='."'password'".' }"'.$disabled.' />'.&mt('Visible input').'</label>';
+    $result.="<label><input type=\"checkbox\" name=\"visible\" onclick='if (this.checked) { this.form.intarg.type=\"text\" } else { this.form.intarg.type=\"password\" }' />".&mt('Visible input').'</label>';
     return $result;
 }
 
@@ -3020,11 +2710,8 @@ sub authform_local {
               kerb_def_dom => 'MSU.EDU',
               @_,
               );
-    my ($loccheck,$locarg,$result,$authtype,$autharg,$jscall,$disabled);
+    my ($loccheck,$locarg,$result,$authtype,$autharg,$jscall);
     my ($authnum,%can_assign) = &get_assignable_auth($in{'domain'});
-    if ($in{'readonly'}) {
-        $disabled = ' disabled="disabled"';
-    }
     if (defined($in{'curr_authtype'})) {
         if ($in{'curr_authtype'} eq 'loc') {
             if ($can_assign{'loc'}) {
@@ -3053,7 +2740,7 @@ sub authform_local {
         if (defined($in{'mode'})) {
             if ($in{'mode'} eq 'modifycourse') {
                 if ($authnum == 1) {
-                    $authtype = '<input type="radio" name="login" value="loc"'.$disabled.' />';
+                    $authtype = '<input type="radio" name="login" value="loc" />';
                 }
             }
         }
@@ -3062,10 +2749,10 @@ sub authform_local {
     if ($authtype eq '') {
         $authtype = '<input type="radio" name="login" value="loc" '.
                     $loccheck.' onchange="'.$jscall.'" onclick="'.
-                    $jscall.'"'.$disabled.' />';
+                    $jscall.'" />';
     }
     $autharg = '<input type="text" size="10" name="locarg" value="'.
-               $locarg.'" onchange="'.$jscall.'"'.$disabled.' />';
+               $locarg.'" onchange="'.$jscall.'" />';
     $result = &mt('[_1] Local Authentication with argument [_2]',
                   '<label>'.$authtype,'</label>'.$autharg);
     return $result;
@@ -3077,11 +2764,8 @@ sub authform_filesystem {
               kerb_def_dom => 'MSU.EDU',
               @_,
               );
-    my ($fsyscheck,$result,$authtype,$autharg,$jscall,$disabled);
+    my ($fsyscheck,$result,$authtype,$autharg,$jscall);
     my ($authnum,%can_assign) = &get_assignable_auth($in{'domain'});
-    if ($in{'readonly'}) {
-        $disabled = ' disabled="disabled"';
-    }
     if (defined($in{'curr_authtype'})) {
         if ($in{'curr_authtype'} eq 'fsys') {
             if ($can_assign{'fsys'}) {
@@ -3107,7 +2791,7 @@ sub authform_filesystem {
         if (defined($in{'mode'})) {
             if ($in{'mode'} eq 'modifycourse') {
                 if ($authnum == 1) {
-                    $authtype = '<input type="radio" name="login" value="fsys"'.$disabled.' />';
+                    $authtype = '<input type="radio" name="login" value="fsys" />';
                 }
             }
         }
@@ -3116,16 +2800,16 @@ sub authform_filesystem {
     if ($authtype eq '') {
         $authtype = '<input type="radio" name="login" value="fsys" '.
                     $fsyscheck.' onchange="'.$jscall.'" onclick="'.
-                    $jscall.'"'.$disabled.' />';
+                    $jscall.'" />';
     }
     $autharg = '<input type="text" size="10" name="fsysarg" value=""'.
-               ' onchange="'.$jscall.'"'.$disabled.' />';
+               ' onchange="'.$jscall.'" />';
     $result = &mt
         ('[_1] Filesystem Authenticated (with initial password [_2])',
          '<label><input type="radio" name="login" value="fsys" '.
-         $fsyscheck.'onchange="'.$jscall.'" onclick="'.$jscall.'"'.$disabled.' />',
+         $fsyscheck.'onchange="'.$jscall.'" onclick="'.$jscall.'" />',
          '</label><input type="password" size="10" name="fsysarg" value="" '.
-                  'onchange="'.$jscall.'"'.$disabled.' />');
+                  'onchange="'.$jscall.'" />');
     return $result;
 }
 
@@ -3147,7 +2831,7 @@ sub get_assignable_auth {
             my $context;
             if ($env{'request.role'} =~ /^au/) {
                 $context = 'author';
-            } elsif ($env{'request.role'} =~ /^(dc|dh)/) {
+            } elsif ($env{'request.role'} =~ /^dc/) {
                 $context = 'domain';
             } elsif ($env{'request.course.id'}) {
                 $context = 'course';
@@ -3171,79 +2855,6 @@ sub get_assignable_auth {
     return ($authnum,%can_assign);
 }
 
-sub check_passwd_rules {
-    my ($domain,$plainpass) = @_;
-    my %passwdconf = &Apache::lonnet::get_passwdconf($domain);
-    my ($min,$max,@chars,@brokerule,$warning);
-    $min = $Apache::lonnet::passwdmin;
-    if (ref($passwdconf{'chars'}) eq 'ARRAY') {
-        if ($passwdconf{'min'} =~ /^\d+$/) {
-            if ($passwdconf{'min'} > $min) {
-                $min = $passwdconf{'min'};
-            }
-        }
-        if ($passwdconf{'max'} =~ /^\d+$/) {
-            $max = $passwdconf{'max'};
-        }
-        @chars = @{$passwdconf{'chars'}};
-    }
-    if (($min) && (length($plainpass) < $min)) {
-        push(@brokerule,'min');
-    }
-    if (($max) && (length($plainpass) > $max)) {
-        push(@brokerule,'max');
-    }
-    if (@chars) {
-        my %rules;
-        map { $rules{$_} = 1; } @chars;
-        if ($rules{'uc'}) {
-            unless ($plainpass =~ /[A-Z]/) {
-                push(@brokerule,'uc');
-            }
-        }
-        if ($rules{'lc'}) {
-            unless ($plainpass =~ /[a-z]/) {
-                push(@brokerule,'lc');
-            }
-        }
-        if ($rules{'num'}) {
-            unless ($plainpass =~ /\d/) {
-                push(@brokerule,'num');
-            }
-        }
-        if ($rules{'spec'}) {
-            unless ($plainpass =~ /[!"#$%&'()*+,\-.\/:;<=>?@[\\\]^_`{|}~]/) {
-                push(@brokerule,'spec');
-            }
-        }
-    }
-    if (@brokerule) {
-        my %rulenames = &Apache::lonlocal::texthash(
-            uc   => 'At least one upper case letter',
-            lc   => 'At least one lower case letter',
-            num  => 'At least one number',
-            spec => 'At least one non-alphanumeric',
-        );
-        $rulenames{'uc'} .= ': ABCDEFGHIJKLMNOPQRSTUVWXYZ';
-        $rulenames{'lc'} .= ': abcdefghijklmnopqrstuvwxyz';
-        $rulenames{'num'} .= ': 0123456789';
-        $rulenames{'spec'} .= ': !&quot;\#$%&amp;\'()*+,-./:;&lt;=&gt;?@[\]^_\`{|}~';
-        $rulenames{'min'} = &mt('Minimum password length: [_1]',$min);
-        $rulenames{'max'} = &mt('Maximum password length: [_1]',$max);
-        $warning = &mt('Password did not satisfy the following:').'<ul>';
-        foreach my $rule ('min','max','uc','ls','num','spec') {
-            if (grep(/^$rule$/,@brokerule)) {
-                $warning .= '<li>'.$rulenames{$rule}.'</li>';
-            }
-        }
-        $warning .= '</ul>';
-    }
-    if (wantarray) {
-        return @brokerule;
-    }
-    return $warning;
-}
-
 ###############################################################
 ##    Get Kerberos Defaults for Domain                 ##
 ###############################################################
@@ -3411,6 +3022,45 @@ sub get_related_words {
     untie %thesaurus_db;
     return @Words;
 }
+###############################################################
+#
+#  Spell checking
+#
+
+=pod
+
+=head1 Spell checking
+
+=over 4
+
+=item * &check_spelling($wordlist $language)
+
+Takes a string containing words and feeds it to an external
+spellcheck program via a pipeline. Returns a string containing
+them mis-spelled words.
+
+Parameters:
+
+=over 4
+
+=item - $wordlist
+
+String that will be fed into the spellcheck program.
+
+=item - $language
+
+Language string that specifies the language for which the spell
+check will be performed.
+
+=back
+
+=back
+
+Note: This sub assumes that aspell is installed.
+
+
+=cut
+
 
 =pod
 
@@ -3418,6 +3068,31 @@ sub get_related_words {
 
 =cut
 
+sub check_spelling {
+    my ($wordlist, $language) = @_;
+    my @misspellings;
+    
+    # Generate the speller and set the langauge.
+    # if explicitly selected:
+
+    my $speller = Text::Aspell->new;
+    if ($language) {
+	$speller->set_option('lang', $language);
+    }
+
+    # Turn the word list into an array of words by splittingon whitespace
+
+    my @words = split(/\s+/, $wordlist);
+
+    foreach my $word (@words) {
+	if(! $speller->check($word)) {
+	    push(@misspellings, $word);
+	}
+    }
+    return join(' ', @misspellings);
+    
+}
+
 # -------------------------------------------------------------- Plaintext name
 =pod
 
@@ -3608,7 +3283,7 @@ sub screenname {
 # ------------------------------------------------------------- Confirm Wrapper
 =pod
 
-=item * &confirmwrapper($message)
+=item confirmwrapper
 
 Wrap messages about completion of operation in box
 
@@ -4024,7 +3699,7 @@ sub user_lang {
 =over 4
 
 =item * &get_previous_attempt($symb, $username, $domain, $course,
-    $getattempt, $regexp, $gradesub, $usec, $identifier)
+    $getattempt, $regexp, $gradesub)
 
 Return string with previous attempt on problem. Arguments:
 
@@ -4046,11 +3721,6 @@ Return string with previous attempt on p
 
 =item * $gradesub: routine that processes the string if it matches $regexp
 
-=item * $usec: section of the desired student
-
-=item * $identifier: counter for student (multiple students one problem) or
-    problem (one student; whole sequence).
-
 =back
 
 The output string is a table containing all desired attempts, if any.
@@ -4058,7 +3728,7 @@ The output string is a table containing
 =cut
 
 sub get_previous_attempt {
-  my ($symb,$username,$domain,$course,$getattempt,$regexp,$gradesub,$usec,$identifier)=@_;
+  my ($symb,$username,$domain,$course,$getattempt,$regexp,$gradesub)=@_;
   my $prevattempts='';
   no strict 'refs';
   if ($symb) {
@@ -4068,18 +3738,13 @@ sub get_previous_attempt {
       my %lasthash=();
       my $version;
       for ($version=1;$version<=$returnhash{'version'};$version++) {
-        foreach my $key (reverse(sort(split(/\:/,$returnhash{$version.':keys'})))) {
-            if ($key =~ /\.rawrndseed$/) {
-                my ($id) = ($key =~ /^(.+)\.rawrndseed$/);
-                $lasthash{$id.'.rndseed'} = $returnhash{$version.':'.$key};
-            } else {
-                $lasthash{$key}=$returnhash{$version.':'.$key};
-            }
+        foreach my $key (sort(split(/\:/,$returnhash{$version.':keys'}))) {
+	  $lasthash{$key}=$returnhash{$version.':'.$key};
         }
       }
       $prevattempts=&start_data_table().&start_data_table_header_row();
       $prevattempts.='<th>'.&mt('History').'</th>';
-      my (%typeparts,%lasthidden,%regraded,%hidestatus);
+      my (%typeparts,%lasthidden);
       my $showsurv=&Apache::lonnet::allowed('vas',$env{'request.course.id'});
       foreach my $key (sort(keys(%lasthash))) {
 	my ($ign,@parts) = split(/\./,$key);
@@ -4096,18 +3761,6 @@ sub get_previous_attempt {
                       $lasthidden{$ign.'.'.$id} = 1;
                   }
               }
-              if ($identifier ne '') {
-                  my $id = join(',',@parts);
-                  if (&Apache::lonnet::EXT("resource.$id.problemstatus",$symb,
-                                               $domain,$username,$usec,undef,$course) =~ /^no/) {
-                      $hidestatus{$ign.'.'.$id} = 1;
-                  }
-              }
-          } elsif ($data eq 'regrader') {
-              if (($identifier ne '') && (@parts)) {
-                  my $id = join(',',@parts);
-                  $regraded{$ign.'.'.$id} = 1;
-              }
           } 
 	} else {
 	  if ($#parts == 0) {
@@ -4119,60 +3772,17 @@ sub get_previous_attempt {
       }
       $prevattempts.=&end_data_table_header_row();
       if ($getattempt eq '') {
-        my (%solved,%resets,%probstatus);
-        if (($identifier ne '') && (keys(%regraded) > 0)) {
-            for ($version=1;$version<=$returnhash{'version'};$version++) {
-                foreach my $id (keys(%regraded)) {
-                    if (($returnhash{$version.':'.$id.'.regrader'}) &&
-                        ($returnhash{$version.':'.$id.'.tries'} eq '') &&
-                        ($returnhash{$version.':'.$id.'.award'} eq '')) {
-                        push(@{$resets{$id}},$version);
-                    }
-                }
-            }
-        }
 	for ($version=1;$version<=$returnhash{'version'};$version++) {
-            my (@hidden,@unsolved);
+            my @hidden;
             if (%typeparts) {
                 foreach my $id (keys(%typeparts)) {
-                    if (($returnhash{$version.':'.$id.'.type'} eq 'anonsurvey') ||
-                        ($returnhash{$version.':'.$id.'.type'} eq 'anonsurveycred')) {
+                    if (($returnhash{$version.':'.$id.'.type'} eq 'anonsurvey') || ($returnhash{$version.':'.$id.'.type'} eq 'anonsurveycred')) {
                         push(@hidden,$id);
-                    } elsif ($identifier ne '') {
-                        unless (($returnhash{$version.':'.$id.'.type'} eq 'survey') ||
-                                ($returnhash{$version.':'.$id.'.type'} eq 'surveycred') ||
-                                ($hidestatus{$id})) {
-                            next if ((ref($resets{$id}) eq 'ARRAY') && grep(/^\Q$version\E$/,@{$resets{$id}}));
-                            if ($returnhash{$version.':'.$id.'.solved'} eq 'correct_by_student') {
-                                push(@{$solved{$id}},$version);
-                            } elsif (($returnhash{$version.':'.$id.'.solved'} ne '') &&
-                                     (ref($solved{$id}) eq 'ARRAY')) {
-                                my $skip;
-                                if (ref($resets{$id}) eq 'ARRAY') {
-                                    foreach my $reset (@{$resets{$id}}) {
-                                        if ($reset > $solved{$id}[-1]) {
-                                            $skip=1;
-                                            last;
-                                        }
-                                    }
-                                }
-                                unless ($skip) {
-                                    my ($ign,$partslist) = split(/\./,$id,2);
-                                    push(@unsolved,$partslist);
-                                }
-                            }
-                        }
                     }
                 }
             }
             $prevattempts.=&start_data_table_row().
-                           '<td>'.&mt('Transaction [_1]',$version);
-            if (@unsolved) {
-                $prevattempts .= '<span class="LC_nobreak"><label>'.
-                                 '<input type="checkbox" name="HIDE'.$identifier.'" value="'.$version.':'.join('_',@unsolved).'" />'.
-                                 &mt('Hide').'</label></span>';
-            }
-            $prevattempts .= '</td>';
+                           '<td>'.&mt('Transaction [_1]',$version).'</td>';
             if (@hidden) {
                 foreach my $key (sort(keys(%lasthash))) {
                     next if ($key =~ /\.foilorder$/);
@@ -4194,15 +3804,9 @@ sub get_previous_attempt {
                         }
                     } else {
                         if ($key =~ /\./) {
-                            my $value = $returnhash{$version.':'.$key};
-                            if ($key =~ /\.rndseed$/) {
-                                my ($id) = ($key =~ /^(.+)\.rndseed$/);
-                                if (exists($returnhash{$version.':'.$id.'.rawrndseed'})) {
-                                    $value = $returnhash{$version.':'.$id.'.rawrndseed'};
-                                }
-                            }
-                            $prevattempts.='<td>'.&format_previous_attempt_value($key,$value).
-                                           '&nbsp;</td>';
+                            my $value = &format_previous_attempt_value($key,
+                                              $returnhash{$version.':'.$key});
+                            $prevattempts.='<td>'.$value.'&nbsp;</td>';
                         } else {
                             $prevattempts.='<td>&nbsp;</td>';
                         }
@@ -4211,15 +3815,9 @@ sub get_previous_attempt {
             } else {
 	        foreach my $key (sort(keys(%lasthash))) {
                     next if ($key =~ /\.foilorder$/);
-                    my $value = $returnhash{$version.':'.$key};
-                    if ($key =~ /\.rndseed$/) {
-                        my ($id) = ($key =~ /^(.+)\.rndseed$/);
-                        if (exists($returnhash{$version.':'.$id.'.rawrndseed'})) {
-                            $value = $returnhash{$version.':'.$id.'.rawrndseed'};
-                        }
-                    }
-                    $prevattempts.='<td>'.&format_previous_attempt_value($key,$value).
-                                   '&nbsp;</td>';
+		    my $value = &format_previous_attempt_value($key,
+			            $returnhash{$version.':'.$key});
+		    $prevattempts.='<td>'.$value.'&nbsp;</td>';
 	        }
             }
 	    $prevattempts.=&end_data_table_row();
@@ -4669,20 +4267,23 @@ sub findallcourses {
 ###############################################
 
 sub blockcheck {
-    my ($setters,$activity,$uname,$udom,$url,$is_course) = @_;
+    my ($setters,$activity,$uname,$udom,$url) = @_;
 
-    if (defined($udom) && defined($uname)) {
-        # If uname and udom are for a course, check for blocks in the course.
-        if (($is_course) || (&Apache::lonnet::is_course($udom,$uname))) {
-            my ($startblock,$endblock,$triggerblock) =
-                &get_blocks($setters,$activity,$udom,$uname,$url);
-            return ($startblock,$endblock,$triggerblock);
-        }
-    } else {
+    if (!defined($udom)) {
         $udom = $env{'user.domain'};
+    }
+    if (!defined($uname)) {
         $uname = $env{'user.name'};
     }
 
+    # If uname and udom are for a course, check for blocks in the course.
+
+    if (&Apache::lonnet::is_course($udom,$uname)) {
+        my ($startblock,$endblock,$triggerblock) = 
+            &get_blocks($setters,$activity,$udom,$uname,$url);
+        return ($startblock,$endblock,$triggerblock);
+    }
+
     my $startblock = 0;
     my $endblock = 0;
     my $triggerblock = '';
@@ -4692,8 +4293,7 @@ sub blockcheck {
     # boards, chat or groups, check for blocking in current course only.
 
     if (($activity eq 'boards' || $activity eq 'chat' ||
-         $activity eq 'groups' || $activity eq 'printout') &&
-        ($env{'request.course.id'})) {
+         $activity eq 'groups') && ($env{'request.course.id'})) {
         foreach my $key (keys(%live_courses)) {
             if ($key ne $env{'request.course.id'}) {
                 delete($live_courses{$key});
@@ -4776,7 +4376,7 @@ sub blockcheck {
                                                                 $tdom,$spec,$trest,$area);
                         }
                     }
-                    my ($author,$adv,$rar) = &Apache::lonnet::set_userprivs(\%userroles,\%allroles);
+                    my ($author,$adv) = &Apache::lonnet::set_userprivs(\%userroles,\%allroles);
                     if ($userroles{'user.priv.'.$checkrole} =~ /evb\&([^\:]*)/) {
                         if ($1) {
                             $no_userblock = 1;
@@ -4798,7 +4398,7 @@ sub blockcheck {
                  ($env{'request.role'} !~ m{^st\./\Q$cdom\E/\Q$cnum\E}));
         next if ($no_userblock);
 
-        # Retrieve blocking times and identity of blocker for course
+        # Retrieve blocking times and identity of locker for course
         # of specified user, unless user has 'evb' privilege.
         
         my ($start,$end,$trigger) = 
@@ -4957,12 +4557,12 @@ sub parse_block_record {
 }
 
 sub blocking_status {
-    my ($activity,$uname,$udom,$url,$is_course) = @_;
+    my ($activity,$uname,$udom,$url) = @_;
     my %setters;
 
 # check for active blocking
     my ($startblock,$endblock,$triggerblock) = 
-        &blockcheck(\%setters,$activity,$uname,$udom,$url,$is_course);
+        &blockcheck(\%setters,$activity,$uname,$udom,$url);
     my $blocked = 0;
     if ($startblock && $endblock) {
         $blocked = 1;
@@ -4974,9 +4574,9 @@ sub blocking_status {
 # build a link to a popup window containing the details
     my $querystring  = "?activity=$activity";
 # $uname and $udom decide whose portfolio the user is trying to look at
-    if (($activity eq 'port') || ($activity eq 'passwd')) {
-        $querystring .= "&amp;udom=$udom"      if ($udom =~ /^$match_domain$/);
-        $querystring .= "&amp;uname=$uname"    if ($uname =~ /^$match_username$/);
+    if ($activity eq 'port') {
+        $querystring .= "&amp;udom=$udom"      if $udom;
+        $querystring .= "&amp;uname=$uname"    if $uname;
     } elsif ($activity eq 'docs') {
         $querystring .= '&amp;url='.&HTML::Entities::encode($url,'&"');
     }
@@ -4995,17 +4595,13 @@ END_MYBLOCK
   
     my $popupUrl = "/adm/blockingstatus/$querystring";
     my $text = &mt('Communication Blocked');
-    my $class = 'LC_comblock';
     if ($activity eq 'docs') {
         $text = &mt('Content Access Blocked');
-        $class = '';
     } elsif ($activity eq 'printout') {
         $text = &mt('Printing Blocked');
-    } elsif ($activity eq 'passwd') {
-        $text = &mt('Password Changing Blocked');
     }
     $output .= <<"END_BLOCK";
-<div class='$class'>
+<div class='LC_comblock'>
   <a onclick='openWindow("$popupUrl","Blocking Table",600,300,"no","no");return false;' href='/adm/blockingstatus/$querystring'
   title='$text'>
   <img class='LC_noBorder LC_middle' title='$text' src='/res/adm/pages/comblock.png' alt='$text'/></a>
@@ -5021,13 +4617,13 @@ END_BLOCK
 ###############################################
 
 sub check_ip_acc {
-    my ($acc,$clientip)=@_;
+    my ($acc)=@_;
     &Apache::lonxml::debug("acc is $acc");
     if (!defined($acc) || $acc =~ /^\s*$/ || $acc =~/^\s*no\s*$/i) {
         return 1;
     }
     my $allowed=0;
-    my $ip=$ENV{'REMOTE_ADDR'} || $clientip || $env{'request.host'};
+    my $ip=$env{'request.host'} || $ENV{'REMOTE_ADDR'};
 
     my $name;
     foreach my $pattern (split(',',$acc)) {
@@ -5123,28 +4719,23 @@ sub get_domainconf {
             if (keys(%{$domconfig{'login'}})) {
                 foreach my $key (keys(%{$domconfig{'login'}})) {
                     if (ref($domconfig{'login'}{$key}) eq 'HASH') {
-                        if (($key eq 'loginvia') || ($key eq 'headtag')) {
-                            if (ref($domconfig{'login'}{$key}) eq 'HASH') {
-                                foreach my $hostname (keys(%{$domconfig{'login'}{$key}})) {
-                                    if (ref($domconfig{'login'}{$key}{$hostname}) eq 'HASH') {
-                                        if ($key eq 'loginvia') {
-                                            if ($domconfig{'login'}{'loginvia'}{$hostname}{'server'}) {
-                                                my $server = $domconfig{'login'}{'loginvia'}{$hostname}{'server'};
-                                                $designhash{$udom.'.login.loginvia'} = $server;
-                                                if ($domconfig{'login'}{'loginvia'}{$hostname}{'serverpath'} eq 'custom') {
-                                                    $designhash{$udom.'.login.loginvia_'.$hostname} = $server.':'.$domconfig{'login'}{'loginvia'}{$hostname}{'custompath'};
-                                                } else {
-                                                    $designhash{$udom.'.login.loginvia_'.$hostname} = $server.':'.$domconfig{'login'}{'loginvia'}{$hostname}{'serverpath'};
-                                                }
+                        if ($key eq 'loginvia') {
+                            if (ref($domconfig{'login'}{'loginvia'}) eq 'HASH') {
+                                foreach my $hostname (keys(%{$domconfig{'login'}{'loginvia'}})) {
+                                    if (ref($domconfig{'login'}{'loginvia'}{$hostname}) eq 'HASH') {
+                                        if ($domconfig{'login'}{'loginvia'}{$hostname}{'server'}) {
+                                            my $server = $domconfig{'login'}{'loginvia'}{$hostname}{'server'};
+                                            $designhash{$udom.'.login.loginvia'} = $server;
+                                            if ($domconfig{'login'}{'loginvia'}{$hostname}{'serverpath'} eq 'custom') {
+
+                                                $designhash{$udom.'.login.loginvia_'.$hostname} = $server.':'.$domconfig{'login'}{'loginvia'}{$hostname}{'custompath'};
+                                            } else {
+                                                $designhash{$udom.'.login.loginvia_'.$hostname} = $server.':'.$domconfig{'login'}{'loginvia'}{$hostname}{'serverpath'};
                                             }
-                                        } elsif ($key eq 'headtag') {
-                                            if ($domconfig{'login'}{'headtag'}{$hostname}{'url'}) {
-                                                $designhash{$udom.'.login.headtag_'.$hostname} = $domconfig{'login'}{'headtag'}{$hostname}{'url'};
+                                            if ($domconfig{'login'}{'loginvia'}{$hostname}{'exempt'}) {
+                                                $designhash{$udom.'.login.loginvia_exempt_'.$hostname} = $domconfig{'login'}{'loginvia'}{$hostname}{'exempt'};
                                             }
                                         }
-                                        if ($domconfig{'login'}{$key}{$hostname}{'exempt'}) {
-                                            $designhash{$udom.'.login.'.$key.'_exempt_'.$hostname} = $domconfig{'login'}{$key}{$hostname}{'exempt'};
-                                        }
                                     }
                                 }
                             }
@@ -5212,7 +4803,7 @@ sub get_legacy_domconf {
     my $designdir=$Apache::lonnet::perlvar{'lonTabDir'}.'/lonDomColors';
     my $designfile =  $designdir.'/'.$udom.'.tab';
     if (-e $designfile) {
-        if ( open (my $fh,'<',$designfile) ) {
+        if ( open (my $fh,"<$designfile") ) {
             while (my $line = <$fh>) {
                 next if ($line =~ /^\#/);
                 chomp($line);
@@ -5310,7 +4901,7 @@ sub designparm {
 
 Inputs: $url (usually will be undef).
 
-Returns: Path to Authoring Space containing the resource or 
+Returns: Path to Construction Space containing the resource or 
          directory being viewed (or for which action is being taken). 
          If $url is provided, and begins /priv/<domain>/<uname>
          the path will be that portion of the $context argument.
@@ -5373,7 +4964,7 @@ Input: (optional) filename from which br
        is appropriate for use in building the breadcrumb trail.
 
 Returns: HTML div with CSTR path and recent box
-         To be included on Authoring Space pages
+         To be included on Construction Space pages
 
 =cut
 
@@ -5388,7 +4979,7 @@ sub CSTR_pageheader {
 
     my $londocroot = $Apache::lonnet::perlvar{'lonDocRoot'};
     my ($udom,$uname,$thisdisfn)=
-        ($trailfile =~ m{^\Q$londocroot\E/priv/([^/]+)/([^/]+)(?:|/(.*))$});
+        ($trailfile =~ m{^\Q$londocroot\E/priv/([^/]+)/([^/]+)/(.*)$});
     my $formaction = "/priv/$udom/$uname/$thisdisfn";
     $formaction =~ s{/+}{/}g;
 
@@ -5404,7 +4995,7 @@ sub CSTR_pageheader {
     my $output =
          '<div>'
         .&Apache::loncommon::help_open_menu('','',3,'Authoring') #FIXME: Broken? Where is it?
-        .'<b>'.&mt('Authoring Space:').'</b> '
+        .'<b>'.&mt('Construction Space:').'</b> '
         .'<form name="dirs" method="post" action="'.$formaction
         .'" target="_top">' #FIXME lonpubdir: target="_parent"
         .&Apache::lonhtmlcommon::crumbs($uname.'/'.$parentpath,'_top','/priv/'.$udom,undef,undef);
@@ -5463,15 +5054,11 @@ Inputs:
 
 =item * $bgcolor, used to override the bgcolor on a webpage to a specific value
 
-=item * $no_inline_link, if true and in remote mode, don't show the
-         'Switch To Inline Menu' link
-
 =item * $args, optional argument valid values are
             no_auto_mt_title -> prevents &mt()ing the title arg
-            use_absolute     -> for external resource or syllabus, this will
-                                contain https://<hostname> if server uses
-                                https (as per hosts.tab), but request is for http
-            hostname         -> hostname, from $r->hostname().
+            inherit_jsmath -> when creating popup window in a page,
+                              should it have jsmath forced on by the
+                              current page
 
 =item * $advtoolsref, optional argument, ref to an array containing
             inlineremote items to be added in "Functions" menu below
@@ -5488,7 +5075,7 @@ other decorations will be returned.
 
 sub bodytag {
     my ($title,$function,$addentries,$bodyonly,$domain,$forcereg,
-        $no_nav_bar,$bgcolor,$no_inline_link,$args,$advtoolsref)=@_;
+        $no_nav_bar,$bgcolor,$args,$advtoolsref)=@_;
 
     my $public;
     if ((($env{'user.name'} eq 'public') && ($env{'user.domain'} eq 'public'))
@@ -5496,8 +5083,6 @@ sub bodytag {
         $public = 1;
     }
     if (!$args->{'no_auto_mt_title'}) { $title = &mt($title); }
-    my $httphost = $args->{'use_absolute'};
-    my $hostname = $args->{'hostname'};
 
     $function = &get_users_function() if (!$function);
     my $img =    &designparm($function.'.img',$domain);
@@ -5513,10 +5098,7 @@ sub bodytag {
     @design{keys(%$addentries)} = @$addentries{keys(%$addentries)}; 
 
  # role and realm
-    my ($role,$realm) = split(m{\./},$env{'request.role'},2);
-    if ($realm) {
-        $realm = '/'.$realm;
-    }
+    my ($role,$realm) = split(/\./,$env{'request.role'},2);
     if ($role  eq 'ca') {
         my ($rdom,$rname) = ($realm =~ m{^/($match_domain)/($match_username)$});
         $realm = &plainname($rname,$rdom);
@@ -5525,14 +5107,6 @@ sub bodytag {
     if ($env{'request.course.id'}) {
         if ($env{'request.role'} !~ /^cr/) {
             $role = &Apache::lonnet::plaintext($role,&course_type());
-        } elsif ($role =~ m{^cr/($match_domain)/\1-domainconfig/(\w+)$}) {
-            if ($env{'request.role.desc'}) {
-                $role = $env{'request.role.desc'};
-            } else {
-                $role = &mt('Helpdesk[_1]','&nbsp;'.$2);
-            }
-        } else {
-            $role = (split(/\//,$role,4))[-1];
         }
         if ($env{'request.course.sec'}) {
             $role .= ('&nbsp;'x2).'-&nbsp;'.&mt('section:').'&nbsp;'.$env{'request.course.sec'};
@@ -5548,16 +5122,18 @@ sub bodytag {
 
 # construct main body tag
     my $bodytag = "<body $extra_body_attr>".
-	&Apache::lontexconvert::init_math_support();
-
-    &get_unprocessed_cgi($ENV{'QUERY_STRING'}, ['inhibitmenu']);
+	&Apache::lontexconvert::init_math_support($args->{'inherit_jsmath'});
 
-    if (($bodyonly) || ($no_nav_bar) || ($env{'form.inhibitmenu'} eq 'yes')) {
+    if ($bodyonly) {
         return $bodytag;
-    }
+    } 
 
+    my $name = &plainname($env{'user.name'},$env{'user.domain'});
     if ($public) {
 	undef($role);
+    } else {
+	$name = &aboutmewrapper($name,$env{'user.name'},$env{'user.domain'},
+                                undef,'LC_menubuttons_link');
     }
     
     my $titleinfo = '<h1>'.$title.'</h1>';
@@ -5572,82 +5148,61 @@ sub bodytag {
         $dc_info =~ s/\s+$//;
     }
 
-    $role = '<span class="LC_nobreak">('.$role.')</span>' if ($role && !$env{'browser.mobile'});
-
-    if ($env{'request.state'} eq 'construct') { $forcereg=1; }
-
+    $role = '<span class="LC_nobreak">('.$role.')</span>' if $role;
+    &get_unprocessed_cgi($ENV{'QUERY_STRING'}, ['inhibitmenu']);
 
+        if ($no_nav_bar || $env{'form.inhibitmenu'} eq 'yes') { 
+            return $bodytag; 
+        } 
 
-    my $funclist;
-    if (($env{'environment.remote'} eq 'on') && ($env{'request.state'} ne 'construct')) {
-        $bodytag .= Apache::lonhtmlcommon::scripttag(Apache::lonmenu::utilityfunctions($httphost), 'start')."\n".
-                    Apache::lonmenu::serverform();
-        my $forbodytag;
-        &Apache::lonmenu::prepare_functions($env{'request.noversionuri'},
-                                            $forcereg,$args->{'group'},
-                                            $args->{'bread_crumbs'},
-                                            $advtoolsref,'','',\$forbodytag);
-        unless (ref($args->{'bread_crumbs'}) eq 'ARRAY') {
-            $funclist = $forbodytag;
-        }
-    } else {
+        if ($env{'request.state'} eq 'construct') { $forcereg=1; }
 
         #    if ($env{'request.state'} eq 'construct') {
         #        $titleinfo = &CSTR_pageheader(); #FIXME: Will be removed once all scripts have their own calls
         #    }
 
-        $bodytag .= Apache::lonhtmlcommon::scripttag(
-            Apache::lonmenu::utilityfunctions($httphost), 'start');
 
-        my ($left,$right) = Apache::lonmenu::primary_menu();
 
         if ($env{'request.noversionuri'} =~ m{^/res/adm/pages/}) {
-            if ($dc_info) {
+             if ($dc_info) {
                  $dc_info = qq|<span class="LC_cusr_subheading">$dc_info</span>|;
-            }
-            $bodytag .= qq|<div id="LC_nav_bar">$left $role<br />
-                           <em>$realm</em> $dc_info</div>|;
+             }
+             $bodytag .= qq|<div id="LC_nav_bar">$name $role<br />
+                <em>$realm</em> $dc_info</div>|;
             return $bodytag;
         }
 
         unless ($env{'request.symb'} =~ m/\.page___\d+___/) {
-            $bodytag .= qq|<div id="LC_nav_bar">$left $role</div>|;
+            $bodytag .= qq|<div id="LC_nav_bar">$name $role</div>|;
         }
 
-        $bodytag .= $right;
+        $bodytag .= Apache::lonhtmlcommon::scripttag(
+            Apache::lonmenu::utilityfunctions(), 'start');
+
+        $bodytag .= Apache::lonmenu::primary_menu();
 
         if ($dc_info) {
             $dc_info = &dc_courseid_toggle($dc_info);
         }
         $bodytag .= qq|<div id="LC_realm">$realm $dc_info</div>|;
 
-        #if directed to not display the secondary menu, don't.
-        if ($args->{'no_secondary_menu'}) {
-            return $bodytag;
-        }
         #don't show menus for public users
         if (!$public){
-            $bodytag .= Apache::lonmenu::secondary_menu($httphost);
+            $bodytag .= Apache::lonmenu::secondary_menu();
             $bodytag .= Apache::lonmenu::serverform();
             $bodytag .= Apache::lonhtmlcommon::scripttag('', 'end');
             if ($env{'request.state'} eq 'construct') {
                 $bodytag .= &Apache::lonmenu::innerregister($forcereg,
-                                $args->{'bread_crumbs'},'','',$hostname);
+                                $args->{'bread_crumbs'});
             } elsif ($forcereg) {
                 $bodytag .= &Apache::lonmenu::innerregister($forcereg,undef,
-                                                            $args->{'group'},
-                                                            $args->{'hide_buttons',
-                                                            $hostname});
+                                                            $args->{'group'});
             } else {
-                my $forbodytag;
-                &Apache::lonmenu::prepare_functions($env{'request.noversionuri'},
-                                                    $forcereg,$args->{'group'},
-                                                    $args->{'bread_crumbs'},
-                                                    $advtoolsref,'',$hostname,
-                                                    \$forbodytag);
-                unless (ref($args->{'bread_crumbs'}) eq 'ARRAY') {
-                    $bodytag .= $forbodytag;
-                }
+                $bodytag .= 
+                    &Apache::lonmenu::prepare_functions($env{'request.noversionuri'},
+                                                        $forcereg,$args->{'group'},
+                                                        $args->{'bread_crumbs'},
+                                                        $advtoolsref);
             }
         }else{
             # this is to seperate menu from content when there's no secondary
@@ -5657,54 +5212,6 @@ sub bodytag {
         }
 
         return $bodytag;
-    }
-
-#
-# Top frame rendering, Remote is up
-#
-
-    my $imgsrc = $img;
-    if ($img =~ /^\/adm/) {
-        $imgsrc = &lonhttpdurl($img);
-    }
-    my $upperleft='<img src="'.$imgsrc.'" alt="'.$function.'" />';
-
-    my $help=($no_inline_link?''
-              :&Apache::loncommon::top_nav_help('Help'));
-
-    # Explicit link to get inline menu
-    my $menu= ($no_inline_link?''
-               :'<a href="/adm/remote?action=collapse" target="_top">'.&mt('Switch to Inline Menu Mode').'</a>');
-
-    if ($dc_info) {
-        $dc_info = qq|<span class="LC_cusr_subheading">($dc_info)</span>|;
-    }
-
-    my $name = &plainname($env{'user.name'},$env{'user.domain'});
-    unless ($public) {
-        $name = &aboutmewrapper($name,$env{'user.name'},$env{'user.domain'},
-                                undef,'LC_menubuttons_link');
-    }
-
-    unless ($env{'form.inhibitmenu'}) {
-        $bodytag .= qq|<div id="LC_nav_bar">$name $role</div>
-                       <ol class="LC_primary_menu LC_floatright LC_right">
-                       <li>$help</li>
-                       <li>$menu</li>
-                       </ol><div id="LC_realm"> $realm $dc_info</div>|;
-    }
-    if ($env{'request.state'} eq 'construct') {
-        if (!$public){
-            if ($env{'request.state'} eq 'construct') {
-                $funclist = &Apache::lonhtmlcommon::scripttag(
-                                &Apache::lonmenu::utilityfunctions($httphost), 'start').
-                            &Apache::lonhtmlcommon::scripttag('','end').
-                            &Apache::lonmenu::innerregister($forcereg,
-                                                            $args->{'bread_crumbs'});
-            }
-        }
-    }
-    return $bodytag."\n".$funclist;
 }
 
 sub dc_courseid_toggle {
@@ -5736,19 +5243,12 @@ sub make_attr_string {
 		delete($attr_ref->{$key});
 	    }
 	}
-        if ($env{'environment.remote'} eq 'on') {
-            $attr_ref->{'onload'}  =
-                &Apache::lonmenu::loadevents().  $on_load;
-            $attr_ref->{'onunload'}=
-                &Apache::lonmenu::unloadevents().$on_unload;
-        } else {  
-	    $attr_ref->{'onload'}  = $on_load;
-	    $attr_ref->{'onunload'}= $on_unload;
-        }
+	$attr_ref->{'onload'}  = $on_load;
+	$attr_ref->{'onunload'}= $on_unload;
     }
 
     my $attr_string;
-    foreach my $attr (sort(keys(%$attr_ref))) {
+    foreach my $attr (keys(%$attr_ref)) {
 	$attr_string .= " $attr=\"".$attr_ref->{$attr}.'" ';
     }
     return $attr_string;
@@ -5778,6 +5278,7 @@ sub endbodytag {
     unless ((ref($args) eq 'HASH') && ($args->{'notbody'})) {
         $endbodytag='</body>';
     }
+    $endbodytag=&Apache::lontexconvert::jsMath_process()."\n".$endbodytag;
     if ( exists( $env{'internal.head.redirect'} ) ) {
         if (!(ref($args) eq 'HASH' && $args->{'noredirectlink'})) {
 	    $endbodytag=
@@ -5878,14 +5379,6 @@ form, .inline {
   vertical-align:middle;
 }
 
-.LC_floatleft {
-  float: left;
-}
-
-.LC_floatright {
-  float: right;
-}
-
 .LC_400Box {
   width:400px;
 }
@@ -5952,17 +5445,6 @@ div.LC_confirm_box .LC_success img {
   vertical-align: middle;
 }
 
-.LC_maxwidth {
-  max-width: 100%;
-  height: auto;
-}
-
-.LC_textsize_mobile {
-  \@media only screen and (max-device-width: 480px) {
-      -webkit-text-size-adjust:100%; -moz-text-size-adjust:100%; -ms-text-size-adjust:100%;
-  }
-}
-
 .LC_icon {
   border: none;
   vertical-align: middle;
@@ -6084,10 +5566,6 @@ table#LC_menubuttons img {
   vertical-align: middle;
 }
 
-.LC_breadcrumbs_hoverable {
-  background: $sidebg;
-}
-
 td.LC_table_cell_checkbox {
   text-align: center;
 }
@@ -6158,11 +5636,6 @@ td.LC_menubuttons_text {
   background: $tabbg;
 }
 
-td.LC_zero_height {
-  line-height: 0;
-  cellpadding: 0;
-}
-
 table.LC_data_table {
   border: 1px solid #000000;
   border-collapse: separate;
@@ -6249,8 +5722,7 @@ table.LC_nested tr.LC_empty_row td {
   padding: 8px;
 }
 
-table.LC_data_table tr.LC_empty_row td,
-table.LC_data_table tr.LC_footer_row td {
+table.LC_data_table tr.LC_empty_row td {
   background-color: $sidebg;
 }
 
@@ -6753,8 +6225,7 @@ table.LC_prior_tries td {
   padding: 6px;
 }
 
-.LC_answer_unknown,
-.LC_answer_warning {
+.LC_answer_unknown {
   background: orange;
   color: black;
   padding: 6px;
@@ -6836,7 +6307,6 @@ table.LC_data_table tr > td.LC_docs_entr
   color: #990000;
 }
 
-.LC_domprefs_email,
 .LC_docs_reinit_warn,
 .LC_docs_ext_edit {
   font-size: x-small;
@@ -6952,7 +6422,7 @@ div.LC_edit_problem_footer,
 div.LC_edit_problem_footer div,
 div.LC_edit_problem_editxml_header,
 div.LC_edit_problem_editxml_header div {
-  z-index: 100;
+  margin-top: 5px;
 }
 
 div.LC_edit_problem_header_title {
@@ -6968,34 +6438,14 @@ table.LC_edit_problem_header_title {
   background: $tabbg;
 }
 
-div.LC_edit_actionbar {
-    background-color: $sidebg;
-    margin: 0;
-    padding: 0;
-    line-height: 200%;
-}
-
-div.LC_edit_actionbar div{
-    padding: 0;
-    margin: 0;
-    display: inline-block;
-}
-
-.LC_edit_opt {
-  padding-left: 1em;
-  white-space: nowrap;
-}
-
-.LC_edit_problem_latexhelper{
-    text-align: right;
-}
-
-#LC_edit_problem_colorful div{
-    margin-left: 40px;
+div.LC_edit_problem_discards {
+  float: left;
+  padding-bottom: 5px;
 }
 
-#LC_edit_problem_codemirror div{
-    margin-left: 0px;
+div.LC_edit_problem_saves {
+  float: right;
+  padding-bottom: 5px;
 }
 
 img.stift {
@@ -7012,7 +6462,6 @@ div.LC_createcourse {
 }
 
 .LC_dccid {
-  float: right;
   margin: 0.2em 0 0 0;
   padding: 0;
   font-size: 90%;
@@ -7085,10 +6534,6 @@ fieldset {
   /* overflow: hidden; */
 }
 
-article.geogebraweb div {
-    margin: 0;
-}
-
 fieldset > legend {
   font-weight: bold;
   padding: 0 5px 0 5px;
@@ -7114,8 +6559,10 @@ fieldset > legend {
 }
 
 ol.LC_primary_menu {
+  float: right;
   margin: 0;
   padding: 0;
+  background-color: $pgbg_or_bgcolor;
 }
 
 ol#LC_PathBreadcrumbs {
@@ -7127,48 +6574,23 @@ ol.LC_primary_menu li {
   vertical-align: middle;
   text-align: left;
   list-style: none;
-  position: relative;
   float: left;
-  z-index: 100; /* will be displayed above codemirror and underneath the help-layer */
-  line-height: 1.5em;
 }
 
-ol.LC_primary_menu li a, 
-ol.LC_primary_menu li p {
+ol.LC_primary_menu li a {
   display: block;
   margin: 0;
   padding: 0 5px 0 10px;
   text-decoration: none;
 }
 
-ol.LC_primary_menu li p span.LC_primary_menu_innertitle {
-  display: inline-block;
-  width: 95%;
-  text-align: left;
-}
-
-ol.LC_primary_menu li p span.LC_primary_menu_innerarrow {
-  display: inline-block;
-  width: 5%;
-  float: right;
-  text-align: right;
-  font-size: 70%;
-}
-
-ol.LC_primary_menu ul {
+ol.LC_primary_menu li ul {
   display: none;
-  width: 15em;
+  width: 10em;
   background-color: $data_table_light;
-  position: absolute;
-  top: 100%;
-}
-
-ol.LC_primary_menu ul ul {
-  left: 100%;
-  top: 0;
 }
 
-ol.LC_primary_menu li:hover > ul, ol.LC_primary_menu li.hover > ul {
+ol.LC_primary_menu li:hover ul, ol.LC_primary_menu li.hover ul {
   display: block;
   position: absolute;
   margin: 0;
@@ -7177,21 +6599,15 @@ ol.LC_primary_menu li:hover > ul, ol.LC_
 }
 
 ol.LC_primary_menu li:hover li, ol.LC_primary_menu li.hover li {
-/* First Submenu -> size should be smaller than the menu title of the whole menu */
   font-size: 90%;
   vertical-align: top;
   float: none;
   border-left: 1px solid black;
   border-right: 1px solid black;
-/* A dark bottom border to visualize different menu options;
-overwritten in the create_submenu routine for the last border-bottom of the menu */
-  border-bottom: 1px solid $data_table_dark;
 }
 
-ol.LC_primary_menu li li p:hover {
-  color:$button_hover;
-  text-decoration:none;
-  background-color:$data_table_dark;
+ol.LC_primary_menu li:hover li a, ol.LC_primary_menu li.hover li a {
+  background-color:$data_table_light;
 }
 
 ol.LC_primary_menu li li a:hover {
@@ -7199,11 +6615,6 @@ ol.LC_primary_menu li li a:hover {
    background-color:$data_table_dark;
 }
 
-/* Font-size equal to the size of the predecessors*/
-ol.LC_primary_menu li:hover li li {
-  font-size: 100%;
-}
-
 ol.LC_primary_menu li img {
   vertical-align: bottom;
   height: 1.1em;
@@ -7292,7 +6703,7 @@ ul#LC_secondary_menu li ul li {
   vertical-align: top;
   border-left: 1px solid black;
   border-right: 1px solid black;
-  background-color: $data_table_light;
+  background-color: $data_table_light
   list-style:none;
   float: none;
 }
@@ -7745,14 +7156,6 @@ ul.LC_funclist li {
 		cursor:pointer;
 }
 
-pre.LC_wordwrap {
-  white-space: pre-wrap;
-  white-space: -moz-pre-wrap;
-  white-space: -pre-wrap;
-  white-space: -o-pre-wrap;
-  word-wrap: break-word;
-}
-
 /*
   styles used by TTH when "Default set of options to pass to tth/m
   when converting TeX" in course settings has been set
@@ -7774,39 +7177,6 @@ span.roman {font-family: serif; font-sty
 span.overacc2 {position: relative;  left: .8em; top: -1.2ex;}
 span.overacc1 {position: relative;  left: .6em; top: -1.2ex;}
 
-#LC_minitab_header {
-  float:left;
-  width:100%;
-  background:#DAE0D2 url("/res/adm/pages/minitabmenu_bg.gif") repeat-x bottom;
-  font-size:93%;
-  line-height:normal;
-  margin: 0.5em 0 0.5em 0;
-}
-#LC_minitab_header ul {
-  margin:0;
-  padding:10px 10px 0;
-  list-style:none;
-}
-#LC_minitab_header li {
-  float:left;
-  background:url("/res/adm/pages/minitabmenu_left.gif") no-repeat left top;
-  margin:0;
-  padding:0 0 0 9px;
-}
-#LC_minitab_header a {
-  display:block;
-  background:url("/res/adm/pages/minitabmenu_right.gif") no-repeat right top;
-  padding:5px 15px 4px 6px;
-}
-#LC_minitab_header #LC_current_minitab {
-  background-image:url("/res/adm/pages/minitabmenu_left_on.gif");
-}
-#LC_minitab_header #LC_current_minitab a {
-  background-image:url("/res/adm/pages/minitabmenu_right_on.gif");
-  padding-bottom:5px;
-}
-
-
 END
 }
 
@@ -7843,7 +7213,6 @@ sub headtag {
     my $function = $args->{'function'} || &get_users_function();
     my $domain   = $args->{'domain'}   || &determinedomain();
     my $bgcolor  = $args->{'bgcolor'}  || &designparm($function.'.pgbg',$domain);
-    my $httphost = $args->{'use_absolute'};
     my $url = join(':',$env{'user.name'},$env{'user.domain'},
 		   $Apache::lonnet::perlvar{'lonVersion'},
 		   #time(),
@@ -7854,23 +7223,20 @@ sub headtag {
 
     my $result =
 	'<head>'.
-	&font_settings($args);
+	&font_settings();
 
-    my $inhibitprint;
-    if ($args->{'print_suppress'}) {
-        $inhibitprint = &print_suppression();
-    }
+    my $inhibitprint = &print_suppression();
 
     if (!$args->{'frameset'}) {
 	$result .= &Apache::lonhtmlcommon::htmlareaheaders();
     }
-    if ($args->{'force_register'}) {
-        $result .= &Apache::lonmenu::registerurl(1);
+    if ($args->{'force_register'} && $env{'request.noversionuri'} !~ m{^/res/adm/pages/}) {
+        $result .= Apache::lonxml::display_title();
     }
     if (!$args->{'no_nav_bar'} 
 	&& !$args->{'only_body'}
 	&& !$args->{'frameset'}) {
-	$result .= &help_menu_js($httphost);
+	$result .= &help_menu_js();
         $result.=&modal_window();
         $result.=&togglebox_script();
         $result.=&wishlist_window();
@@ -7899,107 +7265,15 @@ sub headtag {
 <meta http-equiv="pragma" content="no-cache" />
 <meta http-equiv="Refresh" content="$time; url=$url" />
 ADDMETA
-    } else {
-        unless (($args->{'frameset'}) || ($args->{'js_ready'}) || ($args->{'only_body'}) || ($args->{'no_nav_bar'})) {
-            my $requrl = $env{'request.uri'};
-            if ($requrl eq '') {
-                $requrl = $ENV{'REQUEST_URI'};
-                $requrl =~ s/\?.+$//;
-            }
-            unless (($requrl =~ m{^/adm/(?:switchserver|login|authenticate|logout|groupsort|cleanup|helper|slotrequest|grades)(\?|$)}) ||
-                    (($requrl =~ m{^/res/}) && (($env{'form.submitted'} eq 'scantron') ||
-                     ($env{'form.grade_symb'}) || ($Apache::lonhomework::scantronmode)))) {
-                my $dom_in_use = $Apache::lonnet::perlvar{'lonDefDomain'};
-                unless (&Apache::lonnet::allowed('mau',$dom_in_use)) {
-                    my %domdefs = &Apache::lonnet::get_domain_defaults($dom_in_use);
-                    if (ref($domdefs{'offloadnow'}) eq 'HASH') {
-                        my $lonhost = $Apache::lonnet::perlvar{'lonHostID'};
-                        if ($domdefs{'offloadnow'}{$lonhost}) {
-                            my $newserver = &Apache::lonnet::spareserver(30000,undef,1,$dom_in_use);
-                            if (($newserver) && ($newserver ne $lonhost)) {
-                                my $numsec = 5;
-                                my $timeout = $numsec * 1000;
-                                my ($newurl,$locknum,%locks,$msg);
-                                if ($env{'request.role.adv'}) {
-                                    ($locknum,%locks) = &Apache::lonnet::get_locks();
-                                }
-                                my $disable_submit = 0;
-                                if ($requrl =~ /$LONCAPA::assess_re/) {
-                                    $disable_submit = 1;
-                                }
-                                if ($locknum) {
-                                    my @lockinfo = sort(values(%locks));
-                                    $msg = &mt('Once the following tasks are complete: ')."\\n".
-                                           join(", ",sort(values(%locks)))."\\n".
-                                           &mt('your session will be transferred to a different server, after you click "Roles".');
-                                } else {
-                                    if (($requrl =~ m{^/res/}) && ($env{'form.submitted'} =~ /^part_/)) {
-                                        $msg = &mt('Your LON-CAPA submission has been recorded')."\\n";
-                                    }
-                                    $msg .= &mt('Your current LON-CAPA session will be transferred to a different server in [quant,_1,second].',$numsec);
-                                    $newurl = '/adm/switchserver?otherserver='.$newserver;
-                                    if (($env{'request.role'}) && ($env{'request.role'} ne 'cm')) {
-                                        $newurl .= '&role='.$env{'request.role'};
-                                    }
-                                    if ($env{'request.symb'}) {
-                                        $newurl .= '&symb='.$env{'request.symb'};
-                                    } else {
-                                        $newurl .= '&origurl='.$requrl;
-                                    }
-                                }
-                                &js_escape(\$msg);
-                                $result.=<<OFFLOAD
-<meta http-equiv="pragma" content="no-cache" />
-<script type="text/javascript">
-// <![CDATA[
-function LC_Offload_Now() {
-    var dest = "$newurl";
-    if (dest != '') {
-        window.location.href="$newurl";
-    }
-}
-\$(document).ready(function () {
-    window.alert('$msg');
-    if ($disable_submit) {
-        \$(".LC_hwk_submit").prop("disabled", true);
-        \$( ".LC_textline" ).prop( "readonly", "readonly");
-    }
-    setTimeout('LC_Offload_Now()', $timeout);
-});
-// ]]>
-</script>
-OFFLOAD
-                            }
-                        }
-                    }
-                }
-            }
-        }
     }
     if (!defined($title)) {
 	$title = 'The LearningOnline Network with CAPA';
     }
     if (!$args->{'no_auto_mt_title'}) { $title = &mt($title); }
     $result .= '<title> LON-CAPA '.$title.'</title>'
-	.'<link rel="stylesheet" type="text/css" href="'.$url.'"';
-    if (!$args->{'frameset'}) {
-        $result .= ' /';
-    }
-    $result .= '>'
+	.'<link rel="stylesheet" type="text/css" href="'.$url.'" />'
         .$inhibitprint
 	.$head_extra;
-    my $clientmobile;
-    if (($env{'user.name'} eq '') && ($env{'user.domain'} eq '')) {
-        (undef,undef,undef,undef,undef,undef,$clientmobile) = &decode_user_agent();
-    } else {
-        $clientmobile = $env{'browser.mobile'};
-    }
-    if ($clientmobile) {
-        $result .= '
-<meta name="viewport" content="width=device-width, initial-scale=1.0, user-scalable=0, minimum-scale=1.0, maximum-scale=1.0">
-<meta name="apple-mobile-web-app-capable" content="yes" />';
-    }
-    $result .= '<meta name="google" content="notranslate" />'."\n";
     return $result.'</head>';
 }
 
@@ -8009,21 +7283,15 @@ OFFLOAD
 
 Returns neccessary <meta> to set the proper encoding
 
-Inputs: optional reference to HASH -- $args passed to &headtag()
+Inputs: none
 
 =cut
 
 sub font_settings {
-    my ($args) = @_;
     my $headerstring='';
-    if ((!$env{'browser.mathml'} && $env{'browser.unicode'}) ||
-        ((ref($args) eq 'HASH') && ($args->{'browser.unicode'}))) {
+    if (!$env{'browser.mathml'} && $env{'browser.unicode'}) {
 	$headerstring.=
-	    '<meta http-equiv="Content-Type" content="text/html; charset=utf-8"';
-        if (!$args->{'frameset'}) {
-            $headerstring.= ' /';
-        }
-        $headerstring .= '>'."\n";
+	    '<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />';
     }
     return $headerstring;
 }
@@ -8068,7 +7336,7 @@ sub print_suppression {
         }
         my $cdom = $env{'course.'.$env{'request.course.id'}.'.domain'};
         my $cnum = $env{'course.'.$env{'request.course.id'}.'.num'};
-        my $blocked = &blocking_status('printout',$cnum,$cdom,undef,1);
+        my $blocked = &blocking_status('printout',$cnum,$cdom);
         if ($blocked) {
             my $checkrole = "cm./$cdom/$cnum";
             if ($env{'request.course.sec'} ne '') {
@@ -8115,7 +7383,6 @@ Inputs: none
 =cut
 
 sub xml_begin {
-    my ($is_frameset) = @_;
     my $output='';
 
     if ($env{'browser.mathml'}) {
@@ -8127,12 +7394,9 @@ sub xml_begin {
 	    .'<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1 plus MathML 2.0 plus SVG 1.1//EN" "http://www.w3.org/2002/04/xhtml-math-svg/xhtml-math-svg.dtd">'
             .'<html xmlns:math="http://www.w3.org/1998/Math/MathML" ' 
 	    .'xmlns="http://www.w3.org/1999/xhtml">';
-    } elsif ($is_frameset) {
-        $output='<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Frameset//EN" "http://www.w3.org/TR/html4/frameset.dtd">'."\n".
-                '<html>'."\n";
     } else {
-	$output='<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'."\n".
-                '<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">'."\n";
+	$output='<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">'
+           .'<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">';
     }
     return $output;
 }
@@ -8176,19 +7440,14 @@ $args - additional optional args support
              skip_phases    -> hash ref of 
                                     head -> skip the <html><head> generation
                                     body -> skip all <body> generation
-             no_inline_link -> if true and in remote mode, don't show the
-                                    'Switch To Inline Menu' link
              no_auto_mt_title -> prevent &mt()ing the title arg
+             inherit_jsmath -> when creating popup window in a page,
+                                    should it have jsmath forced on by the
+                                    current page
              bread_crumbs ->             Array containing breadcrumbs
              bread_crumbs_component ->  if exists show it as headline else show only the breadcrumbs
-             bread_crumbs_nomenu -> if true will pass false as the value of $menulink
-                                    to lonhtmlcommon::breadcrumbs
-             group          -> includes the current group, if page is for a
-                               specific group
-             use_absolute   -> for request for external resource or syllabus, this
-                               will contain https://<hostname> if server uses
-                               https (as per hosts.tab), but request is for http
-             hostname       -> hostname, originally from $r->hostname(), (optional).
+             group          -> includes the current group, if page is for a 
+                               specific group  
 
 =back
 
@@ -8204,7 +7463,7 @@ sub start_page {
     my ($result,@advtools);
 
     if (! exists($args->{'skip_phases'}{'head'}) ) {
-        $result .= &xml_begin($args->{'frameset'}) . &headtag($title, $head_extra, $args);
+        $result .= &xml_begin() . &headtag($title, $head_extra, $args);
     }
     
     if (! exists($args->{'skip_phases'}{'body'}) ) {
@@ -8218,8 +7477,8 @@ sub start_page {
                          $args->{'function'},       $args->{'add_entries'},
                          $args->{'only_body'},      $args->{'domain'},
                          $args->{'force_register'}, $args->{'no_nav_bar'},
-                         $args->{'bgcolor'},        $args->{'no_inline_link'},
-                         $args,                     \@advtools);
+                         $args->{'bgcolor'},        $args,
+                         \@advtools);
         }
     }
 
@@ -8252,24 +7511,13 @@ sub start_page {
                 if (@advtools > 0) {
                     &Apache::lonmenu::advtools_crumbs(@advtools);
                 }
-                my $menulink;
-                # if arg: bread_crumbs_nomenu is true pass 0 as $menulink item.
-                if (exists($args->{'bread_crumbs_nomenu'})) {
-                    $menulink = 0;
-                } else {
-                    undef($menulink);
-                }
+
 		#if bread_crumbs_component exists show it as headline else show only the breadcrumbs
 		if(exists($args->{'bread_crumbs_component'})){
-			$result .= &Apache::lonhtmlcommon::breadcrumbs($args->{'bread_crumbs_component'},'',$menulink);
+			$result .= &Apache::lonhtmlcommon::breadcrumbs($args->{'bread_crumbs_component'});
 		}else{
-			$result .= &Apache::lonhtmlcommon::breadcrumbs('','',$menulink);
+			$result .= &Apache::lonhtmlcommon::breadcrumbs();
 		}
-    } elsif (($env{'environment.remote'} eq 'on') &&
-             ($env{'form.inhibitmenu'} ne 'yes') &&
-             ($env{'request.noversionuri'} =~ m{^/res/}) &&
-             ($env{'request.noversionuri'} !~ m{^/res/adm/pages/})) {
-        $result .= '<div style="padding:0;margin:0;clear:both"><hr /></div>';
     }
     return $result;
 }
@@ -8316,13 +7564,9 @@ function set_wishlistlink(title, path) {
         title = document.title;
         title = title.replace(/^LON-CAPA /,'');
     }
-    title = encodeURIComponent(title);
-    title = title.replace("'","\\\'");
     if (!path) {
         path = location.pathname;
     }
-    path = encodeURIComponent(path);
-    path = path.replace("'","\\\'");
     Win = window.open('/adm/wishlist?mode=newLink&setTitle='+title+'&setPath='+path,
                       'wishlistNewLink','width=560,height=350,scrollbars=0');
 }
@@ -8363,15 +7607,14 @@ var modalWindow = {
 		$(".LCmodal-overlay").click(function(){modalWindow.close();});
 	}
 };
-	var openMyModal = function(source,width,height,scrolling,transparency,style)
+	var openMyModal = function(source,width,height,scrolling)
 	{
-                source = source.replace(/'/g,"&#39;");
 		modalWindow.windowId = "myModal";
 		modalWindow.width = width;
 		modalWindow.height = height;
-		modalWindow.content = "<iframe width='"+width+"' height='"+height+"' frameborder='0' scrolling='"+scrolling+"' allowtransparency='"+transparency+"' src='" + source + "' style='"+style+"'></iframe>";
+		modalWindow.content = "<iframe width='"+width+"' height='"+height+"' frameborder='0' scrolling='"+scrolling+"' allowtransparency='true' src='" + source + "'>&lt/iframe>";
 		modalWindow.open();
-	};
+	};	
 // END LON-CAPA Internal -->
 // ]]>
 </script>
@@ -8379,18 +7622,16 @@ ENDMODAL
 }
 
 sub modal_link {
-    my ($link,$linktext,$width,$height,$target,$scrolling,$title,$transparency,$style)=@_;
+    my ($link,$linktext,$width,$height,$target,$scrolling,$title)=@_;
     unless ($width) { $width=480; }
     unless ($height) { $height=400; }
     unless ($scrolling) { $scrolling='yes'; }
-    unless ($transparency) { $transparency='true'; }
-
     my $target_attr;
     if (defined($target)) {
         $target_attr = 'target="'.$target.'"';
     }
     return <<"ENDLINK";
-<a href="$link" $target_attr title="$title" onclick="javascript:openMyModal('$link',$width,$height,'$scrolling','$transparency','$style'); return false;">
+<a href="$link" $target_attr title="$title" onclick="javascript:openMyModal('$link',$width,$height,'$scrolling'); return false;">
            $linktext</a>
 ENDLINK
 }
@@ -8418,10 +7659,10 @@ sub modal_adhoc_inner {
     my $innerwidth=$width-20;
     $content=&js_ready(
                &start_page('Dialog',undef,{'only_body'=>1,'bgcolor'=>'#FFFFFF'}).
-                 &start_scrollbox($width.'px',$innerwidth.'px',$height.'px','myModal','#FFFFFF',undef,1).
-                 $content.
+                 &start_scrollbox($width.'px',$innerwidth.'px',$height.'px').
+                    $content.
                  &end_scrollbox().
-                 &end_page()
+               &end_page()
              );
     return &modal_adhoc_script($funcname,$width,$height,$content);
 }
@@ -8494,9 +7735,8 @@ sub end_togglebox {
 }
 
 sub LCprogressbar_script {
-   my ($id,$number_to_do)=@_;
-   if ($number_to_do) {
-       return(<<ENDPROGRESS);
+   my ($id)=@_;
+   return(<<ENDPROGRESS);
 <script type="text/javascript">
 // <![CDATA[
 \$('#progressbar$id').progressbar({
@@ -8509,43 +7749,21 @@ sub LCprogressbar_script {
 // ]]>
 </script>
 ENDPROGRESS
-   } else {
-       return(<<ENDPROGRESS);
-<script type="text/javascript">
-// <![CDATA[
-\$('#progressbar$id').progressbar({
-  value: false,
-  create: function(event, ui) {
-    \$('.ui-widget-header', this).css({'background':'#F0F0F0'});
-    \$('.ui-progressbar-overlay', this).css({'margin':'0'});
-  }
-});
-// ]]>
-</script>
-ENDPROGRESS
-   }
 }
 
 sub LCprogressbarUpdate_script {
    return(<<ENDPROGRESSUPDATE);
 <style type="text/css">
 .ui-progressbar { position:relative; }
-.progress-label {position: absolute; width: 100%; text-align: center; top: 1px; font-weight: bold; text-shadow: 1px 1px 0 #fff;margin: 0; line-height: 200%; }
 .pblabel { position: absolute; width: 100%; text-align: center; line-height: 1.9em; }
 </style>
 <script type="text/javascript">
 // <![CDATA[
 var LCprogressTxt='---';
 
-function LCupdateProgress(percent,progresstext,id,maxnum) {
+function LCupdateProgress(percent,progresstext,id) {
    LCprogressTxt=progresstext;
-   if ((maxnum == '') || (maxnum == undefined) || (maxnum == null)) {
-       \$('#progressbar'+id).find('.progress-label').text(LCprogressTxt);
-   } else if (percent === \$('#progressbar'+id).progressbar( "value" )) {
-       \$('#progressbar'+id).find('.pblabel').text(LCprogressTxt);
-   } else {
-       \$('#progressbar'+id).progressbar('value',percent);
-   }
+   \$('#progressbar'+id).progressbar('value',percent);
 }
 // ]]>
 </script>
@@ -8557,54 +7775,39 @@ my $LCidcnt;
 my $LCcurrentid;
 
 sub LCprogressbar {
-    my ($r,$number_to_do,$preamble)=@_;
+    my ($r)=(@_);
     $LClastpercent=0;
     $LCidcnt++;
     $LCcurrentid=$$.'_'.$LCidcnt;
-    my ($starting,$content);
-    if ($number_to_do) {
-        $starting=&mt('Starting');
-        $content=(<<ENDPROGBAR);
-$preamble
+    my $starting=&mt('Starting');
+    my $content=(<<ENDPROGBAR);
+<p>
   <div id="progressbar$LCcurrentid">
     <span class="pblabel">$starting</span>
   </div>
+</p>
 ENDPROGBAR
-    } else {
-        $starting=&mt('Loading...');
-        $LClastpercent='false';
-        $content=(<<ENDPROGBAR);
-$preamble
-  <div id="progressbar$LCcurrentid">
-      <div class="progress-label">$starting</div>
-  </div>
-ENDPROGBAR
-    }
-    &r_print($r,$content.&LCprogressbar_script($LCcurrentid,$number_to_do));
+    &r_print($r,$content.&LCprogressbar_script($LCcurrentid));
 }
 
 sub LCprogressbarUpdate {
-    my ($r,$val,$text,$number_to_do)=@_;
-    if ($number_to_do) {
-        unless ($val) { 
-            if ($LClastpercent) {
-                $val=$LClastpercent;
-            } else {
-                $val=0;
-            }
-        }
-        if ($val<0) { $val=0; }
-        if ($val>100) { $val=0; }
-        $LClastpercent=$val;
-        unless ($text) { $text=$val.'%'; }
-    } else {
-        $val = 'false';
+    my ($r,$val,$text)=@_;
+    unless ($val) { 
+       if ($LClastpercent) {
+           $val=$LClastpercent;
+       } else {
+           $val=0;
+       }
     }
+    if ($val<0) { $val=0; }
+    if ($val>100) { $val=0; }
+    $LClastpercent=$val;
+    unless ($text) { $text=$val.'%'; }
     $text=&js_ready($text);
     &r_print($r,<<ENDUPDATE);
 <script type="text/javascript">
 // <![CDATA[
-LCupdateProgress($val,'$text','$LCcurrentid','$number_to_do');
+LCupdateProgress($val,'$text','$LCcurrentid');
 // ]]>
 </script>
 ENDUPDATE
@@ -8678,27 +7881,20 @@ sub validate_page {
 
 
 sub start_scrollbox {
-    my ($outerwidth,$width,$height,$id,$bgcolor,$cursor,$needjsready) = @_;
+    my ($outerwidth,$width,$height,$id,$bgcolor)=@_;
     unless ($outerwidth) { $outerwidth='520px'; }
     unless ($width) { $width='500px'; }
     unless ($height) { $height='200px'; }
     my ($table_id,$div_id,$tdcol);
     if ($id ne '') {
-        $table_id = ' id="table_'.$id.'"';
-        $div_id = ' id="div_'.$id.'"';
+        $table_id = " id='table_$id'";
+        $div_id = " id='div_$id'";
     }
     if ($bgcolor ne '') {
         $tdcol = "background-color: $bgcolor;";
     }
-    my $nicescroll_js;
-    if ($env{'browser.mobile'}) {
-        $nicescroll_js = &nicescroll_javascript('div_'.$id,$cursor,$needjsready);
-    }
     return <<"END";
-$nicescroll_js
-
-<table style="width: $outerwidth; border: 1px solid none;"$table_id><tr><td style="width: $width;$tdcol">
-<div style="overflow:auto; width:$width; height:$height;"$div_id>
+<table style="width: $outerwidth; border: 1px solid none;"$table_id><tr><td style="width: $width;$tdcol"><div style="overflow:auto; width:$width; height: $height;"$div_id>
 END
 }
 
@@ -8706,98 +7902,11 @@ sub end_scrollbox {
     return '</div></td></tr></table>';
 }
 
-sub nicescroll_javascript {
-    my ($id,$cursor,$needjsready,$framecheck,$location) = @_;
-    my %options;
-    if (ref($cursor) eq 'HASH') {
-        %options = %{$cursor};
-    }
-    unless ($options{'railalign'} =~ /^left|right$/) {
-        $options{'railalign'} = 'left';
-    }
-    unless ($options{'cursorcolor'} =~ /^\#\w+$/) {
-        my $function  = &get_users_function();
-        $options{'cursorcolor'} = &designparm($function.'.sidebg',$env{'request.role.domain'});
-        unless ($options{'cursorcolor'} =~ /^\#\w+$/) {
-            $options{'cursorcolor'} = '#00F';
-        }
-    }
-    if ($options{'cursoropacity'} =~ /^[\d.]+$/) {
-        unless ($options{'cursoropacity'} >= 0.0 && $options{'cursoropacity'} <=1.0) {
-            $options{'cursoropacity'}='1.0';
-        }
-    } else {
-        $options{'cursoropacity'}='1.0';
-    }
-    if ($options{'cursorfixedheight'} eq 'none') {
-        delete($options{'cursorfixedheight'});
-    } else {
-        unless ($options{'cursorfixedheight'} =~ /^\d+$/) { $options{'cursorfixedheight'}='50'; }
-    }
-    unless ($options{'railoffset'} =~ /^{[\w\:\d\-,]+}$/) {
-        delete($options{'railoffset'});
-    }
-    my @niceoptions;
-    while (my($key,$value) = each(%options)) {
-        if ($value =~ /^\{.+\}$/) {
-            push(@niceoptions,$key.':'.$value);
-        } else {
-            push(@niceoptions,$key.':"'.$value.'"');
-        }
-    }
-    my $nicescroll_js = '
-$(document).ready(
-      function() {
-          $("#'.$id.'").niceScroll({'.join(',',@niceoptions).'});
-      }
-);
-';
-    if ($framecheck) {
-        $nicescroll_js .= '
-function expand_div(caller) {
-    if (top === self) {
-        document.getElementById("'.$id.'").style.width = "auto";
-        document.getElementById("'.$id.'").style.height = "auto";
-    } else {
-        try {
-            if (parent.frames) {
-                if (parent.frames.length > 1) {
-                    var framesrc = parent.frames[1].location.href;
-                    var currsrc = framesrc.replace(/\#.*$/,"");
-                    if ((caller == "search") || (currsrc == "'.$location.'")) {
-                        document.getElementById("'.$id.'").style.width = "auto";
-                        document.getElementById("'.$id.'").style.height = "auto";
-                    }
-                }
-            }
-        } catch (e) {
-            return;
-        }
-    }
-    return;
-}
-';
-    }
-    if ($needjsready) {
-        $nicescroll_js = '
-<script type="text/javascript">'."\n".$nicescroll_js."\n</script>\n";
-    } else {
-        $nicescroll_js = &Apache::lonhtmlcommon::scripttag($nicescroll_js);
-    }
-    return $nicescroll_js;
-}
-
 sub simple_error_page {
-    my ($r,$title,$msg,$args) = @_;
-    if (ref($args) eq 'HASH') {
-        if (!$args->{'no_auto_mt_msg'}) { $msg = &mt($msg); }
-    } else {
-        $msg = &mt($msg);
-    }
-
+    my ($r,$title,$msg) = @_;
     my $page =
 	&Apache::loncommon::start_page($title).
-	'<p class="LC_error">'.$msg.'</p>'.
+	'<p class="LC_error">'.&mt($msg).'</p>'.
 	&Apache::loncommon::end_page();
     if (ref($r)) {
 	$r->print($page);
@@ -9013,7 +8122,7 @@ role status: active, previous or future.
 sub check_user_status {
     my ($udom,$uname,$cdom,$crs,$role,$sec) = @_;
     my %userinfo = &Apache::lonnet::dump('roles',$udom,$uname);
-    my @uroles = keys(%userinfo);
+    my @uroles = keys %userinfo;
     my $srchstr;
     my $active_chk = 'none';
     my $now = time;
@@ -9090,19 +8199,7 @@ sub get_sections {
     my %sectioncount;
     my $now = time;
 
-    my $check_students = 1;
-    my $only_students = 0;
-    if (ref($possible_roles) eq 'ARRAY') {
-        if (grep(/^st$/,@{$possible_roles})) {
-            if (@{$possible_roles} == 1) {
-                $only_students = 1;
-            }
-        } else {
-            $check_students = 0;
-        }
-    }
-
-    if ($check_students) {
+    if (!defined($possible_roles) || (grep(/^st$/,@$possible_roles))) {
 	my ($classlist) = &Apache::loncoursedata::get_classlist($cdom,$cnum);
 	my $sec_index = &Apache::loncoursedata::CL_SECTION();
 	my $status_index = &Apache::loncoursedata::CL_STATUS();
@@ -9129,9 +8226,6 @@ sub get_sections {
 	    }
 	}
     }
-    if ($only_students) {
-        return %sectioncount;
-    }
     my %courseroles = &Apache::lonnet::dump('nohist_userroles',$cdom,$cnum);
     foreach my $user (sort(keys(%courseroles))) {
 	if ($user !~ /^(\w{2})/) { next; }
@@ -9279,7 +8373,7 @@ sub get_course_users {
                               active   => 'Active',
                               future   => 'Future',
                             );
-        my (%nothide,@possdoms);
+        my %nothide;
         if ($hidepriv) {
             my %coursehash=&Apache::lonnet::coursedescription($cdom.'_'.$cnum);
             foreach my $user (split(/\s*\,\s*/,$coursehash{'nothideprivileged'})) {
@@ -9289,10 +8383,6 @@ sub get_course_users {
                     $nothide{$user} = 1;
                 }
             }
-            my @possdoms = ($cdom);
-            if ($coursehash{'checkforpriv'}) {
-                push(@possdoms,split(/,/,$coursehash{'checkforpriv'}));
-            }
         }
         foreach my $person (sort(keys(%coursepersonnel))) {
             my $match = 0;
@@ -9328,7 +8418,7 @@ sub get_course_users {
                 }
                 if ($uname ne '' && $udom ne '') {
                     if ($hidepriv) {
-                        if ((&Apache::lonnet::privileged($uname,$udom,\@possdoms)) &&
+                        if ((&Apache::lonnet::privileged($uname,$udom)) &&
                             (!$nothide{$uname.':'.$udom})) {
                             next;
                         }
@@ -9416,19 +8506,14 @@ sub get_user_info {
 
 =item * &get_user_quota()
 
-Retrieves quota assigned for storage of user files.
-Default is to report quota for portfolio files.
+Retrieves quota assigned for storage of portfolio files for a user  
 
 Incoming parameters:
 1. user's username
 2. user's domain
-3. quota name - portfolio, author, or course
-   (if no quota name provided, defaults to portfolio).
-4. crstype - official, unofficial, textbook or community, if quota name is
-   course
 
 Returns:
-1. Disk quota (in MB) assigned to student.
+1. Disk quota (in Mb) assigned to student.
 2. (Optional) Type of setting: custom or default
    (individually assigned or default for user's 
    institutional status).
@@ -9439,7 +8524,7 @@ Returns:
 
 If a value has been stored in the user's environment, 
 it will return that, otherwise it returns the maximal default
-defined for the user's institutional status(es) in the domain.
+defined for the user's instituional status(es) in the domain.
 
 =cut
 
@@ -9447,7 +8532,7 @@ defined for the user's institutional sta
 
 
 sub get_user_quota {
-    my ($uname,$udom,$quotaname,$crstype) = @_;
+    my ($uname,$udom) = @_;
     my ($quota,$quotatype,$settingstatus,$defquota);
     if (!defined($udom)) {
         $udom = $env{'user.domain'};
@@ -9462,58 +8547,27 @@ sub get_user_quota {
         $defquota = 0; 
     } else {
         my $inststatus;
-        if ($quotaname eq 'course') {
-            if (($env{'course.'.$udom.'_'.$uname.'.num'} eq $uname) &&
-                ($env{'course.'.$udom.'_'.$uname.'.domain'} eq $udom)) {
-                $quota = $env{'course.'.$udom.'_'.$uname.'.internal.uploadquota'};
-            } else {
-                my %cenv = &Apache::lonnet::coursedescription("$udom/$uname");
-                $quota = $cenv{'internal.uploadquota'};
-            }
+        if ($udom eq $env{'user.domain'} && $uname eq $env{'user.name'}) {
+            $quota = $env{'environment.portfolioquota'};
+            $inststatus = $env{'environment.inststatus'};
+        } else {
+            my %userenv = 
+                &Apache::lonnet::get('environment',['portfolioquota',
+                                     'inststatus'],$udom,$uname);
+            my ($tmp) = keys(%userenv);
+            if ($tmp !~ /^(con_lost|error|no_such_host)/i) {
+                $quota = $userenv{'portfolioquota'};
+                $inststatus = $userenv{'inststatus'};
+            } else {
+                undef(%userenv);
+            }
+        }
+        ($defquota,$settingstatus) = &default_quota($udom,$inststatus);
+        if ($quota eq '') {
+            $quota = $defquota;
+            $quotatype = 'default';
         } else {
-            if ($udom eq $env{'user.domain'} && $uname eq $env{'user.name'}) {
-                if ($quotaname eq 'author') {
-                    $quota = $env{'environment.authorquota'};
-                } else {
-                    $quota = $env{'environment.portfolioquota'};
-                }
-                $inststatus = $env{'environment.inststatus'};
-            } else {
-                my %userenv = 
-                    &Apache::lonnet::get('environment',['portfolioquota',
-                                         'authorquota','inststatus'],$udom,$uname);
-                my ($tmp) = keys(%userenv);
-                if ($tmp !~ /^(con_lost|error|no_such_host)/i) {
-                    if ($quotaname eq 'author') {
-                        $quota = $userenv{'authorquota'};
-                    } else {
-                        $quota = $userenv{'portfolioquota'};
-                    }
-                    $inststatus = $userenv{'inststatus'};
-                } else {
-                    undef(%userenv);
-                }
-            }
-        }
-        if ($quota eq '' || wantarray) {
-            if ($quotaname eq 'course') {
-                my %domdefs = &Apache::lonnet::get_domain_defaults($udom);
-                if (($crstype eq 'official') || ($crstype eq 'unofficial') ||
-                    ($crstype eq 'community') || ($crstype eq 'textbook')) {
-                    $defquota = $domdefs{$crstype.'quota'};
-                }
-                if ($defquota eq '') {
-                    $defquota = 500;
-                }
-            } else {
-                ($defquota,$settingstatus) = &default_quota($udom,$inststatus,$quotaname);
-            }
-            if ($quota eq '') {
-                $quota = $defquota;
-                $quotatype = 'default';
-            } else {
-                $quotatype = 'custom';
-            }
+            $quotatype = 'custom';
         }
     }
     if (wantarray) {
@@ -9533,60 +8587,54 @@ Retrieves default quota assigned for sto
 given an (optional) user's institutional status.
 
 Incoming parameters:
-
 1. domain
 2. (Optional) institutional status(es).  This is a : separated list of 
    status types (e.g., faculty, staff, student etc.)
    which apply to the user for whom the default is being retrieved.
    If the institutional status string in undefined, the domain
-   default quota will be returned.
-3.  quota name - portfolio, author, or course
-   (if no quota name provided, defaults to portfolio).
+   default quota will be returned. 
 
 Returns:
-
-1. Default disk quota (in MB) for user portfolios in the domain.
+1. Default disk quota (in Mb) for user portfolios in the domain.
 2. (Optional) institutional type which determined the value of the
    default quota.
 
 If a value has been stored in the domain's configuration db,
 it will return that, otherwise it returns 20 (for backwards 
 compatibility with domains which have not set up a configuration
-db file; the original statically defined portfolio quota was 20 MB). 
+db file; the original statically defined portfolio quota was 20 Mb). 
 
 If the user's status includes multiple types (e.g., staff and student),
 the largest default quota which applies to the user determines the
 default quota returned.
 
+=back
+
 =cut
 
 ###############################################
 
 
 sub default_quota {
-    my ($udom,$inststatus,$quotaname) = @_;
+    my ($udom,$inststatus) = @_;
     my ($defquota,$settingstatus);
     my %quotahash = &Apache::lonnet::get_dom('configuration',
                                             ['quotas'],$udom);
-    my $key = 'defaultquota';
-    if ($quotaname eq 'author') {
-        $key = 'authorquota';
-    }
     if (ref($quotahash{'quotas'}) eq 'HASH') {
         if ($inststatus ne '') {
             my @statuses = map { &unescape($_); } split(/:/,$inststatus);
             foreach my $item (@statuses) {
-                if (ref($quotahash{'quotas'}{$key}) eq 'HASH') {
-                    if ($quotahash{'quotas'}{$key}{$item} ne '') {
+                if (ref($quotahash{'quotas'}{'defaultquota'}) eq 'HASH') {
+                    if ($quotahash{'quotas'}{'defaultquota'}{$item} ne '') {
                         if ($defquota eq '') {
-                            $defquota = $quotahash{'quotas'}{$key}{$item};
+                            $defquota = $quotahash{'quotas'}{'defaultquota'}{$item};
                             $settingstatus = $item;
-                        } elsif ($quotahash{'quotas'}{$key}{$item} > $defquota) {
-                            $defquota = $quotahash{'quotas'}{$key}{$item};
+                        } elsif ($quotahash{'quotas'}{'defaultquota'}{$item} > $defquota) {
+                            $defquota = $quotahash{'quotas'}{'defaultquota'}{$item};
                             $settingstatus = $item;
                         }
                     }
-                } elsif ($key eq 'defaultquota') {
+                } else {
                     if ($quotahash{'quotas'}{$item} ne '') {
                         if ($defquota eq '') {
                             $defquota = $quotahash{'quotas'}{$item};
@@ -9600,25 +8648,16 @@ sub default_quota {
             }
         }
         if ($defquota eq '') {
-            if (ref($quotahash{'quotas'}{$key}) eq 'HASH') {
-                $defquota = $quotahash{'quotas'}{$key}{'default'};
-            } elsif ($key eq 'defaultquota') {
+            if (ref($quotahash{'quotas'}{'defaultquota'}) eq 'HASH') {
+                $defquota = $quotahash{'quotas'}{'defaultquota'}{'default'};
+            } else {
                 $defquota = $quotahash{'quotas'}{'default'};
             }
             $settingstatus = 'default';
-            if ($defquota eq '') {
-                if ($quotaname eq 'author') {
-                    $defquota = 500;
-                }
-            }
         }
     } else {
         $settingstatus = 'default';
-        if ($quotaname eq 'author') {
-            $defquota = 500;
-        } else {
-            $defquota = 20;
-        }
+        $defquota = 20;
     }
     if (wantarray) {
         return ($defquota,$settingstatus);
@@ -9627,62 +8666,6 @@ sub default_quota {
     }
 }
 
-###############################################
-
-=pod
-
-=item * &excess_filesize_warning()
-
-Returns warning message if upload of file to authoring space, or copying
-of existing file within authoring space will cause quota for the authoring
-space to be exceeded.
-
-Same, if upload of a file directly to a course/community via Course Editor
-will cause quota for uploaded content for the course to be exceeded.
-
-Inputs: 7 
-1. username or coursenum
-2. domain
-3. context ('author' or 'course')
-4. filename of file for which action is being requested
-5. filesize (kB) of file
-6. action being taken: copy or upload.
-7. quotatype (in course context -- official, unofficial, community or textbook).
-
-Returns: 1 scalar: HTML to display containing warning if quota would be exceeded,
-         otherwise return null.
-
-=back
-
-=cut
-
-sub excess_filesize_warning {
-    my ($uname,$udom,$context,$filename,$filesize,$action,$quotatype) = @_;
-    my $current_disk_usage = 0;
-    my $disk_quota = &get_user_quota($uname,$udom,$context,$quotatype); #expressed in MB
-    if ($context eq 'author') {
-        my $authorspace = $Apache::lonnet::perlvar{'lonDocRoot'}."/priv/$udom/$uname";
-        $current_disk_usage = &Apache::lonnet::diskusage($udom,$uname,$authorspace);
-    } else {
-        foreach my $subdir ('docs','supplemental') {
-            $current_disk_usage += &Apache::lonnet::diskusage($udom,$uname,"userfiles/$subdir",1);
-        }
-    }
-    $disk_quota = int($disk_quota * 1000);
-    if (($current_disk_usage + $filesize) > $disk_quota) {
-        return '<p class="LC_warning">'.
-                &mt("Unable to $action [_1]. (size = [_2] kilobytes). Disk quota will be exceeded.",
-                    '<span class="LC_filename">'.$filename.'</span>',$filesize).'</p>'.
-               '<p>'.&mt('Disk quota is [_1] kilobytes. Your current disk usage is [_2] kilobytes.',
-                            $disk_quota,$current_disk_usage).
-               '</p>';
-    }
-    return;
-}
-
-###############################################
-
-
 sub get_secgrprole_info {
     my ($cdom,$cnum,$needroles,$type)  = @_;
     my %sections_count = &get_sections($cdom,$cnum);
@@ -9721,24 +8704,8 @@ sub get_secgrprole_info {
 }
 
 sub user_picker {
-    my ($dom,$srch,$forcenewuser,$caller,$cancreate,$usertype,$context,$fixeddom,$noinstd) = @_;
+    my ($dom,$srch,$forcenewuser,$caller,$cancreate,$usertype,$context) = @_;
     my $currdom = $dom;
-    my @alldoms = &Apache::lonnet::all_domains();
-    if (@alldoms == 1) {
-        my %domsrch = &Apache::lonnet::get_dom('configuration',
-                                               ['directorysrch'],$alldoms[0]);
-        my $domdesc = &Apache::lonnet::domain($alldoms[0],'description');
-        my $showdom = $domdesc;
-        if ($showdom eq '') {
-            $showdom = $dom;
-        }
-        if (ref($domsrch{'directorysrch'}) eq 'HASH') {
-            if ((!$domsrch{'directorysrch'}{'available'}) &&
-                ($domsrch{'directorysrch'}{'lcavailable'} eq '0')) {
-                return (&mt('LON-CAPA directory search is not available in domain: [_1]',$showdom),0);
-            }
-        }
-    }
     my %curr_selected = (
                         srchin => 'dom',
                         srchby => 'lastname',
@@ -9759,7 +8726,7 @@ sub user_picker {
         }
         $srchterm = $srch->{'srchterm'};
     }
-    my %html_lt=&Apache::lonlocal::texthash(
+    my %lt=&Apache::lonlocal::texthash(
                     'usr'       => 'Search criteria',
                     'doma'      => 'Domain/institution to search',
                     'uname'     => 'username',
@@ -9772,8 +8739,6 @@ sub user_picker {
                     'exact'     => 'is',
                     'contains'  => 'contains',
                     'begins'    => 'begins with',
-                                       );
-    my %js_lt=&Apache::lonlocal::texthash(
                     'youm'      => "You must include some text to search for.",
                     'thte'      => "The text you are searching for must contain at least two characters when using a 'begins' type search.",
                     'thet'      => "The text you are searching for must contain at least three characters when using a 'contains' type search.",
@@ -9783,16 +8748,7 @@ sub user_picker {
                     'whse'      => "When searching by last,first you must include at least one character in the first name.",
                      'thfo'     => "The following need to be corrected before the search can be run:",
                                        );
-    &html_escape(\%html_lt);
-    &js_escape(\%js_lt);
-    my $domform;
-    my $allow_blank = 1;
-    if ($fixeddom) {
-        $allow_blank = 0;
-        $domform = &select_dom_form($currdom,'srchdomain',$allow_blank,1,undef,[$currdom]);
-    } else {
-        $domform = &select_dom_form($currdom,'srchdomain',$allow_blank,1);
-    }
+    my $domform = &select_dom_form($currdom,'srchdomain',1,1);
     my $srchinsel = ' <select name="srchin">';
 
     my @srchins = ('crs','dom','alc','instd');
@@ -9804,13 +8760,12 @@ sub user_picker {
         next if ($option eq 'alc');
         next if (($option eq 'crs') && ($env{'form.form'} eq 'requestcrs'));  
         next if ($option eq 'crs' && !$env{'request.course.id'});
-        next if (($option eq 'instd') && ($noinstd));
         if ($curr_selected{'srchin'} eq $option) {
             $srchinsel .= ' 
-   <option value="'.$option.'" selected="selected">'.$html_lt{$option}.'</option>';
+   <option value="'.$option.'" selected="selected">'.$lt{$option}.'</option>';
         } else {
             $srchinsel .= '
-   <option value="'.$option.'">'.$html_lt{$option}.'</option>';
+   <option value="'.$option.'">'.$lt{$option}.'</option>';
         }
     }
     $srchinsel .= "\n  </select>\n";
@@ -9819,10 +8774,10 @@ sub user_picker {
     foreach my $option ('lastname','lastfirst','uname') {
         if ($curr_selected{'srchby'} eq $option) {
             $srchbysel .= '
-   <option value="'.$option.'" selected="selected">'.$html_lt{$option}.'</option>';
+   <option value="'.$option.'" selected="selected">'.$lt{$option}.'</option>';
         } else {
             $srchbysel .= '
-   <option value="'.$option.'">'.$html_lt{$option}.'</option>';
+   <option value="'.$option.'">'.$lt{$option}.'</option>';
          }
     }
     $srchbysel .= "\n  </select>\n";
@@ -9831,10 +8786,10 @@ sub user_picker {
     foreach my $option ('begins','contains','exact') {
         if ($curr_selected{'srchtype'} eq $option) {
             $srchtypesel .= '
-   <option value="'.$option.'" selected="selected">'.$html_lt{$option}.'</option>';
+   <option value="'.$option.'" selected="selected">'.$lt{$option}.'</option>';
         } else {
             $srchtypesel .= '
-   <option value="'.$option.'">'.$html_lt{$option}.'</option>';
+   <option value="'.$option.'">'.$lt{$option}.'</option>';
         }
     }
     $srchtypesel .= "\n  </select>\n";
@@ -9919,46 +8874,46 @@ function validateEntry(callingForm) {
 
     if (srchterm == "") {
         checkok = 0;
-        msg += "$js_lt{'youm'}\\n";
+        msg += "$lt{'youm'}\\n";
     }
 
     if (srchtype== 'begins') {
         if (srchterm.length < 2) {
             checkok = 0;
-            msg += "$js_lt{'thte'}\\n";
+            msg += "$lt{'thte'}\\n";
         }
     }
 
     if (srchtype== 'contains') {
         if (srchterm.length < 3) {
             checkok = 0;
-            msg += "$js_lt{'thet'}\\n";
+            msg += "$lt{'thet'}\\n";
         }
     }
     if (srchin == 'instd') {
         if (srchdomain == '') {
             checkok = 0;
-            msg += "$js_lt{'yomc'}\\n";
+            msg += "$lt{'yomc'}\\n";
         }
     }
     if (srchin == 'dom') {
         if (srchdomain == '') {
             checkok = 0;
-            msg += "$js_lt{'ymcd'}\\n";
+            msg += "$lt{'ymcd'}\\n";
         }
     }
     if (srchby == 'lastfirst') {
         if (srchterm.indexOf(",") == -1) {
             checkok = 0;
-            msg += "$js_lt{'whus'}\\n";
+            msg += "$lt{'whus'}\\n";
         }
         if (srchterm.indexOf(",") == srchterm.length -1) {
             checkok = 0;
-            msg += "$js_lt{'whse'}\\n";
+            msg += "$lt{'whse'}\\n";
         }
     }
     if (checkok == 0) {
-        alert("$js_lt{'thfo'}\\n"+msg);
+        alert("$lt{'thfo'}\\n"+msg);
         return;
     }
     if (checkok == 1) {
@@ -9976,10 +8931,10 @@ $new_user_create
 END_BLOCK
 
     $output .= &Apache::lonhtmlcommon::start_pick_box().
-               &Apache::lonhtmlcommon::row_title($html_lt{'doma'}).
+               &Apache::lonhtmlcommon::row_title($lt{'doma'}).
                $domform.
                &Apache::lonhtmlcommon::row_closure().
-               &Apache::lonhtmlcommon::row_title($html_lt{'usr'}).
+               &Apache::lonhtmlcommon::row_title($lt{'usr'}).
                $srchbysel.
                $srchtypesel. 
                '<input type="text" size="15" name="srchterm" value="'.$srchterm.'" />'.
@@ -9987,165 +8942,61 @@ END_BLOCK
                &Apache::lonhtmlcommon::row_closure(1). 
                &Apache::lonhtmlcommon::end_pick_box().
                '<br />';
-    return ($output,1);
+    return $output;
 }
 
 sub user_rule_check {
     my ($usershash,$checks,$alerts,$rulematch,$inst_results,$curr_rules,$got_rules) = @_;
-    my ($response,%inst_response);
+    my $response;
     if (ref($usershash) eq 'HASH') {
-        if (keys(%{$usershash}) > 1) {
-            my (%by_username,%by_id,%userdoms);
-            my $checkid;
-            if (ref($checks) eq 'HASH') {
-                if ((!defined($checks->{'username'})) && (defined($checks->{'id'}))) {
-                    $checkid = 1;
-                }
-            }
-            foreach my $user (keys(%{$usershash})) {
-                my ($uname,$udom) = split(/:/,$user);
-                if ($checkid) {
-                    if (ref($usershash->{$user}) eq 'HASH') {
-                        if ($usershash->{$user}->{'id'} ne '') {
-                            $by_id{$udom}{$usershash->{$user}->{'id'}} = $uname;
-                            $userdoms{$udom} = 1;
-                            if (ref($inst_results) eq 'HASH') {
-                                $inst_results->{$uname.':'.$udom} = {};
-                            }
-                        }
-                    }
-                } else {
-                    $by_username{$udom}{$uname} = 1;
-                    $userdoms{$udom} = 1;
-                    if (ref($inst_results) eq 'HASH') {
-                        $inst_results->{$uname.':'.$udom} = {};
-                    }
-                }
-            }
-            foreach my $udom (keys(%userdoms)) {
-                if (!$got_rules->{$udom}) {
-                    my %domconfig = &Apache::lonnet::get_dom('configuration',
-                                                             ['usercreation'],$udom);
-                    if (ref($domconfig{'usercreation'}) eq 'HASH') {
-                        foreach my $item ('username','id') {
-                            if (ref($domconfig{'usercreation'}{$item.'_rule'}) eq 'ARRAY') {
-                                $$curr_rules{$udom}{$item} =
-                                    $domconfig{'usercreation'}{$item.'_rule'};
-                            }
-                        }
-                    }
-                    $got_rules->{$udom} = 1;
-                }
+        foreach my $user (keys(%{$usershash})) {
+            my ($uname,$udom) = split(/:/,$user);
+            next if ($udom eq '' || $uname eq '');
+            my ($id,$newuser);
+            if (ref($usershash->{$user}) eq 'HASH') {
+                $newuser = $usershash->{$user}->{'newuser'};
+                $id = $usershash->{$user}->{'id'};
             }
-            if ($checkid) {
-                foreach my $udom (keys(%by_id)) {
-                    my ($outcome,$results) = &Apache::lonnet::get_multiple_instusers($udom,$by_id{$udom},'id');
-                    if ($outcome eq 'ok') {
-                        foreach my $id (keys(%{$by_id{$udom}})) {
-                            my $uname = $by_id{$udom}{$id};
-                            $inst_response{$uname.':'.$udom} = $outcome;
-                        }
-                        if (ref($results) eq 'HASH') {
-                            foreach my $uname (keys(%{$results})) {
-                                if (exists($inst_response{$uname.':'.$udom})) {
-                                    $inst_response{$uname.':'.$udom} = $outcome;
-                                    $inst_results->{$uname.':'.$udom} = $results->{$uname};
-                                }
-                            }
-                        }
-                    }
+            my $inst_response;
+            if (ref($checks) eq 'HASH') {
+                if (defined($checks->{'username'})) {
+                    ($inst_response,%{$inst_results->{$user}}) = 
+                        &Apache::lonnet::get_instuser($udom,$uname);
+                } elsif (defined($checks->{'id'})) {
+                    ($inst_response,%{$inst_results->{$user}}) =
+                        &Apache::lonnet::get_instuser($udom,undef,$id);
                 }
             } else {
-                foreach my $udom (keys(%by_username)) {
-                    my ($outcome,$results) = &Apache::lonnet::get_multiple_instusers($udom,$by_username{$udom});
-                    if ($outcome eq 'ok') {
-                        foreach my $uname (keys(%{$by_username{$udom}})) {
-                            $inst_response{$uname.':'.$udom} = $outcome;
-                        }
-                        if (ref($results) eq 'HASH') {
-                            foreach my $uname (keys(%{$results})) {
-                                $inst_results->{$uname.':'.$udom} = $results->{$uname};
-                            }
-                        }
-                    }
-                }
+                ($inst_response,%{$inst_results->{$user}}) =
+                    &Apache::lonnet::get_instuser($udom,$uname);
+                return;
             }
-        } elsif (keys(%{$usershash}) == 1) {
-            my $user = (keys(%{$usershash}))[0];
-            my ($uname,$udom) = split(/:/,$user);
-            if (($udom ne '') && ($uname ne '')) {
-                if (ref($usershash->{$user}) eq 'HASH') {
-                    if (ref($checks) eq 'HASH') {
-                        if (defined($checks->{'username'})) {
-                            ($inst_response{$user},%{$inst_results->{$user}}) =
-                                &Apache::lonnet::get_instuser($udom,$uname);
-                        } elsif (defined($checks->{'id'})) {
-                            if ($usershash->{$user}->{'id'} ne '') {
-                                ($inst_response{$user},%{$inst_results->{$user}}) =
-                                    &Apache::lonnet::get_instuser($udom,undef,
-                                                                  $usershash->{$user}->{'id'});
-                            } else {
-                                ($inst_response{$user},%{$inst_results->{$user}}) =
-                                    &Apache::lonnet::get_instuser($udom,$uname);
-                            }
+            if (!$got_rules->{$udom}) {
+                my %domconfig = &Apache::lonnet::get_dom('configuration',
+                                                  ['usercreation'],$udom);
+                if (ref($domconfig{'usercreation'}) eq 'HASH') {
+                    foreach my $item ('username','id') {
+                        if (ref($domconfig{'usercreation'}{$item.'_rule'}) eq 'ARRAY') {
+                            $$curr_rules{$udom}{$item} = 
+                                $domconfig{'usercreation'}{$item.'_rule'};
                         }
-                    } else {
-                       ($inst_response{$user},%{$inst_results->{$user}}) =
-                            &Apache::lonnet::get_instuser($udom,$uname);
-                       return;
-                    }
-                    if (!$got_rules->{$udom}) {
-                        my %domconfig = &Apache::lonnet::get_dom('configuration',
-                                                                 ['usercreation'],$udom);
-                        if (ref($domconfig{'usercreation'}) eq 'HASH') {
-                            foreach my $item ('username','id') {
-                                if (ref($domconfig{'usercreation'}{$item.'_rule'}) eq 'ARRAY') {
-                                   $$curr_rules{$udom}{$item} =
-                                       $domconfig{'usercreation'}{$item.'_rule'};
-                                }
-                            }
-                        }
-                        $got_rules->{$udom} = 1;
                     }
                 }
-            } else {
-                return;
-            }
-        } else {
-            return;
-        }
-        foreach my $user (keys(%{$usershash})) {
-            my ($uname,$udom) = split(/:/,$user);
-            next if (($udom eq '') || ($uname eq ''));
-            my $id;
-            if (ref($inst_results) eq 'HASH') {
-                if (ref($inst_results->{$user}) eq 'HASH') {
-                    $id = $inst_results->{$user}->{'id'};
-                }
-            }
-            if ($id eq '') {
-                if (ref($usershash->{$user})) {
-                    $id = $usershash->{$user}->{'id'};
-                }
+                $got_rules->{$udom} = 1;  
             }
             foreach my $item (keys(%{$checks})) {
                 if (ref($$curr_rules{$udom}) eq 'HASH') {
                     if (ref($$curr_rules{$udom}{$item}) eq 'ARRAY') {
                         if (@{$$curr_rules{$udom}{$item}} > 0) {
-                            my %rule_check = &Apache::lonnet::inst_rulecheck($udom,$uname,$id,$item,
-                                                                             $$curr_rules{$udom}{$item});
+                            my %rule_check = &Apache::lonnet::inst_rulecheck($udom,$uname,$id,$item,$$curr_rules{$udom}{$item});
                             foreach my $rule (@{$$curr_rules{$udom}{$item}}) {
                                 if ($rule_check{$rule}) {
                                     $$rulematch{$user}{$item} = $rule;
-                                    if ($inst_response{$user} eq 'ok') {
+                                    if ($inst_response eq 'ok') {
                                         if (ref($inst_results) eq 'HASH') {
                                             if (ref($inst_results->{$user}) eq 'HASH') {
                                                 if (keys(%{$inst_results->{$user}}) == 0) {
                                                     $$alerts{$item}{$udom}{$uname} = 1;
-                                                } elsif ($item eq 'id') {
-                                                    if ($inst_results->{$user}->{'id'} eq '') {
-                                                        $$alerts{$item}{$udom}{$uname} = 1;
-                                                    }
                                                 }
                                             }
                                         }
@@ -10256,14 +9107,7 @@ sub personal_data_fieldtitles {
 
 sub sorted_inst_types {
     my ($dom) = @_;
-    my ($usertypes,$order);
-    my %domdefaults = &Apache::lonnet::get_domain_defaults($dom);
-    if (ref($domdefaults{'inststatus'}) eq 'HASH') {
-        $usertypes = $domdefaults{'inststatus'}{'inststatustypes'};
-        $order = $domdefaults{'inststatus'}{'inststatusorder'};
-    } else {
-        ($usertypes,$order) = &Apache::lonnet::retrieve_inst_usertypes($dom);
-    }
+    my ($usertypes,$order) = &Apache::lonnet::retrieve_inst_usertypes($dom);
     my $othertitle = &mt('All users');
     if ($env{'request.course.id'}) {
         $othertitle  = &mt('Any users');
@@ -10302,7 +9146,7 @@ sub get_institutional_codes {
         foreach (@currxlists) {
             if (m/^([^:]+):(\w*)$/) {
                 unless (grep/^$1$/,@{$allcourses}) {
-                    push(@{$allcourses},$1);
+                    push @{$allcourses},$1;
                     $$LC_code{$1} = $2;
                 }
             }
@@ -10315,7 +9159,7 @@ sub get_institutional_codes {
                 my $sec = $coursecode.$1;
                 my $lc_sec = $2;
                 unless (grep/^$sec$/,@{$allcourses}) {
-                    push(@{$allcourses},$sec);
+                    push @{$allcourses},$sec;
                     $$LC_code{$sec} = $lc_sec;
                 }
             }
@@ -10413,9 +9257,7 @@ reservable_now - ref to hash of student_
 
     Keys in inner hash are:
     (a) symb: either blank or symb to which slot use is restricted.
-    (b) endreserve: end date of reservation period.
-    (c) uniqueperiod: start,end dates when slot is to be uniquely
-        selected.
+    (b) endreserve: end date of reservation period. 
 
 sorted_future - ref to array of student_schedulable slots reservable in
                 the future, ordered by start date of reservation period.
@@ -10426,8 +9268,6 @@ future_reservable - ref to hash of stude
     Keys in inner hash are:
     (a) symb: either blank or symb to which slot use is restricted.
     (b) startreserve:  start date of reservation period.
-    (c) uniqueperiod: start,end dates when slot is to be uniquely
-        selected.
 
 =back
 
@@ -10481,10 +9321,6 @@ sub get_future_slots {
             my $startreserve = $slots{$slot}->{'startreserve'};
             my $endreserve = $slots{$slot}->{'endreserve'};
             my $symb = $slots{$slot}->{'symb'};
-            my $uniqueperiod;
-            if (ref($slots{$slot}->{'uniqueperiod'}) eq 'ARRAY') {
-                $uniqueperiod = join(',',@{$slots{$slot}->{'uniqueperiod'}});
-            }
             if (($startreserve < $now) &&
                 (!$endreserve || $endreserve > $now)) {
                 my $lastres = $endreserve;
@@ -10493,15 +9329,13 @@ sub get_future_slots {
                 }
                 $reservable_now{$slot} = {
                                            symb       => $symb,
-                                           endreserve => $lastres,
-                                           uniqueperiod => $uniqueperiod,   
+                                           endreserve => $lastres
                                          };
             } elsif (($startreserve > $now) &&
                      (!$endreserve || $endreserve > $startreserve)) {
                 $future_reservable{$slot} = {
                                               symb         => $symb,
-                                              startreserve => $startreserve,
-                                              uniqueperiod => $uniqueperiod,
+                                              startreserve => $startreserve
                                             };
             }
         }
@@ -10671,26 +9505,22 @@ sub ask_for_embedded_content {
     my $numexisting = 0;
     my $numunused = 0;
     my ($output,$upload_output,$toplevel,$url,$udom,$uname,$getpropath,$cdom,$cnum,
-        $fileloc,$filename,$delete_output,$modify_output,$title,$symb,$path,$navmap);
+        $fileloc,$filename,$delete_output,$modify_output,$title,$symb,$path);
     my $heading = &mt('Upload embedded files');
     my $buttontext = &mt('Upload');
 
+    my $navmap;
     if ($env{'request.course.id'}) {
-        if ($actionurl eq '/adm/dependencies') {
-            $navmap = Apache::lonnavmaps::navmap->new();
-        }
-        $cdom = $env{'course.'.$env{'request.course.id'}.'.domain'};
-        $cnum = $env{'course.'.$env{'request.course.id'}.'.num'};
+        $navmap = Apache::lonnavmaps::navmap->new();
     }
-    if (($actionurl eq '/adm/portfolio') ||
-        ($actionurl eq '/adm/coursegrp_portfolio')) {
+    if (($actionurl eq '/adm/portfolio') || ($actionurl eq '/adm/coursegrp_portfolio')) {
         my $current_path='/';
         if ($env{'form.currentpath'}) {
             $current_path = $env{'form.currentpath'};
         }
         if ($actionurl eq '/adm/coursegrp_portfolio') {
-            $udom = $cdom;
-            $uname = $cnum;
+            $udom = $env{'course.'.$env{'request.course.id'}.'.domain'};
+            $uname = $env{'course.'.$env{'request.course.id'}.'.num'};
             $url = '/userfiles/groups/'.$env{'form.group'}.'/portfolio';
         } else {
             $udom = $env{'user.domain'};
@@ -10714,70 +9544,43 @@ sub ask_for_embedded_content {
             $toplevel = $url;
             if ($args->{'context'} eq 'paste') {
                 ($cdom,$cnum) = ($url =~ m{^\Q/uploaded/\E($match_domain)/($match_courseid)/});
-                ($path) =
+                ($path) = 
                     ($toplevel =~ m{^(\Q/uploaded/$cdom/$cnum/\E(?:docs|supplemental)/(?:default|\d+)/\d+)/});
                 $fileloc = &Apache::lonnet::filelocation('',$toplevel);
                 $fileloc =~ s{^/}{};
             }
         }
-    } elsif ($actionurl eq '/adm/dependencies') {
+    } elsif ($actionurl eq '/adm/dependencies')  {
         if ($env{'request.course.id'} ne '') {
+            $cdom = $env{'course.'.$env{'request.course.id'}.'.domain'};
+            $cnum =  $env{'course.'.$env{'request.course.id'}.'.num'};
             if (ref($args) eq 'HASH') {
                 $url = $args->{'docs_url'};
                 $title = $args->{'docs_title'};
-                $toplevel = $url;
-                unless ($toplevel =~ m{^/}) {
-                    $toplevel = "/$url";
-                }
+                $toplevel = "/$url";
                 ($rem) = ($toplevel =~ m{^(.+/)[^/]+$});
-                if ($toplevel =~ m{^(\Q/uploaded/$cdom/$cnum/portfolio/syllabus\E)}) {
-                    $path = $1;
-                } else {
-                    ($path) =
-                        ($toplevel =~ m{^(\Q/uploaded/$cdom/$cnum/\E(?:docs|supplemental)/(?:default|\d+)/\d+)/});
-                }
-                if ($toplevel=~/^\/*(uploaded|editupload)/) {
-                    $fileloc = $toplevel;
-                    $fileloc=~ s/^\s*(\S+)\s*$/$1/;
-                    my ($udom,$uname,$fname) =
-                        ($fileloc=~ m{^/+(?:uploaded|editupload)/+($match_domain)/+($match_name)/+(.*)$});
-                    $fileloc = propath($udom,$uname).'/userfiles/'.$fname;
-                } else {
-                    $fileloc = &Apache::lonnet::filelocation('',$toplevel);
-                }
+                ($path) =  
+                    ($toplevel =~ m{^(\Q/uploaded/$cdom/$cnum/\E(?:docs|supplemental)/(?:default|\d+)/\d+)/});
+                $fileloc = &Apache::lonnet::filelocation('',$toplevel);
                 $fileloc =~ s{^/}{};
                 ($filename) = ($fileloc =~ m{.+/([^/]+)$});
                 $heading = &mt('Status of dependencies in [_1]',"$title ($filename)");
             }
         }
-    } elsif ($actionurl eq "/public/$cdom/$cnum/syllabus") {
-        $udom = $cdom;
-        $uname = $cnum;
-        $url = "/uploaded/$cdom/$cnum/portfolio/syllabus";
-        $toplevel = $url;
-        $path = $url;
-        $fileloc = &Apache::lonnet::filelocation('',$toplevel).'/';
-        $fileloc =~ s{^/}{};
-    }
-    foreach my $file (keys(%{$allfiles})) {
-        my $embed_file;
-        if (($path eq "/uploaded/$cdom/$cnum/portfolio/syllabus") && ($file =~ m{^\Q$path/\E(.+)$})) {
-            $embed_file = $1;
-        } else {
-            $embed_file = $file;
-        }
-        my ($absolutepath,$cleaned_file);
+    }
+    my $now = time();
+    foreach my $embed_file (keys(%{$allfiles})) {
+        my $absolutepath;
         if ($embed_file =~ m{^\w+://}) {
-            $cleaned_file = $embed_file;
-            $newfiles{$cleaned_file} = 1;
-            $mapping{$cleaned_file} = $embed_file;
+            $newfiles{$embed_file} = 1;
+            $mapping{$embed_file} = $embed_file;
         } else {
-            $cleaned_file = &clean_path($embed_file);
             if ($embed_file =~ m{^/}) {
                 $absolutepath = $embed_file;
+                $embed_file =~ s{^(/+)}{};
             }
-            if ($cleaned_file =~ m{/}) {
-                my ($path,$fname) = ($cleaned_file =~ m{^(.+)/([^/]*)$});
+            if ($embed_file =~ m{/}) {
+                my ($path,$fname) = ($embed_file =~ m{^(.+)/([^/]*)$});
                 $path = &check_for_traversal($path,$url,$toplevel);
                 my $item = $fname;
                 if ($path ne '') {
@@ -10794,9 +9597,9 @@ sub ask_for_embedded_content {
             } else {
                 $dependencies{$embed_file} = 1;
                 if ($absolutepath) {
-                    $mapping{$cleaned_file} = $absolutepath;
+                    $mapping{$embed_file} = $absolutepath;
                 } else {
-                    $mapping{$cleaned_file} = $embed_file;
+                    $mapping{$embed_file} = $embed_file;
                 }
             }
         }
@@ -10804,8 +9607,7 @@ sub ask_for_embedded_content {
     my $dirptr = 16384;
     foreach my $path (keys(%subdependencies)) {
         $currsubfile{$path} = {};
-        if (($actionurl eq '/adm/portfolio') ||
-            ($actionurl eq '/adm/coursegrp_portfolio')) { 
+        if (($actionurl eq '/adm/portfolio') || ($actionurl eq '/adm/coursegrp_portfolio')) { 
             my ($sublistref,$listerror) =
                 &Apache::lonnet::dirlist($url.$path,$udom,$uname,$getpropath);
             if (ref($sublistref) eq 'ARRAY') {
@@ -10821,15 +9623,9 @@ sub ask_for_embedded_content {
             }
         } elsif (($actionurl eq '/adm/dependencies') ||
                  (($actionurl eq '/adm/coursedocs') && (ref($args) eq 'HASH') &&
-                  ($args->{'context'} eq 'paste')) ||
-                 ($actionurl eq "/public/$cdom/$cnum/syllabus")) {
+                  ($args->{'context'} eq 'paste'))) {
             if ($env{'request.course.id'} ne '') {
-                my $dir;
-                if ($actionurl eq "/public/$cdom/$cnum/syllabus") {
-                    $dir = $fileloc;
-                } else {
-                    ($dir) = ($fileloc =~ m{^(.+/)[^/]+$});
-                }
+                my ($dir) = ($fileloc =~ m{^(.+/)[^/]+$});
                 if ($dir ne '') {
                     my ($sublistref,$listerror) =
                         &Apache::lonnet::dirlist($dir.$path,$cdom,$cnum,$getpropath,undef,'/');
@@ -10877,8 +9673,7 @@ sub ask_for_embedded_content {
         }
     }
     my %currfile;
-    if (($actionurl eq '/adm/portfolio') ||
-        ($actionurl eq '/adm/coursegrp_portfolio')) {
+    if (($actionurl eq '/adm/portfolio') || ($actionurl eq '/adm/coursegrp_portfolio')) {
         my ($dirlistref,$listerror) =
             &Apache::lonnet::dirlist($url,$udom,$uname,$getpropath);
         if (ref($dirlistref) eq 'ARRAY') {
@@ -10894,8 +9689,7 @@ sub ask_for_embedded_content {
         }
     } elsif (($actionurl eq '/adm/dependencies') ||
              (($actionurl eq '/adm/coursedocs') && (ref($args) eq 'HASH') &&
-              ($args->{'context'} eq 'paste')) ||
-             ($actionurl eq "/public/$cdom/$cnum/syllabus")) {
+              ($args->{'context'} eq 'paste'))) {
         if ($env{'request.course.id'} ne '') {
             my ($dir) = ($fileloc =~ m{^(.+/)[^/]+$});
             if ($dir ne '') {
@@ -10930,14 +9724,12 @@ sub ask_for_embedded_content {
                 ($file eq $filename.'.bak') ||
                 ($dependencies{$file})) {
             if ($actionurl eq '/adm/dependencies') {
-                unless ($toplevel =~ m{^\Q/uploaded/$cdom/$cnum/portfolio/syllabus\E}) {
-                    next if (($rem ne '') &&
-                             (($env{"httpref.$rem".$file} ne '') ||
-                              (ref($navmap) &&
-                              (($navmap->getResourceByUrl($rem.$file) ne '') ||
-                               (($file =~ /^(.*\.s?html?)\.bak$/i) &&
-                                ($navmap->getResourceByUrl($rem.$1)))))));
-                }
+                next if (($rem ne '') &&
+                         (($env{"httpref.$rem".$file} ne '') ||
+                          (ref($navmap) &&
+                          (($navmap->getResourceByUrl($rem.$file) ne '') ||
+                           (($file =~ /^(.*\.s?html?)\.bak$/i) &&
+                            ($navmap->getResourceByUrl($rem.$1)))))));
             }
             $unused{$file} = 1;
         }
@@ -10946,38 +9738,28 @@ sub ask_for_embedded_content {
         ($args->{'context'} eq 'paste')) {
         $counter = scalar(keys(%existing));
         $numpathchg = scalar(keys(%pathchanges));
-        return ($output,$counter,$numpathchg,\%existing);
-    } elsif (($actionurl eq "/public/$cdom/$cnum/syllabus") &&
-             (ref($args) eq 'HASH') && ($args->{'context'} eq 'rewrites')) {
-        $counter = scalar(keys(%existing));
-        $numpathchg = scalar(keys(%pathchanges));
-        return ($output,$counter,$numpathchg,\%existing,\%mapping);
+        return ($output,$counter,$numpathchg,\%existing); 
     }
     foreach my $embed_file (sort {lc($a) cmp lc($b)} keys(%newfiles)) {
         if ($actionurl eq '/adm/dependencies') {
             next if ($embed_file =~ m{^\w+://});
         }
         $upload_output .= &start_data_table_row().
-                          '<td valign="top"><img src="'.&icon($embed_file).'" />&nbsp;'.
+                          '<td><img src="'.&icon($embed_file).'" />&nbsp;'.
                           '<span class="LC_filename">'.$embed_file.'</span>';
         unless ($mapping{$embed_file} eq $embed_file) {
-            $upload_output .= '<br /><span class="LC_info" style="font-size:smaller;">'.
-                              &mt('changed from: [_1]',$mapping{$embed_file}).'</span>';
+            $upload_output .= '<br /><span class="LC_info" style="font-size:smaller;">'.&mt('changed from: [_1]',$mapping{$embed_file}).'</span>';
         }
-        $upload_output .= '</td>';
+        $upload_output .= '</td><td>';
         if ($args->{'ignore_remote_references'} && $embed_file =~ m{^\w+://}) { 
-            $upload_output.='<td align="right">'.
-                            '<span class="LC_info LC_fontsize_medium">'.
-                            &mt("URL points to web address").'</span>';
+            $upload_output.='<span class="LC_warning">'.&mt("URL points to other server.").'</span>';
             $numremref++;
         } elsif ($args->{'error_on_invalid_names'}
             && $embed_file ne &Apache::lonnet::clean_filename($embed_file,{'keep_path' => 1,})) {
-            $upload_output.='<td align="right"><span class="LC_warning">'.
-                            &mt('Invalid characters').'</span>';
+            $upload_output.='<span class="LC_warning">'.&mt('Invalid characters').'</span>';
             $numinvalid++;
         } else {
-            $upload_output .= '<td>'.
-                              &embedded_file_element('upload_embedded',$counter,
+            $upload_output .= &embedded_file_element('upload_embedded',$counter,
                                                      $embed_file,\%mapping,
                                                      $allfiles,$codebase,'upload');
             $counter ++;
@@ -11006,9 +9788,8 @@ sub ask_for_embedded_content {
             $counter ++;
         } else {
             $upload_output .= &start_data_table_row().
-                              '<td valign="top"><img src="'.&icon($embed_file).'" />&nbsp;'.
-                              '<span class="LC_filename">'.$embed_file.'</span></td>'.
-                              '<td align="right"><span class="LC_info LC_fontsize_medium">'.&mt('Already exists').'</span></td>'.
+                              '<td><span class="LC_filename">'.$embed_file.'</span></td>';
+                              '<td><span class="LC_warning">'.&mt('Already exists').'</span></td>'.
                               &Apache::loncommon::end_data_table_row()."\n";
         }
     }
@@ -11103,7 +9884,7 @@ sub ask_for_embedded_content {
         $output = '<b>'.&mt('Referenced files').'</b>:<br />';
         if ($applies > 1) {
             $output .=  
-                &mt('No dependencies need to be uploaded, as one of the following applies to each reference:').'<ul>';
+                &mt('No files need to be uploaded, as one of the following applies to each reference:').'<ul>';
             if ($numremref) {
                 $output .= '<li>'.&mt('reference is to a URL which points to another server').'</li>'."\n";
             }
@@ -11146,7 +9927,7 @@ sub ask_for_embedded_content {
             $chgcount ++;
         }
     }
-    if (($counter) || ($numunused)) {
+    if ($counter) {
         if ($numpathchg) {
             $output .= '<input type ="hidden" name="number_pathchange_items" value="'.
                        $numpathchg.'" />'."\n";
@@ -11159,57 +9940,17 @@ sub ask_for_embedded_content {
         } elsif ($actionurl eq '/adm/dependencies') {
             $output .= '<input type="hidden" name="action" value="process_changes" />';
         }
-        $output .= '<input type ="submit" value="'.$buttontext.'" />'."\n".'</form>'."\n";
+        $output .=  '<input type ="submit" value="'.$buttontext.'" />'."\n".'</form>'."\n";
     } elsif ($numpathchg) {
         my %pathchange = ();
         $output .= &modify_html_form('pathchange',$actionurl,$state,\%pathchange,$pathchange_output);
         if (($actionurl eq '/adm/portfolio') || ($actionurl eq '/adm/coursegrp_portfolio')) {
             $output .= '<p>'.&mt('or').'</p>'; 
-        }
+        } 
     }
     return ($output,$counter,$numpathchg);
 }
 
-=pod
-
-=item * clean_path($name)
-
-Performs clean-up of directories, subdirectories and filename in an
-embedded object, referenced in an HTML file which is being uploaded
-to a course or portfolio, where
-"Upload embedded images/multimedia files if HTML file" checkbox was
-checked.
-
-Clean-up is similar to replacements in lonnet::clean_filename()
-except each / between sub-directory and next level is preserved.
-
-=cut
-
-sub clean_path {
-    my ($embed_file) = @_;
-    $embed_file =~s{^/+}{};
-    my @contents;
-    if ($embed_file =~ m{/}) {
-        @contents = split(/\//,$embed_file);
-    } else {
-        @contents = ($embed_file);
-    }
-    my $lastidx = scalar(@contents)-1;
-    for (my $i=0; $i<=$lastidx; $i++) {
-        $contents[$i]=~s{\\}{/}g;
-        $contents[$i]=~s/\s+/\_/g;
-        $contents[$i]=~s{[^/\w\.\-]}{}g;
-        if ($i == $lastidx) {
-            $contents[$i]=~s/\.(\d+)(?=\.)/_$1/g;
-        }
-    }
-    if ($lastidx > 0) {
-        return join('/',@contents);
-    } else {
-        return $contents[0];
-    }
-}
-
 sub embedded_file_element {
     my ($context,$num,$embed_file,$mapping,$allfiles,$codebase,$type) = @_;
     return unless ((ref($mapping) eq 'HASH') && (ref($allfiles) eq 'HASH') &&
@@ -11334,31 +10075,28 @@ sub upload_embedded {
         # Check if extension is valid
         if (($fname =~ /\.(\w+)$/) &&
             (&Apache::loncommon::fileembstyle($1) eq 'hdn')) {
-            $output .= &mt('Invalid file extension ([_1]) - reserved for internal use.',$1)
-                      .' '.&mt('Rename the file with a different extension and re-upload.').'<br />';
+            $output .= &mt('Invalid file extension ([_1]) - reserved for LONCAPA use - rename the file with a different extension and re-upload. ',$1).'<br />';
             next;
         } elsif (($fname =~ /\.(\w+)$/) &&
                  (!defined(&Apache::loncommon::fileembstyle($1)))) {
             $output .= &mt('Unrecognized file extension ([_1]) - rename the file with a proper extension and re-upload.',$1).'<br />';
             next;
         } elsif ($fname=~/\.(\d+)\.(\w+)$/) {
-            $output .= &mt('Filename not allowed - rename the file to remove the number immediately before the file extension([_1]) and re-upload.',$2).'<br />';
+            $output .= &mt('File name not allowed - rename the file to remove the number immediately before the file extension([_1]) and re-upload.',$2).'<br />';
             next;
         }
         $env{'form.embedded_item_'.$i.'.filename'}=$fname;
-        my $subdir = $path;
-        $subdir =~ s{/+$}{};
         if ($context eq 'portfolio') {
             my $result;
             if ($state eq 'existingfile') {
                 $result=
                     &Apache::lonnet::userfileupload('embedded_item_'.$i,'existingfile',
-                                                    $dirpath.$env{'form.currentpath'}.$subdir);
+                                                    $dirpath.$env{'form.currentpath'}.$path);
             } else {
                 $result=
                     &Apache::lonnet::userfileupload('embedded_item_'.$i,'',
                                                     $dirpath.
-                                                    $env{'form.currentpath'}.$subdir);
+                                                    $env{'form.currentpath'}.$path);
                 if ($result !~ m|^/uploaded/|) {
                     $output .= '<span class="LC_error">'
                                .&mt('An error occurred ([_1]) while trying to upload [_2] for embedded element [_3].'
@@ -11370,11 +10108,10 @@ sub upload_embedded {
                                $path.$fname.'</span>').'<br />';     
                 }
             }
-        } elsif (($context eq 'coursedoc') || ($context eq 'syllabus')) {
-            my $extendedsubdir = $dirpath.'/'.$subdir;
-            $extendedsubdir =~ s{/+$}{};
+        } elsif ($context eq 'coursedoc') {
             my $result =
-                &Apache::lonnet::userfileupload('embedded_item_'.$i,$context,$extendedsubdir);
+                &Apache::lonnet::userfileupload('embedded_item_'.$i,'coursedoc',
+                                                $dirpath.'/'.$path);
             if ($result !~ m|^/uploaded/|) {
                 $output .= '<span class="LC_error">'
                            .&mt('An error occurred ([_1]) while trying to upload [_2] for embedded element [_3].'
@@ -11384,9 +10121,6 @@ sub upload_embedded {
             } else {
                 $output .= &mt('Uploaded [_1]','<span class="LC_filename">'.
                            $path.$fname.'</span>').'<br />';
-                if ($context eq 'syllabus') {
-                    &Apache::lonnet::make_public_indefinitely($result);
-                }
             }
         } else {
 # Save the file
@@ -11518,7 +10252,7 @@ sub modify_html_form {
 }
 
 sub modify_html_refs {
-    my ($context,$dirpath,$uname,$udom,$dir_root,$url) = @_;
+    my ($context,$dirpath,$uname,$udom,$dir_root) = @_;
     my $container;
     if ($context eq 'portfolio') {
         $container = $env{'form.container'};
@@ -11527,14 +10261,12 @@ sub modify_html_refs {
     } elsif ($context eq 'manage_dependencies') {
         (undef,undef,$container) = &Apache::lonnet::decode_symb($env{'form.symb'});
         $container = "/$container";
-    } elsif ($context eq 'syllabus') {
-        $container = $url;
     } else {
         $container = $Apache::lonnet::perlvar{'lonDocRoot'}.$env{'form.filename'};
     }
     my (%allfiles,%codebase,$output,$content);
     my @changes = &get_env_multiple('form.namechange');
-    unless ((@changes > 0)  || ($context eq 'syllabus')) {
+    unless (@changes > 0) {
         if (wantarray) {
             return ('',0,0); 
         } else {
@@ -11542,7 +10274,7 @@ sub modify_html_refs {
         }
     }
     if (($context eq 'portfolio') || ($context eq 'coursedoc') || 
-        ($context eq 'manage_dependencies') || ($context eq 'syllabus')) {
+        ($context eq 'manage_dependencies')) {
         unless ($container =~ m{^/uploaded/\Q$udom\E/\Q$uname\E/}) {
             if (wantarray) {
                 return ('',0,0);
@@ -11566,7 +10298,7 @@ sub modify_html_refs {
                 return;
             }
         } 
-        if (open(my $fh,'<',$container)) {
+        if (open(my $fh,"<$container")) {
             $content = join('', <$fh>);
             close($fh);
         } else {
@@ -11598,8 +10330,6 @@ sub modify_html_refs {
                     if ($content =~ m{($attrib_regexp\s*=\s*['"]?)\Q$ref\E(['"]?)}) {
                         my $numchg = ($content =~ s{($attrib_regexp\s*=\s*['"]?)\Q$ref\E(['"]?)}{$1$newname$2}gi);
                         $count += $numchg;
-                        $allfiles{$newname} = $allfiles{$ref};
-                        delete($allfiles{$ref});
                     }
                     if ($env{'form.embedded_codebase_'.$i} ne '') {
                         $codebase = &unescape($env{'form.embedded_codebase_'.$i});
@@ -11608,11 +10338,10 @@ sub modify_html_refs {
                     }
                 }
             }
-            my $skiprewrites;
             if ($count || $codebasecount) {
                 my $saveresult;
                 if (($context eq 'portfolio') || ($context eq 'coursedoc') || 
-                    ($context eq 'manage_dependencies') || ($context eq 'syllabus')) {
+                    ($context eq 'manage_dependencies')) {
                     my $url = &Apache::lonnet::store_edited_file($container,$content,$udom,$uname,\$saveresult);
                     if ($url eq $container) {
                         my ($fname) = ($container =~ m{/([^/]+)$});
@@ -11625,13 +10354,8 @@ sub modify_html_refs {
                                    '<span class="LC_filename">'.
                                    $container.'</span>').'</p>';
                     }
-                    if ($context eq 'syllabus') {
-                        unless ($saveresult eq 'ok') {
-                            $skiprewrites = 1;
-                        }
-                    }
                 } else {
-                    if (open(my $fh,'>',$container)) {
+                    if (open(my $fh,">$container")) {
                         print $fh $content;
                         close($fh);
                         $output = '<p>'.&mt('Updated [quant,_1,reference] in [_2].',
@@ -11645,47 +10369,6 @@ sub modify_html_refs {
                     }
                 }
             }
-            if (($context eq 'syllabus') && (!$skiprewrites)) {
-                my ($actionurl,$state);
-                $actionurl = "/public/$udom/$uname/syllabus";
-                my ($ignore,$num,$numpathchanges,$existing,$mapping) =
-                    &ask_for_embedded_content($actionurl,$state,\%allfiles,
-                                              \%codebase,
-                                              {'context' => 'rewrites',
-                                               'ignore_remote_references' => 1,});
-                if (ref($mapping) eq 'HASH') {
-                    my $rewrites = 0;
-                    foreach my $key (keys(%{$mapping})) {
-                        next if ($key =~ m{^https?://});
-                        my $ref = $mapping->{$key};
-                        my $newname = "/uploaded/$udom/$uname/portfolio/syllabus/$key";
-                        my $attrib;
-                        if (ref($allfiles{$mapping->{$key}}) eq 'ARRAY') {
-                            $attrib = join('|',@{$allfiles{$mapping->{$key}}});
-                        }
-                        if ($content =~ m{($attrib\s*=\s*['"]?)\Q$ref\E(['"]?)}) {
-                            my $numchg = ($content =~ s{($attrib\s*=\s*['"]?)\Q$ref\E(['"]?)}{$1$newname$2}gi);
-                            $rewrites += $numchg;
-                        }
-                    }
-                    if ($rewrites) {
-                        my $saveresult;
-                        my $url = &Apache::lonnet::store_edited_file($container,$content,$udom,$uname,\$saveresult);
-                        if ($url eq $container) {
-                            my ($fname) = ($container =~ m{/([^/]+)$});
-                            $output .= '<p>'.&mt('Rewrote [quant,_1,link] as [quant,_1,absolute link] in [_2].',
-                                            $count,'<span class="LC_filename">'.
-                                            $fname.'</span>').'</p>';
-                        } else {
-                            $output .= '<p class="LC_error">'.
-                                       &mt('Error: could not update links in [_1].',
-                                       '<span class="LC_filename">'.
-                                       $container.'</span>').'</p>';
-
-                        }
-                    }
-                }
-            }
         } else {
             &logthis('Failed to parse '.$container.
                      ' to modify references: '.$parse_result);
@@ -11775,11 +10458,11 @@ sub check_for_upload {
                     if ($currsize < $filesize) {
                         my $extra = $filesize - $currsize;
                         if (($current_disk_usage + $extra) > $disk_quota) {
-                            my $msg = '<p class="LC_warning">'.
+                            my $msg = '<span class="LC_error">'.
                                       &mt('Unable to upload [_1]. (size = [_2] kilobytes). Disk quota will be exceeded if existing (smaller) file with same name (size = [_3] kilobytes) is replaced.',
-                                          '<span class="LC_filename">'.$fname.'</span>',$filesize,$currsize).'</p>'.
-                                      '<p>'.&mt('Disk quota is [_1] kilobytes. Your current disk usage is [_2] kilobytes.',
-                                                   $disk_quota,$current_disk_usage).'</p>';
+                                          '<span class="LC_filename">'.$fname.'</span>',$filesize,$currsize).'</span>'.
+                                      '<br />'.&mt('Disk quota is [_1] kilobytes. Your current disk usage is [_2] kilobytes.',
+                                                   $disk_quota,$current_disk_usage);
                             return ('will_exceed_quota',$msg);
                         }
                     }
@@ -11788,21 +10471,21 @@ sub check_for_upload {
         }
     }
     if (($current_disk_usage + $filesize) > $disk_quota){
-        my $msg = '<p class="LC_warning">'.
-                &mt('Unable to upload [_1]. (size = [_2] kilobytes). Disk quota will be exceeded.','<span class="LC_filename">'.$fname.'</span>',$filesize).'</p>'.
-                  '<p>'.&mt('Disk quota is [_1] kilobytes. Your current disk usage is [_2] kilobytes.',$disk_quota,$current_disk_usage).'</p>';
+        my $msg = '<span class="LC_error">'.
+                &mt('Unable to upload [_1]. (size = [_2] kilobytes). Disk quota will be exceeded.','<span class="LC_filename">'.$fname.'</span>',$filesize).'</span>'.
+                  '<br />'.&mt('Disk quota is [_1] kilobytes. Your current disk usage is [_2] kilobytes.',$disk_quota,$current_disk_usage);
         return ('will_exceed_quota',$msg);
     } elsif ($found_file) {
         if ($locked_file) {
-            my $msg = '<p class="LC_warning">';
+            my $msg = '<span class="LC_error">';
             $msg .= &mt('Unable to upload [_1]. A locked file by that name was found in [_2].','<span class="LC_filename">'.$fname.'</span>','<span class="LC_filename">'.$port_path.$env{'form.currentpath'}.'</span>');
-            $msg .= '</p>';
+            $msg .= '</span><br />';
             $msg .= &mt('You will be able to rename or delete existing [_1] after a grade has been assigned.','<span class="LC_filename">'.$fname.'</span>');
             return ('file_locked',$msg);
         } else {
-            my $msg = '<p class="LC_error">';
+            my $msg = '<span class="LC_error">';
             $msg .= &mt(' A file by that name: [_1] was found in [_2].','<span class="LC_filename">'.$fname.'</span>',$port_path.$env{'form.currentpath'});
-            $msg .= '</p>';
+            $msg .= '</span>';
             return ('existingfile',$msg);
         }
     }
@@ -11893,66 +10576,16 @@ sub decompress_form {
         }
     }
     if ($mimetype =~ m{^application/(x\-)?(compressed|zip)}) {
-        my @camtasia6 = ("$topdir/","$topdir/index.html",
+        my @camtasia = ("$topdir/","$topdir/index.html",
                         "$topdir/media/",
                         "$topdir/media/$topdir.mp4",
                         "$topdir/media/FirstFrame.png",
                         "$topdir/media/player.swf",
                         "$topdir/media/swfobject.js",
                         "$topdir/media/expressInstall.swf");
-        my @camtasia8_1 = ("$topdir/","$topdir/$topdir.html",
-                         "$topdir/$topdir.mp4",
-                         "$topdir/$topdir\_config.xml",
-                         "$topdir/$topdir\_controller.swf",
-                         "$topdir/$topdir\_embed.css",
-                         "$topdir/$topdir\_First_Frame.png",
-                         "$topdir/$topdir\_player.html",
-                         "$topdir/$topdir\_Thumbnails.png",
-                         "$topdir/playerProductInstall.swf",
-                         "$topdir/scripts/",
-                         "$topdir/scripts/config_xml.js",
-                         "$topdir/scripts/handlebars.js",
-                         "$topdir/scripts/jquery-1.7.1.min.js",
-                         "$topdir/scripts/jquery-ui-1.8.15.custom.min.js",
-                         "$topdir/scripts/modernizr.js",
-                         "$topdir/scripts/player-min.js",
-                         "$topdir/scripts/swfobject.js",
-                         "$topdir/skins/",
-                         "$topdir/skins/configuration_express.xml",
-                         "$topdir/skins/express_show/",
-                         "$topdir/skins/express_show/player-min.css",
-                         "$topdir/skins/express_show/spritesheet.png");
-        my @camtasia8_4 = ("$topdir/","$topdir/$topdir.html",
-                         "$topdir/$topdir.mp4",
-                         "$topdir/$topdir\_config.xml",
-                         "$topdir/$topdir\_controller.swf",
-                         "$topdir/$topdir\_embed.css",
-                         "$topdir/$topdir\_First_Frame.png",
-                         "$topdir/$topdir\_player.html",
-                         "$topdir/$topdir\_Thumbnails.png",
-                         "$topdir/playerProductInstall.swf",
-                         "$topdir/scripts/",
-                         "$topdir/scripts/config_xml.js",
-                         "$topdir/scripts/techsmith-smart-player.min.js",
-                         "$topdir/skins/",
-                         "$topdir/skins/configuration_express.xml",
-                         "$topdir/skins/express_show/",
-                         "$topdir/skins/express_show/spritesheet.min.css",
-                         "$topdir/skins/express_show/spritesheet.png",
-                         "$topdir/skins/express_show/techsmith-smart-player.min.css");
-        my @diffs = &compare_arrays(\@paths,\@camtasia6);
+        my @diffs = &compare_arrays(\@paths,\@camtasia);
         if (@diffs == 0) {
-            $is_camtasia = 6;
-        } else {
-            @diffs = &compare_arrays(\@paths,\@camtasia8_1);
-            if (@diffs == 0) {
-                $is_camtasia = 8;
-            } else {
-                @diffs = &compare_arrays(\@paths,\@camtasia8_4);
-                if (@diffs == 0) {
-                    $is_camtasia = 8;
-                }
-            }
+            $is_camtasia = 1;
         }
     }
     my $output;
@@ -11964,7 +10597,8 @@ sub decompress_form {
 function camtasiaToggle() {
     for (var i=0; i<document.uploaded_decompress.autoextract_camtasia.length; i++) {
         if (document.uploaded_decompress.autoextract_camtasia[i].checked) {
-            if (document.uploaded_decompress.autoextract_camtasia[i].value == $is_camtasia) {
+            if (document.uploaded_decompress.autoextract_camtasia[i].value == 1) {
+
                 document.getElementById('camtasia_titles').style.display='block';
             } else {
                 document.getElementById('camtasia_titles').style.display='none';
@@ -12026,7 +10660,7 @@ ENDCAM
     if ($is_camtasia) {
         $output .= $lt{'auto'}.'<br />'.
                    '<span class="LC_nobreak">'.$lt{'proa'}.'<label>'.
-                   '<input type="radio" name="autoextract_camtasia" value="'.$is_camtasia.'" onclick="javascript:camtasiaToggle();" checked="checked" />'.
+                   '<input type="radio" name="autoextract_camtasia" value="1" onclick="javascript:camtasiaToggle();" checked="checked" />'.
                    $lt{'yes'}.'</label>&nbsp;<label>'.
                    '<input type="radio" name="autoextract_camtasia" value="0" onclick="javascript:camtasiaToggle();" />'.
                    $lt{'no'}.'</label></span><br />'.
@@ -12148,22 +10782,10 @@ sub decompress_uploaded_file {
 
 sub process_decompression {
     my ($docudom,$docuname,$file,$destination,$dir_root,$hiddenelem) = @_;
-    unless (($dir_root eq '/userfiles') && ($destination =~ m{^(docs|supplemental)/(default|\d+)/\d+$})) {
-        return '<p class="LC_error">'.&mt('Not extracted.').'<br />'.
-               &mt('Unexpected file path.').'</p>'."\n";
-    }
-    unless (($docudom =~ /^$match_domain$/) && ($docuname =~ /^$match_courseid$/)) {
-        return '<p class="LC_error">'.&mt('Not extracted.').'<br />'.
-               &mt('Unexpected course context.').'</p>'."\n";
-    }
-    unless ($file eq &Apache::lonnet::clean_filename($file)) {
-        return '<p class="LC_error">'.&mt('Not extracted.').'<br />'.
-               &mt('Filename contained unexpected characters.').'</p>'."\n";
-    }
     my ($dir,$error,$warning,$output);
-    if ($file !~ /\.(zip|tar|bz2|gz|tar.gz|tar.bz2|tgz)$/i) {
-        $error = &mt('Filename not a supported archive file type.').
-                 '<br />'.&mt('Filename should end with one of: [_1].',
+    if ($file !~ /\.(zip|tar|bz2|gz|tar.gz|tar.bz2|tgz)$/) {
+        $error = &mt('File name not a supported archive file type.').
+                 '<br />'.&mt('File name should end with one of: [_1].',
                               '.zip, .tar, .bz2, .gz, .tar.gz, .tar.bz2, .tgz');
     } else {
         my $docuhome = &Apache::lonnet::homeserver($docuname,$docudom);
@@ -12194,44 +10816,30 @@ sub process_decompression {
                 }
             }
             my $numskip = scalar(@to_skip);
-            my $numoverwrite = scalar(@to_overwrite);
-            if (($numskip) && (!$numoverwrite)) {
+            if (($numskip > 0) && 
+                ($numskip == $env{'form.archive_itemcount'})) {
                 $warning = &mt('All items in the archive file already exist, and no overwriting of existing files has been requested.');         
             } elsif ($dir eq '') {
                 $error = &mt('Directory containing archive file unavailable.');
             } elsif (!$error) {
                 my ($decompressed,$display);
-                if (($numskip) || ($numoverwrite)) {
+                if ($numskip > 0) {
                     my $tempdir = time.'_'.$$.int(rand(10000));
                     mkdir("$dir/$tempdir",0755);
-                    if (&File::Copy::move("$dir/$file","$dir/$tempdir/$file")) {
-                        ($decompressed,$display) =
-                            &decompress_uploaded_file($file,"$dir/$tempdir");
-                        foreach my $item (@to_skip) {
-                            if (($item ne '') && ($item !~ /\.\./)) {
-                                if (-f "$dir/$tempdir/$item") {
-                                    unlink("$dir/$tempdir/$item");
-                                } elsif (-d "$dir/$tempdir/$item") {
-                                    &File::Path::remove_tree("$dir/$tempdir/$item",{ safe => 1 });
-                                }
-                            }
-                        }
-                        foreach my $item (@to_overwrite) {
-                            if ((-e "$dir/$tempdir/$item") && (-e "$dir/$item")) {
-                                if (($item ne '') && ($item !~ /\.\./)) {
-                                    if (-f "$dir/$item") {
-                                        unlink("$dir/$item");
-                                    } elsif (-d "$dir/$item") {
-                                        &File::Path::remove_tree("$dir/$item",{ safe => 1 });
-                                    }
-                                    &File::Copy::move("$dir/$tempdir/$item","$dir/$item");
-                                }
+                    system("mv $dir/$file $dir/$tempdir/$file");
+                    ($decompressed,$display) = 
+                        &decompress_uploaded_file($file,"$dir/$tempdir");
+                    foreach my $item (@to_skip) {
+                        if (($item ne '') && ($item !~ /\.\./)) {
+                            if (-f "$dir/$tempdir/$item") { 
+                                unlink("$dir/$tempdir/$item");
+                            } elsif (-d "$dir/$tempdir/$item") {
+                                system("rm -rf $dir/$tempdir/$item");
                             }
                         }
-                        if (&File::Copy::move("$dir/$tempdir/$file","$dir/$file")) {
-                            &File::Path::remove_tree("$dir/$tempdir",{ safe => 1 });
-                        }
                     }
+                    system("mv $dir/$tempdir/* $dir");
+                    rmdir("$dir/$tempdir");   
                 } else {
                     ($decompressed,$display) = 
                         &decompress_uploaded_file($file,$dir);
@@ -12249,7 +10857,8 @@ sub process_decompression {
                     if (ref($newdirlistref) eq 'ARRAY') {
                         foreach my $dir_line (@{$newdirlistref}) {
                             my ($item,undef,undef,$testdir)=split(/\&/,$dir_line,5);
-                            unless (($item =~ /^\.+$/) || ($item eq $file)) { 
+                            unless (($item =~ /^\.+$/) || ($item eq $file) || 
+                                    ((@to_skip > 0) && (grep(/^\Q$item\E$/,@to_skip)))) {
                                 push(@newitems,$item);
                                 if ($dirptr&$testdir) {
                                     $is_dir{$item} = 1;
@@ -12284,7 +10893,6 @@ sub process_decompression {
                                                            \%titles,\%children);
                         }
                         if ($env{'form.autoextract_camtasia'}) {
-                            my $version = $env{'form.autoextract_camtasia'};
                             my %displayed;
                             my $total = 1;
                             $env{'form.archive_directory'} = [];
@@ -12303,15 +10911,12 @@ sub process_decompression {
                                     $env{'form.archive_'.$i} = 'display';
                                     $env{'form.archive_title_'.$i} = $env{'form.camtasia_foldername'};
                                     $displayed{'folder'} = $i;
-                                } elsif ((($item eq "$contents[0]/index.html") && ($version == 6)) ||
-                                         (($item eq "$contents[0]/$contents[0]".'.html') && ($version == 8))) {
+                                } elsif ($item eq "$contents[0]/index.html") {
                                     $env{'form.archive_'.$i} = 'display';
                                     $env{'form.archive_title_'.$i} = $env{'form.camtasia_moviename'};
                                     $displayed{'web'} = $i;
                                 } else {
-                                    if ((($item eq "$contents[0]/media") && ($version == 6)) ||
-                                        ((($item eq "$contents[0]/scripts") || ($item eq "$contents[0]/skins") ||
-                                             ($item eq "$contents[0]/skins/express_show")) && ($version == 8))) {
+                                    if ($item eq "$contents[0]/media") {
                                         push(@{$env{'form.archive_directory'}},$i);
                                     }
                                     $env{'form.archive_'.$i} = 'dependency';
@@ -12734,7 +11339,7 @@ END
 sub process_extracted_files {
     my ($context,$docudom,$docuname,$destination,$dir_root,$hiddenelem) = @_;
     my $numitems = $env{'form.archive_count'};
-    return if ((!$numitems) || ($numitems =~ /\D/));
+    return unless ($numitems);
     my @ids=&Apache::lonnet::current_machine_ids();
     my ($prefix,$pathtocheck,$dir,$ishome,$error,$warning,%toplevelitems,%is_dir,
         %folders,%containers,%mapinner,%prompttofetch);
@@ -12747,7 +11352,7 @@ sub process_extracted_files {
     } else {
         $prefix = $Apache::lonnet::perlvar{'lonDocRoot'};
         $pathtocheck = "$dir_root/$docudom/$docuname/$destination";
-        $dir = "$dir_root/$docudom/$docuname";
+        $dir = "$dir_root/$docudom/$docuname";    
     }
     my $currdir = "$dir_root/$destination";
     (my $docstype,$mapinner{'0'}) = ($destination =~ m{^(docs|supplemental)/(\w+)/});
@@ -12756,7 +11361,7 @@ sub process_extracted_files {
         $folders{'0'} = $items[-2];
         if ($env{'form.folderpath'} =~ /\:1$/) {
             $containers{'0'}='page';
-        } else {
+        } else {  
             $containers{'0'}='sequence';
         }
     }
@@ -12836,9 +11441,7 @@ sub process_extracted_files {
                                                         '.'.$containers{$outer},1,1);
                             $newseqid{$i} = $newidx;
                             unless ($errtext) {
-                                $result .=  '<li>'.&mt('Folder: [_1] added to course',
-                                                       &HTML::Entities::encode($docstitle,'<>&"'))..
-                                            '</li>'."\n";
+                                $result .=  '<li>'.&mt('Folder: [_1] added to course',$docstitle).'</li>'."\n";
                             }
                         }
                     } else {
@@ -12847,47 +11450,38 @@ sub process_extracted_files {
                             my $url = '/uploaded/'.$docudom.'/'.$docuname.'/'.
                                       $docstype.'/'.$mapinner{$outer}.'/'.$newidx.'/'.
                                       $title;
-                            if (($outer !~ /\D/) && ($mapinner{$outer} !~ /\D/) && ($newidx !~ /\D/)) {
-                                if (!-e "$prefix$dir/$docstype/$mapinner{$outer}") {
-                                    mkdir("$prefix$dir/$docstype/$mapinner{$outer}",0755);
-                                }
-                                if (!-e "$prefix$dir/$docstype/$mapinner{$outer}/$newidx") {
-                                    mkdir("$prefix$dir/$docstype/$mapinner{$outer}/$newidx");
-                                }
-                                if (-e "$prefix$dir/$docstype/$mapinner{$outer}/$newidx") {
-                                    if (rename("$prefix$path","$prefix$dir/$docstype/$mapinner{$outer}/$newidx/$title")) {
-                                        $newdest{$i} = "$prefix$dir/$docstype/$mapinner{$outer}/$newidx";
-                                        unless ($ishome) {
-                                            my $fetch = "$newdest{$i}/$title";
-                                            $fetch =~ s/^\Q$prefix$dir\E//;
-                                            $prompttofetch{$fetch} = 1;
-                                        }
-                                   }
+                            if (!-e "$prefix$dir/$docstype/$mapinner{$outer}") {
+                                mkdir("$prefix$dir/$docstype/$mapinner{$outer}",0755);
+                            }
+                            if (!-e "$prefix$dir/$docstype/$mapinner{$outer}/$newidx") {
+                                mkdir("$prefix$dir/$docstype/$mapinner{$outer}/$newidx");
+                            }
+                            if (-e "$prefix$dir/$docstype/$mapinner{$outer}/$newidx") {
+                                system("mv $prefix$path $prefix$dir/$docstype/$mapinner{$outer}/$newidx/$title");
+                                $newdest{$i} = "$prefix$dir/$docstype/$mapinner{$outer}/$newidx";
+                                unless ($ishome) {
+                                    my $fetch = "$newdest{$i}/$title";
+                                    $fetch =~ s/^\Q$prefix$dir\E//;
+                                    $prompttofetch{$fetch} = 1;
                                 }
-                                $LONCAPA::map::resources[$newidx]=
-                                    $docstitle.':'.$url.':false:normal:res';
-                                push(@LONCAPA::map::order, $newidx);
-                                my ($outtext,$errtext)=
-                                    &LONCAPA::map::storemap('/uploaded/'.$docudom.'/'.
-                                                            $docuname.'/'.$folders{$outer}.
-                                                            '.'.$containers{$outer},1,1);
-                                unless ($errtext) {
-                                    if (-e "$prefix$dir/$docstype/$mapinner{$outer}/$newidx/$title") {
-                                        $result .= '<li>'.&mt('File: [_1] added to course',
-                                                              &HTML::Entities::encode($docstitle,'<>&"')).
-                                                   '</li>'."\n";
-                                    }
+                            }
+                            $LONCAPA::map::resources[$newidx]=
+                                $docstitle.':'.$url.':false:normal:res';
+                            push(@LONCAPA::map::order, $newidx);
+                            my ($outtext,$errtext)=
+                                &LONCAPA::map::storemap('/uploaded/'.$docudom.'/'.
+                                                        $docuname.'/'.$folders{$outer}.
+                                                        '.'.$containers{$outer},1,1);
+                            unless ($errtext) {
+                                if (-e "$prefix$dir/$docstype/$mapinner{$outer}/$newidx/$title") {
+                                    $result .= '<li>'.&mt('File: [_1] added to course',$docstitle).'</li>'."\n";
                                 }
-                            } else {
-                                $warning .= &mt('Item extracted from archive: [_1] has unexpected path.',
-                                                &HTML::Entities::encode($path,'<>&"')).'<br />';
                             }
                         }
                     }
                 }
             } else {
-                $warning .= &mt('Item extracted from archive: [_1] has unexpected path.',
-                                &HTML::Entities::encode($path,'<>&"')).'<br />';
+                $warning .= &mt('Item extracted from archive: [_1] has unexpected path.',$path).'<br />'; 
             }
         }
         for (my $i=1; $i<=$numitems; $i++) {
@@ -12909,7 +11503,7 @@ sub process_extracted_files {
                         }
                         if ($itemidx eq '') {
                             $itemidx =  0;
-                        }
+                        } 
                         if (grep(/^\Q$referrer{$i}\E$/,@archdirs)) {
                             if ($mapinner{$referrer{$i}}) {
                                 $fullpath = "$prefix$dir/$docstype/$mapinner{$referrer{$i}}";
@@ -12948,9 +11542,7 @@ sub process_extracted_files {
                         }
                         if ($fullpath ne '') {
                             if (-e "$prefix$path") {
-                                unless (rename("$prefix$path","$fullpath/$title")) {
-                                     $warning .= &mt('Failed to rename dependency').'<br />';
-                                }
+                                system("mv $prefix$path $fullpath/$title");
                             }
                             if (-e "$fullpath/$title") {
                                 my $showpath;
@@ -12958,27 +11550,22 @@ sub process_extracted_files {
                                     $showpath = "$relpath/$title";
                                 } else {
                                     $showpath = "/$title";
-                                }
-                                $result .= '<li>'.&mt('[_1] included as a dependency',
-                                                      &HTML::Entities::encode($showpath,'<>&"')).
-                                           '</li>'."\n";
-                                unless ($ishome) {
-                                    my $fetch = "$fullpath/$title";
-                                    $fetch =~ s/^\Q$prefix$dir\E//;
-                                    $prompttofetch{$fetch} = 1;
-                                }
+                                } 
+                                $result .= '<li>'.&mt('[_1] included as a dependency',$showpath).'</li>'."\n";
+                            } 
+                            unless ($ishome) {
+                                my $fetch = "$fullpath/$title";
+                                $fetch =~ s/^\Q$prefix$dir\E//; 
+                                $prompttofetch{$fetch} = 1;
                             }
                         }
                     }
                 } elsif ($env{'form.archive_'.$referrer{$i}} eq 'discard') {
                     $warning .= &mt('[_1] is a dependency of [_2], which was discarded.',
-                                    &HTML::Entities::encode($path,'<>&"'),
-                                    &HTML::Entities::encode($env{'form.archive_content_'.$referrer{$i}},'<>&"')).
-                                '<br />';
+                                    $path,$env{'form.archive_content_'.$referrer{$i}}).'<br />';
                 }
             } else {
-                $warning .= &mt('Item extracted from archive: [_1] has unexpected path.',
-                                &HTML::Entities::encode($path)).'<br />';
+                $warning .= &mt('Item extracted from archive: [_1] has unexpected path.',$path).'<br />'; 
             }
         }
         if (keys(%todelete)) {
@@ -13037,7 +11624,7 @@ sub cleanup_empty_dirs {
             my $numitems = 0;
             foreach my $item (@dircontents) {
                 if (-d "$path/$item") {
-                    &cleanup_empty_dirs("$path/$item");
+                    &recurse_dirs("$path/$item");
                     if (-e "$path/$item") {
                         $numitems ++;
                     }
@@ -13056,7 +11643,7 @@ sub cleanup_empty_dirs {
 
 =pod
 
-=item * &get_folder_hierarchy()
+=item &get_folder_hierarchy()
 
 Provides hierarchy of names of folders/sub-folders containing the current
 item,
@@ -13084,7 +11671,7 @@ sub get_folder_hierarchy {
                 my @pcs = split(/,/,$pcslist);
                 foreach my $pc (@pcs) {
                     if ($pc == 1) {
-                        push(@pathitems,&mt('Main Content'));
+                        push(@pathitems,&mt('Main Course Documents'));
                     } else {
                         my $res = $navmap->getByMapPc($pc);
                         if (ref($res)) {
@@ -13099,7 +11686,7 @@ sub get_folder_hierarchy {
             }
             if ($showitem) {
                 if ($mapres->{ID} eq '0.0') {
-                    push(@pathitems,&mt('Main Content'));
+                    push(@pathitems,&mt('Main Course Documents'));
                 } else {
                     my $maptitle = $mapres->compTitle();
                     $maptitle =~ s/\W+/_/g;
@@ -13166,9 +11753,6 @@ sub get_turnedin_filepath {
                             my $title = $res->compTitle();
                             $title =~ s/\W+/_/g;
                             if ($title ne '') {
-                                if (($pc > 1) && (length($title) > 12)) {
-                                    $title = substr($title,0,12);
-                                }
                                 push(@pathitems,$title);
                             }
                         }
@@ -13177,9 +11761,6 @@ sub get_turnedin_filepath {
                 my $maptitle = $mapres->compTitle();
                 $maptitle =~ s/\W+/_/g;
                 if ($maptitle ne '') {
-                    if (length($maptitle) > 12) {
-                        $maptitle = substr($maptitle,0,12);
-                    }
                     push(@pathitems,$maptitle);
                 }
                 unless ($env{'request.state'} eq 'construct') {
@@ -13220,9 +11801,6 @@ sub get_turnedin_filepath {
                 $restitle = time;
             }
         }
-        if (length($restitle) > 12) {
-            $restitle = substr($restitle,0,12);
-        }
         push(@pathitems,$restitle);
         $path .= join('/',@pathitems);
     }
@@ -13252,15 +11830,12 @@ sub upfile_store {
     $env{'form.upfile'}=~s/\n+/\n/gs;
     $env{'form.upfile'}=~s/\n+$//gs;
 
-    my $datatoken = &valid_datatoken($env{'user.name'}.'_'.$env{'user.domain'}.
-                                     '_enroll_'.$env{'request.course.id'}.'_'.
-                                     time.'_'.$$);
-    return if ($datatoken eq '');
-
+    my $datatoken=$env{'user.name'}.'_'.$env{'user.domain'}.
+	'_enroll_'.$env{'request.course.id'}.'_'.time.'_'.$$;
     {
         my $datafile = $r->dir_config('lonDaemons').
                            '/tmp/'.$datatoken.'.tmp';
-        if ( open(my $fh,'>',$datafile) ) {
+        if ( open(my $fh,">$datafile") ) {
             print $fh $env{'form.upfile'};
             close($fh);
         }
@@ -13270,22 +11845,21 @@ sub upfile_store {
 
 =pod
 
-=item * &load_tmp_file($r,$datatoken)
+=item * &load_tmp_file($r)
 
 Load uploaded file from tmp, $r should be the HTTP Request object,
-$datatoken is the name to assign to the temporary file.
+needs $env{'form.datatoken'},
 sets $env{'form.upfile'} to the contents of the file
 
 =cut
 
 sub load_tmp_file {
-    my ($r,$datatoken) = @_;
-    return if ($datatoken eq '');
+    my $r=shift;
     my @studentdata=();
     {
         my $studentfile = $r->dir_config('lonDaemons').
-                              '/tmp/'.$datatoken.'.tmp';
-        if ( open(my $fh,'<',$studentfile) ) {
+                              '/tmp/'.$env{'form.datatoken'}.'.tmp';
+        if ( open(my $fh,"<$studentfile") ) {
             @studentdata=<$fh>;
             close($fh);
         }
@@ -13293,14 +11867,6 @@ sub load_tmp_file {
     $env{'form.upfile'}=join('',@studentdata);
 }
 
-sub valid_datatoken {
-    my ($datatoken) = @_;
-    if ($datatoken =~ /^$match_username\_$match_domain\_enroll_(|$match_domain\_$match_courseid)\_\d+_\d+$/) {
-        return $datatoken;
-    }
-    return;
-}
-
 =pod
 
 =item * &upfile_record_sep()
@@ -13741,7 +12307,7 @@ sub DrawBarGraph {
         @Labels = @$labels;
     } else {
         for (my $i=0;$i<@{$Values[0]};$i++) {
-            push(@Labels,$i+1);
+            push (@Labels,$i+1);
         }
     }
     #
@@ -14172,28 +12738,18 @@ sub restore_settings {
 
 =item * &build_recipient_list()
 
-Build recipient lists for following types of e-mail:
+Build recipient lists for five types of e-mail:
 (a) Error Reports, (b) Package Updates, (c) lonstatus warnings/errors
-(d) Help requests, (e) Course requests needing approval, (f) loncapa
-module change checking, student/employee ID conflict checks, as
-generated by lonerrorhandler.pm, CHECKRPMS, loncron,
-lonsupportreq.pm, loncoursequeueadmin.pm, searchcat.pl respectively.
+(d) Help requests, (e) Course requests needing approval,  generated by
+lonerrorhandler.pm, CHECKRPMS, loncron, lonsupportreq.pm and
+loncoursequeueadmin.pm respectively.
 
 Inputs:
-defmail (scalar - email address of default recipient),
-mailing type (scalar: errormail, packagesmail, helpdeskmail,
-requestsmail, updatesmail, or idconflictsmail).
-
+defmail (scalar - email address of default recipient), 
+mailing type (scalar - errormail, packagesmail, or helpdeskmail), 
 defdom (domain for which to retrieve configuration settings),
-
-origmail (scalar - email address of recipient from loncapa.conf,
-i.e., predates configuration by DC via domainprefs.pm
-
-$requname username of requester (if mailing type is helpdeskmail)
-
-$requdom domain of requester (if mailing type is helpdeskmail)
-
-$reqemail e-mail address of requester (if mailing type is helpdeskmail)
+origmail (scalar - email address of recipient from loncapa.conf, 
+i.e., predates configuration by DC via domainprefs.pm 
 
 Returns: comma separated list of addresses to which to send e-mail.
 
@@ -14204,11 +12760,11 @@ Returns: comma separated list of address
 ############################################################
 ############################################################
 sub build_recipient_list {
-    my ($defmail,$mailing,$defdom,$origmail,$requname,$requdom,$reqemail) = @_;
+    my ($defmail,$mailing,$defdom,$origmail) = @_;
     my @recipients;
-    my ($otheremails,$lastresort,$allbcc,$addtext);
+    my $otheremails;
     my %domconfig =
-        &Apache::lonnet::get_dom('configuration',['contacts'],$defdom);
+         &Apache::lonnet::get_dom('configuration',['contacts'],$defdom);
     if (ref($domconfig{'contacts'}) eq 'HASH') {
         if (exists($domconfig{'contacts'}{$mailing})) {
             if (ref($domconfig{'contacts'}{$mailing}) eq 'HASH') {
@@ -14220,183 +12776,14 @@ sub build_recipient_list {
                             push(@recipients,$addr);
                         }
                     }
-                }
-                $otheremails = $domconfig{'contacts'}{$mailing}{'others'};
-                if ($mailing eq 'helpdeskmail') {
-                    if ($domconfig{'contacts'}{$mailing}{'bcc'}) {
-                        my @bccs = split(/,/,$domconfig{'contacts'}{$mailing}{'bcc'});
-                        my @ok_bccs;
-                        foreach my $bcc (@bccs) {
-                            $bcc =~ s/^\s+//g;
-                            $bcc =~ s/\s+$//g;
-                            if ($bcc =~ m/^[^\@]+\@[^\@]+$/) {
-                                if (!(grep(/^\Q$bcc\E$/,@ok_bccs))) {
-                                    push(@ok_bccs,$bcc);
-                                }
-                            }
-                        }
-                        if (@ok_bccs > 0) {
-                            $allbcc = join(', ',@ok_bccs);
-                        }
-                    }
-                    $addtext = $domconfig{'contacts'}{$mailing}{'include'};
+                    $otheremails = $domconfig{'contacts'}{$mailing}{'others'};
                 }
             }
         } elsif ($origmail ne '') {
-            $lastresort = $origmail;
-        }
-        if ($mailing eq 'helpdeskmail') {
-            if ((ref($domconfig{'contacts'}{'overrides'}) eq 'HASH') &&
-                (keys(%{$domconfig{'contacts'}{'overrides'}}))) {
-                my ($inststatus,$inststatus_checked);
-                if (($env{'user.name'} ne '') && ($env{'user.domain'} ne '') &&
-                    ($env{'user.domain'} ne 'public')) {
-                    $inststatus_checked = 1;
-                    $inststatus = $env{'environment.inststatus'};
-                }
-                unless ($inststatus_checked) {
-                    if (($requname ne '') && ($requdom ne '')) {
-                        if (($requname =~ /^$match_username$/) &&
-                            ($requdom =~ /^$match_domain$/) &&
-                            (&Apache::lonnet::domain($requdom))) {
-                            my $requhome = &Apache::lonnet::homeserver($requname,
-                                                                      $requdom);
-                            unless ($requhome eq 'no_host') {
-                                my %userenv = &Apache::lonnet::userenvironment($requdom,$requname,'inststatus');
-                                $inststatus = $userenv{'inststatus'};
-                                $inststatus_checked = 1;
-                            }
-                        }
-                    }
-                }
-                unless ($inststatus_checked) {
-                    if ($reqemail =~ /^[^\@]+\@[^\@]+$/) {
-                        my %srch = (srchby     => 'email',
-                                    srchdomain => $defdom,
-                                    srchterm   => $reqemail,
-                                    srchtype   => 'exact');
-                        my %srch_results = &Apache::lonnet::usersearch(\%srch);
-                        foreach my $uname (keys(%srch_results)) {
-                            if (ref($srch_results{$uname}{'inststatus'}) eq 'ARRAY') {
-                                $inststatus = join(',',@{$srch_results{$uname}{'inststatus'}});
-                                $inststatus_checked = 1;
-                                last;
-                            }
-                        }
-                        unless ($inststatus_checked) {
-                            my ($dirsrchres,%srch_results) = &Apache::lonnet::inst_directory_query(\%srch);
-                            if ($dirsrchres eq 'ok') {
-                                foreach my $uname (keys(%srch_results)) {
-                                    if (ref($srch_results{$uname}{'inststatus'}) eq 'ARRAY') {
-                                        $inststatus = join(',',@{$srch_results{$uname}{'inststatus'}});
-                                        $inststatus_checked = 1;
-                                        last;
-                                    }
-                                }
-                            }
-                        }
-                    }
-                }
-                if ($inststatus ne '') {
-                    foreach my $status (split(/\:/,$inststatus)) {
-                        if (ref($domconfig{'contacts'}{'overrides'}{$status}) eq 'HASH') {
-                            my @contacts = ('adminemail','supportemail');
-                            foreach my $item (@contacts) {
-                                if ($domconfig{'contacts'}{'overrides'}{$status}{$item}) {
-                                    my $addr = $domconfig{'contacts'}{'overrides'}{$status};
-                                    if (!grep(/^\Q$addr\E$/,@recipients)) {
-                                        push(@recipients,$addr);
-                                    }
-                                }
-                            }
-                            $otheremails = $domconfig{'contacts'}{'overrides'}{$status}{'others'};
-                            if ($domconfig{'contacts'}{'overrides'}{$status}{'bcc'}) {
-                                my @bccs = split(/,/,$domconfig{'contacts'}{'overrides'}{$status}{'bcc'});
-                                my @ok_bccs;
-                                foreach my $bcc (@bccs) {
-                                    $bcc =~ s/^\s+//g;
-                                    $bcc =~ s/\s+$//g;
-                                    if ($bcc =~ m/^[^\@]+\@[^\@]+$/) {
-                                        if (!(grep(/^\Q$bcc\E$/,@ok_bccs))) {
-                                            push(@ok_bccs,$bcc);
-                                        }
-                                    }
-                                }
-                                if (@ok_bccs > 0) {
-                                    $allbcc = join(', ',@ok_bccs);
-                                }
-                            }
-                            $addtext = $domconfig{'contacts'}{'overrides'}{$status}{'include'};
-                            last;
-                        }
-                    }
-                }
-            }
+            push(@recipients,$origmail);
         }
     } elsif ($origmail ne '') {
-        $lastresort = $origmail;
-    }
-    if (($mailing eq 'helpdeskmail') && ($lastresort ne '')) {
-        unless (grep(/^\Q$defdom\E$/,&Apache::lonnet::current_machine_domains())) {
-            my $lonhost = $Apache::lonnet::perlvar{'lonHostID'};
-            my $machinedom = $Apache::lonnet::perlvar{'lonDefDomain'};
-            my %what = (
-                          perlvar => 1,
-                       );
-            my $primary = &Apache::lonnet::domain($defdom,'primary');
-            if ($primary) {
-                my $gotaddr;
-                my ($result,$returnhash) =
-                    &Apache::lonnet::get_remote_globals($primary,{ perlvar => 1 });
-                if (($result eq 'ok') && (ref($returnhash) eq 'HASH')) {
-                    if ($returnhash->{'lonSupportEMail'} =~ /^[^\@]+\@[^\@]+$/) {
-                        $lastresort = $returnhash->{'lonSupportEMail'};
-                        $gotaddr = 1;
-                    }
-                }
-                unless ($gotaddr) {
-                    my $uintdom = &Apache::lonnet::internet_dom($primary);
-                    my $intdom = &Apache::lonnet::internet_dom($lonhost);
-                    unless ($uintdom eq $intdom) {
-                        my %domconfig =
-                            &Apache::lonnet::get_dom('configuration',['contacts'],$machinedom);
-                        if (ref($domconfig{'contacts'}) eq 'HASH') {
-                            if (ref($domconfig{'contacts'}{'otherdomsmail'}) eq 'HASH') {
-                                my @contacts = ('adminemail','supportemail');
-                                foreach my $item (@contacts) {
-                                    if ($domconfig{'contacts'}{'otherdomsmail'}{$item}) {
-                                        my $addr = $domconfig{'contacts'}{$item};
-                                        if (!grep(/^\Q$addr\E$/,@recipients)) {
-                                            push(@recipients,$addr);
-                                        }
-                                    }
-                                }
-                                if ($domconfig{'contacts'}{'otherdomsmail'}{'others'}) {
-                                    $otheremails = $domconfig{'contacts'}{'otherdomsmail'}{'others'};
-                                }
-                                if ($domconfig{'contacts'}{'otherdomsmail'}{'bcc'}) {
-                                    my @bccs = split(/,/,$domconfig{'contacts'}{'otherdomsmail'}{'bcc'});
-                                    my @ok_bccs;
-                                    foreach my $bcc (@bccs) {
-                                        $bcc =~ s/^\s+//g;
-                                        $bcc =~ s/\s+$//g;
-                                        if ($bcc =~ m/^[^\@]+\@[^\@]+$/) {
-                                            if (!(grep(/^\Q$bcc\E$/,@ok_bccs))) {
-                                                push(@ok_bccs,$bcc);
-                                            }
-                                        }
-                                    }
-                                    if (@ok_bccs > 0) {
-                                        $allbcc = join(', ',@ok_bccs);
-                                    }
-                                }
-                                $addtext = $domconfig{'contacts'}{'otherdomsmail'}{'include'};
-                            }
-                        }
-                    }
-                }
-            }
-        }
+        push(@recipients,$origmail);
     }
     if (defined($defmail)) {
         if ($defmail ne '') {
@@ -14416,21 +12803,8 @@ sub build_recipient_list {
             }
         }
     }
-    if ($mailing eq 'helpdeskmail') {
-        if ((!@recipients) && ($lastresort ne '')) {
-            push(@recipients,$lastresort);
-        }
-    } elsif ($lastresort ne '') {
-        if (!grep(/^\Q$lastresort\E$/,@recipients)) {
-            push(@recipients,$lastresort);
-        }
-    }
-    my $recipientlist = join(',',@recipients);
-    if (wantarray) {
-        return ($recipientlist,$allbcc,$addtext);
-    } else {
-        return $recipientlist;
-    }
+    my $recipientlist = join(',',@recipients); 
+    return $recipientlist;
 }
 
 ############################################################
@@ -14521,8 +12895,6 @@ jsarray (reference to array of categorie
 subcats (reference to hash of arrays containing all subcategories within each 
          category, -recursive)
 
-maxd (reference to hash used to hold max depth for all top-level categories).
-
 Returns: nothing
 
 Side effects: populates trails and allitems hash references.
@@ -14530,7 +12902,7 @@ Side effects: populates trails and allit
 =cut
 
 sub extract_categories {
-    my ($categories,$cats,$trails,$allitems,$idx,$jsarray,$subcats,$maxd) = @_;
+    my ($categories,$cats,$trails,$allitems,$idx,$jsarray,$subcats) = @_;
     if (ref($categories) eq 'HASH') {
         &gather_categories($categories,$cats,$idx,$jsarray);
         if (ref($cats->[0]) eq 'ARRAY') {
@@ -14556,15 +12928,12 @@ sub extract_categories {
                         if (ref($subcats) eq 'HASH') {
                             push(@{$subcats->{$item}},&escape($category).':'.&escape($name).':1');
                         }
-                        &recurse_categories($cats,2,$category,$trails,$allitems,\@parents,$subcats,$maxd);
+                        &recurse_categories($cats,2,$category,$trails,$allitems,\@parents,$subcats);
                     }
                 } else {
                     if (ref($subcats) eq 'HASH') {
                         $subcats->{$item} = [];
                     }
-                    if (ref($maxd) eq 'HASH') {
-                        $maxd->{$name} = 1;
-                    }
                 }
             }
         }
@@ -14574,7 +12943,7 @@ sub extract_categories {
 
 =pod
 
-=item * &recurse_categories()
+=item *&recurse_categories()
 
 Recursively used to generate breadcrumb trails for course categories.
 
@@ -14602,7 +12971,7 @@ Side effects: populates trails and allit
 =cut
 
 sub recurse_categories {
-    my ($cats,$depth,$category,$trails,$allitems,$parents,$subcats,$maxd) = @_;
+    my ($cats,$depth,$category,$trails,$allitems,$parents,$subcats) = @_;
     my $shallower = $depth - 1;
     if (ref($cats->[$depth]{$category}) eq 'ARRAY') {
         for (my $k=0; $k<@{$cats->[$depth]{$category}}; $k++) {
@@ -14629,28 +12998,23 @@ sub recurse_categories {
                 }
             }
             &recurse_categories($cats,$deeper,$name,$trails,$allitems,$parents,
-                                $subcats,$maxd);
+                                $subcats);
             pop(@{$parents});
         }
     } else {
         my $item = &escape($category).':'.&escape($parents->[-1]).':'.$shallower;
-        my $trailstr = join(' &raquo; ',(@{$parents},$category));
+        my $trailstr = join(' -&gt; ',(@{$parents},$category));
         if ($allitems->{$item} eq '') {
             push(@{$trails},$trailstr);
             $allitems->{$item} = scalar(@{$trails})-1;
         }
-        if (ref($maxd) eq 'HASH') {
-            if ($depth > $maxd->{$parents->[0]}) {
-                $maxd->{$parents->[0]} = $depth;
-            }
-        }
     }
     return;
 }
 
 =pod
 
-=item * &assign_categories_table()
+=item *&assign_categories_table()
 
 Create a datatable for display of hierarchical categories in a domain,
 with checkboxes to allow a course to be categorized. 
@@ -14664,19 +13028,16 @@ currcat - scalar with an & separated lis
 
 type    - scalar contains course type (Course or Community).
 
-disabled - scalar (optional) contains disabled="disabled" if input elements are
-           to be readonly (e.g., Domain Helpdesk role viewing course settings).
-
 Returns: $output (markup to be displayed) 
 
 =cut
 
 sub assign_categories_table {
-    my ($cathash,$currcat,$type,$disabled) = @_;
+    my ($cathash,$currcat,$type) = @_;
     my $output;
     if (ref($cathash) eq 'HASH') {
-        my (@cats,@trails,%allitems,%idx,@jsarray,%maxd,@path,$maxdepth);
-        &extract_categories($cathash,\@cats,\@trails,\%allitems,\%idx,\@jsarray,\%maxd);
+        my (@cats,@trails,%allitems,%idx,@jsarray,@path,$maxdepth);
+        &extract_categories($cathash,\@cats,\@trails,\%allitems,\%idx,\@jsarray);
         $maxdepth = scalar(@cats);
         if (@cats > 0) {
             my $itemcount = 0;
@@ -14708,11 +13069,11 @@ sub assign_categories_table {
                     }
                     $table .= '<tr '.$css_class.'><td><span class="LC_nobreak">'.
                               '<input type="checkbox" name="usecategory" value="'.
-                              $item.'"'.$checked.$disabled.' />'.$parent_title.'</span>'.
+                              $item.'"'.$checked.' />'.$parent_title.'</span>'.
                               '<input type="hidden" name="catname" value="'.$parent.'" /></td>';
                     my $depth = 1;
                     push(@path,$parent);
-                    $table .= &assign_category_rows($itemcount,\@cats,$depth,$parent,\@path,\@currcategories,$disabled);
+                    $table .= &assign_category_rows($itemcount,\@cats,$depth,$parent,\@path,\@currcategories);
                     pop(@path);
                     $table .= '</tr><tr><td colspan="'.$maxdepth.'" class="LC_row_separator"></td></tr>';
                     $itemcount ++;
@@ -14730,7 +13091,7 @@ sub assign_categories_table {
 
 =pod
 
-=item * &assign_category_rows()
+=item *&assign_category_rows()
 
 Create a datatable row for display of nested categories in a domain,
 with checkboxes to allow a course to be categorized,called recursively.
@@ -14751,15 +13112,12 @@ path - Array containing all categories b
 
 currcategories - reference to array of current categories assigned to the course
 
-disabled - scalar (optional) contains disabled="disabled" if input elements are
-           to be readonly (e.g., Domain Helpdesk role viewing course settings).
-
 Returns: $output (markup to be displayed).
 
 =cut
 
 sub assign_category_rows {
-    my ($itemcount,$cats,$depth,$parent,$path,$currcategories,$disabled) = @_;
+    my ($itemcount,$cats,$depth,$parent,$path,$currcategories) = @_;
     my ($text,$name,$item,$chgstr);
     if (ref($cats) eq 'ARRAY') {
         my $maxdepth = scalar(@{$cats});
@@ -14767,7 +13125,7 @@ sub assign_category_rows {
             if (ref($cats->[$depth]{$parent}) eq 'ARRAY') {
                 my $numchildren = @{$cats->[$depth]{$parent}};
                 my $css_class = $itemcount%2?' class="LC_odd_row"':'';
-                $text .= '<td><table class="LC_data_table">';
+                $text .= '<td><table class="LC_datatable">';
                 for (my $j=0; $j<$numchildren; $j++) {
                     $name = $cats->[$depth]{$parent}[$j];
                     $item = &escape($name).':'.&escape($parent).':'.$depth;
@@ -14782,12 +13140,12 @@ sub assign_category_rows {
                     }
                     $text .= '<tr><td><span class="LC_nobreak"><label>'.
                              '<input type="checkbox" name="usecategory" value="'.
-                             $item.'"'.$checked.$disabled.' />'.$name.'</label></span>'.
+                             $item.'"'.$checked.' />'.$name.'</label></span>'.
                              '<input type="hidden" name="catname" value="'.$name.'" />'.
                              '</td><td>';
                     if (ref($path) eq 'ARRAY') {
                         push(@{$path},$name);
-                        $text .= &assign_category_rows($itemcount,$cats,$deeper,$name,$path,$currcategories,$disabled);
+                        $text .= &assign_category_rows($itemcount,$cats,$deeper,$name,$path,$currcategories);
                         pop(@{$path});
                     }
                     $text .= '</td></tr>';
@@ -14799,12 +13157,6 @@ sub assign_category_rows {
     return $text;
 }
 
-=pod
-
-=back
-
-=cut
-
 ############################################################
 ############################################################
 
@@ -14821,7 +13173,7 @@ sub commit_customrole {
 }
 
 sub commit_standardrole {
-    my ($udom,$uname,$url,$three,$start,$end,$one,$two,$sec,$context,$credits) = @_;
+    my ($udom,$uname,$url,$three,$start,$end,$one,$two,$sec,$context) = @_;
     my ($output,$logmsg,$linefeed);
     if ($context eq 'auto') {
         $linefeed = "\n";
@@ -14830,7 +13182,7 @@ sub commit_standardrole {
     }  
     if ($three eq 'st') {
         my $result = &commit_studentrole(\$logmsg,$udom,$uname,$url,$three,$start,$end,
-                                         $one,$two,$sec,$context,$credits);
+                                         $one,$two,$sec,$context);
         if (($result =~ /^error/) || ($result eq 'not_in_class') || 
             ($result eq 'unknown_course') || ($result eq 'refused')) {
             $output = $logmsg.' '.&mt('Error: ').$result."\n"; 
@@ -14861,8 +13213,7 @@ sub commit_standardrole {
 }
 
 sub commit_studentrole {
-    my ($logmsg,$udom,$uname,$url,$three,$start,$end,$one,$two,$sec,$context,
-        $credits) = @_;
+    my ($logmsg,$udom,$uname,$url,$three,$start,$end,$one,$two,$sec,$context) = @_;
     my ($result,$linefeed,$oldsecurl,$newsecurl);
     if ($context eq 'auto') {
         $linefeed = "\n";
@@ -14909,11 +13260,7 @@ sub commit_studentrole {
             }
         }
         if (($expire_role_result eq 'ok') || ($secchange == 0)) {
-            $modify_section_result = 
-                &Apache::lonnet::modify_student_enrollment($udom,$uname,undef,undef,
-                                                           undef,undef,undef,$sec,
-                                                           $end,$start,'','',$cid,
-                                                           '',$context,$credits);
+            $modify_section_result = &Apache::lonnet::modify_student_enrollment($udom,$uname,undef,undef,undef,undef,undef,$sec,$end,$start,'','',$cid,'',$context);
             if ($modify_section_result =~ /^ok/) {
                 if ($secchange == 1) {
                     if ($sec eq '') {
@@ -14970,26 +13317,6 @@ sub commit_studentrole {
     return $result;
 }
 
-sub show_role_extent {
-    my ($scope,$context,$role) = @_;
-    $scope =~ s{^/}{};
-    my @courseroles = &Apache::lonuserutils::roles_by_context('course',1);
-    push(@courseroles,'co');
-    my @authorroles = &Apache::lonuserutils::roles_by_context('author');
-    if (($context eq 'course') || (grep(/^\Q$role\E/,@courseroles))) {
-        $scope =~ s{/}{_};
-        return '<span class="LC_cusr_emph">'.$env{'course.'.$scope.'.description'}.'</span>';
-    } elsif (($context eq 'author') || (grep(/^\Q$role\E/,@authorroles))) {
-        my ($audom,$auname) = split(/\//,$scope);
-        return &mt('[_1] Author Space','<span class="LC_cusr_emph">'.
-                   &Apache::loncommon::plainname($auname,$audom).'</span>');
-    } else {
-        $scope =~ s{/$}{};
-        return &mt('Domain: [_1]','<span class="LC_cusr_emph">'.
-                   &Apache::lonnet::domain($scope,'description').'</span>');
-    }
-}
-
 ############################################################
 ############################################################
 
@@ -15018,95 +13345,37 @@ sub check_clone {
                 return ($can_clone, $clonemsg, $cloneid, $clonehome);
             }
         }
-	if (($env{'request.role.domain'} eq $args->{'clonedomain'}) &&
+	if (($env{'request.role.domain'} eq $args->{'clonedomain'}) && 
             (&Apache::lonnet::allowed('ccc',$env{'request.role.domain'}))) {
 	    $can_clone = 1;
 	} else {
-	    my %clonehash = &Apache::lonnet::get('environment',['cloners','internal.coursecode'],
+	    my %clonehash = &Apache::lonnet::get('environment',['cloners'],
 						 $args->{'clonedomain'},$args->{'clonecourse'});
-            if ($clonehash{'cloners'} eq '') {
-                my %domdefs = &Apache::lonnet::get_domain_defaults($args->{'course_domain'});
-                if ($domdefs{'canclone'}) {
-                    unless ($domdefs{'canclone'} eq 'none') {
-                        if ($domdefs{'canclone'} eq 'domain') {
-                            if ($args->{'ccdomain'} eq $args->{'clonedomain'}) {
-                                $can_clone = 1;
-                            }
-                        } elsif (($clonehash{'internal.coursecode'}) && ($args->{'crscode'}) &&
-                                 ($args->{'clonedomain'} eq  $args->{'course_domain'})) {
-                            if (&Apache::lonnet::default_instcode_cloning($args->{'clonedomain'},$domdefs{'canclone'},
-                                                                          $clonehash{'internal.coursecode'},$args->{'crscode'})) {
-                                $can_clone = 1;
-                            }
-                        }
-                    }
-                }
+	    my @cloners = split(/,/,$clonehash{'cloners'});
+            if (grep(/^\*$/,@cloners)) {
+                $can_clone = 1;
+            } elsif (grep(/^\*\:\Q$args->{'ccdomain'}\E$/,@cloners)) {
+                $can_clone = 1;
             } else {
-	        my @cloners = split(/,/,$clonehash{'cloners'});
-                if (grep(/^\*$/,@cloners)) {
-                    $can_clone = 1;
-                } elsif (grep(/^\*\:\Q$args->{'ccdomain'}\E$/,@cloners)) {
-                    $can_clone = 1;
-                } elsif (grep(/^\Q$args->{'ccuname'}\E:\Q$args->{'ccdomain'}\E$/,@cloners)) {
-                    $can_clone = 1;
-                }
-                unless ($can_clone) {
-                    if (($clonehash{'internal.coursecode'}) && ($args->{'crscode'}) &&
-                        ($args->{'clonedomain'} eq  $args->{'course_domain'})) {
-                        my (%gotdomdefaults,%gotcodedefaults);
-                        foreach my $cloner (@cloners) {
-                            if (($cloner ne '*') && ($cloner !~ /^\*\:$match_domain$/) &&
-                                ($cloner !~ /^$match_username\:$match_domain$/) && ($cloner ne '')) {
-                                my (%codedefaults,@code_order);
-                                if (ref($gotcodedefaults{$args->{'clonedomain'}}) eq 'HASH') {
-                                    if (ref($gotcodedefaults{$args->{'clonedomain'}}{'defaults'}) eq 'HASH') {
-                                        %codedefaults = %{$gotcodedefaults{$args->{'clonedomain'}}{'defaults'}};
-                                    }
-                                    if (ref($gotcodedefaults{$args->{'clonedomain'}}{'order'}) eq 'ARRAY') {
-                                        @code_order = @{$gotcodedefaults{$args->{'clonedomain'}}{'order'}};
-                                    }
-                                } else {
-                                    &Apache::lonnet::auto_instcode_defaults($args->{'clonedomain'},
-                                                                            \%codedefaults,
-                                                                            \@code_order);
-                                    $gotcodedefaults{$args->{'clonedomain'}}{'defaults'} = \%codedefaults;
-                                    $gotcodedefaults{$args->{'clonedomain'}}{'order'} = \@code_order;
-                                }
-                                if (@code_order > 0) {
-                                    if (&Apache::lonnet::check_instcode_cloning(\%codedefaults,\@code_order,
-                                                                                $cloner,$clonehash{'internal.coursecode'},
-                                                                                $args->{'crscode'})) {
-                                        $can_clone = 1;
-                                        last;
-                                    }
-                                }
-                            }
-                        }
-                    }
-                }
-            }
-            unless ($can_clone) {
                 my $ccrole = 'cc';
                 if ($args->{'crstype'} eq 'Community') {
                     $ccrole = 'co';
                 }
-                my %roleshash =
-                    &Apache::lonnet::get_my_roles($args->{'ccuname'},
-                                                  $args->{'ccdomain'},
-                                                  'userroles',['active'],[$ccrole],
-                                                  [$args->{'clonedomain'}]);
-                if ($roleshash{$args->{'clonecourse'}.':'.$args->{'clonedomain'}.':'.$ccrole}) {
+	        my %roleshash =
+		    &Apache::lonnet::get_my_roles($args->{'ccuname'},
+					 $args->{'ccdomain'},
+                                         'userroles',['active'],[$ccrole],
+					 [$args->{'clonedomain'}]);
+	        if (($roleshash{$args->{'clonecourse'}.':'.$args->{'clonedomain'}.':'.$ccrole}) || (grep(/^\Q$args->{'ccuname'}\E:\Q$args->{'ccdomain'}\E$/,@cloners))) {
                     $can_clone = 1;
-                } elsif (&Apache::lonnet::is_course_owner($args->{'clonedomain'},$args->{'clonecourse'},
-                                                          $args->{'ccuname'},$args->{'ccdomain'})) {
+                } elsif (&Apache::lonnet::is_course_owner($args->{'clonedomain'},$args->{'clonecourse'},$args->{'ccuname'},$args->{'ccdomain'})) {
                     $can_clone = 1;
-                }
-            }
-            unless ($can_clone) {
-                if ($args->{'crstype'} eq 'Community') {
-                    $clonemsg = &mt('No new community created.').$linefeed.&mt('The new community could not be cloned from the existing community because the new community owner ([_1]) does not have cloning rights in the existing community ([_2]).',$args->{'ccuname'}.':'.$args->{'ccdomain'},$clonedesc{'description'});
                 } else {
-                    $clonemsg = &mt('No new course created.').$linefeed.&mt('The new course could not be cloned from the existing course because the new course owner ([_1]) does not have cloning rights in the existing course ([_2]).',$args->{'ccuname'}.':'.$args->{'ccdomain'},$clonedesc{'description'});
+                    if ($args->{'crstype'} eq 'Community') {
+                        $clonemsg = &mt('No new community created.').$linefeed.&mt('The new community could not be cloned from the existing community because the new community owner ([_1]) does not have cloning rights in the existing community ([_2]).',$args->{'ccuname'}.':'.$args->{'ccdomain'},$clonedesc{'description'});
+                    } else {
+                        $clonemsg = &mt('No new course created.').$linefeed.&mt('The new course could not be cloned from the existing course because the new course owner ([_1]) does not have cloning rights in the existing course ([_2]).',$args->{'ccuname'}.':'.$args->{'ccdomain'},$clonedesc{'description'});
+                    }
 	        }
 	    }
         }
@@ -15115,8 +13384,7 @@ sub check_clone {
 }
 
 sub construct_course {
-    my ($args,$logmsg,$courseid,$crsudom,$crsunum,$udom,$uname,$context,
-        $cnum,$category,$coderef) = @_;
+    my ($args,$logmsg,$courseid,$crsudom,$crsunum,$udom,$uname,$context,$cnum,$category) = @_;
     my $outcome;
     my $linefeed =  '<br />'."\n";
     if ($context eq 'auto') {
@@ -15212,13 +13480,8 @@ sub construct_course {
                    'pch.users.denied',
                    'plc.users.denied',
                    'hidefromcat',
-                   'checkforpriv',
-                   'categories',
-                   'internal.uniquecode'],
+                   'categories'],
                    $$crsudom,$$crsunum);
-        if ($args->{'textbook'}) {
-            $cenv{'internal.textbook'} = $args->{'textbook'};
-        }
     }
 
 #
@@ -15246,9 +13509,6 @@ sub construct_course {
     } else {
         $cenv{'internal.courseowner'} = $args->{'curruser'};
     }
-    if ($args->{'defaultcredits'}) {
-        $cenv{'internal.defaultcredits'} = $args->{'defaultcredits'};
-    }
     my @badclasses = (); # Used to accumulate sections/crosslistings that did not pass classlist access check for course owner.
     if ($args->{'crssections'}) {
         $cenv{'internal.sectionnums'} = '';
@@ -15264,7 +13524,7 @@ sub construct_course {
                 my $addcheck = &Apache::lonnet::auto_new_course($$crsunum,$$crsudom,$class,$cenv{'internal.courseowner'});
                 $cenv{'internal.sectionnums'} .= $item.',';
                 unless ($addcheck eq 'ok') {
-                    push(@badclasses,$class);
+                    push @badclasses, $class;
                 }
             }
             $cenv{'internal.sectionnums'} =~ s/,$//;
@@ -15273,11 +13533,6 @@ sub construct_course {
 # do not hide course coordinator from staff listing, 
 # even if privileged
     $cenv{'nothideprivileged'}=$args->{'ccuname'}.':'.$args->{'ccdomain'};
-# add course coordinator's domain to domains to check for privileged users
-# if different to course domain
-    if ($$crsudom ne $args->{'ccdomain'}) {
-        $cenv{'checkforpriv'} = $args->{'ccdomain'};
-    }
 # add crosslistings
     if ($args->{'crsxlist'}) {
         $cenv{'internal.crosslistings'}='';
@@ -15292,7 +13547,7 @@ sub construct_course {
                 my $addcheck =  &Apache::lonnet::auto_new_course($$crsunum,$$crsudom,$xl,$cenv{'internal.courseowner'});
                 $cenv{'internal.crosslistings'} .= $item.',';
                 unless ($addcheck eq 'ok') {
-                    push(@badclasses,$xl);
+                    push @badclasses, $xl;
                 }
             }
             $cenv{'internal.crosslistings'} =~ s/,$//;
@@ -15327,29 +13582,28 @@ sub construct_course {
     }
     if (@badclasses > 0) {
         my %lt=&Apache::lonlocal::texthash(
-                'tclb' => 'The courses listed below were included as sections or crosslistings affiliated with your new LON-CAPA course.',
-                'howi' => 'However, if automated course roster updates are enabled for this class, these particular sections/crosslistings are not guaranteed to contribute towards enrollment.',
-                'itis' => 'It is possible that rights to access enrollment for these classes will be available through assignment of co-owners.',
+                'tclb' => 'The courses listed below were included as sections or crosslistings affiliated with your new LON-CAPA course.  However, if automated course roster updates are enabled for this class, these particular sections/crosslistings will not contribute towards enrollment, because the user identified as the course owner for this LON-CAPA course',
+                'dnhr' => 'does not have rights to access enrollment in these classes',
+                'adby' => 'as determined by the policies of your institution on access to official classlists'
         );
-        my $badclass_msg = $lt{'tclb'}.$linefeed.$lt{'howi'}.$linefeed.
-                           &mt('That is because the user identified as the course owner ([_1]) does not have rights to access enrollment in these classes, as determined by the policies of your institution on access to official classlists',$cenv{'internal.courseowner'}).$linefeed.$lt{'itis'};
+        my $badclass_msg = $cenv{'internal.courseowner'}.') - '.$lt{'dnhr'}.
+                           ' ('.$lt{'adby'}.')';
         if ($context eq 'auto') {
             $outcome .= $badclass_msg.$linefeed;
-        } else {
             $outcome .= '<div class="LC_warning">'.$badclass_msg.$linefeed.'<ul>'."\n";
-        }
-        foreach my $item (@badclasses) {
+            foreach my $item (@badclasses) {
+                if ($context eq 'auto') {
+                    $outcome .= " - $item\n";
+                } else {
+                    $outcome .= "<li>$item</li>\n";
+                }
+            }
             if ($context eq 'auto') {
-                $outcome .= " - $item\n";
+                $outcome .= $linefeed;
             } else {
-                $outcome .= "<li>$item</li>\n";
+                $outcome .= "</ul><br /><br /></div>\n";
             }
-        }
-        if ($context eq 'auto') {
-            $outcome .= $linefeed;
-        } else {
-            $outcome .= "</ul><br /><br /></div>\n";
-        }
+        } 
     }
     if ($args->{'no_end_date'}) {
         $args->{'endaccess'} = 0;
@@ -15381,9 +13635,6 @@ sub construct_course {
        if ($args->{'setcontent'}) {
            $cenv{'question.email'}=$args->{'ccuname'}.':'.$args->{'ccdomain'};
        }
-       if ($args->{'setcomment'}) {
-           $cenv{'comment.email'}=$args->{'ccuname'}.':'.$args->{'ccdomain'};
-       }
     }
     if ($args->{'reshome'}) {
 	$cenv{'reshome'}=$args->{'reshome'}.'/';
@@ -15406,25 +13657,6 @@ sub construct_course {
 	}
     }
 
-#
-#  generate and store uniquecode (available to course requester), if course should have one.
-#
-    if ($args->{'uniquecode'}) {
-        my ($code,$error) = &make_unique_code($$crsudom,$$crsunum);
-        if ($code) {
-            $cenv{'internal.uniquecode'} = $code;
-            my %crsinfo =
-                &Apache::lonnet::courseiddump($$crsudom,'.',1,'.','.',$$crsunum,undef,undef,'.');
-            if (ref($crsinfo{$$crsudom.'_'.$$crsunum}) eq 'HASH') {
-                $crsinfo{$$crsudom.'_'.$$crsunum}{'uniquecode'} = $code;
-                my $putres = &Apache::lonnet::courseidput($$crsudom,\%crsinfo,$crsuhome,'notime');
-            }
-            if (ref($coderef)) {
-                $$coderef = $code;
-            }
-        }
-    }
-
     if ($args->{'disresdis'}) {
         $cenv{'pch.roles.denied'}='st';
     }
@@ -15493,60 +13725,6 @@ sub construct_course {
     return (1,$outcome);
 }
 
-sub make_unique_code {
-    my ($cdom,$cnum) = @_;
-    # get lock on uniquecodes db
-    my $lockhash = {
-                      $cnum."\0".'uniquecodes' => $env{'user.name'}.
-                                                  ':'.$env{'user.domain'},
-                   };
-    my $tries = 0;
-    my $gotlock = &Apache::lonnet::newput_dom('uniquecodes',$lockhash,$cdom);
-    my ($code,$error);
-
-    while (($gotlock ne 'ok') && ($tries<3)) {
-        $tries ++;
-        sleep 1;
-        $gotlock = &Apache::lonnet::newput_dom('uniquecodes',$lockhash,$cdom);
-    }
-    if ($gotlock eq 'ok') {
-        my %currcodes = &Apache::lonnet::dump_dom('uniquecodes',$cdom);
-        my $gotcode;
-        my $attempts = 0;
-        while ((!$gotcode) && ($attempts < 100)) {
-            $code = &generate_code();
-            if (!exists($currcodes{$code})) {
-                $gotcode = 1;
-                unless (&Apache::lonnet::newput_dom('uniquecodes',{ $code => $cnum },$cdom) eq 'ok') {
-                    $error = 'nostore';
-                }
-            }
-            $attempts ++;
-        }
-        my @del_lock = ($cnum."\0".'uniquecodes');
-        my $dellockoutcome = &Apache::lonnet::del_dom('uniquecodes',\@del_lock,$cdom);
-    } else {
-        $error = 'nolock';
-    }
-    return ($code,$error);
-}
-
-sub generate_code {
-    my $code;
-    my @letts = qw(B C D G H J K M N P Q R S T V W X Z);
-    for (my $i=0; $i<6; $i++) {
-        my $lettnum = int (rand 2);
-        my $item = '';
-        if ($lettnum) {
-            $item = $letts[int( rand(18) )];
-        } else {
-            $item = 1+int( rand(8) );
-        }
-        $code .= $item;
-    }
-    return $code;
-}
-
 ############################################################
 ############################################################
 
@@ -15574,12 +13752,11 @@ sub group_term {
 }
 
 sub course_types {
-    my @types = ('official','unofficial','community','textbook');
+    my @types = ('official','unofficial','community');
     my %typename = (
                          official   => 'Official course',
                          unofficial => 'Unofficial course',
                          community  => 'Community',
-                         textbook   => 'Textbook course',
                    );
     return (\@types,\%typename);
 }
@@ -15640,7 +13817,7 @@ sub escape_url {
     my ($url)   = @_;
     my @urlslices = split(/\//, $url,-1);
     my $lastitem = &escape(pop(@urlslices));
-    return &HTML::Entities::encode(join('/',@urlslices),"'").'/'.$lastitem;
+    return join('/',@urlslices).'/'.$lastitem;
 }
 
 sub compare_arrays {
@@ -15694,37 +13871,10 @@ sub init_user_environment {
 	    opendir(DIR,$lonids);
 	    while ($filename=readdir(DIR)) {
 		if ($filename=~/^$username\_\d+\_$domain\_$authhost\.id$/) {
-                    if (tie(my %oldenv,'GDBM_File',"$lonids/$filename",
-                            &GDBM_READER(),0640)) {
-                        my $linkedfile;
-                        if (exists($oldenv{'user.linkedenv'})) {
-                            $linkedfile = $oldenv{'user.linkedenv'};
-                        }
-                        untie(%oldenv);
-                        if (unlink("$lonids/$filename")) {
-                            if ($linkedfile =~ /^[a-f0-9]+_linked$/) {
-                                if (-l "$lonids/$linkedfile.id") {
-                                    unlink("$lonids/$linkedfile.id");
-                                }
-                            }
-                        }
-                    } else {
-                        unlink($lonids.'/'.$filename);
-                    }
+		    unlink($lonids.'/'.$filename);
 		}
 	    }
 	    closedir(DIR);
-# If there is a undeleted lockfile for the user's paste buffer remove it.
-            my $namespace = 'nohist_courseeditor';
-            my $lockingkey = 'paste'."\0".'locked_num';
-            my %lockhash = &Apache::lonnet::get($namespace,[$lockingkey],
-                                                $domain,$username);
-            if (exists($lockhash{$lockingkey})) {
-                my $delresult = &Apache::lonnet::del($namespace,[$lockingkey],$domain,$username);
-                unless ($delresult eq 'ok') {
-                    &Apache::lonnet::logthis("Failed to delete paste buffer locking key in $namespace for ".$username.":".$domain." Result was: $delresult");
-                }
-            }
 	}
 # Give them a new cookie
 	my $id = ($args->{'robot'} ? 'robot'.$args->{'robot'}
@@ -15738,8 +13888,8 @@ sub init_user_environment {
     }
 # ------------------------------------ Check browser type and MathML capability
 
-    my ($httpbrowser,$clientbrowser,$clientversion,$clientmathml,$clientunicode,
-        $clientos,$clientmobile,$clientinfo,$clientosversion) = &decode_user_agent($r);
+    my ($httpbrowser,$clientbrowser,$clientversion,$clientmathml,
+        $clientunicode,$clientos) = &decode_user_agent($r);
 
 # ------------------------------------------------------------- Get environment
 
@@ -15770,9 +13920,6 @@ sub init_user_environment {
 	     "browser.mathml"     => $clientmathml,
 	     "browser.unicode"    => $clientunicode,
 	     "browser.os"         => $clientos,
-             "browser.mobile"     => $clientmobile,
-             "browser.info"       => $clientinfo,
-             "browser.osversion"  => $clientosversion,
 	     "server.domain"      => $Apache::lonnet::perlvar{'lonDefDomain'},
 	     "request.course.fn"  => '',
 	     "request.course.uri" => '',
@@ -15792,50 +13939,36 @@ sub init_user_environment {
 	    $env{'browser.interface'}=$form->{'interface'};
 	}
 
-        if ($form->{'iptoken'}) {
-            my $lonhost = $r->dir_config('lonHostID');
-            $initial_env{"user.noloadbalance"} = $lonhost;
-            $env{'user.noloadbalance'} = $lonhost;
+        my %is_adv = ( is_adv => $env{'user.adv'} );
+        my %domdef;
+        unless ($domain eq 'public') {
+            %domdef = &Apache::lonnet::get_domain_defaults($domain);
         }
 
-        if ($form->{'noloadbalance'}) {
-            my @hosts = &Apache::lonnet::current_machine_ids();
-            my $hosthere = $form->{'noloadbalance'};
-            if (grep(/^\Q$hosthere\E$/,@hosts)) {
-                $initial_env{"user.noloadbalance"} = $hosthere;
-                $env{'user.noloadbalance'} = $hosthere;
-            }
+        foreach my $tool ('aboutme','blog','webdav','portfolio') {
+            $userenv{'availabletools.'.$tool} = 
+                &Apache::lonnet::usertools_access($username,$domain,$tool,'reload',
+                                                  undef,\%userenv,\%domdef,\%is_adv);
         }
 
-        unless ($domain eq 'public') {
-            my %is_adv = ( is_adv => $env{'user.adv'} );
-            my %domdef = &Apache::lonnet::get_domain_defaults($domain);
-
-            foreach my $tool ('aboutme','blog','webdav','portfolio') {
-                $userenv{'availabletools.'.$tool} = 
-                    &Apache::lonnet::usertools_access($username,$domain,$tool,'reload',
-                                                      undef,\%userenv,\%domdef,\%is_adv);
-            }
-
-            foreach my $crstype ('official','unofficial','community','textbook') {
-                $userenv{'canrequest.'.$crstype} =
-                    &Apache::lonnet::usertools_access($username,$domain,$crstype,
-                                                      'reload','requestcourses',
-                                                      \%userenv,\%domdef,\%is_adv);
-            }
-
-            $userenv{'canrequest.author'} =
-                &Apache::lonnet::usertools_access($username,$domain,'requestauthor',
-                                                  'reload','requestauthor',
+        foreach my $crstype ('official','unofficial','community') {
+            $userenv{'canrequest.'.$crstype} =
+                &Apache::lonnet::usertools_access($username,$domain,$crstype,
+                                                  'reload','requestcourses',
                                                   \%userenv,\%domdef,\%is_adv);
-            my %reqauthor = &Apache::lonnet::get('requestauthor',['author_status','author'],
-                                                 $domain,$username);
-            my $reqstatus = $reqauthor{'author_status'};
-            if ($reqstatus eq 'approval' || $reqstatus eq 'approved') {
-                if (ref($reqauthor{'author'}) eq 'HASH') {
-                    $userenv{'requestauthorqueued'} = $reqstatus.':'.
-                                                      $reqauthor{'author'}{'timestamp'};
-                }
+        }
+
+        $userenv{'canrequest.author'} =
+            &Apache::lonnet::usertools_access($username,$domain,'requestauthor',
+                                        'reload','requestauthor',
+                                        \%userenv,\%domdef,\%is_adv);
+        my %reqauthor = &Apache::lonnet::get('requestauthor',['author_status','author'],
+                                             $domain,$username);
+        my $reqstatus = $reqauthor{'author_status'};
+        if ($reqstatus eq 'approval' || $reqstatus eq 'approved') { 
+            if (ref($reqauthor{'author'}) eq 'HASH') {
+                $userenv{'requestauthorqueued'} = $reqstatus.':'.
+                                                  $reqauthor{'author'}{'timestamp'};
             }
         }
 
@@ -15923,745 +14056,36 @@ sub clean_symb {
     return ($symb,$enc);
 }
 
-############################################################
-############################################################
-
-=pod
-
-=head1 Routines for building display used to search for courses
-
-
-=over 4
-
-=item * &build_filters()
-
-Create markup for a table used to set filters to use when selecting
-courses in a domain.  Used by lonpickcourse.pm, lonmodifycourse.pm
-and quotacheck.pl
-
-
-Inputs:
-
-filterlist - anonymous array of fields to include as potential filters
-
-crstype - course type
-
-roleelement - fifth arg in selectcourse_link() populates fifth arg in javascript: opencrsbrowser() function, used
-              to pop-open a course selector (will contain "extra element").
-
-multelement - if multiple course selections will be allowed, this will be a hidden form element: name: multiple; value: 1
-
-filter - anonymous hash of criteria and their values
-
-action - form action
-
-numfiltersref - ref to scalar (count of number of elements in institutional codes -- e.g., 4 for year, semester, department, and number)
-
-caller - caller context (e.g., set to 'modifycourse' when routine is called from lonmodifycourse.pm)
-
-cloneruname - username of owner of new course who wants to clone
-
-clonerudom - domain of owner of new course who wants to clone
-
-typeelem - text to use for left column in row containing course type (i.e., Course, Community or Course/Community)
-
-codetitlesref - reference to array of titles of components in institutional codes (official courses)
-
-codedom - domain
-
-formname - value of form element named "form".
-
-fixeddom - domain, if fixed.
-
-prevphase - value to assign to form element named "phase" when going back to the previous screen
-
-cnameelement - name of form element in form on opener page which will receive title of selected course
-
-cnumelement - name of form element in form on opener page which will receive courseID  of selected course
-
-cdomelement - name of form element in form on opener page which will receive domain of selected course
-
-setroles - includes access constraint identifier when setting a roles-based condition for acces to a portfolio file
-
-clonetext - hidden form elements containing list of courses cloneable by intended course owner when DC creates a course
-
-clonewarning - warning message about missing information for intended course owner when DC creates a course
-
-
-Returns: $output - HTML for display of search criteria, and hidden form elements.
-
-
-Side Effects: None
-
-=cut
-
-# ---------------------------------------------- search for courses based on last activity etc.
-
-sub build_filters {
-    my ($filterlist,$crstype,$roleelement,$multelement,$filter,$action,
-        $numtitlesref,$caller,$cloneruname,$clonerudom,$typeelement,
-        $codetitlesref,$codedom,$formname,$fixeddom,$prevphase,
-        $cnameelement,$cnumelement,$cdomelement,$setroles,
-        $clonetext,$clonewarning) = @_;
-    my ($list,$jscript);
-    my $onchange = 'javascript:updateFilters(this)';
-    my ($domainselectform,$sincefilterform,$createdfilterform,
-        $ownerdomselectform,$persondomselectform,$instcodeform,
-        $typeselectform,$instcodetitle);
-    if ($formname eq '') {
-        $formname = $caller;
-    }
-    foreach my $item (@{$filterlist}) {
-        unless (($item eq 'descriptfilter') || ($item eq 'instcodefilter') ||
-                ($item eq 'sincefilter') || ($item eq 'createdfilter')) {
-            if ($item eq 'domainfilter') {
-                $filter->{$item} = &LONCAPA::clean_domain($filter->{$item});
-            } elsif ($item eq 'coursefilter') {
-                $filter->{$item} = &LONCAPA::clean_courseid($filter->{$item});
-            } elsif ($item eq 'ownerfilter') {
-                $filter->{$item} = &LONCAPA::clean_username($filter->{$item});
-            } elsif ($item eq 'ownerdomfilter') {
-                $filter->{'ownerdomfilter'} =
-                    &LONCAPA::clean_domain($filter->{$item});
-                $ownerdomselectform = &select_dom_form($filter->{'ownerdomfilter'},
-                                                       'ownerdomfilter',1);
-            } elsif ($item eq 'personfilter') {
-                $filter->{$item} = &LONCAPA::clean_username($filter->{$item});
-            } elsif ($item eq 'persondomfilter') {
-                $persondomselectform = &select_dom_form($filter->{'persondomfilter'},
-                                                        'persondomfilter',1);
-            } else {
-                $filter->{$item} =~ s/\W//g;
-            }
-            if (!$filter->{$item}) {
-                $filter->{$item} = '';
-            }
-        }
-        if ($item eq 'domainfilter') {
-            my $allow_blank = 1;
-            if ($formname eq 'portform') {
-                $allow_blank=0;
-            } elsif ($formname eq 'studentform') {
-                $allow_blank=0;
-            }
-            if ($fixeddom) {
-                $domainselectform = '<input type="hidden" name="domainfilter"'.
-                                    ' value="'.$codedom.'" />'.
-                                    &Apache::lonnet::domain($codedom,'description');
+sub build_release_hashes {
+    my ($checkparms,$checkresponsetypes,$checkcrstypes,$anonsurvey,$randomizetry) = @_;
+    return unless((ref($checkparms) eq 'HASH') && (ref($checkresponsetypes) eq 'HASH') &&
+                  (ref($checkcrstypes) eq 'HASH') && (ref($anonsurvey) eq 'HASH') &&
+                  (ref($randomizetry) eq 'HASH'));
+    foreach my $key (keys(%Apache::lonnet::needsrelease)) {
+        my ($item,$name,$value) = split(/:/,$key);
+        if ($item eq 'parameter') {
+            if (ref($checkparms->{$name}) eq 'ARRAY') {
+                unless(grep(/^\Q$name\E$/,@{$checkparms->{$name}})) {
+                    push(@{$checkparms->{$name}},$value);
+                }
             } else {
-                $domainselectform = &select_dom_form($filter->{$item},
-                                                     'domainfilter',
-                                                      $allow_blank,'',$onchange);
+                push(@{$checkparms->{$name}},$value);
             }
-        } else {
-            $list->{$item} = &HTML::Entities::encode($filter->{$item},'<>&"');
-        }
-    }
-
-    # last course activity filter and selection
-    $sincefilterform = &timebased_select_form('sincefilter',$filter);
-
-    # course created filter and selection
-    if (exists($filter->{'createdfilter'})) {
-        $createdfilterform = &timebased_select_form('createdfilter',$filter);
-    }
-
-    my %lt = &Apache::lonlocal::texthash(
-                'cac' => "$crstype Activity",
-                'ccr' => "$crstype Created",
-                'cde' => "$crstype Title",
-                'cdo' => "$crstype Domain",
-                'ins' => 'Institutional Code',
-                'inc' => 'Institutional Categorization',
-                'cow' => "$crstype Owner/Co-owner",
-                'cop' => "$crstype Personnel Includes",
-                'cog' => 'Type',
-             );
-
-    if (($formname eq 'ccrs') || ($formname eq 'requestcrs')) {
-        my $typeval = 'Course';
-        if ($crstype eq 'Community') {
-            $typeval = 'Community';
-        }
-        $typeselectform = '<input type="hidden" name="type" value="'.$typeval.'" />';
-    } else {
-        $typeselectform =  '<select name="type" size="1"';
-        if ($onchange) {
-            $typeselectform .= ' onchange="'.$onchange.'"';
-        }
-        $typeselectform .= '>'."\n";
-        foreach my $posstype ('Course','Community') {
-            $typeselectform.='<option value="'.$posstype.'"'.
-                ($posstype eq $crstype ? ' selected="selected" ' : ''). ">".&mt($posstype)."</option>\n";
-        }
-        $typeselectform.="</select>";
-    }
-
-    my ($cloneableonlyform,$cloneabletitle);
-    if (exists($filter->{'cloneableonly'})) {
-        my $cloneableon = '';
-        my $cloneableoff = ' checked="checked"';
-        if ($filter->{'cloneableonly'}) {
-            $cloneableon = $cloneableoff;
-            $cloneableoff = '';
-        }
-        $cloneableonlyform = '<span class="LC_nobreak"><label><input type="radio" name="cloneableonly" value="1" '.$cloneableon.'/>&nbsp;'.&mt('Required').'</label>'.('&nbsp;'x3).'<label><input type="radio" name="cloneableonly" value="" '.$cloneableoff.' />&nbsp;'.&mt('No restriction').'</label></span>';
-        if ($formname eq 'ccrs') {
-            $cloneabletitle = &mt('Cloneable for [_1]',$cloneruname.':'.$clonerudom);
-        } else {
-            $cloneabletitle = &mt('Cloneable by you');
-        }
-    }
-    my $officialjs;
-    if ($crstype eq 'Course') {
-        if (exists($filter->{'instcodefilter'})) {
-#            if (($fixeddom) || ($formname eq 'requestcrs') ||
-#                ($formname eq 'modifycourse') || ($formname eq 'filterpicker')) {
-            if ($codedom) {
-                $officialjs = 1;
-                ($instcodeform,$jscript,$$numtitlesref) =
-                    &Apache::courseclassifier::instcode_selectors($codedom,'filterpicker',
-                                                                  $officialjs,$codetitlesref);
-                if ($jscript) {
-                    $jscript = '<script type="text/javascript">'."\n".
-                               '// <![CDATA['."\n".
-                               $jscript."\n".
-                               '// ]]>'."\n".
-                               '</script>'."\n";
-                }
-            }
-            if ($instcodeform eq '') {
-                $instcodeform =
-                    '<input type="text" name="instcodefilter" size="10" value="'.
-                    $list->{'instcodefilter'}.'" />';
-                $instcodetitle = $lt{'ins'};
-            } else {
-                $instcodetitle = $lt{'inc'};
+        } elsif ($item eq 'resourcetag') {
+            if ($name eq 'responsetype') {
+                $checkresponsetypes->{$value} = $Apache::lonnet::needsrelease{$key}
             }
-            if ($fixeddom) {
-                $instcodetitle .= '<br />('.$codedom.')';
+        } elsif ($item eq 'course') {
+            if ($name eq 'crstype') {
+                $checkcrstypes->{$value} = $Apache::lonnet::needsrelease{$key};
             }
         }
     }
-    my $output = qq|
-<form method="post" name="filterpicker" action="$action">
-<input type="hidden" name="form" value="$formname" />
-|;
-    if ($formname eq 'modifycourse') {
-        $output .= '<input type="hidden" name="phase" value="courselist" />'."\n".
-                   '<input type="hidden" name="prevphase" value="'.
-                   $prevphase.'" />'."\n";
-    } elsif ($formname eq 'quotacheck') {
-        $output .= qq|
-<input type="hidden" name="sortby" value="" />
-<input type="hidden" name="sortorder" value="" />
-|;
-    } else {
-        my $name_input;
-        if ($cnameelement ne '') {
-            $name_input = '<input type="hidden" name="cnameelement" value="'.
-                          $cnameelement.'" />';
-        }
-        $output .= qq|
-<input type="hidden" name="cnumelement" value="$cnumelement" />
-<input type="hidden" name="cdomelement" value="$cdomelement" />
-$name_input
-$roleelement
-$multelement
-$typeelement
-|;
-        if ($formname eq 'portform') {
-            $output .= '<input type="hidden" name="setroles" value="'.$setroles.'" />'."\n";
-        }
-    }
-    if ($fixeddom) {
-        $output .= '<input type="hidden" name="fixeddom" value="'.$fixeddom.'" />'."\n";
-    }
-    $output .= "<br />\n".&Apache::lonhtmlcommon::start_pick_box();
-    if ($sincefilterform) {
-        $output .= &Apache::lonhtmlcommon::row_title($lt{'cac'})
-                  .$sincefilterform
-                  .&Apache::lonhtmlcommon::row_closure();
-    }
-    if ($createdfilterform) {
-        $output .= &Apache::lonhtmlcommon::row_title($lt{'ccr'})
-                  .$createdfilterform
-                  .&Apache::lonhtmlcommon::row_closure();
-    }
-    if ($domainselectform) {
-        $output .= &Apache::lonhtmlcommon::row_title($lt{'cdo'})
-                  .$domainselectform
-                  .&Apache::lonhtmlcommon::row_closure();
-    }
-    if ($typeselectform) {
-        if (($formname eq 'ccrs') || ($formname eq 'requestcrs')) {
-            $output .= $typeselectform;
-        } else {
-            $output .= &Apache::lonhtmlcommon::row_title($lt{'cog'})
-                      .$typeselectform
-                      .&Apache::lonhtmlcommon::row_closure();
-        }
-    }
-    if ($instcodeform) {
-        $output .= &Apache::lonhtmlcommon::row_title($instcodetitle)
-                  .$instcodeform
-                  .&Apache::lonhtmlcommon::row_closure();
-    }
-    if (exists($filter->{'ownerfilter'})) {
-        $output .= &Apache::lonhtmlcommon::row_title($lt{'cow'}).
-                   '<table><tr><td>'.&mt('Username').'<br />'.
-                   '<input type="text" name="ownerfilter" size="20" value="'.
-                   $list->{'ownerfilter'}.'" /></td><td>'.&mt('Domain').'<br />'.
-                   $ownerdomselectform.'</td></tr></table>'.
-                   &Apache::lonhtmlcommon::row_closure();
-    }
-    if (exists($filter->{'personfilter'})) {
-        $output .= &Apache::lonhtmlcommon::row_title($lt{'cop'}).
-                   '<table><tr><td>'.&mt('Username').'<br />'.
-                   '<input type="text" name="personfilter" size="20" value="'.
-                   $list->{'personfilter'}.'" /></td><td>'.&mt('Domain').'<br />'.
-                   $persondomselectform.'</td></tr></table>'.
-                   &Apache::lonhtmlcommon::row_closure();
-    }
-    if (exists($filter->{'coursefilter'})) {
-        $output .= &Apache::lonhtmlcommon::row_title(&mt('LON-CAPA course ID'))
-                  .'<input type="text" name="coursefilter" size="25" value="'
-                  .$list->{'coursefilter'}.'" />'
-                  .&Apache::lonhtmlcommon::row_closure();
-    }
-    if ($cloneableonlyform) {
-        $output .= &Apache::lonhtmlcommon::row_title($cloneabletitle).
-                   $cloneableonlyform.&Apache::lonhtmlcommon::row_closure();
-    }
-    if (exists($filter->{'descriptfilter'})) {
-        $output .= &Apache::lonhtmlcommon::row_title($lt{'cde'})
-                  .'<input type="text" name="descriptfilter" size="40" value="'
-                  .$list->{'descriptfilter'}.'" />'
-                  .&Apache::lonhtmlcommon::row_closure(1);
-    }
-    $output .= &Apache::lonhtmlcommon::end_pick_box().'<p>'.$clonetext."\n".
-               '<input type="hidden" name="updater" value="" />'."\n".
-               '<input type="submit" name="gosearch" value="'.
-               &mt('Search').'" /></p>'."\n".'</form>'."\n".'<hr />'."\n";
-    return $jscript.$clonewarning.$output;
-}
-
-=pod
-
-=item * &timebased_select_form()
-
-Create markup for a dropdown list used to select a time-based
-filter e.g., Course Activity, Course Created, when searching for courses
-or communities
-
-Inputs:
-
-item - name of form element (sincefilter or createdfilter)
-
-filter - anonymous hash of criteria and their values
-
-Returns: HTML for a select box contained a blank, then six time selections,
-         with value set in incoming form variables currently selected.
-
-Side Effects: None
-
-=cut
-
-sub timebased_select_form {
-    my ($item,$filter) = @_;
-    if (ref($filter) eq 'HASH') {
-        $filter->{$item} =~ s/[^\d-]//g;
-        if (!$filter->{$item}) { $filter->{$item}=-1; }
-        return &select_form(
-                            $filter->{$item},
-                            $item,
-                            {      '-1' => '',
-                                '86400' => &mt('today'),
-                               '604800' => &mt('last week'),
-                              '2592000' => &mt('last month'),
-                              '7776000' => &mt('last three months'),
-                             '15552000' => &mt('last six months'),
-                             '31104000' => &mt('last year'),
-                    'select_form_order' =>
-                           ['-1','86400','604800','2592000','7776000',
-                            '15552000','31104000']});
-    }
-}
-
-=pod
-
-=item * &js_changer()
-
-Create script tag containing Javascript used to submit course search form
-when course type or domain is changed, and also to hide 'Searching ...' on
-page load completion for page showing search result.
-
-Inputs: None
-
-Returns: markup containing updateFilters() and hideSearching() javascript functions.
-
-Side Effects: None
-
-=cut
-
-sub js_changer {
-    return <<ENDJS;
-<script type="text/javascript">
-// <![CDATA[
-function updateFilters(caller) {
-    if (typeof(caller) != "undefined") {
-        document.filterpicker.updater.value = caller.name;
-    }
-    document.filterpicker.submit();
-}
-
-function hideSearching() {
-    if (document.getElementById('searching')) {
-        document.getElementById('searching').style.display = 'none';
-    }
+    ($anonsurvey->{major},$anonsurvey->{minor}) = split(/\./,$Apache::lonnet::needsrelease{'parameter:type:anonsurvey'});
+    ($randomizetry->{major},$randomizetry->{minor}) = split(/\./,$Apache::lonnet::needsrelease{'parameter:type:randomizetry'});
     return;
 }
 
-// ]]>
-</script>
-
-ENDJS
-}
-
-=pod
-
-=item * &search_courses()
-
-Process selected filters form course search form and pass to lonnet::courseiddump
-to retrieve a hash for which keys are courseIDs which match the selected filters.
-
-Inputs:
-
-dom - domain being searched
-
-type - course type ('Course' or 'Community' or '.' if any).
-
-filter - anonymous hash of criteria and their values
-
-numtitles - for institutional codes - number of categories
-
-cloneruname - optional username of new course owner
-
-clonerudom - optional domain of new course owner
-
-domcloner - optional "domcloner" flag; has value=1 if user has ccc priv in domain being filtered by,
-            (used when DC is using course creation form)
-
-codetitles - reference to array of titles of components in institutional codes (official courses).
-
-cc_clone - escaped comma separated list of courses for which course cloner has active CC role
-           (and so can clone automatically)
-
-reqcrsdom - domain of new course, where search_courses is used to identify potential courses to clone
-
-reqinstcode - institutional code of new course, where search_courses is used to identify potential
-              courses to clone
-
-Returns: %courses - hash of courses satisfying search criteria, keys = course IDs, values are corresponding colon-separated escaped description, institutional code, owner and type.
-
-
-Side Effects: None
-
-=cut
-
-
-sub search_courses {
-    my ($dom,$type,$filter,$numtitles,$cloneruname,$clonerudom,$domcloner,$codetitles,
-        $cc_clone,$reqcrsdom,$reqinstcode) = @_;
-    my (%courses,%showcourses,$cloner);
-    if (($filter->{'ownerfilter'} ne '') ||
-        ($filter->{'ownerdomfilter'} ne '')) {
-        $filter->{'combownerfilter'} = $filter->{'ownerfilter'}.':'.
-                                       $filter->{'ownerdomfilter'};
-    }
-    foreach my $item ('descriptfilter','coursefilter','combownerfilter') {
-        if (!$filter->{$item}) {
-            $filter->{$item}='.';
-        }
-    }
-    my $now = time;
-    my $timefilter =
-       ($filter->{'sincefilter'}==-1?1:$now-$filter->{'sincefilter'});
-    my ($createdbefore,$createdafter);
-    if (($filter->{'createdfilter'} ne '') && ($filter->{'createdfilter'} !=-1)) {
-        $createdbefore = $now;
-        $createdafter = $now-$filter->{'createdfilter'};
-    }
-    my ($instcodefilter,$regexpok);
-    if ($numtitles) {
-        if ($env{'form.official'} eq 'on') {
-            $instcodefilter =
-                &Apache::courseclassifier::instcode_search_str($dom,$numtitles,$codetitles);
-            $regexpok = 1;
-        } elsif ($env{'form.official'} eq 'off') {
-            $instcodefilter = &Apache::courseclassifier::instcode_search_str($dom,$numtitles,$codetitles);
-            unless ($instcodefilter eq '') {
-                $regexpok = -1;
-            }
-        }
-    } else {
-        $instcodefilter = $filter->{'instcodefilter'};
-    }
-    if ($instcodefilter eq '') { $instcodefilter = '.'; }
-    if ($type eq '') { $type = '.'; }
-
-    if (($clonerudom ne '') && ($cloneruname ne '')) {
-        $cloner = $cloneruname.':'.$clonerudom;
-    }
-    %courses = &Apache::lonnet::courseiddump($dom,
-                                             $filter->{'descriptfilter'},
-                                             $timefilter,
-                                             $instcodefilter,
-                                             $filter->{'combownerfilter'},
-                                             $filter->{'coursefilter'},
-                                             undef,undef,$type,$regexpok,undef,undef,
-                                             undef,undef,$cloner,$cc_clone,
-                                             $filter->{'cloneableonly'},
-                                             $createdbefore,$createdafter,undef,
-                                             $domcloner,undef,$reqcrsdom,$reqinstcode);
-    if (($filter->{'personfilter'} ne '') && ($filter->{'persondomfilter'} ne '')) {
-        my $ccrole;
-        if ($type eq 'Community') {
-            $ccrole = 'co';
-        } else {
-            $ccrole = 'cc';
-        }
-        my %rolehash = &Apache::lonnet::get_my_roles($filter->{'personfilter'},
-                                                     $filter->{'persondomfilter'},
-                                                     'userroles',undef,
-                                                     [$ccrole,'in','ad','ep','ta','cr'],
-                                                     $dom);
-        foreach my $role (keys(%rolehash)) {
-            my ($cnum,$cdom,$courserole) = split(':',$role);
-            my $cid = $cdom.'_'.$cnum;
-            if (exists($courses{$cid})) {
-                if (ref($courses{$cid}) eq 'HASH') {
-                    if (ref($courses{$cid}{roles}) eq 'ARRAY') {
-                        if (!grep(/^\Q$courserole\E$/,@{$courses{$cid}{roles}})) {
-                            push(@{$courses{$cid}{roles}},$courserole);
-                        }
-                    } else {
-                        $courses{$cid}{roles} = [$courserole];
-                    }
-                    $showcourses{$cid} = $courses{$cid};
-                }
-            }
-        }
-        %courses = %showcourses;
-    }
-    return %courses;
-}
-
-=pod
-
-=back
-
-=head1 Routines for version requirements for current course.
-
-=over 4
-
-=item * &check_release_required()
-
-Compares required LON-CAPA version with version on server, and
-if required version is newer looks for a server with the required version.
-
-Looks first at servers in user's owen domain; if none suitable, looks at
-servers in course's domain are permitted to host sessions for user's domain.
-
-Inputs:
-
-$loncaparev - Version on current server (format: Major.Minor.Subrelease-datestamp)
-
-$courseid - Course ID of current course
-
-$rolecode - User's current role in course (for switchserver query string).
-
-$required - LON-CAPA version needed by course (format: Major.Minor).
-
-
-Returns:
-
-$switchserver - query string tp append to /adm/switchserver call (if
-                current server's LON-CAPA version is too old.
-
-$warning - Message is displayed if no suitable server could be found.
-
-=cut
-
-sub check_release_required {
-    my ($loncaparev,$courseid,$rolecode,$required) = @_;
-    my ($switchserver,$warning);
-    if ($required ne '') {
-        my ($reqdmajor,$reqdminor) = ($required =~ /^(\d+)\.(\d+)$/);
-        my ($major,$minor) = ($loncaparev =~ /^\'?(\d+)\.(\d+)\.[\w.\-]+\'?$/);
-        if ($reqdmajor ne '' && $reqdminor ne '') {
-            my $otherserver;
-            if (($major eq '' && $minor eq '') ||
-                (($reqdmajor > $major) || (($reqdmajor == $major) && ($reqdminor > $minor)))) {
-                my ($userdomserver) = &Apache::lonnet::choose_server($env{'user.domain'},undef,$required,1);
-                my $switchlcrev =
-                    &Apache::lonnet::get_server_loncaparev($env{'user.domain'},
-                                                           $userdomserver);
-                my ($swmajor,$swminor) = ($switchlcrev =~ /^\'?(\d+)\.(\d+)\.[\w.\-]+\'?$/);
-                if (($swmajor eq '' && $swminor eq '') || ($reqdmajor > $swmajor) ||
-                    (($reqdmajor == $swmajor) && ($reqdminor > $swminor))) {
-                    my $cdom = $env{'course.'.$courseid.'.domain'};
-                    if ($cdom ne $env{'user.domain'}) {
-                        my ($coursedomserver,$coursehostname) = &Apache::lonnet::choose_server($cdom,undef,$required,1);
-                        my $serverhomeID = &Apache::lonnet::get_server_homeID($coursehostname);
-                        my $serverhomedom = &Apache::lonnet::host_domain($serverhomeID);
-                        my %defdomdefaults = &Apache::lonnet::get_domain_defaults($serverhomedom);
-                        my %udomdefaults = &Apache::lonnet::get_domain_defaults($env{'user.domain'});
-                        my $remoterev = &Apache::lonnet::get_server_loncaparev($serverhomedom,$coursedomserver);
-                        my $canhost =
-                            &Apache::lonnet::can_host_session($env{'user.domain'},
-                                                              $coursedomserver,
-                                                              $remoterev,
-                                                              $udomdefaults{'remotesessions'},
-                                                              $defdomdefaults{'hostedsessions'});
-
-                        if ($canhost) {
-                            $otherserver = $coursedomserver;
-                        } else {
-                            $warning = &mt('Requires LON-CAPA version [_1].',$env{'course.'.$courseid.'.internal.releaserequired'}).'<br />'. &mt("No suitable server could be found amongst servers in either your own domain or in the course's domain.");
-                        }
-                    } else {
-                        $warning = &mt('Requires LON-CAPA version [_1].',$env{'course.'.$courseid.'.internal.releaserequired'}).'<br />'.&mt("No suitable server could be found amongst servers in your own domain (which is also the course's domain).");
-                    }
-                } else {
-                    $otherserver = $userdomserver;
-                }
-            }
-            if ($otherserver ne '') {
-                $switchserver = 'otherserver='.$otherserver.'&amp;role='.$rolecode;
-            }
-        }
-    }
-    return ($switchserver,$warning);
-}
-
-=pod
-
-=item * &check_release_result()
-
-Inputs:
-
-$switchwarning - Warning message if no suitable server found to host session.
-
-$switchserver - query string to append to /adm/switchserver containing lonHostID
-                and current role.
-
-Returns: HTML to display with information about requirement to switch server.
-         Either displaying warning with link to Roles/Courses screen or
-         display link to switchserver.
-
-=cut
-
-sub check_release_result {
-    my ($switchwarning,$switchserver) = @_;
-    my $output = &start_page('Selected course unavailable on this server').
-                 '<p class="LC_warning">';
-    if ($switchwarning) {
-        $output .= $switchwarning.'<br /><a href="/adm/roles">';
-        if (&show_course()) {
-            $output .= &mt('Display courses');
-        } else {
-            $output .= &mt('Display roles');
-        }
-        $output .= '</a>';
-    } elsif ($switchserver) {
-        $output .= &mt('This course requires a newer version of LON-CAPA than is installed on this server.').
-                   '<br />'.
-                   '<a href="/adm/switchserver?'.$switchserver.'">'.
-                   &mt('Switch Server').
-                   '</a>';
-    }
-    $output .= '</p>'.&end_page();
-    return $output;
-}
-
-=pod
-
-=item * &needs_coursereinit()
-
-Determine if course contents stored for user's session needs to be
-refreshed, because content has changed since "Big Hash" last tied.
-
-Check for change is made if time last checked is more than 10 minutes ago
-(by default).
-
-Inputs:
-
-$loncaparev - Version on current server (format: Major.Minor.Subrelease-datestamp)
-
-$interval (optional) - Time which may elapse (in s) between last check for content
-                       change in current course. (default: 600 s).
-
-Returns: an array; first element is:
-
-=over 4
-
-'switch' - if content updates mean user's session
-           needs to be switched to a server running a newer LON-CAPA version
-
-'update' - if course session needs to be refreshed (i.e., Big Hash needs to be reloaded)
-           on current server hosting user's session
-
-''       - if no action required.
-
-=back
-
-If first item element is 'switch':
-
-second item is $switchwarning - Warning message if no suitable server found to host session.
-
-third item is $switchserver - query string to append to /adm/switchserver containing lonHostID
-                              and current role.
-
-otherwise: no other elements returned.
-
-=back
-
-=cut
-
-sub needs_coursereinit {
-    my ($loncaparev,$interval) = @_;
-    return() unless ($env{'request.course.id'} && $env{'request.course.tied'});
-    my $cdom = $env{'course.'.$env{'request.course.id'}.'.domain'};
-    my $cnum = $env{'course.'.$env{'request.course.id'}.'.num'};
-    my $now = time;
-    if ($interval eq '') {
-        $interval = 600;
-    }
-    if (($now-$env{'request.course.timechecked'})>$interval) {
-        my $lastchange = &Apache::lonnet::get_coursechange($cdom,$cnum);
-        &Apache::lonnet::appenv({'request.course.timechecked'=>$now});
-        if ($lastchange > $env{'request.course.tied'}) {
-            my %curr_reqd_hash = &Apache::lonnet::userenvironment($cdom,$cnum,'internal.releaserequired');
-            if ($curr_reqd_hash{'internal.releaserequired'} ne '') {
-                my $required = $env{'course.'.$cdom.'_'.$cnum.'.internal.releaserequired'};
-                if ($curr_reqd_hash{'internal.releaserequired'} ne $required) {
-                    &Apache::lonnet::appenv({'course.'.$cdom.'_'.$cnum.'.internal.releaserequired' =>
-                                             $curr_reqd_hash{'internal.releaserequired'}});
-                    my ($switchserver,$switchwarning) =
-                        &check_release_required($loncaparev,$cdom.'_'.$cnum,$env{'request.role'},
-                                                $curr_reqd_hash{'internal.releaserequired'});
-                    if ($switchwarning ne '' || $switchserver ne '') {
-                        return ('switch',$switchwarning,$switchserver);
-                    }
-                }
-            }
-            return ('update');
-        }
-    }
-    return ();
-}
-
 sub update_content_constraints {
     my ($cdom,$cnum,$chome,$cid) = @_;
     my %curr_reqd_hash = &Apache::lonnet::userenvironment($cdom,$cnum,'internal.releaserequired');
@@ -16699,32 +14123,6 @@ sub update_content_constraints {
     return;
 }
 
-sub allmaps_incourse {
-    my ($cdom,$cnum,$chome,$cid) = @_;
-    if ($cdom eq '' || $cnum eq '' || $chome eq '' || $cid eq '') {
-        $cid = $env{'request.course.id'};
-        $cdom = $env{'course.'.$cid.'.domain'};
-        $cnum = $env{'course.'.$cid.'.num'};
-        $chome = $env{'course.'.$cid.'.home'};
-    }
-    my %allmaps = ();
-    my $lastchange =
-        &Apache::lonnet::get_coursechange($cdom,$cnum);
-    if ($lastchange > $env{'request.course.tied'}) {
-        my ($furl,$ferr) = &Apache::lonuserstate::readmap("$cdom/$cnum");
-        unless ($ferr) {
-            &update_content_constraints($cdom,$cnum,$chome,$cid);
-        }
-    }
-    my $navmap = Apache::lonnavmaps::navmap->new();
-    if (defined($navmap)) {
-        foreach my $res ($navmap->retrieveResources(undef,sub { $_[0]->is_map() },1,0,1)) {
-            $allmaps{$res->src()} = 1;
-        }
-    }
-    return \%allmaps;
-}
-
 sub parse_supplemental_title {
     my ($title) = @_;
 
@@ -16748,33 +14146,9 @@ sub parse_supplemental_title {
     return $title;
 }
 
-sub recurse_supplemental {
-    my ($cnum,$cdom,$suppmap,$numfiles,$errors) = @_;
-    if ($suppmap) {
-        my ($errtext,$fatal) = &LONCAPA::map::mapread('/uploaded/'.$cdom.'/'.$cnum.'/'.$suppmap);
-        if ($fatal) {
-            $errors ++;
-        } else {
-            if ($#LONCAPA::map::resources > 0) {
-                foreach my $res (@LONCAPA::map::resources) {
-                    my ($title,$src,$ext,$type,$status)=split(/\:/,$res);
-                    if (($src ne '') && ($status eq 'res')) {
-                        if ($src =~ m{^\Q/uploaded/$cdom/$cnum/\E(supplemental_\d+\.sequence)$}) {
-                            ($numfiles,$errors) = &recurse_supplemental($cnum,$cdom,$1,$numfiles,$errors);
-                        } else {
-                            $numfiles ++;
-                        }
-                    }
-                }
-            }
-        }
-    }
-    return ($numfiles,$errors);
-}
-
 sub symb_to_docspath {
-    my ($symb,$navmapref) = @_;
-    return unless ($symb && ref($navmapref));
+    my ($symb) = @_;
+    return unless ($symb);
     my ($mapurl,$id,$resurl) = &Apache::lonnet::decode_symb($symb);
     if ($resurl=~/\.(sequence|page)$/) {
         $mapurl=$resurl;
@@ -16782,11 +14156,9 @@ sub symb_to_docspath {
         $mapurl=$env{'course.'.$env{'request.course.id'}.'.url'};
     }
     my $mapresobj;
-    unless (ref($$navmapref)) {
-        $$navmapref = Apache::lonnavmaps::navmap->new();
-    }
-    if (ref($$navmapref)) {
-        $mapresobj = $$navmapref->getResourceByUrl($mapurl);
+    my $navmap = Apache::lonnavmaps::navmap->new();
+    if (ref($navmap)) {
+        $mapresobj = $navmap->getResourceByUrl($mapurl);
     }
     $mapurl=~s{^.*/([^/]+)\.(\w+)$}{$1};
     my $type=$2;
@@ -16796,14 +14168,14 @@ sub symb_to_docspath {
         if ($pcslist ne '') {
             foreach my $pc (split(/,/,$pcslist)) {
                 next if ($pc <= 1);
-                my $res = $$navmapref->getByMapPc($pc);
+                my $res = $navmap->getByMapPc($pc);
                 if (ref($res)) {
                     my $thisurl = $res->src();
                     $thisurl=~s{^.*/([^/]+)\.\w+$}{$1};
                     my $thistitle = $res->title();
                     $path .= '&'.
                              &Apache::lonhtmlcommon::entity_encode($thisurl).'&'.
-                             &escape($thistitle).
+                             &Apache::lonhtmlcommon::entity_encode($thistitle).
                              ':'.$res->randompick().
                              ':'.$res->randomout().
                              ':'.$res->encrypted().
@@ -16815,11 +14187,11 @@ sub symb_to_docspath {
         $path =~ s/^\&//;
         my $maptitle = $mapresobj->title();
         if ($mapurl eq 'default') {
-            $maptitle = 'Main Content';
+            $maptitle = 'Main Course Documents';
         }
         $path .= (($path ne '')? '&' : '').
                  &Apache::lonhtmlcommon::entity_encode($mapurl).'&'.
-                 &escape($maptitle).
+                 &Apache::lonhtmlcommon::entity_encode($maptitle).
                  ':'.$mapresobj->randompick().
                  ':'.$mapresobj->randomout().
                  ':'.$mapresobj->encrypted().
@@ -16829,46 +14201,45 @@ sub symb_to_docspath {
         my $maptitle = &Apache::lonnet::gettitle($mapurl);
         my $ispage = (($type eq 'page')? 1 : '');
         if ($mapurl eq 'default') {
-            $maptitle = 'Main Content';
+            $maptitle = 'Main Course Documents';
         }
         $path = &Apache::lonhtmlcommon::entity_encode($mapurl).'&'.
-                &escape($maptitle).':::::'.$ispage;
+                &Apache::lonhtmlcommon::entity_encode($maptitle).':::::'.$ispage;
     }
     unless ($mapurl eq 'default') {
         $path = 'default&'.
-                &escape('Main Content').
+                &Apache::lonhtmlcommon::entity_encode('Main Course Documents').
                 ':::::&'.$path;
     }
     return $path;
 }
 
 sub captcha_display {
-    my ($context,$lonhost,$defdom) = @_;
+    my ($context,$lonhost) = @_;
     my ($output,$error);
-    my ($captcha,$pubkey,$privkey,$version) =
-        &get_captcha_config($context,$lonhost,$defdom);
+    my ($captcha,$pubkey,$privkey) = &get_captcha_config($context,$lonhost);
     if ($captcha eq 'original') {
         $output = &create_captcha();
         unless ($output) {
-            $error = 'captcha';
+            $error = 'captcha'; 
         }
     } elsif ($captcha eq 'recaptcha') {
-        $output = &create_recaptcha($pubkey,$version);
+        $output = &create_recaptcha($pubkey);
         unless ($output) {
-            $error = 'recaptcha';
+            $error = 'recaptcha'; 
         }
     }
-    return ($output,$error,$captcha,$version);
+    return ($output,$error);
 }
 
 sub captcha_response {
-    my ($context,$lonhost,$defdom) = @_;
+    my ($context,$lonhost) = @_;
     my ($captcha_chk,$captcha_error);
-    my ($captcha,$pubkey,$privkey,$version) = &get_captcha_config($context,$lonhost,$defdom);
+    my ($captcha,$pubkey,$privkey) = &get_captcha_config($context,$lonhost);
     if ($captcha eq 'original') {
         ($captcha_chk,$captcha_error) = &check_captcha();
     } elsif ($captcha eq 'recaptcha') {
-        $captcha_chk = &check_recaptcha($privkey,$version);
+        $captcha_chk = &check_recaptcha($privkey);
     } else {
         $captcha_chk = 1;
     }
@@ -16876,8 +14247,8 @@ sub captcha_response {
 }
 
 sub get_captcha_config {
-    my ($context,$lonhost,$dom_in_effect) = @_;
-    my ($captcha,$pubkey,$privkey,$version,$hashtocheck);
+    my ($context,$lonhost) = @_;
+    my ($captcha,$pubkey,$privkey,$hashtocheck);
     my $hostname = &Apache::lonnet::hostname($lonhost);
     my $serverhomeID = &Apache::lonnet::get_server_homeID($hostname);
     my $serverhomedom = &Apache::lonnet::host_domain($serverhomeID);
@@ -16893,10 +14264,6 @@ sub get_captcha_config {
                     }
                     if ($privkey && $pubkey) {
                         $captcha = 'recaptcha';
-                        $version = $hashtocheck->{'recaptchaversion'};
-                        if ($version ne '2') {
-                            $version = 1;
-                        }
                     } else {
                         $captcha = 'original';
                     }
@@ -16914,39 +14281,14 @@ sub get_captcha_config {
             $privkey = $domconfhash{$serverhomedom.'.login.recaptchakeys_private'};
             if ($privkey && $pubkey) {
                 $captcha = 'recaptcha';
-                $version = $domconfhash{$serverhomedom.'.login.recaptchaversion'};
-                if ($version ne '2') {
-                    $version = 1;
-                }
             } else {
                 $captcha = 'original';
             }
         } elsif ($domconfhash{$serverhomedom.'.login.captcha'} eq 'original') {
             $captcha = 'original';
         }
-    } elsif ($context eq 'passwords') {
-        if ($dom_in_effect) {
-            my %passwdconf = &Apache::lonnet::get_passwdconf($dom_in_effect);
-            if ($passwdconf{'captcha'} eq 'recaptcha') {
-                if (ref($passwdconf{'recaptchakeys'}) eq 'HASH') {
-                    $pubkey = $passwdconf{'recaptchakeys'}{'public'};
-                    $privkey = $passwdconf{'recaptchakeys'}{'private'};
-                }
-                if ($privkey && $pubkey) {
-                    $captcha = 'recaptcha';
-                    $version = $passwdconf{'recaptchaversion'};
-                    if ($version ne '2') {
-                        $version = 1;
-                    }
-                } else {
-                    $captcha = 'original';
-                }
-            } elsif ($passwdconf{'captcha'} ne 'notused') {
-                $captcha = 'original';
-            }
-        }
     }
-    return ($captcha,$pubkey,$privkey,$version);
+    return ($captcha,$pubkey,$privkey);
 }
 
 sub create_captcha {
@@ -16963,9 +14305,8 @@ sub create_captcha {
         if (-e $Apache::lonnet::perlvar{'lonCaptchaDir'}.'/'.$md5sum.'.png') {
             $output = '<input type="hidden" name="crypt" value="'.$md5sum.'" />'."\n".
                       &mt('Type in the letters/numbers shown below').'&nbsp;'.
-                      '<input type="text" size="5" name="code" value="" autocomplete="off" />'.
-                      '<br />'.
-                      '<img src="'.$captcha_params{'www_output_dir'}.'/'.$md5sum.'.png" alt="captcha" />';
+                     '<input type="text" size="5" name="code" value="" /><br />'.
+                     '<img src="'.$captcha_params{'www_output_dir'}.'/'.$md5sum.'.png" />';
             last;
         }
     }
@@ -16991,7 +14332,7 @@ sub check_captcha {
                       output_folder => $captcha_params{'output_dir'},
                       data_folder   => $captcha_params{'db_dir'},
                   );
-    $captcha_chk = $captcha->check_code($code,$md5sum);
+    my $captcha_chk = $captcha->check_code($code,$md5sum);
     my %captcha_hash = (
                         0       => 'Code not checked (file error)',
                        -1      => 'Failed: code expired',
@@ -17005,317 +14346,37 @@ sub check_captcha {
 }
 
 sub create_recaptcha {
-    my ($pubkey,$version) = @_;
-    if ($version >= 2) {
-        return '<div class="g-recaptcha" data-sitekey="'.$pubkey.'"></div>';
-    } else {
-        my $use_ssl;
-        if ($ENV{'SERVER_PORT'} == 443) {
-            $use_ssl = 1;
-        }
-        my $captcha = Captcha::reCAPTCHA->new;
-        return $captcha->get_options_setter({theme => 'white'})."\n".
-               $captcha->get_html($pubkey,undef,$use_ssl).
-               &mt('If the text is hard to read, [_1] will replace them.',
-                   '<img src="/res/adm/pages/refresh.gif" alt="reCAPTCHA refresh" />').
-               '<br /><br />';
-     }
+    my ($pubkey) = @_;
+    my $captcha = Captcha::reCAPTCHA->new;
+    return $captcha->get_options_setter({theme => 'white'})."\n".
+           $captcha->get_html($pubkey).
+           &mt('If either word is hard to read, [_1] will replace them.',
+               '<image src="/res/adm/pages/refresh.gif" alt="reCAPTCHA refresh" />').
+           '<br /><br />';
 }
 
 sub check_recaptcha {
-    my ($privkey,$version) = @_;
+    my ($privkey) = @_;
     my $captcha_chk;
-    if ($version >= 2) {
-        my $ua = LWP::UserAgent->new;
-        $ua->timeout(10);
-        my %info = (
-                     secret   => $privkey,
-                     response => $env{'form.g-recaptcha-response'},
-                     remoteip => $ENV{'REMOTE_ADDR'},
-                   );
-        my $response = $ua->post('https://www.google.com/recaptcha/api/siteverify',\%info);
-        if ($response->is_success)  {
-            my $data = JSON::DWIW->from_json($response->decoded_content);
-            if (ref($data) eq 'HASH') {
-                if ($data->{'success'}) {
-                    $captcha_chk = 1;
-                }
-            }
-        }
-    } else {
-        my $captcha = Captcha::reCAPTCHA->new;
-        my $captcha_result =
-            $captcha->check_answer(
-                                    $privkey,
-                                    $ENV{'REMOTE_ADDR'},
-                                    $env{'form.recaptcha_challenge_field'},
-                                    $env{'form.recaptcha_response_field'},
-                                  );
-        if ($captcha_result->{is_valid}) {
-            $captcha_chk = 1;
-        }
+    my $captcha = Captcha::reCAPTCHA->new;
+    my $captcha_result =
+        $captcha->check_answer(
+                                $privkey,
+                                $ENV{'REMOTE_ADDR'},
+                                $env{'form.recaptcha_challenge_field'},
+                                $env{'form.recaptcha_response_field'},
+                              );
+    if ($captcha_result->{is_valid}) {
+        $captcha_chk = 1;
     }
     return $captcha_chk;
 }
 
-sub emailusername_info {
-    my @fields = ('firstname','lastname','institution','web','location','officialemail','id');
-    my %titles = &Apache::lonlocal::texthash (
-                     lastname      => 'Last Name',
-                     firstname     => 'First Name',
-                     institution   => 'School/college/university',
-                     location      => "School's city, state/province, country",
-                     web           => "School's web address",
-                     officialemail => 'E-mail address at institution (if different)',
-                     id            => 'Student/Employee ID',
-                 );
-    return (\@fields,\%titles);
-}
+=pod
 
-sub cleanup_html {
-    my ($incoming) = @_;
-    my $outgoing;
-    if ($incoming ne '') {
-        $outgoing = $incoming;
-        $outgoing =~ s/;/&#059;/g;
-        $outgoing =~ s/\#/&#035;/g;
-        $outgoing =~ s/\&/&#038;/g;
-        $outgoing =~ s/</&#060;/g;
-        $outgoing =~ s/>/&#062;/g;
-        $outgoing =~ s/\(/&#040/g;
-        $outgoing =~ s/\)/&#041;/g;
-        $outgoing =~ s/"/&#034;/g;
-        $outgoing =~ s/'/&#039;/g;
-        $outgoing =~ s/\$/&#036;/g;
-        $outgoing =~ s{/}{&#047;}g;
-        $outgoing =~ s/=/&#061;/g;
-        $outgoing =~ s/\\/&#092;/g
-    }
-    return $outgoing;
-}
-
-# Checks for critical messages and returns a redirect url if one exists.
-# $interval indicates how often to check for messages.
-sub critical_redirect {
-    my ($interval) = @_;
-    if ((time-$env{'user.criticalcheck.time'})>$interval) {
-        my @what=&Apache::lonnet::dump('critical', $env{'user.domain'},
-                                        $env{'user.name'});
-        &Apache::lonnet::appenv({'user.criticalcheck.time'=>time});
-        my $redirecturl;
-        if ($what[0]) {
-            if (($what[0] ne 'con_lost') && ($what[0]!~/^error\:/)) {
-                $redirecturl='/adm/email?critical=display';
-                my $url=&Apache::lonnet::absolute_url().$redirecturl;
-                return (1, $url);
-            }
-        }
-    }
-    return ();
-}
+=back
 
-# Use:
-#   my $answer=reply("encrypt:passwd:$udom:$uname:$upass",$tryserver);
-#
-##################################################
-#          password associated functions         #
-##################################################
-sub des_keys {
-    # Make a new key for DES encryption.
-    # Each key has two parts which are returned separately.
-    # Please note:  Each key must be passed through the &hex function
-    # before it is output to the web browser.  The hex versions cannot
-    # be used to decrypt.
-    my @hexstr=('0','1','2','3','4','5','6','7',
-                '8','9','a','b','c','d','e','f');
-    my $lkey='';
-    for (0..7) {
-        $lkey.=$hexstr[rand(15)];
-    }
-    my $ukey='';
-    for (0..7) {
-        $ukey.=$hexstr[rand(15)];
-    }
-    return ($lkey,$ukey);
-}
-
-sub des_decrypt {
-    my ($key,$cyphertext) = @_;
-    my $keybin=pack("H16",$key);
-    my $cypher;
-    if ($Crypt::DES::VERSION>=2.03) {
-        $cypher=new Crypt::DES $keybin;
-    } else {
-        $cypher=new DES $keybin;
-    }
-    my $plaintext='';
-    my $cypherlength = length($cyphertext);
-    my $numchunks = int($cypherlength/32);
-    for (my $j=0; $j<$numchunks; $j++) {
-        my $start = $j*32;
-        my $cypherblock = substr($cyphertext,$start,32);
-        my $chunk =
-            $cypher->decrypt(unpack("a8",pack("H16",substr($cypherblock,0,16))));
-        $chunk .=
-            $cypher->decrypt(unpack("a8",pack("H16",substr($cypherblock,16,16))));
-        $chunk=substr($chunk,1,ord(substr($chunk,0,1)) );
-        $plaintext .= $chunk;
-    }
-    return $plaintext;
-}
-
-sub is_nonframeable {
-    my ($url,$absolute,$hostname,$ip,$nocache) = @_;
-    my ($remprotocol,$remhost) = ($url =~ m{^(https?)\://(([a-z0-9]+(-[a-z0-9]+)*\.)+[a-z]{2,})}i);
-    return if (($remprotocol eq '') || ($remhost eq ''));
-
-    $remprotocol = lc($remprotocol);
-    $remhost = lc($remhost);
-    my $remport = 80;
-    if ($remprotocol eq 'https') {
-        $remport = 443;
-    }
-    my ($result,$cached) = &Apache::lonnet::is_cached_new('noiframe',$remhost.':'.$remport);
-    if ($cached) {
-        unless ($nocache) {
-            if ($result) {
-                return 1;
-            } else {
-                return 0;
-            }
-        }
-    }
-    my $uselink;
-    my $request = new HTTP::Request('HEAD',$url);
-    my $ua = LWP::UserAgent->new;
-    $ua->timeout(5);
-    my $response=$ua->request($request);
-    if ($response->is_success()) {
-        my $secpolicy = lc($response->header('content-security-policy'));
-        my $xframeop = lc($response->header('x-frame-options'));
-        $secpolicy =~ s/^\s+|\s+$//g;
-        $xframeop =~ s/^\s+|\s+$//g;
-        if (($secpolicy ne '') || ($xframeop ne '')) {
-            my $remotehost = $remprotocol.'://'.$remhost;
-            my ($origin,$protocol,$port);
-            if ($ENV{'SERVER_PORT'} =~/^\d+$/) {
-                $port = $ENV{'SERVER_PORT'};
-            } else {
-                $port = 80;
-            }
-            if ($absolute eq '') {
-                $protocol = 'http:';
-                if ($port == 443) {
-                    $protocol = 'https:';
-                }
-                $origin = $protocol.'//'.lc($hostname);
-            } else {
-                $origin = lc($absolute);
-                ($protocol,$hostname) = ($absolute =~ m{^(https?:)//([^/]+)$});
-            }
-            if (($secpolicy) && ($secpolicy =~ /\Qframe-ancestors\E([^;]*)(;|$)/)) {
-                my $framepolicy = $1;
-                $framepolicy =~ s/^\s+|\s+$//g;
-                my @policies = split(/\s+/,$framepolicy);
-                if (@policies) {
-                    if (grep(/^\Q'none'\E$/,@policies)) {
-                        $uselink = 1;
-                    } else {
-                        $uselink = 1;
-                        if ((grep(/^\Q*\E$/,@policies)) || (grep(/^\Q$protocol\E$/,@policies)) ||
-                                (($origin ne '') && (grep(/^\Q$origin\E$/,@policies))) ||
-                                (($ip ne '') && (grep(/^\Q$ip\E$/,@policies)))) {
-                            undef($uselink);
-                        }
-                        if ($uselink) {
-                            if (grep(/^\Q'self'\E$/,@policies)) {
-                                if (($origin ne '') && ($remotehost eq $origin)) {
-                                    undef($uselink);
-                                }
-                            }
-                        }
-                        if ($uselink) {
-                            my @possok;
-                            if ($ip ne '') {
-                                push(@possok,$ip);
-                            }
-                            my $hoststr = '';
-                            foreach my $part (reverse(split(/\./,$hostname))) {
-                                if ($hoststr eq '') {
-                                    $hoststr = $part;
-                                } else {
-                                    $hoststr = "$part.$hoststr";
-                                }
-                                if ($hoststr eq $hostname) {
-                                    push(@possok,$hostname);
-                                } else {
-                                    push(@possok,"*.$hoststr");
-                                }
-                            }
-                            if (@possok) {
-                                foreach my $poss (@possok) {
-                                    last if (!$uselink);
-                                    foreach my $policy (@policies) {
-                                        if ($policy =~ m{^(\Q$protocol\E//|)\Q$poss\E(\Q:$port\E|)$}) {
-                                            undef($uselink);
-                                            last;
-                                        }
-                                    }
-                                }
-                            }
-                        }
-                    }
-                }
-            } elsif ($xframeop ne '') {
-                $uselink = 1;
-                my @policies = split(/\s*,\s*/,$xframeop);
-                if (@policies) {
-                    unless (grep(/^deny$/,@policies)) {
-                        if ($origin ne '') {
-                            if (grep(/^sameorigin$/,@policies)) {
-                                if ($remotehost eq $origin) {
-                                    undef($uselink);
-                                }
-                            }
-                            if ($uselink) {
-                                foreach my $policy (@policies) {
-                                    if ($policy =~ /^allow-from\s*(.+)$/) {
-                                        my $allowfrom = $1;
-                                        if (($allowfrom ne '') && ($allowfrom eq $origin)) {
-                                            undef($uselink);
-                                            last;
-                                        }
-                                    }
-                                }
-                            }
-                        }
-                    }
-                }
-            }
-        }
-    }
-    if ($nocache) {
-        if ($cached) {
-            my $devalidate;
-            if ($uselink && !$result) {
-                $devalidate = 1;
-            } elsif (!$uselink && $result) {
-                $devalidate = 1;
-            }
-            if ($devalidate) {
-                &Apache::lonnet::devalidate_cache_new('noiframe',$remhost.':'.$remport);
-            }
-        }
-    } else {
-        if ($uselink) {
-            $result = 1;
-        } else {
-            $result = 0;
-        }
-        &Apache::lonnet::do_cache_new('noiframe',$remhost.':'.$remport,$result,3600);
-    }
-    return $uselink;
-}
+=cut
 
 1;
 __END__;