Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 20 additions & 22 deletions Plugin.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,15 @@

use Backend\Classes\Controller as BackendController;
use Backend\Models\UserRole;
use Barryvdh\Debugbar\Facades\Debugbar;
use Barryvdh\Debugbar\LaravelDebugbar;
use Barryvdh\Debugbar\SymfonyHttpDriver;
use Fruitcake\LaravelDebugbar\Facades\Debugbar;
use Fruitcake\LaravelDebugbar\LaravelDebugbar;
use Cms\Classes\Controller as CmsController;
use Cms\Classes\Layout;
use Cms\Classes\Page;
use Config;
use Event;
use Illuminate\Contracts\Http\Kernel as HttpKernelContract;
use Illuminate\Foundation\AliasLoader;
use Illuminate\Session\SessionManager;
use System\Classes\CombineAssets;
use System\Classes\PluginBase;
use Twig\Extension\ProfilerExtension;
Expand Down Expand Up @@ -61,16 +59,12 @@ public function register()
$this->app->register(\Winter\Debugbar\Classes\ServiceProvider::class);

// Replace the LaravelDebugbar with the WinterDebugbar
// Laravel Debugbar 4 configures its own HTTP driver (LaravelHttpDriver), so we only
// need to swap in Winter's Debugbar subclass for the custom JavascriptRenderer styling.
// Resolve through the container so the constructor (Application $app, Request $request)
// dependencies are injected (Laravel Debugbar 4 requires both).
$this->app->singleton(LaravelDebugbar::class, function ($app) {
$debugbar = new WinterDebugbar($app);

if ($app->bound(SessionManager::class)) {
$sessionManager = $app->make(SessionManager::class);
$httpDriver = new SymfonyHttpDriver($sessionManager);
$debugbar->setHttpDriver($httpDriver);
}

return $debugbar;
return $app->make(WinterDebugbar::class);
});

// Register alias
Expand Down Expand Up @@ -118,10 +112,12 @@ public function addGlobalCollectors()
// Disable original models collector because it will be replaced
Config::set('debugbar.collectors.models', false);

/** @var \Barryvdh\Debugbar\LaravelDebugbar $debugBar */
$debugBar = $this->app->make(\Barryvdh\Debugbar\LaravelDebugbar::class);
/** @var LaravelDebugbar $debugBar */
$debugBar = $this->app->make(LaravelDebugbar::class);
$modelsCollector = $this->app->make(ModelsCollector::class);
$debugBar->addCollector($modelsCollector);
if (!$debugBar->hasCollector($modelsCollector->getName())) {
$debugBar->addCollector($modelsCollector);
}
}
}

