EWPP-4991: Use drupal code review component from openeuropa.

composer.json

@@ -15,7 +15,7 @@
         "drupal/datetime_testing": "^1.0-beta4",
         "drupal/drupal-extension": "^5.0",
         "drush/drush": "^12.4",
-        "openeuropa/code-review": "^2.0.0-alpha6",
+        "openeuropa/code-review-drupal": "^1.0.0-alpha",
         "openeuropa/task-runner-drupal-project-symlink": "^1.0",
         "phpspec/prophecy-phpunit": "^2",
         "symfony/phpunit-bridge": "^6.2"
@@ -28,6 +28,14 @@
         {
             "type": "composer",
             "url": "https://packages.drupal.org/8"
+        },
+        {
+            "type": "git",
+            "url": "https://github.com/openeuropa/DrupalDriver.git"
+        },
+        {
+            "type": "git",
+            "url": "https://github.com/openeuropa/code-review-drupal.git"
         }
     ],
     "autoload": {
@@ -65,7 +73,9 @@
             "cweagans/composer-patches": true,
             "dealerdirect/phpcodesniffer-composer-installer": true,
             "drupal/core-composer-scaffold": true,
-            "phpro/grumphp": true
+            "phpro/grumphp": true,
+            "phpro/grumphp-shim": true,
+            "phpstan/extension-installer": true
         }
     }
 }

grumphp.yml.dist

@@ -1,18 +1,12 @@
 imports:
-  - { resource: vendor/openeuropa/code-review/dist/drupal-conventions.yml }
-parameters:
-  tasks.phpcs.ignore_patterns:
-    - build/
-    - node_modules/
-    - vendor/
-  tasks.phpcs.triggered_by:
-    - inc
-    - install
-    - module
-    - php
-    - theme
-    - yml
-
+  - { resource: ./vendor/openeuropa/code-review-drupal/dist/oe-component-conventions.yml }
 grumphp:
+  tasks:
+    phpparser:
+      ignore_patterns:
+        - vendor/
   git_hook_variables:
     EXEC_GRUMPHP_COMMAND: 'docker-compose exec -T web'
+
+parameters:
+  tasks.phpstan.configuration: phpstan.neon.dist

phpstan.neon.dist

@@ -0,0 +1,2 @@
+includes:
+  - ./vendor/openeuropa/code-review-drupal/dist/phpstan.drupal.neon

src/Controller/ExtensionsController.php

@@ -63,6 +63,13 @@ class ExtensionsController extends ControllerBase {
    */
   protected $updateHookRegistry;
 
+  /**
+   * The theme extension list.
+   *
+   * @var \Drupal\Core\Extension\ThemeExtensionList
+   */
+  protected ThemeExtensionList $themeExtensionList;
+
   /**
    * ExtensionsController constructor.
    *
@@ -78,14 +85,17 @@ class ExtensionsController extends ControllerBase {
    *   The location of the manifest file.
    * @param \Drupal\Core\Update\UpdateHookRegistry $updateHookRegistry
    *   The update hook registry.
+   * @param \Drupal\Core\Extension\ThemeExtensionList $theme_extension_list
+   *   The theme extension list.
    */
-  public function __construct(ModuleExtensionList $extension_list_module, ThemeHandlerInterface $theme_handler, LoggerChannelFactoryInterface $logger_factory, EventDispatcherInterface $eventDispatcher, string $manifest_file_location, UpdateHookRegistry $updateHookRegistry) {
+  public function __construct(ModuleExtensionList $extension_list_module, ThemeHandlerInterface $theme_handler, LoggerChannelFactoryInterface $logger_factory, EventDispatcherInterface $eventDispatcher, string $manifest_file_location, UpdateHookRegistry $updateHookRegistry, ThemeExtensionList $theme_extension_list) {
     $this->moduleExtensionList = $extension_list_module;
     $this->themeHandler = $theme_handler;
     $this->logger = $logger_factory->get('dashboard_agent');
     $this->eventDispatcher = $eventDispatcher;
     $this->manifestFileLocation = $manifest_file_location;
     $this->updateHookRegistry = $updateHookRegistry;
+    $this->themeExtensionList = $theme_extension_list;
   }
 
   /**
@@ -98,7 +108,8 @@ public static function create(ContainerInterface $container) {
       $container->get('logger.factory'),
       $container->get('event_dispatcher'),
       $container->getParameter('oe_dashboard_agent.manifest_file_location'),
-      $container->get('update.update_hook_registry')
+      $container->get('update.update_hook_registry'),
+      $container->get('extension.list.theme')
     );
   }
 
@@ -122,7 +133,7 @@ public function extensions() {
 
     // Get all available themes.
     try {
-      $themes = $this->themeHandler->rebuildThemeData();
+      $themes = $this->themeExtensionList->reset()->getList();
       // Sort themes by name.
       uasort($themes, [ThemeExtensionList::class, 'sortByName']);
     }
@@ -185,7 +196,7 @@ public function extensions() {
     }
 
     // Add Drupal and PHP versions.
-    $info['drupal_version'] = \DRUPAL::VERSION;
+    $info['drupal_version'] = \Drupal::VERSION;
     $info['php_version'] = phpversion();
 
     $this->addSiteVersion($info);