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

Fehler bei ynewsletter mit yform? #49

Open
erraiva opened this issue Apr 28, 2022 · 4 comments
Open

Fehler bei ynewsletter mit yform? #49

erraiva opened this issue Apr 28, 2022 · 4 comments

Comments

@erraiva
Copy link

erraiva commented Apr 28, 2022

Einen Fehler bekomme ich, wenn ich nach der Installation des ynewsletter bei yform in das SetUp gehe:
Declaration of rex_yform_validate_ynewsletter_auth::getDescription() must be compatible with rex_yform_base_abstract::getDescription(): string. Erzeugt wird der Fehler laut Logfile in
redaxo/src/addons/ynewsletter/lib/yform/validate/ynewsletter_auth.php, Zeile 52.
Wenn ich ynewsletter deaktiviere funktioniert der SetUp Link bei yform wieder.
LG Heinz

@erraiva
Copy link
Author

erraiva commented Apr 28, 2022

Bei den github-Versionen der beiden habe ich den Fehler nicht. Kann das sein?

@tyrant88
Copy link
Contributor

tyrant88 commented Oct 7, 2022

Ist in Version 1.5.1 seit dem 15. März gefixt.

@brandes-webdesign
Copy link

Ich habe mir eben das Addon in der Version 1.5.1 heruntergeladen über den Installer (Redaxo Backend)
und auch hier einmal in den Download (Zip-Datei) geschaut:
https://redaxo.org/download/addons/ynewsletter/

Dort ist der Fehler immer noch drinnen.

Datei: ynewsletter/lib/yform/validate/ynewsletter_auth.php

Zeile 52: public function getDescription(): string
Zeile 57: public function getDefinitions(): array

Anscheinend wurde die Code-Anpassung beim erstellen der Version 1.5.1 nicht mit eingefügt / übernommen.

Mit der Code Anpassung klappt es jedenfalls.

@tyrant88
Copy link
Contributor

Ooops, da hatte ich mich verguckt, es wurde erst nach dem 1.5.1 Release gefixt.

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

No branches or pull requests

3 participants