[Bug 34433] New: 'Custom cover image' in lightbox is untranslatable
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 Bug ID: 34433 Summary: 'Custom cover image' in lightbox is untranslatable Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: I18N/L10N Assignee: koha-bugs@lists.koha-community.org Reporter: caroline.cyr-la-rose@inlibro.com QA Contact: testopia@bugs.koha-community.org CC: frederic@tamil.fr Created attachment 153974 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153974&action=edit Sample records In the staff interface, when viewing a custom cover image in the lightbox, at the bottom, it says 'Custom cover image'. This string is not translatable. To test: Note: I attached a file of sample (MARC21) records that I use for various image sources, in which one record has a URL to an image in 956$u, you can import this file into Koha, if needed - Download the attached marc file (coverimages.mrc) - In the Koha staff interface, go to Cataloging > Stage records for import - Click the "Choose file" or "Browse" button and choose the downloaded file - Click "Upload file" - Click "Stage for import" - Click "View detail of the enqueued job" - Click "View batch" - Click "Import this batch into the catalog" 1. Enable CustomCoverImages 1.1. Go to Administration > System preferences 1.2. Search for CustomCoverImages 1.3. Select "Show" in CustomCoverImages 1.4. Set CustomCoverImagesURL to {956$u} 1.5. Click "Save all Enhanced content preferences" 2. Install new language 2.1. gulp po:update --lang fr-CA 2.2. Search for "Custom cover image" in the staff-prog.po file, make sure it is translated 2.3. Install translations ./misc/translator/translate install fr-CA 2.4. In the staff interface, go to Administration > System preferences 2.5. Search for language 2.6. Check the box next to the newly installed language 2.7. Click "Save all I18N/L10N preferences" 3. View the image in the lightbox in the staff interface 3.1. If using the attached records, search the catalog in the staff interface for "As you like it"; If not using the file, add an image URL in 956$u, or finagle something with CustomCoverImagesURL 3.2. Click on the title to view the detailed record 3.3. Click on the image to view it in the lightbox --> In the image box, notice that it says "Custom cover image" 3.4. Click the X to close the lightbox 5. Change the language of the staff interface 6. Reopen the lightbox --> The caption of the lightbox is still "Custom cover image" in the other language even if the string was translated in the po file -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Version|unspecified |master Assignee|koha-bugs@lists.koha-commun |oleonard@myacpl.org |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 153993 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153993&action=edit Bug 34433: 'Custom cover image' in lightbox is untranslatable In the staff interface, when viewing a custom cover image in the lightbox, at the bottom, it says 'Custom cover image'. This string is not translatable. The string is missing the _() i18n function. This patch also corrects the "Coce image from Amazon.com" which was missing the correct i18n function too. If necessary, import the MARC records attached to this bug to use in testing. 1. Enable CustomCoverImages 1.1. Go to Administration > System preferences 1.2. Search for CustomCoverImages 1.3. Select "Show" in CustomCoverImages 1.4. Set CustomCoverImagesURL to {956$u} 1.5. Click "Save all Enhanced content preferences" 2. Install a new language 2.1. gulp po:update --lang fr-CA 2.2. Search for "Custom cover image" and "Coce image from Amazon.com" in fr-CA-staff-prog.po file, make sure it is translated 2.3. Install translations ./misc/translator/translate install fr-CA 2.4. In the staff interface, go to Administration > System preferences 2.5. Search for language 2.6. Check the box next to the newly installed language 2.7. Click "Save all I18N/L10N preferences" 3. View the image in the lightbox in the staff interface 3.1. If using the attached records, search the catalog in the staff interface for "As you like it"; If not using the file, add an image URL in 956$u, or finagle something with CustomCoverImagesURL 3.2. Click on the title to view the detailed record 3.3. Click on the image to view it in the lightbox Test the lightbox in the default English translation and in your newly-translated language. In both cases the caption in the lightbox footer should be correct. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #153993|0 |1 is obsolete| | --- Comment #2 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- Created attachment 153994 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153994&action=edit Bug 34433: 'Custom cover image' in lightbox is untranslatable In the staff interface, when viewing a custom cover image in the lightbox, at the bottom, it says 'Custom cover image'. This string is not translatable. The string is missing the _() i18n function. This patch also corrects the "Coce image from Amazon.com" which was missing the correct i18n function too. If necessary, import the MARC records attached to this bug to use in testing. 1. Enable CustomCoverImages 1.1. Go to Administration > System preferences 1.2. Search for CustomCoverImages 1.3. Select "Show" in CustomCoverImages 1.4. Set CustomCoverImagesURL to {956$u} 1.5. Click "Save all Enhanced content preferences" 2. Install a new language 2.1. gulp po:update --lang fr-CA 2.2. Search for "Custom cover image" and "Coce image from Amazon.com" in fr-CA-staff-prog.po file, make sure it is translated 2.3. Install translations ./misc/translator/translate install fr-CA 2.4. In the staff interface, go to Administration > System preferences 2.5. Search for language 2.6. Check the box next to the newly installed language 2.7. Click "Save all I18N/L10N preferences" 3. View the image in the lightbox in the staff interface 3.1. If using the attached records, search the catalog in the staff interface for "As you like it"; If not using the file, add an image URL in 956$u, or finagle something with CustomCoverImagesURL 3.2. Click on the title to view the detailed record 3.3. Click on the image to view it in the lightbox Test the lightbox in the default English translation and in your newly-translated language. In both cases the caption in the lightbox footer should be correct. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 --- Comment #3 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- Thanks Owen! You did it so fast that I still had my tab open! I only had to apply, install and refresh XD -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #153994|0 |1 is obsolete| | --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 154118 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154118&action=edit Bug 34433: 'Custom cover image' in lightbox is untranslatable In the staff interface, when viewing a custom cover image in the lightbox, at the bottom, it says 'Custom cover image'. This string is not translatable. The string is missing the _() i18n function. This patch also corrects the "Coce image from Amazon.com" which was missing the correct i18n function too. If necessary, import the MARC records attached to this bug to use in testing. 1. Enable CustomCoverImages 1.1. Go to Administration > System preferences 1.2. Search for CustomCoverImages 1.3. Select "Show" in CustomCoverImages 1.4. Set CustomCoverImagesURL to {956$u} 1.5. Click "Save all Enhanced content preferences" 2. Install a new language 2.1. gulp po:update --lang fr-CA 2.2. Search for "Custom cover image" and "Coce image from Amazon.com" in fr-CA-staff-prog.po file, make sure it is translated 2.3. Install translations ./misc/translator/translate install fr-CA 2.4. In the staff interface, go to Administration > System preferences 2.5. Search for language 2.6. Check the box next to the newly installed language 2.7. Click "Save all I18N/L10N preferences" 3. View the image in the lightbox in the staff interface 3.1. If using the attached records, search the catalog in the staff interface for "As you like it"; If not using the file, add an image URL in 956$u, or finagle something with CustomCoverImagesURL 3.2. Click on the title to view the detailed record 3.3. Click on the image to view it in the lightbox Test the lightbox in the default English translation and in your newly-translated language. In both cases the caption in the lightbox footer should be correct. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |23.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 --- Comment #5 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 23.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34433 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)|23.11.00 |23.11.00,23.05.04 released in| | CC| |fridolin.somers@biblibre.co | |m Status|Pushed to master |Pushed to stable --- Comment #6 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to 23.05.x for 23.05.04 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org