lekoala / silverstripe-debugbar

code/DebugBarController.php

<?php

namespace LeKoala\DebugBar;

use DebugBar\OpenHandler;
use SilverStripe\Control\Controller;
use SilverStripe\Control\HTTPRequest;

/**
 * A open handler controller for DebugBar
 *
 * @author Koala
 */
class DebugBarController extends Controller
{
    public function index(HTTPRequest $request)

The parameter $request is not used and could be removed.

    {
        if (!DebugBar::config()->get('enable_storage')) {
            return $this->httpError(404, 'Storage not enabled');
        }
        $debugbar = DebugBar::getDebugBar();
        if (!$debugbar) {

$debugbar is of type DebugBar\DebugBar, thus it always evaluated to true.

            return $this->httpError(404, 'DebugBar not enabled');
        }
        $openHandler = new OpenHandler($debugbar);
        $openHandler->handle();
        exit(); // Handle will echo and set headers

Using exit here is not recommended.

    }
}