Bug 4131

Summary: link on subscripton page says 'create routing' even when one exists
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: SerialsAssignee: Nicole C. Engard <nengard>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: minor    
Priority: PATCH-Sent (DO NOT USE) CC: chris, cnighswonger, gcollum, oleonard
Version: Main   
Hardware: PC   
OS: All   
URL: cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=5
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: menu on subscription
patch

Description Chris Cormack 2010-05-21 01:23:30 UTC


---- Reported by nengard@gmail.com 2010-02-07 16:39:20 ----

Created an attachment
menu on subscription

When on the subscription page there is a link to 'create routing list' on the menu on the left even if there is already a routing list created. it should probably say 'edit routing list' or maybe even move the link to a button on the top of the subscription.



--- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:23 UTC  ---

This bug was previously known as _bug_ 4131 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4131
Imported an attachment (id=1860)

Actual time not defined. Setting to 0.0
The original submitter of attachment 1860 [details] is unknown.
   Reassigning to the person who moved it here: chris@bigballofwax.co.nz.

Comment 1 Garry Collum 2010-05-22 14:08:43 UTC
The menu for that contains 'create routing list' appears on several serials pages.  At first, I had the idea of calling it 'Create/Edit Routing List,' but that's far too lengthy, IMO.  I propose that we just call the link 'Routing List.'
Comment 2 Nicole C. Engard 2010-05-23 20:53:20 UTC
works for me
Comment 3 Nicole C. Engard 2010-06-02 12:33:40 UTC
anyone else have anything to say? If not I'll edit the template
Comment 4 Nicole C. Engard 2010-06-03 00:03:40 UTC
Found the problem - the template was referring to the wrong variable to check what to show on the menu.  Patch coming.
Comment 5 Nicole C. Engard 2010-06-03 00:05:24 UTC
Created attachment 2208 [details] [review]
patch
Comment 6 Chris Cormack 2011-01-11 23:43:29 UTC
Pushed to master, please test

Chris N, I have NOT verified this applies to 3.2.x