[CLEANUP] Extract method Color::shouldRenderAsHex

Resolves #852.
MyIntervals · Jan 30, 2025 · 7dc3be2 · 7dc3be2
1 parent c8a7b5f
commit 7dc3be2
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/src/Value/Color.php b/src/Value/Color.php
@@ -224,12 +224,7 @@ public function __toString(): string
 
     public function render(OutputFormat $outputFormat): string
     {
-        // Shorthand RGB color values
-        if (
-            $outputFormat->getRGBHashNotation()
-            && $this->getRealName() === 'rgb'
-            && $this->allComponentsAreNumbers()
-        ) {
+        if ($this->shouldRenderAsHex($outputFormat)) {
             return $this->renderAsHex();
         }
 
@@ -240,6 +235,14 @@ public function render(OutputFormat $outputFormat): string
         return parent::render($outputFormat);
     }
 
+    private function shouldRenderAsHex(OutputFormat $outputFormat): bool
+    {
+        return
+            $outputFormat->getRGBHashNotation()
+            && $this->getRealName() === 'rgb'
+            && $this->allComponentsAreNumbers();
+    }
+
     /**
      * The function name is a concatenation of the array keys of the components, which is passed to the constructor.
      * However, this can be changed by calling {@see CSSFunction::setName},