Skip to content

Commit 234cd57

Browse files
ACMS-4504: Update ConfigSubscriber with DI.
1 parent af754cf commit 234cd57

File tree

2 files changed

+15
-4
lines changed

2 files changed

+15
-4
lines changed

modules/acquia_cms_headless/modules/acquia_cms_headless_ui/acquia_cms_headless_ui.services.yml

+1
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ services:
33
class: 'Drupal\acquia_cms_headless_ui\EventSubscriber\ConfigSubscriber'
44
arguments:
55
- '@plugin.manager.menu.local_task'
6+
- '@config.factory'
67
tags:
78
- { name: event_subscriber }
89

modules/acquia_cms_headless/modules/acquia_cms_headless_ui/src/EventSubscriber/ConfigSubscriber.php

+14-4
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
use Drupal\Core\Config\ConfigCrudEvent;
66
use Drupal\Core\Config\ConfigEvents;
7+
use Drupal\Core\Config\ConfigFactoryInterface;
78
use Drupal\Core\Menu\LocalTaskManager;
89
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
910

@@ -24,14 +25,24 @@ final class ConfigSubscriber implements EventSubscriberInterface {
2425
*/
2526
private $localTaskManager;
2627

28+
/**
29+
* The config factory.
30+
*
31+
* @var \Drupal\Core\Config\ConfigFactoryInterface
32+
*/
33+
protected $configFactory;
34+
2735
/**
2836
* ConfigSubscriber constructor.
2937
*
3038
* @param \Drupal\Core\Menu\LocalTaskManager $local_task_manager
3139
* The local task plugin manager.
40+
* @param \Drupal\Core\Config\ConfigFactoryInterface $config_factory
41+
* The config factory.
3242
*/
33-
public function __construct(LocalTaskManager $local_task_manager) {
43+
public function __construct(LocalTaskManager $local_task_manager, ConfigFactoryInterface $config_factory) {
3444
$this->localTaskManager = $local_task_manager;
45+
$this->configFactory = $config_factory;
3546
}
3647

3748
/**
@@ -74,9 +85,8 @@ public static function getSubscribedEvents(): array {
7485
* @param array $configurations
7586
* An array of drupal configurations.
7687
*/
77-
function updatePageConfigurations(string $config_name, array $configurations) {
78-
$configFactory = \Drupal::service('config.factory');
79-
$config = $configFactory->getEditable($config_name);
88+
public function updatePageConfigurations(string $config_name, array $configurations) {
89+
$config = $this->configFactory->getEditable($config_name);
8090
$need_save = FALSE;
8191
if ($config) {
8292
foreach ($configurations as $key => $value) {

0 commit comments

Comments
 (0)