Bug 1689 - Circulation Rules doesn't take the default in account
Summary: Circulation Rules doesn't take the default in account
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) normal
Assignee: Chris Cormack
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-12-21 02:46 UTC by Chris Cormack
Modified: 2012-10-25 20:00 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Cormack 2010-05-21 00:33:29 UTC 


---- Reported by Eric.Begin@inLibro.com 2007-12-21 14:46:04 ----

no-zebra (probably reproable on zebra too)

In the circulation rules, if we only set the default branch (*) Total/Default value to 25,5 (all other values or all other branches are empty), we are able to loan a 6th item to a borrower without any warning.

I was expecting to get a Too many issue type of warning.

It looks like some fallback checks are missing.



---- Additional Comments From chris@bigballofwax.co.nz 2007-12-28 22:14:11 ----

It doesnt seem to be checking the number of issues allowed at all. It is using the value for the date check though.

I will go to check CanBookBeIssued now



---- Additional Comments From chris@bigballofwax.co.nz 2007-12-28 22:38:04 ----

fixed and patch sent, will resolve when accepted



--- Bug imported by chris@bigballofwax.co.nz 2010-05-21 00:33 UTC  ---

This bug was previously known as _bug_ 1689 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=1689

Unknown operating system Linux - Other. Setting to default OS "All".
Actual time not defined. Setting to 0.0
The original reporter of this bug does not have
   an account here. Reassigning to the person who moved
   it here: chris@bigballofwax.co.nz.
   Previous reporter was Eric.Begin@inLibro.com.