Summary: | Untranslatable "Please make sure all selected titles have a pickup location set" | ||
---|---|---|---|
Product: | Koha | Reporter: | Caroline Cyr La Rose <caroline.cyr-la-rose> |
Component: | I18N/L10N | Assignee: | Owen Leonard <oleonard> |
Status: | Pushed to oldstable --- | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | minor | ||
Priority: | P5 - low | CC: | f.demians, fridolin.somers, jonathan.druart, lucas, m.de.rooy, philippe.blouin |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
24.05.00,23.11.06,23.05.12
|
|
Circulation function: | |||
Bug Depends on: | 28273 | ||
Bug Blocks: | 37017 | ||
Attachments: |
Bug 36872: Untranslatable strings in request.tt
Bug 36872: Untranslatable strings in request.tt Bug 36872: Untranslatable strings in request.tt |
Description
Caroline Cyr La Rose
2024-05-15 18:56:24 UTC
Created attachment 166826 [details] [review] Bug 36872: Untranslatable strings in request.tt There are errors in the JavaScript in the holds template in the staff client which results in strings not being translatable. This patch corrects the errors. To test, apply the patch and run through the process of placing a hold on multiple items in the staff interface. - At the hold confirmation step, try to submit the form without selecting pickup locations for one or more titles. - You should get an alert, "Please make sure all selected titles have a pickup location set". - Uncheck the checkboxes next to each title you're placing a hold on. - Submit the form. You should get an error: "Please select at least one title". - Test the translation process with a language, e.g. fr-FR: - In KTD, run: gulp po:update --lang fr-FR - Check fr-FR-staff-prog.po for the line referring to koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt:1499 - You should see the string "Please make sure all selected titles have a pickup location set". Created attachment 166827 [details] [review] Bug 36872: Untranslatable strings in request.tt There are errors in the JavaScript in the holds template in the staff client which results in strings not being translatable. This patch corrects the errors. To test, apply the patch and run through the process of placing a hold on multiple items in the staff interface. - At the hold confirmation step, try to submit the form without selecting pickup locations for one or more titles. - You should get an alert, "Please make sure all selected titles have a pickup location set". - Uncheck the checkboxes next to each title you're placing a hold on. - Submit the form. You should get an error: "Please select at least one title". - Test the translation process with a language, e.g. fr-FR: - In KTD, run: gulp po:update --lang fr-FR - Check fr-FR-staff-prog.po for the line referring to koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt:1499 - You should see the string "Please make sure all selected titles have a pickup location set". Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Thanks Owen! Created attachment 166881 [details] [review] Bug 36872: Untranslatable strings in request.tt There are errors in the JavaScript in the holds template in the staff client which results in strings not being translatable. This patch corrects the errors. To test, apply the patch and run through the process of placing a hold on multiple items in the staff interface. - At the hold confirmation step, try to submit the form without selecting pickup locations for one or more titles. - You should get an alert, "Please make sure all selected titles have a pickup location set". - Uncheck the checkboxes next to each title you're placing a hold on. - Submit the form. You should get an error: "Please select at least one title". - Test the translation process with a language, e.g. fr-FR: - In KTD, run: gulp po:update --lang fr-FR - Check fr-FR-staff-prog.po for the line referring to koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt:1499 - You should see the string "Please make sure all selected titles have a pickup location set". Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Pushed for 24.05! Well done everyone, thank you! Are those \n needed at all? Looks like this is since Bug 28273 And there are other places : > git grep '" + "\\n"' origin/main origin/main:koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt: msg = _("- Please select a pickup location for the item" + "\n") origin/main:koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt: msg = _("- Please select a pickup location for this hold" + "\n"); origin/main:koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt: msg = _("- Please select a pickup location for this hold" + "\n"); Backported to 23.05.x for upcoming 23.05.12 (In reply to Fridolin Somers from comment #7) > Looks like this is since Bug 28273 > > And there are other places : > > git grep '" + "\\n"' origin/main > origin/main:koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt: > msg = _("- Please select a pickup location for the item" + "\n") > origin/main:koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt: > msg = _("- Please select a pickup location for this hold" + "\n"); > origin/main:koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt: > msg = _("- Please select a pickup location for this hold" + "\n"); Fridolin, do we need to open a follow-up bug? (In reply to Lucas Gass from comment #9) > (In reply to Fridolin Somers from comment #7) > > Looks like this is since Bug 28273 > > > > And there are other places : > > > git grep '" + "\\n"' origin/main > > origin/main:koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt: > > msg = _("- Please select a pickup location for the item" + "\n") > > origin/main:koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt: > > msg = _("- Please select a pickup location for this hold" + "\n"); > > origin/main:koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt: > > msg = _("- Please select a pickup location for this hold" + "\n"); > > Fridolin, do we need to open a follow-up bug? Yes, I think it would be good. Created Bug 37017 |