To recreate: 1. Go to Item types administration. 2. Set some items types to be limited to a branch you are not logged in as. 3. Find some items from the branch you are logged in at and go to Batch item modification. 4. Add the barcodes and attempt to edit the item type to the one you limited in step 2. 5. You can change these item types to ones that your branch should not be able to.
Created attachment 154660 [details] [review] Bug 34257: Limit item types by branch in batchMod.pl To test: 1. Go to Item types administration. 2. Set some items types to be limited to a branch you are not logged in as. 3. Find some items from the branch you are logged in at and go to Batch item modification. 4. Add the barcodes and attempt to edit the item type to the one you limited in step 2. 5. You can change these item types to ones that your branch should not be able to. 6. Apply patch 7. Try steps 4 -5 again but this time you should not see item types in the dropdown that are limited to other branches.
Created attachment 154686 [details] [review] Bug 34257: Limit item types by branch in batchMod.pl To test: 1. Go to Item types administration. 2. Set some items types to be limited to a branch you are not logged in as. 3. Find some items from the branch you are logged in at and go to Batch item modification. 4. Add the barcodes and attempt to edit the item type to the one you limited in step 2. 5. You can change these item types to ones that your branch should not be able to. 6. Apply patch 7. Try steps 4 -5 again but this time you should not see item types in the dropdown that are limited to other branches. Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Created attachment 155182 [details] [review] Bug 34257: Limit item types by branch in batchMod.pl To test: 1. Go to Item types administration. 2. Set some items types to be limited to a branch you are not logged in as. 3. Find some items from the branch you are logged in at and go to Batch item modification. 4. Add the barcodes and attempt to edit the item type to the one you limited in step 2. 5. You can change these item types to ones that your branch should not be able to. 6. Apply patch 7. Try steps 4 -5 again but this time you should not see item types in the dropdown that are limited to other branches. Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Created attachment 155183 [details] [review] Bug 34257: (QA follow-up): tidy up code
Works, makes sense, QA script happy, code looks good :) Unless Katrin wants to look at something in particular and finds an issue this should be good to go. I'll ping them about it.
(In reply to Victor Grousset/tuxayo from comment #5) > Works, makes sense, QA script happy, code looks good :) > > Unless Katrin wants to look at something in particular and finds an issue > this should be good to go. I'll ping them about it. I think I just got distracted - sorry for that. Updating QA contact to Victor, please Pass QA.
Alright, let's go :)
Pushed to master for 23.11. Nice work everyone, thanks!
Pushed to 23.05.x for 23.05.04
Nice work everyone! Pushed to oldstable for 22.11.x