search

FriendsOfREDAXO / quick_navigation

Backend-Navigation für Artikel, Medien, yForm Tabellen, Artikelverlauf
MIT License
68 stars 8 forks source link

Warning: Undefined variable "mode" nach update #84

Closed alxndr-w closed 6 years ago

alxndr-w commented 6 years ago

Warning: Undefined variable "mode" in rex_fragment "quick_favs.php" in /homepages/.../htdocs/beta/redaxo/src/core/lib/fragment.php on line 213

skerbis commented 6 years ago

Nach dem Reload sollte es weg sein - oder? @alexplusde -- Die mode-var ist neu in 3.x. Das Notice sollte nach dem Update nicht mehr auftreten.

alxndr-w commented 6 years ago

das stimmt.

skerbis commented 6 years ago

Anscheinend wird das Fragment bereits aufgerufen obwohl die neue Class noch nicht geladen ist. (Ist meine Vermutung) @gharlan ?

skerbis commented 6 years ago

Ich kann den Fehler mit der aktuellen 3.3.0 nicht reproduzieren. @alexplusde Wie sieht's bei Dir aus?

alxndr-w commented 6 years ago

3.3 ist noch nicht im Installer, oder? Es geht sicherlich darum, wenn man aus einer Version updatet, die mode noch nicht im Fragment hatte, hin zur Version, die sie einsetzt.

Ich denke mal, das hat hiermit zu tun: https://github.com/redaxo/redaxo/issues/702#issuecomment-359547286

Ein Teil des Codes ist nach dem Update schon aktualisiert, ein Teil eben noch nicht verfügbar.

skerbis commented 6 years ago

3.3 ist im Installer - ok, dann liegt das Problem mit der notice wohl nicht bei mir. Danke Hier kommt noch eine neue mod-var dazu.

alxndr-w commented 6 years ago

Habe mir nochmal darüber Gedanken gemacht.

Das Fragment sollte vorab prüfen, ob $this->mode (und andere) existiert. Das Problem hatte ich in meinem DSGVO-Addon auch, wenn ich nachträglich Parameter in das Fragment hinzufüge. Hier ist es überschaubar, weil ja nach dem Reload die Variable zur Verfügung steht.