Probleme mit der PHP8-Kompatibilität von Modulen

bernd · 19.04.2021 21:13:37

Sodele und der Rest:

o.k.
Scroll to top - Section List - Section Picker - Smart Image - Socialmedia made easy - Two Click Content - User extend - User Groups - User search - User Statistics - vrgr - WYSIWYG2

Reviews and Ratings
deprecated gefixt, aktuelle Version im Anhang

RSS Feed all
da hatte sich anscheinend das bereits letztes Jahr reparierte "posted_by" wieder eingeschlichen
aktuelle Version im Anhang

System Information
da zickt der MySQL-Tab mit Doctrine - sprich der Fix von Bianka aus dem letzten Jahr funktioniert nicht mehr.
Das müsste sich Bianka nochmal anschauen ...

Nicht angeschaut
- alle Module die Florian bereits in #1 hatte
- Bakery (da wird ja gerade ohnehin an der neuen Version gestrickt)
- Fancy Box (braucht man die überhaupt noch?)
- Minified css & js
- shortURL (kann ich hier lokal nicht testen)
- WebBird Library / Profiles und die dazugehörige jQuery Library (denke, da ist Bianka schneller, wenn sie mal drüber schaut)
- Module aus "Archiv"

Ich mach dann spätestens morgen noch eine Zusammenfassung der offenen Wehwehchen ...

Last edited by bernd (19.04.2021 21:34:02)

florian

bernd · 20.04.2021 23:06:35

Und die Zusammenfassung der noch offenen Problemkinder.
Falls jemand sachdienliche Hinweise oder Reparatur-Lösungen hat: nur her damit angel

DirList
die Warnings von Florian aus #8 kann ich nicht reproduzieren,
erhalte aber im FE nach wie vor einen Fatal error

Call to undefined method dirlist::sortFilesAscending() in /modules/dirlist/class.dirlist.php:716

wobei das "sortFilesAscending" jeweils das wiederspiegelt was im BE als Sortierung eingestellt ist.
Unter PHP7.4 funktioniert es, irgendwas missfällt also PHP8 an der class.dirlist ich komme aber einfach nicht dahinter was.

Inline Wrapper
steigt bereits bei der Installation nach paar "Required parameter follows" Deprecated Meldungen mit einem Fatal error aus

Cannot redeclare str_contains() in /modules/inlinewrapper/functions.php on line 372

da will sich ja mrbaseman drum kümmern
Edit: Status: gefixt - Aktuelle Version in #39

ErrorLog Viewer
hat noch ein paar Warnings nur in der Table-Ansicht die wohl mit den @ zusammenhängen in der tool.php Zeile 165 ff

MultilingualSwitcher
erzeugt Warnings

Trying to access array offset on value of type bool in /framework/class.frontend.php on line 131 und 132

funktioniert ansonsten aber sauber.
Die Warning von mastermind aus #16 kann ich nicht reproduzieren,
das könnte was OS-spezifisches sein, da müsste vielleicht mal ein regex-Spezi drüberschauen.

Multiple Page Permission Settings
da müsste sich mal jemand in der class.menu.php Zeile 77

while ( $loop && ( ( $option = each( $children[$parent] ) ) || ( $parent > $root_id ) ) )

anschauen, wie man das each da rausdröselt, ich bin scheints zu "blöd" dazu

PostIts
funktioniert mittlerweile soweit, auch mit der neuen TWIG-Version hat nur noch ein Problem mit der Seitenübergreifenden Einbindung des CSS in der javascript/postits.js

jQuery.include([
		WB_URL + '/modules/cwsoft-postits/css/postits.css'
	]);

jQuery.include is not a function

Sachdienliche Hinweise erbeten ...
Edit: Status: gefixt - Aktuelle Version in #35

System Information
kommt auf dem "MySQL"-Tab mit Doctrine wieder nicht klar

Fatal error: Uncaught Error: Call to undefined method database::conn() in /modules/sysinfo/funcs/mysqlinfo.php:31

Da hatte ja webbird letztes Jahr schon mal nachgebessert, vielleicht könnte sie da noch mal drüber schauen?

