Summary: | Batch check out fails to inform or trap | ||
---|---|---|---|
Product: | Koha | Reporter: | Christopher Brannon <cbrannon> |
Component: | Circulation | Assignee: | Bugs List <koha-bugs> |
Status: | NEW --- | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | david, gmcharlt, jonathan.druart, koha, kyle.m.hall, martin.renvoize, patrick.robitaille, sbrown |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | 37396 | ||
Bug Blocks: | |||
Attachments: | Bug 23780: Add DEBT info to batch checkouts info |
Description
Christopher Brannon
2019-10-08 16:45:58 UTC
Created attachment 94216 [details] [review] Bug 23780: Add DEBT info to batch checkouts info DEBT can be impossible or needs confirmation. Test plan: Create an invoice of 50 (and make sure 50 is above the limit) Check an item out using the batch checkouts feature => You should see the warning in the "Information" column (In reply to Jonathan Druart from comment #1) > Created attachment 94216 [details] [review] [review] > Bug 23780: Add DEBT info to batch checkouts info > > DEBT can be impossible or needs confirmation. > > Test plan: > Create an invoice of 50 (and make sure 50 is above the limit) > Check an item out using the batch checkouts feature > => You should see the warning in the "Information" column Technically, it works for the fines, but if I have multiple reasons, it only shows one reason and lets you blow past the other unknowingly. For this to work as expected, it needs to show ALL reasons. To test: * Set checkout limit to 5 items. * Set MaxFine to 3 * Set borrower with a manual invoice of $5. * Batch checkout 6 items. You will see information on every item that the patron has a debt of 5.00, but on the 6th item, you should ALSO see "Too many checked out". The problem is other items are not already checked out to trap this. SO, what needs to happen is either A) the batch checkout needs to "Look ahead" at all issues and evaluate all potential blocks, or B) the batch checkout needs to run through the same process again on all the checked items and catch the next block. Process A would be preferable so you don't have to override over and over again through each scenario. I want to sign off, because this is a step in the right direction, but it does not address the multiple blocks addressed in the description of the problem. Ok, got it now.
Actually it's not the "multiple blocks" that is the problem, but the "too many" that cannot be guessed.
For instance if you already have 5 checkouts and over the fine limit, you will get the 2 warnings.
> * Batch checkout 6 items. You will see information on every item that the patron has a debt of 5.00, but on the 6th item, you should ALSO see "Too many checked out".
Until the limit is reached we cannot guess it in advance, because we cannot simulate a checkout. We could add an additional screen for this specific case, but it's much more complex that the first patch I submitted.
Well, either way, if all we are being told about is the fines and not the number of checkouts, overriding the fines completely bypasses other blocks, and this is a HUGE problem. Either Koha is going to have to look ahead at all potential blocks before checkout, or it is going to have to run through the checkout process multiple times. Theoretically, if all 6 items are showing the fines block, and you have all 6 items checked for override, shouldn't it check each item in again and then run into the next block on the 6th item? It seems what is happening is an overall override, rather than a specific block override. The override for the fines is also overriding or bypassing the block for the item limit. I'm wondering if Koha needs to have a smarter override system, in that the override targets a specific block, and is not just a general, ignore everything override. I'd suggest to separate the two issues as one is fixed and the other one is complex and will certainly not be fixed soon. I'm not sure how to trigger this when testing. Before applying the patch, and if I add an invoice for $50 (which is over the limit), I get this message on batch check out screen: Cannot check out! Charges: Patron has outstanding charges of 50.00. Checkouts are BLOCKED because fine balance is OVER THE LIMIT. [Make payment][Pay all fines] Patch still applies. (In reply to David Nind from comment #6) > I'm not sure how to trigger this when testing. > > Before applying the patch, and if I add an invoice for $50 (which is over > the limit), I get this message on batch check out screen: > > Cannot check out! > Charges: Patron has outstanding charges of 50.00. Checkouts are BLOCKED > because fine balance is OVER THE LIMIT. [Make payment][Pay all fines] > > Patch still applies. You are right, it has been fixed by bug 19382. Hm, so should we change the bug title to indicate a part of this has already been fixed? +1 Are any cases still failing here.. or are we in agreement the bug no longer exists? Still exists, see comment 3. *** Bug 37092 has been marked as a duplicate of this bug. *** |