search

contao / managed-edition

Contao Managed Edition
17 stars 14 forks source link

PHP Fehlermeldungen werden auf dem Bildschirm ausgegeben #24

Closed Mynyx closed 6 years ago

Mynyx commented 7 years ago

PHP Fehlermeldungen werden stets auf dem Bildschirm ausgegeben, eine Konfigurationsmöglichkeit wie in Contao 3.5 fehlt.

aschempp commented 7 years ago

Das sollte eigentlich nicht der Fall sein? Die Ausgabe von Fehlern hängt davon ab welchen Entry Point du benutzt, ob app.php oder app_dev.php?

Mynyx commented 7 years ago

Ich benutze den app.php Entry Point.

ausi commented 7 years ago

Kann ich bestätigen, der app.php entry point zeigt fehlermeldungen an:

contao-error

Ist aber vermutlich ein contao/core-bundle issue unabhängig von der managed edition.

aschempp commented 7 years ago

Ihr meint die Meldung die im Inhalt steht? Das ist nur im Backend/bei abgemeldetem Benutzer der Fall 😉

ausi commented 7 years ago

Stimmt, sorry. Im frontend wird keine Fehlermeldung ausgegeben 👍

leofeyer commented 7 years ago

Das ist doch auch keine PHP-Fehlermeldung, sondern unsere Pretty Error Screen.

Mynyx commented 7 years ago

Nein, ich meine native PHP Fehlermeldungen.

ausi commented 7 years ago

Das ist doch auch keine PHP-Fehlermeldung, sondern unsere Pretty Error Screen.

Im pretty error screen vom backend wird die message der exception ausgegeben, das betrifft auch PHP-Fehlermeldungen. Allerdings ist das auch in Ordnung so denke ich, da es nur das Backend betrifft.

Nein, ich meine native PHP Fehlermeldungen.

Kannst du ein Beispiel nennen wie man das reproduzieren kann? Und eventuell einen screenshot zeigen damit wir von der selben Art von Fehlermeldung sprechen?

leofeyer commented 7 years ago

Aber der Pretty-Error-Handler ist doch auch im Frontend aktiv, oder?

ausi commented 7 years ago

Ja, aber im Frontend wird nicht das Backend-Error-Template geladen (PrettyErrorScreenListener.php:124) und nur im backend template wird die fehlermeldung ausgegeben...

leofeyer commented 7 years ago

Müssen wir also etwas ändern oder kann ich das Ticket schließen?

ausi commented 7 years ago

@Mynyx hat nicht mehr geantwortet auf meine Frage.

IMO kann es geschlossen werden.

fritzmg commented 7 years ago

Hm, I just had something like this:

screen shot 2017-09-16 at 15 36 12

@Mynyx did you mean that?

dmolineus commented 7 years ago

Can confirm it. Also saw some unformatted fatal errors. Maybe the in PHP 7 introduced throwable errors are not handled correctly? It's just a guess, didn't look into the code.

leofeyer commented 6 years ago

How do I reproduce this?

dmolineus commented 6 years ago

@leofeyer I think you accidently closed this issue.

To repoduce it, just add a syntax error to your code which will throw an fatal error.

Hm, no. I can't reproduce it anymore.

leofeyer commented 6 years ago

Hm, no. I can't reproduce it anymore.

Me neither, that's why I closed it. Maybe it was an issue with one of the vendor packages.