Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.3] Consistent Database Name Validation #3386

Merged
merged 4 commits into from
Feb 16, 2025

Conversation

heelc29
Copy link
Member

@heelc29 heelc29 commented Feb 2, 2025

Pull Request für Issue #3360 .

Zusammenfassung der Änderungen

add strings

INSTL_DATABASE_NAME_MSG_MYSQL="Der Datenbankname ist ungültig. Der Datenbankname darf die folgenden Zeichen nicht enthalten: \ /"
INSTL_DATABASE_NAME_MSG_POSTGRES="Der Datenbankname ist ungültig. Der Datenbankname muss mit einem Buchstaben beginnen, gefolgt von alphanumerischen Zeichen."

Wo wird der Sprachstring angezeigt / Wie kann getestet werden

backend (com_config)
image
image

@heelc29 heelc29 added this to the 5.3.0v1 milestone Feb 2, 2025
@heelc29 heelc29 requested a review from a team February 2, 2025 12:37
@heelc29 heelc29 self-assigned this Feb 2, 2025
@heelc29 heelc29 linked an issue Feb 2, 2025 that may be closed by this pull request
@tecpromotion
Copy link
Member

Derzeit sprechen wir in der Konfiguration nur von "Datenbank" statt "Datenbankname".

COM_CONFIG_FIELD_DATABASE_NAME_LABEL="Datenbank"

Core im Vergleich: https://github.com/joomla/joomla-cms/blob/95fc2cb38c6ad9c8c3e824ddecdddfa734a90db5/administrator/language/en-GB/com_config.ini#L60

Verwirrt das den Nutzer?

@heelc29
Copy link
Member Author

heelc29 commented Feb 2, 2025

Hatte die Übersetzung aus der Installation übernommen, da diese hier im Core identisch sind.

Stimmt in der Installation verwenden wir Datenbankname:

INSTL_DATABASE_NAME_LABEL="Datenbankname"

Ob es irritierend ist, kann ich so nicht sagen... Datenbank ist auch gleichzeitig die Bezeichnung für das ganze Fieldset.
Die Datenbank ist ungültig. Die Datenbank darf die folgenden Zeichen nicht enthalten: \ /
Die Datenbank ist ungültig. Die Datenbank muss mit einem Buchstaben beginnen, gefolgt von alphanumerischen Zeichen.

@tecpromotion
Copy link
Member

@zero-24 was meinst du?

@tecpromotion tecpromotion requested review from a team, tecpromotion and zero-24 and removed request for a team February 3, 2025 07:17
Copy link
Member

@tecpromotion tecpromotion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @heelc29
We'll use this version for now.

@tecpromotion tecpromotion merged commit 6501040 into joomlagerman:5.3-dev Feb 16, 2025
@zero-24
Copy link
Member

zero-24 commented Feb 16, 2025

Entschuldigt deine Nachricht ist untergegangen, würde das auch hier erstmal so lassen

@heelc29 heelc29 deleted the issue-3360 branch March 15, 2025 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

[5.3] Consistent Database Name Validation
3 participants