We are unable to remove users from serials routing lists after the 24.05 upgrade. Clicking the delete button does nothing. Possibly related to CSRF changes.
I verified this is present on main as well. While troubleshooting, I also found that the "Save and preview slip" button was not previewing the slip. Also, using the "Print list" button from the Serial collection information screen was display the standard staff interface headers when trying to print the slip. I will have a patch up for this later today.
Created attachment 171278 [details] [review] Bug 37873: Unable to delete user from routing list or preview/print routing list slip This patch restores the ability to delete a recipient from a serials routing list. It also fixes the slip preview when using the "Save and preview slip", and removes the koha staff intranet head from the slip when using 'Print list' from the Serial collection information screen. To test: 1) Login to the staff client. 2) Open the Serials module. 3) + New subscription to add a new serial. 4) Create a new subscription. a) Search for a record (I searched for 'dog') and chose that record. b) Under 'When receiving this serial', select 'Create an item record' c) Under the 'Item type' dropdown, select anything. d) Click Next -> (and confirm the popup). e) Select any 'First issue date'. f) Select any 'Frequency'. g) Enter a number larger than 5 in the 'Subscription length'. h) Select any 'Subscription start date'. i) Select your favorite numbering pattern. j) Verify your prediction pattern using "Test prediction pattern". k) Click "Save subscription". 5) On the 'Subscription details' screen, click "Receive". 6) Under the first volume, click "Add item" then "Save". 7) Repeat step 6 to receive one more issue. 8) On the 'Serial collection information' page click "+ Create routing list" 9) Click "+ Add recipients". 10) Search for and add several recipients. 11) Click "Close" to exit the 'Add recipients' page. 12) Click "Delete" next to any recipient. *Note that the recipient is not deleted. 13) Click "Save" on the 'Modify routing list for ...' page. 14) Click "Save and preview routing slip" button. *Note you are redirected to the Subscription details page, but no routing slip preview appears. 15) Click "Serial collection" from the left-side menu. 16) Click "Print list" on either of the issues you previously received. *Note the routing slip includes the staff intranet header, menu, etc. 17) Apply patch. 18) Return to the 'Serial collection information' screen for your subscription. 19) Click "Edit routing list". 20) Repeat steps 12-16 and verify the issues are fixed. -"Delete" button works to remove a recipient from a list. -"Save and preview routing list" button pops-up a preview before redirecting. -"Print list" pops-up a vanilla routing list to print. Sponsored-by: Westlake Porter Public Library
Created attachment 171292 [details] [review] Bug 37873: Unable to delete user from routing list or preview/print routing list slip This patch restores the ability to delete a recipient from a serials routing list. It also fixes the slip preview when using the "Save and preview slip", and removes the koha staff intranet head from the slip when using 'Print list' from the Serial collection information screen. To test: 1) Login to the staff client. 2) Open the Serials module. 3) + New subscription to add a new serial. 4) Create a new subscription. a) Search for a record (I searched for 'dog') and chose that record. b) Under 'When receiving this serial', select 'Create an item record' c) Under the 'Item type' dropdown, select anything. d) Click Next -> (and confirm the popup). e) Select any 'First issue date'. f) Select any 'Frequency'. g) Enter a number larger than 5 in the 'Subscription length'. h) Select any 'Subscription start date'. i) Select your favorite numbering pattern. j) Verify your prediction pattern using "Test prediction pattern". k) Click "Save subscription". 5) On the 'Subscription details' screen, click "Receive". 6) Under the first volume, click "Add item" then "Save". 7) Repeat step 6 to receive one more issue. 8) On the 'Serial collection information' page click "+ Create routing list" 9) Click "+ Add recipients". 10) Search for and add several recipients. 11) Click "Close" to exit the 'Add recipients' page. 12) Click "Delete" next to any recipient. *Note that the recipient is not deleted. 13) Click "Save" on the 'Modify routing list for ...' page. 14) Click "Save and preview routing slip" button. *Note you are redirected to the Subscription details page, but no routing slip preview appears. 15) Click "Serial collection" from the left-side menu. 16) Click "Print list" on either of the issues you previously received. *Note the routing slip includes the staff intranet header, menu, etc. 17) Apply patch. 18) Return to the 'Serial collection information' screen for your subscription. 19) Click "Edit routing list". 20) Repeat steps 12-16 and verify the issues are fixed. -"Delete" button works to remove a recipient from a list. -"Save and preview routing list" button pops-up a preview before redirecting. -"Print list" pops-up a vanilla routing list to print. Sponsored-by: Westlake Porter Public Library Signed-off-by: Olivier V <olivier.vezina@inLibro.com>
Created attachment 171446 [details] [review] Bug 37873: Unable to delete user from routing list or preview/print routing list slip This patch restores the ability to delete a recipient from a serials routing list. It also fixes the slip preview when using the "Save and preview slip", and removes the koha staff intranet head from the slip when using 'Print list' from the Serial collection information screen. To test: 1) Login to the staff client. 2) Open the Serials module. 3) + New subscription to add a new serial. 4) Create a new subscription. a) Search for a record (I searched for 'dog') and chose that record. b) Under 'When receiving this serial', select 'Create an item record' c) Under the 'Item type' dropdown, select anything. d) Click Next -> (and confirm the popup). e) Select any 'First issue date'. f) Select any 'Frequency'. g) Enter a number larger than 5 in the 'Subscription length'. h) Select any 'Subscription start date'. i) Select your favorite numbering pattern. j) Verify your prediction pattern using "Test prediction pattern". k) Click "Save subscription". 5) On the 'Subscription details' screen, click "Receive". 6) Under the first volume, click "Add item" then "Save". 7) Repeat step 6 to receive one more issue. 8) On the 'Serial collection information' page click "+ Create routing list" 9) Click "+ Add recipients". 10) Search for and add several recipients. 11) Click "Close" to exit the 'Add recipients' page. 12) Click "Delete" next to any recipient. *Note that the recipient is not deleted. 13) Click "Save" on the 'Modify routing list for ...' page. 14) Click "Save and preview routing slip" button. *Note you are redirected to the Subscription details page, but no routing slip preview appears. 15) Click "Serial collection" from the left-side menu. 16) Click "Print list" on either of the issues you previously received. *Note the routing slip includes the staff intranet header, menu, etc. 17) Apply patch. 18) Return to the 'Serial collection information' screen for your subscription. 19) Click "Edit routing list". 20) Repeat steps 12-16 and verify the issues are fixed. -"Delete" button works to remove a recipient from a list. -"Save and preview routing list" button pops-up a preview before redirecting. -"Print list" pops-up a vanilla routing list to print. Sponsored-by: Westlake Porter Public Library Signed-off-by: Olivier V <olivier.vezina@inLibro.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Thanks for the fix here.. Passing QA
Pushed for 24.11! Well done everyone, thank you!
Doesn't apply cleanly to 24.05.x, please rebase
Created attachment 171944 [details] [review] Bug 37873: [24.05.x] Unable to delete user from routing list or preview/print routing list slip This patch restores the ability to delete a recipient from a serials routing list. It also fixes the slip preview when using the "Save and preview slip", and removes the koha staff intranet head from the slip when using 'Print list' from the Serial collection information screen. To test: 1) Login to the staff client. 2) Open the Serials module. 3) + New subscription to add a new serial. 4) Create a new subscription. a) Search for a record (I searched for 'dog') and chose that record. b) Under 'When receiving this serial', select 'Create an item record' c) Under the 'Item type' dropdown, select anything. d) Click Next -> (and confirm the popup). e) Select any 'First issue date'. f) Select any 'Frequency'. g) Enter a number larger than 5 in the 'Subscription length'. h) Select any 'Subscription start date'. i) Select your favorite numbering pattern. j) Verify your prediction pattern using "Test prediction pattern". k) Click "Save subscription". 5) On the 'Subscription details' screen, click "Receive". 6) Under the first volume, click "Add item" then "Save". 7) Repeat step 6 to receive one more issue. 8) On the 'Serial collection information' page click "+ Create routing list" 9) Click "+ Add recipients". 10) Search for and add several recipients. 11) Click "Close" to exit the 'Add recipients' page. 12) Click "Delete" next to any recipient. *Note that the recipient is not deleted. 13) Click "Save" on the 'Modify routing list for ...' page. 14) Click "Save and preview routing slip" button. *Note you are redirected to the Subscription details page, but no routing slip preview appears. 15) Click "Serial collection" from the left-side menu. 16) Click "Print list" on either of the issues you previously received. *Note the routing slip includes the staff intranet header, menu, etc. 17) Apply patch. 18) Return to the 'Serial collection information' screen for your subscription. 19) Click "Edit routing list". 20) Repeat steps 12-16 and verify the issues are fixed. -"Delete" button works to remove a recipient from a list. -"Save and preview routing list" button pops-up a preview before redirecting. -"Print list" pops-up a vanilla routing list to print. Sponsored-by: Westlake Porter Public Library
Rebased for 24.05.x
Thanks CJ! Backported to 24.05. for upcoming 24.05.04
CUD not in 23.11.x
Bug fix, no change to UI, no changes required to the documentation.