--- rat/lonpageflip.pm	2014/06/26 18:28:25	1.80.8.6
+++ rat/lonpageflip.pm	2014/12/11 13:16:49	1.86
@@ -2,7 +2,7 @@
 #
 # Page flip handler
 #
-# $Id: lonpageflip.pm,v 1.80.8.6 2014/06/26 18:28:25 raeburn Exp $
+# $Id: lonpageflip.pm,v 1.86 2014/12/11 13:16:49 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -283,7 +283,7 @@ sub handler {
 	     
 	  return REDIRECT;
       }
-      if ($direction eq 'return') { 
+      if ($direction eq 'return') {
 # -------------------------------------------------------- Return to last known
          my $last;
          if (tie(%hash,'GDBM_File',$env{'request.course.fn'}.'_symb.db',
@@ -306,12 +306,12 @@ sub handler {
             untie %hash;
          } else {
 	    $newloc='/adm/navmaps';
-         }
-	 &Apache::loncommon::content_type($r,'text/html');
-	 $r->header_out(Location => 
-			&Apache::lonnet::absolute_url().$newloc);
+         }  
+	     &Apache::loncommon::content_type($r,'text/html');
+	     $r->header_out(Location => 
+			    &Apache::lonnet::absolute_url().$newloc);
 	     
-	 return REDIRECT;
+	     return REDIRECT;
       }
       $currenturl=~s/^https?\:\/\///;
       $currenturl=~s/^[^\/]+//;
@@ -328,11 +328,11 @@ sub handler {
          if ($last) {
 	     $currenturl=&Apache::lonnet::clutter((&Apache::lonnet::decode_symb($last))[2]);
 	 } else {
-	     &Apache::loncommon::content_type($r,'text/html');
-	     $r->header_out(Location => 
-			    &Apache::lonnet::absolute_url().
-			    '/adm/noidea.html');
-	     return REDIRECT;
+		 &Apache::loncommon::content_type($r,'text/html');
+		 $r->header_out(Location => 
+				&Apache::lonnet::absolute_url().
+				'/adm/noidea.html');
+		 return REDIRECT;
          }
       }
 # ------------------------------------------- Do we have any idea where we are?
@@ -390,7 +390,7 @@ sub handler {
 	      untie(%hash);
 # --------------------------------------------------------- Store position info
               $cachehash{$startoutmap}{'last_direction'}=[$direction,'notasymb'];
-              foreach my $thismap (keys %cachehash) {
+              foreach my $thismap (keys(%cachehash)) {
 		  my $mapnum=$cachehash{$thismap}->{'mapnum'};
 		  delete($cachehash{$thismap}->{'mapnum'});
 		  &Apache::lonnet::symblist($thismap,
@@ -407,11 +407,11 @@ sub handler {
 		  }
 # ------------------------------------- Check for and display critical messages
                   my ($redirect, $url) = &Apache::loncommon::critical_redirect(300);
-                  unless ($redirect) {
+                  unless ($redirect) { 
                       $url=&Apache::lonnet::absolute_url().$redirecturl;
-                      $url = &add_get_param($url, { 'symb' => $redirectsymb});
+		      $url = &add_get_param($url, { 'symb' => $redirectsymb});
                   }
-                  &Apache::loncommon::content_type($r,'text/html');
+		  &Apache::loncommon::content_type($r,'text/html');
                   $r->header_out(Location => $url);
                   return REDIRECT;
 	      } else {
@@ -461,7 +461,7 @@ ENDSTART
                           &Apache::loncommon::start_page('No Resource')
                          .'<h2>'.$lt{'title'}.'</h2>'
                          .'<p>'.$lt{'explain'}.'</p>');
-		  }
+                  }
                   $r->print(
                       &Apache::lonhtmlcommon::actionbox(
                           ['<a href="/adm/flip?postdata=return:">'.$lt{'back'}.'</a></li>',