--- loncom/interface/londocs.pm	2012/12/02 18:44:04	1.516
+++ loncom/interface/londocs.pm	2013/01/10 17:10:30	1.531
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Documents
 #
-# $Id: londocs.pm,v 1.516 2012/12/02 18:44:04 raeburn Exp $
+# $Id: londocs.pm,v 1.531 2013/01/10 17:10:30 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -40,7 +40,6 @@ use Apache::lonxml;
 use Apache::lonclonecourse;
 use Apache::lonnavmaps;
 use Apache::lonnavdisplay();
-use Apache::lonuserstate();
 use Apache::lonextresedit();
 use HTML::Entities;
 use HTML::TokeParser;
@@ -153,7 +152,7 @@ sub dumpcourse {
             $r->print(&endContentScreen());
             return '';
         }
-	my ($ca,$cd)=split(/\@/,$env{'form.authorspace'});
+	my ($ca,$cd)=split(/\:/,$env{'form.authorspace'});
 	$r->print('<h3>'.&mt('Copying Files').'</h3>');
 	my $title=$env{'form.authorfolder'};
 	$title=&clean($title);
@@ -271,7 +270,9 @@ sub dumpcourse {
 
 sub group_import {
     my ($coursenum, $coursedom, $folder, $container, $caller, @files) = @_;
-
+    my ($donechk,$allmaps,%hierarchy,%titles,%addedmaps,%removefrommap,
+        %removeparam,$importuploaded,$fixuperrors);
+    $allmaps = {};
     while (@files) {
 	my ($name, $url, $residx) = @{ shift(@files) };
         if (($url =~ m{^/uploaded/\Q$coursedom\E/\Q$coursenum\E/(default_\d+\.)(page|sequence)$})
@@ -297,6 +298,24 @@ sub group_import {
             }
         }
 	if ($url) {
+            if (($caller eq 'londocs') &&
+                ($folder =~ /^default/)) {
+                unless ($donechk) {
+                    my $chome = &Apache::lonnet::homeserver($coursenum,$coursedom);
+                    my $cid = $coursedom.'_'.$coursenum;
+                    $allmaps =
+                        &Apache::loncommon::allmaps_incourse($coursedom,$coursenum,
+                                                             $chome,$cid);
+                    $donechk = 1;
+                }
+                if ($url =~ m{^/uploaded/\Q$coursedom\E/\Q$coursenum\E/(default_\d+\.)(page|sequence)$}) {
+                    &contained_map_check($url,$folder,\%removefrommap,\%removeparam,
+                                         \%addedmaps,\%hierarchy,\%titles,$allmaps);
+                    $importuploaded = 1;
+                } elsif ($url =~ m{^/res/.+\.(page|sequence)$}) {
+                    next if ($allmaps->{$url});
+                }
+            }
 	    if (!$residx
 		|| defined($LONCAPA::map::zombies[$residx])) {
 		$residx = &LONCAPA::map::getresidx($url,$residx);
@@ -310,7 +329,27 @@ sub group_import {
 		join(':', ($name, $url, $ext, 'normal', 'res'));
 	}
     }
-    return &storemap($coursenum, $coursedom, $folder.'.'.$container,1);
+    if ($importuploaded) {
+        my %import_errors;
+        my %updated = (
+                          removefrommap => \%removefrommap,
+                          removeparam   => \%removeparam,
+                      );
+        &apply_fixups($folder,1,$coursedom,$coursenum,\%import_errors,\%updated);
+        if (keys(%import_errors) > 0) {
+            $fixuperrors =
+                '<p span class="LC_warning">'."\n".
+                &mt('The following files are either dependencies of a web page or references within a folder and/or composite page for which errors occurred during import:')."\n".
+                '<ul>'."\n";
+            foreach my $key (sort(keys(%import_errors))) {
+                $fixuperrors .= '<li>'.$key.'</li>'."\n";
+            }
+            $fixuperrors .= '</ul></p>'."\n";
+        }
+    }
+    my ($errtext,$fatal) =
+        &storemap($coursenum, $coursedom, $folder.'.'.$container,1);
+    return ($errtext,$fatal,$fixuperrors);
 }
 
 sub log_docs {
@@ -391,18 +430,15 @@ sub docs_change_log {
     my %orderhash;
     my $container='sequence';
     my $pathitem;
-    if ($env{'form.pagepath'}) {
+    if ($env{'form.folderpath'} =~ /\:1$/) {
         $container='page';
-        $pathitem = '<input type="hidden" name="pagepath" value="'.
-                    &HTML::Entities::encode($env{'form.pagepath'},'<>&"').'" />';
-    } else {
-        my $folderpath=$env{'form.folderpath'};
-        if ($folderpath eq '') {
-            $folderpath = 'default&'.&escape(&mt('Main '.$crstype.' Documents'));
-        }
-        $pathitem = '<input type="hidden" name="folderpath" value="'.
-                    &HTML::Entities::encode($folderpath,'<>&"').'" />';
     }
+    my $folderpath=$env{'form.folderpath'};
+    if ($folderpath eq '') {
+        $folderpath = 'default&'.&escape(&mt('Main '.$crstype.' Documents').':::::');
+    }
+    $pathitem = '<input type="hidden" name="folderpath" value="'.
+                &HTML::Entities::encode($folderpath,'<>&"').'" />';
     my $readfile="/uploaded/$coursedom/$coursenum/$folder.$container";
     my $jumpto = $readfile;
     $jumpto =~ s{^/}{};
@@ -654,7 +690,7 @@ sub print_paste_buffer {
         $is_external = 1;
     }
 
-    my ($canpaste,$nopaste,$othercrs,$areachange,$is_uploaded_map);
+    my ($canpaste,$nopaste,$othercrs,$areachange);
     if ($folder =~ /^supplemental/) {
         $canpaste = &supp_pasteable($env{'docs.markedcopy_url'});
         unless ($canpaste) {
@@ -761,16 +797,9 @@ sub print_paste_buffer {
             }
         }
         $r->print('<br /><input type="submit" name="pastemarked" value="'.&mt('Paste').'" />'.$othercrs);
-        if ($container eq 'page') {
-            $r->print('
-        <input type="hidden" name="pagepath" value="'.&HTML::Entities::encode($env{'form.pagepath'},'<>&"').'" />
-        <input type="hidden" name="pagesymb" value="'.&HTML::Entities::encode($env{'form.pagesymb'},'<>&"').'" />
-');
-        } else {
-            $r->print('
+        $r->print('
         <input type="hidden" name="folderpath" value="'.&HTML::Entities::encode($env{'form.folderpath'},'<>&"').'" />
 ');
-        }
         $r->print('</form>');
     } else {
         $r->print(&mt('Paste buffer contains:').' '.$buffer.
@@ -905,30 +934,23 @@ sub do_paste_from_buffer {
         %dbcopies,%zombies,%params,%docmoves,%mapmoves,%newsubdir,%newurls);
     $oldurl = $url;
     if ($is_map) {
-        if ($folder =~ /^default/) {
-            my $lastchange = &Apache::lonnet::get_coursechange($coursedom,$coursenum);
-            if ($lastchange > $env{'request.course.tied'}) {
-                &reinit_role($coursedom,$coursenum,$env{"course.$env{'request.course.id'}.home"}); 
-            }
-        }
 # If pasting a map, check if map contains other maps
-        my (%allmaps,%hierarchy,%titles);
+        my ($allmaps,%hierarchy,%titles);
+        $allmaps = {};
         if ($folder =~ /^default/) {
-            my $navmap = Apache::lonnavmaps::navmap->new();
-            if (defined($navmap)) {
-                foreach my $res ($navmap->retrieveResources(undef,sub { $_[0]->is_map() },1,0,1)) {
-                    $allmaps{$res->src()} = 1;
-                }
-            }
+            $allmaps =
+                &Apache::loncommon::allmaps_incourse($coursedom,$coursenum,
+                                                     $env{"course.$env{'request.course.id'}.home"},
+                                                     $env{'request.course.id'});
         }
         &contained_map_check($url,$folder,\%removefrommap,\%removeparam,
-                             \%addedmaps,\%hierarchy,\%titles,\%allmaps);
+                             \%addedmaps,\%hierarchy,\%titles,$allmaps);
         if ($url=~ m{^/uploaded/}) {
             my $newurl;
             unless ($env{'form.docs.markedcopy_options'} eq 'move') {
                 ($newurl,my $error) = 
                     &get_newmap_url($url,$folder,$prefixchg,$coursedom,$coursenum,
-                                    $srcdom,$srcnum,\$title,\%allmaps,\%newurls);
+                                    $srcdom,$srcnum,\$title,$allmaps,\%newurls);
                 if ($error) {
                     return ($error);
                 }
@@ -944,14 +966,14 @@ sub do_paste_from_buffer {
             if (($srcdom ne $coursedom) || ($srcnum ne $coursenum) || ($prefixchg) ||
                 (($newurl ne '') && ($newurl ne $url))) {
                 unless (&url_paste_fixups($url,$folder,$prefixchg,$coursedom,$coursenum,
-                                          \%allmaps,\%rewrites,\%retitles,\%copies,\%dbcopies,
+                                          $allmaps,\%rewrites,\%retitles,\%copies,\%dbcopies,
                                           \%zombies,\%params,\%mapmoves,\%mapchanges,\%tomove,
                                           \%newsubdir,\%newurls)) {
                     $mapmoves{$url} = 1;
                 }
                 $url = $newurl;
             } elsif ($env{'docs.markedcopy_nested'}) {
-                &url_paste_fixups($url,$folder,$prefixchg,$coursedom,$coursenum,\%allmaps,\%rewrites,
+                &url_paste_fixups($url,$folder,$prefixchg,$coursedom,$coursenum,$allmaps,\%rewrites,
                                   \%retitles,\%copies,\%dbcopies,\%zombies,\%params,\%mapmoves,
                                   \%mapchanges,\%tomove,\%newsubdir,\%newurls); 
             }
@@ -960,7 +982,7 @@ sub do_paste_from_buffer {
             &Apache::lonnet::delenv('docs.markedcopy');
 # if pasting published map (main content are only) check map is not already in course
             if ($folder =~ /^default/) {
-                if ($allmaps{$url}) {
+                if ((ref($allmaps) eq 'HASH') && ($allmaps->{$url})) {
                     return (&mt('Paste failed: only one instance of a particular published sequence or page is allowed within each course.'));
                 }
             }
@@ -1026,11 +1048,28 @@ sub do_paste_from_buffer {
     }
 # Apply any changes to maps, or copy dependencies for uploaded HTML pages 
     my ($result,$save_err);
+    my %updated = (
+                    rewrites      => \%rewrites,
+                    zombies       => \%zombies,
+                    removefrommap => \%removefrommap,
+                    removeparam   => \%removeparam,
+                    dbcopies      => \%dbcopies,
+                    retitles      => \%retitles,
+                  );
+    my %info = (
+                   newsubdir => \%newsubdir,
+                   params    => \%params,
+                   before    => \%before,
+                   after     => \%after,
+               );
+    my %moves = (
+                   copies   => \%copies,
+                   docmoves => \%docmoves,
+                   mapmoves => \%mapmoves,
+                );
     $result =
-        &apply_fixups($folder,$is_map,$prefixchg,$coursedom,$coursenum,$oldurl,
-                      $url,\%removefrommap,\%removeparam,\%rewrites,\%retitles,
-                      \%copies,\%dbcopies,\%zombies,\%params,\%docmoves,
-                      \%mapmoves,\%newsubdir,$errors,\%before,\%after);
+        &apply_fixups($folder,$is_map,$coursedom,$coursenum,$errors,
+                      \%updated,\%info,\%moves,$prefixchg,$oldurl,$url,'paste');
     if ($result eq 'ok') {
         if ($is_map) { 
             my ($errtext,$fatal) = &mapread($coursenum,$coursedom,
@@ -1190,7 +1229,7 @@ sub contained_map_check {
                         $titles->{$url}{$token->[2]->{'id'}} = $token->[2]->{'title'};
                     } else {
                         if ($allmaps->{$ressrc}) {
-                            $removefrommap->{$url}{$token->[2]->{'id'}} = $ressrc; 
+                            $removefrommap->{$url}{$token->[2]->{'id'}} = $ressrc;
                         } elsif (ref($addedmaps->{$ressrc}) eq 'ARRAY') {
                             $removefrommap->{$url}{$token->[2]->{'id'}} = $ressrc;
                         } else {
@@ -1214,15 +1253,6 @@ sub contained_map_check {
     return;
 }
 
-sub reinit_role {
-    my ($cdom,$cnum,$chome) = @_;
-    my ($furl,$ferr) = &Apache::lonuserstate::readmap("$cdom/$cnum");
-    unless ($ferr) {
-        &Apache::loncommon::update_content_constraints($cdom,$cnum,$chome,$cdom.'_'.$cnum);
-    }
-    return;
-}
-
 sub url_paste_fixups {
     my ($oldurl,$folder,$prefixchg,$cdom,$cnum,$allmaps,$rewrites,$retitles,$copies,
         $dbcopies,$zombies,$params,$mapmoves,$mapchanges,$tomove,$newsubdir,$newurls) = @_;
@@ -1341,23 +1371,71 @@ sub url_paste_fixups {
 }
 
 sub apply_fixups {
-    my ($folder,$is_map,$prefixchg,$cdom,$cnum,$oldurl,$url,$removefrommap,
-        $removeparam,$rewrites,$retitles,$copies,$dbcopies,$zombies,$params,
-        $docmoves,$mapmoves,$newsubdir,$errors,$before,$after) = @_;
-    foreach my $key (keys(%{$copies}),keys(%{$docmoves})) {
+    my ($folder,$is_map,$cdom,$cnum,$errors,$updated,$info,$moves,$prefixchg,
+        $oldurl,$url,$caller) = @_;
+    my (%rewrites,%zombies,%removefrommap,%removeparam,%dbcopies,%retitles,
+        %params,%newsubdir,%before,%after,%copies,%docmoves,%mapmoves);
+    if (ref($updated) eq 'HASH') {
+        if (ref($updated->{'rewrites'}) eq 'HASH') {
+            %rewrites = %{$updated->{'rewrites'}};
+        }
+        if (ref($updated->{'zombies'}) eq 'HASH') {
+            %zombies = %{$updated->{'zombies'}};
+        }
+        if (ref($updated->{'removefrommap'}) eq 'HASH') {
+            %removefrommap = %{$updated->{'removefrommap'}};
+        }
+        if (ref($updated->{'removeparam'}) eq 'HASH') {
+            %removeparam = %{$updated->{'removeparam'}};
+        }
+        if (ref($updated->{'dbcopies'}) eq 'HASH') {
+            %dbcopies = %{$updated->{'dbcopies'}};
+        }
+        if (ref($updated->{'retitles'}) eq 'HASH') {
+            %retitles = %{$updated->{'retitles'}};
+        }
+    }
+    if (ref($info) eq 'HASH') {
+        if (ref($info->{'newsubdir'}) eq 'HASH') {
+            %newsubdir = %{$info->{'newsubdir'}};
+        }
+        if (ref($info->{'params'}) eq 'HASH') {
+            %params = %{$info->{'params'}};
+        }
+        if (ref($info->{'before'}) eq 'HASH') {
+            %before = %{$info->{'before'}};
+        }
+        if (ref($info->{'after'}) eq 'HASH') {
+            %after = %{$info->{'after'}};
+        }
+    }
+    if (ref($moves) eq 'HASH') {
+        if (ref($moves->{'copies'}) eq 'HASH') {
+            %copies = %{$moves->{'copies'}};
+        }
+        if (ref($moves->{'docmoves'}) eq 'HASH') {
+            %docmoves = %{$moves->{'docmoves'}};
+        }
+        if (ref($moves->{'mapmoves'}) eq 'HASH') {
+            %mapmoves = %{$moves->{'mapmoves'}};
+        }
+    }
+    foreach my $key (keys(%copies),keys(%docmoves)) {
         my @allcopies;
-        if (ref($copies->{$key}) eq 'HASH') {
-            my %added;
-            foreach my $innerkey (keys(%{$copies->{$key}})) {
-                if (($innerkey ne '') && (!$added{$innerkey})) {
-                    push(@allcopies,$innerkey);
-                    $added{$innerkey} = 1;
+        if (exists($copies{$key})) {
+            if (ref($copies{$key}) eq 'HASH') {
+                my %added;
+                foreach my $innerkey (keys(%{$copies{$key}})) {
+                    if (($innerkey ne '') && (!$added{$innerkey})) {
+                        push(@allcopies,$innerkey);
+                        $added{$innerkey} = 1;
+                    }
                 }
+                undef(%added);
             }
-            undef(%added);
         }
         if ($key eq $oldurl) {
-            if ((exists($docmoves->{$key}))) {
+            if ((exists($docmoves{$key}))) {
                 unless (grep(/^\Q$oldurl\E/,@allcopies)) {
                     push(@allcopies,$oldurl);
                 }
@@ -1371,16 +1449,16 @@ sub apply_fixups {
                     my $content = &Apache::lonnet::getfile($item);
                     unless ($content eq '-1') {
                         my $storefn;
-                        if (($key eq $oldurl) && (ref($docmoves) eq 'HASH') && (exists($docmoves->{$key}))) {
-                            $storefn = $docmoves->{$key};
+                        if (($key eq $oldurl) && (exists($docmoves{$key}))) {
+                            $storefn = $docmoves{$key};
                         } else {
                             $storefn = $relpath;
                             $storefn =~s{^/uploaded/$match_domain/$match_courseid/}{};
-                            if ($prefixchg) {
-                                $storefn =~ s/^\Q$before->{'doc'}\E/$after->{'doc'}/;
+                            if ($prefixchg && $before{'doc'} && $after{'doc'}) {
+                                $storefn =~ s/^\Q$before{'doc'}\E/$after{'doc'}/;
                             }
-                            if ($newsubdir->{$key}) {
-                                $storefn =~ s#^(docs|supplemental)/\Q$oldsubdir\E/#$1/$newsubdir->{$key}#;
+                            if ($newsubdir{$key}) {
+                                $storefn =~ s#^(docs|supplemental)/\Q$oldsubdir\E/#$1/$newsubdir{$key}#;
                             }
                         }
                         &copy_dependencies($item,$storefn,$relpath,$errors,\$content);
@@ -1388,7 +1466,7 @@ sub apply_fixups {
                             &Apache::lonclonecourse::writefile($env{'request.course.id'},
                                                                $storefn.$fname,$content);
                         if ($copyurl eq '/adm/notfound.html') {
-                            if ((ref($docmoves) eq 'HASH') && (exists($docmoves->{$oldurl}))) {
+                            if (exists($docmoves{$oldurl})) {
                                 return &mt('Paste failed: an error occurred copying the file.');
                             } elsif (ref($errors) eq 'HASH') {
                                 $errors->{$item} = 1;
@@ -1399,14 +1477,14 @@ sub apply_fixups {
             }
         }
     }
-    foreach my $key (keys(%{$mapmoves})) {
+    foreach my $key (keys(%mapmoves)) {
         my $storefn=$key;
         $storefn=~s{^/uploaded/$match_domain/$match_courseid/}{};
-        if ($prefixchg) {
-            $storefn =~ s/^\Q$before->{'map'}\E/$after->{'map'}/;
+        if ($prefixchg && $before{'map'} && $after{'map'}) {
+            $storefn =~ s/^\Q$before{'map'}\E/$after{'map'}/;
         }
-        if ($newsubdir->{$key}) {
-            $storefn =~ s/^((?:default|supplemental)_)(\d+)/$1$newsubdir->{$key}/;
+        if ($newsubdir{$key}) {
+            $storefn =~ s/^((?:default|supplemental)_)(\d+)/$1$newsubdir{$key}/;
         }
         my $mapcontent = &Apache::lonnet::getfile($key);
         if ($mapcontent eq '-1') {
@@ -1426,48 +1504,39 @@ sub apply_fixups {
     }
     my %updates;
     if ($is_map) {
-        foreach my $key (keys(%{$rewrites})) {
-            $updates{$key} = 1;
-        }
-        foreach my $key (keys(%{$zombies})) {
-            $updates{$key} = 1;
-        }
-        foreach my $key (keys(%{$removefrommap})) {
-            $updates{$key} = 1;
-        }
-        foreach my $key (keys(%{$removeparam})) {
-            $updates{$key} = 1;
-        }
-        foreach my $key (keys(%{$dbcopies})) {
-            $updates{$key} = 1;
-        }
-        foreach my $key (keys(%{$retitles})) {
-            $updates{$key} = 1;
+        if (ref($updated) eq 'HASH') {
+            foreach my $type (keys(%{$updated})) {
+                if (ref($updated->{$type}) eq 'HASH') {
+                    foreach my $key (keys(%{$updated->{$type}})) {
+                        $updates{$key} = 1;
+                    }
+                }
+            }
         }
         foreach my $key (keys(%updates)) {
             my (%torewrite,%toretitle,%toremove,%remparam,%currparam,%zombie,%newdb);
-            if (ref($rewrites->{$key}) eq 'HASH') {
-                %torewrite = %{$rewrites->{$key}};
+            if (ref($rewrites{$key}) eq 'HASH') {
+                %torewrite = %{$rewrites{$key}};
             }
-            if (ref($retitles->{$key}) eq 'HASH') {
-                %toretitle = %{$retitles->{$key}};
+            if (ref($retitles{$key}) eq 'HASH') {
+                %toretitle = %{$retitles{$key}};
             }
-            if (ref($removefrommap->{$key}) eq 'HASH') {
-                %toremove = %{$removefrommap->{$key}};
+            if (ref($removefrommap{$key}) eq 'HASH') {
+                %toremove = %{$removefrommap{$key}};
             }
-            if (ref($removeparam->{$key}) eq 'HASH') {
-                %remparam = %{$removeparam->{$key}};
+            if (ref($removeparam{$key}) eq 'HASH') {
+                %remparam = %{$removeparam{$key}};
             }
-            if (ref($zombies->{$key}) eq 'HASH') {
-                %zombie = %{$zombies->{$key}};
+            if (ref($zombies{$key}) eq 'HASH') {
+                %zombie = %{$zombies{$key}};
             }
-            if (ref($dbcopies->{$key}) eq 'HASH') {
-                foreach my $item (keys(%{$dbcopies->{$key}})) {
+            if (ref($dbcopies{$key}) eq 'HASH') {
+                foreach my $item (keys(%{$dbcopies{$key}})) {
                     $newdb{$item} = &dbcopy($item);
                 }
             }
-            if (ref($params->{$key}) eq 'HASH') {
-                %currparam = %{$params->{$key}};
+            if (ref($params{$key}) eq 'HASH') {
+                %currparam = %{$params{$key}};
             }
             my ($errtext,$fatal) = &LONCAPA::map::mapread($key);
             if ($fatal) {
@@ -1481,40 +1550,41 @@ sub apply_fixups {
                     }
                 }
             }
-            for (my $i=0; $i<@LONCAPA::map::resources; $i++) {
-                if (defined($LONCAPA::map::resources[$i])) {
+            for (my $i=0; $i<@LONCAPA::map::order; $i++) {
+                my $idx = $LONCAPA::map::order[$i];
+                if (defined($LONCAPA::map::resources[$idx])) {
                     my $changed;
-                    my ($title,$src,$ext,$type)=split(/\:/,$LONCAPA::map::resources[$i]);
-                    if ($toremove{$src} eq $i) {
+                    my ($title,$src,$ext,$type)=split(/\:/,$LONCAPA::map::resources[$idx]);
+                    if ((exists($toremove{$idx})) && ($toremove{$idx} eq $src)) {
                         splice(@LONCAPA::map::order,$i,1);
-                        if (ref($currparam{$i}) eq 'ARRAY') {
-                            foreach my $name (@{$currparam{$i}}) {
-                                &LONCAPA::map::delparameter($i,'parameter_'.$name);
+                        if (ref($currparam{$idx}) eq 'ARRAY') {
+                            foreach my $name (@{$currparam{$idx}}) {
+                                &LONCAPA::map::delparameter($idx,'parameter_'.$name);
                             }
                         }
                         next;
                     }
                     my $origsrc = $src;
-                    if ((exists($toretitle{$src})) && ($toretitle{$src} eq $i)) {
+                    if ((exists($toretitle{$src})) && ($toretitle{$src} eq $idx)) {
                         if ($title =~ m{^\d+\Q___&amp;&amp;&amp;___\E$match_username\Q___&amp;&amp;&amp;___\E$match_domain\Q___&amp;&amp;&amp;___\E(.+)$}) {
                             $changed = 1;
                         }
                     }
-                    if ((exists($torewrite{$src})) && ($torewrite{$src} eq $i)) {
+                    if ((exists($torewrite{$src})) && ($torewrite{$src} eq $idx)) {
                         $src =~ s{^/(uploaded|adm|public)/$match_domain/$match_courseid/}{/$1/$cdom/$cnum/};
                         if ($origsrc =~ m{^/uploaded/}) {
-                            if ($prefixchg) {
+                            if ($prefixchg && $before{'map'} && $after{'map'}) {
                                 if ($src =~ /\.(page|sequence)$/) {
-                                    $src =~ s#^(/uploaded/$match_domain/$match_courseid/)\Q$before->{'map'}\E#$1$after->{'map'}#;
+                                    $src =~ s#^(/uploaded/$match_domain/$match_courseid/)\Q$before{'map'}\E#$1$after{'map'}#;
                                 } else {
-                                    $src =~ s#^(/uploaded/$match_domain/$match_courseid/)\Q$before->{'doc'}\E#$1$after->{'doc'}#;
+                                    $src =~ s#^(/uploaded/$match_domain/$match_courseid/)\Q$before{'doc'}\E#$1$after{'doc'}#;
                                 }
                             }
-                            if ($newsubdir->{$origsrc}) {
+                            if ($newsubdir{$origsrc}) {
                                 if ($src =~ /\.(page|sequence)$/) {
-                                    $src =~ s#^(/uploaded/$match_domain/$match_courseid/(?:default|supplemental)_)(\d+)#$1$newsubdir->{$origsrc}#;
+                                    $src =~ s#^(/uploaded/$match_domain/$match_courseid/(?:default|supplemental)_)(\d+)#$1$newsubdir{$origsrc}#;
                                 } else {
-                                    $src =~ s#^(/uploaded/$match_domain/$match_courseid/\w+/)(\d+)#$1$newsubdir->{$origsrc}#;
+                                    $src =~ s#^(/uploaded/$match_domain/$match_courseid/\w+/)(\d+)#$1$newsubdir{$origsrc}#;
                                 }
                             }
                         }
@@ -1524,7 +1594,7 @@ sub apply_fixups {
                         $changed = 1;
                     }
                     if ($changed) {
-                        $LONCAPA::map::resources[$i] = join(':',($title,$src,$ext,$type));
+                        $LONCAPA::map::resources[$idx] = join(':',($title,$src,$ext,$type));
                     }
                 }
             }
@@ -1542,21 +1612,23 @@ sub apply_fixups {
             } else {
                 $storefn = $key;
                 $storefn=~s{^/uploaded/$match_domain/$match_courseid/}{};
-                if ($prefixchg) {
-                    $storefn =~ s/^\Q$before->{'map'}\E/$after->{'map'}/;
+                if ($prefixchg && $before{'map'} && $after{'map'}) {
+                    $storefn =~ s/^\Q$before{'map'}\E/$after{'map'}/;
                 }
-                if ($newsubdir->{$key}) {
-                    $storefn =~ s/^((?:default|supplemental)_)(\d+)/$1$newsubdir->{$key}/;
+                if ($newsubdir{$key}) {
+                    $storefn =~ s/^((?:default|supplemental)_)(\d+)/$1$newsubdir{$key}/;
                 }
             }
             my $report;
             if ($folder !~ /^supplemental/) {
                 $report = 1;
             }
-            my ($outtext,$errtext) =
+            (my $outtext,$errtext) =
                 &LONCAPA::map::storemap("/uploaded/$cdom/$cnum/$storefn",1,$report);
             if ($errtext) {
-                return &mt('Paste failed: an error occurred saving the folder or page.');
+                if ($caller eq 'paste') {
+                    return &mt('Paste failed: an error occurred saving the folder or page.');
+                }
             }
         }
     }
@@ -1692,16 +1764,23 @@ sub handle_edit_cmd {
 sub editor {
     my ($r,$coursenum,$coursedom,$folder,$allowed,$upload_output,$crstype,
         $supplementalflag,$orderhash,$iconpath,$pathitem)=@_;
-    my $container= ($env{'form.pagepath'}) ? 'page'
-		                           : 'sequence';
-    my ($randompick,$ishidden,$isencrypted,$plain,$is_random_order);
+    my ($randompick,$ishidden,$isencrypted,$plain,$is_random_order,$container);
     if ($allowed) {
-        (my $breadcrumbtrail,$randompick,$ishidden,$isencrypted,$plain,$is_random_order) =
+        (my $breadcrumbtrail,$randompick,$ishidden,$isencrypted,$plain,
+         $is_random_order,$container) =
             &Apache::lonhtmlcommon::docs_breadcrumbs($allowed,$crstype,1);
         $r->print($breadcrumbtrail);
+    } elsif ($env{'form.folderpath'} =~ /\:1$/) {
+        $container = 'page'; 
+    } else {
+        $container = 'sequence';
     }
 
-    my $jumpto = "uploaded/$coursedom/$coursenum/$folder.$container";
+    my $jumpto;
+
+    unless ($supplementalflag) {
+        $jumpto = "'uploaded/$coursedom/$coursenum/$folder.$container'";
+    }
 
     unless ($allowed) {
         $randompick = -1;
@@ -1793,9 +1872,13 @@ sub editor {
 		    push(@imports, [$name, $url, $residx]);
 		}
 	    }
-	    ($errtext,$fatal)=&group_import($coursenum, $coursedom, $folder,
-					    $container,'londocs',@imports);
+            ($errtext,$fatal,my $fixuperrors) =
+                &group_import($coursenum, $coursedom, $folder,$container,
+                              'londocs',@imports);
 	    return $errtext if ($fatal);
+            if ($fixuperrors) {
+                $r->print($fixuperrors);
+            }
 	}
 # Loading a complete map
 	if ($env{'form.loadmap'}) {
@@ -1862,13 +1945,23 @@ sub editor {
         unless ($name) { $idx++; next; }
         $output .= &entryline($idx,$name,$url,$folder,$allowed,$res,
                               $coursenum,$coursedom,$crstype,
-                              $pathitem,$supplementalflag);
+                              $pathitem,$supplementalflag,$container);
         $idx++;
         $shown++;
     }
     &Apache::loncommon::end_data_table_count();
 
     if (($allowed) || ($supplementalflag && $folder eq 'supplemental')) {
+        my $toolslink = '<table><tr><td>'
+                       .&Apache::loncommon::help_open_menu('Navigation Screen',
+                                                           'Navigation_Screen',undef,'RAT')
+                       .'</td><td class="LC_middle">'.&mt('Tools:').'</td>'
+                       .'<td align="left"><ul id="LC_toolbar">'
+                       .'<li><a href="/adm/coursedocs?forcesupplement=1&amp;command=editsupp" '
+                       .'id="LC_content_toolbar_edittoplevel" '
+                       .'class="LC_toolbarItem" '
+                       .'title="'.&mt('Supplemental Content Editor').'">'
+                       .'</a></li></ul></td></tr></table><br />';
         if ($shown) {
             if ($allowed) {
                 $to_show = &Apache::loncommon::start_scrollbox('900px','880px','400px','contentscroll')
@@ -1886,19 +1979,15 @@ sub editor {
                            .'<br style="line-height:2px;" />'
                            .&Apache::loncommon::end_scrollbox();
             } else {
-                $to_show = '<table><tr><td>'.&Apache::loncommon::help_open_menu('Navigation Screen','Navigation_Screen',undef,'RAT')
-                           .'</td><td class="LC_middle">'.&mt('Tools:').'</td>'
-                           .'<td align="left"><ul id="LC_toolbar">'
-                           .'<li><a href="/adm/coursedocs?forcesupplement=1" '
-                           .'id="LC_content_toolbar_edittoplevel" '
-                           .'class="LC_toolbarItem" '
-                           .'title="'.&mt('Supplemental Content Editor').'">'
-                           .'</a></li></ul></td></tr></table><br />'
+                $to_show .= $toolslink
                            .&Apache::loncommon::start_data_table('LC_tableOfContent')
                            .$output.' '
                            .&Apache::loncommon::end_data_table();
             }
         } else {
+            if (!$allowed) {
+                $to_show .= $toolslink;
+            }
             $to_show .= &Apache::loncommon::start_scrollbox('400px','380px','200px','contentscroll')
                        .'<div class="LC_info" id="contentlist">'
                        .&mt('Currently no documents.')
@@ -1948,7 +2037,7 @@ sub process_file_upload {
         my $errtext='';
         my $fatal=0;
         my $container='sequence';
-        if ($env{'form.pagepath'}) {
+        if ($env{'form.folderpath'} =~ /:1$/) {
             $container='page';
         }
         ($errtext,$fatal)=
@@ -2038,7 +2127,6 @@ sub process_file_upload {
                 my $archiveurl = &HTML::Entities::encode($url,'<>&"');
                 my %archiveitems = (
                     folderpath => $env{'form.folderpath'},
-                    pagepath   => $env{'form.pagepath'},
                     cmd        => $nextphase,
                     newidx     => $newidx,
                     position   => $position,
@@ -2093,16 +2181,14 @@ sub is_supplemental_title {
 
 sub entryline {
     my ($index,$title,$url,$folder,$allowed,$residx,$coursenum,$coursedom,
-        $crstype,$pathitem,$supplementalflag)=@_;
-    my ($foldertitle,$pagetitle,$renametitle);
+        $crstype,$pathitem,$supplementalflag,$container)=@_;
+    my ($foldertitle,$renametitle);
     if (&is_supplemental_title($title)) {
 	($title,$foldertitle,$renametitle) = &Apache::loncommon::parse_supplemental_title($title);
-	$pagetitle = $foldertitle;
     } else {
 	$title=&HTML::Entities::encode($title,'"<>&\'');
 	$renametitle=$title;
 	$foldertitle=$title;
-	$pagetitle=$title;
     }
 
     my $orderidx=$LONCAPA::map::order[$index];
@@ -2113,19 +2199,17 @@ sub entryline {
     my $line=&Apache::loncommon::start_data_table_row();
     my ($form_start,$form_end,$form_common);
 # Edit commands
-    my ($container, $type, $esc_path, $path, $symb);
+    my ($type, $esc_path, $path, $symb);
+    if ($container eq 'page') {
+        $type = 'page';
+    } else {
+        $type = 'folder';
+    }
     if ($env{'form.folderpath'}) {
-	$type = 'folder';
-        $container = 'sequence';
 	$esc_path=&escape($env{'form.folderpath'});
 	$path = &HTML::Entities::encode($env{'form.folderpath'},'<>&"');
 	# $htmlfoldername=&HTML::Entities::encode($env{'form.foldername'},'<>&"');
     }
-    if ($env{'form.pagepath'}) {
-        $type = $container = 'page';
-        $esc_path=&escape($env{'form.pagepath'});
-	$path = &HTML::Entities::encode($env{'form.pagepath'},'<>&"');
-    }
     my $isexternal;
     if ($residx) {
         my $currurl = $url;
@@ -2142,7 +2226,7 @@ sub entryline {
                                                  &Apache::lonnet::declutter($currurl));
         }
     }
-    my %lt;
+    my ($renamelink,%lt);
     if ($allowed) {
 	my $incindex=$index+1;
 	my $selectbox='';
@@ -2239,7 +2323,7 @@ sub entryline {
                 $nocopy=1;
             }
         }
-        my ($copylink,$cutlink,$removelink,$renamelink);
+        my ($copylink,$cutlink,$removelink);
 	my $skip_confirm = 0;
 	if ( $folder =~ /^supplemental/
 	     || ($url =~ m{( /smppg$
@@ -2258,7 +2342,7 @@ sub entryline {
 ENDCOPY
         } else {
 	    $copylink=(<<ENDCOPY);
-<a href="javascript:markcopy('$esc_path','$index','$renametitle','$container','$symb','$folder');" class="LC_docs_copy">$lt{'cp'}</a>
+<a href="javascript:markcopy('$esc_path','$index','$renametitle','$container','$folder');" class="LC_docs_copy">$lt{'cp'}</a>
 ENDCOPY
         }
 	if ($nocut) {
@@ -2267,7 +2351,7 @@ ENDCOPY
 ENDCUT
         } else {
 	    $cutlink=(<<ENDCUT);
-<a href="javascript:cutres('$esc_path','$index','$renametitle','$container','$symb','$folder',$skip_confirm);" class="LC_docs_cut">$lt{'ct'}</a>
+<a href="javascript:cutres('$esc_path','$index','$renametitle','$container','$folder',$skip_confirm);" class="LC_docs_cut">$lt{'ct'}</a>
 ENDCUT
         }
         if ($noremove) {
@@ -2276,12 +2360,14 @@ ENDCUT
 ENDREM
         } else {
             $removelink=(<<ENDREM);
-<a href='javascript:removeres("$esc_path","$index","$renametitle","$container","$symb",$skip_confirm);' class="LC_docs_remove">$lt{'rm'}</a>
+<a href='javascript:removeres("$esc_path","$index","$renametitle",$skip_confirm);' class="LC_docs_remove">$lt{'rm'}</a>
 ENDREM
         }
-        $renamelink=(<<ENDREN);
-<a href='javascript:changename("$esc_path","$index","$renametitle","$container","$symb");' class="LC_docs_rename">$lt{'rn'}</a>
+        unless ($isexternal) {
+            $renamelink=(<<ENDREN);
+<a href='javascript:changename("$esc_path","$index","$renametitle");' class="LC_docs_rename">$lt{'rn'}</a>
 ENDREN
+        }
 	$form_start = '
    <form action="/adm/coursedocs" method="post">
 ';
@@ -2313,7 +2399,6 @@ END
 </td>
 <td class="LC_docs_entry_commands"><span class="LC_nobreak">
 $removelink
-$renamelink
 $cutlink
 $copylink
 </span>
@@ -2325,22 +2410,18 @@ END
     my ($extension)=($url=~/\.(\w+)$/);
     my $uploaded=($url=~/^\/*uploaded\//);
     my $icon=&Apache::loncommon::icon($url);
-    my $isfolder=0;
-    my $ispage=0;
-    my $folderarg;
-    my $pagearg;
-    my $pagefile;
+    my $isfolder;
+    my $ispage;
+    my $containerarg;
     if ($uploaded) {
         if (($extension eq 'sequence') || ($extension eq 'page')) {
             $url=~/\Q$coursenum\E\/([\/\w]+)\.\Q$extension\E$/;
-            my $containerarg = $1;
+            $containerarg = $1;
 	    if ($extension eq 'sequence') {
 	        $icon=$iconpath.'navmap.folder.closed.gif';
-                $folderarg=$containerarg;
                 $isfolder=1;
             } else {
                 $icon=$iconpath.'page.gif';
-                $pagearg=$containerarg;
                 $ispage=1;
             }
             if ($allowed) {
@@ -2353,15 +2434,11 @@ END
 	}
     }
 
-    my $editlink;
+    my ($editlink,$extresform);
     my $orig_url = $url;
     $orig_url=~s{http(&colon;|:)//https(&colon;|:)//}{https$2//};
     $url=~s{^http(|s)(&colon;|:)//}{/adm/wrapper/ext/};
     if (!$supplementalflag && $residx && $symb) {
-        if ($container eq 'page') {
-            $url=&Apache::lonnet::clutter((&Apache::lonnet::decode_symb($symb))[2]);
-            $url.=(($url=~/\?/)?'&':'?').'symb='.&escape($symb);
-        } 
         if ((!$isfolder) && (!$ispage)) {
 	    (undef,undef,$url)=&Apache::lonnet::decode_symb($symb);
 	    $url=&Apache::lonnet::clutter($url);
@@ -2386,17 +2463,17 @@ END
 	}
     }
     my ($rand_pick_text,$rand_order_text);
-    if ($isfolder || $extension eq 'sequence') {
+    if ($isfolder || $ispage || $extension eq 'sequence' || $extension eq 'page') {
 	my $foldername=&escape($foldertitle);
 	my $folderpath=$env{'form.folderpath'};
 	if ($folderpath) { $folderpath.='&' };
         if (!$allowed && $supplementalflag) {
-            $folderpath.=$folderarg.'&'.$foldername;
+            $folderpath.=$containerarg.'&'.$foldername;
             $url.='folderpath='.&escape($folderpath);
         } else {
 # Append randompick number, hidden, and encrypted with ":" to foldername,
 # so it gets transferred between levels
-	    $folderpath.=$folderarg.'&'.$foldername.
+	    $folderpath.=$containerarg.'&'.$foldername.
                                               ':'.(&LONCAPA::map::getparameter($orderidx,
                                               'parameter_randompick'))[0]
                                                .':'.((&LONCAPA::map::getparameter($orderidx,
@@ -2404,7 +2481,8 @@ END
                                                .':'.((&LONCAPA::map::getparameter($orderidx,
                                               'parameter_encrypturl'))[0]=~/^yes$/i)
                                                .':'.((&LONCAPA::map::getparameter($orderidx,
-                                              'parameter_randomorder'))[0]=~/^yes$/i);
+                                              'parameter_randomorder'))[0]=~/^yes$/i)
+                                               .':'.$ispage;
 	    $url.='folderpath='.&escape($folderpath);
             my $rpicknum = (&LONCAPA::map::getparameter($orderidx,
                                                         'parameter_randompick'))[0];
@@ -2438,20 +2516,12 @@ $form_common.'
             $url .= '&amp;idx='.$orderidx;
         }
     }
-    if ($ispage) {
-        my $pagename=&escape($pagetitle);
-        my $pagepath;
-        my $folderpath=$env{'form.folderpath'};
-        if ($folderpath) { $pagepath = $folderpath.'&' };
-        $pagepath.=$pagearg.'&'.$pagename;
-	$url.='pagepath='.&escape($pagepath).
-	    '&amp;pagesymb='.&escape($symb);
-    }
+    my ($tdalign,$tdwidth);
     if ($allowed) {
         my $fileloc = 
             &Apache::lonnet::declutter(&Apache::lonnet::filelocation('',$orig_url));
         if ($isexternal) {
-            $editlink = '&nbsp;'.
+            ($editlink,$extresform) = 
                 &Apache::lonextresedit::extedit_form(0,$residx,$orig_url,$title,$pathitem);
         } elsif (!$isfolder && !$ispage) {
             my ($cfile,$home,$switchserver,$forceedit,$forceview) = 
@@ -2463,21 +2533,23 @@ $form_common.'
                                                             $forceedit,
                                                             undef,$symb,
                                                             &escape($env{'form.folderpath'}),
-                                                            $renametitle);
+                                                            $renametitle,'','',1);
                 if ($jscall) {
-                    $editlink = '&nbsp;<a class="LC_docs_ext_edit" href="javascript:'.
-                                $jscall.'" >'.&mt('Edit').'</a>';
+                    $editlink = '<a class="LC_docs_ext_edit" href="javascript:'.
+                                $jscall.'" >'.&mt('Edit').'</a>&nbsp;'."\n";
                 }
             }
         }
+        $tdalign = ' align="right" valign="top"';
+        $tdwidth = ' width="80%"';
     }
     my $reinit;
     if ($crstype eq 'Community') {
         $reinit = &mt('(re-initialize community to access)');
     } else {
         $reinit = &mt('(re-initialize course to access)');
-    }  
-    $line.='<td>';
+    }
+    $line.='<td class="LC_docs_entry_commands"'.$tdalign.'><span class="LC_nobreak">'.$editlink.$renamelink;
     if (($url=~m{/adm/(coursedocs|supplemental)}) || (!$allowed && $url)) {
        $line.='<a href="'.$url.'"><img src="'.$icon.'" alt="" class="LC_icon" /></a>';
     } elsif ($url) {
@@ -2486,7 +2558,7 @@ $form_common.'
     } else {
        $line.='<img src="'.$icon.'" alt="" class="LC_icon" />';
     }
-    $line.='</td><td>';
+    $line.='</span></td><td'.$tdwidth.'>';
     if (($url=~m{/adm/(coursedocs|supplemental)}) || (!$allowed && $url)) {
        $line.='<a href="'.$url.'">'.$title.'</a>';
     } elsif ($url) {
@@ -2495,7 +2567,7 @@ $form_common.'
     } else {
        $line.=$title.' <span class="LC_docs_reinit_warn">'.$reinit.'</span>';
     }
-    $line.=$editlink."</td>";
+    $line.="$extresform</td>";
     $rand_pick_text = '&nbsp;' if ($rand_pick_text eq '');
     $rand_order_text = '&nbsp;' if ($rand_order_text eq '');
     if (($allowed) && ($folder!~/^supplemental/)) {
@@ -2645,7 +2717,7 @@ sub checkonthis {
              $r->print('<span class="LC_error">'.&mt('connection down').'</span>');
           } elsif ($result eq 'not_found') {
 	      unless ($url=~/\$/) {
-		  $r->print('<span class="LC_error">'.&mt('not found').'</b></span>');
+		  $r->print('<span class="LC_error">'.&mt('not found').'</span>');
 	      } else {
 		  $r->print('<span class="LC_error">'.&mt('unable to verify variable URL').'</span>');
 	      }
@@ -2691,7 +2763,7 @@ sub list_symbs {
             $r->print(&Apache::loncommon::start_data_table_row().
                       '<td>'.$res->compTitle().'</td>'.
                       '<td>'.$res->symb().'</td>'.
-                      &Apache::loncommon::start_data_table_row());
+                      &Apache::loncommon::end_data_table_row());
             $count ++;
         }
         if (!$count) {
@@ -2701,6 +2773,7 @@ sub list_symbs {
         }
         $r->print(&Apache::loncommon::end_data_table());
     }
+    $r->print(&endContentScreen());
 }
 
 
@@ -2731,6 +2804,7 @@ sub verifycontent {
    }
    &untiehash();
    $r->print('<p class="LC_success">'.&mt('Done').'</p>');
+    $r->print(&endContentScreen());
 }
 
 
@@ -2810,7 +2884,7 @@ sub checkversions {
     if ($env{'form.timerange'} eq 'all') {
 # show all documents
 	$header=&mt('All Documents in '.$crstype);
-	$allsel=1;
+	$allsel=' selected="selected"';
 	foreach my $key (keys(%hash)) {
 	    if ($key=~/^ids\_(\/res\/.+)$/) {
 		my $src=$1;
@@ -2831,19 +2905,19 @@ sub checkversions {
 		.&mt('seconds');
 	    if ($env{'form.timerange'}==-1) {
 		$seltext='since start of course';
-		$startsel='selected';
+		$startsel=' selected="selected"';
 		$env{'form.timerange'}=time;
 	    }
 	    $starttime=time-$env{'form.timerange'};
 	    if ($env{'form.timerange'}==2592000) {
 		$seltext=&mt('during the last month').' ('.&Apache::lonlocal::locallocaltime($starttime).')';
-		$monthsel='selected';
+		$monthsel=' selected="selected"';
 	    } elsif ($env{'form.timerange'}==604800) {
 		$seltext=&mt('during the last week').' ('.&Apache::lonlocal::locallocaltime($starttime).')';
-		$weeksel='selected';
+		$weeksel=' selected="selected"';
 	    } elsif ($env{'form.timerange'}==86400) {
 		$seltext=&mt('since yesterday').' ('.&Apache::lonlocal::locallocaltime($starttime).')';
-		$daysel='selected';
+		$daysel=' selected="selected"';
 	    }
 	    $header=&mt('Content changed').' '.$seltext;
 	} else {
@@ -2881,11 +2955,11 @@ sub checkversions {
 <fieldset>
 <legend>$lt{'cd'}</legend>
 <select name="timerange">
-<option value='all' $allsel>$lt{'al'}</option>
-<option value="-1" $startsel>$lt{'st'}</option>
-<option value="2592000" $monthsel>$lt{'lm'}</option>
-<option value="604800" $weeksel>$lt{'lw'}</option>
-<option value="86400" $daysel>$lt{'sy'}</option>
+<option value='all'$allsel>$lt{'al'}</option>
+<option value="-1"$startsel>$lt{'st'}</option>
+<option value="2592000"$monthsel>$lt{'lm'}</option>
+<option value="604800"$weeksel>$lt{'lw'}</option>
+<option value="86400"$daysel>$lt{'sy'}</option>
 </select>
 <input type="submit" name="display" value="$lt{'sd'}" />
 </fieldset>
@@ -2901,111 +2975,93 @@ $lt{'sc'}: <input type="submit" name="se
 <hr />
 <h4>$lt{'vers'}</h4>
 <input type="submit" name="setversions" value="$lt{'save'}" />
-<table border="0">
 ENDHEADERS
     #number of columns for version history
-    my $num_ver_col = 1;
     $r->print(
-    &Apache::loncommon::start_data_table().
-    &Apache::loncommon::start_data_table_header_row().
-    '<th>'.&mt('Resources').'</th>'.
-    "<th>$lt{'mr'}</th>".
-    "<th>$lt{'ve'}</th>".
-    "<th>$lt{'vu'}</th>".
-    '<th colspan="'.$num_ver_col.'">'.&mt('History').'</th>'.
-    '</b>');
+        &Apache::loncommon::start_data_table().
+        &Apache::loncommon::start_data_table_header_row().
+        '<th>'.&mt('Resources').'</th>'.
+        "<th>$lt{'mr'}</th>".
+        "<th>$lt{'ve'}</th>".
+        "<th>$lt{'vu'}</th>".
+        '<th>'.&mt('History').'</th>'.
+        &Apache::loncommon::end_data_table_header_row()
+    );
     foreach my $key (sort(keys(%changes))) {
         #excludes not versionable problems from resource version history:
-        if ($changes{$key}>$starttime && $key !~ /^\/res\/lib\/templates/) {
-	    my ($root,$extension)=($key=~/^(.*)\.(\w+)$/);
-	    my $currentversion=&Apache::lonnet::getversion($key);
-	    if ($currentversion<0) {
-                $currentversion='<span class="LC_error">'.&mt('Could not be determined.').'</span>';
-	    }
-	    my $linkurl=&Apache::lonnet::clutter($key);
+        next unless ($changes{$key}>$starttime && $key !~ /^\/res\/lib\/templates/);
+        my ($root,$extension)=($key=~/^(.*)\.(\w+)$/);
+        my $currentversion=&Apache::lonnet::getversion($key);
+        if ($currentversion<0) {
+            $currentversion='<span class="LC_error">'.&mt('Could not be determined.').'</span>';
+        }
+        my $linkurl=&Apache::lonnet::clutter($key);
         $r->print(
-            &Apache::loncommon::end_data_table_header_row().
             &Apache::loncommon::start_data_table_row().
-            '<td><b>'.&Apache::lonnet::gettitle($linkurl).'</b><br>'.
+            '<td><b>'.&Apache::lonnet::gettitle($linkurl).'</b><br />'.
             '<a href="'.$linkurl.'" target="cat">'.$linkurl.'</a></td>'.
-            '<td align="right">'.$currentversion.'<span class="LC_fontsize_medium"><br>('.
+            '<td align="right">'.$currentversion.'<span class="LC_fontsize_medium"><br />('.
             &Apache::lonlocal::locallocaltime(&Apache::lonnet::metadata($root.'.'.$extension,'lastrevisiondate')).')</span></td>'.
-            '<td align="right">');
-# Used in course
-	    my $usedversion=$hash{'version_'.$linkurl};
-	    if (($usedversion) && ($usedversion ne 'mostrecent')) {
-                if($usedversion != $currentversion){
+            '<td align="right">'
+        );
+        # Used in course
+        my $usedversion=$hash{'version_'.$linkurl};
+        if (($usedversion) && ($usedversion ne 'mostrecent')) {
+                if ($usedversion != $currentversion) {
                     $r->print('<span class="LC_warning">'.$usedversion.'</span>');
-                }else{
+                } else {
                     $r->print($usedversion);
                 }
-	    } else {
-		$r->print($currentversion);
-	    }
-	    $r->print('</td><td title="'.$lt{'vu'}.'">');
-# Set version
-	    $r->print(&Apache::loncommon::select_form($setversions{$linkurl},
-						      'set_version_'.$linkurl,
-						      {'select_form_order' =>
-						       ['',1..$currentversion,'mostrecent'],
-						       '' => '',
-						       'mostrecent' => &mt('most recent'),
-						       map {$_,$_} (1..$currentversion)}));
-	    my $lastold=1;
-	    for (my $prevvers=1;$prevvers<$currentversion;$prevvers++) {
-		my $url=$root.'.'.$prevvers.'.'.$extension;
-		if (&Apache::lonnet::metadata($url,'lastrevisiondate')<
-		    $starttime) {
-		    $lastold=$prevvers;
-		}
-	    }
-            #
-            # Code to figure out how many version entries should go in
-            # each of the four columns
-            my $entries_per_col = 0;
-            my $num_entries = ($currentversion-$lastold);
-            if ($num_entries % $num_ver_col == 0) {
-                $entries_per_col = $num_entries/$num_ver_col;
             } else {
-                $entries_per_col = $num_entries/$num_ver_col + 1;
+                $r->print($currentversion);
             }
-            my $entries_count = 0;
-            $r->print('<td valign="top"><span class="LC_fontsize_medium">');
-            my $cols_output = 1;
-            for (my $prevvers=$lastold;$prevvers<$currentversion;$prevvers++) {
-		my $url=$root.'.'.$prevvers.'.'.$extension;
-		$r->print('<span class="LC_nobreak"><a href="'.&Apache::lonnet::clutter($url).
-			  '">'.&mt('Version').' '.$prevvers.'</a> ('.
-			  &Apache::lonlocal::locallocaltime(
-                                &Apache::lonnet::metadata($url,
-                                                          'lastrevisiondate')
-                                                            ).
-			  ')');
-		if (&Apache::loncommon::fileembstyle($extension) eq 'ssi') {
-                    $r->print(' <a href="/adm/diff?filename='.
-			      &Apache::lonnet::clutter($root.'.'.$extension).
-			      '&versionone='.$prevvers.
-			      '" target="diffs">'.&mt('Diffs').'</a>');
-		}
-		$r->print('</span><br />');
-                if (++$entries_count % $entries_per_col == 0) {
-                    $r->print('</span></td>');
-                    if ($cols_output != $num_ver_col) {
-                        $r->print('<td valign="top"><span class="LC_fontsize_medium">');
-                        $cols_output++;
-                    }
-                }
-	    }
-            while($cols_output++ < $num_ver_col) {
-                $r->print('</span></td><td>');
+        $r->print('</td><td title="'.$lt{'vu'}.'">');
+        # Set version
+        $r->print(&Apache::loncommon::select_form(
+            $setversions{$linkurl},
+            'set_version_'.$linkurl,
+            {'select_form_order' => ['',1..$currentversion,'mostrecent'],
+             '' => '',
+             'mostrecent' => &mt('most recent'),
+             map {$_,$_} (1..$currentversion)}));
+        my $lastold=1;
+        for (my $prevvers=1;$prevvers<$currentversion;$prevvers++) {
+            my $url=$root.'.'.$prevvers.'.'.$extension;
+            if (&Apache::lonnet::metadata($url,'lastrevisiondate')<$starttime) {
+                $lastold=$prevvers;
+            }
+        }
+        $r->print('</td>');
+        # List all available versions
+        $r->print('<td valign="top"><span class="LC_fontsize_medium">');
+        for (my $prevvers=$lastold;$prevvers<$currentversion;$prevvers++) {
+            my $url=$root.'.'.$prevvers.'.'.$extension;
+            $r->print(
+                '<span class="LC_nobreak">'
+               .'<a href="'.&Apache::lonnet::clutter($url).'">'
+               .&mt('Version [_1]',$prevvers).'</a>'
+               .' ('.&Apache::lonlocal::locallocaltime(
+                         &Apache::lonnet::metadata($url,'lastrevisiondate'))
+               .')');
+            if (&Apache::loncommon::fileembstyle($extension) eq 'ssi') {
+                $r->print(
+                    ' <a href="/adm/diff?filename='.
+                    &Apache::lonnet::clutter($root.'.'.$extension).
+                    &HTML::Entities::encode('&versionone='.$prevvers,'"<>&').
+                    '" target="diffs">'.&mt('Diffs').'</a>');
             }
-	}
+            $r->print('</span><br />');
+        }
+        $r->print('</span></td>'.&Apache::loncommon::end_data_table_row());
     }
-    $r->print('</td>'.&Apache::loncommon::end_data_table_row().
-            &Apache::loncommon::end_data_table().
-            '<input type="submit" name="setversions" value="'.$lt{'save'}.'" />');
+    $r->print(
+        &Apache::loncommon::end_data_table().
+        '<input type="submit" name="setversions" value="'.$lt{'save'}.'" />'.
+        '</form>'
+    );
 
     &untiehash();
+    $r->print(&endContentScreen());
 }
 
 sub mark_hash_old {
@@ -3037,11 +3093,6 @@ sub changewarning {
     my $pathvar='folderpath';
     my $path=&escape($env{'form.folderpath'});
     if (!defined($url)) {
-	if (defined($env{'form.pagepath'})) {
-	    $pathvar='pagepath';
-	    $path=&escape($env{'form.pagepath'});
-	    $path.='&amp;pagesymb='.&escape($env{'form.pagesymb'});
-	}
 	$url='/adm/coursedocs?'.$pathvar.'='.$path;
     }
     my $course_type = &Apache::loncommon::course_type();
@@ -3215,8 +3266,8 @@ sub handler {
 # Get the parameters that may be needed
 #
     &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
-                                            ['folderpath','pagepath',
-                                             'pagesymb','forcesupplement','forcestandard',
+                                            ['folderpath',
+                                             'forcesupplement','forcestandard',
                                              'tools','symb','command','supppath']);
 
 # standard=1: this is a "new-style" course with an uploaded map as top level
@@ -3231,7 +3282,7 @@ sub handler {
 
 
     my $supplementalflag=($env{'form.folderpath'}=~/^supplemental/);
-    if (($env{'form.folderpath'}=~/^default/) || $env{'form.folderpath'} eq "" || ($env{'form.pagepath'})) {
+    if (($env{'form.folderpath'}=~/^default/) || ($env{'form.folderpath'} eq "")) {
        $supplementalflag=0;
     }
     if ($env{'form.forcesupplement'}) { $supplementalflag=1; }
@@ -3250,112 +3301,54 @@ sub handler {
 
 # Do we directly jump somewhere?
 
-   if ($env{'form.command'} eq 'direct') {
-       my ($mapurl,$id,$resurl);
+   if (($env{'form.command'} eq 'direct') || ($env{'form.command'} eq 'directnav')) {
        if ($env{'form.symb'} ne '') {
-           ($mapurl,$id,$resurl) = &Apache::lonnet::decode_symb($env{'form.symb'});
-           if ($resurl=~/\.(sequence|page)$/) {
-               $mapurl=$resurl;
-           } elsif ($resurl eq 'adm/navmaps') {
-               $mapurl=$env{'course.'.$env{'request.course.id'}.'.url'};
-           }
-           my $mapresobj;
-           my $navmap = Apache::lonnavmaps::navmap->new();
-           if (ref($navmap)) {
-               $mapresobj = $navmap->getResourceByUrl($mapurl);
-           }
-           $mapurl=~s{^.*/([^/]+)\.(\w+)$}{$1};
-           my $type=$2;
-           my $path;
-           if (ref($mapresobj)) {
-               my $pcslist = $mapresobj->map_hierarchy();
-               if ($pcslist ne '') {
-                   foreach my $pc (split(/,/,$pcslist)) {
-                       next if ($pc <= 1);
-                       my $res = $navmap->getByMapPc($pc);
-                       if (ref($res)) {
-                           my $thisurl = $res->src();
-                           $thisurl=~s{^.*/([^/]+)\.\w+$}{$1}; 
-                           my $thistitle = $res->title();
-                           $path .= '&'.
-                                    &Apache::lonhtmlcommon::entity_encode($thisurl).'&'.
-                                    &Apache::lonhtmlcommon::entity_encode($thistitle).
-                                    ':'.$res->randompick().
-                                    ':'.$res->randomout().
-                                    ':'.$res->encrypted().
-                                    ':'.$res->randomorder();
-                       }
-                   }
-               }
-               $path =~ s/^\&//;
-               my $maptitle = $mapresobj->title();
-               if ($mapurl eq 'default') {
-                   $maptitle = 'Main Course Documents';
-               }
-               $path .= (($path ne '')? '&' : '').
-                    &Apache::lonhtmlcommon::entity_encode($mapurl).'&'.
-                    &Apache::lonhtmlcommon::entity_encode($maptitle).
-                    ':'.$mapresobj->randompick().
-                    ':'.$mapresobj->randomout().
-                    ':'.$mapresobj->encrypted().
-                    ':'.$mapresobj->randomorder();
-           } else {
-               my $maptitle = &Apache::lonnet::gettitle($mapurl);
-               if ($mapurl eq 'default') {
-                   $maptitle = 'Main Course Documents';
-               }
-               $path = &Apache::lonhtmlcommon::entity_encode($mapurl).'&'.
-                       &Apache::lonhtmlcommon::entity_encode($maptitle).'::::';
-           }
-           unless ($mapurl eq 'default') {
-               $path = 'default&'.
-                       &Apache::lonhtmlcommon::entity_encode('Main Course Documents').
-                       '::::&'.$path;
-           }
-           if ($type eq 'sequence') {
-               $env{'form.folderpath'}=$path;
-               $env{'form.pagepath'}='';
-           } else {
-               $env{'form.pagepath'}=$path;
-               $env{'form.folderpath'}='';
-           }
+           $env{'form.folderpath'}=
+               &Apache::loncommon::symb_to_docspath($env{'form.symb'});
+           &Apache::lonnet::appenv({'docs.exit.'.$env{'request.course.id'} =>
+               $env{'form.command'}.'_'.$env{'form.symb'}});
        } elsif ($env{'form.supppath'} ne '') {
            $env{'form.folderpath'}=$env{'form.supppath'};
+           &Apache::lonnet::appenv({'docs.exit.'.$env{'request.course.id'} =>
+               $env{'form.command'}.'_'.$env{'form.supppath'}});
        }
    } elsif ($env{'form.command'} eq 'editdocs') {
-        $env{'form.folderpath'} = 'default&'.
-                                  &Apache::lonhtmlcommon::entity_encode('Main Course Content');
-        $env{'form.pagepath'}='';
+       $env{'form.folderpath'} = 'default&'.
+                                 &Apache::lonhtmlcommon::entity_encode('Main Course Content').
+                                 ':::::';
+       &Apache::lonnet::appenv({'docs.exit.'.$env{'request.course.id'} => $env{'form.command'}});
    } elsif ($env{'form.command'} eq 'editsupp') {
-        $env{'form.folderpath'} = 'default&'.
+       $env{'form.folderpath'} = 'supplemental&'.
                                   &Apache::lonhtmlcommon::entity_encode('Supplemental Content');
-        $env{'form.pagepath'}='';
+       &Apache::lonnet::appenv({'docs.exit.'.$env{'request.course.id'} => '/adm/supplemental'});
+   } elsif ($env{'form.command'} eq 'contents') {
+       &Apache::lonnet::appenv({'docs.exit.'.$env{'request.course.id'} => '/adm/navmaps'});
+   } elsif ($env{'form.command'} eq 'home') {
+       &Apache::lonnet::appenv({'docs.exit.'.$env{'request.course.id'} => '/adm/menu'});
    }
 
+
 # Where do we store these for when we come back?
     my $stored_folderpath='docs_folderpath';
     if ($supplementalflag) {
        $stored_folderpath='docs_sup_folderpath';
     }
 
-# No folderpath, no pagepath, and in edit mode, see if we have something stored
-    if ((!$env{'form.folderpath'}) && (!$env{'form.pagepath'}) && $allowed) {
+# No folderpath, and in edit mode, see if we have something stored
+    if ((!$env{'form.folderpath'}) && $allowed) {
         &Apache::loncommon::restore_course_settings($stored_folderpath,
                                           {'folderpath' => 'scalar'});
+        unless (&unescape($env{'form.folderpath'}) =~ m{^(default|supplemental)&}) {
+            undef($env{'form.folderpath'});
+        }
     }
    
 # If we are not allowed to make changes, all we can see are supplemental docs
     if (!$allowed) {
-        $env{'form.pagepath'}='';
         unless ($env{'form.folderpath'} =~ /^supplemental/) {
             $env{'form.folderpath'} = &supplemental_base();
         }
     }
-# If we still not have a folderpath, see if we can resurrect at pagepath
-    if (!$env{'form.folderpath'} && $allowed) {
-        &Apache::loncommon::restore_course_settings($stored_folderpath,
-                                              {'pagepath' => 'scalar'});
-    }
 # Make the zeroth entry in supplemental docs page paths, so we can get to top level
     if ($env{'form.folderpath'} =~ /^supplemental_\d+/) {
         $env{'form.folderpath'} = &supplemental_base()
@@ -3363,11 +3356,12 @@ sub handler {
                                   $env{'form.folderpath'};
     }
 # If after all of this, we still don't have any paths, make them
-    unless (($env{'form.pagepath'}) || ($env{'form.folderpath'})) {
+    unless ($env{'form.folderpath'}) {
        if ($supplementalflag) {
           $env{'form.folderpath'}=&supplemental_base();
        } else {
-          $env{'form.folderpath'}='default';
+          $env{'form.folderpath'}='default'.&escape(&mt('Main '.$crstype.' Documents')).
+                                  ':::::';
        }
     }
 
@@ -3375,38 +3369,29 @@ sub handler {
     unless ($toolsflag) {
         if ($allowed) {
             &Apache::loncommon::store_course_settings($stored_folderpath,
-                                                      {'pagepath' => 'scalar',
-                                                       'folderpath' => 'scalar'});
+                                                      {'folderpath' => 'scalar'});
         }
+        my $folderpath;
         if ($env{'form.folderpath'}) {
-	    my (@folderpath)=split('&',$env{'form.folderpath'});
-	    $env{'form.foldername'}=&unescape(pop(@folderpath));
-	    $env{'form.folder'}=pop(@folderpath);
-            $container='sequence';
-        }
-        if ($env{'form.pagepath'}) {
-            my (@pagepath)=split('&',$env{'form.pagepath'});
-            $env{'form.pagename'}=&unescape(pop(@pagepath));
-            $env{'form.folder'}=pop(@pagepath);
-            $container='page';
-            $containertag = '<input type="hidden" name="pagepath" value="" />'.
-	                    '<input type="hidden" name="pagesymb" value="" />';
-            $pathitem = 
-                '<input type="hidden" name="pagepath" value="'.&HTML::Entities::encode($env{'form.pagepath'},'<>&"').'" />'.
-	        '<input type="hidden" name="pagesymb" value="'.&HTML::Entities::encode($env{'form.pagesymb'},'<>&"').'" />'.
-                '<input type="hidden" name="folderpath" value="" />';
+            $folderpath = $env{'form.folderpath'};
+	    my (@folders)=split('&',$env{'form.folderpath'});
+	    $env{'form.foldername'}=&unescape(pop(@folders));
+            if ($env{'form.foldername'} =~ /\:1$/) {
+                $container = 'page';
+            } else {
+                $container = 'sequence';
+            }
+	    $env{'form.folder'}=pop(@folders);
         } else {
-            my $folderpath=$env{'form.folderpath'};
-            if (!$folderpath) {
-                if ($env{'form.folder'} eq '' ||
-                    $env{'form.folder'} eq 'supplemental') {
-                    $folderpath='default&'.
-                        &escape(&mt('Main '.$crstype.' Documents'));
-                }
+            if ($env{'form.folder'} eq '' ||
+                $env{'form.folder'} eq 'supplemental') {
+                $folderpath='default&'.
+                            &escape(&mt('Main '.$crstype.' Documents')).
+                            ':::::';
             }
-            $containertag = '<input type="hidden" name="folderpath" value="" />';
-            $pathitem = '<input type="hidden" name="folderpath" value="'.&HTML::Entities::encode($folderpath,'<>&"').'" />';
         }
+        $containertag = '<input type="hidden" name="folderpath" value="" />';
+        $pathitem = '<input type="hidden" name="folderpath" value="'.&HTML::Entities::encode($folderpath,'<>&"').'" />';
         if ($r->uri=~/^\/adm\/coursedocs\/showdoc\/(.*)$/) {
            $showdoc='/'.$1;
         }
@@ -3444,7 +3429,7 @@ sub handler {
                 @tabids = ('002','ee2','ff2');
             } else {
                 @tabids = ('aa1','bb1','cc1','ff1');
-                unless ($env{'form.pagepath'}) {
+                unless ($env{'form.folderpath'} =~ /\:1$/) {
                     unshift(@tabids,'001');
                     push(@tabids,('dd1','ee1'));
                 }
@@ -3558,9 +3543,6 @@ sub handler {
   } elsif ((!$showdoc) && (!$uploadphase)) {
 # -----------------------------------------------------------------------------
        my %lt=&Apache::lonlocal::texthash(
-                'uplm' => 'Upload a new main '.lc($crstype).' document',
-                'upls' => 'Upload a new supplemental '.lc($crstype).' document',
-                'impp' => 'Import a document',
 		'copm' => 'All documents out of a published map into this folder',
                 'upfi' => 'Upload File',
                 'upld' => 'Import Content',
@@ -3570,10 +3552,8 @@ sub handler {
                 'impm' => 'Import from Assembled Map',
                 'selm' => 'Select Map',
                 'load' => 'Load Map',
-                'reco' => 'Recover Deleted Documents',
                 'newf' => 'New Folder',
                 'newp' => 'New Composite Page',
-                'extr' => 'External Resource',
                 'syll' => 'Syllabus',
                 'navc' => 'Table of Contents',
                 'sipa' => 'Simple Course Page',
@@ -3584,7 +3564,7 @@ sub handler {
                 'mypi' => 'My Personal Information Page',
                 'grpo' => 'Group Portfolio',
                 'rost' => 'Course Roster',
-				'abou' => 'Personal Information Page for a User',
+                'abou' => 'Personal Information Page for a User',
                 'imsf' => 'IMS Import',
                 'imsl' => 'Import IMS package',
                 'cms'  => 'Origin of IMS package',
@@ -3592,9 +3572,6 @@ sub handler {
                 'file' =>  'File',
                 'title' => 'Title',
                 'comment' => 'Comment',
-                'url'  => 'URL',
-                'prev' => 'Preview',
-                'lnk'  => 'Add Link', 
                 'parse' => 'Upload embedded images/multimedia files if HTML file',
 					  );
 # -----------------------------------------------------------------------------
@@ -3751,10 +3728,6 @@ HIDDENFORM
        }
        my $folderseq='/uploaded/'.$coursedom.'/'.$coursenum.'/default_new.sequence';
        my $pageseq = '/uploaded/'.$coursedom.'/'.$coursenum.'/default_new.page';
-	my $container='sequence';
-	if ($env{'form.pagepath'}) {
-	    $container='page';
-	}
 	my $readfile='/uploaded/'.$coursedom.'/'.$coursenum.'/'.$folder.'.'.$container;
 
 	my $newnavform=(<<NNFORM);
@@ -3912,10 +3885,16 @@ NGFFORM
 
 
         my @importdoc = (
-        {'<img class="LC_noBorder LC_middle" src="/res/adm/pages/extres.png" alt="'.$lt{extr}.'" onclick="toggleUpload(\'ext\');" />'=>$extresourcesform},
-        {'<img class="LC_noBorder LC_middle" src="/res/adm/pages/ims.png" alt="'.$lt{imsf}.'" onclick="javascript:toggleUpload(\'ims\');" />'=>$imspform},
-        {'<img class="LC_noBorder_LC_middle" src="/res/adm/pages/pdfupload.png" alt="'.$lt{upl}.'" onclick="javascript:toggleUpload(\'doc\');" />'=>$fileuploadform,
-        });
+        {'<img class="LC_noBorder LC_middle" src="/res/adm/pages/extres.png" alt="'.$lt{extr}.'" onclick="toggleUpload(\'ext\');" />'=>$extresourcesform}
+        );
+        unless ($container eq 'page') {
+            push(@importdoc,
+                {'<img class="LC_noBorder LC_middle" src="/res/adm/pages/ims.png" alt="'.$lt{imsf}.'" onclick="javascript:toggleUpload(\'ims\');" />'=>$imspform}
+            );
+        }
+        push(@importdoc,
+            {'<img class="LC_noBorder_LC_middle" src="/res/adm/pages/pdfupload.png" alt="'.$lt{upl}.'" onclick="javascript:toggleUpload(\'doc\');" />'=>$fileuploadform}
+        );
         $fileuploadform =  &create_form_ul(&create_list_elements(@importdoc));
 
         @gradingforma=(
@@ -3940,7 +3919,7 @@ my %orderhash = (
                 'bb' => ['Published Content',$importpubform],
                 'cc' => ['Grading Resources',$gradingform],
                 );
-unless ($env{'form.pagepath'}) {
+unless ($container eq 'page') {
     $orderhash{'00'} = ['Newfolder',$newfolderform];
     $orderhash{'dd'} = ['Collaboration',$communityform];
     $orderhash{'ee'} = ['Special Pages',$specialdocumentsform];
@@ -3973,7 +3952,6 @@ unless ($env{'form.pagepath'}) {
        } elsif ($allowed) {
 	  $env{'form.folderpath'} = $savefolderpath;
        }
-       $env{'form.pagepath'} = '';
        $pathitem = '<input type="hidden" name="folderpath" value="'.
                     &HTML::Entities::encode($env{'form.folderpath'},'<>&"').'" />';
        if ($allowed) {
@@ -4142,12 +4120,9 @@ sub decompression_info {
     my $docuname=$env{'course.'.$env{'request.course.id'}.'.num'};
     my $container='sequence';
     my ($pathitem,$hiddenelem);
-    my @hiddens = ('newidx','comment','position');
-    if ($env{'form.pagepath'}) {
+    my @hiddens = ('newidx','comment','position','folderpath');
+    if ($env{'form.folderpath'} =~ /\:1$/) {
         $container='page';
-        $pathitem = 'pagepath';
-    } else {
-        $pathitem = 'folderpath';
     }
     unshift(@hiddens,$pathitem);
     foreach my $item (@hiddens) {
@@ -4337,15 +4312,15 @@ sub generate_edit_table {
     my $form;
     my $activetab;
     my $active;
-    if($env{'form.active'} ne ''){
+    if (($env{'form.active'} ne '') && ($env{'form.active'} ne 'aa')) {
         $activetab = $env{'form.active'};
     }
     my $backicon = $iconpath.'clickhere.gif';
-    my $backtext = &mt('Exit');
+    my $backtext = &mt('Exit Editor');
     $form = '<div class="LC_Box" style="margin:0;">'.
             '<ul id="navigation'.$tid.'" class="LC_TabContent">'."\n".
             '<li class="goback">'.
-            '<a href="javascript:toContents('."'$jumpto'".');">'.
+            '<a href="javascript:toContents('.$jumpto.');">'.
             '<img src="'.$backicon.'" class="LC_icon" style="border: none; vertical-align: top;"'.
             '  alt="'.$backtext.'" />'.$backtext.'</a></li>'."\n".
             '<li>'.
@@ -4434,22 +4409,38 @@ sub editing_js {
 
     my $crstype = &Apache::loncommon::course_type();
     my $docs_folderpath = &HTML::Entities::encode($env{'environment.internal.'.$env{'request.course.id'}.'.docs_folderpath.folderpath'},'<>&"');
-    my $docs_pagepath = &HTML::Entities::encode($env{'environment.internal.'.$env{'request.course.id'}.'.docs_folderpath.pagepath'},'<>&"');
     my $main_container_page;
-    if ($docs_folderpath eq '') {
-        if ($docs_pagepath ne '') {
-            $main_container_page = 1;
-        }
+    if (&HTML::Entities::decode($env{'environment.internal.'.$env{'request.course.id'}.'.docs_folderpath.folderpath'}) =~ /\:1$/) {
+        $main_container_page = 1;
     }
-    my $toplevelmain = 'default&Main%20'.$crstype.'%20Documents';
+    my $toplevelmain = 'default&Main%20'.$crstype.'%20Documents%3A%3A%3A%3A%3A';
     my $toplevelsupp = &supplemental_base();
 
-    my $backtourl = '/adm/navmaps';
-    if ($supplementalflag) {
+    my $backtourl;
+    if ($env{'docs.exit.'.$env{'request.course.id'}} =~ /^direct_(.+)$/) {
+        my $caller = $1;
+        if ($caller =~ /^supplemental/) {
+            $backtourl = '/adm/supplemental?folderpath='.&escape($caller);
+        } else {
+            my ($map,$id,$res)=&Apache::lonnet::decode_symb($caller);
+            $res = &Apache::lonnet::clutter($res);
+            if (&Apache::lonnet::is_on_map($res)) {
+                $backtourl = &HTML::Entities::encode(&Apache::lonnet::clutter($res),'<>&"').'?symb='.
+                             &HTML::Entities::encode($caller,'<>&"');
+            }
+        }
+    } elsif ($env{'docs.exit.'.$env{'request.course.id'}} eq '/adm/menu') {
+        $backtourl = '/adm/menu';
+    } elsif ($supplementalflag) {
         $backtourl = '/adm/supplemental';
+    } else {
+        $backtourl = '/adm/navmaps';
     }
 
-    my $fieldsets = "'ext','doc','ims'";
+    my $fieldsets = "'ext','doc'";
+    unless ($main_container_page) {
+        $fieldsets .=",'ims'";
+    }
     if ($supplementalflag) {
         $fieldsets = "'suppext','suppdoc'";
     }
@@ -4579,64 +4570,40 @@ function makeims(imsform) {
     imsform.submit();
 }
 
-function changename(folderpath,index,oldtitle,container,pagesymb) {
+function changename(folderpath,index,oldtitle) {
 var title=prompt('$lt{"p_chn"}',oldtitle);
 if (title) {
 this.document.forms.renameform.markcopy.value=-1;
 this.document.forms.renameform.title.value=title;
 this.document.forms.renameform.cmd.value='rename_'+index;
-if (container == 'sequence') {
-    this.document.forms.renameform.folderpath.value=folderpath;
-}
-if (container == 'page') {
-    this.document.forms.renameform.pagepath.value=folderpath;
-    this.document.forms.renameform.pagesymb.value=pagesymb;
-}
+this.document.forms.renameform.folderpath.value=folderpath;
 this.document.forms.renameform.submit();
 }
 }
 
-function removeres(folderpath,index,oldtitle,container,pagesymb,skip_confirm) {
+function removeres(folderpath,index,oldtitle,skip_confirm) {
 if (skip_confirm || confirm('$lt{"p_rmr1"}\\n\\n$lt{"p_rmr2a"} "'+oldtitle+'" $lt{"p_rmr2b"}')) {
 this.document.forms.renameform.markcopy.value=-1;
 this.document.forms.renameform.cmd.value='del_'+index;
-if (container == 'sequence') {
-    this.document.forms.renameform.folderpath.value=folderpath;
-}
-if (container == 'page') {
-    this.document.forms.renameform.pagepath.value=folderpath;
-    this.document.forms.renameform.pagesymb.value=pagesymb;
-}
+this.document.forms.renameform.folderpath.value=folderpath;
 this.document.forms.renameform.submit();
 }
 }
 
-function cutres(folderpath,index,oldtitle,container,pagesymb,folder,skip_confirm) {
+function cutres(folderpath,index,oldtitle,container,folder,skip_confirm) {
 if (skip_confirm || confirm('$lt{"p_ctr1a"}\\n$lt{"p_ctr1b"}\\n\\n$lt{"p_ctr2a"} "'+oldtitle+'" $lt{"p_ctr2b"}')) {
 this.document.forms.renameform.cmd.value='cut_'+index;
 this.document.forms.renameform.markcopy.value=index;
 this.document.forms.renameform.copyfolder.value=folder+'.'+container;
-if (container == 'sequence') {
-    this.document.forms.renameform.folderpath.value=folderpath;
-}
-if (container == 'page') {
-    this.document.forms.renameform.pagepath.value=folderpath;
-    this.document.forms.renameform.pagesymb.value=pagesymb;
-}
+this.document.forms.renameform.folderpath.value=folderpath;
 this.document.forms.renameform.submit();
 }
 }
 
-function markcopy(folderpath,index,oldtitle,container,pagesymb,folder) {
+function markcopy(folderpath,index,oldtitle,container,folder) {
 this.document.forms.renameform.markcopy.value=index;
 this.document.forms.renameform.copyfolder.value=folder+'.'+container;
-if (container == 'sequence') {
 this.document.forms.renameform.folderpath.value=folderpath;
-}
-if (container == 'page') {
-this.document.forms.renameform.pagepath.value=folderpath;
-this.document.forms.renameform.pagesymb.value=pagesymb;
-}
 this.document.forms.renameform.submit();
 }
 
@@ -4734,9 +4701,6 @@ function showPage(current, pageId, nav,
         toggleMap();
         if (nav == 'mainnav') {
             var storedpath = "$docs_folderpath";
-            if (storedpath == '') {
-                storedpath = "$docs_pagepath";
-            }
             var storedpage = "$main_container_page";
             var reg = new RegExp("^supplemental");
             if (pageId == 'mainCourseDocuments') {
@@ -4772,9 +4736,8 @@ function showPage(current, pageId, nav,
 
 function toContents(jumpto) {
     var newurl = '$backtourl';
-    if (jumpto != '') {
+    if ((newurl == '/adm/navmaps') && (jumpto != '')) {
         newurl = newurl+'?postdata='+jumpto;
-;
     }
     location.href=newurl;
 }
@@ -4966,8 +4929,6 @@ Return hash with valid author names
 
 =item contained_map_check()
 
-=item reinit_role()
-
 =item url_paste_fixups()
 
 =item apply_fixups()