Go to file
Kyle M Hall 8a975f8d30 Bug 11258: fix another case where holds queue made transfer requests that contradict the library holds policy
This patch fixes a problem where the holds queue generator
was making requests where the pickup library is the
same as the item's library but not the patron's branch,
even if there is a "Default holds policy by item type" rule that states
this item can only fill holds for patrons of the same library as the
item.

Test Plan:
1) Create a test record with 2 items with different itemtypes
2) Set the Default holds policy by item type for the first
   item to "From any library"
3) Set the Default holds policy by item type for the second
   item to "From home library"
4) Place a record level hold for a patron from another library,
   but for pickup at the same library as the item is from
5) Rebuild the holds queue
6) View the holds queue, note the item is listed, though this
   patron cannot place a hold on this item
7) Apply this patch
8) Repeat step 5, note the hold is no longer in the queue

Signed-off-by: Liz Rea <liz@catalyst.net.nz>
automated tests pass, functional tests pass. Bug replicated, eradicated by patch.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
I finally managed to reproduce this, patch works as described.
Passes tests and QA script, provided tests fail without patch, but
succeed with the patch.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 7966037747)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
2014-08-13 10:30:02 -04:00
C4 Bug 11258: fix another case where holds queue made transfer requests that contradict the library holds policy 2014-08-13 10:30:02 -04:00
Koha Bug 11779: (follow-up) improve GetLoggedInBranchcode() and add test cases 2014-06-06 07:46:22 -04:00
OpenILS Bug 9239 QA follow-up: remove stray debug code 2013-03-16 21:32:34 -04:00
acqui Bug 7267: Add account number to German PDF template 2014-08-13 09:53:51 -04:00
admin Bug 9114: Make frameworks import/export routines correctly use UTF-8 2014-06-13 08:45:38 -04:00
authorities Bug 10808: (follow-up) reformat auth_finder.pl 2014-01-14 12:11:47 -03:00
basket Bug 10605: fix encoding issue on basket email (INTRANET) 2014-01-14 12:15:00 -03:00
catalogue Bug 11885: (follow-up) remove tabs 2014-08-13 09:47:21 -04:00
cataloguing Bug 11715: require authentication for the cataloging Z39.50 search 2014-06-26 06:31:49 -04:00
circ Bug 10731 - IntranetSlipPrinterJS is not working for hold slips [v3.10/v3.12] 2013-11-11 10:54:03 -03:00
debian Bug 11757: remove dependency on POE 2014-06-06 07:28:18 -04:00
docs Bug 7143: Cleaning up docs/history.txt 2013-12-16 09:43:10 -03:00
errors Housekeeping in errors scripts 2010-05-12 07:29:03 -04:00
etc Bug 10729: Add phrases configuration for ICU 2014-06-06 09:17:32 -04:00
install_misc Bug 11757: remove dependency on POE 2014-06-06 07:28:18 -04:00
installer Increment version for 3.12.14 2014-07-25 12:41:25 -04:00
koha-tmpl Bug 10825: don't display enum/chron twice for items received via the serials module 2014-08-13 10:24:26 -04:00
labels Bug 11222: fix crash that can occur in search for items to add to label batch 2013-12-17 12:10:53 -03:00
members Bug 11184: correct attribute cloning for the patron editor 2014-08-13 09:43:49 -04:00
misc Bug 12081: make tmpl_process3.pl delete ts temp files 2014-08-13 10:26:38 -04:00
offline_circ Bug 8220 - Allow koc uploads to go to process queue instead of being applied directly. 2013-03-21 20:35:37 -04:00
opac Bug 11647: (follow-up) add comment expressing that user must be logged in 2014-06-13 08:42:34 -04:00
patroncards Bug 10527: remove disused routine C4::Branch::get_branch_code_from_name 2013-07-25 12:33:14 -03:00
plugins Bug 7804 - Add Koha Plugin System - QA Followup 2 2013-03-20 14:50:19 -04:00
reports Bug 11715: require authentication for various staff scripts 2014-06-26 06:32:17 -04:00
reserve Bug 10663: QA Followup: Typo in comment in renewscript 2013-08-16 12:46:15 -03:00
reviews Bug 1623 - Provide view of approved comments 2011-12-27 18:26:50 +01:00
rotating_collections Bug 9605: rotating collections permissions are wrong 2013-03-30 22:11:05 -04:00
selenium Adding selenium tests for filterMembers 2009-09-30 11:30:37 +02:00
serials Bug 11715: require authentication for various staff scripts 2014-06-26 06:32:17 -04:00
services Bug 11715: require authentication for various staff scripts 2014-06-26 06:32:17 -04:00
skel Bug 7804 - Add Koha Plugin System 2013-03-20 14:49:47 -04:00
sms Bug 10611: Use mysql_auto_reconnect instead of ping 2014-06-06 07:18:51 -04:00
suggestion Bug 4068: (follow-up) use URI::Escape 2014-08-13 09:05:38 -04:00
svc Bug 11704: Make */svc/report print the correct headers 2014-06-06 07:24:34 -04:00
t Bug 11258: fix another case where holds queue made transfer requests that contradict the library holds policy 2014-08-13 10:30:02 -04:00
tags Bug 10022 - Multiple tags moderation doesn't work on translated page 2013-05-01 06:44:31 -04:00
test Bug 5449: JSON malformed in Koha - Blocker with jQuery 1.4.x 2011-03-12 08:53:41 +13:00
tmp/modified_authorities
…
tools Bug 11493: Revert "Bug 6869 Batch item modification fails whit accented characters" 2014-06-06 07:18:23 -04:00
virtualshelves Bug 7788: [SIGNED-OFF] Followup: GetShelf call in addbybiblio script corrected 2012-05-22 12:06:26 +02:00
xt Bug 12076: better detect an untranslatable template construct 2014-08-13 10:01:11 -04:00
.htaccess Fix file permissions: if it is not a script, it should not be executable. 2010-04-16 00:40:34 -04:00
.mailmap 7439 Mailmap for master 2012-01-27 12:27:58 +01:00
INSTALL Bug 7759, update of install files to use background indexing (and some whitespace tidy) 2012-04-20 16:11:52 +02:00
INSTALL.debian Bug 8092 follow-up: Add optional dependency on CHI 2012-06-09 13:08:18 +02:00
INSTALL.fedora7 Bug 11757: remove dependency on POE 2014-06-06 07:28:18 -04:00
INSTALL.opensuse Bug 11757: remove dependency on POE 2014-06-06 07:28:18 -04:00
INSTALL.ubuntu Bug 7764: (follow-up) editorial tweaks 2013-11-13 16:48:41 -03:00
LICENSE Bug 9440 - update Koha's LICENSE file from GPL2 to GPL3 2013-02-12 08:52:10 -05:00
MANIFEST.SKIP Bug 9546 : Updating make manifest tardist 2013-02-06 23:54:46 -05:00
Makefile.PL Bug 10729: Add phrases configuration for ICU 2014-06-06 09:17:32 -04:00
README Bug 9440 - update Koha's LICENSE file from GPL2 to GPL3 2013-02-12 08:52:10 -05:00
README.robots Bug 6411 add another example to README.robots 2011-07-05 14:48:05 +12:00
about.pl Bug 11772: Show warning on About page:System information if no active currency is defined 2014-06-06 09:20:43 -04:00
changelanguage.pl Bug 6755 Problems with switching languages 2011-09-23 09:47:09 +12:00
edithelp.pl Bug 11661: sanitize file names supplied to edithelp.pl 2014-02-06 09:34:10 -03:00
fix-perl-path.PL installer: improvements to fix-path-perl.PL on Win32 2007-12-20 19:20:12 -06:00
help.pl Bug 8949: Trying to view help on a page without any help results in a nasty error 2012-11-05 08:39:23 -05:00
install-CPAN.pl Bug 5370: Fix all the references to koha.org 2010-11-08 09:41:49 +13:00
koha_perl_deps.pl bug 10548: fix count of missing required dependencies by koha_perl_deps.pl 2013-07-16 14:54:30 -03:00
kohaversion.pl Increment version for 3.12.14 2014-07-25 12:41:25 -04:00
mainpage.pl Revert "Bug 7167: New version for updatedatabase" 2012-12-27 14:02:56 -05:00
rewrite-config.PL Bug 10729: Add phrases configuration for ICU 2014-06-06 09:17:32 -04:00

README

Koha is a free software integrated library system.

Koha is distributed under the GNU GPL version 3 or later.
Please read the file LICENSE for more details.

To install or upgrade Koha, please see the INSTALL file appropriate
to your platform.

Report bugs at http://bugs.koha-community.org/

Visit the Koha Project website at http://www.koha-community.org/