Forums / Developer / Debug filters out variable reference errors

Debug filters out variable reference errors

Previous topic

Developer

Next topic

Author	Message
Kristof Coomans	Monday 04 September 2006 3:07:20 am I discovered some code in lib/ezutils/classes/ezdebug.php function eZDebugErrorHandler (added on trunk in rev. rev. 12364): if ( preg_match( "/variable(.*?)reference/", $errstr ) ) return; Is it good practice to silently discard these messages? I don't think so. independent eZ Publish developer and service provider \| http://blog.coomanskristof.be \| http://ezpedia.org
Clemens T	Monday 04 September 2006 6:00:40 am Very true Kristof, It's probably their way of saying... Stop reporting all those reference errors :P Greets, Clemens
Kåre Køhler Høvik	Tuesday 05 September 2006 5:34:55 am Hi The development crew here at eZ use Xdebug, so we see these warnings/errors anyway. However, I do agree we should show these warnings in the future, but it should be introduced in a major release, for example 3.9. If we fix this in a stable3.8 sub release, alot of people will get warnings they don't know where come from. Kåre Høvik

Author

Message

Kristof Coomans

Monday 04 September 2006 3:07:20 am

I discovered some code in lib/ezutils/classes/ezdebug.php function eZDebugErrorHandler (added on trunk in rev. rev. 12364):

if ( preg_match( "/variable(.*?)reference/", $errstr ) )
    return;

Is it good practice to silently discard these messages? I don't think so.

independent eZ Publish developer and service provider | http://blog.coomanskristof.be | http://ezpedia.org

Clemens T

Monday 04 September 2006 6:00:40 am

Very true Kristof,

It's probably their way of saying...

Stop reporting all those reference errors :P

Greets,
Clemens

Kåre Køhler Høvik

Tuesday 05 September 2006 5:34:55 am

The development crew here at eZ use Xdebug, so we see these warnings/errors anyway.

However, I do agree we should show these warnings in the future, but it should be introduced in a major release, for example 3.9. If we fix this in a stable3.8 sub release, alot of people will get warnings they don't know where come from.

Kåre Høvik