Bug 39987 - Batch item deletion breadcrumb uses wrong link
Summary: Batch item deletion breadcrumb uses wrong link
Status: Pushed to oldoldstable
Alias: None
Product: Koha
Classification: Unclassified
Component: Staff interface (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Nick Clemens (kidclamp)
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-05-23 18:30 UTC by Nick Clemens (kidclamp)
Modified: 2025-07-07 15:32 UTC (History)
5 users (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
This fixes the 'Batch item deletion' breadcrumb link when batch deleting items in cataloguing. If you clicked on the link, it would incorrectly take you to the 'Batch item modification' page.
Version(s) released in:
25.11.00,25.05.01,24.11.06
Circulation function:


Attachments
Bug 39987: Correct breadcrumbs link to batch item deletion (1.48 KB, patch)
2025-05-23 18:53 UTC, Nick Clemens (kidclamp)
Details | Diff | Splinter Review
Bug 39987: Correct breadcrumbs link to batch item deletion (1.53 KB, patch)
2025-05-23 19:14 UTC, Roman Dolny
Details | Diff | Splinter Review
Bug 39987: Correct breadcrumbs link to batch item deletion (1.59 KB, patch)
2025-05-28 09:31 UTC, Baptiste Wojtkowski (bwoj)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens (kidclamp) 2025-05-23 18:30:29 UTC
Currently it points to:
http://localhost:8081/cgi-bin/koha/tools/batchMod.pl

Should be:
http://localhost:8081/cgi-bin/koha/tools/batchMod.pl?del=1
Comment 1 Nick Clemens (kidclamp) 2025-05-23 18:53:41 UTC
Created attachment 182777 [details] [review]
Bug 39987: Correct breadcrumbs link to batch item deletion

This patch adds th del=1 param to the batchMod link in the breadcrumbs to ensure we return to batch item
deletion and not modification

To test:
1 - Browse to Cataloging->Batch item deletion
2 - Click the 'Batch item deletion' link in the breadcrumbs
3 - You are now at 'Batch item modification'
4 - Apply patch
5 - Repeat, you stay on the correct page
6 - Complete a batch item deletion and confirm the breadcrum is correct at each stage.
Comment 2 Roman Dolny 2025-05-23 19:14:19 UTC
Created attachment 182778 [details] [review]
Bug 39987: Correct breadcrumbs link to batch item deletion

This patch adds th del=1 param to the batchMod link in the breadcrumbs to ensure we return to batch item
deletion and not modification

To test:
1 - Browse to Cataloging->Batch item deletion
2 - Click the 'Batch item deletion' link in the breadcrumbs
3 - You are now at 'Batch item modification'
4 - Apply patch
5 - Repeat, you stay on the correct page
6 - Complete a batch item deletion and confirm the breadcrum is correct at each stage.

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Comment 3 Baptiste Wojtkowski (bwoj) 2025-05-28 09:31:25 UTC
Created attachment 182811 [details] [review]
Bug 39987: Correct breadcrumbs link to batch item deletion

This patch adds th del=1 param to the batchMod link in the breadcrumbs to ensure we return to batch item
deletion and not modification

To test:
1 - Browse to Cataloging->Batch item deletion
2 - Click the 'Batch item deletion' link in the breadcrumbs
3 - You are now at 'Batch item modification'
4 - Apply patch
5 - Repeat, you stay on the correct page
6 - Complete a batch item deletion and confirm the breadcrum is correct at each stage.

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Signed-off-by: Baptiste Wojtkowski <baptiste.wojtkowski@biblibre.com>
Comment 4 Lucas Gass (lukeg) 2025-06-02 20:59:25 UTC
Nice work everyone!

Pushed to main for 25.11
Comment 5 Fridolin Somers 2025-06-19 07:27:46 UTC
I see in 25.05.x
Comment 6 Fridolin Somers 2025-06-19 07:28:27 UTC
Pushed to 24.11.x for 24.11.06
Comment 7 Paul Derscheid 2025-06-19 09:02:03 UTC
Nice work everyone!

Pushed to 25.05.x for 25.05.03
Comment 8 Jesse Maseto 2025-07-07 15:32:43 UTC
This will be pushed to 24.05.x for 24.05.12