Incomplete PHPDoc signature (#41960)

Author: Denitz

administrator/components/com_media/src/Provider/ProviderManagerHelperTrait.php

@@ -70,10 +70,11 @@ public function getProviderManager(): ProviderManager
     /**
      * Returns a provider for the given id.
      *
-     * @return  ProviderInterface
+     * @param   string  $id
      *
-     * @throws  \Exception
+     * @return  ProviderInterface
      *
+     * @throws \Exception
      * @since   4.1.0
      */
     public function getProvider(string $id): ProviderInterface
@@ -84,10 +85,11 @@ public function getProvider(string $id): ProviderInterface
     /**
      * Return an adapter for the given name.
      *
-     * @return  AdapterInterface
+     * @param   string  $name
      *
-     * @throws  \Exception
+     * @return  AdapterInterface
      *
+     * @throws \Exception
      * @since   4.1.0
      */
     public function getAdapter(string $name): AdapterInterface
@@ -98,10 +100,11 @@ public function getAdapter(string $name): AdapterInterface
     /**
      * Returns an array with the adapter name as key and the path of the file.
      *
-     * @return  array
+     * @param   string  $path
      *
-     * @throws  \InvalidArgumentException
+     * @return  array
      *
+     * @throws \Exception
      * @since   4.1.0
      */
     protected function resolveAdapterAndPath(string $path): array

plugins/editors/tinymce/src/Provider/TinyMCEProvider.php

@@ -50,6 +50,7 @@ final class TinyMCEProvider extends AbstractEditorProvider
      * @param   Registry                 $params
      * @param   CMSApplicationInterface  $application
      * @param   DispatcherInterface      $dispatcher
+     * @param   DatabaseInterface        $database
      *
      * @since  5.0.0
      */

plugins/system/httpheaders/src/Extension/Httpheaders.php

@@ -160,6 +160,8 @@ public static function getSubscribedEvents(): array
     /**
      * The `applyHashesToCspRule` method makes sure the csp hashes are added to the csp header when enabled
      *
+     * @param   Event  $event
+     *
      * @return  void
      *
      * @since   4.0.0
@@ -235,6 +237,8 @@ public function applyHashesToCspRule(Event $event): void
     /**
      * The `setHttpHeaders` method handle the setting of the configured HTTP Headers
      *
+     * @param   Event  $event
+     *
      * @return  void
      *
      * @since   4.0.0

plugins/workflow/featuring/src/Extension/Featuring.php

@@ -180,6 +180,8 @@ protected function enhanceItemForm(Form $form, $data)
      *
      * @param   DisplayEvent  $event
      *
+     * @return  void
+     *
      * @since   4.0.0
      */
     public function onAfterDisplay(DisplayEvent $event)
@@ -195,7 +197,7 @@ public function onAfterDisplay(DisplayEvent $event)
         $singularsection = Inflector::singularize($section);
 
         if (!$this->isSupported($component . '.' . $singularsection)) {
-            return true;
+            return;
         }
 
         // List of related batch functions we need to hide
@@ -227,8 +229,6 @@ public function onAfterDisplay(DisplayEvent $event)
 		";
 
         $this->getApplication()->getDocument()->addScriptDeclaration($js);
-
-        return true;
     }
 
     /**

plugins/workflow/publishing/src/Extension/Publishing.php

@@ -194,7 +194,9 @@ protected function enhanceItemForm(Form $form, $data)
     /**
      * Manipulate the generic list view
      *
-     * @param   DisplayEvent    $event
+     * @param   DisplayEvent  $event
+     *
+     * @return  void
      *
      * @since   4.0.0
      */
@@ -211,7 +213,7 @@ public function onAfterDisplay(DisplayEvent $event)
         $singularsection = Inflector::singularize($section);
 
         if (!$this->isSupported($component . '.' . $singularsection)) {
-            return true;
+            return;
         }
 
         // That's the hard coded list from the AdminController publish method => change, when it's make dynamic in the future
@@ -246,8 +248,6 @@ public function onAfterDisplay(DisplayEvent $event)
 		";
 
         $this->getApplication()->getDocument()->addScriptDeclaration($js);
-
-        return true;
     }
 
     /**