Expand All @@ -130,8 +126,8 @@ public function addGlobalCollectors()
*/
public function addFrontendCollectors()
{
/** @var \Barryvdh\Debugbar\LaravelDebugbar $debugBar */
$debugBar = $this->app->make(\Barryvdh\Debugbar\LaravelDebugbar::class);
/** @var LaravelDebugbar $debugBar */
$debugBar = $this->app->make(LaravelDebugbar::class);

if (Config::get('debugbar.collectors.cms', true)) {
// Disable route collector as the CMS collector presents this info instead
Expand Down Expand Up @@ -164,8 +160,8 @@ public function addFrontendCollectors()
*/
public function addBackendCollectors()
{
/** @var \Barryvdh\Debugbar\LaravelDebugbar $debugBar */
$debugBar = $this->app->make(\Barryvdh\Debugbar\LaravelDebugbar::class);
/** @var LaravelDebugbar $debugBar */
$debugBar = $this->app->make(LaravelDebugbar::class);

if (Config::get('debugbar.collectors.backend', true)) {
Event::listen('backend.page.beforeDisplay', function (BackendController $controller, $action, array $params) use ($debugBar) {
Expand All @@ -183,7 +179,7 @@ public function addBackendCollectors()
protected function registerCmsTwigExtensions()
{
$profile = new Profile;
$debugBar = $this->app->make(\Barryvdh\Debugbar\LaravelDebugbar::class);
$debugBar = $this->app->make(LaravelDebugbar::class);

Event::listen('cms.page.beforeDisplay', function ($controller, $url, $page) use ($profile, $debugBar) {
$twig = $controller->getTwig();
Expand All @@ -197,9 +193,11 @@ protected function registerCmsTwigExtensions()
}
});

// php-debugbar 2.x (shipped with Laravel Debugbar 4) no longer bundles a Twig profile
// collector, so only register one when the bridge class is actually available.
if (class_exists(\DebugBar\Bridge\NamespacedTwigProfileCollector::class)) {
$debugBar->addCollector(new \DebugBar\Bridge\NamespacedTwigProfileCollector($profile));
} else {
} elseif (class_exists(\DebugBar\Bridge\TwigProfileCollector::class)) {
$debugBar->addCollector(new \DebugBar\Bridge\TwigProfileCollector($profile));
}
}
Expand Down
19 changes: 0 additions & 19 deletions classes/JavascriptRenderer.php

This file was deleted.

24 changes: 21 additions & 3 deletions classes/ServiceProvider.php
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
<?php namespace Winter\Debugbar\Classes;

use Barryvdh\Debugbar\ServiceProvider as BaseServiceProvider;
use Fruitcake\LaravelDebugbar\ServiceProvider as BaseServiceProvider;
use Illuminate\Contracts\Events\Dispatcher;
use Illuminate\Contracts\Http\Kernel;
use Winter\Debugbar\Middleware\InjectDebugbar;

Expand All @@ -9,14 +10,31 @@
*/
class ServiceProvider extends BaseServiceProvider
{
/**
* Bootstrap the application events.
*
* Laravel Debugbar 4 no longer registers an injection middleware; instead the base
* service provider injects the Debugbar via a RequestHandled event listener. We still
* need Winter's InjectDebugbar middleware so that injection and request storage can be
* gated by the winter.debugbar permissions, so we push it onto the kernel here. The
* middleware short-circuits the base listener (by disabling the Debugbar) whenever the
* current user is not permitted to view it.
*/
public function boot(Dispatcher $events): void
{
parent::boot($events);

$this->registerMiddleware(InjectDebugbar::class);
}

/**
* Register the Debugbar Middleware
*
* @param string $middleware
*/
protected function registerMiddleware($middleware)
protected function registerMiddleware(string $middleware): void
{
$kernel = $this->app[Kernel::class];
$kernel->pushMiddleware(InjectDebugbar::class);
$kernel->pushMiddleware($middleware);
}
}
25 changes: 16 additions & 9 deletions classes/WinterDebugbar.php
Original file line number Diff line number Diff line change
@@ -1,21 +1,28 @@
<?php namespace Winter\Debugbar\Classes;

use Barryvdh\Debugbar\LaravelDebugbar;
use DebugBar\JavascriptRenderer;
use Fruitcake\LaravelDebugbar\LaravelDebugbar;

class WinterDebugbar extends LaravelDebugbar
{
/**
* Returns a winterized JavascriptRenderer for this instance
* Returns the JavascriptRenderer for this instance with Winter's styling applied.
*
* @param string $baseUrl
* @param string $basePathng
* @return JavascriptRenderer
* Laravel Debugbar 4 performs the renderer configuration inside getJavascriptRenderer(),
* so we defer to the parent to build and cache the renderer and then register Winter's
* stylesheet via the renderer's asset API (the renderer is cached, so the CSS is only
* added the first time it is built).
*/
public function getJavascriptRenderer($baseUrl = null, $basePath = null)
public function getJavascriptRenderer(?string $baseUrl = null, ?string $basePath = null): JavascriptRenderer
{
if ($this->jsRenderer === null) {
$this->jsRenderer = new JavascriptRenderer($this, $baseUrl, $basePath);
$alreadyBuilt = $this->jsRenderer !== null;

$renderer = parent::getJavascriptRenderer($baseUrl, $basePath);

if (!$alreadyBuilt) {
$renderer->addAssets(cssFiles: ['debugbar.css'], basePath: __DIR__ . '/../assets/css');
}
return $this->jsRenderer;

return $renderer;
}
}
6 changes: 3 additions & 3 deletions collectors/BackendCollector.php
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ public function __construct(Controller $controller, $action, array $params = [])
/**
* {@inheritDoc}
*/
public function collect()
public function collect(): array
{
$ajaxHandler = $this->controller->getAjaxHandler();

Expand Down Expand Up @@ -100,15 +100,15 @@ protected function getReflector($handler)
/**
* {@inheritDoc}
*/
public function getName()
public function getName(): string
{
return 'backend';
}

/**
* {@inheritDoc}
*/
public function getWidgets()
public function getWidgets(): array
{
return [
'route' => [
Expand Down
6 changes: 3 additions & 3 deletions collectors/CmsCollector.php
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ public function __construct(Controller $controller, $url, Page $page)
/**
* {@inheritDoc}
*/
public function collect()
public function collect(): array
{
$ajaxHandler = $this->controller->getAjaxHandler();

Expand Down Expand Up @@ -103,15 +103,15 @@ protected function getReflector($handler)
/**
* {@inheritDoc}
*/
public function getName()
public function getName(): string
{
return 'cms';
}

/**
* {@inheritDoc}
*/
public function getWidgets()
public function getWidgets(): array
{
return [
'route' => [
Expand Down
6 changes: 3 additions & 3 deletions collectors/ComponentsCollector.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ public function __construct(Controller $controller, Page $page, Layout $layout)
/**
* {@inheritDoc}
*/
public function collect()
public function collect(): array
{
/** @var ComponentBase[]|object $components */
$components = [];
Expand Down Expand Up @@ -58,15 +58,15 @@ protected function makeComponentDetails($componentObj): array
/**
* {@inheritDoc}
*/
public function getName()
public function getName(): string
{
return 'components';
}

/**
* {@inheritDoc}
*/
public function getWidgets()
public function getWidgets(): array
{
return [
'components' => [
Expand Down
6 changes: 3 additions & 3 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@
"source": "https://github.com/wintercms/wn-debugbar-plugin"
},
"require": {
"php": ">=7.0",
"php": ">=8.3",
"composer/installers": "~1.0",
"barryvdh/laravel-debugbar": "^3.15.0"
"fruitcake/laravel-debugbar": "^4.2"
},
"replace": {
"rainlab/debugbar-plugin": ">=1.0.5"
Expand All @@ -34,7 +34,7 @@
"installer-name": "debugbar",
"laravel": {
"dont-discover": [
"barryvdh/laravel-debugbar"
"fruitcake/laravel-debugbar"
]
}
}
Expand Down
Loading