Page 2 of 2

Re: Fatal Error

Posted: Wed Sep 18, 2013 6:46 pm
by Timbalu
Das ist sehr schön, auch wenn ich nicht wirklich zufrieden bin, dass da im error das falsche Plugin auftaucht!

Wenn du schon alles neu machst, dann aber gleich auf die 1.7.3! :)

Re: Fatal Error

Posted: Thu Sep 19, 2013 10:53 am
by Timbalu
Ich finde das hört sich verdächtigt nach benötigtem inlude_once(), statt include S9Y_INCLUDE_PATH . 'include/plugin_internal.inc.php'; an!

Es gibt im Code Stellen, wo das mit Letzterem eingebunden wird. Die haben aber eigentlich Sicherungen, dass eigentlich keine doppelten includes vorkommen sollten. Aus irgendeinem Grund kann es aber (sehr selten, ich habe bis jetzt nur 3 mal davon gehört) zu Fehlern kommen, die diese Sicherungen aufheben.

Garvin, sollten wir die Stellen sicherheitshalber also grundsätzlich auf include_once() umschreiben?

Re: Fatal Error

Posted: Tue Sep 24, 2013 1:16 pm
by garvinhicking
Garvin, sollten wir die Stellen sicherheitshalber also grundsätzlich auf include_once() umschreiben?
Ja, gern!

Re: Fatal Error

Posted: Tue Sep 24, 2013 1:29 pm
by onli
Prüft bitte auch: Das sollte in 2.0 alles schon auf include_once stehen. Das anzupassen war gar nicht so harmlos, weil dann natürlich auch die Guard-Konstanten wegfallen sollten... Evtl. also von da übernehmen, oder es besser 2.0 überlassen bzw dort nachtragen, falls ich ne Stelle übersehen habe.

Re: Fatal Error

Posted: Tue Sep 24, 2013 2:08 pm
by Timbalu
Ich meinte diese beiden hier:
https://github.com/s9y/Serendipity/blob ... .php#L1683
und
https://github.com/s9y/Serendipity/blob ... nc.php#L18
Ist da noch mehr?

Außerdem sieht es so aus, als sollte das den 2.0 tree nicht betreffen.