Seite 2 von 2
Re: con 4.9.12: PHP-Warnung Declaration of cApiArticleLanguageVe
Verfasst: Di 26. Sep 2017, 09:18
von Bernbacher
Sorry, Schreibfehler: ist natürlich 4.9.12 und nicht 4.9.1
.
Re: con 4.9.12: PHP-Warnung Declaration of cApiArticleLanguageVe
Verfasst: Di 26. Sep 2017, 09:44
von Oldperl
Servus,
kein Problem!
Alternativ mal bei Frederic für die 4.9.13 RC anfragen, näheres in diesem Thread
https://forum.contenido.org/viewtopic.php?f=98&t=38587
Auch kannst Du versuchen die Klasse cString aus dem GIT zu nehmen. Die Bugbehebungen von Frederic sind zumeist gegen den aktuellen GIT-Zweig gemacht und daher können entsprechende dort bereits vorhandene Änderungen und Einbauten in der 4.9.12 natürlich noch fehlen.
Gruß aus Franken
Ortwin
Re: con 4.9.12: PHP-Warnung Declaration of cApiArticleLanguageVe
Verfasst: Di 9. Jan 2018, 14:08
von rethus
Hab derzeit das gleiche Problem nach einem Upgrade, nur das in der Artikelansicht bei jedem gelöschten Artikel eine unschöne Fehlermeldung ausgegeben wird:
Ursache liegt darin begründet:
cApiArticleLanguageVersionCollection erbt von cApiArticleLanguageCollection
In Zeile 54 wird dessen create-Methode mit einem Array als Parameter definiert :
https://git.contenido.org/projects/CON/ ... ion.php#54
in der Elternklasse sind dies jedoch einzelne Parameter:
https://git.contenido.org/projects/CON/ ... age.php#83
Im neuen Dev-Branch ist an dieser Stelle noch nichts verändert worden, ein laden von "neueren Zip's" bringt demnach nichts.
Hier muss man sich wohl für eine Variante entscheiden, und diese dann Systemweit umsetzen.
Re: con 4.9.12: PHP-Warnung Declaration of cApiArticleLanguageVe
Verfasst: Di 9. Jan 2018, 14:43
von Oldperl
Servus,
rethus hat geschrieben: ↑Di 9. Jan 2018, 14:08
Im neuen Dev-Branch ist an dieser Stelle noch nichts verändert worden, ein laden von "neueren Zip's" bringt demnach nichts.
Hier muss man sich wohl für eine Variante entscheiden, und diese dann Systemweit umsetzen.
Im develop-Branch ist da sehr wohl etwas verändert. Guckst Du hier
https://git.contenido.org/projects/CON/ ... develop#51
Man muss natürlich auch den entsprechenden Branch vorher wählen...
Gruß aus Franken
Ortwin
Re: con 4.9.12: PHP-Warnung Declaration of cApiArticleLanguageVe
Verfasst: Di 9. Jan 2018, 15:15
von rethus
Kleine Ursache, große Wirkung
Klassischer Fehler: zwischen Rückenlehne und Monitor.
Für jeden, der exakt diesen Bug beheben möchte, hier der Link zum richtigen Commit, der alle veränderten Dateien beinhaltet:
https://git.contenido.org/projects/CON/ ... 5b4b6bafbb
Danke für die clearification
1. Commit :
https://git.contenido.org/projects/CON/ ... 5b4b6bafbb
2. Commit :
https://git.contenido.org/projects/CON/ ... 63a1a0693a
3. Commit :
https://git.contenido.org/projects/CON/ ... 1c292d4589
Re: con 4.9.12: PHP-Warnung Declaration of cApiArticleLanguageVe
Verfasst: Di 9. Jan 2018, 15:18
von Oldperl
rethus hat geschrieben: ↑Di 9. Jan 2018, 15:15
Danke für die clearification
Immer wieder gerne!
Gruß aus Franken
Ortwin
Re: con 4.9.12: PHP-Warnung Declaration of cApiArticleLanguageVe
Verfasst: Di 9. Jan 2018, 15:24
von rethus
Sehe gerade... die Änderung der Funktion hat auch Auswirkungen auf andere Dateien, in denen der "alte Aufruf mit einzelnen Parametern" noch verwendet wurde.
Beispiel:
https://git.contenido.org/projects/CON/ ... evelop#116
Wäre gut, wenn man hier eine Übersicht der Dateien hätte, auf die sich diese Änderung noch auswirkt.
Hab den betreffenden Commit für eine betroffene Datei, die ich gefunden habe nun oben bei der Angabe des anderen Commit ergänzt.
Re: con 4.9.12: PHP-Warnung Declaration of cApiArticleLanguageVe
Verfasst: Mi 10. Jan 2018, 09:24
von rethus
Kurzer Nachtrag:
Wenn Ihr die Anpassungen aus dem development-Branch nutzt, müsst Ihr auch dafür sorgen, das die neue cString::toLowerCase() Funktion vorhanden ist. Diese wird nun in class.articlelanguage.php verwendet.
Für die Klasse cString wurde jedoch die Vererbungslinie verändert, was wieder weitere Abhängigkeiten mit sich bringt.
Der Vollständigkeit halber hier der Hinweis, das auch folgender Commit vollständig eingepflegt werden muss:
https://git.contenido.org/projects/CON/ ... 1c292d4589
Dann sollte es soweit laufen.
Wem das alles zu viel Bastelei für eine blöde "strict-Meldung" ist, der kann getrost auf das nächste Contenido-Release warten
Re: con 4.9.12: PHP-Warnung Declaration of cApiArticleLanguageVe
Verfasst: Do 11. Jan 2018, 11:46
von frederic.schneider_4fb
Diese Anpassungen waren echt "lustig"