--- loncom/interface/lonsearchcat.pm	2023/12/30 03:56:48	1.331.4.16.2.2
+++ loncom/interface/lonsearchcat.pm	2015/06/16 20:24:39	1.346
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Search Catalog
 #
-# $Id: lonsearchcat.pm,v 1.331.4.16.2.2 2023/12/30 03:56:48 raeburn Exp $
+# $Id: lonsearchcat.pm,v 1.346 2015/06/16 20:24:39 damieng Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -176,16 +176,7 @@ sub handler {
     } 
     my $crumb_text = 'Portfolio Search';
     if (@allowed_searches ==2) {
-       $crumb_text = 'Portfolio and Content Library 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';
-        }
+       $crumb_text = 'Portfolio and Catalog Search'; 
     }
     &Apache::lonhtmlcommon::add_breadcrumb
        ({href=>'/adm/searchcat?'.
@@ -194,7 +185,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/) {
@@ -308,7 +299,7 @@ END
     }
     #
     if ($env{'form.searchmode'} eq 'advanced') {
-        my $srchtype = 'Content Library';
+        my $srchtype = 'Catalog';
         if ($env{'form.area'} eq 'portfolio') {
             $srchtype = 'Portfolio';
         }
@@ -337,10 +328,10 @@ END
                  ['query','customquery','customshow',
                   'libraries','pretty_string','domains']);
         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'},$domainsref);
         }
     } elsif(($env{'form.phase'} eq 'basic_search') ||
             ($env{'form.phase'} eq 'adv_search')) {
@@ -442,7 +433,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.
@@ -474,7 +465,7 @@ sub setup_basic_search {
     my ($r,$area,$hidden_fields,$closebutton) = @_;
     # Define interface components
     my %lt = &Apache::lonlocal::texthash (
-                              res => 'Content Library Search',
+                              res => 'LON-CAPA Catalog Search',
                               portfolio => 'Portfolio Search',
     );
     my ($userelatedwords,$onlysearchdomain,$inclext,$adv_search_link,$scrout);
@@ -488,31 +479,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?'.
@@ -599,7 +573,7 @@ sub print_advanced_search_form{
 $closebutton
 </p>
 END
-    my $srchtype = 'Content Library';
+    my $srchtype = 'Catalog';
     my $jscript;
     if ($env{'form.area'} eq 'portfolio') {
         $srchtype = 'Portfolio';
@@ -687,22 +661,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') {
@@ -1890,8 +1852,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 +1870,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 +1912,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 +2018,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( 
@@ -2094,34 +2051,15 @@ sub print_sort_form {
         &Apache::lonnet::logthis(&Apache::lonmysql::get_error());
         return;
     }
-    my $js =<<END;
-<script type="text/javascript">
-// <![CDATA[
-    function change_sort() {
-        var newloc = "/adm/searchcat?phase=results";
-        newloc += "&persistent_db_id=$env{'form.persistent_db_id'}";
-        newloc += "&sortby=";
-        newloc += document.forms.statusform.elements.sortby.value;
-        parent.resultsframe.location= newloc;
-    }
-// ]]>
-</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',undef);
     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
@@ -2372,7 +2310,7 @@ results into MySQL.
 ######################################################################
 sub run_search {
     my ($r,$query,$customquery,$customshow,$serverlist,
-        $pretty_string,$area,$domainsref,$target) = @_;
+        $pretty_string,$area,$domainsref) = @_;
     my $tabletype = 'metadata';
     if ($area eq 'portfolio') {
         $tabletype = 'portfolio_search';
@@ -2381,15 +2319,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
@@ -2478,7 +2411,6 @@ END
     ## Prepare for the big loop.
     my $hitcountsum;
     my $oldhitcountsum;
-    my $displaycount;
     my %matches;
     my $server; 
     my $status;
@@ -2525,7 +2457,7 @@ END
                 &update_status($r,
                        &mt('waiting on [_1]',join(' ',keys(%Server_status))));
             }
-            sleep(0.1); 
+            sleep(0.1);
         }
         #
         # Loop through the servers we have contacted but do not
@@ -2578,8 +2510,8 @@ END
 
                     if ($area eq 'portfolio') {
                         next if (defined($matches{$Fields{'url'}}));
-                        # Skip unless access control set to public or passphrase-protected
-                        next unless (($Fields{'scope'} eq 'public') || ($Fields{'scope'} eq 'guest'));
+                        # Skip if inaccessible
+                        next if (!&Apache::lonnet::portfolio_access($Fields{'url'}));
                         $matches{$Fields{'url'}} = 1; 
                     }
                     #
@@ -2603,10 +2535,8 @@ END
             last if ($connection->aborted());
             if ($oldhitcountsum < $hitcountsum) {
                 &update_count_status($r,$hitcountsum);
-                if (($hitcountsum <= $env{'form.show'}) ||
-                    (!$displaycount && $hitcountsum)) {
+                if ($hitcountsum <= $env{'form.show'}) {
                     reload_result_frame($r);
-                    $displaycount = $hitcountsum;
                 }
                 $oldhitcountsum = $hitcountsum;
             }
@@ -2712,11 +2642,11 @@ 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
@@ -3269,7 +3199,6 @@ sub print_frames_interface {
     my $results_link = &results_link();
     my $js = <<JS;
 <script type="text/javascript">
-// <![CDATA[
 var targetwin = opener;
 var queue = '';
 
@@ -3292,7 +3221,6 @@ function done_loading_results() {
         update_results(new_location);
     }
 }
-// ]]>
 </script>
 JS
 
@@ -3404,7 +3332,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 +3465,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 +3511,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 +3527,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;
 }