Attachment 37447 Details for Bug 13947 – Bug 13947: Correct call to GetBiblio to receive correct return value

[patch] Bug 13947: Correct call to GetBiblio to receive correct return value

Bug-13947-Correct-call-to-GetBiblio-to-receive-cor.patch (text/plain), 1.51 KB, created by Jacek Ablewicz on 2015-04-02 16:05:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jacek Ablewicz

Created: 2015-04-02 16:05:59 UTC

Size: 1.51 KB

patch

obsolete

>From 79c3170df590d372a24919a45ced80b2ce68f00f Mon Sep 17 00:00:00 2001
>From: Mark Tompsett <mtompset@hotmail.com>
>Date: Thu, 2 Apr 2015 11:15:14 -0400
>Subject: [PATCH] Bug 13947: Correct call to GetBiblio to receive correct
> return value
>
>When you attempt to edit the manual history for a subscription,
>the title is blank. This is a result of an incorrect use of the
>GetBiblio function.
>
>TEST PLAN
>---------
>1) Log in to staff client
>2) Serials
>3) Search for an existing serial with manual history checked.
>   -- creating one as needed is left as an exercise to the reader.
>4) Click on the title of the serial for the details page.
>5) Go to the Planning tab and click 'Edit History'
>   -- Heading will be 'Subscription history for' without a
>      title.
>6) Apply patch
>7) Refresh page.
>   -- Heading will include the title correctly.
>8) run koha qa test tools
>
>Signed-off-by: Jacek Ablewicz <abl@biblos.pk.edu.pl>
>---
> serials/subscription-history.pl |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/serials/subscription-history.pl b/serials/subscription-history.pl
>index 9310b5d..267afcb 100755
>--- a/serials/subscription-history.pl
>+++ b/serials/subscription-history.pl
>@@ -71,7 +71,7 @@ if($op && $op eq 'mod') {
>     exit;
> } else {
>     my $history = GetSubscriptionHistoryFromSubscriptionId($subscriptionid);
>-    my (undef, $biblio) = GetBiblio($history->{'biblionumber'});
>+    my $biblio  = GetBiblio($history->{'biblionumber'});
> 
>     $template->param(
>         subscriptionid  => $subscriptionid,
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 13947: 37443 | 37447 | 37461