milesj / utility

[Deprecated] A CakePHP plugin for common utility classes.
MIT License
69 stars 24 forks source link

Debug-level overridden #30

Closed lftbrts closed 10 years ago

lftbrts commented 10 years ago

The AjaxHandlerComponent::initialize() overrides the debug-level with a value of 0. Afterwards the setting is never reset to its original value. This behavior has a serious impact on other components, etc. Is this piece of source code an artifact or necessary?

milesj commented 10 years ago

It disables debug so that the AJAX responses aren't malformed. How is this breaking other components if the override only applies to a single AJAX request? Are you trying to debug it?

lftbrts commented 10 years ago

The more I think about it, the more I see that setting the debug level in the initialize() method is perfectly okay.