]> git.koha-community.org Git - koha.git/commit
Bug 26999: (follow-up) Simplify code
authorTomas Cohen Arazi <tomascohen@theke.io>
Mon, 15 Mar 2021 11:33:49 +0000 (08:33 -0300)
committerFridolin Somers <fridolin.somers@biblibre.com>
Tue, 13 Apr 2021 15:03:33 +0000 (17:03 +0200)
commit7ca4b8e15c968b763d58b428a1aed26a293d08b8
tree14e911fb367467b32db00b939c88bce316fd5709
parent1d863a6dfc71f7193f533c06c2906de7d17bacfb
Bug 26999: (follow-up) Simplify code

Doing
$ git grep pickup_locations_code

shows there's some calculated data that is not actually used anywhere.
We can get rid of it.

This patch also reuses $item_object (which is in the same loop scope) to
avoid an extra DB call.

To test:
1. Run:
   $ git grep pickup_locations_code
=> FAIL: It is only used/set as a comma separated string, inside
request.pl
2. Apply this patch
3. Repeat 1
=> SUCCESS: The unused stuff is not there anymore
4. Open the page for placing some holds
=> SUCCESS: It doesn't explode
5. Sign off :-D

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit 0fd2be61e8376ad0db0f8ecf1eacde706cec6816)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
reserve/request.pl