Completed
Push — master ( 2a7a8f...d27882 )
by Lars
01:25
created
src/voku/helper/HtmlMin.php 3 patches
Indentation   +115 added lines, -115 removed lines patch added patch discarded remove patch
@@ -778,206 +778,206 @@  discard block
 block discarded – undo
778 778
         return \in_array($tag_name, self::$optional_end_tags, true)
779 779
                ||
780 780
                (
781
-                   $tag_name === 'li'
781
+                    $tag_name === 'li'
782 782
                    &&
783 783
                    (
784
-                       $nextSibling === null
784
+                        $nextSibling === null
785 785
                        ||
786 786
                        (
787
-                           $nextSibling instanceof \DOMElement
787
+                            $nextSibling instanceof \DOMElement
788 788
                            &&
789 789
                            $nextSibling->tagName === 'li'
790
-                       )
791
-                   )
792
-               )
790
+                        )
791
+                    )
792
+                )
793 793
                ||
794 794
                (
795
-                   $tag_name === 'rp'
795
+                    $tag_name === 'rp'
796 796
                    &&
797 797
                    (
798
-                       $nextSibling === null
798
+                        $nextSibling === null
799 799
                        ||
800 800
                        (
801
-                           $nextSibling instanceof \DOMElement
801
+                            $nextSibling instanceof \DOMElement
802 802
                            &&
803 803
                            (
804
-                               $nextSibling->tagName === 'rp'
804
+                                $nextSibling->tagName === 'rp'
805 805
                                ||
806 806
                                $nextSibling->tagName === 'rt'
807
-                           )
808
-                       )
809
-                   )
810
-               )
807
+                            )
808
+                        )
809
+                    )
810
+                )
811 811
                ||
812 812
                (
813
-                   $tag_name === 'tr'
813
+                    $tag_name === 'tr'
814 814
                    &&
815 815
                    (
816
-                       $nextSibling === null
816
+                        $nextSibling === null
817 817
                        ||
818 818
                        (
819
-                           $nextSibling instanceof \DOMElement
819
+                            $nextSibling instanceof \DOMElement
820 820
                            &&
821 821
                            $nextSibling->tagName === 'tr'
822
-                       )
823
-                   )
824
-               )
822
+                        )
823
+                    )
824
+                )
825 825
                ||
826 826
                (
827
-                   $tag_name === 'source'
827
+                    $tag_name === 'source'
828 828
                    &&
829 829
                    (
830
-                       $parent_tag_name === 'audio'
830
+                        $parent_tag_name === 'audio'
831 831
                        ||
832 832
                        $parent_tag_name === 'video'
833 833
                        ||
834 834
                        $parent_tag_name === 'picture'
835 835
                        ||
836 836
                        $parent_tag_name === 'source'
837
-                   )
837
+                    )
838 838
                    &&
839 839
                    (
840
-                       $nextSibling === null
840
+                        $nextSibling === null
841 841
                        ||
842 842
                        (
843
-                           $nextSibling instanceof \DOMElement
843
+                            $nextSibling instanceof \DOMElement
844 844
                            &&
845 845
                            $nextSibling->tagName === 'source'
846
-                       )
847
-                   )
848
-               )
846
+                        )
847
+                    )
848
+                )
849 849
                ||
850 850
                (
851
-                   (
852
-                       $tag_name === 'td'
851
+                    (
852
+                        $tag_name === 'td'
853 853
                        ||
854 854
                        $tag_name === 'th'
855
-                   )
855
+                    )
856 856
                    &&
857 857
                    (
858
-                       $nextSibling === null
858
+                        $nextSibling === null
859 859
                        ||
860 860
                        (
861
-                           $nextSibling instanceof \DOMElement
861
+                            $nextSibling instanceof \DOMElement
862 862
                            &&
863 863
                            (
864
-                               $nextSibling->tagName === 'td'
864
+                                $nextSibling->tagName === 'td'
865 865
                                ||
866 866
                                $nextSibling->tagName === 'th'
867
-                           )
868
-                       )
869
-                   )
870
-               )
867
+                            )
868
+                        )
869
+                    )
870
+                )
871 871
                ||
