Attachment 127347 Details for Bug 29416 – Bug 29416: (bug 28941 follow-up) Restore suggestion prefill

[patch] Bug 29416: (bug 28941 follow-up) Restore suggestion prefill

Bug-29416-bug-28941-follow-up-Restore-suggestion-p.patch (text/plain), 1.44 KB, created by Jonathan Druart on 2021-11-05 09:18:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2021-11-05 09:18:34 UTC

Size: 1.44 KB

patch

obsolete

>From 06ea356d4fd81bd996d73b82b6f2d74aec353a46 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Fri, 5 Nov 2021 10:13:47 +0100
>Subject: [PATCH] Bug 29416: (bug 28941 follow-up) Restore suggestion prefill
>
>From commit 586bed1319592e05f5dc3acf64a1dba8cae69d6b
>     Bug 28941: Filter suggestion inputs at the OPAC
>
>But later in the script we send { suggestion => %$suggestion } to the
>template and so erase the variable previously passed.
>
>Test plan:
>Login at the OPAC
>Go to the bibliographic detail page
>Suggest for purchase
>=> The form must be prefilled!
>---
> opac/opac-suggestions.pl | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/opac/opac-suggestions.pl b/opac/opac-suggestions.pl
>index 6f6e91b45a5..9c965041ce2 100755
>--- a/opac/opac-suggestions.pl
>+++ b/opac/opac-suggestions.pl
>@@ -254,7 +254,7 @@ my @mandatoryfields;
>     }
>     if ( $biblionumber ) {
>         my $biblio = Koha::Biblios->find($biblionumber);
>-        $template->param(
>+        $suggestion = {
>             biblionumber    => $biblio->biblionumber,
>             title           => $biblio->title,
>             author          => $biblio->author,
>@@ -263,7 +263,7 @@ my @mandatoryfields;
>             publishercode   => $biblio->biblioitem->publishercode,
>             collectiontitle => $biblio->biblioitem->collectiontitle,
>             place           => $biblio->biblioitem->place,
>-        );
>+        };
>     }
> }
> 
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 29416: 127347 | 127354 | 127356