ArjanSchouten / HtmlMinifier

src/Minifiers/Html/BooleanAttributeMinifier.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
                 ([\'"])?                    # optional to use a quote
                 (\1|true|false|([\s>"\']))    # match the boolean attribute name again or true|false
                 \2?                         # match the quote again
-            /xi', function ($match) {
+            /xi', function($match) {
                 if (isset($match[4])) {
                     return ' '.$match[1];
                 }

src/Minifiers/Html/EmptyAttributeMinifier.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
                 \2                                              # Backreference to the matched quote
                 \s*
             /x',
-            function ($match) {
+            function($match) {
                 if ($this->isBooleanAttribute($match[1])) {
                     return Html::isLastAttribute($match[0]) ? $match[1] : $match[1].' ';
                 }

src/Minifiers/Html/JavascriptEventsMinifier.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
                 ["\']?              # Match an optional quote
                 \s*javascript:      # Match the text "javascript:" which should be removed
             /xis',
-            function ($match) {
+            function($match) {
                 return str_replace('javascript:', '', $match[0]);
             }, $context->getContents());
 

src/Minifiers/Html/RedundantAttributeMinifier.php

Spacing +3 added lines, -3 removed lines

@@ -40,8 +40,8 @@ 
      */
     public function process(MinifyContext $context)
     {
-        Collection::make($this->redundantAttributes)->each(function ($attributes, $element) use (&$context) {
-            Collection::make($attributes)->each(function ($value, $attribute) use ($element, &$context) {
+        Collection::make($this->redundantAttributes)->each(function($attributes, $element) use (&$context) {
+            Collection::make($attributes)->each(function($value, $attribute) use ($element, &$context) {
                 $contents = preg_replace_callback(
                     '/
                         '.$element.'                    # Match the given element
@@ -55,7 +55,7 @@ 
                             \s*
                         )
                     /xis',
-                    function ($match) {
+                    function($match) {
                         return $this->removeAttribute($match[0], $match[2]);
                     }, $context->getContents());
 

src/PlaceholderContainer.php

Spacing +3 added lines, -3 removed lines

@@ -79,21 +79,21 @@
      */
     protected function removeNestedPlaceholders($originalContent)
     {
-        return preg_replace_callback('/'.Constants::PLACEHOLDER_PATTERN.'/', function ($match) {
+        return preg_replace_callback('/'.Constants::PLACEHOLDER_PATTERN.'/', function($match) {
             return $this->get($match[0]);
         }, $originalContent);
     }
 
     public function getPlaceholderSize()
     {
-        return $this->keys()->sum(function ($key) {
+        return $this->keys()->sum(function($key) {
             return mb_strlen($key, '8bit');
         });
     }
 
     public function getOriginalSize()
     {
-        return $this->values()->sum(function ($value) {
+        return $this->values()->sum(function($value) {
             return mb_strlen($value, '8bit');
         });
     }

src/Placeholders/CommentPlaceholder.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
      */
     protected function setCDataPlaceholder($contents, PlaceholderContainer $placeholderContainer)
     {
-        return preg_replace_callback('/<!\[CDATA\[((?!\]\]>).)*\]\]>/s', function ($match) use ($placeholderContainer) {
+        return preg_replace_callback('/<!\[CDATA\[((?!\]\]>).)*\]\]>/s', function($match) use ($placeholderContainer) {
             return $placeholderContainer->addPlaceholder($match[0]);
         }, $contents);
     }
@@ -66,7 +66,7 @@ 
                     >                   # Match the endif with the captured dashes
                 )
             /xis',
-            function ($match) use ($placeholderContainer) {
+            function($match) use ($placeholderContainer) {
                 if (!empty(preg_replace('/\s*/', '', $match[3]))) {
                     return $placeholderContainer->addPlaceholder($match[1]).$match[3].$placeholderContainer->addPlaceholder($match[4]);
                 } else {

src/Placeholders/Php/PhpPlaceholder.php

Spacing +2 added lines, -2 removed lines

@@ -16,10 +16,10 @@
     public function process($context)
     {
         $contents = $context->getContents();
-        $contents = preg_replace_callback('/<\?=((?!\?>).)*\?>/s', function ($match) use ($context) {
+        $contents = preg_replace_callback('/<\?=((?!\?>).)*\?>/s', function($match) use ($context) {
             return $context->getPlaceholderContainer()->addPlaceholder($match[0]);
         }, $contents);
-        $contents = preg_replace_callback('/<\?php((?!\?>).)*(\?>)?/s', function ($match) use ($context) {
+        $contents = preg_replace_callback('/<\?php((?!\?>).)*(\?>)?/s', function($match) use ($context) {
             return $context->getPlaceholderContainer()->addPlaceholder($match[0]);
         }, $contents);
 

src/Placeholders/WhitespacePlaceholder.php

Spacing +4 added lines, -4 removed lines

@@ -63,7 +63,7 @@ 
                 \s+                             # Match minimal 1 whitespace between the elements
                 <('.$elementsRegex.')           # Match the start of the next inline element
             /xi',
-            function ($match) use ($placeholderContainer) {
+            function($match) use ($placeholderContainer) {
                 // Where going to respect one space between the inline elements.
                 $placeholder = $placeholderContainer->addPlaceholder(' ');
 
@@ -93,7 +93,7 @@ 
                 )
                 <('.$elementsRegex.')       # Match starting tag
             /xis',
-            function ($match) use ($placeholderContainer) {
+            function($match) use ($placeholderContainer) {
                 return $this->replaceWhitespacesInInlineElements($match[1], $placeholderContainer).'<'.$match[3];
             }, $contents);
     }
@@ -108,7 +108,7 @@ 
      */
     private function replaceWhitespacesInInlineElements($element, PlaceholderContainer $placeholderContainer)
     {
-        return preg_replace_callback('/>\s/', function ($match) use ($placeholderContainer) {
+        return preg_replace_callback('/>\s/', function($match) use ($placeholderContainer) {
             return '>'.$placeholderContainer->addPlaceholder(' ');
         }, $element);
     }
@@ -134,7 +134,7 @@ 
             ((?:(?!<\/\2>).)+)
             (<\/\2>)/xis';
 
-        return preg_replace_callback($pattern, function ($match) use ($placeholderContainer) {
+        return preg_replace_callback($pattern, function($match) use ($placeholderContainer) {
             return $match[1].$placeholderContainer->addPlaceholder($match[3]).$match[4];
         }, $contents);
     }

src/MinifyContext.php

Doc Comments +3 added lines

@@ -78,6 +78,9 @@
         $this->statistics->createReferencePoint($this->calculateInputLength($input), $keyName);
     }
 
+    /**
+     * @param string $input
+     */
     private function calculateInputLength($input)
     {
         return mb_strlen($input, '8bit') + $this->placeholderContainer->getOriginalSize() - $this->placeholderContainer->getPlaceholderSize();