]> git.koha-community.org Git - koha.git/commit
Bug 33238: Fix bad suggestions query (Bug 29886 follow-up)
authorJanusz Kaczmarek <januszop@gmail.com>
Wed, 15 Mar 2023 19:25:53 +0000 (20:25 +0100)
committerLucas Gass <lucas@bywatersolutions.com>
Fri, 5 May 2023 22:29:15 +0000 (22:29 +0000)
commitf849dd313cde6dfdd03aba5652fb9429f093c901
treec43103b54e411dab9f30fd5fb552eaa96f255fe6
parent435c79b8a47b8ee47fd493ca4ac17a6413e9ba00
Bug 33238: Fix bad suggestions query (Bug 29886 follow-up)

With IndependentBranches Koha explodes with Error 500 when a non
superlibrarian tries to add to basket from suggestions.  In errorlog you
would find "DBIx::Class::Storage::DBI::_dbh_execute(): DBI Exception:
DBD::mysql::st execute failed: Column 'branchcode' in where clause is
ambiguous".

Test plan:
==========
1. Have a Koha instance with IndependentBranches, some accepted
   suggestions, and a vendor defined.
2. Be a non superlibrarian librarian with permissions to
   manage acquisitions.
3. Create a new basket and try to add an item to the basket
   "From a suggestion".
4. Koha would generate error 500.
5. Apply the patch.
6. Try to add an item to the basket "From a suggestion".
7. You should succeed.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 68b2f29d483290d5dfd859dce9b9fe7a387f3ab1)
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
(cherry picked from commit d4fb68703098f25bd587a8fdfc6cd5b4d884c4f9)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Koha/Suggestions.pm