Fehler nach Update auf 1.7

[MarioH]

Hallo zusammen!

Ich habe dann heute mal ein Update auf das Pre-Release 1.7 versucht. Nach dem Update bekomme ich direkt beim Aufruf des Blogs einen "Error Testing Mode". Live zu bestaunen auf http://blog.hommel-net.de.

Was kann das sein. PHP-Version ist momentan noch die 5.2.17.

Gruß
Mario

[yellowled]

Was kann das sein.

Offensichtlich ein Fehler im Browser-Kompatibilitäts-Plugin. (Das, aber das nur am Rande, ohnehin längst überflüssig ist.) Ich sehe den Fehler (serendipity_event_browsercompatibility::event_hook() ist irgendwie falsch deklariert), kann ihn als Nicht-Coder aber leider nicht beheben.

Für den Moment kannst Du einfach das Verzeichnis plugins/serendipity_event_browsercompatibility per FTP löschen (womit Du das Plugin „hart“ deinstallierst) oder in der serendipity_config.inc.php serendipity[production] auf true setzen, um die Fehlerausgabe zu unterbinden (sagte Garvin via Twitter).

In beiden Fällen wäre es gut, wenn Du die Fehlermeldung als Textdatei sichern könntest, damit wir später darauf Zugriff haben um den Fehler zu beheben.

YL

[Timbalu]

Arrgh! So ein Mist! Dieses Plugin wurde mit 1.7 entfernt!

Du musst es per Hand löschen oder die entsprechende Zeile ändern.
So müsste diese Zeile dann aussehen

Code: Select all

 function event_hook($event, &$bag, &$eventData, $addData = null) {

Eine händische Löschung ist zu empfehlen, da dieses Plugin nicht mehr benötigt und supported wird.

Wir werden nicht umhin kommen für diesen Fall eine automatische Löschung in das nächste RC einzubauen.

[yellowled]

Dieses Plugin wurde mit 1.7 entfernt!

Betrifft das nur Plugins, die in 1.7 entfernt wurden? Das können ja ausser Browsercompat nicht viele (lies: wahrscheinlich keine) sein …

YL

[MarioH]

Hallo Matthias,

ich habe das Plugin-Verzeichnis gelöscht, jetzt kann man die Seite wieder aufrufen, danke.

Das Plugin war irgendwie noch von Anbeginn der Zeit drin.

Ich habe die Fehlerseite mal als Textdatei hochgeladen. Jetzt steht noch die Umschaltung auf PHP 5.4 an.

Gruß
Mario

[Timbalu]

Ja, denn vermutlich wurde es schon entfernt, bevor Grischa im Sommer diese Art von Fehlern in allen anderen Plugins bugfixte. Ich denke auch es ist das Einzigste. Und daran dass das jetzt ein Problem werden könnte hat leider niemand gedacht. Sorry!

[yellowled]

Das Problem ist noch ein ganz anderes: Was ist mit den Plugins auf Spartacus, die zwar auf Spartacus bereits aktualisiert wurden, vom Benutzer lokal aber nicht? Ja, das ist eigentlich das Problem des Benutzers, aber der kommt dann trotzdem nicht mehr ins Blog …

YL

[Timbalu]

Das ist jetzt geraten, oder beruht das auf Erfahrung?
Das Backend an sich müsste erreichbar sein.
Die Gefahr an sich besteht.... aber das kann man nicht wirklich umcoden, ohne allzuviel Mühe für unsauber geschriebene Plugins aufzuwenden.

Wir sollten aber immerhin vor der Installation darauf hinweisen! Ich schreibe mal ein Note ins Blog.

Edit: Ich glaube Garvin arbeitet schon dran....

[yellowled]

Das ist jetzt geraten, oder beruht das auf Erfahrung?

Das sagte Garvin auf Twitter.

Das Backend an sich müsste erreichbar sein.

War's bei Mario auch nicht.

Wir sollten aber immerhin vor der Installation darauf hinweisen! Ich schreibe mal ein Note ins Blog.

Garvin hat den Blogeintrag zurückgezogen, außerdem habe ich gerade Hinweise in en/de hier gepostet.

YL

[garvinhicking]

Ich glaub ich spinne.

Ich war mir SICHER dass das Problem recht einfach reproduzierbar wäre, weil das browsercompatibility plugin kein $addData in hook_event() definiert.

Jetzt habe ich gerade hier mit meinem PHP 5.4.4-10 lokal experimentieren wollen, habe das browsercompatibility-plugin von der 1.5 installiert - und mein Blog läuft aber damit problemlos, ich kriege die Fehler gar nicht.

Hö?!?

Kriegst du, Ian, das bei Dir lokal irgendwie reproduziert? Ohne es reproduzieren zu können, kann ich nicht am Fix arbeiten. Ich stelle mir vor, in der plugin_api.inc.php einfach den error_reporting Level vor dem include oder aufruf von hook_event runterzusetzen, und danach wieder auf den normalen level, damit die Warning bei unterschiedlicher Signatur erst gar nicht erscheint.

Aber ich krieg den Fehler hier gar nicht. Hilfe?!

Grüße,
Garvin

[MarioH]

Hallo,

also, ich hatte zwischenzeitlich die PHP-Version auf 5.4.10 geändert. Habe dann das alte Plugin wieder hochgeladen und eingebunden. Und: Kein Fehler.

Ich habe dann mal die PHP-Version wieder auf 5.2.17 zurückgestellt, der Fehler ist wieder da.

Anscheinend tritt der Fehler in dem Plugin nur mit der PHP 5.2.x auf. Mit 5.3.x habe ich noch nicht probieren können, ich habe hier noch eine lokale Installation damit, ich werde das mal testen und hier wieder Bescheid geben.

Gruß
Mario

[garvinhicking]

Hi!

Danke. In der Tat kann ich das mit meinem PHP 5.2 auch so reproduzieren. Weiß garnicht genau woher das konkret rührt.

Ich habe gerade einen kleinen Patch committed in den GIT master, aber der löst das bei mir noch nicht vollständig. Muss mir das genauer anschauen, werde aber wohl erst im Laufe der Woche dazu kommen. Sehr frickelig.

Grundsätzlich sollte es sonst helfen in der serendipity_config_local.inc.php einfach $serendipity['production'] = true; zu setzen, dann wird der sensible Errorhandler rausgekickt.

Grüße,
Garvin

[Timbalu]

Ich war mir SICHER dass das Problem recht einfach reproduzierbar wäre, weil das browsercompatibility plugin kein $addData in hook_event() definiert.

Ups, war kurz mal weg...
Meins hat es in 1.1

Code: Select all

 function event_hook($event, &$bag, &$eventData) {

und das muss wie gesagt anders lauten.

Mal ins Unreine gesprochen....
Eigentlich wäre es ja das einfachste man lieferte die korrigierte Version mit und könnte dann ein automatisches de-install über function uninstall oder so einbauen....

[Timbalu]

Mein zur Feier des Tages neu aufgesetztes PHP 5.4.11 meldet mir - wenn ich das Plugin wieder hinzufüge - aber auch gleich den Fehler "Declaration of serendipity_event_browsercompatibility::event_hook() should be compatible..." (dann wohl eher wahrscheinlich, weil ich zum testen den

Code: Select all

set_error_handler($serendipity['errorhandler'], E_ALL & ~E_NOTICE);// ^ E_STRICT);

schärfer gestellt habe.

[MarioH]

In meiner Testumgebung mit PHP 5.3. erzeugt das Plugin auch den Fehler.

Gruß
Mario