Merge pull request #11 from peterkraume/ispackageactive

[BUGFIX] Avoid global state usage in Services.php
georgringer · Oct 22, 2021 · 08fa15e · 08fa15e
2 parents 91275d9 + 5683afb
commit 08fa15e
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/Configuration/Services.php b/Configuration/Services.php
@@ -4,15 +4,17 @@
 namespace GeorgRinger\Doc;
 
 use GeorgRinger\Doc\Widgets\Provider\ExtDocButtonProvider;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
 use Symfony\Component\DependencyInjection\Reference;
 use TYPO3\CMS\Core\Utility\ExtensionManagementUtility;
+use TYPO3\CMS\Dashboard\Dashboard;
 use TYPO3\CMS\Dashboard\Widgets\CtaWidget;
 
-return function (ContainerConfigurator $configurator) {
+return static function (ContainerConfigurator $configurator, ContainerBuilder $containerBuilder) {
     $services = $configurator->services();
 
-    if (ExtensionManagementUtility::isLoaded('dashboard')) {
+    if ($containerBuilder->hasDefinition(Dashboard::class)) {
         $services->set('dashboard.widget.gotoProjectDocumentation')
             ->class(CtaWidget::class)
             ->arg('$view', new Reference('dashboard.views.widget'))