872 872
                (
873
-                   (
874
-                       $tag_name === 'dd'
873
+                    (
874
+                        $tag_name === 'dd'
875 875
                        ||
876 876
                        $tag_name === 'dt'
877
-                   )
877
+                    )
878 878
                    &&
879 879
                    (
880
-                       (
881
-                           $nextSibling === null
880
+                        (
881
+                            $nextSibling === null
882 882
                            &&
883 883
                            $tag_name === 'dd'
884
-                       )
884
+                        )
885 885
                        ||
886 886
                        (
887
-                           $nextSibling instanceof \DOMElement
887
+                            $nextSibling instanceof \DOMElement
888 888
                            &&
889 889
                            (
890
-                               $nextSibling->tagName === 'dd'
890
+                                $nextSibling->tagName === 'dd'
891 891
                                ||
892 892
                                $nextSibling->tagName === 'dt'
893
-                           )
894
-                       )
895
-                   )
896
-               )
893
+                            )
894
+                        )
895
+                    )
896
+                )
897 897
                ||
898 898
                (
899
-                   $tag_name === 'option'
899
+                    $tag_name === 'option'
900 900
                    &&
901 901
                    (
902
-                       $nextSibling === null
902
+                        $nextSibling === null
903 903
                        ||
904 904
                        (
905
-                           $nextSibling instanceof \DOMElement
905
+                            $nextSibling instanceof \DOMElement
906 906
                            &&
907 907
                            (
908
-                               $nextSibling->tagName === 'option'
908
+                                $nextSibling->tagName === 'option'
909 909
                                ||
910 910
                                $nextSibling->tagName === 'optgroup'
911
-                           )
912
-                       )
913
-                   )
914
-               )
911
+                            )
912
+                        )
913
+                    )
914
+                )
915 915
                ||
916 916
                (
917
-                   $tag_name === 'p'
917
+                    $tag_name === 'p'
918 918
                    &&
919 919
                    (
920
-                       (
921
-                           $nextSibling === null
920
+                        (
921
+                            $nextSibling === null
922 922
                            &&
923 923
                            (
924
-                               $node->parentNode !== null
924
+                                $node->parentNode !== null
925 925
                                &&
926 926
                                !\in_array(
927
-                                   $node->parentNode->nodeName,
928
-                                   [
929
-                                       'a',
930
-                                       'audio',
931
-                                       'del',
932
-                                       'ins',
933
-                                       'map',
934
-                                       'noscript',
935
-                                       'video',
936
-                                   ],
937
-                                   true
938
-                               )
939
-                           )
940
-                       )
927
+                                    $node->parentNode->nodeName,
928
+                                    [
929
+                                        'a',
930
+                                        'audio',
931
+                                        'del',
932
+                                        'ins',
933
+                                        'map',
934
+                                        'noscript',
935
+                                        'video',
936
+                                    ],
937
+                                    true
938
+                                )
939
+                            )
940
+                        )
941 941
                        ||
942 942
                        (
943
-                           $nextSibling instanceof \DOMElement
943
+                            $nextSibling instanceof \DOMElement
944 944
                            &&
945 945
                            \in_array(
946
-                               $nextSibling->tagName,
947
-                               [
948
-                                   'address',
949
-                                   'article',
950
-                                   'aside',
951
-                                   'blockquote',
952
-                                   'dir',
953
-                                   'div',
954
-                                   'dl',
955
-                                   'fieldset',
956
-                                   'footer',
957
-                                   'form',
958
-                                   'h1',
959
-                                   'h2',
960
-                                   'h3',
961
-                                   'h4',
962
-                                   'h5',
963
-                                   'h6',
964
-                                   'header',
965
-                                   'hgroup',
966
-                                   'hr',
967
-                                   'menu',
968
-                                   'nav',
969
-                                   'ol',
970
-                                   'p',
971
-                                   'pre',
972
-                                   'section',
973
-                                   'table',
974
-                                   'ul',
975
-                               ],
976
-                               true
977
-                           )
978
-                       )
979
-                   )
980
-               );
946
+                                $nextSibling->tagName,
947
+                                [
948
+                                    'address',
949
+                                    'article',
950
+                                    'aside',
951
+                                    'blockquote',
952
+                                    'dir',
953
+                                    'div',
954
+                                    'dl',
955
+                                    'fieldset',
956
+                                    'footer',
957
+                                    'form',
958
+                                    'h1',
959
+                                    'h2',
960
+                                    'h3',
961
+                                    'h4',
962
+                                    'h5',
963
+                                    'h6',
964
+                                    'header',
965
+                                    'hgroup',
966
+                                    'hr',
967
+                                    'menu',
968
+                                    'nav',
969
+                                    'ol',
970
+                                    'p',
971
+                                    'pre',
972
+                                    'section',
973
+                                    'table',
974
+                                    'ul',
975
+                                ],
976
+                                true
977
+                            )
978
+                        )
979
+                    )
980
+                );
981 981
     }
