Attachment 60728 Details for Bug 18181 – Bug 18181: Can't tell which subscriptions already have a routing list if seeing all subs attached to a biblio

[patch] Bug 18181: Can't tell which subscriptions already have a routing list if seeing all subs attached to a biblio

Bug-18181-Cant-tell-which-subscriptions-already-ha.patch (text/plain), 1.44 KB, created by Aleisha Amohia on 2017-02-28 02:24:35 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Aleisha Amohia

Created: 2017-02-28 02:24:35 UTC

Size: 1.44 KB

patch

obsolete

>From 4dd7bffe1f11778928bdee12b593a05904329709 Mon Sep 17 00:00:00 2001
>From: Aleisha Amohia <aleishaamohia@hotmail.com>
>Date: Tue, 28 Feb 2017 02:20:01 +0000
>Subject: [PATCH] Bug 18181: Can't tell which subscriptions already have a
> routing list if seeing all subs attached to a biblio
>
>To test:
>1) Go to Serials -> search for a subscription
>2) Add a new routing list for a subscription
>3) After saving, you should be redirected to the subscription detail
>page
>4) Click Serial collection on the left sidebar menu
>5) Notice there is a button that says 'Edit routing list' next to this
>subscription in the first table
>6) Click 'See any subscription attached to this biblio' link
>7) Notice it no longer says edit next to this subscription, it says
>'Create routing list'
>8) Apply patch and refresh page
>9) Should now show 'Edit' button like expected
>
>Sponsored-by: Catalyst IT
>---
> serials/serials-collection.pl | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/serials/serials-collection.pl b/serials/serials-collection.pl
>index 47d07eb..5182cbd 100755
>--- a/serials/serials-collection.pl
>+++ b/serials/serials-collection.pl
>@@ -146,6 +146,7 @@ my $subscriptionidlist="";
> foreach my $subscription (@$subscriptiondescs){
>   $subscriptionidlist.=$subscription->{'subscriptionid'}."," ;
>   $biblionumber = $subscription->{'bibnum'} unless ($biblionumber);
>+  $subscription->{'hasRouting'} = check_routing($subscription->{'subscriptionid'});
> }
> 
> chop $subscriptionidlist;
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 18181: 60728 | 61457 | 61480