Summary: | Subscriptions titles are replaced by "---" when duplicated | ||
---|---|---|---|
Product: | Koha | Reporter: | Adrien SAURAT <adrien.saurat> |
Component: | Serials | Assignee: | Adrien SAURAT <adrien.saurat> |
Status: | CLOSED FIXED | QA Contact: | Paul Poulain <paul.poulain> |
Severity: | enhancement | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, paul.poulain |
Version: | 3.6 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
proposed patch
Bug 7357: display serial title instead of '---' |
Description
Adrien SAURAT
2011-12-14 13:33:58 UTC
I agree that the current display is not ideal, but perhaps we can find a way to do both? Indicate that there is more than one subscription for this title and display more than -- ? Perhaps Owen or someone else has a good idea? Could the subscriptions for the title be grouped somehow in the same table line? So each title gets a row, then as many subscriptions are put into a table of their own, with columns for the rest of the data? Our customer just asked for the title to appear instead of this '---' string. I really think it would be enough, and don't see any real advantage in doing something complicated here. I'll wait for a community decision, but my vote would go for the simple removal of this "replace TITLE with ---" rule. I've done some experimenting with it and have not been able to come up with a solution that I think works better. For now I would agree that letting duplicate titles appear is probably the best option. Created attachment 7090 [details] [review] proposed patch Created attachment 7126 [details] [review] Bug 7357: display serial title instead of '---' The full title of a serial is now displayed even if several lines have the same title. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> QA comment: simple patch, no comment. This patch will be needed when introducing DataTable to this list This fix was included in the 3.6.x branch prior to 3.6.4. |