Bug 7285 - ILSDI service AuthenticatePatron doesn't work
Summary: ILSDI service AuthenticatePatron doesn't work
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: 3.6
Hardware: All All
: PATCH-Sent (DO NOT USE) blocker (vote)
Assignee: Alex Arnaud
QA Contact: Ian Walls
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-02 08:09 UTC by Alex Arnaud
Modified: 2019-06-27 09:24 UTC (History)
2 users (show)

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


Attachments
Proposed patch (852 bytes, patch)
2011-12-02 08:21 UTC, Alex Arnaud
Details | Diff | Splinter Review
bug/7285 Use C4::Auth::checkpw instead of checkpw only (904 bytes, patch)
2011-12-02 08:56 UTC, Chris Cormack
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Arnaud 2011-12-02 08:09:41 UTC
I get this error: "Undefined subroutine &C4::ILSDI::Services::checkpw called at /home/alex/workspace/versions/koha34/C4/ILSDI/Services.pm line 318." when i call AuthenticatePatron service.
Comment 1 Alex Arnaud 2011-12-02 08:21:26 UTC Comment hidden (obsolete)
Comment 2 Alex Arnaud 2011-12-02 08:45:16 UTC
for testing:

kohaopac/cgi-bin/koha/ilsdi.pl?service=AuthenticatePatron&username=<USERNAME>&password=<PASSWORD>
Comment 3 Chris Cormack 2011-12-02 08:56:19 UTC
Very simple change, easily tested, signing off
Comment 4 Chris Cormack 2011-12-02 08:56:48 UTC
Created attachment 6516 [details] [review]
bug/7285 Use C4::Auth::checkpw instead of checkpw only

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Comment 5 Ian Walls 2011-12-02 13:52:12 UTC
Just changes subroutine call to explicitly use the right module.  Only affects the one script in ILSDI, so marking as Passed QA
Comment 6 Paul Poulain 2011-12-06 18:00:21 UTC
patch pushed, please test