Bug 15853

Summary: Add author and link columns to opac course reserves table
Product: Koha Reporter: Nick Clemens <nick>
Component: Course reservesAssignee: Nick Clemens <nick>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: amy, f.demians, jean-manuel.broust, jonathan.druart, kyle, nicole
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: Sponsored Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 15853 - Add author and link columns to opac course reserves table
Bug 15853 - Add author and link columns to opac course reserves table
[SIGNED-OFF] Bug 15853 - Add author and link columns to opac course reserves table
Bug 15853 - Add author and link columns to opac course reserves table

Description Nick Clemens 2016-02-17 23:07:44 UTC
To be done:

1 - Add an author column to the opac course reserves table
2 - Add a 'link' column to the opac course reserves table, populated from the 856$u field
Comment 1 Nick Clemens 2016-06-03 18:52:47 UTC Comment hidden (obsolete)
Comment 2 jmbroust 2016-06-29 13:59:43 UTC
We tried to apply the patch on sandboxes 2 and 6 but it failed each time with no logs (blocked on web installer twice). So maybe there is a problem.
Comment 3 Nick Clemens 2016-06-30 13:34:54 UTC Comment hidden (obsolete)
Comment 4 Nick Clemens 2016-06-30 13:35:37 UTC
(In reply to jmbroust from comment #2)
> We tried to apply the patch on sandboxes 2 and 6 but it failed each time
> with no logs (blocked on web installer twice). So maybe there is a problem.
Rebased (it was my patch on another bug that caused the conflict :-) )
Comment 5 Owen Leonard 2016-07-12 15:19:57 UTC Comment hidden (obsolete)
Comment 6 Jonathan Druart 2016-07-15 13:41:14 UTC
Created attachment 53433 [details] [review]
Bug 15853 - Add author and link columns to opac course reserves table

To test:
1 - Add some course reserves
2 - View the table in staff and opac, there is no column for author or
link
3 - Apply patch
4 - You shoudl see these columns
5 - Link should contain item uri if present, fallback to title 856$u,
  then be blank if niether

Sponsored by:
    Southeastern Wisconsin Information Technology Exchange, Inc (SWITCH)
    (http://caspian.switchinc.org/)

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Might be good to add this table to the column visibility tool if you
plan to add more columns.
Comment 7 Jonathan Druart 2016-07-15 13:42:14 UTC
Nick, you should check if the release note script handles the "Sponsored by:" on multiple lines.
Comment 8 Kyle M Hall 2016-07-15 18:04:27 UTC
Pushed to master for 16.11, thanks Nick!
Comment 9 Frédéric Demians 2016-08-01 13:42:27 UTC
Pushed in 16.05. Will be in 16.05.02.