Skip to content

fix(lang): Replace generic 'his/he' with singular 'they'#5646

Merged
Deltik merged 1 commit into
masterfrom
e107help/5642
May 19, 2026
Merged

fix(lang): Replace generic 'his/he' with singular 'they'#5646
Deltik merged 1 commit into
masterfrom
e107help/5642

Conversation

@e107help

@e107help e107help Bot commented May 19, 2026

Copy link
Copy Markdown
Contributor

Reported by @BillyBoy0823 in #5642. Two generic-singular his/he references in user-facing English LAN strings.

Changes

  • e107_languages/English/admin/lan_prefs.php PRFLAN_38 (line 41): "if a user post too fast, he will be redirected" → "if a user posts too fast, they will be redirected" (singular-they plus a subject-verb agreement nit)
  • e107_languages/English/admin/lan_prefs.php PRFLAN_91 (line 84): "his IP will be automatically banned" → "their IP will be automatically banned"

Why the scope shrank

Initial commit also touched e107_docs/help/English/Administrators and e107_docs/help/English/Forums. PR #5596 (feat/admin-docs-rich-renderer) rewrites all 21 English help files into a new line-based format, so the help-doc edits were either (a) on a line that is replaced wholesale by the rewrite (Administrators), or (b) already applied in #5596's tree on the same line (Forums). Both dropped to avoid conflicting with #5596; thanks to @Deltik for the catch.

Backwards compatibility

LAN constants are keyed by name, not by value. PRFLAN_38 and PRFLAN_91 keep their identifiers; only the English value changes. Translation packs in e107translations override the English string as they always have, so this is a no-op for any non-English install.

Test plan

  • Admin > Preferences renders the flood-protection and auto-ban descriptions with the new wording.
  • Non-English packs continue to override the constants regardless.

Comment thread e107_docs/help/English/Administrators Outdated
Two user-facing English LAN values used masculine generic pronouns for
an unspecified user. Switches them to singular 'they' in PRFLAN_38 and
PRFLAN_91 of e107_languages/English/admin/lan_prefs.php.

PRFLAN_38 also drops a subject-verb agreement nit ("a user post" ->
"a user posts") that falls out of the same edit.

Reported by @BillyBoy0823 in #5642.
@e107help e107help Bot force-pushed the e107help/5642 branch from 662818f to 19e1f3f Compare May 19, 2026 05:37
@Deltik Deltik merged commit 560ac03 into master May 19, 2026
44 checks passed
@Deltik Deltik deleted the e107help/5642 branch May 19, 2026 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants