Bug 38687

Summary: Batch extend due dates should be called Batch modify due dates
Product: Koha Reporter: hebah
Component: ToolsAssignee: Nicolas Hunstein <nicolas.hunstein>
Status: Failed QA --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: aude.charillon, caroline.cyr-la-rose, janne.seppanen, laura
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: String patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 38687: update "batch modify due dates" and description "Modify the due date checkouts in batch"
Bug 38687: update "batch modify due dates" and description "Modify the due date checkouts in batch"

Description hebah 2024-12-12 15:27:22 UTC
The batch extend due dates tool can be used not just to extend due dates but to set earlier ones. Calling it 'batch modify due dates' better describes its function and makes it more consistent with other tool names.
Comment 1 Aude Charillon 2025-01-13 16:35:53 UTC
I would support this change of name.

The description for the tool on the Tools page should also be updated to something clearer. At the moment, the description is: "Modify the due date checkouts in batch". I would suggest replacing it by: "Modify checkout due dates in batch".
Comment 2 Nicolas Hunstein 2025-01-16 12:15:02 UTC
Created attachment 176651 [details] [review]
Bug 38687: update "batch modify due dates" and description "Modify the due date checkouts in batch"

to test:
- go to Tool
- check designation and description of "batch modify due date"
- verify designation ist "batch modify due date" and description is "Modify the due date checkouts in batch"

- apply patch
- verify instead of "batch modify due date" it is now "Batch modify due dates" and instead of "Modify the due date checkouts in batch" it is "Modify checkout due dates in batch"
Comment 3 Janne Seppänen 2025-02-21 12:38:36 UTC
Tried testing on the ByWater Solutions sandbox, but applying the patch ends up in error:

Bug 38687 - Batch extend due dates should be called Batch modify due dates

176651 - Bug 38687: update "batch modify due dates" and description "Modify the due date checkouts in batch"

Apply? [(y)es, (n)o, (i)nteractive] Applying: Bug 38687: update "batch modify due dates" and description "Modify the due date checkouts in batch"
Using index info to reconstruct a base tree...
M	koha-tmpl/intranet-tmpl/prog/en/includes/permissions.inc
M	koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc
M	koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_extend_due_dates.tt
M	koha-tmpl/intranet-tmpl/prog/en/modules/tools/tools-home.tt
Falling back to patching base and 3-way merge...
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/tools/tools-home.tt
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/tools/tools-home.tt
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_extend_due_dates.tt
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_extend_due_dates.tt
Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc
Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/permissions.inc
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/includes/permissions.inc
error: Failed to merge in the changes.
Patch failed at 0001 Bug 38687: update "batch modify due dates" and description "Modify the due date checkouts in batch"
[33mhint: Use 'git am --show-current-patch=diff' to see the failed patch[m
When you have resolved this problem run "git bz apply --continue".
If you would prefer to skip this patch, instead run "git bz apply --skip".
To restore the original branch and stop patching run "git bz apply --abort".
Patch left in /tmp/Bug-38687-update-batch-modify-due-dates-and-descri-55dxiw_1.patch
Comment 4 Owen Leonard 2025-02-24 15:39:45 UTC
Created attachment 178580 [details] [review]
Bug 38687: update "batch modify due dates" and description "Modify the due date checkouts in batch"

to test:
- go to Tool
- check designation and description of "batch modify due date"
- verify designation ist "batch modify due date" and description is
  "Modify the due date checkouts in batch"

- apply patch
- verify instead of "batch modify due date" it is now "Batch modify due
  dates" and instead of "Modify the due date checkouts in batch" it is
  "Modify checkout due dates in batch"
Comment 5 Caroline Cyr La Rose 2025-03-31 12:11:53 UTC
When I apply the patch, the tool disappears. I think it has to do with the permission name change. But I'm not sure what I'm doing wrong, or what I should be doing...

I did git bz apply and then I did restart_all on my ktd. I also did ./misc/installer/data/updatedatabase.pl inside ktd shell.
Comment 6 Janne Seppänen 2025-04-04 10:23:27 UTC
Tested this in the BibLibre sandbox, and the patch installs without errors. However I'm getting the same issue as Caroline, the tool has now disappeared entirely from the Tools list.
Comment 7 Katrin Fischer 2025-04-06 11:23:42 UTC
This will require a database update for the change of the permission name.
There might also be some other changes required to make this fully work (I wonder if we have checks on the permission in .pl files as well).