ArjanSchouten / HtmlMinifier

src/Placeholders/WhitespacePlaceholder.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace ArjanSchouten\HtmlMinifier\Placeholders;
 
 use ArjanSchouten\HtmlMinifier\PlaceholderContainer;
-use ArjanSchouten\HtmlMinifier\Repositories\HtmlInlineElementsRepository;
 
 class WhitespacePlaceholder implements PlaceholderInterface
 {

Spacing +6 added lines, -6 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);
     }
@@ -125,7 +125,7 @@ 
 
         $pattern = '/
             (
-                <(' . $htmlTags . ')                    # Match html start tag and capture
+                <(' . $htmlTags.')                    # Match html start tag and capture
                     (?:
                         [^"\'>]*|"[^"]*"|\'[^\']*\'
                     )*                                  # Match all attributes
@@ -140,8 +140,8 @@ 
             (<\/\2>)                                    # Match end tag by back referencing the start tag
             /xis';
 
-        $contents = preg_replace_callback($pattern, function ($match) use ($placeholderContainer) {
-            return $match[1] . $placeholderContainer->addPlaceholder($match[3]) . $match[4];
+        $contents = preg_replace_callback($pattern, function($match) use ($placeholderContainer) {
+            return $match[1].$placeholderContainer->addPlaceholder($match[3]).$match[4];
         }, $contents);
 
         return $contents;

src/Minifiers/Html/AttributeQuoteMinifier.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
                             )*                              # special part of "unrolling the loop"
                     )       # use a the "unrolling the loop" technique to be able to skip escaped quotes like ="\""
                 \1?         # match the same quote symbol as matched before
-            /x', function ($match) {
+            /x', function($match) {
             return $this->minifyAttribute($match);
         }, $context->getContents()));
     }

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/CommentMinifier.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
                 <!          # search for the start of a comment
                     [^>]*   # search for everything without a ">"
                 >           # match the end of the comment
-            /xs', function ($match) {
+            /xs', function($match) {
             if (Str::contains(strtolower($match[0]), 'doctype')) {
                 return $match[0];
             }

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 {