When the force_checkout permission was added, the script circulation.pl was altered to allow overriding any block on the patron's account (expired, gonenoaddress, etc) The last follow-up on that bug, however, moved the button to override into the section for restrictions, limiting the scope for users to the case where a patron had a debarment. To recreate: 1 - Log in to Koha with a user with force_checkout permission (superlibrarian ok too) 2 - Mark a patron as expired, lost card, gone no address 3 - Go to 'Check out' page for the patorn 4 - Note you cannot override the restrictions 5 - Add a manual restriction/debarment 6 - Go to checkout again, now you can override 7 - Remove the debarment 8 - Add manually ot URL of checkout page '&foreallow=1' 9 - Note you can now override the other restrictions I think the intent here should be that 'force_checkout' allows overriding any of the statuses
Created attachment 158062 [details] [review] Bug 35188: Show checkout override for all noisseus statuses This patch moves the override button up one level into the list of noissues in order to allow overriding any blocking status To test: 1 - Log in to Koha with a user with force_checkout permission (superlibrarian ok too) 2 - Mark a patron as expired, lost card, gone no address 3 - Go to 'Check out' page for the patorn 4 - Note you cannot override the restrictions 5 - Add a manual restriction/debarment 6 - Go to checkout again, now you can override 7 - Remove the debarment 8 - Add manually ot URL of checkout page '&foreallow=1' 9 - Note you can now override the other restrictions 10 - Apply patch 11 - Reload patron 12 - Note you now see the override button on the check out screen 13 - Add a debarment 14 - Confirm the button is still there 15 - Confirm overriding allows the checkout (with or without a debarment)
*** Bug 34588 has been marked as a duplicate of this bug. ***
Created attachment 158271 [details] [review] Bug 35188: Show checkout override for all noisseus statuses This patch moves the override button up one level into the list of noissues in order to allow overriding any blocking status To test: 1 - Log in to Koha with a user with force_checkout permission (superlibrarian ok too) 2 - Mark a patron as expired, lost card, gone no address 3 - Go to 'Check out' page for the patorn 4 - Note you cannot override the restrictions 5 - Add a manual restriction/debarment 6 - Go to checkout again, now you can override 7 - Remove the debarment 8 - Add manually ot URL of checkout page '&foreallow=1' 9 - Note you can now override the other restrictions 10 - Apply patch 11 - Reload patron 12 - Note you now see the override button on the check out screen 13 - Add a debarment 14 - Confirm the button is still there 15 - Confirm overriding allows the checkout (with or without a debarment) Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Created attachment 158413 [details] [review] Bug 35188: Show checkout override for all noisseus statuses This patch moves the override button up one level into the list of noissues in order to allow overriding any blocking status To test: 1 - Log in to Koha with a user with force_checkout permission (superlibrarian ok too) 2 - Mark a patron as expired, lost card, gone no address 3 - Go to 'Check out' page for the patorn 4 - Note you cannot override the restrictions 5 - Add a manual restriction/debarment 6 - Go to checkout again, now you can override 7 - Remove the debarment 8 - Add manually ot URL of checkout page '&foreallow=1' 9 - Note you can now override the other restrictions 10 - Apply patch 11 - Reload patron 12 - Note you now see the override button on the check out screen 13 - Add a debarment 14 - Confirm the button is still there 15 - Confirm overriding allows the checkout (with or without a debarment) Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Works, makes sense, QA script happy, code looks good, passing QA :)
Pushed to master for 23.11. Nice work everyone, thanks!
Does not apply on 23.05.x Please provide rebased patch
Created attachment 158803 [details] [review] Bug 35188: [23.05.x] Show checkout override for all noissues statuses This patch moves the override button up one level into the list of noissues in order to allow overriding any blocking status To test: 1 - Log in to Koha with a user with force_checkout permission (superlibrarian ok too) 2 - Mark a patron as expired, lost card, gone no address 3 - Go to 'Check out' page for the patorn 4 - Note you cannot override the restrictions 5 - Add a manual restriction/debarment 6 - Go to checkout again, now you can override 7 - Remove the debarment 8 - Add manually ot URL of checkout page '&foreallow=1' 9 - Note you can now override the other restrictions 10 - Apply patch 11 - Reload patron 12 - Note you now see the override button on the check out screen 13 - Add a debarment 14 - Confirm the button is still there 15 - Confirm overriding allows the checkout (with or without a debarment) Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>