--- loncom/interface/lonsearchcat.pm	2022/05/29 21:59:55	1.331.4.16.2.1
+++ loncom/interface/lonsearchcat.pm	2013/08/13 13:02:57	1.335
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Search Catalog
 #
-# $Id: lonsearchcat.pm,v 1.331.4.16.2.1 2022/05/29 21:59:55 raeburn Exp $
+# $Id: lonsearchcat.pm,v 1.335 2013/08/13 13:02:57 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -52,6 +52,8 @@ search (on a server basis) is displayed
 
 =head1 Internals
 
+=over 4
+
 =cut
 
 ###############################################################################
@@ -78,7 +80,6 @@ use Apache::lonnavmaps;
 use Apache::lonindexer();
 use Apache::lonwishlist();
 use LONCAPA;
-use Time::HiRes qw(sleep);
 
 ######################################################################
 ######################################################################
@@ -144,7 +145,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'});
     }
     ##
@@ -171,22 +172,13 @@ sub handler {
     &Apache::lonhtmlcommon::clear_breadcrumbs();
 
     my @allowed_searches = ('portfolio');
-    if (&Apache::lonnet::allowed('bre',$env{'request.role.domain'}) eq 'F') {
+    if (&Apache::lonnet::allowed('bre',$env{'request.role.domain'})) {
         push(@allowed_searches,'res');
     } 
     my $crumb_text = 'Portfolio Search';
     if (@allowed_searches ==2) {
        $crumb_text = 'Portfolio and Catalog Search'; 
     }
-    my $target = '_top';
-    if ((($env{'request.lti.login'}) && ($env{'request.lti.target'} eq 'iframe')) ||
-        (($env{'request.deeplink.login'}) && ($env{'request.deeplink.target'} eq '_self'))) {
-        if ($env{'form.phase'} =~ /^(sort|run_search)$/) {
-            $target = '_parent';
-        } else {
-            $target = '_self';
-        }
-    }
     &Apache::lonhtmlcommon::add_breadcrumb
        ({href=>'/adm/searchcat?'.
 	       &Apache::loncommon::inhibit_menu_check().
@@ -194,7 +186,7 @@ sub handler {
                '&launch='.$env{'form.launch'}.
                '&mode='.$env{'form.mode'},
               text=>"$crumb_text",
-              target=>$target,
+              target=>'_top',
               bug=>'Searching',});
     #
     if ($env{'form.phase'} !~ m/(basic|adv|course)_search/) {
@@ -203,15 +195,12 @@ sub handler {
                 &Apache::lonnet::logthis('lonsearchcat:'.
                                          'Unable to recover data from '.
                                          $persistent_db_file);
-                my $msg =
-                    &mt('We were unable to retrieve data describing your search.').
-                    ' '.&mt('This is a serious error and has been logged.').
-                    '<br />'.
-                    &mt('Please alert your LON-CAPA administrator.');
-                &Apache::loncommon::simple_error_page(
-                    $r,'Search Error',
-                    $msg,
-                    {'no_auto_mt_msg' => 1});
+		my $msg =
+		    'We were unable to retrieve data describing your search. '.
+		    'This is a serious error and has been logged. '.
+		    'Please alert your LON-CAPA administrator.';
+		&Apache::loncommon::simple_error_page($r,'Search Error',
+						      $msg);
 		return OK;
             }
         }
@@ -332,31 +321,31 @@ END
         &display_results($r,$importbutton,$closebutton,$diropendb,
                          $env{'form.area'});
     } elsif ($env{'form.phase'} =~ /^(sort|run_search)$/) {
-        my ($query,$customquery,$customshow,$libraries,$pretty_string,$domainsref) =
+        my ($query,$customquery,$customshow,$libraries,$pretty_string) =
             &get_persistent_data($persistent_db_file,
                  ['query','customquery','customshow',
-                  'libraries','pretty_string','domains']);
+                  'libraries','pretty_string']);
         if ($env{'form.phase'} eq 'sort') {
-            &print_sort_form($r,$pretty_string,$target);
+            &print_sort_form($r,$pretty_string);
         } elsif ($env{'form.phase'} eq 'run_search') {
             &run_search($r,$query,$customquery,$customshow,
-                        $libraries,$pretty_string,$env{'form.area'},$domainsref,$target);
+                        $libraries,$pretty_string,$env{'form.area'});
         }
     } elsif(($env{'form.phase'} eq 'basic_search') ||
             ($env{'form.phase'} eq 'adv_search')) {
         #
         # We are running a search, try to parse it
-        my ($query,$customquery,$customshow,$libraries,$domains) = 
-            (undef,undef,undef,undef,undef);
+        my ($query,$customquery,$customshow,$libraries) = 
+            (undef,undef,undef,undef);
         my $pretty_string;
         if ($env{'form.phase'} eq 'basic_search') {
-            ($query,$pretty_string,$libraries,$domains) = 
+            ($query,$pretty_string,$libraries) = 
                 &parse_basic_search($r,$closebutton,$hidden_fields);
             return OK if (! defined($query));
             &make_persistent({ basicexp => $env{'form.basicexp'}},
                              $persistent_db_file);
         } else {                      # Advanced search
-            ($query,$customquery,$customshow,$libraries,$pretty_string,$domains) 
+            ($query,$customquery,$customshow,$libraries,$pretty_string) 
                 = &parse_advanced_search($r,$closebutton,$hidden_fields);
             return OK if (! defined($query));
         }
@@ -364,8 +353,7 @@ END
                            customquery => $customquery,
                            customshow => $customshow,
                            libraries => $libraries,
-                           pretty_string => $pretty_string,
-                           domains => $domains },
+                           pretty_string => $pretty_string },
                          $persistent_db_file);
         #
         # Set up table