Forum
hab ich nochmal angeschaut und Warning und Deprecated Meldungen entfernt.
Aktuelle Version im Anhang

Newsreader
die Warnings in der Vorschau im Backend beseitigt
aktuelle Version im Anhang

Das war's erstmal.
So weit ich das beurteilen kann:
Sind alle anderen Module fit für PHP8 und MySQL8.
Laufen alle Module (ausser System Information (s.o.) auch problemlos mit Doctrine.

Last edited by bernd (28.04.2021 23:18:29)

florian

kant · 23.04.2021 14:19:32

Hallo Bernd

Modul Multiple Page Permission Settings

file: ~wbce/modules/page_permission/class.menu.php
line: 77
statt

[== PHP ==]
while ( $loop && ( ( $option = each( $children[$parent] ) ) || ( $parent > $root_id ) ) )

dann

[== PHP ==]
foreach( $children[$parent]  as $option['value'] )
   {
       if( ($loop == false) || ($parent < $root_id) ) break;

bitte einmal ausprobieren - hier scheint's zu gehen
[local MAMP/PHP 8.0.0 - WBCE 1.4.5? (git) ]

Falls dich noch keiner gelobt hat für Deine Fleißarbeit: ich tue es! smile wink thumb_up
(und falls doch: sorry, ich hab's überlesen ;-) )

Gruß
Kant

florian, stefanek

mrbaseman · 23.04.2021 17:34:51

Beim inlinewrapper bin ich noch nicht zum Ziel gekommen. Ich hab zwar errors und warnings unter php8 weg bekommen, hab dann aber eine leere Seite bekommen. Da muss ich noch bisschen weiter untersuchen.

@bernd, beim rss_feed_all ist mir das wohl entgangen. Eine PM mit dem Hinweis auf einen Fix wäre manchmal geschickt. Was genau in dem Fall schief gelaufen ist, hab ich jetzt aber nicht recherchiert...

bernd · 23.04.2021 20:02:32

@kant:
danke für den Lösungsvorschlag, funktioniert aber leider nicht wie erwartet sad
Anstatt des korrekten Seitenbaums (unter PHP7.4 - wo das "each" ja nur deprecated ist):

bekomme ich mit deiner Änderung das:

Die Unterseiten werden also komplett unterschlagen, dafür werden die Seiten auf der 1. Ebene immer weiter eingerückt (als ob alle Unterseiten der jeweilig drüberstehenden Seite wären) ...

bernd · 23.04.2021 20:05:37

@mrbaseman:
ist ja kein Beinbruch - hatte da nur ein Déjà-vu weil ich beim letztjährigen Frühjahrsputz schonmal dran rumgegrübelt hatte.
Aber ich gelobe Besserung bzgl. Benachrichtigungen ;-)

mrbaseman · 23.04.2021 21:32:16

@bernd: ich seh gerade, dass ich das auf meine Kappe nehmen muss. Ich hab die Änderung gerade bei mir eingechecked und bin dabei über einen Patch von dir von vor einem Jahr gestolpert. Da war genau die Änderung schon drin, ist aber irgendwie nicht in meine Sourcen rein gewandert. Vielleicht gab's beim Anwenden einen Konflikt mit Whitespaces drum herum oder irgendwie so

mrbaseman · 23.04.2021 21:46:51

bernd wrote:

Inline Wrapper
steigt bereits bei der Installation nach paar "Required parameter follows" Deprecated Meldungen mit einem Fatal error aus
Cannot redeclare str_contains() in /modules/inlinewrapper/functions.php on line 372
da will sich ja mrbaseman drum kümmern

danke für den Tipp! bei mir kam diese Meldung irgendwie nicht, aber das war der entscheidende Hinweis, denn bei php8 gibt es jetzt genau diese Funktion, die genau das macht, was hier bereits im Modul enthaltne war. Ein if(!function_exists()){...} drum herum ist da also die Lösung...

die php8 kompatible Version anbei. Bei der Suche nach der Ursache habe ich gestern schon ein paar Quoting-Probleme bei den sql-Statements ausgebügelt.

Last edited by mrbaseman (23.04.2021 22:01:14)

bernd · 23.04.2021 22:09:37

mrbaseman wrote:

die php8 kompatible Version anbei

Prima, lässt sich installieren und funktioniert so weit.
Allerdings sind die "Required parameter follows ..." Deprecated Meldungen noch vorhanden
Sollte reichen Zeile 34 so zu ändern:

function doPost ($pUrl, $pCookie, $pPostVariables, &$pHeaders) {

und Zeile 129 so:

function doGet ($pUrl, $pRequestHeaders, $pCookie, &$pHeaders, $pContent = true, $pAcceptedContentTypes = false) {

Wenn du das gleich noch mit einbauen magst ...

Last edited by bernd (24.04.2021 00:03:09)

florian

bernd · 24.04.2021 01:13:23

So,
Posits
ist auch gefixt.
Die Twig-Einbindung ist so gelöst das es sowohl mit WBCE-Versionen <= 1.4.5 als auch mit 1.5.x funktioniert (könnte man evtl. so auch für den AFE realisieren?)
Der Fehler mit der jQuery.include ist behoben durch jQuery.insert - ist irgendwie an mir Dummerchen vorbeigegangen das die include gar nicht mehr geladen wird ops
Aktualisierte Version im Anhang.

florian

mrbaseman · 24.04.2021 08:46:07

bernd wrote:

mrbaseman wrote:
die php8 kompatible Version anbei
Prima, lässt sich installieren und funktioniert so weit.
Allerdings sind die "Required parameter follows ..." Deprecated Meldungen noch vorhanden
Sollte reichen Zeile 34 so zu ändern:
function doPost ($pUrl, $pCookie, $pPostVariables, &$pHeaders) {
und Zeile 129 so:
function doGet ($pUrl, $pRequestHeaders, $pCookie, &$pHeaders, $pContent = true, $pAcceptedContentTypes = false) {
Wenn du das gleich noch mit einbauen magst ...

Gut, mach ich noch. Irgendwie hab ich wohl das Error-Logging komplett verbogen, sonst hätte ich das ja auch selbst sehen müssen.

bernd · 25.04.2021 02:37:26

Irgendwie hab ich wohl das Error-Logging komplett verbogen, sonst hätte ich das ja auch selbst sehen müssen.

Guck mal vorsichtshalber nach ob in deiner php.ini "ignore_repeated_errors" und "ignore_repeated_source" auf "off" steht.
Das hat mir am Anfang bischen Kopfzerbrechen bereitet.

Wobei ich z.Zt. auch immer mal wieder feststelle das grade Deprecated Meldungen - trotz dieser Einstellungen - nur einmal erscheinen solange der Server (in meinem Fall lighttpd) läuft. Ich hab mir deshalb angewöhnt vor dem finalen check ob ich auch alles erwischt habe den Server mal kurz neu zu starten.

florian · 25.04.2021 18:34:18

Zu
https://forum.wbce.org/viewtopic.php?pid=36575#p36575

Kann sonst man eventuell aus einem anderen Modul eine Funktion zum Generieren des Seitenbaums adaptieren?

mrbaseman · 25.04.2021 21:16:11

bernd wrote:

...
Allerdings sind die "Required parameter follows ..." Deprecated Meldungen noch vorhanden
Sollte reichen Zeile 34 so zu ändern:
function doPost ($pUrl, $pCookie, $pPostVariables, &$pHeaders) {
und Zeile 129 so:
function doGet ($pUrl, $pRequestHeaders, $pCookie, &$pHeaders, $pContent = true, $pAcceptedContentTypes = false) {
Wenn du das gleich noch mit einbauen magst ...

hab ich in der Version 2.9.7 jetzt auch mit drin und selbst noch eine warning gefunden.

bernd wrote:

Guck mal vorsichtshalber nach ob in deiner php.ini "ignore_repeated_errors" und "ignore_repeated_source" auf "off" steht.
Das hat mir am Anfang bischen Kopfzerbrechen bereitet.
Wobei ich z.Zt. auch immer mal wieder feststelle das grade Deprecated Meldungen - trotz dieser Einstellungen - nur einmal erscheinen solange der Server (in meinem Fall lighttpd) läuft. Ich hab mir deshalb angewöhnt vor dem finalen check ob ich auch alles erwischt habe den Server mal kurz neu zu starten.

die php.ini passt soweit ich das sehe. Und bei E_ALL | E_STRICT sollten doch die deprecated Warnings auch mit dabei sein, wobei mir gerade auffällt, dass das in den Grundeinstellungen wohl nochmal überschrieben wird. Aber auch dort habe ich auf "Fehler und Warnungen Anzeigen (Entwicklung)" gestellt.

florian

Slugger · 07.07.2021 15:43:27

Hat jemand hierfür eine Lösung?

2021-07-07T14:34:41+00:00 [Warning] /modules/teasers/kram/getsettings.php:[50] from /modules/znip_childs2tabs/view.php:[86] "Undefined variable $TEXT"
2021-07-07T14:34:41+00:00 [Warning] /modules/teasers/kram/getsettings.php:[50] from /modules/znip_childs2tabs/view.php:[86] "Trying to access array offset on value of type null"

florian · 08.07.2021 06:54:50

So, jetzt habe ich's auch kapiert. Da wird also mit childs2tabs (ein Modul, das es offiziell nicht mehr gibt) ein Teasers-Abschnitt (ein Modul, das archiviert ist) gezogen. Erstaunlich, dass das überhaupt funktioniert, aber sei's drum.
Jedenfalls, die Fehler im Errorlog werden versursacht, weil durch den ich sag mal indirekten Zugriff auf die Teasers-Bestandteile aus irgendwelchen Gründen die globale Sprachdatei nicht geladen wird. Warum, weiß ich jetzt gerade nicht, denn wenn eine Seite mit einem Teasers-Abschnitt direkt aufgerufen wird, funktioniert's ja.

Ich kann da jetzt wegen meiner sehr überschaubaren PHP-Fähigkeiten nur zwei Holzhammerlösungen anbieten:
1.) Entweder nach Zeile 28

if ($modus == 'slider') {$sectionclasses = $sliderclasses;}

einfügen:

if (!isset($TEXT)) {
	$TEXT = array('SAVE' => 'speichern');
}

oder Zeile 50

<input type="image"  style="float:left;" class="tsrs_set_submit" src="'.WB_URL.'/modules/'.$mod_dir.'/img/save.png" alt="'.$TEXT['SAVE'].'" title="'.$TEXT['SAVE'].'" />

stumpf ändern zu

<input type="image"  style="float:left;" class="tsrs_set_submit" src="'.WB_URL.'/modules/'.$mod_dir.'/img/save.png"  />

Slugger

Slugger · 11.07.2021 11:29:25

backup 2.9.13 Modul:

[Warning] /modules/backup/tool.php:[43] from /modules/backup/tool.php:[151] get_backup_files "String offset cast occurred"

florian · 11.07.2021 16:19:42

Danke für den Hinweis, ist gefixt in 2.9.14

tennstar · 11.07.2021 16:33:06

wollte heute mini hero 0.5.1 unter php8 wieder in eine meiner Seiten einbauen ist leider nicht mehr möglich.

"Die übertragene Datei ist ungültig" wird als Fehlermeldung angezeigt und auf den Seiten wo mini hero eingebaut war kann das Modul nicht mehr bearbeitet werden.

Es scheint auch einen Konflikt mit mit "mini gal2" zu geben, dieses Modul war auch nach dem Update nicht mehr funktionsbereit.

Erst nachdem alle beide entfernt waren und minigal2 erneut installiert wurde funktionierte es wieder.

florian · 11.07.2021 16:39:17

Kann ich nicht bestätigen.

Bitte lieber wie gehabt einen eigenen Thread für modulspezifische Probleme aufmachen.

In diesem Thread hier sollten nur Meldungen gepostet werden, die nachweislich (selbe Konstellation (Modul-/WBCE-Version), nur PHP-Versionswechsel), reproduzierbar und dokumentierbar (Errorlog) mit PHP 8 Probleme haben.

Last edited by florian (11.07.2021 17:02:53)

Slugger · 14.07.2021 07:36:39

Errorlog viewer 1.1.0

[Warning] /modules/errorlogger/tool.php:[165] from /admin/admintools/tool.php:[100] "Undefined array key 1"
[Warning] /modules/errorlogger/tool.php:[165] from /admin/admintools/tool.php:[100] "Trying to access array offset on value of type null"
[Warning] /modules/errorlogger/tool.php:[174] from /admin/admintools/tool.php:[100] "Undefined array key 1"
[Warning] /modules/errorlogger/tool.php:[177] from /admin/admintools/tool.php:[100] "Undefined array key 2"
[Warning] /modules/errorlogger/tool.php:[177] from /admin/admintools/tool.php:[100] "Trying to access array offset on value of type null"
[Warning] /modules/errorlogger/tool.php:[178] from /admin/admintools/tool.php:[100] "Undefined array key 2"
[Warning] /modules/errorlogger/tool.php:[178] from /admin/admintools/tool.php:[100] "Trying to access array offset on value of type null"
[Warning] /modules/errorlogger/tool.php:[179] from /admin/admintools/tool.php:[100] "Undefined array key 3"
[Warning] /modules/errorlogger/tool.php:[179] from /admin/admintools/tool.php:[100] "Trying to access array offset on  value of type null"
[Warning] /modules/errorlogger/tool.php:[180] from /admin/admintools/tool.php:[100] "Undefined array key 3"
[Warning] /modules/errorlogger/tool.php:[180] from /admin/admintools/tool.php:[100] "Trying to access array offset on value of type null"

Last edited by Slugger (14.07.2021 07:38:13)

florian · 14.07.2021 07:39:57

Ja, ist bekannt, dafür muss das Modul umfassend umgebaut werden.
Siehe https://forum.wbce.org/viewtopic.php?id=4390 und https://github.com/WBCE/WBCE_CMS/issues/508

Slugger

mac-schierer · 05.09.2022 14:16:13

florian wrote:

---
Zu https://forum.wbce.org/viewtopic.php?pid=36460#p36460
Procalendar warf doch einige Notices, gefixte Version hängt an.

Wie schaut es bei den Droplets von procalender aus? Sind da auch Korrekturen notwendig?

florian · 05.09.2022 15:05:14

Wenn sie noch funktionieren und im Errorlog keine Fehler aufploppen, nicht... sorry, ich habe gerade keine Testinstallation, um das zu checken. Probier es aus, und wenn Probleme auftreten, poste sie hier, dann schaue ich mir das an.

mac-schierer · 05.09.2022 15:23:39

florian wrote:

Wenn sie noch funktionieren und im Errorlog keine Fehler aufploppen, nicht... sorry, ich habe gerade keine Testinstallation, um das zu checken. Probier es aus, und wenn Probleme auftreten, poste sie hier, dann schaue ich mir das an.

Der Errorlog bleibt leer und das Frontend zeigt eine weiße Seite. Bin dann auf diesen Forenbeitrag gestoßen: https://forum.wbce.org/viewtopic.php?id=4726

Und siehe da. Die Korrektur von $dateend hat geholfen.

WBCE CMS Forum

#26 19.04.2021 21:13:37

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#27 20.04.2021 23:06:35

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#28 23.04.2021 14:19:32

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#29 23.04.2021 17:34:51

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#30 23.04.2021 20:02:32

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#31 23.04.2021 20:05:37

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#32 23.04.2021 21:32:16

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#33 23.04.2021 21:46:51

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#34 23.04.2021 22:09:37

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#35 24.04.2021 01:13:23

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#36 24.04.2021 08:46:07

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#37 25.04.2021 02:37:26

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#38 25.04.2021 18:34:18

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#39 25.04.2021 21:16:11

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#40 07.07.2021 15:43:27

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#41 08.07.2021 06:54:50

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#42 11.07.2021 11:29:25

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#43 11.07.2021 16:19:42

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#44 11.07.2021 16:33:06

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#45 11.07.2021 16:39:17

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#46 14.07.2021 07:36:39

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#47 14.07.2021 07:39:57

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#48 05.09.2022 14:16:13

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#49 05.09.2022 15:05:14

Re: Probleme mit der PHP8-Kompatibilität von Modulen

#50 05.09.2022 15:23:39

Re: Probleme mit der PHP8-Kompatibilität von Modulen

Board footer