Love Fuel?    Donate

FuelPHP Forums

Ask your question about FuelPHP in the appropriate forum, or help others by answering their questions.
Profiler output fix
  • Hi there,
    if you add info debugging information that will contain html characters this will be displayed accordingly in the profiling box, which might not be what you expect.
    Example:
    \Log::info('previous_value=foo&current_value=var');
    
    Will show
    previous_value=foo¤t_value=var
    
    since ¤ will be transformed as ¤ by browsers, despite the missing semicolon. In order to fix it it's enough to change
    $return_output .='<div><pre>'.$log['data'].'</pre></div>';
    
    to
    $return_output .='<div><pre>'.htmlspecialchars($log['data']).'</pre></div>';
    
    in vendor/phpquickprofiler/display.php:362 (BTW, This forum has the same problem. I had to transform the html chars in order to post it in a more or less readable way. And anyway it messed up all the formatting.) Cheers
  • Nano, If you notice bugs like this, please create an issue for them on http://github.com/fuel/core/issues so someone can look at it. We don't use the forum for bug reports. Issue is corrected in 1.3/develop.
  • Thanks a lot, mate.
    Will do, from now on. Cheers

Howdy, Stranger!

It looks like you're new here. If you want to get involved, click one of these buttons!

In this Discussion