Bug 4131 - link on subscripton page says 'create routing' even when one exists
Summary: link on subscripton page says 'create routing' even when one exists
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Serials (show other bugs)
Version: Main
Hardware: PC All
: PATCH-Sent (DO NOT USE) minor
Assignee: Nicole C. Engard
QA Contact: Bugs List
URL: cgi-bin/koha/serials/subscription-det...
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-07 04:39 UTC by Nicole C. Engard
Modified: 2012-10-25 23:04 UTC (History)
4 users (show)

See Also:
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 (37.54 KB, image/png)
2010-02-07 16:39 UTC, Chris Cormack
Details
patch (1.74 KB, patch)
2010-06-03 00:05 UTC, Nicole C. Engard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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