--- loncom/interface/lonsearchcat.pm	2016/08/04 21:23:57	1.331.4.12
+++ loncom/interface/lonsearchcat.pm	2014/02/22 18:12:09	1.342
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Search Catalog
 #
-# $Id: lonsearchcat.pm,v 1.331.4.12 2016/08/04 21:23:57 raeburn Exp $
+# $Id: lonsearchcat.pm,v 1.342 2014/02/22 18:12:09 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -78,7 +78,6 @@ use Apache::lonnavmaps;
 use Apache::lonindexer();
 use Apache::lonwishlist();
 use LONCAPA;
-use Time::HiRes qw(sleep);
 
 ######################################################################
 ######################################################################
@@ -144,7 +143,7 @@ sub handler {
     ##     this once, so the pause indicator is deleted
     ##
     if (exists($env{'form.pause'})) {
-        sleep(0.1);
+        sleep(1);
         delete($env{'form.pause'});
     }
     ##
@@ -194,7 +193,7 @@ sub handler {
                 &Apache::lonnet::logthis('lonsearchcat:'.
                                          'Unable to recover data from '.
                                          $persistent_db_file);
-                my $msg =
+		my $msg =
                     &mt('We were unable to retrieve data describing your search.').
                     ' '.&mt('This is a serious error and has been logged.').
                     '<br />'.
@@ -433,7 +432,7 @@ sub hidden_field {
 =pod
 
 =over 4
-
+ 
 =item &print_basic_search_form() 
 
 Prints the form for the basic search.  Sorry the name is so cryptic.
@@ -1912,11 +1911,10 @@ sub copyright_check {
     my (undef,undef,$resdom,$resname) = split('/',
                                               $Metadata->{'url'});
     # Check for priv
-    if ($Metadata->{'copyright'} eq 'priv') {
-        unless (($env{'user.name'} eq $resname) &&
-                ($env{'user.domain'} eq $resdom)) {
-            return 0;
-        }
+    if (($Metadata->{'copyright'} eq 'priv') && 
+        (($env{'user.name'} ne $resname) &&
+         ($env{'user.domain'} ne $resdom))) {
+        return 0;
     }
     # Check for domain
     if (($Metadata->{'copyright'} eq 'domain') &&
@@ -2066,7 +2064,7 @@ sub print_sort_form {
 </script>
 END
 
-    my $start_page = &Apache::loncommon::start_page('Results');
+    my $start_page = &Apache::loncommon::start_page('Results',$js);
     my $breadcrumbs=
         &Apache::lonhtmlcommon::breadcrumbs('Searching','Searching',
 					    $env{'form.catalogmode'} ne 'import');
@@ -2230,21 +2228,8 @@ SCRIPT
     $r->rflush();
 }
 
-sub reload_result_frame {
-    my ($r) = @_;
-    my $newloc = '/adm/searchcat?phase=results&persistent_db_id='.
-        $env{'form.persistent_db_id'};
-    $r->print(<<SCRIPT);
-<script type="text/javascript">
-    parent.update_results("$newloc");
-</script>
-SCRIPT
-
-    $r->rflush();
-}
-
 {
-    my $max_time  = 60;  # seconds for the search to complete
+    my $max_time  = 300;  # seconds for the search to complete
     my $start_time = 0;
     my $last_time = 0;
 
@@ -2425,7 +2410,6 @@ END
     ##
     ## Prepare for the big loop.
     my $hitcountsum;
-    my $oldhitcountsum;
     my %matches;
     my $server; 
     my $status;
@@ -2472,7 +2456,7 @@ END
                 &update_status($r,
                        &mt('waiting on [_1]',join(' ',keys(%Server_status))));
             }
-            sleep(0.1); 
+            sleep(1); 
         }
         #
         # Loop through the servers we have contacted but do not
@@ -2548,13 +2532,7 @@ END
                 delete($Server_status{$server});
             }
             last if ($connection->aborted());
-            if ($oldhitcountsum < $hitcountsum) {
-                &update_count_status($r,$hitcountsum);
-                if ($hitcountsum <= $env{'form.show'}) {
-                    reload_result_frame($r);
-                }
-                $oldhitcountsum = $hitcountsum;
-            }
+            &update_count_status($r,$hitcountsum);
         }
         last if ($connection->aborted());
         &update_seconds($r);
@@ -2648,8 +2626,6 @@ sub display_results {
     if ($env{'form.catalogmode'} eq 'import') {
         if (! tie(%groupsearch_db,'GDBM_File',$diropendb,
                   &GDBM_WRCREAT(),0640)) {
-            # NOTE: this can happen when a previous request to searchcat?phase=results gets interrupted
-            # (%groupsearch_db is not untied)
             $r->print('<p class="LC_error">'.
               &mt('Unable to save import results.').
               '</p>'.
@@ -2657,11 +2633,7 @@ sub display_results {
 		      &Apache::loncommon::end_page());
             $r->rflush();
             return;
-        }
-        # untie %groupsearch_db if the connection gets aborted before the end
-        $r->register_cleanup(sub {
-            untie %groupsearch_db if (tied(%groupsearch_db));
-        }); 
+        } 
     }
     ##
     ## Prepare the table for querying
@@ -2803,7 +2775,8 @@ sub display_results {
     );
 
     if ($total_results == 0) {
-        $r->print('<p class="LC_info">'.&mt('There are currently no results.').'</p>'.
+        $r->print('<meta HTTP-EQUIV="Refresh" CONTENT="2" />'.
+                  '<p class="LC_info">'.&mt('There are currently no results.').'</p>'.
                   "</form>".
 		  &Apache::loncommon::end_page());
         return;
@@ -3176,14 +3149,6 @@ SCRIPT
 </script>
 SCRIPT
 
-    $js.=<<SCRIPT;
-<script type="text/javascript">
-    \$(document).ready(function() {
-        parent.done_loading_results();
-    });
-</script>
-SCRIPT
-
     my $start_page  = &Apache::loncommon::start_page(undef,$js,
 						     {'only_body' =>1,
                                                       'add_wishlist' =>1,
@@ -3217,26 +3182,6 @@ sub print_frames_interface {
 // <![CDATA[
 var targetwin = opener;
 var queue = '';
-
-var loading_results = true;
-var need_reloading = false;
-var new_location;
-function update_results(newloc) {
-    if (loading_results) {
-        need_reloading = true;
-        new_location = newloc;
-    } else {
-        loading_results = true;
-        resultsframe.location = newloc;
-    }
-}
-function done_loading_results() {
-    loading_results = false;
-    if (need_reloading) {
-        need_reloading = false;
-        update_results(new_location);
-    }
-}
 // ]]>
 </script>
 JS
@@ -3349,7 +3294,7 @@ sub detailed_citation_view {
         '<b>'.$prefix.
         '<img src="'.&Apache::loncommon::icon($values{'url'}).'" alt="" />'.'&nbsp;'.
         '<a href="'.$jumpurl.'?inhibitmenu=yes" '.
-        'target="preview" onclick="openMyModal(this.href, 500, 500, \'yes\');return false;">'.$values{'title'}."</a></b>\n".
+        'target="preview">'.$values{'title'}."</a></b>\n".
         &display_tools($values{'title'}, $jumpurl).
         "<p>\n".
         '<b>'.$values{'author'}.'</b>,'.
@@ -3482,7 +3427,7 @@ sub summary_view {
     my $link = '<br />'.&display_url($jumpurl,1).'<br />';
     $result .=
         '<a href="'.$jumpurl.'?inhibitmenu=yes"'.
-        ' target="preview" onclick="openMyModal(this.href, 500, 500, \'yes\');return false;">'.$values{'title'}.'</a>'.
+        ' target="preview">'.$values{'title'}.'</a>'.
         &display_tools($values{'title'}, $jumpurl).<<END;
 <br />
 $link<br />
@@ -3528,7 +3473,7 @@ sub compact_view {
     }
     $jumpurl = &HTML::Entities::encode($jumpurl,'<>&"');
     $result.=' <span class="LC_nobreak">'.
-	'<a href="'.$jumpurl.'?inhibitmenu=yes" target="preview" onclick="openMyModal(this.href, 500, 500, \'yes\');return false;">'.
+	'<a href="'.$jumpurl.'?inhibitmenu=yes" target="preview">'.
         &HTML::Entities::encode($values{'title'},'<>&"').'</a></span> '.
         &display_tools($values{'title'}, $jumpurl).
 	$link.' <b>'.$values{'author'}.'</b> ('.$values{'domain'}.')';
@@ -3544,22 +3489,12 @@ sub display_url {
     } elsif ($url=~m{^(http://|/uploaded/)}) {
 	$link='<span class="LC_filename">'.$url.'</span>';
     } else {
-        # replace the links to open in a new window
-        # (because the search opens in a new window, it gets
-        # confusing when the links open a tab in the
-        # parent window; ideally we should not force windows)
-        my $onclick = " onclick=\"window.open(this.href, '_blank', 'toolbar=1,location=1,menubar=0');return false;\"";
         $link=&Apache::lonhtmlcommon::crumbs(
                   $url,
                   'preview',
                   '',
                   (($env{'form.catalogmode'} eq 'import')?'parent.statusframe.document.forms.statusform':''),
-                  $skiplast,$onclick).' ';
-        # replace the links to open in a new window
-        # (because the search opens in a new window, it gets
-        # confusing when the links open a tab in the
-        # parent window; ideally we should not force windows)
-        $link =~ s/(href="[^"]*")/\1 onclick="window.open(this.href, '_blank', 'toolbar=1,location=1,menubar=0');return false;"/g;
+                  $skiplast).' ';
     }
     return $link;
 }
@@ -3865,7 +3800,7 @@ Cleans the global %groupsearch_db by rem
 ######################################################################
 sub start_fresh_session {
     delete $groupsearch_db{'mode_catalog'};
-    foreach (keys(%groupsearch_db)) {
+    foreach (keys %groupsearch_db) {
         if ($_ =~ /^pre_/) {
             delete $groupsearch_db{$_};
         }