Summary: | Batch extend due dates should be called Batch modify due dates | ||
---|---|---|---|
Product: | Koha | Reporter: | hebah |
Component: | Tools | Assignee: | 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
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". 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" 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 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" 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. 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. 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). |