@@ -439,9 +427,7 @@ sub hidden_field {
 
 ######################################################################
 
-=pod
-
-=over 4
+=pod 
 
 =item &print_basic_search_form() 
 
@@ -459,7 +445,7 @@ sub print_basic_search_form {
 					    $env{'form.catalogmode'} ne 'import');
     my $scrout = &Apache::loncommon::start_page('Content Library').$bread_crumb;
 # Search form for resource space 
-    if (&Apache::lonnet::allowed('bre',$env{'request.role.domain'}) eq 'F') {
+    if (&Apache::lonnet::allowed('bre',$env{'request.role.domain'})) {
         $scrout .= &setup_basic_search($r,'res',$hidden_fields,$closebutton);
         $scrout .= '<hr /><br />';
     }
@@ -488,31 +474,14 @@ sub setup_basic_search {
                      .&mt('use related words')
                      .'</label>';
 
-    my $anydom = 1;
-    if ($area eq 'res') {
-        unless (&Apache::lonnet::allowed('bre','/res/') eq 'F') {
-            $anydom = 0;
-        }
-    }
-    my $singledom;
-    my ($disabled,$checked);
-    if ($anydom) {
-       $singledom = $r->dir_config('lonDefDomain');
-       if ($env{'form.domains'} eq $singledom) {
-           $checked = 1;
-       }
-    } else {
-       $singledom = $env{'user.domain'};
-       $disabled = ' disabled="disabled"';
-       $checked = 1;
-    }
     $onlysearchdomain = '<label>'
                        .&Apache::lonhtmlcommon::checkbox(
-                           'domains',$checked,
-                           $singledom,$disabled)
+                           'domains',
+                           $env{'form.domains'},
+                           $r->dir_config('lonDefDomain'))
                        .' '
                        .&mt('only search domain [_1]'
-                               ,'<b>'.$singledom.'</b>')
+                               ,'<b>'.$r->dir_config('lonDefDomain').'</b>')
                        .'</label>';
 
     $adv_search_link = '<a href="/adm/searchcat?'.
@@ -687,22 +656,10 @@ function additional_metadata() {
                    $env{'form.category'})
               .&Apache::lonhtmlcommon::row_closure();
 
-    my $anydomain = 1;
-    if ($env{'form.area'} ne 'portfolio') {
-        unless (&Apache::lonnet::allowed('bre','/res/')) {
-            $anydomain = 0;
-        }
-    }
-
-    $scrout .= &Apache::lonhtmlcommon::row_title(&titlefield(&mt('Domains')));
-    if ($anydomain) {
-        $scrout .= &Apache::loncommon::domain_select('domains',
-                   $env{'form.domains'},1);
-    } else {
-        $scrout .= &Apache::loncommon::select_dom_form($env{'user.domain'},
-                                                       'domains','','','',[$env{'user.domain'}],'',1);
-    }
-    $scrout .= &Apache::lonhtmlcommon::row_closure();
+    $scrout .= &Apache::lonhtmlcommon::row_title(&titlefield(&mt('Domains')))
+              .&Apache::loncommon::domain_select('domains',
+                   $env{'form.domains'},1)
+              .&Apache::lonhtmlcommon::row_closure();
 
     # Misc metadata
     if ($env{'form.area'} ne 'portfolio') {
@@ -1055,9 +1012,8 @@ Outputs: array of values.  Returns undef
 
 This function is the reverse of &make_persistent();
 Retrieve persistent data from %persistent_db.  Retrieved items will have their
-values unescaped.  If the item is 'domains; then the returned
-value will be a hash pointer.  Otherwise, if the item contains
-commas (before unescaping), the returned value will be an array pointer.
+values unescaped.  If the item contains commas (before unescaping), the
+returned value will be an array pointer. 
 
 =cut
 
@@ -1075,25 +1031,13 @@ sub get_persistent_data {
             push @Values, undef;
             next;
         }
-        if ($name eq 'domains') {
-            my %valueshash;
-            my @items= map { &unescape($_); } split(',',$persistent_db{$name});
-            foreach my $item (@items) {
-                if ($item =~ /=/) {
-                    my ($key,$val) = map { &unescape($_); } split(/=/,$item);
-                    $valueshash{$key} = $val;
-                }
-            }
-            push(@Values,\%valueshash);
+        my @values = map { 
+            &unescape($_);
+        } split(',',$persistent_db{$name});
+        if (@values <= 1) {
+            push @Values,$values[0];
         } else {
-            my @values = map { 
-                &unescape($_);
-            } split(',',$persistent_db{$name});
-            if (@values <= 1) {
-                push @Values,$values[0];
-            } else {
-                push @Values,\@values;
-            }
+            push @Values,\@values;
         }
     }
     untie (%persistent_db);
@@ -1111,9 +1055,7 @@ Inputs: Hash of values to save, filename
 
 Store variables away to the %persistent_db.
 Values will be escaped.  Values that are array pointers will have their
-elements escaped and concatenated in a comma separated string. Values 
-that are hash pointers will have their keys and values escaped and 
-concatenated in a comma separated string.
+elements escaped and concatenated in a comma separated string.  
 
 =cut
 
@@ -1125,17 +1067,8 @@ sub make_persistent {
     return undef if (! tie(%persistent_db,'GDBM_File',
                            $filename,&GDBM_WRCREAT(),0640));
     foreach my $name (keys(%save)) {
-        my @values=();
-        if (ref($save{$name}) eq 'ARRAY') {
-            @values = @{$save{$name}};
-        } elsif (ref($save{$name}) eq 'HASH') {
-            foreach my $key (%{$save{$name}}) {
-                push(@values,&escape($key).'='.&escape($save{$name}{$key}));
-            }
-        } else {
-            @values = $save{$name};
-        }
-        # We handle array and hash references, but not recursively.
+        my @values = (ref($save{$name}) ? @{$save{$name}} : ($save{$name}));
+        # We handle array references, but not recursively.
         my $store = join(',', map { &escape($_); } @values );
         $persistent_db{$name} = $store;
     }
@@ -1452,8 +1385,7 @@ sub parse_advanced_search {
     ##
     ## Deal with restrictions to given domains
     ## 
-    my ($libraries_to_query,$pretty_domains_string,$domains_to_query) = 
-        &parse_domain_restrictions();
+    my ($libraries_to_query,$pretty_domains_string) = &parse_domain_restrictions();
     if ($pretty_domains_string) {
        $pretty_search_string .= $pretty_domains_string."<br />\n";
     }
@@ -1469,12 +1401,11 @@ sub parse_advanced_search {
     }
     #&Apache::lonnet::logthis('advanced query = '.$/.$query);
     return ($query,$customquery,$customshow,$libraries_to_query,
-            $pretty_search_string,$domains_to_query);
+            $pretty_search_string);
 }
 
 sub parse_domain_restrictions {
     my $libraries_to_query = undef;
-    my $domains_to_query = undef;
     # $env{'form.domains'} can be either a scalar or an array reference.
     # We need an array.
     if (! exists($env{'form.domains'}) || $env{'form.domains'} eq '') {
@@ -1498,32 +1429,12 @@ sub parse_domain_restrictions {
         foreach (sort @allowed_domains) {
             $pretty_domains_string .= "<b>".$_."</b> ";
         }
-	my %library_servers = &Apache::lonnet::get_unique_servers(\@allowed_domains,
-						                  'library');
-        my (%older_library_servers,%okdoms,%domains_for_id);
-        map { $okdoms{$_} = 1; } @allowed_domains;
-        foreach my $key (keys(%library_servers)) {
-            if (&Apache::lonnet::get_server_loncaparev('',$key) =~ /^\'?(\d+)\.(\d+)/) {
-                my $major = $1;
-                my $minor = $2;
-                if (($major < 2) || (($major == 2) && ($minor < 11))) {
-                    map { $older_library_servers{$_} = 1; }
-                        &Apache::lonnet::machine_ids($library_servers{$key});
-                } else {
-                    my %possdoms;
-                    map { $possdoms{$_}=1 if ($okdoms{$_}); }
-                        &Apache::lonnet::machine_domains($library_servers{$key});
-                    $domains_for_id{$key} = join(',',sort(keys(%possdoms)));
-                }
-            }
-        }
-        my %servers = (%library_servers,%older_library_servers);
+	my %servers = &Apache::lonnet::get_unique_servers(\@allowed_domains,
+						   'library');
 	$libraries_to_query = [keys(%servers)];
-        $domains_to_query = \%domains_for_id;
     }
     return ($libraries_to_query,
-            $pretty_domains_string,
-            $domains_to_query);
+            $pretty_domains_string);
 }
 
 ######################################################################
@@ -1552,8 +1463,7 @@ sub parse_basic_search {
 	$env{"form.$_"}=&unescape($env{"form.$_"});
 	$env{"form.$_"}=~s/[^\w\/\s\(\)\=\-\"\']//g;
     }
-    my ($libraries_to_query,$pretty_domains_string,$domains_to_query) = 
-        &parse_domain_restrictions();
+    my ($libraries_to_query,$pretty_domains_string) = &parse_domain_restrictions();
     #
     # Check to see if enough of a query is filled in
     my $search_string = $env{'form.basicexp'};
@@ -1599,7 +1509,7 @@ sub parse_basic_search {
     $pretty_search_string =~ s:^<br /> and ::;
     &Apache::lonnet::logthis('simple search final query = '.$/.$final_query);
     return ($final_query,$pretty_search_string,
-            $libraries_to_query,$domains_to_query);
+            $libraries_to_query);
 }
 
 
@@ -1890,8 +1800,6 @@ sub build_date_queries {
             my (undef,undef,undef,$cbday,$cbmon,$cbyear) = localtime($cbefore);
             # Correct for year being relative to 1900
             $cayear+=1900; $cbyear+=1900;
-            # Correct month; localtime gives month 0..11 but MySQL expects 1..12
-            $camon++; $cbmon++;
             my $cquery=
                 '(creationdate BETWEEN '.
                 "'".$cayear.'-'.$camon.'-'.$caday."'".
@@ -1910,8 +1818,6 @@ sub build_date_queries {
             my (undef,undef,undef,$mbday,$mbmon,$mbyear) = localtime($mbefore);
             # Correct for year being relative to 1900
             $mayear+=1900; $mbyear+=1900;
-            # Correct month; localtime gives month 0..11 but MySQL expects 1..12
-            $mamon++; $mbmon++;
             my $mquery=
                 '(lastrevisiondate BETWEEN '.
                 "'".$mayear.'-'.$mamon.'-'.$maday."'".
@@ -1954,11 +1860,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') &&
@@ -2061,7 +1966,7 @@ a link to change the search query.
 ######################################################################
 ######################################################################
 sub print_sort_form {
-    my ($r,$pretty_query_string,$target) = @_;
+    my ($r,$pretty_query_string) = @_;
 
     ##
     my %SortableFields=&Apache::lonlocal::texthash( 
@@ -2108,20 +2013,15 @@ sub print_sort_form {
 </script>
 END
 
-    my $args;
-    if ($target eq '_parent') {
-        $args = {'links_target' => $target};
-    }
-    my $start_page = &Apache::loncommon::start_page('Results',undef,$args);
+    my $start_page = &Apache::loncommon::start_page('Results',$js);
     my $breadcrumbs=
         &Apache::lonhtmlcommon::breadcrumbs('Searching','Searching',
-					    $env{'form.catalogmode'} ne 'import',
-                                            '','','','','','',$target);
+					    $env{'form.catalogmode'} ne 'import');
 
     my $result = <<END;
 $start_page
 $breadcrumbs
-<form name="statusform" action="" method="post" target="$target">
+<form name="statusform" action="" method="post" target="_top">
 <input type="hidden" name="catalogmode" value="import" />
 <input type="hidden" name="acts" value="" />
 END
@@ -2139,7 +2039,7 @@ END
 #    $result.="</select>\n";
     my $revise = &revise_button();
     $result.='<p>'
-            .&mt('Total of [quant,_1,match,matches] to your query.',$total_results)
+            .&mt('There are [_1] matches to your query.',$total_results)
             .' '.$revise.'</p>'
             .'<p>'.&mt('Search: ').$pretty_query_string
             .'</p></form>';
@@ -2277,21 +2177,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;
 
@@ -2372,7 +2259,7 @@ results into MySQL.
 ######################################################################
 sub run_search {
     my ($r,$query,$customquery,$customshow,$serverlist,
-        $pretty_string,$area,$domainsref,$target) = @_;
+        $pretty_string,$area) = @_;
     my $tabletype = 'metadata';
     if ($area eq 'portfolio') {
         $tabletype = 'portfolio_search';
@@ -2381,15 +2268,10 @@ sub run_search {
     #
     # Print run_search header
     #
-    my $args;
-    if ($target eq '_parent') {
-        $args = {'links_target' => $target};
-    }
-    my $start_page = &Apache::loncommon::start_page('Search Status',undef,$args);
+    my $start_page = &Apache::loncommon::start_page('Search Status',undef);
     my $breadcrumbs =
 	&Apache::lonhtmlcommon::breadcrumbs('Searching','Searching',
-					    $env{'form.catalogmode'} ne 'import',
-                                            '','','','','','',$target);
+					    $env{'form.catalogmode'} ne 'import');
     $r->print(<<END);
 $start_page
 $breadcrumbs
@@ -2409,39 +2291,16 @@ END
     $r->rflush();
     #
     # Determine the servers we need to contact.
-    my (@Servers_to_contact,%domains_by_server);
+    my @Servers_to_contact;
     if (defined($serverlist)) {
         if (ref($serverlist) eq 'ARRAY') {
             @Servers_to_contact = @$serverlist;
         } else {
             @Servers_to_contact = ($serverlist);
         }
-        if (ref($domainsref) eq 'HASH') {
-            foreach my $server (@Servers_to_contact) {
-                $domains_by_server{$server} = $domainsref->{$server};
-            }
-        }
     } else {
-	my %library_servers = &Apache::lonnet::unique_library();
-        my (%all_library_servers, %older_library_servers);
-        foreach my $key (keys(%library_servers)) {
-            if (&Apache::lonnet::get_server_loncaparev('',$key) =~ /^\'?(\d+)\.(\d+)/) {
-                my $major = $1;
-                my $minor = $2;
-                if (($major < 2) || (($major == 2) && ($minor < 11))) {
-                    map { $older_library_servers{$_} = 1; }
-                        &Apache::lonnet::machine_ids($library_servers{$key});
-                }
-            }
-        }
-        %all_library_servers = (%library_servers,%older_library_servers);
+	my %all_library_servers = &Apache::lonnet::unique_library();
         @Servers_to_contact = sort(keys(%all_library_servers));
-        foreach my $server (@Servers_to_contact) {
-            my %possdoms;
-            map { $possdoms{$_}=1;  } &Apache::lonnet::machine_domains($all_library_servers{$server});
-            $domains_by_server{$server} =
-                join(',',sort(&Apache::lonnet::machine_domains($all_library_servers{$server})));
-        }
     }
     my %Server_status;
     #
@@ -2477,8 +2336,6 @@ END
     ##
     ## Prepare for the big loop.
     my $hitcountsum;
-    my $oldhitcountsum;
-    my $displaycount;
     my %matches;
     my $server; 
     my $status;
@@ -2513,7 +2370,7 @@ END
             my $server = shift(@Servers_to_contact);
             &update_status($r,&mt('contacting [_1]',$server));
             my $reply=&Apache::lonnet::metadata_query($query,$customquery,
-                                                      $customshow,[$server],\%domains_by_server);
+                                                      $customshow,[$server]);
             ($server) = keys(%$reply);
             $Server_status{$server} = $reply->{$server};
         } else {
@@ -2525,7 +2382,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
@@ -2601,15 +2458,7 @@ END
                 delete($Server_status{$server});
             }
             last if ($connection->aborted());
-            if ($oldhitcountsum < $hitcountsum) {
-                &update_count_status($r,$hitcountsum);
-                if (($hitcountsum <= $env{'form.show'}) ||
-                    (!$displaycount && $hitcountsum)) {
-                    reload_result_frame($r);
-                    $displaycount = $hitcountsum;
-                }
-                $oldhitcountsum = $hitcountsum;
-            }
+            &update_count_status($r,$hitcountsum);
         }
         last if ($connection->aborted());
         &update_seconds($r);
@@ -2703,8 +2552,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>'.
@@ -2712,11 +2559,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
@@ -2858,7 +2701,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;
@@ -3231,18 +3075,9 @@ 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,
-                                                      'add_modal' =>1});
+                                                      'add_wishlist' =>1});
     my $result=<<END;
 $start_page
 <form name="results" method="post" action="/adm/searchcat">
@@ -3272,26 +3107,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
@@ -3404,7 +3219,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>,'.
@@ -3537,7 +3352,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 />
@@ -3583,7 +3398,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'}.')';
@@ -3599,17 +3414,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',
                   '',
-                  '',
-                  $skiplast,$onclick).' ';
+                  (($env{'form.catalogmode'} eq 'import')?'parent.statusframe.document.forms.statusform':''),
+                  $skiplast).' ';
     }
     return $link;
 }
@@ -3915,7 +3725,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{$_};
         }