[CLEANUP] Rector: Add return type void to function like without any r…

…eturn (MyIntervals#597) This applies the rule AddVoidReturnTypeWhereNoReturnRector. For Details see: https://github.com/rectorphp/rector/blob/main/docs/rector_rules_overview.md#addvoidreturntypewherenoreturnrector Signed-off-by: Daniel Ziegenberg <[email protected]>
ziegenberg · Jun 21, 2024 · c69b1f8 · c69b1f8
1 parent 3163c9f
commit c69b1f8
Show file tree

Hide file tree

Showing 23 changed files with 72 additions and 193 deletions.
diff --git a/src/CSSList/CSSList.php b/src/CSSList/CSSList.php
@@ -57,12 +57,10 @@ public function __construct($iLineNo = 0)
     }
 
     /**
-     * @return void
-     *
      * @throws UnexpectedTokenException
      * @throws SourceException
      */
-    public static function parseList(ParserState $oParserState, CSSList $oList)
+    public static function parseList(ParserState $oParserState, CSSList $oList): void
     {
         $bIsRoot = $oList instanceof Document;
         if (is_string($oParserState)) {
@@ -260,10 +258,8 @@ public function getLineNo()
      * Prepends an item to the list of contents.
      *
      * @param RuleSet|CSSList|Import|Charset $oItem
-     *
-     * @return void
      */
-    public function prepend($oItem)
+    public function prepend($oItem): void
     {
         array_unshift($this->aContents, $oItem);
     }
@@ -272,10 +268,8 @@ public function prepend($oItem)
      * Appends an item to the list of contents.
      *
      * @param RuleSet|CSSList|Import|Charset $oItem
-     *
-     * @return void
      */
-    public function append($oItem)
+    public function append($oItem): void
     {
         $this->aContents[] = $oItem;
     }
@@ -286,10 +280,8 @@ public function append($oItem)
      * @param int $iOffset
      * @param int $iLength
      * @param array<int, RuleSet|CSSList|Import|Charset> $mReplacement
-     *
-     * @return void
      */
-    public function splice($iOffset, $iLength = null, $mReplacement = null)
+    public function splice($iOffset, $iLength = null, $mReplacement = null): void
     {
         array_splice($this->aContents, $iOffset, $iLength, $mReplacement);
     }
@@ -355,7 +347,7 @@ public function replace($oOldItem, $mNewItem)
     /**
      * @param array<int, RuleSet|Import|Charset|CSSList> $aContents
      */
-    public function setContents(array $aContents)
+    public function setContents(array $aContents): void
     {
         $this->aContents = [];
         foreach ($aContents as $content) {
@@ -368,10 +360,8 @@ public function setContents(array $aContents)
      *
      * @param DeclarationBlock|array<array-key, Selector>|string $mSelector the selectors to match
      * @param bool $bRemoveAll whether to stop at the first declaration block found or remove all blocks
-     *
-     * @return void
      */
-    public function removeDeclarationBlockBySelector($mSelector, $bRemoveAll = false)
+    public function removeDeclarationBlockBySelector($mSelector, $bRemoveAll = false): void
     {
         if ($mSelector instanceof DeclarationBlock) {
             $mSelector = $mSelector->getSelectors();
@@ -466,10 +456,8 @@ public function getContents()
 
     /**
      * @param array<array-key, Comment> $aComments
-     *
-     * @return void
      */
-    public function addComments(array $aComments)
+    public function addComments(array $aComments): void
     {
         $this->aComments = array_merge($this->aComments, $aComments);
     }
@@ -484,10 +472,8 @@ public function getComments()
 
     /**
      * @param array<array-key, Comment> $aComments
-     *
-     * @return void
      */
-    public function setComments(array $aComments)
+    public function setComments(array $aComments): void
     {
         $this->aComments = $aComments;
     }

diff --git a/src/CSSList/Document.php b/src/CSSList/Document.php
@@ -113,11 +113,9 @@ public function getSelectorsBySpecificity($sSpecificitySearch = null)
     /**
      * Expands all shorthand properties to their long value.
      *
-     * @return void
-     *
      * @deprecated This will be removed without substitution in version 10.0.
      */
-    public function expandShorthands()
+    public function expandShorthands(): void
     {
         foreach ($this->getAllDeclarationBlocks() as $oDeclaration) {
             $oDeclaration->expandShorthands();
@@ -127,11 +125,9 @@ public function expandShorthands()
     /**
      * Create shorthands properties whenever possible.
      *
-     * @return void
-     *
      * @deprecated This will be removed without substitution in version 10.0.
      */
-    public function createShorthands()
+    public function createShorthands(): void
     {
         foreach ($this->getAllDeclarationBlocks() as $oDeclaration) {
             $oDeclaration->createShorthands();

diff --git a/src/CSSList/KeyFrame.php b/src/CSSList/KeyFrame.php
@@ -30,7 +30,7 @@ public function __construct($iLineNo = 0)
     /**
      * @param string $vendorKeyFrame
      */
-    public function setVendorKeyFrame($vendorKeyFrame)
+    public function setVendorKeyFrame($vendorKeyFrame): void
     {
         $this->vendorKeyFrame = $vendorKeyFrame;
     }
@@ -46,7 +46,7 @@ public function getVendorKeyFrame()
     /**
      * @param string $animationName
      */
-    public function setAnimationName($animationName)
+    public function setAnimationName($animationName): void
     {
         $this->animationName = $animationName;
     }

diff --git a/src/Comment/Comment.php b/src/Comment/Comment.php
@@ -45,10 +45,8 @@ public function getLineNo()
 
     /**
      * @param string $sComment
-     *
-     * @return void
      */
-    public function setComment($sComment)
+    public function setComment($sComment): void
     {
         $this->sComment = $sComment;
     }

diff --git a/src/OutputFormat.php b/src/OutputFormat.php
@@ -274,10 +274,7 @@ public function nextLevel()
         return $this->oNextLevelFormat;
     }
 
-    /**
-     * @return void
-     */
-    public function beLenient()
+    public function beLenient(): void
     {
         $this->bIgnoreExceptions = true;
     }

diff --git a/src/Parser.php b/src/Parser.php
@@ -33,20 +33,16 @@ public function __construct($sText, Settings $oParserSettings = null, $iLineNo =
      * Sets the charset to be used if the CSS does not contain an `@charset` declaration.
      *
      * @param string $sCharset
-     *
-     * @return void
      */
-    public function setCharset($sCharset)
+    public function setCharset($sCharset): void
     {
         $this->oParserState->setCharset($sCharset);
     }
 
     /**
      * Returns the charset that is used if the CSS does not contain an `@charset` declaration.
-     *
-     * @return void
      */
-    public function getCharset()
+    public function getCharset(): void
     {
         // Note: The `return` statement is missing here. This is a bug that needs to be fixed.
         $this->oParserState->getCharset();

diff --git a/src/Parsing/Anchor.php b/src/Parsing/Anchor.php
@@ -24,10 +24,7 @@ public function __construct($iPosition, ParserState $oParserState)
         $this->oParserState = $oParserState;
     }
 
-    /**
-     * @return void
-     */
-    public function backtrack()
+    public function backtrack(): void
     {
         $this->oParserState->setPosition($this->iPosition);
     }

diff --git a/src/Parsing/ParserState.php b/src/Parsing/ParserState.php
@@ -69,10 +69,8 @@ public function __construct($sText, Settings $oParserSettings, $iLineNo = 1)
      * Sets the charset to be used if the CSS does not contain an `@charset` declaration.
      *
      * @param string $sCharset
-     *
-     * @return void
      */
-    public function setCharset($sCharset)
+    public function setCharset($sCharset): void
     {
         $this->sCharset = $sCharset;
         $this->aText = $this->strsplit($this->sText);
@@ -123,10 +121,8 @@ public function anchor(): Anchor
 
     /**
      * @param int $iPosition
-     *
-     * @return void
      */
-    public function setPosition($iPosition)
+    public function setPosition($iPosition): void
     {
         $this->iCurrentPosition = $iPosition;
     }
@@ -426,10 +422,8 @@ public function streql($sString1, $sString2, $bCaseInsensitive = true): bool
 
     /**
      * @param int $iAmount
-     *
-     * @return void
      */
-    public function backtrack($iAmount)
+    public function backtrack($iAmount): void
     {
         $this->iCurrentPosition -= $iAmount;
     }

diff --git a/src/Property/CSSNamespace.php b/src/Property/CSSNamespace.php
@@ -83,20 +83,16 @@ public function getPrefix()
 
     /**
      * @param string $mUrl
-     *
-     * @return void
      */
-    public function setUrl($mUrl)
+    public function setUrl($mUrl): void
     {
         $this->mUrl = $mUrl;
     }
 
     /**
      * @param string $sPrefix
-     *
-     * @return void
      */
-    public function setPrefix($sPrefix)
+    public function setPrefix($sPrefix): void
     {
         $this->sPrefix = $sPrefix;
     }
@@ -123,10 +119,8 @@ public function atRuleArgs(): array
 
     /**
      * @param array<array-key, Comment> $aComments
-     *
-     * @return void
      */
-    public function addComments(array $aComments)
+    public function addComments(array $aComments): void
     {
         $this->aComments = array_merge($this->aComments, $aComments);
     }
@@ -141,10 +135,8 @@ public function getComments()
 
     /**
      * @param array<array-key, Comment> $aComments
-     *
-     * @return void
      */
-    public function setComments(array $aComments)
+    public function setComments(array $aComments): void
     {
         $this->aComments = $aComments;
     }

diff --git a/src/Property/Charset.php b/src/Property/Charset.php
@@ -55,7 +55,7 @@ public function getLineNo()
      *
      * @return void
      */
-    public function setCharset($sCharset)
+    public function setCharset($sCharset): void
     {
         $sCharset = $sCharset instanceof CSSString ? $sCharset : new CSSString($sCharset);
         $this->oCharset = $sCharset;
@@ -100,7 +100,7 @@ public function atRuleArgs()
      *
      * @return void
      */
-    public function addComments(array $aComments)
+    public function addComments(array $aComments): void
     {
         $this->aComments = array_merge($this->aComments, $aComments);
     }
@@ -118,7 +118,7 @@ public function getComments()
      *
      * @return void
      */
-    public function setComments(array $aComments)
+    public function setComments(array $aComments): void
     {
         $this->aComments = $aComments;
     }

diff --git a/src/Property/Import.php b/src/Property/Import.php
@@ -54,10 +54,8 @@ public function getLineNo()
 
     /**
      * @param URL $oLocation
-     *
-     * @return void
      */
-    public function setLocation($oLocation)
+    public function setLocation($oLocation): void
     {
         $this->oLocation = $oLocation;
     }
@@ -103,10 +101,8 @@ public function atRuleArgs(): array
 
     /**
      * @param array<array-key, Comment> $aComments
-     *
-     * @return void
      */
-    public function addComments(array $aComments)
+    public function addComments(array $aComments): void
     {
         $this->aComments = array_merge($this->aComments, $aComments);
     }
@@ -121,10 +117,8 @@ public function getComments()
 
     /**
      * @param array<array-key, Comment> $aComments
-     *
-     * @return void
      */
-    public function setComments(array $aComments)
+    public function setComments(array $aComments): void
     {
         $this->aComments = $aComments;
     }

diff --git a/src/Property/Selector.php b/src/Property/Selector.php
@@ -107,7 +107,7 @@ public function getSelector()
      *
      * @return void
      */
-    public function setSelector($sSelector)
+    public function setSelector($sSelector): void
     {
         $this->sSelector = trim($sSelector);
         $this->iSpecificity = null;