镜像站点 · 本页由第三方 GitHub 只读镜像提供，非 GitHub 官方站点，不接受任何登录或凭据输入。前往 github.com
Skip to content

i18n: Variable %uid is displayed incorrectly at Transifex #52004

Closed
Bug
#52005
Closed
i18n: Variable %uid is displayed incorrectly at Transifex#52004
Bug
#52005
Labels
2. developingWork in progress32-feedbackbugfeature: language/translations (l10n/i18n)Localization and translation mattersfeature: ldap

Description

@rakekniven
rakekniven
opened on Apr 7, 2025

Do not use %uid.

server/apps/user_ldap/lib/Connection.php

Line 542 in a42563f

$this->l10n->t('Login filter does not contain %uid place holder'),

Use %s like in Line 493 and so many other areas.

Variable %uid result in:

Image

Please see that string is partly marked as protected and therefore translators are unable to translate it easily.

Activity

  1. rakekniven
    self-assigned this
    on Apr 7, 2025
  2. github-project-automation
    added this to 🧑‍🤝‍🧑 Community triageon Apr 7, 2025
  3. github-project-automation
    moved this to To triage in 🧑‍🤝‍🧑 Community triageon Apr 7, 2025
  4. rakekniven
    assigned
    come-nc
    and unassigned
    rakekniven
    on Apr 7, 2025
  5. susnux
    moved this from To triage to Confirmed in 🧑‍🤝‍🧑 Community triageon Apr 7, 2025
  6. susnux
    added the
    Bug
    issue type on Apr 7, 2025
  7. susnux
    added
    bug
    1. to developAccepted and waiting to be taken care of
    feature: language/translations (l10n/i18n)Localization and translation matters
    on Apr 7, 2025

  8. susnux commented on Apr 7, 2025

    @susnux
    susnux
    on Apr 7, 2025
    Contributor

    Use %s like in Line 493 and so many other areas.

    In this case its a string not a placeholder

  9. susnux
    unassigned
    come-nc
    on Apr 7, 2025
  10. szaimen
    added
    32-feedback
    2. developingWork in progress
    and removed
    1. to developAccepted and waiting to be taken care of
    on Apr 7, 2025
  11. susnux
    closed this as completedin #52005on Apr 7, 2025
  12. github-project-automation
    moved this from Confirmed to Done in 🧑‍🤝‍🧑 Community triageon Apr 7, 2025
  13. rakekniven
    changed the title [-]i18n: Variable %uid is displayed worth at Transifex[/-] [+]i18n: Variable %uid is displayed incorrectly at Transifex[/+] on Apr 7, 2025
  14. joshtrichards
    added
    feature: ldap
    on Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    2. developingWork in progress32-feedbackbugfeature: language/translations (l10n/i18n)Localization and translation mattersfeature: ldap

    Type

    Bug

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions