Completed
Push — master ( ecebd3...709799 )
by Lars
01:29
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.
Spacing   +30 added lines, -30 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace voku\helper;
6 6
 
@@ -452,7 +452,7 @@  discard block
 block discarded – undo
452 452
         # http://www.whatwg.org/specs/web-apps/current-work/multipage/syntax.html#attributes-0
453 453
         $omitquotes = $attribute->value != '' && 0 == \preg_match('/["\'=<>` \t\r\n\f]+/', $attribute->value);
454 454
         $attr_val = $attribute->value;
455
-        $attrstr .= ($omitquotes ? '' : '"') . $attr_val . ($omitquotes ? '' : '"');
455
+        $attrstr .= ($omitquotes ? '' : '"').$attr_val.($omitquotes ? '' : '"');
456 456
         $attrstr .= ' ';
457 457
       }
458 458
     }
@@ -578,11 +578,11 @@  discard block
 block discarded – undo
578 578
 
579 579
       } elseif ($child instanceof \DOMElement) {
580 580
 
581
-        $htmlstr .= trim('<' . $child->tagName . ' ' . $this->domNodeAttributesToString($child));
582
-        $htmlstr .= '>' . $this->domNodeToString($child);
581
+        $htmlstr .= trim('<'.$child->tagName.' '.$this->domNodeAttributesToString($child));
582
+        $htmlstr .= '>'.$this->domNodeToString($child);
583 583
 
584 584
         if (!$this->domNodeClosingTagOptional($child)) {
585
-          $htmlstr .= '</' . $child->tagName . '>';
585
+          $htmlstr .= '</'.$child->tagName.'>';
586 586
         }
587 587
 
588 588
       } elseif ($child instanceof \DOMText) {
@@ -605,7 +605,7 @@  discard block
 block discarded – undo
605 605
 
606 606
       } else {
607 607
 
608
-        throw new \Exception('Error by: ' . print_r($child, true));
608
+        throw new \Exception('Error by: '.print_r($child, true));
609 609
 
610 610
       }
611 611
     }
@@ -660,7 +660,7 @@  discard block
 block discarded – undo
660 660
    */
661 661
   public function minify($html, $decodeUtf8Specials = false): string
