search

WBCE / News-with-Images

Extended news module which allows to have multiple images per post
0 stars 4 forks source link

$wb->preprocess erzeugt Notices #56

Closed instantflorian closed 5 years ago

instantflorian commented 5 years ago

https://forum.wbce.org/viewtopic.php?pid=25196#p25196

instantflorian commented 5 years ago

[quote] Hm.. ja, das kommt, weil die preprocess Methode bei WBCE "rausgeworfen" wurde und das ist wahrscheinlich nicht bei den Modulerstellern angekommen, die Notice nicht gesehen worden o.ä. [/quote] Hm... ich hab das jetzt mal geprüft, eine Notice sehe ich nicht obwohl ich bei Grundeinstellungen auf "Fehler und Warnungen anzeigen" geschaltet habe und bewusst produzierte Notices (Test mit Code2 > echo $foo; => Notice: Undefined variable) erscheinen. Wo würde die NWI-spezifische Notice denn ausgegeben?

webbird commented 5 years ago

Spezifische Notices gibt es nicht, eine Notice ist eine Notice. Die Funktion ist zwar als obsolet markiert, existiert aber nach wie vor. (class.frontend.php ab Zeile 346.)

Edit: https://github.com/WBCE/WBCE_CMS/blob/641303f3955cd3e36d5ce4447c09f49f3beb0441/wbce/framework/class.frontend.php#L346

mrbaseman commented 5 years ago

da die Funktion in WB schon rausgeflogen ist und wblink auch in WBCE schon seit längerer Zeit per Outputfilter eingefügt werden, können wir auch auf preprocess verzichten: https://github.com/WBCE/News-with-Images/commit/5e0f055c8dd6855d7a6a076de2df7fa05e6d25d6

webbird commented 5 years ago

Spricht nichts gegen, aber es kann keine Notices geben, wenn die Funktion noch vorhanden ist.

webbird commented 5 years ago

Ist erledigt.