[TASK] Cleanup some more type hints

extcode · Oct 15, 2024 · fb44159 · fb44159
1 parent 3e235e2
commit fb44159
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 41 deletions.
diff --git a/Classes/Domain/Model/Cart/BeVariant.php b/Classes/Domain/Model/Cart/BeVariant.php
@@ -619,10 +619,7 @@ public function getAdditional(string $key)
         return $this->additional[$key];
     }
 
-    /**
-     * @param string $key
-     */
-    public function setAdditional($key, mixed $value): void
+    public function setAdditional(string $key, mixed $value): void
     {
         $this->additional[$key] = $value;
     }

diff --git a/Classes/Event/CheckProductAvailabilityEvent.php b/Classes/Event/CheckProductAvailabilityEvent.php
@@ -77,9 +77,6 @@ public function setMessages(array $messages): void
         $this->messages = $messages;
     }
 
-    /**
-     * @param FlashMessage $message
-     */
     public function addMessage(FlashMessage $message): void
     {
         $this->messages[] = $message;

diff --git a/Classes/Hooks/ItemsProcFunc.php b/Classes/Hooks/ItemsProcFunc.php
@@ -33,8 +33,6 @@ public function __construct()
 
     /**
      * Itemsproc function to extend the selection of templateLayouts in the plugin
-     *
-     * @param array &$config configuration array
      */
     public function user_templateLayout(array &$config): void
     {
@@ -98,12 +96,8 @@ protected function getExtKey($listType)
 
     /**
      * Reduce the template layouts by the ones that are not allowed in given colPos
-     *
-     * @param array $templateLayouts
-     * @param int $currentColPos
-     * @return array
      */
-    protected function reduceTemplateLayouts($templateLayouts, $currentColPos)
+    protected function reduceTemplateLayouts(array $templateLayouts, int $currentColPos): array
     {
         $currentColPos = (int)$currentColPos;
         $restrictions = [];
@@ -131,11 +125,8 @@ protected function reduceTemplateLayouts($templateLayouts, $currentColPos)
 
     /**
      * Get page id, if negative, then it is a "after record"
-     *
-     * @param int $pid
-     * @return int
      */
-    protected function getPageId($pid)
+    protected function getPageId(int $pid): int
     {
         $pid = (int)$pid;
 

diff --git a/Classes/Utility/TemplateLayout.php b/Classes/Utility/TemplateLayout.php
@@ -17,13 +17,8 @@ class TemplateLayout implements SingletonInterface
 {
     /**
      * Get available template layouts for a certain page
-     *
-     * @param int $pageUid
-     * @param string $extKey
-     * @param string $pluginName
-     * @return array
      */
-    public function getAvailableTemplateLayouts($pageUid, $extKey, $pluginName)
+    public function getAvailableTemplateLayouts(int $pageUid, string $extKey, string $pluginName): array
     {
         $templateLayouts = [];
         $pluginName = GeneralUtility::camelCaseToLowerCaseUnderscored($pluginName);
@@ -57,12 +52,8 @@ public function getAvailableTemplateLayouts($pageUid, $extKey, $pluginName)
 
     /**
      * Get template layouts defined in TsConfig
-     *
-     * @param $pageUid
-     * @param string $pluginName
-     * @return array
      */
-    protected function getTemplateLayoutsFromTsConfig($pageUid, $extKey, $pluginName)
+    protected function getTemplateLayoutsFromTsConfig(int $pageUid, string $extKey, string $pluginName): array
     {
         $templateLayouts = [];
         $pagesTsConfig = BackendUtility::getPagesTSconfig($pageUid);

diff --git a/Classes/ViewHelpers/FieldNameViewHelper.php b/Classes/ViewHelpers/FieldNameViewHelper.php
@@ -51,12 +51,7 @@ public function render()
         return $fieldName;
     }
 
-    /**
-     * @param BeVariant $variant
-     *
-     * @return string
-     */
-    protected function getVariantFieldName($variant)
+    protected function getVariantFieldName(BeVariant $variant): string
     {
         $fieldName = '';
 

diff --git a/Classes/ViewHelpers/Form/IsServiceAvailableAtPriceViewHelper.php b/Classes/ViewHelpers/Form/IsServiceAvailableAtPriceViewHelper.php
@@ -38,12 +38,7 @@ public function initializeArguments(): void
         );
     }
 
-    /**
-     * @param array|null $arguments
-     * @return bool
-     * @api
-     */
-    protected static function evaluateCondition($arguments = null)
+    protected static function evaluateCondition(array $arguments = null): bool
     {
         $service = $arguments['service'];
         $price = $arguments['price'];