662 662
   {
663
-    $html = (string)$html;
663
+    $html = (string) $html;
664 664
     if (!isset($html[0])) {
665 665
       return '';
666 666
     }
@@ -696,10 +696,10 @@  discard block
 block discarded – undo
696 696
     // -------------------------------------------------------------------------
697 697
 
698 698
     // Remove extra white-space(s) between HTML attribute(s)
699
-    $html = (string)\preg_replace_callback(
699
+    $html = (string) \preg_replace_callback(
700 700
         '#<([^\/\s<>!]+)(?:\s+([^<>]*?)\s*|\s*)(\/?)>#',
701
-        function ($matches) {
702
-          return '<' . $matches[1] . (string)\preg_replace('#([^\s=]+)(\=([\'"]?)(.*?)\3)?(\s+|$)#s', ' $1$2', $matches[2]) . $matches[3] . '>';
701
+        function($matches) {
702
+          return '<'.$matches[1].(string) \preg_replace('#([^\s=]+)(\=([\'"]?)(.*?)\3)?(\s+|$)#s', ' $1$2', $matches[2]).$matches[3].'>';
703 703
         },
704 704
         $html
705 705
     );
@@ -707,15 +707,15 @@  discard block
 block discarded – undo
707 707
 
708 708
     if ($this->doRemoveSpacesBetweenTags === true) {
709 709
       // Remove spaces that are between > and <
710
-      $html = (string)\preg_replace('/(>) (<)/', '>$2', $html);
710
+      $html = (string) \preg_replace('/(>) (<)/', '>$2', $html);
711 711
     }
712 712
 
713 713
     // -------------------------------------------------------------------------
714 714
     // Restore protected HTML-code.
715 715
     // -------------------------------------------------------------------------
716 716
 
717
-    $html = (string)\preg_replace_callback(
718
-        '/<(?<element>' . $this->protectedChildNodesHelper . ')(?<attributes> [^>]*)?>(?<value>.*?)<\/' . $this->protectedChildNodesHelper . '>/',
717
+    $html = (string) \preg_replace_callback(
718
+        '/<(?<element>'.$this->protectedChildNodesHelper.')(?<attributes> [^>]*)?>(?<value>.*?)<\/'.$this->protectedChildNodesHelper.'>/',
719 719
         [$this, 'restoreProtectedHtml'],
720 720
         $html
721 721
     );
@@ -736,14 +736,14 @@  discard block
 block discarded – undo
736 736
 
737 737
     $html = \str_replace(
738 738
         [
739
-            'html>' . "\n",
740
-            "\n" . '<html',
741
-            'html/>' . "\n",
742
-            "\n" . '</html',
743
-            'head>' . "\n",
744
-            "\n" . '<head',
745
-            'head/>' . "\n",
746
-            "\n" . '</head',
739
+            'html>'."\n",
740
+            "\n".'<html',
741
+            'html/>'."\n",
742
+            "\n".'</html',
743
+            'head>'."\n",
744
+            "\n".'<head',
745
+            'head/>'."\n",
746
+            "\n".'</head',
747 747
         ],
748 748
         [
749 749
             'html>',
@@ -762,10 +762,10 @@  discard block
 block discarded – undo
762 762
     $replace = [];
763 763
     $replacement = [];
764 764
     foreach (self::$selfClosingTags as $selfClosingTag) {
765
-      $replace[] = '<' . $selfClosingTag . '/>';
766
-      $replacement[] = '<' . $selfClosingTag . '>';
767
-      $replace[] = '<' . $selfClosingTag . ' />';
768
-      $replacement[] = '<' . $selfClosingTag . '>';
765
+      $replace[] = '<'.$selfClosingTag.'/>';
766
+      $replacement[] = '<'.$selfClosingTag.'>';
767
+      $replace[] = '<'.$selfClosingTag.' />';
768
+      $replacement[] = '<'.$selfClosingTag.'>';
769 769
     }
770 770
     $html = \str_replace(
771 771
         $replace,
@@ -868,7 +868,7 @@  discard block
 block discarded – undo
868 868
     }
869 869
 
870 870
     $attrs = [];
871
-    foreach ((array)$attributes as $attrName => $attrValue) {
871
+    foreach ((array) $attributes as $attrName => $attrValue) {
872 872
 
873 873
       if (isset(self::$booleanAttributes[$attrName])) {
874 874
         continue;
@@ -947,7 +947,7 @@  discard block
 block discarded – undo
947 947
       }
948 948
 
949 949
       $this->protectedChildNodes[$counter] = $element->text();
950
-      $element->getNode()->nodeValue = '<' . $this->protectedChildNodesHelper . ' data-' . $this->protectedChildNodesHelper . '="' . $counter . '"></' . $this->protectedChildNodesHelper . '>';
950
+      $element->getNode()->nodeValue = '<'.$this->protectedChildNodesHelper.' data-'.$this->protectedChildNodesHelper.'="'.$counter.'"></'.$this->protectedChildNodesHelper.'>';
951 951
 
952 952
       ++$counter;
953 953
     }
@@ -962,11 +962,11 @@  discard block
 block discarded – undo
962 962
         continue;
963 963
       }
964 964
 
965
-      $this->protectedChildNodes[$counter] = '<!--' . $text . '-->';
965
+      $this->protectedChildNodes[$counter] = '<!--'.$text.'-->';
966 966
 
967 967
       /* @var $node \DOMComment */
968 968
       $node = $element->getNode();
969
-      $child = new \DOMText('<' . $this->protectedChildNodesHelper . ' data-' . $this->protectedChildNodesHelper . '="' . $counter . '"></' . $this->protectedChildNodesHelper . '>');
969
+      $child = new \DOMText('<'.$this->protectedChildNodesHelper.' data-'.$this->protectedChildNodesHelper.'="'.$counter.'"></'.$this->protectedChildNodesHelper.'>');
970 970
       $element->getNode()->parentNode->replaceChild($child, $node);
971 971
 
972 972
       ++$counter;
@@ -1163,7 +1163,7 @@  discard block
 block discarded – undo
1163 1163
         continue;
1164 1164
       }
1165 1165
 
1166
-      $attrValue .= \trim($class) . ' ';
1166
+      $attrValue .= \trim($class).' ';
1167 1167
     }
1168 1168
     $attrValue = \trim($attrValue);
1169 1169
 
Please login to merge, or discard this patch.
Indentation   +68 added lines, -68 removed lines patch added patch discarded remove patch
@@ -500,102 +500,102 @@
 block discarded – undo
500 500
     return \in_array($tag_name, self::$optional_end_tags, true)
501 501
            ||
502 502
            (
503
-               $tag_name == 'li'
503
+                $tag_name == 'li'
504 504
                &&
505 505
                (
506
-                   $nextSibling === null
506
+                    $nextSibling === null
507 507
                    ||
508 508
                    (
509
-                       $nextSibling instanceof \DOMElement
509
+                        $nextSibling instanceof \DOMElement
510 510
                        &&
511 511
                        $nextSibling->tagName == 'li'
512
-                   )
513
-               )
514
-           )
512
+                    )
513
+                )
514
+            )
515 515
            ||
516 516
            (
517
-               $tag_name == 'td'
517
+                $tag_name == 'td'
518 518
                &&
519 519
                (
520
-                   $nextSibling === null
520
+                    $nextSibling === null
521 521
                    ||
522 522
                    (
523
-                       $nextSibling instanceof \DOMElement
523
+                        $nextSibling instanceof \DOMElement
524 524
                        &&
525 525
                        (
526
-                           $nextSibling->tagName == 'td'
526
+                            $nextSibling->tagName == 'td'
527 527
                            ||
528 528
                            $nextSibling->tagName == 'th'
529
-                       )
530
-                   )
531
-               )
532
-           )
529
+                        )
530
+                    )
531
+                )
532
+            )
533 533
            ||
