Re: Some core code to review

On Tue, Aug 21, 2012 at 11:56 AM, Juan Ezquerro LLanes <arrase@gmail.com> wrote:
> Is that a bug? 'log' index is twice?
>
> File: CAKE/lib/Cake/Utility/Debugger.php line 55
>
> protected $_templates = array(
> 'log' => array(
> 'trace' => '{:reference} - {:path}, line {:line}',
> 'error' => "{:error} ({:code}): {:description} in [{:file}, line {:line}]"
> ),
> 'js' => array(
> 'error' => '',
> 'info' => '',
> 'trace' => '<pre class="stack-trace">{:trace}</pre>',
> 'code' => '',
> 'context' => '',
> 'links' => array(),
> 'escapeContext' => true,
> ),
> 'html' => array(
> 'trace' => '<pre class="cake-error trace"><b>Trace</b>
> <p>{:trace}</p></pre>',
> 'context' => '<pre class="cake-error context"><b>Context</b>
> <p>{:context}</p></pre>',
> 'escapeContext' => true,
> ),
> 'txt' => array(
> 'error' => "{:error}: {:code} :: {:description} on line {:line} of
> {:path}\n{:info}",
> 'code' => '',
> 'info' => ''
> ),
> 'base' => array(
> 'traceLine' => '{:reference} - {:path}, line {:line}',
> 'trace' => "Trace:\n{:trace}\n",
> 'context' => "Context:\n{:context}\n",
> ),
> 'log' => array(),
> );

Perhaps it was done for some debugging reason -- although I can't
think of why -- and then forgotten about. If that's the case, maybe it
would have been better to comment it out rather than override it
temporarily so that it's more noticeable.

It's weird that PHP allows this, too.

--
You received this message because you are subscribed to the Google Groups "CakePHP" group.
To post to this group, send email to cake-php@googlegroups.com.
To unsubscribe from this group, send email to cake-php+unsubscribe@googlegroups.com.
Visit this group at http://groups.google.com/group/cake-php?hl=en-US.