Bug 32222

Summary: Capitalization: id
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: TemplatesAssignee: Katrin Fischer <katrin.fischer>
Status: CLOSED FIXED QA Contact: Martin Renvoize <martin.renvoize>
Severity: trivial    
Priority: P5 - low CC: dcook, lucas, martin.renvoize
Version: MainKeywords: Academy
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32945
Change sponsored?: --- Patch complexity: String patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00,22.11.03
Attachments: Bug 32222: Fix capitalization in various "id is invalid"
Bug 32222: Fix capitalization in various "id is invalid"
Bug 32222: Fix capitalization in various "id is invalid"

Description Katrin Fischer 2022-11-16 00:15:55 UTC
ID is an abbreviation and should be written with capitals.

See: https://www.oxfordlearnersdictionaries.com/definition/english/identification

Found these instances while translating:

koha-tmpl/intranet-tmpl/prog/en/modules/admin/identity_provider_domains.tt:73
An error occurred trying to open the identity provider domain for editing. The passed id is invalid.

koha-tmpl/intranet-tmpl/prog/en/modules/admin/identity_providers.tt:69
An error occurred trying to open the identity provider for editing. The passed id is invalid.
Comment 1 Katrin Fischer 2023-02-05 00:53:42 UTC
Created attachment 146181 [details] [review]
Bug 32222: Fix capitalization in various "id is invalid"

ID is an abbreviation and should be written with capitals.

See: https://www.oxfordlearnersdictionaries.com/definition/english/identification

This fixes all occurences of "id is invalid" to read "ID is invalid"

To test:
* As some are hard to trigger warnings, the best way here is to review the patch.
Comment 2 Owen Leonard 2023-02-06 13:46:30 UTC
Created attachment 146271 [details] [review]
Bug 32222: Fix capitalization in various "id is invalid"

ID is an abbreviation and should be written with capitals.

See: https://www.oxfordlearnersdictionaries.com/definition/english/identification

This fixes all occurences of "id is invalid" to read "ID is invalid"

To test:
* As some are hard to trigger warnings, the best way here is to review the patch.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Katrin Fischer 2023-02-06 17:59:15 UTC
Thanks Owen :)
Comment 4 Martin Renvoize 2023-02-07 13:12:27 UTC
Created attachment 146322 [details] [review]
Bug 32222: Fix capitalization in various "id is invalid"

ID is an abbreviation and should be written with capitals.

See: https://www.oxfordlearnersdictionaries.com/definition/english/identification

This fixes all occurences of "id is invalid" to read "ID is invalid"

To test:
* As some are hard to trigger warnings, the best way here is to review the patch.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 5 Martin Renvoize 2023-02-07 13:13:44 UTC
Good catch.. Passing QA on this one..

However, there are more cases of 'id' littered around the templates.. not all that easy to find but there.. a number in syspref descriptions I believe (rg " id " helped me spot a few)... maybe a follow-up bug could be done for the rest?
Comment 6 Tomás Cohen Arazi 2023-02-07 13:51:19 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 7 Jacob O'Mara 2023-02-08 16:37:45 UTC
Nice work, thanks everyone!

Pushed to 22.11.x for the next release.
Comment 8 Lucas Gass 2023-02-15 21:14:03 UTC
Doesn't apply cleanly to 22.05.x, no backport