To recreate: 1. Go to circ/circulation.pl and check an item out to a patron. 2. Backdate it to sometime in the past using the "Specify due date (MM/DD/YYYY) : " input 3. A modal appears to "Please confirm checkout". The date is no longer populated in that field. 4. You must add the date again here. This is due to Bug 18885.
This highlights the fact that there are multiple '#duedatespec' on the same page.
Created attachment 167124 [details] [review] Bug 36459: Do not set an empty flatpickr explicitly To test; 1. Go to circ/circulation.pl and check an item out to a patron. 2. Backdate it to sometime in the past using the "Specify due date (MM/DD/YYYY) : " input 3. A modal appears to "Please confirm checkout". The date is no longer populated in that field. 4. You must add the date again here. 5. APPLY PATCH 6. Try again and this time the confimration date should be set correctly. 7. Follow the test plan from Bug 18885 to make sure on site checkouts still work correctly.
(In reply to Lucas Gass from comment #1) > This highlights the fact that there are multiple '#duedatespec' on the same > page. I didn't address this with my patch but we should do so.
Created attachment 167127 [details] [review] Bug 36459: Do not set an empty flatpickr explicitly To test; 1. Go to circ/circulation.pl and check an item out to a patron. 2. Backdate it to sometime in the past using the "Specify due date (MM/DD/YYYY) : " input 3. A modal appears to "Please confirm checkout". The date is no longer populated in that field. 4. You must add the date again here. 5. APPLY PATCH 6. Try again and this time the confimration date should be set correctly. 7. Follow the test plan from Bug 18885 to make sure on site checkouts still work correctly. Signed-off-by: David Nind <david@davidnind.com>
This does break 18885, I'm afraid: 1. Have OnSiteCheckouts enabled and OnSiteAutoCheck disabled 2. Go to circulation.pl 3. Check the checkbox for on-site checkout --> The checkout date is auto-populated with today's date 4. Check out an item --> When the page reloads, the checkbox is no longer checked, but the checkout date is still populated In the use case you're aiming to fix here, though, on-site checkouts was never checked in the first place...so why is #duedatespec being cleared at all? I think there's something deeper going on here.
(In reply to Emily Lamancusa from comment #5) > This does break 18885, I'm afraid: > > 1. Have OnSiteCheckouts enabled and OnSiteAutoCheck disabled > 2. Go to circulation.pl > 3. Check the checkbox for on-site checkout > --> The checkout date is auto-populated with today's date > 4. Check out an item > --> When the page reloads, the checkbox is no longer checked, but the > checkout date is still populated > > In the use case you're aiming to fix here, though, on-site checkouts was > never checked in the first place...so why is #duedatespec being cleared at > all? I think there's something deeper going on here. Are you sure? In my testing steps 1-4 only work when my patch from this bug is applied. Without this patch: -I check the on-site checkbox, today's date is populated. -I checkout an item, both the checkbox and the due date is cleared With my patch: -I check the on-site checkbox, today's date is populated. -I checkout an item, the checkbox is cleared but the date remains.
Created attachment 167676 [details] [review] Bug 36459: Do not duplicate duedatespec IDs To test; 1. Go to circ/circulation.pl and check an item out to a patron. 2. Backdate it to sometime in the past using the "Specify due date (MM/DD/YYYY) : " input 3. A modal appears to "Please confirm checkout". The date is no longer populated in that field. 4. You must add the date again here. 5. APPLY PATCH 6. Try again and this time the confimration date should be set correctly. 7. Follow the test plan from Bug 18885 to make sure on site checkouts still work correctly.
Completely new patch, so resetting to NSO.
Created attachment 167684 [details] [review] Bug 36459: Do not duplicate duedatespec IDs To test; 1. Go to circ/circulation.pl and check an item out to a patron. 2. Backdate it to sometime in the past using the "Specify due date (MM/DD/YYYY) : " input 3. A modal appears to "Please confirm checkout". The date is no longer populated in that field. 4. You must add the date again here. 5. APPLY PATCH 6. Try again and this time the confimration date should be set correctly. 7. Follow the test plan from Bug 18885 to make sure on site checkouts still work correctly. Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 167693 [details] [review] Bug 36459: Do not duplicate duedatespec IDs To test; 1. Go to circ/circulation.pl and check an item out to a patron. 2. Backdate it to sometime in the past using the "Specify due date (MM/DD/YYYY) : " input 3. A modal appears to "Please confirm checkout". The date is no longer populated in that field. 4. You must add the date again here. 5. APPLY PATCH 6. Try again and this time the confimration date should be set correctly. 7. Follow the test plan from Bug 18885 to make sure on site checkouts still work correctly. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Thanks for the new patch and signoff! Looks good and works as expected. Passing QA
Thanks for all the hard work! Pushed to main for the next 24.11.00 release as RM Assistant