Bug 34942 - Typo: brower
Summary: Typo: brower
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial
Assignee: David Nind
QA Contact: Katrin Fischer
URL:
Keywords: Academy
Depends on:
Blocks:
 
Reported: 2023-09-28 06:52 UTC by Magnus Enger
Modified: 2024-07-04 20:40 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: String patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.11.00,23.05.05,22.11.11
Circulation function:


Attachments
Bug 34942: Fix typo - 'brower' to 'browser' (2.01 KB, patch)
2023-09-30 09:43 UTC, David Nind
Details | Diff | Splinter Review
Bug 34942: Fix typo - 'brower' to 'browser' (2.07 KB, patch)
2023-10-02 20:39 UTC, Andrew Fuerste-Henry
Details | Diff | Splinter Review
Bug 34942: Fix typo - 'brower' to 'browser' (2.13 KB, patch)
2023-10-03 09:59 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Magnus Enger 2023-09-28 06:52:51 UTC
$ grep -rn --exclude=*.po brower *
koha-tmpl/intranet-tmpl/prog/en/includes/cateditor-ui.inc:873:                if( !confirm( _("This will retrieve macros stored in the brower, save them in the database, and delete them from the browser. Proceed?") ) ){
koha-tmpl/intranet-tmpl/prog/nb-NO/includes/cateditor-ui.inc:873:                if( !confirm( _("This will retrieve macros stored in the brower, save them in the database, and delete them from the browser. Proceed?") ) ){
Comment 1 David Nind 2023-09-30 09:43:58 UTC
Created attachment 156439 [details] [review]
Bug 34942: Fix typo - 'brower' to 'browser'

This fixes a typo in a message used in the advanced cataloguing
editor when macros are converted from being stored in the browser to
being stored in the database (bug 17268 - Advanced cataloging editor
- rancor - macros are lost when browser storage cleared).

Test plan:
1. Search for 'brower' in the codebase - there should be one occurance:
   grep -rn --exclude=*.po brower *
2. Apply the patch.
3. Run the search in step 1 again, there should now be no occurances.
4. Review the diff for the patch a nd make sure that the change makes
   sense.
5. Sign off! 8-)

Signed-off-by: David Nind <david@davidnind.com>
Comment 2 David Nind 2023-09-30 10:04:13 UTC
I'm not sure of the best way to update the nb-NO files - is this handled by the translation process?
Comment 3 Magnus Enger 2023-10-02 06:29:05 UTC
(In reply to David Nind from comment #2)
> I'm not sure of the best way to update the nb-NO files - is this handled by
> the translation process?

Yes it is. Thanks for the fix!
Comment 4 Andrew Fuerste-Henry 2023-10-02 20:39:53 UTC
Created attachment 156463 [details] [review]
Bug 34942: Fix typo - 'brower' to 'browser'

This fixes a typo in a message used in the advanced cataloguing
editor when macros are converted from being stored in the browser to
being stored in the database (bug 17268 - Advanced cataloging editor
- rancor - macros are lost when browser storage cleared).

Test plan:
1. Search for 'brower' in the codebase - there should be one occurance:
   grep -rn --exclude=*.po brower *
2. Apply the patch.
3. Run the search in step 1 again, there should now be no occurances.
4. Review the diff for the patch a nd make sure that the change makes
   sense.
5. Sign off! 8-)

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Comment 5 Katrin Fischer 2023-10-03 09:59:44 UTC
Created attachment 156486 [details] [review]
Bug 34942: Fix typo - 'brower' to 'browser'

This fixes a typo in a message used in the advanced cataloguing
editor when macros are converted from being stored in the browser to
being stored in the database (bug 17268 - Advanced cataloging editor
- rancor - macros are lost when browser storage cleared).

Test plan:
1. Search for 'brower' in the codebase - there should be one occurance:
   grep -rn --exclude=*.po brower *
2. Apply the patch.
3. Run the search in step 1 again, there should now be no occurances.
4. Review the diff for the patch a nd make sure that the change makes
   sense.
5. Sign off! 8-)

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 6 Tomás Cohen Arazi (tcohen) 2023-10-03 13:23:59 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 7 Fridolin Somers 2023-10-05 19:26:06 UTC
Pushed to 23.05.x for 23.05.05
Comment 8 Jacob O'Mara 2023-10-17 13:45:45 UTC
Nice work everyone!

Pushed to oldstable for 22.11.x