Bug 22030 - OverDrive requires configuration for field passed as username
Summary: OverDrive requires configuration for field passed as username
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: P5 - low major (vote)
Assignee: Nick Clemens
QA Contact: Martin Renvoize
URL:
Keywords:
Depends on: 21082
Blocks:
  Show dependency treegraph
 
Reported: 2018-12-20 02:07 UTC by Nick Clemens
Modified: 2020-01-06 20:15 UTC (History)
7 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 22030: Send cardnumber to overdrive instead of username (1.77 KB, patch)
2018-12-20 02:12 UTC, Nick Clemens
Details | Diff | Splinter Review
DO NOT PUSH - example of building filters for uri parameters (2.69 KB, patch)
2018-12-20 22:24 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 22030: Use preference to determine username sent to overdrive (5.46 KB, patch)
2019-01-07 15:33 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 22030: Use preference to determine username sent to overdrive (5.57 KB, patch)
2019-01-07 15:54 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 22030: Use preference to determine username sent to overdrive (5.60 KB, patch)
2019-01-07 18:34 UTC, Martin Renvoize
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens 2018-12-20 02:07:11 UTC
The overdrive API wants cardnumbers, not usernames
Comment 1 Nick Clemens 2018-12-20 02:12:17 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2018-12-20 22:24:14 UTC Comment hidden (obsolete)
Comment 3 Nick Clemens 2019-01-07 15:33:12 UTC Comment hidden (obsolete)
Comment 4 Nick Clemens 2019-01-07 15:54:56 UTC
Created attachment 83693 [details] [review]
Bug 22030: Use preference to determine username sent to overdrive

Overdrive configuration generally defaults to cardnumber, however, they
have confirmed that some libraries use usernames. We need to allow for
both scenarios.

To test:
1 - Have an OverDrive account setup with SIP authentication
    Note: You can apply for a testing account at developer.overdrive.com
    and setup an environment
2 - Fill in all your OverDrive system preferences
3 - Test with a patron whose username is their cardnumber
4 - Confirm their overdrive account tab on opac loads
5 - Change the username to be another value like "borked_wont_work"
6 - Note the overdrive account tab won't load
7 - Apply patch, update database, not new system preference
OverDriveUsername (default to 'cardnumber)
8 - Note the OD account loads successfully
9 - Change the system preference to 'user name' - the account load fails
Comment 5 Martin Renvoize 2019-01-07 18:34:29 UTC
Created attachment 83702 [details] [review]
Bug 22030: Use preference to determine username sent to overdrive

Overdrive configuration generally defaults to cardnumber, however, they
have confirmed that some libraries use usernames. We need to allow for
both scenarios.

To test:
1 - Have an OverDrive account setup with SIP authentication
    Note: You can apply for a testing account at developer.overdrive.com
    and setup an environment
2 - Fill in all your OverDrive system preferences
3 - Test with a patron whose username is their cardnumber
4 - Confirm their overdrive account tab on opac loads
5 - Change the username to be another value like "borked_wont_work"
6 - Note the overdrive account tab won't load
7 - Apply patch, update database, not new system preference
OverDriveUsername (default to 'cardnumber)
8 - Note the OD account loads successfully
9 - Change the system preference to 'user name' - the account load fails

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 6 Martin Renvoize 2019-01-07 18:34:58 UTC
Trivia fix, no regressions.. going straight for QA here.
Comment 7 Nick Clemens 2019-01-08 15:56:01 UTC
Awesome work all!

Pushed to master for 19.05
Comment 8 Martin Renvoize 2019-01-10 13:45:44 UTC
Pushed to 18.11.x for 18.11.02
Comment 9 Jesse Maseto 2019-01-11 20:19:24 UTC
Pushed to 18.05.x for 18.05.08
Comment 10 Fridolin Somers 2019-01-16 08:29:46 UTC
Depends on Bug 21082 not in 17.11.x