982 982
 
983 983
     protected function domNodeToString(\DOMNode $node): string
@@ -1009,9 +1009,9 @@  discard block
 block discarded – undo
1009 1009
                     }
1010 1010
 
1011 1011
                     $html .= '<!DOCTYPE ' . $child->name . ''
1012
-                             . ($child->publicId ? ' ' . $tmpTypePublic . ' "' . $child->publicId . '"' : '')
1013
-                             . ($child->systemId ? ' ' . $tmpTypeSystem . ' "' . $child->systemId . '"' : '')
1014
-                             . '>';
1012
+                              . ($child->publicId ? ' ' . $tmpTypePublic . ' "' . $child->publicId . '"' : '')
1013
+                              . ($child->systemId ? ' ' . $tmpTypeSystem . ' "' . $child->systemId . '"' : '')
1014
+                              . '>';
1015 1015
                 }
1016 1016
             } elseif ($child instanceof \DOMElement) {
1017 1017
                 $html .= \rtrim('<' . $child->tagName . ' ' . $this->domNodeAttributesToString($child));
Please login to merge, or discard this patch.
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1187,7 +1187,7 @@
 block discarded – undo
1187 1187
     /**
1188 1188
      * @param \DOMNode $node
1189 1189
      *
1190
-     * @return \DOMNode|null
1190
+     * @return null|\DOMElement
1191 1191
      */
1192 1192
     protected function getNextSiblingOfTypeDOMElement(\DOMNode $node)
1193 1193
     {
Please login to merge, or discard this patch.
Spacing   +28 added lines, -28 removed lines patch added patch discarded remove patch
@@ -572,7 +572,7 @@  discard block
 block discarded – undo
572 572
                     $attr_val = $attribute->value;
573 573
                 }
574 574
 
575
-                $attr_str .= ($omit_quotes ? '' : $quoteTmp) . $attr_val . ($omit_quotes ? '' : $quoteTmp);
575
+                $attr_str .= ($omit_quotes ? '' : $quoteTmp).$attr_val.($omit_quotes ? '' : $quoteTmp);
576 576
                 $attr_str .= ' ';
577 577
             }
578 578
         }
@@ -869,21 +869,21 @@  discard block
 block discarded – undo
869 869
                         $tmpTypePublic = 'PUBLIC';
870 870
                     }
871 871
 
872
-                    $html .= '<!DOCTYPE ' . $child->name . ''
873
-                             . ($child->publicId ? ' ' . $tmpTypePublic . ' "' . $child->publicId . '"' : '')
874
-                             . ($child->systemId ? ' ' . $tmpTypeSystem . ' "' . $child->systemId . '"' : '')
872
+                    $html .= '<!DOCTYPE '.$child->name.''
873
+                             . ($child->publicId ? ' '.$tmpTypePublic.' "'.$child->publicId.'"' : '')
874
+                             . ($child->systemId ? ' '.$tmpTypeSystem.' "'.$child->systemId.'"' : '')
875 875
                              . '>';
876 876
                 }
