Make sure math methods extraction checks for word boundaries

Otherwise, 'max' could be confused with 'minmax' Fixes #356
matthiasmullie · Jan 6, 2021 · b88dcc6 · b88dcc6
1 parent 227f190
commit b88dcc6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/CSS.php b/src/CSS.php
@@ -685,7 +685,7 @@ protected function stripWhitespace($content)
     protected function extractMath()
     {
         $functions = array('calc', 'clamp', 'min', 'max');
-        $pattern = '/('. implode('|', $functions) .')(\(.+?)(?=$|;|})/m';
+        $pattern = '/\b('. implode('|', $functions) .')(\(.+?)(?=$|;|})/m';
 
         // PHP only supports $this inside anonymous functions since 5.4
         $minifier = $this;

diff --git a/tests/css/CSSTest.php b/tests/css/CSSTest.php
@@ -833,6 +833,12 @@ public function dataProvider()
             '.space-x-4>:not([hidden])~:not([hidden]){--tw-space-x-reverse:0;margin-right:calc(1rem * var(--tw-space-x-reverse));margin-left:calc(1rem * calc(1 - var(--tw-space-x-reverse)))}',
         );
 
+        // https://github.com/matthiasmullie/minify/issues/356
+        $tests[] = array(
+            '.testclass{ grid-template-columns:minmax(0,1fr) minmax(0,1fr) minmax(0,1fr); }',
+            '.testclass{grid-template-columns:minmax(0,1fr) minmax(0,1fr) minmax(0,1fr)}',
+        );
+
         return $tests;
     }