This problem is fixed in main by Bug 35402. ( commit 5fe1ad8ac66072ef7e06d21d0ccc48d388ecaf79 ) - <button type="button" data-dismiss="modal" class="btn btn-default" accesskey="y"><i class="fa fa-check"></i> OK (Y)</button> - <button type="button" data-dismiss="modal" class="btn btn-default openWin" data-url="transfer-slip.pl?transferitem=[% itemnumber | uri %]&branchcode=[% transfer | uri %]&op=slip" accesskey="p"><i class="fa fa-print"></i> Print slip (P)</button> + <button type="button" data-bs-dismiss="modal" class="btn btn-default" accesskey="y"><i class="fa fa-check"></i> OK (Y)</button> + <button type="button" data-bs-dismiss="modal" class="btn btn-default print openWin" data-url="transfer-slip.pl?transferitem=[% itemnumber | uri %]&branchcode=[% transfer | uri %]&op=slip" accesskey="p"><i class="fa fa-print"></i> Print slip (P)</button> We need the same fix in 24.05.x and below.
This fix we just need to add the print class, I think.
Created attachment 172937 [details] [review] Bug 38199: [24.05.x] Add back missing print class To test: 1. Find an item to checkout that doesn't belong to the branch you are logged in at. 2. Check the item out. 3. Go to Circulation > Check in 4. Using the check in input in the main area ("Check in"), check the item in 5. A modal should appear, "Please return this item to {branch}" 6. Click "Print slip" 7. Deal with the browsers print dialouge. ( print, cancel, or save should work for testing) 8. After the modal closes focus is not brought back to the barcode field. 9. Check the item in again, this should trigger the "Wrong transfer detected, please return item to: {branch}" modal. 10. Click "Print transfer slip", here the modal closes and properly gives focus to the #barcode input. 11. APPLY PATCH, clear browser cache. 12. Try all the steps again, each time focus should be restored to the #barcode input
Backported to 24.05.x for upcoming 24.05.05
Comes from : Bug 34478: Manual fix - add cud- ops or remove form submit - returns.tt / checkin-search-box.inc Not in 23.11.x