ArjanSchouten / HtmlMinifier

src/Laravel/Command/ViewCompilerCommand.php

Doc Comments +3 added lines

@@ -146,6 +146,9 @@
         $this->table(['Minification strategy', 'Size (KB)', 'Saved (KB)', 'Size (%)'], $rows);
     }
 
+    /**
+     * @param integer $new
+     */
     private function calculateImprovementPercentage($new, $old)
     {
         return ($new - $old) / $old * 100;

Spacing +8 added lines, -8 removed lines

@@ -52,7 +52,7 @@ 
      */
     protected function setupCompiler()
     {
-        Blade::extend(function ($value, $compiler) {
+        Blade::extend(function($value, $compiler) {
             BladePlaceholder::setBladeTags($this->getBladeTags($compiler));
 
             $context = new MinifyContext(new PlaceholderContainer());
@@ -123,24 +123,24 @@ 
         $totalBytesSavedPercentage = 0;
 
         $lastReferencePoint = null;
-        $rows = $referencePoints->map(function (ReferencePoint $referencePoint) use (&$lastReferencePoint, &$totalBytesSavedPercentage) {
+        $rows = $referencePoints->map(function(ReferencePoint $referencePoint) use (&$lastReferencePoint, &$totalBytesSavedPercentage) {
             $bytesSaved = '';
             $bytesSavedPercentage = '';
             if ($lastReferencePoint != null) {
                 $bytesSaved = $lastReferencePoint->getKiloBytes() - $referencePoint->getKiloBytes();
                 $totalBytesSavedPercentage += $bytesSavedPercentage = $this->calculateImprovementPercentage($referencePoint->getKiloBytes(), $lastReferencePoint->getKiloBytes());
-                $bytesSavedPercentage = round(abs($bytesSavedPercentage),1).'%';
+                $bytesSavedPercentage = round(abs($bytesSavedPercentage), 1).'%';
             }
             $lastReferencePoint = $referencePoint;
 
-            return [$referencePoint->getName(), round($referencePoint->getKiloBytes(),1), round($bytesSaved,1), $bytesSavedPercentage];
+            return [$referencePoint->getName(), round($referencePoint->getKiloBytes(), 1), round($bytesSaved, 1), $bytesSavedPercentage];
         });
 
         $rows[] = [
             'Total',
-            round($referencePoints->last()->getKiloBytes(),1),
-            abs(round($referencePoints->last()->getKiloBytes() - $referencePoints->first()->getKiloBytes(),1)),
-            abs(round($totalBytesSavedPercentage,1)).'%'
+            round($referencePoints->last()->getKiloBytes(), 1),
+            abs(round($referencePoints->last()->getKiloBytes() - $referencePoints->first()->getKiloBytes(), 1)),
+            abs(round($totalBytesSavedPercentage, 1)).'%'
         ];
 
         $this->table(['Minification strategy', 'Size (KB)', 'Saved (KB)', 'Size (%)'], $rows);
@@ -159,7 +159,7 @@ 
     protected function getOptions()
     {
         $options = Collection::make(Options::options())
-            ->map(function (Option $option) {
+            ->map(function(Option $option) {
                 return [
                     $option->getName(),
                     null,

src/Measurements/MeasurementInterface.php

Doc Comments +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     /**
      * Get all the steps which are measured.
      *
-     * @return array
+     * @return ReferencePoint[]
      */
     public function getReferencePoints();
 }

src/Measurements/ReferencePoint.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     }
 
     /**
-     * @return float
+     * @return integer
      */
     public function getKiloBytes()
     {

src/MinifyContext.php

Doc Comments +3 added lines

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

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 +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/Laravel/HtmlMinifierServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
      */
     protected function registerBladeMinifier()
     {
-        $this->app->singleton('blade.compiler.min', function () {
+        $this->app->singleton('blade.compiler.min', function() {
             return new Minify();
         });
     }
@@ -46,7 +46,7 @@ 
      */
     protected function registerPhpMinifier()
     {
-        $this->app->singleton('php.min', function () {
+        $this->app->singleton('php.min', function() {
             return new Minify();
         });
     }

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];
             }