Completed
Push — master ( 9cd978...fb0caf )
by Lars
01:58
created
src/voku/helper/HtmlMin.php 3 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -482,8 +482,8 @@
 block discarded – undo
482 482
   }
483 483
 
484 484
   /**
485
-   * @param $html
486
-   * @param $decodeUtf8Specials
485
+   * @param string $html
486
+   * @param boolean $decodeUtf8Specials
487 487
    *
488 488
    * @return string
489 489
    */
Please login to merge, or discard this patch.
Indentation   +48 added lines, -48 removed lines patch added patch discarded remove patch
@@ -495,72 +495,72 @@
 block discarded – undo
495 495
     return in_array($tag_name, self::$optional_end_tags)
496 496
            ||
497 497
            (
498
-               $tag_name == 'li'
498
+                $tag_name == 'li'
499 499
                &&
500 500
                (
501
-                   $nextSibling === null
501
+                    $nextSibling === null
502 502
                    ||
503 503
                    (
504
-                       $nextSibling instanceof \DOMElement
504
+                        $nextSibling instanceof \DOMElement
505 505
                        &&
506 506
                        $nextSibling->tagName == $tag_name
507
-                   )
508
-               )
509
-           )
507
+                    )
508
+                )
509
+            )
510 510
            ||
511 511
            (
512
-               $tag_name == 'p'
512
+                $tag_name == 'p'
513 513
                &&
514 514
                (
515
-                   (
516
-                       $nextSibling === null
515
+                    (
516
+                        $nextSibling === null
517 517
                        &&
518 518
                        (
519
-                           $node->parentNode !== null
519
+                            $node->parentNode !== null
520 520
                            &&
521 521
                            $node->parentNode->tagName != 'a'
522
-                       )
523
-                   )
522
+                        )
523
+                    )
524 524
                    ||
525 525
                    (
526
-                       $nextSibling instanceof \DOMElement
526
+                        $nextSibling instanceof \DOMElement
527 527
                        &&
528 528
                        in_array(
529
-                           $nextSibling->tagName,
530
-                           array(
531
-                               'address',
532
-                               'article',
533
-                               'aside',
534
-                               'blockquote',
535
-                               'dir',
536
-                               'div',
537
-                               'dl',
538
-                               'fieldset',
539
-                               'footer',
540
-                               'form',
541
-                               'h1',
542
-                               'h2',
543
-                               'h3',
544
-                               'h4',
545
-                               'h5',
546
-                               'h6',
547
-                               'header',
548
-                               'hgroup',
549
-                               'hr',
550
-                               'menu',
551
-                               'nav',
552
-                               'ol',
553
-                               'p',
554
-                               'pre',
555
-                               'section',
556
-                               'table',
557
-                               'ul',
558
-                           ),
559
-                           true
560
-                       )
561
-                   )
562
-               )
563
-           );
529
+                            $nextSibling->tagName,
530
+                            array(
531
+                                'address',
532
+                                'article',
533
+                                'aside',
534
+                                'blockquote',
535
+                                'dir',
536
+                                'div',
537
+                                'dl',
538
+                                'fieldset',
539
+                                'footer',
540
+                                'form',
541
+                                'h1',
542
+                                'h2',
543
+                                'h3',
544
+                                'h4',
545
+                                'h5',
546
+                                'h6',
547
+                                'header',
548
+                                'hgroup',
549
+                                'hr',
550
+                                'menu',
551
+                                'nav',
552
+                                'ol',
553
+                                'p',
554
+                                'pre',
555
+                                'section',
556
+                                'table',
557
+                                'ul',
558
+                            ),
559
+                            true
560
+                        )
561
+                    )
562
+                )
563
+            );
564 564
   }
565 565
 
566 566
   protected function domNodeToString(\DOMNode $node)
Please login to merge, or discard this patch.
Spacing   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -450,7 +450,7 @@  discard block
 block discarded – undo
450 450
         # http://www.whatwg.org/specs/web-apps/current-work/multipage/syntax.html#attributes-0
451 451
         $omitquotes = $attribute->value != '' && 0 == \preg_match('/["\'=<>` \t\r\n\f]+/', $attribute->value);
