[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com Resolution|--- |FIXED Status|Pushed to oldstable |RESOLVED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 --- Comment #20 from Matt Blenkinsop --- Nice work everyone! Pushed to oldstable for 22.11.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Matt Blenkinsop changed: What|Removed |Added Status|Pushed to stable|Pushed to oldstable Version(s)|23.11.00,23.05.02 |23.11.00,23.05.02,22.11.08 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 --- Comment #19 from Martin Renvoize --- Thanks for all the hard work! Pushed to 23.05.x for the next release -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Martin Renvoize changed: What|Removed |Added Version(s)|23.11.00|23.11.00,23.05.02 released in|| Status|Pushed to master|Pushed to stable -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 --- Comment #18 from Tomás Cohen Arazi --- Pushed to master for 23.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Tomás Cohen Arazi changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||23.11.00 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 --- Comment #17 from Tomás Cohen Arazi --- Created attachment 153293 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153293=edit Bug 33817: (QA follow-up) Fix tidyness This patch fixes some reported tidyness issues with the patches. Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Katrin Fischer changed: What|Removed |Added Attachment #152725|0 |1 is obsolete|| --- Comment #16 from Katrin Fischer --- Created attachment 153230 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153230=edit Bug 33817: (follow-up) Fix spec and messages This patch should resolve the issues highlighted at signoff time by Victor. Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Katrin Fischer changed: What|Removed |Added Attachment #152724|0 |1 is obsolete|| --- Comment #15 from Katrin Fischer --- Created attachment 153229 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153229=edit Bug 33817: Enforce server-side Signed-off-by: Martin Renvoize Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Katrin Fischer changed: What|Removed |Added Attachment #152723|0 |1 is obsolete|| --- Comment #14 from Katrin Fischer --- Created attachment 153228 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153228=edit Bug 33817: Prevent item bundle that is checked out to be modified While an item bundle is checked out, we don't want any items to be added or removed from the bundle. To test: * Create a record with LDR Pos. 7 = c * Add an item for your bundle * Add one or more existing items from other records to the bundle * Check out the bundle item * Verify it appears correctly on the patron account * Verify you can still add/remove items from the bundle * Apply patch set * Verify that the add/remove links are now disabled (greyed out) Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 --- Comment #13 from Katrin Fischer --- ...problem descriptions and test plans in commit messages please? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Katrin Fischer changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |katrin.fisc...@bsz-bw.de |y.org | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 --- Comment #12 from Martin Renvoize --- Created attachment 152725 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=152725=edit Bug 33817: (follow-up) Fix spec and messages This patch should resolve the issues highlighted at signoff time by Victor. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Martin Renvoize changed: What|Removed |Added Attachment #151663|0 |1 is obsolete|| --- Comment #11 from Martin Renvoize --- Created attachment 152724 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=152724=edit Bug 33817: Enforce server-side Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Martin Renvoize changed: What|Removed |Added Attachment #151662|0 |1 is obsolete|| --- Comment #10 from Martin Renvoize --- Created attachment 152723 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=152723=edit Bug 33817: Prevent item bundle that is checked out to be modified Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Victor Grousset/tuxayo changed: What|Removed |Added CC||vic...@tuxayo.net --- Comment #9 from Victor Grousset/tuxayo --- Found this when testing for server side enforcement: 0. apply patches 1. have a bundle 2. have it's page opened and it's manage bundle menu too 3. in another tab, check the bundle out 4. back to the 1st tab, try to add another item to the bundle 5. error: "Failure: Item 'BARCODE' belongs to another bundle" this is false! note: API response: 409 and error bundle_checked_out, nothing wrong here it seems 6. try to remove an item 7. nothing happens, which is okay 8. API response: 501 not implemented, error: No response rule for "409" Is the wrong message UI at step 5 a big deal? Same question for API response at step 8. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Martin Renvoize changed: What|Removed |Added Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Martin Renvoize changed: What|Removed |Added Status|Needs Signoff |Signed Off --- Comment #8 from Martin Renvoize --- All working as originally intended and covered by tests too.. Signing off. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Martin Renvoize changed: What|Removed |Added Attachment #151660|0 |1 is obsolete|| --- Comment #7 from Martin Renvoize --- Created attachment 151663 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=151663=edit Bug 33817: Enforce server-side Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Martin Renvoize changed: What|Removed |Added Attachment #151623|0 |1 is obsolete|| --- Comment #6 from Martin Renvoize --- Created attachment 151662 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=151662=edit Bug 33817: Prevent item bundle that is checked out to be modified Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 --- Comment #5 from Martin Renvoize --- Thanks for your work here Jonathan.., testing not, but the code looks solid :) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 --- Comment #4 from Jonathan Druart --- Created attachment 151660 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=151660=edit Bug 33817: Enforce server-side -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Jonathan Druart changed: What|Removed |Added Status|In Discussion |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com --- Comment #3 from Martin Renvoize --- From IRC we came to the conclusion: * Bundles should not be editable whilst checked out (Removing and Adding should both be blocked at client and server side) * Items should not be available for adding to a bundle when they are individually checked out (Again, this should be blocked at both the client and the server). The real world idea of this was that you physically had a 'box' you were packing your items into to create the bundle.. you scan your items into the box and out of the box.. thus they should physically be there. We then track their presence at the checkin/checkout actions. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Jonathan Druart changed: What|Removed |Added Blocks||33568 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33568 [Bug 33568] Use the REST API to display items on the biblio detail view -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 --- Comment #2 from Jonathan Druart --- Created attachment 151623 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=151623=edit Bug 33817: Prevent item bundle that is checked out to be modified -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Jonathan Druart changed: What|Removed |Added Status|ASSIGNED|In Discussion --- Comment #1 from Jonathan Druart --- From bug 28854 comment 515 (In reply to Martin Renvoize from comment #269) > (In reply to Katrin Fischer from comment #224) > > d) Should we be able to change contents of a bundle while it is checked out? > > At the moment it's possible. Maybe a 'safety question'? > > (question/suggestion) > > Fixed in follow-up. This is not working, there is some code in the template to disable the buttons but it does not work. I have a patch to fix it, but I am wondering if it is correct to disable the edition of a bundle item that is checked out. There is no code server-side to prevent that (and so it's currently possible). Should we fix or remove this restriction? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33817] Composition of an item bundle can be changed if checked out
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33817 Jonathan Druart changed: What|Removed |Added Summary|Composition of an item |Composition of an item |bundle can be changed |bundle can be changed if ||checked out -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
