Bug 36091 - Spelling: Use "card number" instead of cardnumber in text
Summary: Spelling: Use "card number" instead of cardnumber in text
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-02-14 07:46 UTC by Magnus Enger
Modified: 2024-03-19 15:12 UTC (History)
5 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:
24.05.00,23.11.04


Attachments
Bug 36091: Spelling: Use "card number" instead of cardnumber in text (9.64 KB, patch)
2024-02-14 14:33 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 36091: Spelling: Use "card number" instead of cardnumber in text (9.78 KB, patch)
2024-02-15 07:29 UTC, Magnus Enger
Details | Diff | Splinter Review
Bug 36091: Spelling: Use "card number" instead of cardnumber in text (9.86 KB, patch)
2024-02-23 10:07 UTC, Julian Maurice
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Magnus Enger 2024-02-14 07:46:09 UTC
Not sure which spelling is correct, but consistency is good. 

$ grep -rn "Patron cardnumber" *
koha-tmpl/intranet-tmpl/prog/en/includes/csv_headers/reports/cash_register_stats.tt:4:        "[% t("Manager name") | html %]"[% sep | html %]"[% t("Patron cardnumber") | html %]"[% sep | html %]"[% t("Patron name") | html %]"[% sep | html %]"[% t("Transaction library") | html %]"[% sep | html %]"[% t("Transaction date") | html %]"[% sep | html %]"[% t("Updated") | html %]"[% sep | html %]"[% t("Transaction type") | html %]"[% sep |html %]"[% t("Notes") | html %]"[% sep | html %]"[% t("Amount") | html %]"[% sep | html %]"[% t("Title") | html %]"[% sep | html %]"[% t("Barcode") | html %]"[% sep | html %]"[% t("Item type") | html %]"
grep: misc/translator/po/.git/objects/pack/pack-b0a884343922aff65f857e780adda0f2bcc2203f.pack: binary file matches
grep: misc/translator/po/.git/objects/pack/tmp_pack_O9fPOy: binary file matches
misc/release_notes/release_notes_20_05_00.html:1588:<li><a href="http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24329">[24329]</a> Patron cardnumber change times are lost during upgrade for bug 3820</li>
misc/release_notes/release_notes_20_05_00.md:1194:- [[24329]](http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24329) Patron cardnumber change times are lost during upgrade for bug 3820

$ grep -rn "Patron card number" *
koha-tmpl/intranet-tmpl/prog/en/modules/reports/cash_register_stats.tt:172:            <th>Patron card number</th>
Comment 1 Owen Leonard 2024-02-14 13:54:21 UTC
I think we should add something like this to our Terminology page (https://wiki.koha-community.org/wiki/Terminology):

'card number (noun)
    When referring to a patron's library card, use "card number," not "cardnumber." "cardnumber" should be used only when referring to the database column.'
Comment 2 Owen Leonard 2024-02-14 14:33:27 UTC Comment hidden (obsolete)
Comment 3 Magnus Enger 2024-02-15 07:29:56 UTC
Created attachment 162170 [details] [review]
Bug 36091: Spelling: Use "card number" instead of cardnumber in text

This patch replaces the term "cardnumber" with "card number" in labels
and text where the patron's library card number is referred to. Where
the database column is referenced "cardnumber" is kept.

A full test plan would be very time-consuming and I think examining the
patch should be verification enough for changes which are limited to
adding a space.

Signed-off-by: Magnus Enger <magnus@libriotech.no>
Verified that the patch only changes "visible" text, and not
references to the database column.
Comment 4 Julian Maurice 2024-02-23 10:07:38 UTC
Created attachment 162370 [details] [review]
Bug 36091: Spelling: Use "card number" instead of cardnumber in text

This patch replaces the term "cardnumber" with "card number" in labels
and text where the patron's library card number is referred to. Where
the database column is referenced "cardnumber" is kept.

A full test plan would be very time-consuming and I think examining the
patch should be verification enough for changes which are limited to
adding a space.

Signed-off-by: Magnus Enger <magnus@libriotech.no>
Verified that the patch only changes "visible" text, and not
references to the database column.
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Comment 5 Katrin Fischer 2024-03-07 14:04:29 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 6 Fridolin Somers 2024-03-11 09:25:38 UTC
Pushed to 23.11.x for 23.11.04
Comment 7 Lucas Gass 2024-03-19 15:12:09 UTC
Doesn't apply to 23.05.x cleanly, no baackport.