452 452
         $attr_val = $attribute->value;
453
-        $attrstr .= ($omitquotes ? '' : '"') . $attr_val . ($omitquotes ? '' : '"');
453
+        $attrstr .= ($omitquotes ? '' : '"').$attr_val.($omitquotes ? '' : '"');
454 454
         $attrstr .= ' ';
455 455
       }
456 456
     }
@@ -576,11 +576,11 @@  discard block
 block discarded – undo
576 576
 
577 577
       } else if ($child instanceof \DOMElement) {
578 578
 
579
-        $htmlstr .= trim('<' . $child->tagName . ' ' . $this->domNodeAttributesToString($child));
580
-        $htmlstr .= '>' . $this->domNodeToString($child);
579
+        $htmlstr .= trim('<'.$child->tagName.' '.$this->domNodeAttributesToString($child));
580
+        $htmlstr .= '>'.$this->domNodeToString($child);
581 581
 
582 582
         if (!$this->domNodeClosingTagOptional($child)) {
583
-          $htmlstr .= '</' . $child->tagName . '>';
583
+          $htmlstr .= '</'.$child->tagName.'>';
584 584
         }
585 585
 
586 586
       } else if ($child instanceof \DOMText) {
@@ -603,7 +603,7 @@  discard block
 block discarded – undo
603 603
 
604 604
       } else {
605 605
 
606
-        throw new \Exception('Error by: ' . print_r($child, true));
606
+        throw new \Exception('Error by: '.print_r($child, true));
607 607
 
608 608
       }
609 609
     }
@@ -653,7 +653,7 @@  discard block
 block discarded – undo
653 653
    */
654 654
   public function minify($html, $decodeUtf8Specials = false)
