Bug 15128: Revert Display all purchase suggestions whether or not suggestion was...
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Fri, 1 Jul 2016 15:55:39 +0000 (16:55 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 8 Jul 2016 14:09:49 +0000 (14:09 +0000)
This does not work correctly and should be handle on another bug report.
The behavior existed prior to this patchset.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
opac/opac-suggestions.pl

index 52c0775..1cc96c2 100755 (executable)
@@ -148,14 +148,16 @@ if ( $op eq "add_confirm" ) {
         $suggestion->{branchcode} = $input->param('branchcode') || C4::Context->userenv->{"branch"};
 
         &NewSuggestion($suggestion);
+
+        # delete empty fields, to avoid filter in "SearchSuggestion"
+        foreach my $field ( qw( title author publishercode copyrightdate place collectiontitle isbn STATUS ) ) {
+            delete $suggestion->{$field}; #clear search filters (except borrower related) to show all suggestions after placing a new one
+        }
+        $suggestions_loop = &SearchSuggestion($suggestion);
+
         push @messages, { type => 'info', code => 'success_on_inserted' };
 
     }
-    # delete empty fields, to avoid filter in "SearchSuggestion" and load all suggestions for display
-    foreach my $field ( qw( title author publishercode copyrightdate place collectiontitle isbn STATUS ) ) {
-        delete $suggestion->{$field}; #clear search filters (except borrower related) to show all suggestions after placing a new one
-    }
-    $suggestions_loop = &SearchSuggestion($suggestion);
     $op = 'else';
 }