[CLEANUP] Avoid Hungarian notation in RuleSet

Also improve some names. (This class still in in urgent need
of refactoring to improve type safety and make clear naming
easier.)

Part of #756

Author: oliverklee

src/RuleSet/RuleSet.php

@@ -25,6 +25,8 @@
 abstract class RuleSet implements Renderable, Commentable
 {
     /**
+     * the rules in this rule set, using the property or rule name as the key
+     *
      * @var array<string, Rule>
      */
     private $rules = [];
@@ -69,9 +71,9 @@ public static function parseRuleSet(ParserState $parserState, RuleSet $ruleSet):
                     $rule = Rule::parse($parserState);
                 } catch (UnexpectedTokenException $e) {
                     try {
-                        $sConsume = $parserState->consumeUntil(["\n", ';', '}'], true);
+                        $consumedText = $parserState->consumeUntil(["\n", ';', '}'], true);
                         // We need to “unfind” the matches to the end of the ruleSet as this will be matched later
-                        if ($parserState->streql(\substr($sConsume, -1), '}')) {
+                        if ($parserState->streql(\substr($consumedText, -1), '}')) {
                             $parserState->backtrack(1);
                         } else {
                             while ($parserState->comes(';')) {
@@ -101,33 +103,33 @@ public function getLineNo(): int
         return $this->lineNumber;
     }
 
-    public function addRule(Rule $rule, ?Rule $oSibling = null): void
+    public function addRule(Rule $ruleToAdd, ?Rule $sibling = null): void
     {
-        $sRule = $rule->getRule();
-        if (!isset($this->rules[$sRule])) {
-            $this->rules[$sRule] = [];
+        $propertyOrRuleName = $ruleToAdd->getRule();
+        if (!isset($this->rules[$propertyOrRuleName])) {
+            $this->rules[$propertyOrRuleName] = [];
         }
 
-        $position = \count($this->rules[$sRule]);
+        $position = \count($this->rules[$propertyOrRuleName]);
 
-        if ($oSibling !== null) {
-            $iSiblingPos = \array_search($oSibling, $this->rules[$sRule], true);
-            if ($iSiblingPos !== false) {
-                $position = $iSiblingPos;
-                $rule->setPosition($oSibling->getLineNo(), $oSibling->getColNo() - 1);
+        if ($sibling !== null) {
+            $siblingPosition = \array_search($sibling, $this->rules[$propertyOrRuleName], true);
+            if ($siblingPosition !== false) {
+                $position = $siblingPosition;
+                $ruleToAdd->setPosition($sibling->getLineNo(), $sibling->getColNo() - 1);
             }
         }
-        if ($rule->getLineNo() === 0 && $rule->getColNo() === 0) {
+        if ($ruleToAdd->getLineNo() === 0 && $ruleToAdd->getColNo() === 0) {
             //this node is added manually, give it the next best line
             $rules = $this->getRules();
-            $pos = \count($rules);
-            if ($pos > 0) {
-                $last = $rules[$pos - 1];
-                $rule->setPosition($last->getLineNo() + 1, 0);
+            $rulesCount = \count($rules);
+            if ($rulesCount > 0) {
+                $last = $rules[$rulesCount - 1];
+                $ruleToAdd->setPosition($last->getLineNo() + 1, 0);
             }
         }
 
-        \array_splice($this->rules[$sRule], $position, 0, [$rule]);
+        \array_splice($this->rules[$propertyOrRuleName], $position, 0, [$ruleToAdd]);
     }
 
     /**
@@ -138,32 +140,33 @@ public function addRule(Rule $rule, ?Rule $oSibling = null): void
      * @example $ruleSet->getRules('font-')
      *          //returns an array of all rules either beginning with font- or matching font.
      *
-     * @param Rule|string|null $mRule
+     * @param Rule|string|null $searchPattern
      *        Pattern to search for. If null, returns all rules.
      *        If the pattern ends with a dash, all rules starting with the pattern are returned
      *        as well as one matching the pattern with the dash excluded.
      *        Passing a Rule behaves like calling `getRules($mRule->getRule())`.
      *
      * @return array<int, Rule>
      */
-    public function getRules($mRule = null)
+    public function getRules($searchPattern = null)
     {
-        if ($mRule instanceof Rule) {
-            $mRule = $mRule->getRule();
+        if ($searchPattern instanceof Rule) {
+            $searchPattern = $searchPattern->getRule();
         }
         /** @var array<int, Rule> $result */
         $result = [];
-        foreach ($this->rules as $sName => $rules) {
+        foreach ($this->rules as $propertyOrRuleName => $rule) {
             // Either no search rule is given or the search rule matches the found rule exactly
             // or the search rule ends in “-” and the found rule starts with the search rule.
             if (
-                !$mRule || $sName === $mRule
+                !$searchPattern || $propertyOrRuleName === $searchPattern
                 || (
-                    \strrpos($mRule, '-') === \strlen($mRule) - \strlen('-')
-                    && (\strpos($sName, $mRule) === 0 || $sName === \substr($mRule, 0, -1))
+                    \strrpos($searchPattern, '-') === \strlen($searchPattern) - \strlen('-')
+                    && (\strpos($propertyOrRuleName, $searchPattern) === 0
+                        || $propertyOrRuleName === \substr($searchPattern, 0, -1))
                 )
             ) {
-                $result = \array_merge($result, $rules);
+                $result = \array_merge($result, $rule);
             }
         }
         \usort($result, static function (Rule $first, Rule $second): int {
@@ -172,6 +175,7 @@ public function getRules($mRule = null)
             }
             return $first->getLineNo() - $second->getLineNo();
         });
+
         return $result;
     }
 
@@ -196,18 +200,18 @@ public function setRules(array $rules): void
      * like `{ background-color: green; background-color; rgba(0, 127, 0, 0.7); }` will only yield an associative array
      * containing the rgba-valued rule while `getRules()` would yield an indexed array containing both.
      *
-     * @param Rule|string|null $mRule $mRule
+     * @param Rule|string|null $searchPattern
      *        Pattern to search for. If null, returns all rules. If the pattern ends with a dash,
      *        all rules starting with the pattern are returned as well as one matching the pattern with the dash
      *        excluded. Passing a Rule behaves like calling `getRules($mRule->getRule())`.
      *
      * @return array<string, Rule>
      */
-    public function getRulesAssoc($mRule = null)
+    public function getRulesAssoc($searchPattern = null)
     {
         /** @var array<string, Rule> $result */
         $result = [];
-        foreach ($this->getRules($mRule) as $rule) {
+        foreach ($this->getRules($searchPattern) as $rule) {
             $result[$rule->getRule()] = $rule;
         }
         return $result;
@@ -222,34 +226,35 @@ public function getRulesAssoc($mRule = null)
      * Note: this is different from pre-v.2.0 behaviour of PHP-CSS-Parser, where passing a Rule instance would
      * remove all rules with the same name. To get the old behaviour, use `removeRule($rule->getRule())`.
      *
-     * @param Rule|string|null $mRule
-     *        pattern to remove. If $mRule is null, all rules are removed. If the pattern ends in a dash,
+     * @param Rule|string|null $searchPattern
+     *        pattern to remove. If null, all rules are removed. If the pattern ends in a dash,
      *        all rules starting with the pattern are removed as well as one matching the pattern with the dash
      *        excluded. Passing a Rule behaves matches by identity.
      */
-    public function removeRule($mRule): void
+    public function removeRule($searchPattern): void
     {
-        if ($mRule instanceof Rule) {
-            $sRule = $mRule->getRule();
-            if (!isset($this->rules[$sRule])) {
+        if ($searchPattern instanceof Rule) {
+            $propertyToRemove = $searchPattern->getRule();
+            if (!isset($this->rules[$propertyToRemove])) {
                 return;
             }
-            foreach ($this->rules[$sRule] as $key => $rule) {
-                if ($rule === $mRule) {
-                    unset($this->rules[$sRule][$key]);
+            foreach ($this->rules[$propertyToRemove] as $key => $rule) {
+                if ($rule === $searchPattern) {
+                    unset($this->rules[$propertyToRemove][$key]);
                 }
             }
         } else {
-            foreach ($this->rules as $sName => $rules) {
+            foreach ($this->rules as $propertyOrRuleName => $rules) {
                 // Either no search rule is given or the search rule matches the found rule exactly
                 // or the search rule ends in “-” and the found rule starts with the search rule or equals it
                 // (without the trailing dash).
                 if (
-                    !$mRule || $sName === $mRule
-                    || (\strrpos($mRule, '-') === \strlen($mRule) - \strlen('-')
-                        && (\strpos($sName, $mRule) === 0 || $sName === \substr($mRule, 0, -1)))
+                    !$searchPattern || $propertyOrRuleName === $searchPattern
+                    || (\strrpos($searchPattern, '-') === \strlen($searchPattern) - \strlen('-')
+                        && (\strpos($propertyOrRuleName, $searchPattern) === 0
+                            || $propertyOrRuleName === \substr($searchPattern, 0, -1)))
                 ) {
-                    unset($this->rules[$sName]);
+                    unset($this->rules[$propertyOrRuleName]);
                 }
             }
         }
@@ -270,22 +275,22 @@ protected function renderRules(OutputFormat $outputFormat)
     {
         $result = '';
         $isFirst = true;
-        $oNextLevel = $outputFormat->nextLevel();
+        $nextLevelFormat = $outputFormat->nextLevel();
         foreach ($this->rules as $rules) {
             foreach ($rules as $rule) {
-                $sRendered = $oNextLevel->safely(static function () use ($rule, $oNextLevel): string {
-                    return $rule->render($oNextLevel);
+                $renderedRule = $nextLevelFormat->safely(static function () use ($rule, $nextLevelFormat): string {
+                    return $rule->render($nextLevelFormat);
                 });
-                if ($sRendered === null) {
+                if ($renderedRule === null) {
                     continue;
                 }
                 if ($isFirst) {
                     $isFirst = false;
-                    $result .= $oNextLevel->spaceBeforeRules();
+                    $result .= $nextLevelFormat->spaceBeforeRules();
                 } else {
-                    $result .= $oNextLevel->spaceBetweenRules();
+                    $result .= $nextLevelFormat->spaceBetweenRules();
                 }
-                $result .= $sRendered;
+                $result .= $renderedRule;
             }
         }