ISTWCMS-5544 Refactor add a warning message when deleting content

ISTWCMS-5544 Refactor using OO way to add a warning message when deleting content

src/EventSubscriber/UwDeleteFormEventSubscriber.php

@@ -2,9 +2,10 @@
 
 namespace Drupal\uw_cfg_common\EventSubscriber;
 
+use Drupal\Core\Session\AccountProxyInterface;
 use Drupal\core_event_dispatcher\Event\Form\FormAlterEvent;
 use Drupal\Core\StringTranslation\StringTranslationTrait;
-use Drupal\hook_event_dispatcher\HookEventDispatcherInterface;
+use Drupal\core_event_dispatcher\FormHookEvents;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 
 /**
@@ -14,12 +15,29 @@ class UwDeleteFormEventSubscriber implements EventSubscriberInterface {
 
   use StringTranslationTrait;
 
+  /**
+   * Current user.
+   *
+   * @var \Drupal\Core\Session\AccountProxyInterface
+   */
+  protected $currentUser;
+
+  /**
+   * Default constructor.
+   *
+   * @param \Drupal\Core\Session\AccountProxyInterface $currentUser
+   *   Current user.
+   */
+  public function __construct(AccountProxyInterface $currentUser) {
+    $this->currentUser = $currentUser;
+  }
+
   /**
    * {@inheritdoc}
    */
   public static function getSubscribedEvents() {
     return [
-      HookEventDispatcherInterface::FORM_ALTER => 'alterForm',
+      FormHookEvents::FORM_ALTER => 'alterForm',
     ];
   }
 
@@ -73,9 +91,6 @@ class UwDeleteFormEventSubscriber implements EventSubscriberInterface {
       ],
     ];
 
-    // Get current user.
-    $user = \Drupal::currentUser();
-
     // Loop all names in array.
     foreach ($names as $key => $type_names) {
       foreach ($type_names as $name) {
@@ -85,8 +100,8 @@ class UwDeleteFormEventSubscriber implements EventSubscriberInterface {
         // The user has 'delete any' and 'delete own' permissions.
         if ($key == 'ct_names') {
           if (($form['#form_id'] == 'node_' . $name . '_delete_form') &&
-            ($user->hasPermission('delete any ' . $name . ' content') ||
-              $user->hasPermission('delete own ' . $name . ' content'))) {
+            ($this->currentUser->hasPermission('delete any ' . $name . ' content') ||
+              $this->currentUser->hasPermission('delete own ' . $name . ' content'))) {
             $form['description']['#markup'] = $custom_waring_message;
             break;
           }
@@ -94,7 +109,7 @@ class UwDeleteFormEventSubscriber implements EventSubscriberInterface {
         // The user has 'delete terms' permission.
         if ($key == 'vocab_names') {
           if (($form['#form_id'] == 'taxonomy_term_' . $name . '_delete_form') &&
-            $user->hasPermission('delete terms in ' . $name)) {
+            $this->currentUser->hasPermission('delete terms in ' . $name)) {
             $form['description']['#markup'] = $custom_waring_message;
             break;
           }
@@ -102,8 +117,8 @@ class UwDeleteFormEventSubscriber implements EventSubscriberInterface {
         // The user has 'delete any' permission.
         if ($key == 'media_names') {
           if (($form['#form_id'] == 'media_' . $name . '_delete_form') &&
-            ($user->hasPermission('delete any ' . $name . ' media') ||
-              $user->hasPermission('delete own ' . $name . ' media'))) {
+            ($this->currentUser->hasPermission('delete any ' . $name . ' media') ||
+              $this->currentUser->hasPermission('delete own ' . $name . ' media'))) {
             $form['description']['#markup'] = $custom_waring_message;
             break;
           }