Bug 16317

Summary: Attempt to share private list results in error
Product: Koha Reporter: Kyle M Hall (khall) <kyle>
Component: OPACAssignee: Kyle M Hall (khall) <kyle>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: major    
Priority: P5 - low CC: brendan, julian.maurice, m.de.rooy, oleonard
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 16317 - Attempt to share private list results in error
[SIGNED-OFF] Bug 16317 - Attempt to share private list results in error
Bug 16317 - Attempt to share private list results in error

Description Kyle M Hall (khall) 2016-04-21 12:50:40 UTC
When sharing a private list, the the receiver of the share link will get the following error when they are logged in and try to click the link:

DBIx::Class::Storage::DBI::_dbh_execute(): Unknown column 'sharedate desc' in 'order clause' at /home/koha/kohaclone/Koha/Objects.pm line 150
Comment 1 Kyle M Hall (khall) 2016-04-21 12:52:38 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2016-04-21 12:56:06 UTC
*** Bug 16287 has been marked as a duplicate of this bug. ***
Comment 3 Owen Leonard 2016-04-21 13:10:16 UTC Comment hidden (obsolete)
Comment 4 Marcel de Rooy 2016-04-22 06:39:25 UTC
Created attachment 50561 [details] [review]
Bug 16317 - Attempt to share private list results in error

When sharing a private list, the the receiver of the share link will get
the following error when they are logged in and try to click the link:

DBIx::Class::Storage::DBI::_dbh_execute(): Unknown column 'sharedate
desc' in 'order clause' at /home/koha/kohaclone/Koha/Objects.pm line 150

Test Plan:
1) Create a private list
2) Share that list to another account
3) Log in as the other account
4) Click the link in the email you recieved
5) Note the error
6) Apply this patch
7) Click the link again
8) You should see no error and be able to view the list!

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 5 Brendan Gallagher 2016-04-22 22:49:07 UTC
Pushed to Master - Should be in the May 2016 Release.  Thanks!
Comment 6 Julian Maurice 2016-04-25 12:17:11 UTC
Patch pushed to 3.22.x, will be in 3.22.6