[TASK] Remove the IE hack in Rule

Author: oliverklee

CHANGELOG.md

@@ -37,6 +37,7 @@ Please also have a look at our
 
 ### Removed
 
+- Remove the IE hack in `Rule` (#995))
 - Remove `OutputFormat::level()` (#874)
 - Remove expansion of shorthand properties (#838)
 - Remove `Parser::setCharset/getCharset` (#808)

config/phpstan-baseline.neon

@@ -240,12 +240,6 @@ parameters:
 			count: 1
 			path: ../src/Rule/Rule.php
 
-		-
-			message: '#^Construct empty\(\) is not allowed\. Use more strict comparison\.$#'
-			identifier: empty.notAllowed
-			count: 1
-			path: ../src/Rule/Rule.php
-
 		-
 			message: '#^Only booleans are allowed in an if condition, Sabberworm\\CSS\\Value\\RuleValueList\|string\|null given\.$#'
 			identifier: if.condNotBoolean
@@ -264,12 +258,6 @@ parameters:
 			count: 1
 			path: ../src/Rule/Rule.php
 
-		-
-			message: '#^Parameters should have "string" types as the only types passed to this method$#'
-			identifier: typePerfect.narrowPublicClassMethodParamType
-			count: 1
-			path: ../src/Rule/Rule.php
-
 		-
 			message: '#^Call to an undefined method Sabberworm\\CSS\\OutputFormat\:\:comments\(\)\.$#'
 			identifier: method.notFound

src/Rule/Rule.php

@@ -36,11 +36,6 @@ class Rule implements Renderable, Commentable
      */
     private $isImportant = false;
 
-    /**
-     * @var array<int, int>
-     */
-    private $ieHack = [];
-
     /**
      * @var int
      */
@@ -94,7 +89,7 @@ public static function parse(ParserState $parserState): Rule
         if ($parserState->getSettings()->lenientParsing) {
             while ($parserState->comes('\\')) {
                 $parserState->consume('\\');
-                $rule->addIeHack($parserState->consume());
+                $parserState->consume();
                 $parserState->consumeWhiteSpace();
             }
         }
@@ -220,30 +215,6 @@ public function addValue($value, $type = ' '): void
         }
     }
 
-    /**
-     * @param int $modifier
-     */
-    public function addIeHack($modifier): void
-    {
-        $this->ieHack[] = $modifier;
-    }
-
-    /**
-     * @param array<int, int> $modifiers
-     */
-    public function setIeHack(array $modifiers): void
-    {
-        $this->ieHack = $modifiers;
-    }
-
-    /**
-     * @return array<int, int>
-     */
-    public function getIeHack()
-    {
-        return $this->ieHack;
-    }
-
     /**
      * @param bool $isImportant
      */
@@ -276,9 +247,6 @@ public function render(OutputFormat $outputFormat): string
         } else {
             $result .= $this->value;
         }
-        if (!empty($this->ieHack)) {
-            $result .= ' \\' . \implode('\\', $this->ieHack);
-        }
         if ($this->isImportant) {
             $result .= ' !important';
         }

tests/ParserTest.php

@@ -1060,28 +1060,6 @@ public function unexpectedTokenExceptionLineNo(): void
         }
     }
 
-    /**
-     * @test
-     */
-    public function ieHacksStrictParsing(): void
-    {
-        $this->expectException(UnexpectedTokenException::class);
-
-        // We can't strictly parse IE hacks.
-        self::parsedStructureForFile('ie-hacks', Settings::create()->beStrict());
-    }
-
-    /**
-     * @test
-     */
-    public function ieHacksParsing(): void
-    {
-        $document = self::parsedStructureForFile('ie-hacks', Settings::create()->withLenientParsing(true));
-        $expected = 'p {padding-right: .75rem \\9;background-image: none \\9;color: red \\9\\0;'
-            . 'background-color: red \\9\\0;background-color: red \\9\\0 !important;content: "red 	\\0";content: "red઼";}';
-        self::assertSame($expected, $document->render());
-    }
-
     /**
      * @depends files
      *