Remove use of PHP language features incompatible with 5.6

Author: JakeQZ

src/CSSList/CSSBlockList.php

@@ -71,7 +71,7 @@ protected function allRuleSets(array &$aResult)
      *
      * @see RuleSet->getRules()
      */
-    public function getAllValues($element = null, bool $searchInFunctionArguments = false): array
+    public function getAllValues($element = null, $searchInFunctionArguments = false)
     {
         $searchString = null;
         if ($element === null) {

tests/Unit/CSSList/CSSBlockListTest.php

@@ -20,8 +20,10 @@ final class CSSBlockListTest extends TestCase
 {
     /**
      * @test
+     *
+     * @return void
      */
-    public function getAllValuesWhenNoContentSetReturnsEmptyArray(): void
+    public function getAllValuesWhenNoContentSetReturnsEmptyArray()
     {
         $subject = new ConcreteCSSBlockList();
 
@@ -30,8 +32,10 @@ public function getAllValuesWhenNoContentSetReturnsEmptyArray(): void
 
     /**
      * @test
+     *
+     * @return void
      */
-    public function getAllValuesReturnsOneValueDirectlySetAsContent(): void
+    public function getAllValuesReturnsOneValueDirectlySetAsContent()
     {
         $subject = new ConcreteCSSBlockList();
 
@@ -50,8 +54,10 @@ public function getAllValuesReturnsOneValueDirectlySetAsContent(): void
 
     /**
      * @test
+     *
+     * @return void
      */
-    public function getAllValuesReturnsMultipleValuesDirectlySetAsContentInOneDeclarationBlock(): void
+    public function getAllValuesReturnsMultipleValuesDirectlySetAsContentInOneDeclarationBlock()
     {
         $subject = new ConcreteCSSBlockList();
 
@@ -74,8 +80,10 @@ public function getAllValuesReturnsMultipleValuesDirectlySetAsContentInOneDeclar
 
     /**
      * @test
+     *
+     * @return void
      */
-    public function getAllValuesReturnsMultipleValuesDirectlySetAsContentInMultipleDeclarationBlocks(): void
+    public function getAllValuesReturnsMultipleValuesDirectlySetAsContentInMultipleDeclarationBlocks()
     {
         $subject = new ConcreteCSSBlockList();
 
@@ -99,8 +107,10 @@ public function getAllValuesReturnsMultipleValuesDirectlySetAsContentInMultipleD
 
     /**
      * @test
+     *
+     * @return void
      */
-    public function getAllValuesReturnsValuesWithinAtRuleBlockList(): void
+    public function getAllValuesReturnsValuesWithinAtRuleBlockList()
     {
         $subject = new ConcreteCSSBlockList();
 
@@ -121,8 +131,10 @@ public function getAllValuesReturnsValuesWithinAtRuleBlockList(): void
 
     /**
      * @test
+     *
+     * @return void
      */
-    public function getAllValuesWithElementProvidedReturnsOnlyValuesWithinThatElement(): void
+    public function getAllValuesWithElementProvidedReturnsOnlyValuesWithinThatElement()
     {
         $subject = new ConcreteCSSBlockList();
 
@@ -146,8 +158,10 @@ public function getAllValuesWithElementProvidedReturnsOnlyValuesWithinThatElemen
 
     /**
      * @test
+     *
+     * @return void
      */
-    public function getAllValuesWithSearchStringProvidedReturnsOnlyValuesFromMatchingRules(): void
+    public function getAllValuesWithSearchStringProvidedReturnsOnlyValuesFromMatchingRules()
     {
         $subject = new ConcreteCSSBlockList();
 
@@ -170,8 +184,10 @@ public function getAllValuesWithSearchStringProvidedReturnsOnlyValuesFromMatchin
 
     /**
      * @test
+     *
+     * @return void
      */
-    public function getAllValuesByDefaultDoesNotReturnValuesInFunctionArguments(): void
+    public function getAllValuesByDefaultDoesNotReturnValuesInFunctionArguments()
     {
         $subject = new ConcreteCSSBlockList();
 
@@ -191,8 +207,10 @@ public function getAllValuesByDefaultDoesNotReturnValuesInFunctionArguments(): v
 
     /**
      * @test
+     *
+     * @return void
      */
-    public function getAllValuesWithSearchInFunctionArgumentsReturnsValuesInFunctionArguments(): void
+    public function getAllValuesWithSearchInFunctionArgumentsReturnsValuesInFunctionArguments()
     {
         $subject = new ConcreteCSSBlockList();
 

tests/Unit/CSSList/Fixtures/ConcreteCSSBlockList.php

@@ -9,12 +9,20 @@
 
 final class ConcreteCSSBlockList extends CSSBlockList
 {
-    public function isRootList(): bool
+    /**
+     * @return never
+     */
+    public function isRootList()
     {
         throw new \BadMethodCallException('Not implemented', 1740395831);
     }
 
-    public function render(OutputFormat $outputFormat): string
+    /**
+     * @param OutputFormat|null $outputFormat
+     *
+     * @return never
+     */
+    public function render($outputFormat)
     {
         throw new \BadMethodCallException('Not implemented', 1740395836);
     }