655 655
   {
656
-    $html = (string)$html;
656
+    $html = (string) $html;
657 657
     if (!isset($html[0])) {
658 658
       return '';
659 659
     }
@@ -689,10 +689,10 @@  discard block
 block discarded – undo
689 689
     // -------------------------------------------------------------------------
690 690
 
691 691
     // Remove extra white-space(s) between HTML attribute(s)
692
-    $html = (string)\preg_replace_callback(
692
+    $html = (string) \preg_replace_callback(
693 693
         '#<([^\/\s<>!]+)(?:\s+([^<>]*?)\s*|\s*)(\/?)>#',
694
-        function ($matches) {
695
-          return '<' . $matches[1] . (string)\preg_replace('#([^\s=]+)(\=([\'"]?)(.*?)\3)?(\s+|$)#s', ' $1$2', $matches[2]) . $matches[3] . '>';
694
+        function($matches) {
695
+          return '<'.$matches[1].(string) \preg_replace('#([^\s=]+)(\=([\'"]?)(.*?)\3)?(\s+|$)#s', ' $1$2', $matches[2]).$matches[3].'>';
696 696
         },
697 697
         $html
698 698
     );
@@ -700,15 +700,15 @@  discard block
 block discarded – undo
700 700
 
701 701
     if ($this->doRemoveSpacesBetweenTags === true) {
702 702
       // Remove spaces that are between > and <
703
-      $html = (string)\preg_replace('/(>) (<)/', '>$2', $html);
703
+      $html = (string) \preg_replace('/(>) (<)/', '>$2', $html);
704 704
     }
705 705
 
706 706
     // -------------------------------------------------------------------------
707 707
     // Restore protected HTML-code.
708 708
     // -------------------------------------------------------------------------
709 709
 
710
-    $html = (string)\preg_replace_callback(
711
-        '/<(?<element>' . $this->protectedChildNodesHelper . ')(?<attributes> [^>]*)?>(?<value>.*?)<\/' . $this->protectedChildNodesHelper . '>/',
710
+    $html = (string) \preg_replace_callback(
711
+        '/<(?<element>'.$this->protectedChildNodesHelper.')(?<attributes> [^>]*)?>(?<value>.*?)<\/'.$this->protectedChildNodesHelper.'>/',
712 712
         array($this, 'restoreProtectedHtml'),
713 713
         $html
714 714
     );
@@ -729,14 +729,14 @@  discard block
 block discarded – undo
729 729
 
730 730
     $html = \str_replace(
731 731
         array(
732
-            'html>' . "\n",
733
-            "\n" . '<html',
734
-            'html/>' . "\n",
735
-            "\n" . '</html',
736
-            'head>' . "\n",
737
-            "\n" . '<head',
738
-            'head/>' . "\n",
739
-            "\n" . '</head',
732
+            'html>'."\n",
733
+            "\n".'<html',
734
+            'html/>'."\n",
735
+            "\n".'</html',
736
+            'head>'."\n",
737
+            "\n".'<head',
738
+            'head/>'."\n",
739
+            "\n".'</head',
740 740
         ),
741 741
         array(
742 742
             'html>',
@@ -755,10 +755,10 @@  discard block
 block discarded – undo
755 755
     $replace = array();
756 756
     $replacement = array();
757 757
     foreach (self::$selfClosingTags as $selfClosingTag) {
758
-      $replace[] = '<' . $selfClosingTag . '/>';
759
-      $replacement[] = '<' . $selfClosingTag . '>';
760
-      $replace[] = '<' . $selfClosingTag . ' />';
761
-      $replacement[] = '<' . $selfClosingTag . '>';
758
+      $replace[] = '<'.$selfClosingTag.'/>';
759
+      $replacement[] = '<'.$selfClosingTag.'>';
760
+      $replace[] = '<'.$selfClosingTag.' />';
761
+      $replacement[] = '<'.$selfClosingTag.'>';
762 762
     }
763 763
     $html = \str_replace(
764 764
         $replace,
@@ -861,7 +861,7 @@  discard block
 block discarded – undo
861 861
     }
862 862
 
863 863
     $attrs = array();
864
-    foreach ((array)$attributes as $attrName => $attrValue) {
864
+    foreach ((array) $attributes as $attrName => $attrValue) {
865 865
 
866 866
       if (isset(self::$booleanAttributes[$attrName])) {
867 867
         continue;
@@ -940,7 +940,7 @@  discard block
 block discarded – undo
940 940
       }
941 941
 
942 942
       $this->protectedChildNodes[$counter] = $element->text();
943
-      $element->getNode()->nodeValue = '<' . $this->protectedChildNodesHelper . ' data-' . $this->protectedChildNodesHelper . '="' . $counter . '"></' . $this->protectedChildNodesHelper . '>';
943
+      $element->getNode()->nodeValue = '<'.$this->protectedChildNodesHelper.' data-'.$this->protectedChildNodesHelper.'="'.$counter.'"></'.$this->protectedChildNodesHelper.'>';
944 944
 
945 945
       ++$counter;
946 946
     }
@@ -955,11 +955,11 @@  discard block
 block discarded – undo
955 955
         continue;
956 956
       }
957 957
 
958
-      $this->protectedChildNodes[$counter] = '<!--' . $text . '-->';
958
+      $this->protectedChildNodes[$counter] = '<!--'.$text.'-->';
959 959
 
960 960
       /* @var $node \DOMComment */
961 961
       $node = $element->getNode();
962
-      $child = new \DOMText('<' . $this->protectedChildNodesHelper . ' data-' . $this->protectedChildNodesHelper . '="' . $counter . '"></' . $this->protectedChildNodesHelper . '>');
962
+      $child = new \DOMText('<'.$this->protectedChildNodesHelper.' data-'.$this->protectedChildNodesHelper.'="'.$counter.'"></'.$this->protectedChildNodesHelper.'>');
963 963
       $element->getNode()->parentNode->replaceChild($child, $node);
964 964
 
965 965
       ++$counter;
@@ -1156,7 +1156,7 @@  discard block
 block discarded – undo
1156 1156
         continue;
1157 1157
       }
1158 1158
 
1159
-      $attrValue .= \trim($class) . ' ';
1159
+      $attrValue .= \trim($class).' ';
1160 1160
     }
1161 1161
     $attrValue = \trim($attrValue);
1162 1162
 
Please login to merge, or discard this patch.