877 877
             } elseif ($child instanceof \DOMElement) {
878
-                $html .= \rtrim('<' . $child->tagName . ' ' . $this->domNodeAttributesToString($child));
879
-                $html .= '>' . $this->domNodeToString($child);
878
+                $html .= \rtrim('<'.$child->tagName.' '.$this->domNodeAttributesToString($child));
879
+                $html .= '>'.$this->domNodeToString($child);
880 880
 
881 881
                 if (
882 882
                     !$this->doRemoveOmittedHtmlTags
883 883
                     ||
884 884
                     !$this->domNodeClosingTagOptional($child)
885 885
                 ) {
886
-                    $html .= '</' . $child->tagName . '>';
886
+                    $html .= '</'.$child->tagName.'>';
887 887
                 }
888 888
 
889 889
                 if (!$this->doRemoveWhitespaceAroundTags) {
@@ -947,7 +947,7 @@  discard block
 block discarded – undo
947 947
                     $html .= $child->wholeText;
948 948
                 }
949 949
             } elseif ($child instanceof \DOMComment) {
950
-                $html .= '<!--' . $child->textContent . '-->';
950
+                $html .= '<!--'.$child->textContent.'-->';
951 951
             }
952 952
         }
953 953
 
@@ -1155,8 +1155,8 @@  discard block
 block discarded – undo
