-
-
Notifications
You must be signed in to change notification settings - Fork 46
[5.0] Remove outdated captcha for new installation #2940
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
Conversation
script.php
Outdated
'/administrator/language/de-DE/plg_captcha_recaptcha_invisible.ini', | ||
'/administrator/language/de-DE/plg_captcha_recaptcha_invisible.sys.ini', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Das recaptcha_invisible
wird ja nicht deinstalliert, sondern nur entsperrt und bleibt weiterhin installiert.
Problem: Da wir es ja nicht mitbekommen, ob im nachhinein das Plugin (manuell) deinstalliert wird, würden bei einer Deinstallation die deutschen Übersetzungen weiterhin verbleiben.
Passt es dann, dass das Plugin in J5 nur noch in englischer Sprache vorhanden ist - es wird ja auch nicht mehr vom Core mit "Updates" versorgt!?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, aber die 4 ini-files wurden doch gelöscht im PR: https://github.com/joomla/joomla-cms/pull/41530/files#diff-cbe3a16b61f80e0af9df2e3c9b0a5b5689af5b3e54e66c1909102fd84db42193
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ja das betrifft nur die Neuinstallation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In Crowdin wird das dann auch automatisch gelöscht sein.
Was meinst du @zero-24 ?
Wir wollen wir das lösen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ja crowdin wird die Datei löschen. Wir haben zwei Möglichkeiten. Entweder behalten wir die Sprachdateien oder wir nehmen die invisible nur noch nicht ins lösch script mit auf und löschen die dann erst mit 6.x
Das ist doch eine gute Idee. Heißt bei Neuinstallationen wird sie nicht mehr mitinstalliert und mit 6 aktiv gelöscht! |
Co-authored-by: Tobias Zulauf <zero-24@users.noreply.github.com>
Pull Request für Issue #2931 .
Zusammenfassung der Änderungen
remove 4 .ini files
Wo wird der Sprachstring angezeigt / Wie kann getestet werden
backend