Main Koha release repository https://koha-community.org
Find a file
Tomas Cohen Arazi 1eb2c4337b
Bug 31395: Only try to cancel holds if item found
This patch makes the code dealing with waiting holds with cancellation
requests be dependent on the fact an item has been found.

The returns.pl controller is a bit messy as the real return takes place
outside the main `if ($item)` block. This should be refactored and
probably run inside a transaction...

In the meantime this patch will make the job.

To test:
1. Try to return an invalid barcode (e.g. ASDQWE)
=> FAIL: Things explode
2. Apply this patch
3. Repeat 1
=> SUCCESS: Doesn't explode!
4. Verify that returning an item with a waiting hold with cancellation
   requests still cancells the hold.
=> SUCCESS: It does!
5. Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
2022-08-19 09:56:43 -03:00
acqui Bug 30718: Pass DT to filter_by_lates 2022-08-19 08:26:37 -03:00
admin Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
api Bug 24857: No words 2022-08-09 14:00:40 -03:00
authorities Bug 29333: Fix encoding of imported UNIMARC authorities 2022-07-08 15:43:33 -03:00
basket Bug 31309: Remove GetItemsInfo from basket/sendbasket 2022-08-16 09:22:14 -03:00
bin Bug 20582: Turn Koha into a Mojolicious application 2020-10-06 12:00:04 +02:00
C4 Bug 30718: Fix renewal with due date 2022-08-19 08:26:39 -03:00
catalogue Bug 30779: Remove _update_import_record_marc and update tests 2022-08-18 09:02:04 -03:00
cataloguing Bug 30941: (QA follow-up) Add exec flag to scripts 2022-08-19 08:55:09 -03:00
circ Bug 31395: Only try to cancel holds if item found 2022-08-19 09:56:43 -03:00
clubs Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
course_reserves Bug 30409: barcodedecode() should always trim barcode 2022-06-14 07:54:58 -03:00
debian Bug 30684: (follow-up) Use log_warning_msg for the "not running" warning 2022-08-18 09:00:39 -03:00
docs Bug 30808: Add the 22.05 release team. 2022-05-25 23:56:12 -10:00
errors
etc Bug 31276: Report results are limited to 999,999 no matter the actual number of results 2022-08-16 09:14:18 -03:00
ill Bug 28909: Allow illview to use backend template 2022-08-09 13:21:39 -03:00
installer Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
Koha Bug 30941: (QA follow-up) Import methods explicitly 2022-08-19 09:49:36 -03:00
koha-tmpl Bug 30941: Add cataloguing plugins for 146$bcdef (UNIMARC) 2022-08-19 08:55:09 -03:00
labels Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
lib/CGI/Session/Serialize
members Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
misc Bug 31282: Fix broken characters in patron_emailer.pl verbose mode 2022-08-19 09:56:13 -03:00
offline_circ Bug 30275: Rename issues.renewals to issues.renewals_count 2022-07-05 09:45:55 -03:00
opac Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
patron_lists
patroncards Bug 24001: Fix patron card template edition 2022-04-28 10:49:20 -10:00
plugins
pos Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
recalls Bug 30924: Add missing branchtransfers.reason value for recall cancellation 2022-06-13 10:30:51 -03:00
reports Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
reserve Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
reviews
rotating_collections
serials Bug 30718: Fix serials/showpredictionpattern.pl 2022-08-19 08:26:37 -03:00
services
skel
suggestion Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
svc Bug 26632: Remove two more occurrences 2022-08-17 09:24:09 -03:00
t Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
tags Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
tmp/modified_authorities
tools Bug 30718: Fix scheduler 2022-08-19 08:26:38 -03:00
virtualshelves Bug 31320: Remove GetItemsInfo from virtualshelves/sendshelf.pl 2022-08-16 09:23:25 -03:00
xt Bug 27619: (QA follow-up) Remove xt/sample_notices.t 2022-05-11 11:28:48 +01:00
.editorconfig
.eslintrc.json
.gitignore
.htaccess Fix file permissions: if it is not a script, it should not be executable. 2010-04-16 00:40:34 -04:00
.mailmap 22.05.00: Update mailmap 2022-05-25 23:56:12 -10:00
.perlcriticrc
.proverc.dist
.scss-lint.yml
about.pl Bug 27667: Display a warning for records missing in Elasticsearch 2022-07-18 14:53:15 -03:00
app.psgi
changelanguage.pl
cpanfile Bug 25669: (follow-up) Minor fixes 2022-06-23 11:30:44 -03:00
fix-perl-path.PL
gulpfile.js Bug 30373: Enable translation of UNIMARC frameworks 2022-04-21 13:41:35 -10:00
help.pl
INSTALL Bug 26617: Update INSTALL file to include koha-testing-docker and Gitlab links 2020-10-15 12:56:30 +02:00
Koha.pm Bug 28787: DBRev 22.06.00.035 2022-08-09 13:42:21 -03:00
koha_perl_deps.pl
kohaversion.pl
LICENSE
mainpage.pl Bug 30650: Add filter_by_scheduled_today 2022-07-29 15:00:47 -03:00
Makefile.PL
MANIFEST.SKIP
package.json
README
README.md
README.robots
rewrite-config.PL
yarn.lock

Koha is a free software integrated library system (ILS).

Koha is distributed under the GNU GPL version 3 or later.

Note: Koha does not accept pull requests from git hosting sites.

Note: This project has its own bug tracker, to report a bug or submit a patch visit http://bugs.koha-community.org.

For guidelines on submitting patches for Koha please visit https://wiki.koha-community.org/wiki/SubmitingAPatch

The developers handbook can be found at https://wiki.koha-community.org/wiki/Developer_handbook

http://koha-community.org/

Koha Logo