1155 1155
         if (\strpos($html, ' ') !== false) {
1156 1156
             $html = (string) \preg_replace_callback(
1157 1157
                 '#<([^/\s<>!]+)(?:\s+([^<>]*?)\s*|\s*)(/?)>#',
1158
-                static function ($matches) {
1159
-                    return '<' . $matches[1] . \preg_replace('#([^\s=]+)(=([\'"]?)(.*?)\3)?(\s+|$)#su', ' $1$2', $matches[2]) . $matches[3] . '>';
1158
+                static function($matches) {
1159
+                    return '<'.$matches[1].\preg_replace('#([^\s=]+)(=([\'"]?)(.*?)\3)?(\s+|$)#su', ' $1$2', $matches[2]).$matches[3].'>';
1160 1160
                 },
1161 1161
                 $html
1162 1162
             );
@@ -1176,7 +1176,7 @@  discard block
 block discarded – undo
1176 1176
 
1177 1177
         if (\strpos($html, $this->protectedChildNodesHelper) !== false) {
1178 1178
             $html = (string) \preg_replace_callback(
1179
-                '/<(?<element>' . $this->protectedChildNodesHelper . ')(?<attributes> [^>]*)?>(?<value>.*?)<\/' . $this->protectedChildNodesHelper . '>/',
1179
+                '/<(?<element>'.$this->protectedChildNodesHelper.')(?<attributes> [^>]*)?>(?<value>.*?)<\/'.$this->protectedChildNodesHelper.'>/',
1180 1180
                 [$this, 'restoreProtectedHtml'],
1181 1181
                 $html
1182 1182
             );
@@ -1196,14 +1196,14 @@  discard block
 block discarded – undo
1196 1196
 
1197 1197
         $html = \str_replace(
1198 1198
             [
1199
-                'html>' . "\n",
1200
-                "\n" . '<html',
1201
-                'html/>' . "\n",
1202
-                "\n" . '</html',
1203
-                'head>' . "\n",
1204
-                "\n" . '<head',
1205
-                'head/>' . "\n",
1206
-                "\n" . '</head',
1199
+                'html>'."\n",
1200
+                "\n".'<html',
1201
+                'html/>'."\n",
1202
+                "\n".'</html',
1203
+                'head>'."\n",
1204
+                "\n".'<head',
1205
+                'head/>'."\n",
1206
+                "\n".'</head',
1207 1207
             ],
1208 1208
             [
1209 1209
                 'html>',
@@ -1222,11 +1222,11 @@  discard block
 block discarded – undo
1222 1222
         $replace = [];
1223 1223
         $replacement = [];
1224 1224
         foreach (self::$selfClosingTags as $selfClosingTag) {
1225
-            $replace[] = '<' . $selfClosingTag . '/>';
1226
-            $replacement[] = '<' . $selfClosingTag . '>';
1227
-            $replace[] = '<' . $selfClosingTag . ' />';
1228
-            $replacement[] = '<' . $selfClosingTag . '>';
1229
-            $replace[] = '></' . $selfClosingTag . '>';
1225
+            $replace[] = '<'.$selfClosingTag.'/>';
1226
+            $replacement[] = '<'.$selfClosingTag.'>';
1227
+            $replace[] = '<'.$selfClosingTag.' />';
1228
+            $replacement[] = '<'.$selfClosingTag.'>';
1229
+            $replace[] = '></'.$selfClosingTag.'>';
1230 1230
             $replacement[] = '>';
1231 1231
         }
1232 1232
         $html = \str_replace(
@@ -1417,7 +1417,7 @@  discard block
 block discarded – undo
1417 1417
             $this->protectedChildNodes[$this->protected_tags_counter] = $element->parentNode()->innerHtml();
1418 1418
             $parentNode = $element->getNode()->parentNode;
1419 1419
             if ($parentNode !== null) {
1420
-                $parentNode->nodeValue = '<' . $this->protectedChildNodesHelper . ' data-' . $this->protectedChildNodesHelper . '="' . $this->protected_tags_counter . '"></' . $this->protectedChildNodesHelper . '>';
1420
+                $parentNode->nodeValue = '<'.$this->protectedChildNodesHelper.' data-'.$this->protectedChildNodesHelper.'="'.$this->protected_tags_counter.'"></'.$this->protectedChildNodesHelper.'>';
1421 1421
             }
1422 1422
 
1423 1423
             ++$this->protected_tags_counter;
@@ -1451,7 +1451,7 @@  discard block
 block discarded – undo
1451 1451
             }
1452 1452
 
1453 1453
             $this->protectedChildNodes[$this->protected_tags_counter] = $element->innerhtml;
1454
-            $element->getNode()->nodeValue = '<' . $this->protectedChildNodesHelper . ' data-' . $this->protectedChildNodesHelper . '="' . $this->protected_tags_counter . '"></' . $this->protectedChildNodesHelper . '>';
1454
+            $element->getNode()->nodeValue = '<'.$this->protectedChildNodesHelper.' data-'.$this->protectedChildNodesHelper.'="'.$this->protected_tags_counter.'"></'.$this->protectedChildNodesHelper.'>';
1455 1455
 
1456 1456
             ++$this->protected_tags_counter;
1457 1457
         }
@@ -1468,11 +1468,11 @@  discard block
 block discarded – undo
1468 1468
                 continue;
1469 1469
             }
1470 1470
 
1471
-            $this->protectedChildNodes[$this->protected_tags_counter] = '<!--' . $text . '-->';
1471
+            $this->protectedChildNodes[$this->protected_tags_counter] = '<!--'.$text.'-->';
1472 1472
 
1473 1473
             /* @var $node \DOMComment */
1474 1474
             $node = $element->getNode();
1475
-            $child = new \DOMText('<' . $this->protectedChildNodesHelper . ' data-' . $this->protectedChildNodesHelper . '="' . $this->protected_tags_counter . '"></' . $this->protectedChildNodesHelper . '>');
1475
+            $child = new \DOMText('<'.$this->protectedChildNodesHelper.' data-'.$this->protectedChildNodesHelper.'="'.$this->protected_tags_counter.'"></'.$this->protectedChildNodesHelper.'>');
1476 1476
             $parentNode = $element->getNode()->parentNode;
1477 1477
             if ($parentNode !== null) {
1478 1478
                 $parentNode->replaceChild($child, $node);
Please login to merge, or discard this patch.
src/voku/helper/HtmlMinDomObserverOptimizeAttributes.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -264,7 +264,7 @@  discard block
 block discarded – undo
264 264
                 $attrName === 'action'
265 265
             )
266 266
         ) {
267
-            $attrValue = \str_replace($scheme . '://', '//', $attrValue);
267
+            $attrValue = \str_replace($scheme.'://', '//', $attrValue);
268 268
         }
269 269
 
270 270
         return $attrValue;
@@ -293,7 +293,7 @@  discard block
 block discarded – undo
293 293
                 continue;
294 294
             }
295 295
 
296
-            $attrValue .= \trim($class) . ' ';
296
+            $attrValue .= \trim($class).' ';
297 297
         }
298 298
 
299 299
         return \trim($attrValue);
Please login to merge, or discard this patch.