534 534
            (
535
-               $tag_name == 'p'
535
+                $tag_name == 'p'
536 536
                &&
537 537
                (
538
-                   (
539
-                       $nextSibling === null
538
+                    (
539
+                        $nextSibling === null
540 540
                        &&
541 541
                        (
542
-                           $node->parentNode !== null
542
+                            $node->parentNode !== null
543 543
                            &&
544 544
                            !\in_array(
545
-                               $node->parentNode->tagName,
546
-                               [
547
-                                   'a',
548
-                                   'audio',
549
-                                   'del',
550
-                                   'ins',
551
-                                   'map',
552
-                                   'noscript',
553
-                                   'video'
554
-                               ],
555
-                               true
556
-                           )
557
-                       )
558
-                   )
545
+                                $node->parentNode->tagName,
546
+                                [
547
+                                    'a',
548
+                                    'audio',
549
+                                    'del',
550
+                                    'ins',
551
+                                    'map',
552
+                                    'noscript',
553
+                                    'video'
554
+                                ],
555
+                                true
556
+                            )
557
+                        )
558
+                    )
559 559
                    ||
560 560
                    (
561
-                       $nextSibling instanceof \DOMElement
561
+                        $nextSibling instanceof \DOMElement
562 562
                        &&
563 563
                        \in_array(
564
-                           $nextSibling->tagName,
565
-                           [
566
-                               'address',
567
-                               'article',
568
-                               'aside',
569
-                               'blockquote',
570
-                               'dir',
571
-                               'div',
572
-                               'dl',
573
-                               'fieldset',
574
-                               'footer',
575
-                               'form',
576
-                               'h1',
577
-                               'h2',
578
-                               'h3',
579
-                               'h4',
580
-                               'h5',
581
-                               'h6',
582
-                               'header',
583
-                               'hgroup',
584
-                               'hr',
585
-                               'menu',
586
-                               'nav',
587
-                               'ol',
588
-                               'p',
589
-                               'pre',
590
-                               'section',
591
-                               'table',
592
-                               'ul',
593
-                           ],
594
-                           true
595
-                       )
596
-                   )
597
-               )
598
-           );
564
+                            $nextSibling->tagName,
565
+                            [
566
+                                'address',
567
+                                'article',
568
+                                'aside',
569
+                                'blockquote',
570
+                                'dir',
571
+                                'div',
572
+                                'dl',
573
+                                'fieldset',
574
+                                'footer',
575
+                                'form',
576
+                                'h1',
577
+                                'h2',
578
+                                'h3',
579
+                                'h4',
580
+                                'h5',
581
+                                'h6',
582
+                                'header',
583
+                                'hgroup',
584
+                                'hr',
585
+                                'menu',
586
+                                'nav',
587
+                                'ol',
588
+                                'p',
589
+                                'pre',
590
+                                'section',
591
+                                'table',
592
+                                'ul',
593
+                            ],
594
+                            true
595
+                        )
596
+                    )
597
+                )
598
+            );
599 599
   }
600 600
 
601 601
   protected function domNodeToString(\DOMNode $node): string
Please login to merge, or discard this patch.