The problem is that "...bundled_items/item/{bundled_item_id}" gives 404, should just be "bundled_items/{bundled_item_id}".
Created attachment 188019 [details] [review] Bug 41035: remove invalid path component 'item' To test: 1. Create a record as a collection with leader 007 set to c 2. Add some items on the same branch to the bundle 3. Check one item in 4. Observe that you get a 404 error and the item is not removed from the bundle 5. Apply the patch 6. Repeat step 3 7. Observe that the item is correctly removed (unlinked) from the bundle 8. Sign off
Created attachment 190652 [details] [review] Bug 41035: remove invalid path component 'item' To test: 1. Create a record as a collection with leader 007 set to c 2. Add some items on the same branch to the bundle 3. Check one item in 4. Observe that you get a 404 error and the item is not removed from the bundle 5. Apply the patch 6. Repeat step 3 7. Observe that the item is correctly removed (unlinked) from the bundle 8. Sign off Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Created attachment 192192 [details] [review] Bug 41035: remove invalid path component 'item' To test: 1. Create a record as a collection with leader 007 set to c 2. Add some items on the same branch to the bundle 3. Check one item in 4. Observe that you get a 404 error and the item is not removed from the bundle 5. Apply the patch 6. Repeat step 3 7. Observe that the item is correctly removed (unlinked) from the bundle 8. Sign off Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
This works and is consistent with how the items API defines the endpoint for remove from bundle: "/items/{item_id}/bundled_items/{bundled_item_id}": delete: x-mojo-to: Items#remove_from_bundle operationId: removeFromBundle tags: - items summary: Remove item from bundle Nice work. Passing QA
Testing note: You can use the following leader to create the record: 00000npc a2200000 4500
Nice work everyone! Pushed to main for 26.05