loncom/interface/loncommon.pm - diff

Return to loncommon.pm CVS log

Up to [LON-CAPA] / loncom / interface

Diff for /loncom/interface/loncommon.pm between versions 1.1075.2.108 and 1.1131

-version 1.1075.2.108, 2016/08/13 20:20:23
+version 1.1131, 2013/05/26 22:00:24
  Line 69  use Apache::lontexconvert();
  use Apache::lonclonecourse();
  use Apache::lonuserutils();
  use Apache::lonuserstate();
- use Apache::courseclassifier();
  use LONCAPA qw(:DEFAULT :match);
  use DateTime::TimeZone;
- use DateTime::Locale;
+ use DateTime::Locale::Catalog;
- use Encode();
+ use Text::Aspell;
  use Authen::Captcha;
  use Captcha::reCAPTCHA;
- use JSON::DWIW;
- use LWP::UserAgent;
- use Crypt::DES;
- use DynaLoader; # for Crypt::DES version
  # ---------------------------------------------- Designs
  use vars qw(%defaultdesign);
- Line 164  sub ssi_with_retries {
+ Line 159  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;
- Line 198  BEGIN {
+ Line 194  BEGIN {
              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);
- Line 535  ENDAUTHORBRW
+ Line 532  ENDAUTHORBRW
  sub coursebrowser_javascript {
      my ($domainfilter,$sec_element,$formname,$role_element,$crstype,
-         $credits_element,$instcode) = @_;
+         $credits_element) = @_;
      my $wintitle = 'Course_Browser';
      if ($crstype eq 'Community') {
          $wintitle = 'Community_Browser';
- Line 586  sub coursebrowser_javascript {
+ Line 583  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;
- Line 675  if (!Array.prototype.indexOf) {
+ Line 666  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));
- Line 911  sub check_uncheck_jscript {
+ Line 902  sub check_uncheck_jscript {
  function checkAll(field) {
      if (field.length > 0) {
          for (i = 0; i < field.length; i++) {
              if (!field[i].disabled) {
                  field[i].checked = true;
              }
          }
      } else {
          if (!field.disabled) {
              field.checked = true;
          }
      }
- Line 967  sub select_datelocale {
+ Line 958  sub select_datelocale {
          }
          $output .= '> </option>';
      }
-     my @languages = &Apache::lonlocal::preferred_languages();
      my (@possibles,%locale_names);
-     my @locales = DateTime::Locale->ids();
+     my @locales = DateTime::Locale::Catalog::Locales;
-     foreach my $id (@locales) {
+     foreach my $locale (@locales) {
-         if ($id ne '') {
+         if (ref($locale) eq 'HASH') {
-             my ($en_terr,$native_terr);
+             my $id = $locale->{'id'};
-             my $loc = DateTime::Locale->load($id);
+             if ($id ne '') {
-             if (ref($loc)) {
+                 my $en_terr = $locale->{'en_territory'};
-                 $en_terr = $loc->name();
+                 my $native_terr = $locale->{'native_territory'};
-                 $native_terr = $loc->native_name();
+                 my @languages = &Apache::lonlocal::preferred_languages();
                  if (grep(/^en$/,@languages) || !@languages) {
                      if ($en_terr ne '') {
                          $locale_names{$id} = '('.$en_terr.')';
- Line 990  sub select_datelocale {
+ Line 980  sub select_datelocale {
                          $locale_names{$id} = '('.$en_terr.')';
                      }
                  }
-                 $locale_names{$id} = Encode::encode('UTF-8',$locale_names{$id});
+                 push (@possibles,$id);
-                 push(@possibles,$id);
              }
          }
      }
- Line 1002  sub select_datelocale {
+ Line 991  sub select_datelocale {
          }
          $output.=">$item";
          if ($locale_names{$item} ne '') {
-             $output.='  '.$locale_names{$item};
+             $output.="  $locale_names{$item}</option>\n";
          }
          $output.="</option>\n";
      }
- Line 1028  sub select_language {
+ Line 1017  sub select_language {
  =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
  =item * &linked_select_forms(...)
  linked_select_forms returns a string containing a <script></script> block
- Line 1248  sub help_open_topic {
+ Line 1264  sub help_open_topic {
      $topic=~s/\W/\_/g;
      if (!$stayOnPage) {
-         if ($env{'browser.mobile'}) {
+ 	$link = "javascript:openMyModal('/adm/help/${filename}.hlp',$width,$height,'yes');";
- 	    $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'))";
-         }
      } 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 {
- Line 1303  sub helpLatexCheatsheet {
+ Line 1315  sub helpLatexCheatsheet {
      unless ($not_author) {
          $out .= ' <span>'
  	       .&help_open_topic('Authoring_Output_Tags',&mt('Output Tags'),$stayOnPage,undef,600)
- 	       .'</span> <span>'
+ 	       .'</span>';
-                .&help_open_topic('Authoring_Multilingual_Problems',&mt('Languages'),$stayOnPage,undef,600)
-                .'</span>';
      }
      $out .= '</span>'; # End cheatsheet
      return $out;
- Line 1368  sub help_open_menu {
+ Line 1378  sub help_open_menu {
  sub top_nav_help {
      my ($text) = @_;
      $text = &mt($text);
-     my $stay_on_page;
+     my $stay_on_page = 1;
-     unless ($env{'environment.remote'} eq 'on') {
-         $stay_on_page = 1;
+     my $link = ($stay_on_page) ? "javascript:helpMenu('display')"
-     }
+ 	                     : "javascript:helpMenu('open')";
-     my ($link,$banner_link);
+     my $banner_link = &update_help_link(undef,undef,undef,undef,$stay_on_page);
-     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 $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,*",},});
- Line 1433  function helpMenu(target) {
+ Line 1435  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('$start_page\\n<frame name="bannerframe" src="'+banner_link+'" />\\n<frame name="bodyframe" src="$details_link" />\\n$end_page')
-     caller.document.writeln('<frame name="bodyframe" src="$details_link" marginwidth="0" marginheight="0" frameborder="0">\\n$end_page');
+     caller.document.close()
-     caller.document.close();
+     caller.focus()
-     caller.focus();
  }
  // END LON-CAPA Internal -->
  // ]]>
- Line 1748  RESIZE
+ Line 1749  RESIZE
  =head1 Excel and CSV file utility routines
+ =over 4
  =cut
  ###############################################################
- Line 1755  RESIZE
+ Line 1758  RESIZE
  =pod
- =over 4
  =item * &csv_translate($text)
  Translate $text to allow it to be output as a 'comma separated values'
- Line 2205  The optional $onchange argument specifie
+ Line 2206  The optional $onchange argument specifie
  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.
  =cut
- Line 2223  sub select_dom_form {
+ Line 2224  sub select_dom_form {
      }
      if ($includeempty) { @domains=('',@domains); }
      if (ref($excdoms) eq 'ARRAY') {
          map { $exclude{$_} = 1; } @{$excdoms};
      }
      my $selectdomain = "<select name=\"$name\" size=\"1\"$onchange>\n";
      foreach my $dom (@domains) {
- Line 2348  Outputs:
+ Line 2349  Outputs:
  =item * $clientos
- =item * $clientmobile
- =item * $clientinfo
- =item * $clientosversion
  =back
  =back
- Line 2372  sub decode_user_agent {
+ Line 2367  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;
- Line 2385  sub decode_user_agent {
+ Line 2378  sub decode_user_agent {
  	}
      }
      my $clientos='unknown';
-     my $clientinfo;
      if (($httpbrowser=~/linux/i) ||
          ($httpbrowser=~/unix/i) ||
          ($httpbrowser=~/ux/i) ||
- Line 2395  sub decode_user_agent {
+ Line 2387  sub decode_user_agent {
      if ($httpbrowser=~/next/i) { $clientos='next'; }
      if (($httpbrowser=~/mac/i) ||
          ($httpbrowser=~/powerpc/i)) { $clientos='mac'; }
-     if ($httpbrowser=~/win/i) {
+     if ($httpbrowser=~/win/i) { $clientos='win'; }
-         $clientos='win';
-         if ($httpbrowser =~/Windows\s+NT\s+(\d+\.\d+)/i) {
-             $clientosversion = $1;
-         }
-     }
      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,
+             $clientunicode,$clientos,);
-             $clientosversion);
  }
  ###############################################################
- Line 2579  sub authform_nochange {
+ Line 2557  sub authform_nochange {
                kerb_def_dom => 'MSU.EDU',
                @_,
            );
      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');
- Line 3075  sub get_related_words {
+ Line 3053  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
- Line 3082  sub get_related_words {
+ Line 3099  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
- Line 3272  sub screenname {
+ Line 3314  sub screenname {
  # ------------------------------------------------------------- Confirm Wrapper
  =pod
- =item * &confirmwrapper($message)
+ =item confirmwrapper
  Wrap messages about completion of operation in box
- Line 3688  sub user_lang {
+ Line 3730  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:
- Line 3710  Return string with previous attempt on p
+ Line 3752  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.
- Line 3722  The output string is a table containing
+ Line 3759  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) {
- Line 3732  sub get_previous_attempt {
+ Line 3769  sub get_previous_attempt {
        my %lasthash=();
        my $version;
        for ($version=1;$version<=$returnhash{'version'};$version++) {
-         foreach my $key (reverse(sort(split(/\:/,$returnhash{$version.':keys'})))) {
+         foreach my $key (sort(split(/\:/,$returnhash{$version.':keys'}))) {
-             if ($key =~ /\.rawrndseed$/) {
+ 	  $lasthash{$key}=$returnhash{$version.':'.$key};
-                 my ($id) = ($key =~ /^(.+)\.rawrndseed$/);
-                 $lasthash{$id.'.rndseed'} = $returnhash{$version.':'.$key};
-             } else {
-                 $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);
- Line 3760  sub get_previous_attempt {
+ Line 3792  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) {
- Line 3783  sub get_previous_attempt {
+ Line 3803  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') ||
+                     if (($returnhash{$version.':'.$id.'.type'} eq 'anonsurvey') || ($returnhash{$version.':'.$id.'.type'} eq 'anonsurveycred')) {
-                         ($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);
+                            '<td>'.&mt('Transaction [_1]',$version).'</td>';
-             if (@unsolved) {
-                 $prevattempts .= '<span class="LC_nobreak"><label>'.
-                                  '<input type="checkbox" name="HIDE'.$identifier.'" value="'.$version.':'.join('_',@unsolved).'" />'.
-                                  &mt('Hide').'</label></span>';
-             }
-             $prevattempts .= '</td>';
              if (@hidden) {
                  foreach my $key (sort(keys(%lasthash))) {
                      next if ($key =~ /\.foilorder$/);
- Line 3858  sub get_previous_attempt {
+ Line 3835  sub get_previous_attempt {
                          }
                      } else {
                          if ($key =~ /\./) {
-                             my $value = $returnhash{$version.':'.$key};
+                             my $value = &format_previous_attempt_value($key,
-                             if ($key =~ /\.rndseed$/) {
+                                               $returnhash{$version.':'.$key});
-                                 my ($id) = ($key =~ /^(.+)\.rndseed$/);
+                             $prevattempts.='<td>'.$value.'&nbsp;</td>';
-                                 if (exists($returnhash{$version.':'.$id.'.rawrndseed'})) {
-                                     $value = $returnhash{$version.':'.$id.'.rawrndseed'};
-                                 }
-                             }
-                             $prevattempts.='<td>'.&format_previous_attempt_value($key,$value).
-                                            '&nbsp;</td>';
                          } else {
                              $prevattempts.='<td>&nbsp;</td>';
                          }
- Line 3875  sub get_previous_attempt {
+ Line 3846  sub get_previous_attempt {
              } else {
  	        foreach my $key (sort(keys(%lasthash))) {
                      next if ($key =~ /\.foilorder$/);
-                     my $value = $returnhash{$version.':'.$key};
+ 		    my $value = &format_previous_attempt_value($key,
-                     if ($key =~ /\.rndseed$/) {
+ 			            $returnhash{$version.':'.$key});
-                         my ($id) = ($key =~ /^(.+)\.rndseed$/);
+ 		    $prevattempts.='<td>'.$value.'&nbsp;</td>';
-                         if (exists($returnhash{$version.':'.$id.'.rawrndseed'})) {
-                             $value = $returnhash{$version.':'.$id.'.rawrndseed'};
-                         }
-                     }
-                     $prevattempts.='<td>'.&format_previous_attempt_value($key,$value).
-                                    '&nbsp;</td>';
  	        }
              }
  	    $prevattempts.=&end_data_table_row();
- Line 4333  sub findallcourses {
+ Line 4298  sub findallcourses {
  ###############################################
  sub blockcheck {
-     my ($setters,$activity,$uname,$udom,$url,$is_course) = @_;
+     my ($setters,$activity,$uname,$udom,$url) = @_;
-     if (defined($udom) && defined($uname)) {
+     if (!defined($udom)) {
-         # 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 {
          $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 = '';
- Line 4356  sub blockcheck {
+ Line 4324  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') &&
+          $activity eq 'groups') && ($env{'request.course.id'})) {
-         ($env{'request.course.id'})) {
          foreach my $key (keys(%live_courses)) {
              if ($key ne $env{'request.course.id'}) {
                  delete($live_courses{$key});
- Line 4621  sub parse_block_record {
+ Line 4588  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;
- Line 4638  sub blocking_status {
+ Line 4605  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')) {
+     if ($activity eq 'port') {
-         $querystring .= "&amp;udom=$udom"      if ($udom =~ /^$match_domain$/);
+         $querystring .= "&amp;udom=$udom"      if $udom;
-         $querystring .= "&amp;uname=$uname"    if ($uname =~ /^$match_username$/);
+         $querystring .= "&amp;uname=$uname"    if $uname;
      } elsif ($activity eq 'docs') {
          $querystring .= '&amp;url='.&HTML::Entities::encode($url,'&"');
      }
- Line 4659  END_MYBLOCK
+ Line 4626  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>
- Line 4685  END_BLOCK
+ Line 4648  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{'request.host'} || $ENV{'REMOTE_ADDR'} || $clientip;
+     my $ip=$env{'request.host'} || $ENV{'REMOTE_ADDR'};
      my $name;
      foreach my $pattern (split(',',$acc)) {
- Line 4787  sub get_domainconf {
+ Line 4750  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 ($key eq 'loginvia') {
-                             if (ref($domconfig{'login'}{$key}) eq 'HASH') {
+                             if (ref($domconfig{'login'}{'loginvia'}) eq 'HASH') {
-                                 foreach my $hostname (keys(%{$domconfig{'login'}{$key}})) {
+                                 foreach my $hostname (keys(%{$domconfig{'login'}{'loginvia'}})) {
-                                     if (ref($domconfig{'login'}{$key}{$hostname}) eq 'HASH') {
+                                     if (ref($domconfig{'login'}{'loginvia'}{$hostname}) eq 'HASH') {
-                                         if ($key eq 'loginvia') {
+                                         if ($domconfig{'login'}{'loginvia'}{$hostname}{'server'}) {
-                                             if ($domconfig{'login'}{'loginvia'}{$hostname}{'server'}) {
+                                             my $server = $domconfig{'login'}{'loginvia'}{$hostname}{'server'};
-                                                 my $server = $domconfig{'login'}{'loginvia'}{$hostname}{'server'};
+                                             $designhash{$udom.'.login.loginvia'} = $server;
-                                                 $designhash{$udom.'.login.loginvia'} = $server;
+                                             if ($domconfig{'login'}{'loginvia'}{$hostname}{'serverpath'} eq 'custom') {
-                                                 if ($domconfig{'login'}{'loginvia'}{$hostname}{'serverpath'} eq 'custom') {
-                                                     $designhash{$udom.'.login.loginvia_'.$hostname} = $server.':'.$domconfig{'login'}{'loginvia'}{$hostname}{'custompath'};
+                                                 $designhash{$udom.'.login.loginvia_'.$hostname} = $server.':'.$domconfig{'login'}{'loginvia'}{$hostname}{'custompath'};
-                                                 } else {
+                                             } else {
-                                                     $designhash{$udom.'.login.loginvia_'.$hostname} = $server.':'.$domconfig{'login'}{'loginvia'}{$hostname}{'serverpath'};
+                                                 $designhash{$udom.'.login.loginvia_'.$hostname} = $server.':'.$domconfig{'login'}{'loginvia'}{$hostname}{'serverpath'};
-                                                 }
                                              }
-                                         } elsif ($key eq 'headtag') {
+                                             if ($domconfig{'login'}{'loginvia'}{$hostname}{'exempt'}) {
-                                             if ($domconfig{'login'}{'headtag'}{$hostname}{'url'}) {
+                                                 $designhash{$udom.'.login.loginvia_exempt_'.$hostname} = $domconfig{'login'}{'loginvia'}{$hostname}{'exempt'};
-                                                 $designhash{$udom.'.login.headtag_'.$hostname} = $domconfig{'login'}{'headtag'}{$hostname}{'url'};
                                              }
                                          }
-                                         if ($domconfig{'login'}{$key}{$hostname}{'exempt'}) {
-                                             $designhash{$udom.'.login.'.$key.'_exempt_'.$hostname} = $domconfig{'login'}{$key}{$hostname}{'exempt'};
-                                         }
                                      }
                                  }
                              }
- Line 4974  sub designparm {
+ Line 4932  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.
- Line 5037  Input: (optional) filename from which br
+ Line 4995  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
- Line 5068  sub CSTR_pageheader {
+ Line 5026  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);
- Line 5127  Inputs:
+ Line 5085  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
+             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
- Line 5148  other decorations will be returned.
+ Line 5106  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'))
- Line 5156  sub bodytag {
+ Line 5114  sub bodytag {
          $public = 1;
      }
      if (!$args->{'no_auto_mt_title'}) { $title = &mt($title); }
-     my $httphost = $args->{'use_absolute'};
      $function = &get_users_function() if (!$function);
      my $img =    &designparm($function.'.img',$domain);
- Line 5172  sub bodytag {
+ Line 5129  sub bodytag {
      @design{keys(%$addentries)} = @$addentries{keys(%$addentries)};
   # role and realm
-     my ($role,$realm) = split(m{\./},$env{'request.role'},2);
+     my ($role,$realm) = split(/\./,$env{'request.role'},2);
-     if ($realm) {
-         $realm = '/'.$realm;
-     }
      if ($role  eq 'ca') {
          my ($rdom,$rname) = ($realm =~ m{^/($match_domain)/($match_username)$});
          $realm = &plainname($rname,$rdom);
- Line 5199  sub bodytag {
+ Line 5153  sub bodytag {
  # construct main body tag
      my $bodytag = "<body $extra_body_attr>".
- 	&Apache::lontexconvert::init_math_support();
+ 	&Apache::lontexconvert::init_math_support($args->{'inherit_jsmath'});
      &get_unprocessed_cgi($ENV{'QUERY_STRING'}, ['inhibitmenu']);
- Line 5223  sub bodytag {
+ Line 5177  sub bodytag {
          $dc_info =~ s/\s+$//;
      }
-     $role = '<span class="LC_nobreak">('.$role.')</span>' if ($role && !$env{'browser.mobile'});
+     $role = '<span class="LC_nobreak">('.$role.')</span>' if $role;
-     if ($env{'request.state'} eq 'construct') { $forcereg=1; }
+         if ($env{'request.state'} eq 'construct') { $forcereg=1; }
-     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') {
          #        $titleinfo = &CSTR_pageheader(); #FIXME: Will be removed once all scripts have their own calls
          #    }
          $bodytag .= Apache::lonhtmlcommon::scripttag(
-             Apache::lonmenu::utilityfunctions($httphost), 'start');
+             Apache::lonmenu::utilityfunctions(), '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 />
+              $bodytag .= qq|<div id="LC_nav_bar">$left $role<br />
-                            <em>$realm</em> $dc_info</div>|;
+                 <em>$realm</em> $dc_info</div>|;
              return $bodytag;
          }
- Line 5272  sub bodytag {
+ Line 5210  sub bodytag {
          }
          $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'});
              } elsif ($forcereg) {
                  $bodytag .= &Apache::lonmenu::innerregister($forcereg,undef,
                                                              $args->{'group'});
              } else {
-                 my $forbodytag;
+                 $bodytag .=
-                 &Apache::lonmenu::prepare_functions($env{'request.noversionuri'},
+                     &Apache::lonmenu::prepare_functions($env{'request.noversionuri'},
-                                                     $forcereg,$args->{'group'},
+                                                         $forcereg,$args->{'group'},
-                                                     $args->{'bread_crumbs'},
+                                                         $args->{'bread_crumbs'},
-                                                     $advtoolsref,'',\$forbodytag);
+                                                         $advtoolsref);
-                 unless (ref($args->{'bread_crumbs'}) eq 'ARRAY') {
-                     $bodytag .= $forbodytag;
-                 }
              }
          }else{
              # this is to seperate menu from content when there's no secondary
- Line 5305  sub bodytag {
+ Line 5236  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 {
- Line 5384  sub make_attr_string {
+ Line 5267  sub make_attr_string {
  		delete($attr_ref->{$key});
  	    }
  	}
-         if ($env{'environment.remote'} eq 'on') {
+ 	$attr_ref->{'onload'}  = $on_load;
-             $attr_ref->{'onload'}  =
+ 	$attr_ref->{'onunload'}= $on_unload;
-                 &Apache::lonmenu::loadevents().  $on_load;
-             $attr_ref->{'onunload'}=
-                 &Apache::lonmenu::unloadevents().$on_unload;
-         } else {
- 	    $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;
- Line 5426  sub endbodytag {
+ Line 5302  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=
- Line 5600  div.LC_confirm_box .LC_success img {
+ Line 5477  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;
- Line 5732  table#LC_menubuttons img {
+ Line 5598  table#LC_menubuttons img {
    vertical-align: middle;
  }
- .LC_breadcrumbs_hoverable {
-   background: $sidebg;
- }
  td.LC_table_cell_checkbox {
    text-align: center;
  }
- Line 6624  div.LC_edit_problem_saves {
+ Line 6486  div.LC_edit_problem_saves {
    white-space: nowrap;
  }
- .LC_edit_problem_latexhelper{
-     text-align: right;
- }
- #LC_edit_problem_colorful div{
-     margin-left: 40px;
- }
  img.stift {
    border-width: 0;
    vertical-align: middle;
- Line 6719  fieldset {
+ Line 6573  fieldset {
    /* overflow: hidden; */
  }
- article.geogebraweb div {
-     margin: 0;
- }
  fieldset > legend {
    font-weight: bold;
    padding: 0 5px 0 5px;
- Line 6859  ul#LC_secondary_menu li {
+ Line 6709  ul#LC_secondary_menu li {
    font-weight: bold;
    line-height: 1.8em;
    border-right: 1px solid black;
-   vertical-align: middle;
    float: left;
  }
- Line 7402  sub headtag {
+ Line 7251  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(),
- Line 7413  sub headtag {
+ Line 7261  sub headtag {
      my $result =
  	'<head>'.
- 	&font_settings($args);
+ 	&font_settings();
-     my $inhibitprint;
+     my $inhibitprint = &print_suppression();
-     if ($args->{'print_suppress'}) {
-         $inhibitprint = &print_suppression();
-     }
      if (!$args->{'frameset'}) {
  	$result .= &Apache::lonhtmlcommon::htmlareaheaders();
      }
-     if ($args->{'force_register'}) {
+     if ($args->{'force_register'} && $env{'request.noversionuri'} !~ m{^/res/adm/pages/}) {
-         $result .= &Apache::lonmenu::registerurl(1);
+         $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();
- Line 7458  sub headtag {
+ Line 7303  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.'"';
+ 	.'<link rel="stylesheet" type="text/css" href="'.$url.'" />'
-     if (!$args->{'frameset'}) {
-         $result .= ' /';
-     }
-     $result .= '>'
          .$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" />';
-     }
      return $result.'</head>';
  }
- Line 7567  OFFLOAD
+ Line 7321  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'}) ||
+     if (!$env{'browser.mathml'} && $env{'browser.unicode'}) {
-         ((ref($args) eq 'HASH') && ($args->{'browser.unicode'}))) {
  	$headerstring.=
- 	    '<meta http-equiv="Content-Type" content="text/html; charset=utf-8"';
+ 	    '<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />';
-         if (!$args->{'frameset'}) {
-             $headerstring.= ' /';
-         }
-         $headerstring .= '>'."\n";
      }
      return $headerstring;
  }
- Line 7626  sub print_suppression {
+ Line 7374  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 '') {
- Line 7673  Inputs: none
+ Line 7421  Inputs: none
  =cut
  sub xml_begin {
-     my ($is_frameset) = @_;
      my $output='';
      if ($env{'browser.mathml'}) {
- Line 7685  sub xml_begin {
+ Line 7432  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".
+ 	$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">'."\n";
+            .'<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">';
      }
      return $output;
  }
- Line 7734  $args - additional optional args support
+ Line 7478  $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
               group          -> includes the current group, if page is for a
                                 specific group
  =back
- Line 7756  sub start_page {
+ Line 7501  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'}) ) {
- Line 7770  sub start_page {
+ Line 7515  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->{'bgcolor'},        $args,
-                          $args,                     \@advtools);
+                          \@advtools);
          }
      }
- Line 7811  sub start_page {
+ Line 7556  sub start_page {
  		}else{
  			$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;
  }
- Line 7862  function set_wishlistlink(title, path) {
+ Line 7602  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');
  }
- Line 7909  var modalWindow = {
+ Line 7645  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("'","&#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>
- Line 7925  ENDMODAL
+ Line 7660  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
  }
- Line 7964  sub modal_adhoc_inner {
+ Line 7697  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).
+                  &start_scrollbox($width.'px',$innerwidth.'px',$height.'px').
-                  $content.
+                     $content.
                   &end_scrollbox().
-                  &end_page()
+                &end_page()
               );
      return &modal_adhoc_script($funcname,$width,$height,$content);
  }
- Line 8184  sub validate_page {
+ Line 7917  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.'"';
+         $table_id = " id='table_$id'";
-         $div_id = ' id="div_'.$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
  }
- Line 8212  sub end_scrollbox {
+ Line 7938  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) = @_;
+     my ($r,$title,$msg) = @_;
-     if (ref($args) eq 'HASH') {
-         if (!$args->{'no_auto_mt_msg'}) { $msg = &mt($msg); }
-     } else {
-         $msg = &mt($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);
- Line 8519  role status: active, previous or future.
+ Line 8158  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;
- Line 8608  sub get_sections {
+ Line 8247  sub get_sections {
          }
      }
      if ($check_students) {
  	my ($classlist) = &Apache::loncoursedata::get_classlist($cdom,$cnum);
  	my $sec_index = &Apache::loncoursedata::CL_SECTION();
  	my $status_index = &Apache::loncoursedata::CL_STATUS();
- Line 8922  sub get_user_info {
+ Line 8561  sub get_user_info {
  =item * &get_user_quota()
- Retrieves quota assigned for storage of user files.
+ Retrieves quota assigned for storage of portfolio files for a user
- Default is to report quota for portfolio files.
  Incoming parameters:
 . user's username
 . user's domain
-. quota name - portfolio, author, or course
-    (if no quota name provided, defaults to portfolio).
-. crstype - official, unofficial, textbook or community, if quota name is
-    course
  Returns:
-. Disk quota (in MB) assigned to student.
+. Disk quota (in Mb) assigned to student.
 . (Optional) Type of setting: custom or default
     (individually assigned or default for user's
     institutional status).
- Line 8945  Returns:
+ Line 8579  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
- Line 8953  defined for the user's institutional sta
+ Line 8587  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'};
- Line 8968  sub get_user_quota {
+ Line 8602  sub get_user_quota {
          $defquota = 0;
      } else {
          my $inststatus;
-         if ($quotaname eq 'course') {
+         if ($udom eq $env{'user.domain'} && $uname eq $env{'user.name'}) {
-             if (($env{'course.'.$udom.'_'.$uname.'.num'} eq $uname) &&
+             $quota = $env{'environment.portfolioquota'};
-                 ($env{'course.'.$udom.'_'.$uname.'.domain'} eq $udom)) {
+             $inststatus = $env{'environment.inststatus'};
-                 $quota = $env{'course.'.$udom.'_'.$uname.'.internal.uploadquota'};
+         } else {
-             } else {
+             my %userenv =
-                 my %cenv = &Apache::lonnet::coursedescription("$udom/$uname");
+                 &Apache::lonnet::get('environment',['portfolioquota',
-                 $quota = $cenv{'internal.uploadquota'};
+                                      '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'}) {
+             $quotatype = 'custom';
-                 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';
-             }
          }
      }
      if (wantarray) {
- Line 9039  Retrieves default quota assigned for sto
+ Line 8642  Retrieves default quota assigned for sto
  given an (optional) user's institutional status.
  Incoming parameters:
 . domain
 . (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.
-.  quota name - portfolio, author, or course
-    (if no quota name provided, defaults to portfolio).
  Returns:
+. Default disk quota (in Mb) for user portfolios in the domain.
-. Default disk quota (in MB) for user portfolios in the domain.
 . (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 (ref($quotahash{'quotas'}{'defaultquota'}) eq 'HASH') {
-                     if ($quotahash{'quotas'}{$key}{$item} ne '') {
+                     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) {
+                         } elsif ($quotahash{'quotas'}{'defaultquota'}{$item} > $defquota) {
-                             $defquota = $quotahash{'quotas'}{$key}{$item};
+                             $defquota = $quotahash{'quotas'}{'defaultquota'}{$item};
                              $settingstatus = $item;
                          }
                      }
-                 } elsif ($key eq 'defaultquota') {
+                 } else {
                      if ($quotahash{'quotas'}{$item} ne '') {
                          if ($defquota eq '') {
                              $defquota = $quotahash{'quotas'}{$item};
- Line 9106  sub default_quota {
+ Line 8703  sub default_quota {
              }
          }
          if ($defquota eq '') {
-             if (ref($quotahash{'quotas'}{$key}) eq 'HASH') {
+             if (ref($quotahash{'quotas'}{'defaultquota'}) eq 'HASH') {
-                 $defquota = $quotahash{'quotas'}{$key}{'default'};
+                 $defquota = $quotahash{'quotas'}{'defaultquota'}{'default'};
-             } elsif ($key eq 'defaultquota') {
+             } else {
                  $defquota = $quotahash{'quotas'}{'default'};
              }
              $settingstatus = 'default';
-             if ($defquota eq '') {
-                 if ($quotaname eq 'author') {
-                     $defquota = 500;
-                 }
-             }
          }
      } else {
          $settingstatus = 'default';
-         if ($quotaname eq 'author') {
+         $defquota = 20;
-             $defquota = 500;
-         } else {
-             $defquota = 20;
-         }
      }
      if (wantarray) {
          return ($defquota,$settingstatus);
- Line 9133  sub default_quota {
+ Line 8721  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
-. username or coursenum
-. domain
-. context ('author' or 'course')
-. filename of file for which action is being requested
-. filesize (kB) of file
-. action being taken: copy or upload.
-. 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);
- Line 9249  sub user_picker {
+ Line 8781  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',
- Line 9262  sub user_picker {
+ Line 8794  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.",
- Line 9273  sub user_picker {
+ Line 8803  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 = &select_dom_form($currdom,'srchdomain',1,1);
      my $srchinsel = ' <select name="srchin">';
- Line 9289  sub user_picker {
+ Line 8817  sub user_picker {
          next if ($option eq 'crs' && !$env{'request.course.id'});
          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";
- Line 9301  sub user_picker {
+ Line 8829  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";
- Line 9313  sub user_picker {
+ Line 8841  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";
- Line 9401  function validateEntry(callingForm) {
+ Line 8929  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) {
- Line 9458  $new_user_create
+ Line 8986  $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.'" />'.
- Line 9474  END_BLOCK
+ Line 9002  END_BLOCK
  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) {
+         foreach my $user (keys(%{$usershash})) {
-             my (%by_username,%by_id,%userdoms);
+             my ($uname,$udom) = split(/:/,$user);
-             my $checkid;
+             next if ($udom eq '' || $uname eq '');
-             if (ref($checks) eq 'HASH') {
+             my ($id,$newuser);
-                 if ((!defined($checks->{'username'})) && (defined($checks->{'id'}))) {
+             if (ref($usershash->{$user}) eq 'HASH') {
-                     $checkid = 1;
+                 $newuser = $usershash->{$user}->{'newuser'};
-                 }
+                 $id = $usershash->{$user}->{'id'};
-             }
-             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;
-                 }
              }
-             if ($checkid) {
+             my $inst_response;
-                 foreach my $udom (keys(%by_id)) {
+             if (ref($checks) eq 'HASH') {
-                     my ($outcome,$results) = &Apache::lonnet::get_multiple_instusers($udom,$by_id{$udom},'id');
+                 if (defined($checks->{'username'})) {
-                     if ($outcome eq 'ok') {
+                     ($inst_response,%{$inst_results->{$user}}) =
-                         foreach my $id (keys(%{$by_id{$udom}})) {
+                         &Apache::lonnet::get_instuser($udom,$uname);
-                             my $uname = $by_id{$udom}{$id};
+                 } elsif (defined($checks->{'id'})) {
-                             $inst_response{$uname.':'.$udom} = $outcome;
+                     ($inst_response,%{$inst_results->{$user}}) =
-                         }
+                         &Apache::lonnet::get_instuser($udom,undef,$id);
-                         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};
-                                 }
-                             }
-                         }
-                     }
                  }
              } else {
-                 foreach my $udom (keys(%by_username)) {
+                 ($inst_response,%{$inst_results->{$user}}) =
-                     my ($outcome,$results) = &Apache::lonnet::get_multiple_instusers($udom,$by_username{$udom});
+                     &Apache::lonnet::get_instuser($udom,$uname);
-                     if ($outcome eq 'ok') {
+                 return;
-                         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};
-                             }
-                         }
-                     }
-                 }
              }
-         } elsif (keys(%{$usershash}) == 1) {
+             if (!$got_rules->{$udom}) {
-             my $user = (keys(%{$usershash}))[0];
+                 my %domconfig = &Apache::lonnet::get_dom('configuration',
-             my ($uname,$udom) = split(/:/,$user);
+                                                   ['usercreation'],$udom);
-             if (($udom ne '') && ($uname ne '')) {
+                 if (ref($domconfig{'usercreation'}) eq 'HASH') {
-                 if (ref($usershash->{$user}) eq 'HASH') {
+                     foreach my $item ('username','id') {
-                     if (ref($checks) eq 'HASH') {
+                         if (ref($domconfig{'usercreation'}{$item.'_rule'}) eq 'ARRAY') {
-                         if (defined($checks->{'username'})) {
+                             $$curr_rules{$udom}{$item} =
-                             ($inst_response{$user},%{$inst_results->{$user}}) =
+                                 $domconfig{'usercreation'}{$item.'_rule'};
-                                 &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);
-                             }
                          }
-                     } 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 {
+                 $got_rules->{$udom} = 1;
-                 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'};
-                 }
              }
              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,
+                             my %rule_check = &Apache::lonnet::inst_rulecheck($udom,$uname,$id,$item,$$curr_rules{$udom}{$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;
-                                                     }
                                                  }
                                              }
                                          }
- Line 9738  sub personal_data_fieldtitles {
+ Line 9162  sub personal_data_fieldtitles {
  sub sorted_inst_types {
      my ($dom) = @_;
-     my ($usertypes,$order);
+     my ($usertypes,$order) = &Apache::lonnet::retrieve_inst_usertypes($dom);
-     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 $othertitle = &mt('All users');
      if ($env{'request.course.id'}) {
          $othertitle  = &mt('Any users');
- Line 9895  reservable_now - ref to hash of student_
+ Line 9312  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.
  sorted_future - ref to array of student_schedulable slots reservable in
                  the future, ordered by start date of reservation period.
- Line 9908  future_reservable - ref to hash of stude
+ Line 9323  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
- Line 9963  sub get_future_slots {
+ Line 9376  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;
- Line 9975  sub get_future_slots {
+ Line 9384  sub get_future_slots {
                  }
                  $reservable_now{$slot} = {
                                             symb       => $symb,
-                                            endreserve => $lastres,
+                                            endreserve => $lastres
-                                            uniqueperiod => $uniqueperiod,
                                           };
              } elsif (($startreserve > $now) &&
                       (!$endreserve || $endreserve > $startreserve)) {
                  $future_reservable{$slot} = {
                                                symb         => $symb,
-                                               startreserve => $startreserve,
+                                               startreserve => $startreserve
-                                               uniqueperiod => $uniqueperiod,
                                              };
              }
          }
- Line 10153  sub ask_for_embedded_content {
+ Line 9560  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,$cdom,$cnum);
      if ($env{'request.course.id'}) {
          if ($actionurl eq '/adm/dependencies') {
              $navmap = Apache::lonnavmaps::navmap->new();
- Line 10164  sub ask_for_embedded_content {
+ Line 9572  sub ask_for_embedded_content {
          $cdom = $env{'course.'.$env{'request.course.id'}.'.domain'};
          $cnum = $env{'course.'.$env{'request.course.id'}.'.num'};
      }
      if (($actionurl eq '/adm/portfolio') ||
          ($actionurl eq '/adm/coursegrp_portfolio')) {
          my $current_path='/';
          if ($env{'form.currentpath'}) {
- Line 10196  sub ask_for_embedded_content {
+ Line 9604  sub ask_for_embedded_content {
              $toplevel = $url;
              if ($args->{'context'} eq 'paste') {
                  ($cdom,$cnum) = ($url =~ m{^\Q/uploaded/\E($match_domain)/($match_courseid)/});
                  ($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 '') {
              if (ref($args) eq 'HASH') {
                  $url = $args->{'docs_url'};
                  $title = $args->{'docs_title'};
                  $toplevel = $url;
                  unless ($toplevel =~ m{^/}) {
                      $toplevel = "/$url";
                  }
- Line 10218  sub ask_for_embedded_content {
+ Line 9626  sub ask_for_embedded_content {
                      ($path) =
                          ($toplevel =~ m{^(\Q/uploaded/$cdom/$cnum/\E(?:docs|supplemental)/(?:default|\d+)/\d+)/});
                  }
-                 if ($toplevel=~/^\/*(uploaded|editupload)/) {
+                 $fileloc = &Apache::lonnet::filelocation('',$toplevel);
-                     $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);
-                 }
                  $fileloc =~ s{^/}{};
                  ($filename) = ($fileloc =~ m{.+/([^/]+)$});
                  $heading = &mt('Status of dependencies in [_1]',"$title ($filename)");
- Line 10248  sub ask_for_embedded_content {
+ Line 9648  sub ask_for_embedded_content {
          } else {
              $embed_file = $file;
          }
-         my ($absolutepath,$cleaned_file);
+         my $absolutepath;
          if ($embed_file =~ m{^\w+://}) {
-             $cleaned_file = $embed_file;
+             $newfiles{$embed_file} = 1;
-             $newfiles{$cleaned_file} = 1;
+             $mapping{$embed_file} = $embed_file;
-             $mapping{$cleaned_file} = $embed_file;
          } else {
-             $cleaned_file = &clean_path($embed_file);
              if ($embed_file =~ m{^/}) {
                  $absolutepath = $embed_file;
+                 $embed_file =~ s{^(/+)}{};
              }
-             if ($cleaned_file =~ m{/}) {
+             if ($embed_file =~ m{/}) {
-                 my ($path,$fname) = ($cleaned_file =~ m{^(.+)/([^/]*)$});
+                 my ($path,$fname) = ($embed_file =~ m{^(.+)/([^/]*)$});
                  $path = &check_for_traversal($path,$url,$toplevel);
                  my $item = $fname;
                  if ($path ne '') {
- Line 10276  sub ask_for_embedded_content {
+ Line 9675  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;
                  }
              }
          }
- Line 10286  sub ask_for_embedded_content {
+ Line 9685  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')) {
              my ($sublistref,$listerror) =
                  &Apache::lonnet::dirlist($url.$path,$udom,$uname,$getpropath);
              if (ref($sublistref) eq 'ARRAY') {
- Line 10429  sub ask_for_embedded_content {
+ Line 9828  sub ask_for_embedded_content {
          $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));
- Line 10652  sub ask_for_embedded_content {
+ Line 10051  sub ask_for_embedded_content {
      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') &&
- Line 10816  sub upload_embedded {
+ Line 10175  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)
+             $output .= &mt('Invalid file extension ([_1]) - reserved for LONCAPA use - rename the file with a different extension and re-upload. ',$1).'<br />';
-                       .' '.&mt('Rename the file with a different extension and re-upload.').'<br />';
              next;
          } elsif (($fname =~ /\.(\w+)$/) &&
                   (!defined(&Apache::loncommon::fileembstyle($1)))) {
- Line 11016  sub modify_html_refs {
+ Line 10374  sub modify_html_refs {
      }
      my (%allfiles,%codebase,$output,$content);
      my @changes = &get_env_multiple('form.namechange');
-     unless ((@changes > 0)  || ($context eq 'syllabus')) {
+     unless ((@changes > 0) || ($context eq 'syllabus')) {
          if (wantarray) {
              return ('',0,0);
          } else {
- Line 11081  sub modify_html_refs {
+ Line 10439  sub modify_html_refs {
                          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});
- Line 11151  sub modify_html_refs {
+ Line 10508  sub modify_html_refs {
                          }
                      }
                      if ($rewrites) {
                          my $saveresult;
                          my $url = &Apache::lonnet::store_edited_file($container,$content,$udom,$uname,\$saveresult);
                          if ($url eq $container) {
                              my ($fname) = ($container =~ m{/([^/]+)$});
- Line 11257  sub check_for_upload {
+ Line 10614  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>'.
+                                           '<span class="LC_filename">'.$fname.'</span>',$filesize,$currsize).'</span>'.
-                                       '<p>'.&mt('Disk quota is [_1] kilobytes. Your current disk usage is [_2] kilobytes.',
+                                       '<br />'.&mt('Disk quota is [_1] kilobytes. Your current disk usage is [_2] kilobytes.',
-                                                    $disk_quota,$current_disk_usage).'</p>';
+                                                    $disk_quota,$current_disk_usage);
                              return ('will_exceed_quota',$msg);
                          }
                      }
- Line 11270  sub check_for_upload {
+ Line 10627  sub check_for_upload {
          }
      }
      if (($current_disk_usage + $filesize) > $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.','<span class="LC_filename">'.$fname.'</span>',$filesize).'</p>'.
+                 &mt('Unable to upload [_1]. (size = [_2] kilobytes). Disk quota will be exceeded.','<span class="LC_filename">'.$fname.'</span>',$filesize).'</span>'.
-                   '<p>'.&mt('Disk quota is [_1] kilobytes. Your current disk usage is [_2] kilobytes.',$disk_quota,$current_disk_usage).'</p>';
+                   '<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);
          }
      }
- Line 11375  sub decompress_form {
+ Line 10732  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",
+         my @diffs = &compare_arrays(\@paths,\@camtasia);
-                          "$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);
          if (@diffs == 0) {
-             $is_camtasia = 6;
+             $is_camtasia = 1;
-         } 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;
-                 }
-             }
          }
      }
      my $output;
- Line 11446  sub decompress_form {
+ Line 10753  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';
- Line 11508  ENDCAM
+ Line 10816  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 />'.
- Line 11631  sub decompress_uploaded_file {
+ Line 10939  sub decompress_uploaded_file {
  sub process_decompression {
      my ($docudom,$docuname,$file,$destination,$dir_root,$hiddenelem) = @_;
      my ($dir,$error,$warning,$output);
-     if ($file !~ /\.(zip|tar|bz2|gz|tar.gz|tar.bz2|tgz)$/i) {
+     if ($file !~ /\.(zip|tar|bz2|gz|tar.gz|tar.bz2|tgz)$/) {
          $error = &mt('Filename not a supported archive file type.').
                   '<br />'.&mt('Filename should end with one of: [_1].',
                                '.zip, .tar, .bz2, .gz, .tar.gz, .tar.bz2, .tgz');
- Line 11741  sub process_decompression {
+ Line 11049  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'} = [];
- Line 11760  sub process_decompression {
+ Line 11067  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)) ||
+                                 } elsif ($item eq "$contents[0]/index.html") {
-                                          (($item eq "$contents[0]/$contents[0]".'.html') && ($version == 8))) {
                                      $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)) ||
+                                     if ($item eq "$contents[0]/media") {
-                                         ((($item eq "$contents[0]/scripts") || ($item eq "$contents[0]/skins") ||
-                                              ($item eq "$contents[0]/skins/express_show")) && ($version == 8))) {
                                          push(@{$env{'form.archive_directory'}},$i);
                                      }
                                      $env{'form.archive_'.$i} = 'dependency';
- Line 12213  sub process_extracted_files {
+ Line 11517  sub process_extracted_files {
          $folders{'0'} = $items[-2];
          if ($env{'form.folderpath'} =~ /\:1$/) {
              $containers{'0'}='page';
          } else {
              $containers{'0'}='sequence';
          }
      }
- Line 12333  sub process_extracted_files {
+ Line 11637  sub process_extracted_files {
                      }
                  }
              } else {
                  $warning .= &mt('Item extracted from archive: [_1] has unexpected path.',$path).'<br />';
              }
          }
          for (my $i=1; $i<=$numitems; $i++) {
- Line 12355  sub process_extracted_files {
+ Line 11659  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}}";
- Line 12402  sub process_extracted_files {
+ Line 11706  sub process_extracted_files {
                                      $showpath = "$relpath/$title";
                                  } else {
                                      $showpath = "/$title";
                                  }
                                  $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;
                              }
                          }
- Line 12417  sub process_extracted_files {
+ Line 11721  sub process_extracted_files {
                                      $path,$env{'form.archive_content_'.$referrer{$i}}).'<br />';
                  }
              } else {
                  $warning .= &mt('Item extracted from archive: [_1] has unexpected path.',$path).'<br />';
              }
          }
          if (keys(%todelete)) {
- Line 12495  sub cleanup_empty_dirs {
+ Line 11799  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,
- Line 12605  sub get_turnedin_filepath {
+ Line 11909  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);
                              }
                          }
- Line 12616  sub get_turnedin_filepath {
+ Line 11917  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') {
- Line 12659  sub get_turnedin_filepath {
+ Line 11957  sub get_turnedin_filepath {
                  $restitle = time;
              }
          }
-         if (length($restitle) > 12) {
-             $restitle = substr($restitle,0,12);
-         }
          push(@pathitems,$restitle);
          $path .= join('/',@pathitems);
      }
- Line 13599  sub restore_settings {
+ Line 12894  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
+ (d) Help requests, (e) Course requests needing approval,  generated by
- module change checking, student/employee ID conflict checks, as
+ lonerrorhandler.pm, CHECKRPMS, loncron, lonsupportreq.pm and
- generated by lonerrorhandler.pm, CHECKRPMS, loncron,
+ loncoursequeueadmin.pm respectively.
- lonsupportreq.pm, loncoursequeueadmin.pm, searchcat.pl respectively.
  Inputs:
  defmail (scalar - email address of default recipient),
- mailing type (scalar: errormail, packagesmail, helpdeskmail,
+ mailing type (scalar - errormail, packagesmail, or helpdeskmail),
- requestsmail, updatesmail, or idconflictsmail).
  defdom (domain for which to retrieve configuration settings),
+ origmail (scalar - email address of recipient from loncapa.conf,
- origmail (scalar - email address of recipient from loncapa.conf,
+ i.e., predates configuration by DC via domainprefs.pm
- i.e., predates configuration by DC via domainprefs.pm
  Returns: comma separated list of addresses to which to send e-mail.
- Line 13808  sub extract_categories {
+ Line 13099  sub extract_categories {
  =pod
- =item * &recurse_categories()
+ =item *&recurse_categories()
  Recursively used to generate breadcrumb trails for course categories.
- Line 13879  sub recurse_categories {
+ Line 13170  sub recurse_categories {
  =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.
- Line 13956  sub assign_categories_table {
+ Line 13247  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.
- Line 13990  sub assign_category_rows {
+ Line 13281  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;
- Line 14022  sub assign_category_rows {
+ Line 13313  sub assign_category_rows {
      return $text;
  }
- =pod
- =back
- =cut
  ############################################################
  ############################################################
- Line 14158  sub commit_studentrole {
+ Line 13443  sub commit_studentrole {
                      }
                  }
              } else {
                  if ($secchange) {
                      $$logmsg .= &mt('Error when attempting section change for [_1] from old section "[_2]" to new section: "[_3]" in course [_4] -error:',$uname,$oldsec,$sec,$cid).' '.$modify_section_result.$linefeed;
                  } else {
                      $$logmsg .= &mt('Error when attempting to modify role for [_1] for section: "[_2]" in course [_3] -error:',$uname,$sec,$cid).' '.$modify_section_result.$linefeed;
- Line 14245  sub check_clone {
+ Line 13530  sub check_clone {
              (&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 @cloners = split(/,/,$clonehash{'cloners'});
-                 my %domdefs = &Apache::lonnet::get_domain_defaults($args->{'course_domain'});
+             if (grep(/^\*$/,@cloners)) {
-                 if ($domdefs{'canclone'}) {
+                 $can_clone = 1;
-                     unless ($domdefs{'canclone'} eq 'none') {
+             } elsif (grep(/^\*\:\Q$args->{'ccdomain'}\E$/,@cloners)) {
-                         if ($domdefs{'canclone'} eq 'domain') {
+                 $can_clone = 1;
-                             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;
-                             }
-                         }
-                     }
-                 }
              } 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 =
+ 	        my %roleshash =
-                     &Apache::lonnet::get_my_roles($args->{'ccuname'},
+ 		    &Apache::lonnet::get_my_roles($args->{'ccuname'},
-                                                   $args->{'ccdomain'},
+ 					 $args->{'ccdomain'},
-                                                   'userroles',['active'],[$ccrole],
+                                          'userroles',['active'],[$ccrole],
-                                                   [$args->{'clonedomain'}]);
+ 					 [$args->{'clonedomain'}]);
-                 if ($roleshash{$args->{'clonecourse'}.':'.$args->{'clonedomain'}.':'.$ccrole}) {
+ 	        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'},
+                 } elsif (&Apache::lonnet::is_course_owner($args->{'clonedomain'},$args->{'clonecourse'},$args->{'ccuname'},$args->{'ccdomain'})) {
-                                                           $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'});
+                     }
  	        }
  	    }
          }
- Line 14338  sub check_clone {
+ Line 13565  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') {
- Line 14435  sub construct_course {
+ Line 13662  sub construct_course {
                     'plc.users.denied',
                     'hidefromcat',
                     'checkforpriv',
-                    'categories',
+                    'categories'],
-                    'internal.uniquecode'],
                     $$crsudom,$$crsunum);
-         if ($args->{'textbook'}) {
-             $cenv{'internal.textbook'} = $args->{'textbook'};
-         }
      }
  #
- Line 14624  sub construct_course {
+ Line 13847  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';
      }
- Line 14711  sub construct_course {
+ Line 13915  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;
- }
  ############################################################
  ############################################################
- Line 14792  sub group_term {
+ Line 13942  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);
  }
- Line 14858  sub escape_url {
+ Line 14007  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 {
- Line 14916  sub init_user_environment {
+ Line 14065  sub init_user_environment {
  		}
  	    }
  	    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'}
- Line 14940  sub init_user_environment {
+ Line 14078  sub init_user_environment {
      }
  # ------------------------------------ Check browser type and MathML capability
-     my ($httpbrowser,$clientbrowser,$clientversion,$clientmathml,$clientunicode,
+     my ($httpbrowser,$clientbrowser,$clientversion,$clientmathml,
-         $clientos,$clientmobile,$clientinfo,$clientosversion) = &decode_user_agent($r);
+         $clientunicode,$clientos) = &decode_user_agent($r);
  # ------------------------------------------------------------- Get environment
- Line 14972  sub init_user_environment {
+ Line 14110  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" => '',
- Line 14994  sub init_user_environment {
+ Line 14129  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') {
- Line 15012  sub init_user_environment {
+ Line 14141  sub init_user_environment {
                                                    undef,\%userenv,\%domdef,\%is_adv);
          }
-         foreach my $crstype ('official','unofficial','community','textbook') {
+         foreach my $crstype ('official','unofficial','community') {
              $userenv{'canrequest.'.$crstype} =
                  &Apache::lonnet::usertools_access($username,$domain,$crstype,
                                                    'reload','requestcourses',
- Line 15026  sub init_user_environment {
+ Line 14155  sub init_user_environment {
          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'};
- Line 15117  sub clean_symb {
+ Line 14246  sub clean_symb {
      return ($symb,$enc);
  }
- ############################################################
+ sub build_release_hashes {
- ############################################################
+     my ($checkparms,$checkresponsetypes,$checkcrstypes,$anonsurvey,$randomizetry) = @_;
+     return unless((ref($checkparms) eq 'HASH') && (ref($checkresponsetypes) eq 'HASH') &&
- =pod
+                   (ref($checkcrstypes) eq 'HASH') && (ref($anonsurvey) eq 'HASH') &&
+                   (ref($randomizetry) eq 'HASH'));
- =head1 Routines for building display used to search for courses
+     foreach my $key (keys(%Apache::lonnet::needsrelease)) {
+         my ($item,$name,$value) = split(/:/,$key);
+         if ($item eq 'parameter') {
- =over 4
+             if (ref($checkparms->{$name}) eq 'ARRAY') {
+                 unless(grep(/^\Q$name\E$/,@{$checkparms->{$name}})) {
- =item * &build_filters()
+                     push(@{$checkparms->{$name}},$value);
+                 }
- 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;
+                 push(@{$checkparms->{$name}},$value);
              }
-             if (!$filter->{$item}) {
+         } elsif ($item eq 'resourcetag') {
-                 $filter->{$item} = '';
+             if ($name eq 'responsetype') {
-             }
+                 $checkresponsetypes->{$value} = $Apache::lonnet::needsrelease{$key}
-         }
-         if ($item eq 'domainfilter') {
-             my $allow_blank = 1;
-             if ($formname eq 'portform') {
-                 $allow_blank=0;
-             } elsif ($formname eq 'studentform') {
-                 $allow_blank=0;
              }
-             if ($fixeddom) {
+         } elsif ($item eq 'course') {
-                 $domainselectform = '<input type="hidden" name="domainfilter"'.
+             if ($name eq 'crstype') {
-                                     ' value="'.$codedom.'" />'.
+                 $checkcrstypes->{$value} = $Apache::lonnet::needsrelease{$key};
-                                     &Apache::lonnet::domain($codedom,'description');
-             } else {
-                 $domainselectform = &select_dom_form($filter->{$item},
-                                                      'domainfilter',
-                                                       $allow_blank,'',$onchange);
              }
-         } else {
-             $list->{$item} = &HTML::Entities::encode($filter->{$item},'<>&"');
          }
      }
+     ($anonsurvey->{major},$anonsurvey->{minor}) = split(/\./,$Apache::lonnet::needsrelease{'parameter:type:anonsurvey'});
-     # last course activity filter and selection
+     ($randomizetry->{major},$randomizetry->{minor}) = split(/\./,$Apache::lonnet::needsrelease{'parameter:type:randomizetry'});
-     $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'};
-             }
-             if ($fixeddom) {
-                 $instcodetitle .= '<br />('.$codedom.')';
-             }
-         }
-     }
-     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';
-     }
      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');
- Line 15942  sub parse_supplemental_title {
+ Line 14362  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) = @_;
      return unless ($symb);
- Line 15995  sub symb_to_docspath {
+ Line 14391  sub symb_to_docspath {
                      my $thistitle = $res->title();
                      $path .= '&'.
                               &Apache::lonhtmlcommon::entity_encode($thisurl).'&'.
-                              &escape($thistitle).
+                              &Apache::lonhtmlcommon::entity_encode($thistitle).
                               ':'.$res->randompick().
                               ':'.$res->randomout().
                               ':'.$res->encrypted().
- Line 16011  sub symb_to_docspath {
+ Line 14407  sub symb_to_docspath {
          }
          $path .= (($path ne '')? '&' : '').
                   &Apache::lonhtmlcommon::entity_encode($mapurl).'&'.
-                  &escape($maptitle).
+                  &Apache::lonhtmlcommon::entity_encode($maptitle).
                   ':'.$mapresobj->randompick().
                   ':'.$mapresobj->randomout().
                   ':'.$mapresobj->encrypted().
- Line 16024  sub symb_to_docspath {
+ Line 14420  sub symb_to_docspath {
              $maptitle = 'Main Content';
          }
          $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 Content').
                  ':::::&'.$path;
      }
      return $path;
- Line 16037  sub symb_to_docspath {
+ Line 14433  sub symb_to_docspath {
  sub captcha_display {
      my ($context,$lonhost) = @_;
      my ($output,$error);
-     my ($captcha,$pubkey,$privkey,$version) =
+     my ($captcha,$pubkey,$privkey) = &get_captcha_config($context,$lonhost);
-         &get_captcha_config($context,$lonhost);
      if ($captcha eq 'original') {
          $output = &create_captcha();
          unless ($output) {
              $error = 'captcha';
          }
      } elsif ($captcha eq 'recaptcha') {
-         $output = &create_recaptcha($pubkey,$version);
+         $output = &create_recaptcha($pubkey);
          unless ($output) {
              $error = 'recaptcha';
          }
      }
-     return ($output,$error,$captcha,$version);
+     return ($output,$error);
  }
  sub captcha_response {
      my ($context,$lonhost) = @_;
      my ($captcha_chk,$captcha_error);
-     my ($captcha,$pubkey,$privkey.$version) = &get_captcha_config($context,$lonhost);
+     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;
      }
- Line 16069  sub captcha_response {
+ Line 14464  sub captcha_response {
  sub get_captcha_config {
      my ($context,$lonhost) = @_;
-     my ($captcha,$pubkey,$privkey,$version,$hashtocheck);
+     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);
- Line 16085  sub get_captcha_config {
+ Line 14480  sub get_captcha_config {
                      }
                      if ($privkey && $pubkey) {
                          $captcha = 'recaptcha';
-                         $version = $hashtocheck->{'recaptchaversion'};
-                         if ($version ne '2') {
-                             $version = 1;
-                         }
                      } else {
                          $captcha = 'original';
                      }
- Line 16106  sub get_captcha_config {
+ Line 14497  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';
              }
- Line 16117  sub get_captcha_config {
+ Line 14504  sub get_captcha_config {
              $captcha = 'original';
          }
      }
-     return ($captcha,$pubkey,$privkey,$version);
+     return ($captcha,$pubkey,$privkey);
  }
  sub create_captcha {
- Line 16134  sub create_captcha {
+ Line 14521  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" />'.
+                      '<input type="text" size="5" name="code" value="" /><br />'.
-                       '<br />'.
+                      '<img src="'.$captcha_params{'www_output_dir'}.'/'.$md5sum.'.png" />';
-                       '<img src="'.$captcha_params{'www_output_dir'}.'/'.$md5sum.'.png" alt="captcha" />';
              last;
          }
      }
- Line 16176  sub check_captcha {
+ Line 14562  sub check_captcha {
  }
  sub create_recaptcha {
-     my ($pubkey,$version) = @_;
+     my ($pubkey) = @_;
-     if ($version >= 2) {
+     my $captcha = Captcha::reCAPTCHA->new;
-         return '<div class="g-recaptcha" data-sitekey="'.$pubkey.'"></div>';
+     return $captcha->get_options_setter({theme => 'white'})."\n".
-     } else {
+            $captcha->get_html($pubkey).
-         my $use_ssl;
+            &mt('If either word is hard to read, [_1] will replace them.',
-         if ($ENV{'SERVER_PORT'} == 443) {
+                '<image src="/res/adm/pages/refresh.gif" alt="reCAPTCHA refresh" />').
-             $use_ssl = 1;
+            '<br /><br />';
-         }
-         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 />';
-      }
  }
  sub check_recaptcha {
-     my ($privkey,$version) = @_;
+     my ($privkey) = @_;
      my $captcha_chk;
-     if ($version >= 2) {
+     my $captcha = Captcha::reCAPTCHA->new;
-         my $ua = LWP::UserAgent->new;
+     my $captcha_result =
-         $ua->timeout(10);
+         $captcha->check_answer(
-         my %info = (
+                                 $privkey,
-                      secret   => $privkey,
+                                 $ENV{'REMOTE_ADDR'},
-                      response => $env{'form.g-recaptcha-response'},
+                                 $env{'form.recaptcha_challenge_field'},
-                      remoteip => $ENV{'REMOTE_ADDR'},
+                                 $env{'form.recaptcha_response_field'},
-                    );
+                               );
-         my $response = $ua->post('https://www.google.com/recaptcha/api/siteverify',\%info);
+     if ($captcha_result->{is_valid}) {
-         if ($response->is_success)  {
+         $captcha_chk = 1;
-             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;
-         }
      }
      return $captcha_chk;
  }
- sub emailusername_info {
+ =pod
-     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);
- }
- sub cleanup_html {
+ =back
-     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 ();
- }
- # Use:
+ =cut
- #   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;
- }
 ;
  __END__;

FreeBSD-CVSweb <freebsd-cvsweb@FreeBSD.org>

Removed from v.1.1075.2.108
changed lines
	Added in v.1.1131