Bug 9857 - Follow-up - Fix for searches with su= or su:
authorKatrin Fischer <katrin.fischer.83@web.de>
Fri, 18 Aug 2017 08:36:59 +0000 (08:36 +0000)
committerMason James <mtj@kohaaloha.com>
Tue, 24 Oct 2017 03:42:12 +0000 (16:42 +1300)
When the initial search is su=.../su:... the links was
not constructed correctly. With this change, it should
be the case.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Mason James <mtj@kohaaloha.com>

Koha/SuggestionEngine/Plugin/AuthorityFile.pm
Koha/SuggestionEngine/Plugin/ExplodedTerms.pm

index fad0a58..cfb34da 100644 (file)
@@ -30,8 +30,7 @@ Plugin to get suggestions from Koha's authority file
 
 =cut
 
-use strict;
-use warnings;
+use Modern::Perl;
 use Carp;
 
 use base qw(Koha::SuggestionEngine::Base);
index c8a3339..7129d9b 100644 (file)
@@ -31,8 +31,7 @@ subjects to subject searches.
 
 =cut
 
-use strict;
-use warnings;
+use Modern::Perl;
 use Carp;
 use C4::Templates qw(gettemplate); # This is necessary for translatability
 
@@ -86,7 +85,7 @@ sub get_suggestions {
     foreach my $index (@indexes) {
         my $thissearch = $search;
         $thissearch = "$index:$thissearch"
-          unless ( $thissearch =~ s/OP!/$index=/g );
+          unless ( $thissearch =~ s/OP!/$index:/g );
         $template->{VARS}->{index} = $index;
         my $label = $template->output;
         push @results,