Re: Some core code to review

I think its some mistake, as well.

its probably best to test the class with the second log key removed and to send a pull request on

https://github.com/cakephp/cakephp/pulls 

Am Freitag, 24. August 2012 20:35:28 UTC+2 schrieb cricket:

On Tue, Aug 21, 2012 at 11:56 AM, Juan Ezquerro LLanes <arr...@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.