[5.3] Use null coalescing assignment operator for libraries code (#44920)

Author: joomdonation

libraries/src/Button/TransitionButton.php

@@ -62,7 +62,7 @@ public function render(?int $value = null, ?int $row = null, array $options = []
     {
         $default  = $this->unknownState;
 
-        $options['tip_title'] = $options['tip_title'] ?? ($options['title'] ?? $default['title']);
+        $options['tip_title'] ??= $options['title'] ?? $default['title'];
 
         return parent::render($value, $row, $options);
     }

libraries/src/Categories/Categories.php

@@ -117,9 +117,9 @@ public function __construct($options)
         $this->_key        = isset($options['key']) && $options['key'] ? $options['key'] : 'id';
         $this->_statefield = $options['statefield'] ?? 'state';
 
-        $options['access']      = $options['access'] ?? 'true';
-        $options['published']   = $options['published'] ?? 1;
-        $options['countItems']  = $options['countItems'] ?? 0;
+        $options['access'] ??= 'true';
+        $options['published'] ??= 1;
+        $options['countItems'] ??= 0;
         $options['currentlang'] = Multilanguage::isEnabled() ? Factory::getLanguage()->getTag() : 0;
 
         $this->_options = $options;

libraries/src/Editor/Button/ButtonsRegistry.php

@@ -89,8 +89,8 @@ public function initRegistry(array $options = []): ButtonsRegistryInterface
         $this->initialised = true;
 
         $options['subject']         = $this;
-        $options['editorType']      = $options['editorType'] ?? '';
-        $options['disabledButtons'] = $options['disabledButtons'] ?? [];
+        $options['editorType'] ??= '';
+        $options['disabledButtons'] ??= [];
 
         $event      = new EditorButtonsSetupEvent('onEditorButtonsSetup', $options);
         $dispatcher = $this->getDispatcher();

libraries/src/Editor/Editor.php

@@ -97,7 +97,7 @@ public function __construct(string $editor = 'none', ?DispatcherInterface $dispa
         $this->_name = $editor;
 
         /** @var  EditorsRegistry  $registry */
-        $registry = $registry ?? Factory::getContainer()->get(EditorsRegistry::class);
+        $registry ??= Factory::getContainer()->get(EditorsRegistry::class);
 
         if ($registry->has($editor)) {
             $this->provider = $registry->get($editor);
@@ -203,9 +203,9 @@ public function initialise()
     public function display($name, $html, $width, $height, $col, $row, $buttons = true, $id = null, $asset = null, $author = null, $params = [])
     {
         if ($this->provider) {
-            $params['buttons'] = $params['buttons'] ?? $buttons;
-            $params['asset']   = $params['asset'] ?? $asset;
-            $params['author']  = $params['author'] ?? $author;
+            $params['buttons'] ??= $buttons;
+            $params['asset'] ??= $asset;
+            $params['author'] ??= $author;
             $content           = $html ?? '';
 
             return $this->provider->display($name, $content, [

libraries/src/Event/Model/PrepareDataEvent.php

@@ -45,7 +45,7 @@ class PrepareDataEvent extends ModelEvent
     public function __construct($name, array $arguments = [])
     {
         // This event has a dummy subject for now
-        $this->arguments['subject'] = $this->arguments['subject'] ?? new \stdClass();
+        $this->arguments['subject'] ??= new \stdClass();
 
         parent::__construct($name, $arguments);
 

libraries/src/Event/Module/AfterRenderModulesEvent.php

@@ -45,7 +45,7 @@ class AfterRenderModulesEvent extends ModuleEvent
     public function __construct($name, array $arguments = [])
     {
         // This event has a dummy subject for now
-        $this->arguments['subject'] = $this->arguments['subject'] ?? new \stdClass();
+        $this->arguments['subject'] ??= new \stdClass();
 
         parent::__construct($name, $arguments);
 

libraries/src/Event/Module/ModuleListEvent.php

@@ -43,7 +43,7 @@ abstract class ModuleListEvent extends ModuleEvent
     public function __construct($name, array $arguments = [])
     {
         // This event has a dummy subject for now
-        $this->arguments['subject'] = $this->arguments['subject'] ?? new \stdClass();
+        $this->arguments['subject'] ??= new \stdClass();
 
         parent::__construct($name, $arguments);
 

libraries/src/Event/Plugin/AjaxEvent.php

@@ -82,7 +82,7 @@ public function getApplication(): AbstractApplication
      */
     public function addResult($data): void
     {
-        $this->arguments['result'] = $this->arguments['result'] ?? [];
+        $this->arguments['result'] ??= [];
 
         if (\is_array($this->arguments['result'])) {
             $this->arguments['result'][] = $data;

libraries/src/Event/Result/ResultAware.php

@@ -63,7 +63,7 @@ public function addResult($data): void
         $this->typeCheckResult($data);
 
         // Append the result. We use the arguments property directly to allow this to work on immutable events.
-        $this->arguments['result']   = $this->arguments['result'] ?? [];
+        $this->arguments['result'] ??= [];
         $this->arguments['result'][] = $data;
     }
 

libraries/src/Event/View/DisplayEvent.php

@@ -58,8 +58,8 @@ public function __construct($name, array $arguments = [])
         if (!\array_key_exists('extensionName', $arguments) || !\array_key_exists('section', $arguments)) {
             $parts = explode('.', $arguments['extension']);
 
-            $arguments['extensionName'] = $arguments['extensionName'] ?? $parts[0];
-            $arguments['section']       = $arguments['section'] ?? $parts[1];
+            $arguments['extensionName'] ??= $parts[0];
+            $arguments['section'] ??= $parts[1];
         }
 
         parent::__construct($name, $arguments);