If you do a batch checkout and the patron is blocked for some reason it shows you a list of the barcodes you scanned, but not warning or confirmation that none of the items were checked out.
Created attachment 44212 [details] warning? This image shows that the patron owes money but it doesn't say that it won't be checking out - there are checkboxes next to all of the ones with error. Then when you confirm it shows you the same titles again but no title on the page to indicate that they did not in fact checkout
Here is a video of what I'm seeing as a big usability issue: http://screencast.com/t/lx6C2zKPxd
Created attachment 44340 [details] [review] Bug 15091: Batch Checkout: Set titles above result tables This patch is to clarify the results that are displayed while performing a batch checkout. It is not the full solution for what is described in the video in comment #2, but it tells the user what the information in the table is about. A second patch is planned to address the case where the issues are confirmed but not can be fulfilled because the user's fines reach the threshold while checking out the items.
Created attachment 44343 [details] [review] Bug 15091: Batch Checkout: Tell the user what happens while checking out This patch slightly changes the behaviour of Batch checkout. - If an item is successfully checked out, it is marked as such in column "Information" - If checkouts are confirmed and an item can not be checked out because e.g. the user reaches the fines threshold, an information is displayed in column "Information" To test: - Watch video from comment #2 - Apply both patches - Set sysprefs to allow override while checking out - Prepare a batch with invalid barcodes, with valid barcodes and with enough items that the user will reach the threshold while checking out all items - Verify that a title appears above the tables while performing the batch checkout - The first table "Batch checkout confirmation" should show - Items that could be checked out - Items that are not checked out (e.g. invalid barcode) - Items that need confirmation (e.g. debt) - After confirmation, more items should be checked out (as long the threshold is not reached - Items above the threshold will have a message "The patron has a debt..." Maybe a more sophisticated solution could be possible, but this patch should address the main concerns from comment #2
Created attachment 44344 [details] [review] Bug 15091: Batch Checkout: Tell the user what happens while checking out This patch slightly changes the behaviour of Batch checkout. - If an item is successfully checked out, it is marked as such in column "Information" - If checkouts are confirmed and an item can not be checked out because e.g. the user reaches the fines threshold, an information is displayed in column "Information" To test: - Watch video from comment #2 - Apply both patches - Set sysprefs to allow override while checking out - Prepare a batch with invalid barcodes, with valid barcodes and with enough items that the user will reach the threshold while checking out all items - Verify that a title appears above the tables while performing the batch checkout - The first table "Batch checkout confirmation" should show - Items that could be checked out - Items that are not checked out (e.g. invalid barcode) - Items that need confirmation (e.g. debt) - After confirmation, more items should be checked out (as long the threshold is not reached - Items above the threshold will have a message "The patron has a debt..." Maybe a more sophisticated solution could be possible, but this patch should address the main concerns from comment #2
Created attachment 44345 [details] [review] Bug 15091: Batch Checkout: Set titles above result tables This patch is to clarify the results that are displayed while performing a batch checkout. It is not the full solution for what is described in the video in comment #2, but it tells the user what the information in the table is about. A second patch is planned to address the case where the issues are confirmed but not can be fulfilled because the user's fines reach the threshold while checking out the items. Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Created attachment 44346 [details] [review] Bug 15091: Batch Checkout: Tell the user what happens while checking out This patch slightly changes the behaviour of Batch checkout. - If an item is successfully checked out, it is marked as such in column "Information" - If checkouts are confirmed and an item can not be checked out because e.g. the user reaches the fines threshold, an information is displayed in column "Information" To test: - Watch video from comment #2 - Apply both patches - Set sysprefs to allow override while checking out - Prepare a batch with invalid barcodes, with valid barcodes and with enough items that the user will reach the threshold while checking out all items - Verify that a title appears above the tables while performing the batch checkout - The first table "Batch checkout confirmation" should show - Items that could be checked out - Items that are not checked out (e.g. invalid barcode) - Items that need confirmation (e.g. debt) - After confirmation, more items should be checked out (as long the threshold is not reached - Items above the threshold will have a message "The patron has a debt..." Maybe a more sophisticated solution could be possible, but this patch should address the main concerns from comment #2 Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
I don't think the second patch is the right way to fix this problem. It seems that the complete tool should be blocked if the patron is not able to check out. I will try to provide a counter patch.
Created attachment 44465 [details] [review] Bug 15091: Do not allow to use batch checkout if checkout is not allowed If the noissue param is passed to the template, the batch checkout should not be allowed. This patch globally fixes the problem raised by bug 15091. A generic message "cannot check out" is displayed and some more specific ones could be added later. For now it only trigger the "fine balance is over the limit" message. Test plan: Confirm there is no regression on the original behavior and that the issue raised on bug 15091 is fixed.
Marc and Nicole, Could you please test this patch, I think it's a global fix for this issue.
Jonathan, I tested yout counter patch with following setting: a) Set loan fee for an item to 5.01 b) Set threshold to 10 c) Do a batch checkout (6 items) for a patron with 0 fines d) The 'Confirm checkout' dialog shows first item with no Information, following intems with information "The patron has a debt of 5.01" e) Click 'Checkout or renew' f) Message appears: "Cannot check out! Checkouts are BLOCKED because fine balance is OVER THE LIMIT." g) Go to Checkout page. 2 of the 6 items are checked out, charges now are on 10.02 This is almost the same confusing situation as described and screencasted by Nicole. With the patch I proposed and signed-off by Nicole, the user sees what happens.
Created attachment 44491 [details] [review] Bug 15091: Do not allow to use batch checkout if checkout is not allowed If the noissue param is passed to the template, the batch checkout should not be allowed. This patch globally fixes the problem raised by bug 15091. A generic message "cannot check out" is displayed and some more specific ones could be added later. For now it only trigger the "fine balance is over the limit" message. Test plan: Confirm there is no regression on the original behavior and that the issue raised on bug 15091 is fixed.
Created attachment 44492 [details] [review] Bug 15091: DEBT is IMPOSSIBLE, not NEEDSCONFIRMATION
Could you please try again with the last patches?
Hi Jonathan, If the patron is over the limit the message appears now as appropriate when going to tab "Batch checkout". One thing remains: There is no info in the information column about items that sucsessfully were checked out (like it was in my patch "Batch Checkout: Tell the user what happens while checking out"). The column "Information" now remains empty for sucessfully checked out items. See discussion with Nicole on IRC around: http://irc.koha-community.org/koha/2015-11-03#i_1750313
(In reply to Marc Véron from comment #15) > Hi Jonathan, > > If the patron is over the limit the message appears now as appropriate when > going to tab "Batch checkout". > > One thing remains: There is no info in the information column about items > that > sucsessfully were checked out (like it was in my patch "Batch Checkout: Tell > the user what happens while checking out"). The column "Information" now > remains empty for sucessfully checked out items. > > See discussion with Nicole on IRC around: > http://irc.koha-community.org/koha/2015-11-03#i_1750313 That is the expected behavior: if the item has been checked out, nothing in the information column. But Nicole and you agree that it's better, we could add it.
I feel there are many improvements we could make here, but as it stands it's usable whereas before it was way too unclear. I'm okay with it going in this way and then us doing some improvements on look and feel and notifications. Nicole
Created attachment 44958 [details] [review] Bug 15091: Do not allow to use batch checkout if checkout is not allowed If the noissue param is passed to the template, the batch checkout should not be allowed. This patch globally fixes the problem raised by bug 15091. A generic message "cannot check out" is displayed and some more specific ones could be added later. For now it only trigger the "fine balance is over the limit" message. Test plan: Confirm there is no regression on the original behavior and that the issue raised on bug 15091 is fixed. Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 44959 [details] [review] Bug 15091: DEBT is IMPOSSIBLE, not NEEDSCONFIRMATION Signed-off-by: Marc Véron <veron@veron.ch>
Set Jonathan as assibnee because he made patches #2 and #3
I think there is a room for improvement here - I'd think to be really clear a positive messsage "checked out" and a negative message "not checked out" would really make things easier.
I will pass this as a partial improvement - please keep the patches coming (on new bugs of course :)) !
Created attachment 44981 [details] [review] [PASSED QA] Bug 15091: Batch Checkout: Set titles above result tables This patch is to clarify the results that are displayed while performing a batch checkout. It is not the full solution for what is described in the video in comment #2, but it tells the user what the information in the table is about. A second patch is planned to address the case where the issues are confirmed but not can be fulfilled because the user's fines reach the threshold while checking out the items. Signed-off-by: Nicole Engard <nengard@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 44982 [details] [review] [PASSED QA] Bug 15091: Do not allow to use batch checkout if checkout is not allowed If the noissue param is passed to the template, the batch checkout should not be allowed. This patch globally fixes the problem raised by bug 15091. A generic message "cannot check out" is displayed and some more specific ones could be added later. For now it only trigger the "fine balance is over the limit" message. Test plan: Confirm there is no regression on the original behavior and that the issue raised on bug 15091 is fixed. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 44983 [details] [review] [PASSED QA] Bug 15091: DEBT is IMPOSSIBLE, not NEEDSCONFIRMATION Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Patches pushed to master. Thanks Marc and Jonathan!