[Koha-bugs] [Bug 21702] mancredit.pl incorrectly passes user_id instead of the patron id
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Mon Oct 29 23:00:12 CET 2018
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21702
Josef Moravec <josef.moravec at gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #81464|0 |1
is obsolete| |
Attachment #81465|0 |1
is obsolete| |
--- Comment #5 from Josef Moravec <josef.moravec at gmail.com> ---
Created attachment 81593
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81593&action=edit
Bug 21702: Make mancredit.pl use the patron id for the staff user
I think unsafe SQL modes and the fact that manager_id has no FK allowed this to
go unnoticed. But now we catch it!
To test:
- On a patron, try adding a new manual credit of any type
=> FAIL: It fails telling the userid of the logged user is not a valid integer!
- Apply this patch
- Try adding a manual credit of any type
=> SUCCESS: Manual credit added!
- Sign off!
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Andrew Isherwood <andrew.isherwood at ptfs-europe.com>
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list