Completed
Pull Request — master (#244)
by
unknown
02:21
created
classes/external/php/yui-php-cssmin-bundled/Minifier.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -72,9 +72,9 @@  discard block
 block discarded – undo
72 72
     public function __construct($raisePhpLimits = true)
73 73
     {
74 74
         $this->raisePhpLimits = (bool) $raisePhpLimits;
75
-        $this->memoryLimit = 128 * 1048576; // 128MB in bytes
76
-        $this->pcreBacktrackLimit = 1000 * 1000;
77
-        $this->pcreRecursionLimit = 500 * 1000;
75
+        $this->memoryLimit = 128*1048576; // 128MB in bytes
76
+        $this->pcreBacktrackLimit = 1000*1000;
77
+        $this->pcreRecursionLimit = 500*1000;
78 78
         $this->hexToNamedColorsMap = Colors::getHexToNamedMap();
79 79
         $this->namedToHexColorsMap = Colors::getNamedToHexMap();
80 80
         $this->namedToHexColorsRegex = sprintf(
@@ -176,8 +176,8 @@  discard block
 block discarded – undo
176 176
      */
177 177
     private function setShortenZeroValuesRegexes()
178 178
     {
179
-        $zeroRegex = '0'. $this->unitsGroupRegex;
180
-        $numOrPosRegex = '('. $this->numRegex .'|top|left|bottom|right|center) ';
179
+        $zeroRegex = '0'.$this->unitsGroupRegex;
180
+        $numOrPosRegex = '('.$this->numRegex.'|top|left|bottom|right|center) ';
181 181
         $oneZeroSafeProperties = array(
182 182
             '(?:line-)?height',
183 183
             '(?:(?:min|max)-)?width',
@@ -195,14 +195,14 @@  discard block
 block discarded – undo
195 195
         );
196 196
 
197 197
         // First zero regex
198
-        $regex = '/(^|;)('. implode('|', $oneZeroSafeProperties) .'):%s/Si';
198
+        $regex = '/(^|;)('.implode('|', $oneZeroSafeProperties).'):%s/Si';
199 199
         $this->shortenOneZeroesRegex = sprintf($regex, $zeroRegex);
200 200
 
201 201
         // Multiple zeroes regexes
202 202
         $regex = '/(^|;)(margin|padding|border-(?:width|radius)|background-position):%s/Si';
203
-        $this->shortenTwoZeroesRegex = sprintf($regex, $numOrPosRegex . $zeroRegex);
204
-        $this->shortenThreeZeroesRegex = sprintf($regex, $numOrPosRegex . $numOrPosRegex . $zeroRegex);
205
-        $this->shortenFourZeroesRegex = sprintf($regex, $numOrPosRegex . $numOrPosRegex . $numOrPosRegex . $zeroRegex);
203
+        $this->shortenTwoZeroesRegex = sprintf($regex, $numOrPosRegex.$zeroRegex);
204
+        $this->shortenThreeZeroesRegex = sprintf($regex, $numOrPosRegex.$numOrPosRegex.$zeroRegex);
205
+        $this->shortenFourZeroesRegex = sprintf($regex, $numOrPosRegex.$numOrPosRegex.$numOrPosRegex.$zeroRegex);
206 206
     }
207 207
 
208 208
     /**
@@ -389,7 +389,7 @@  discard block
 block discarded – undo
389 389
             $dataStartIndex = $matchStartIndex + 4; // url( length
390 390
             $searchOffset = $matchStartIndex + strlen($m[0][0]);
391 391
             $terminator = $m[1][0]; // ', " or empty (not quoted)
392
-            $terminatorRegex = '/(?<!\\\\)'. (strlen($terminator) === 0 ? '' : $terminator.'\s*') .'(\))/S';
392
+            $terminatorRegex = '/(?<!\\\\)'.(strlen($terminator) === 0 ? '' : $terminator.'\s*').'(\))/S';
393 393
 
394 394
             $ret .= substr($css, $substrOffset, $matchStartIndex - $substrOffset);
395 395
 
@@ -404,7 +404,7 @@  discard block
 block discarded – undo
404 404
                     $token = preg_replace('/\s+/S', '', $token);
405 405
                 }
406 406
 
407
-                $ret .= 'url('. $this->registerPreservedToken(trim($token)) .')';
407
+                $ret .= 'url('.$this->registerPreservedToken(trim($token)).')';
408 408
             // No end terminator found, re-add the whole match. Should we throw/warn here?
409 409
             } else {
410 410
                 $ret .= substr($css, $matchStartIndex, $searchOffset - $matchStartIndex);
@@ -425,7 +425,7 @@  discard block
 block discarded – undo
425 425
      */
426 426
     private function processCommentsCallback($matches)
427 427
     {
428
-        return '/*'. $this->registerCommentToken($matches[1]) .'*/';
428
+        return '/*'.$this->registerCommentToken($matches[1]).'*/';
429 429
     }
430 430
 
431 431
     /**
@@ -435,7 +435,7 @@  discard block
 block discarded – undo
435 435
      */
436 436
     private function processOldIeSpecificMatrixDefinitionCallback($matches)
437 437
     {
438
-        return 'filter:progid:DXImageTransform.Microsoft.Matrix('. $this->registerPreservedToken($matches[1]) .')';
438
+        return 'filter:progid:DXImageTransform.Microsoft.Matrix('.$this->registerPreservedToken($matches[1]).')';
439 439
     }
440 440
 
441 441
     /**
@@ -458,7 +458,7 @@  discard block
 block discarded – undo
458 458
         // minify alpha opacity in filter strings
459 459
         $match = str_ireplace('progid:DXImageTransform.Microsoft.Alpha(Opacity=', 'alpha(opacity=', $match);
460 460
 
461
-        return $quote . $this->registerPreservedToken($match) . $quote;
461
+        return $quote.$this->registerPreservedToken($match).$quote;
462 462
     }
463 463
 
464 464
     /**
@@ -469,7 +469,7 @@  discard block
 block discarded – undo
469 469
      */
470 470
     private function processImportUnquotedUrlAtRulesCallback($matches)
471 471
     {
472
-        return '@import url('. $this->registerPreservedToken($matches[1]) .')'. $matches[2];
472
+        return '@import url('.$this->registerPreservedToken($matches[1]).')'.$matches[2];
473 473
     }
474 474
 
475 475
     /**
@@ -480,7 +480,7 @@  discard block
 block discarded – undo
480 480
     private function processComments($css)
481 481
     {
482 482
         foreach ($this->comments as $commentId => $comment) {
483
-            $commentIdString = '/*'. $commentId .'*/';
483
+            $commentIdString = '/*'.$commentId.'*/';
484 484
 
485 485
             // ! in the first position of the comment means preserve
486 486
             // so push to the preserved tokens keeping the !
@@ -531,7 +531,7 @@  discard block
 block discarded – undo
531 531
             $blockEndPos = strpos($css, '}', $blockStartPos);
532 532
             // When ending curly brace is missing, let's
533 533
             // behave like there was one at the end of the block...
534
-            if ( false === $blockEndPos ) {
534
+            if (false === $blockEndPos) {
535 535
                 $blockEndPos = strlen($css) - 1;
536 536
             }
537 537
             $nextBlockStartPos = strpos($css, '{', $blockStartPos + 1);
@@ -543,7 +543,7 @@  discard block
 block discarded – undo
543 543
             } else {
544 544
                 $ruleBody = substr($css, $blockStartPos + 1, $blockEndPos - $blockStartPos - 1);
545 545
                 $ruleBodyToken = $this->registerRuleBodyToken($this->processRuleBody($ruleBody));
546
-                $ret .= '{'. $ruleBodyToken .'}';
546
+                $ret .= '{'.$ruleBodyToken.'}';
547 547
                 $searchOffset = $blockEndPos + 1;
548 548
             }
549 549
 
@@ -643,8 +643,8 @@  discard block
 block discarded – undo
643 643
         $body = preg_replace('/([ :,(])\+(\.?\d+)/S', '$1$2', $body);
644 644
 
645 645
         // shorten ms to s
646
-        $body = preg_replace_callback('/([ :,(])(-?)(\d{3,})ms/Si', function ($matches) {
647
-            return $matches[1] . $matches[2] . ((int) $matches[3] / 1000) .'s';
646
+        $body = preg_replace_callback('/([ :,(])(-?)(\d{3,})ms/Si', function($matches) {
647
+            return $matches[1].$matches[2].((int) $matches[3]/1000).'s';
648 648
         }, $body);
649 649
 
650 650
         // Remove leading zeros from integer and float numbers.
@@ -756,13 +756,13 @@  discard block
 block discarded – undo
756 756
         $css = preg_replace('/::(before|after|first-(?:line|letter))(\{|,)/Si', ':$1$2', $css);
757 757
 
758 758
         // Retain space for special IE6 cases
759
-        $css = preg_replace_callback('/:first-(line|letter)(\{|,)/Si', function ($matches) {
760
-            return ':first-'. strtolower($matches[1]) .' '. $matches[2];
759
+        $css = preg_replace_callback('/:first-(line|letter)(\{|,)/Si', function($matches) {
760
+            return ':first-'.strtolower($matches[1]).' '.$matches[2];
761 761
         }, $css);
762 762
 
763 763
         // Find a fraction that may used in some @media queries such as: (min-aspect-ratio: 1/1)
764 764
         // Add token to add the "/" back in later
765
-        $css = preg_replace('/\(([a-z-]+):([0-9]+)\/([0-9]+)\)/Si', '($1:$2'. self::QUERY_FRACTION .'$3)', $css);
765
+        $css = preg_replace('/\(([a-z-]+):([0-9]+)\/([0-9]+)\)/Si', '($1:$2'.self::QUERY_FRACTION.'$3)', $css);
766 766
 
767 767
         // Remove empty rule blocks up to 2 levels deep.
768 768
         $css = preg_replace(array_fill(0, 2, '/(\{)[^{};\/\n]+\{\}/S'), '$1', $css);
@@ -810,7 +810,7 @@  discard block
 block discarded – undo
810 810
         }
811 811
 
812 812
         // @import handling
813
-        $css = preg_replace_callback($this->importRegex, function ($matches) use (&$imports) {
813
+        $css = preg_replace_callback($this->importRegex, function($matches) use (&$imports) {
814 814
             // Keep all @import at-rules found for later
815 815
             $imports .= $matches[0];
816 816
             // Delete all @import at-rules
@@ -818,7 +818,7 @@  discard block
 block discarded – undo
818 818
         }, $css);
819 819
 
820 820
         // @namespace handling
821
-        $css = preg_replace_callback($this->namespaceRegex, function ($matches) use (&$namespaces) {
821
+        $css = preg_replace_callback($this->namespaceRegex, function($matches) use (&$namespaces) {
822 822
             // Keep all @namespace at-rules found for later
823 823
             $namespaces .= $matches[0];
824 824
             // Delete all @namespace at-rules
@@ -829,7 +829,7 @@  discard block
 block discarded – undo
829 829
         // 1. @charset first
830 830
         // 2. @imports below @charset
831 831
         // 3. @namespaces below @imports
832
-        $css = $charset . $imports . $namespaces . $css;
832
+        $css = $charset.$imports.$namespaces.$css;
833 833
 
834 834
         return $css;
835 835
     }
@@ -883,10 +883,10 @@  discard block
 block discarded – undo
883 883
         // Restore space after rgb() or hsl() function in some cases such as:
884 884
         // background-image: linear-gradient(to bottom, rgb(210,180,140) 10%, rgb(255,0,0) 90%);
885 885
         if (!empty($terminator) && !preg_match('/[ ,);]/S', $terminator)) {
886
-            $terminator = ' '. $terminator;
886
+            $terminator = ' '.$terminator;
887 887
         }
888 888
 
889
-        return '#'. implode('', $hexColors) . $terminator;
889
+        return '#'.implode('', $hexColors).$terminator;
890 890
     }
891 891
 
892 892
     /**
@@ -900,16 +900,16 @@  discard block
 block discarded – undo
900 900
 
901 901
         // Shorten suitable 6 chars HEX colors
902 902
         if (strlen($hex) === 6 && preg_match('/^([0-9a-f])\1([0-9a-f])\2([0-9a-f])\3$/Si', $hex, $m)) {
903
-            $hex = $m[1] . $m[2] . $m[3];
903
+            $hex = $m[1].$m[2].$m[3];
904 904
         }
905 905
 
906 906
         // Lowercase
907
-        $hex = '#'. strtolower($hex);
907
+        $hex = '#'.strtolower($hex);
908 908
 
909 909
         // Replace Hex colors with shorter color names
910 910
         $color = array_key_exists($hex, $this->hexToNamedColorsMap) ? $this->hexToNamedColorsMap[$hex] : $hex;
911 911
 
912
-        return $color . $matches[2];
912
+        return $color.$matches[2];
913 913
     }
914 914
 
915 915
     /**
@@ -920,7 +920,7 @@  discard block
 block discarded – undo
920 920
      */
921 921
     private function shortenNamedColorsCallback($matches)
922 922
     {
923
-        return $matches[1] . $this->namedToHexColorsMap[strtolower($matches[2])] . $matches[3];
923
+        return $matches[1].$this->namedToHexColorsMap[strtolower($matches[2])].$matches[3];
924 924
     }
925 925
 
926 926
     /**
Please login to merge, or discard this patch.
classes/autoptimizeUtils.php 1 patch
Spacing   +76 added lines, -76 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * General helpers.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
@@ -16,15 +16,15 @@  discard block
 block discarded – undo
16 16
      *
17 17
      * @return bool
18 18
      */
19
-    public static function mbstring_available( $override = null )
19
+    public static function mbstring_available($override = null)
20 20
     {
21 21
         static $available = null;
22 22
 
23
-        if ( null === $available ) {
24
-            $available = \extension_loaded( 'mbstring' );
23
+        if (null === $available) {
24
+            $available = \extension_loaded('mbstring');
25 25
         }
26 26
 
27
-        if ( null !== $override ) {
27
+        if (null !== $override) {
28 28
             $available = $override;
29 29
         }
30 30
 
@@ -42,12 +42,12 @@  discard block
 block discarded – undo
42 42
      *
43 43
      * @return int|false
44 44
      */
45
-    public static function strpos( $haystack, $needle, $offset = 0, $encoding = null )
45
+    public static function strpos($haystack, $needle, $offset = 0, $encoding = null)
46 46
     {
47
-        if ( self::mbstring_available() ) {
48
-            return ( null === $encoding ) ? \mb_strpos( $haystack, $needle, $offset ) : \mb_strlen( $haystack, $needle, $offset, $encoding );
47
+        if (self::mbstring_available()) {
48
+            return (null === $encoding) ? \mb_strpos($haystack, $needle, $offset) : \mb_strlen($haystack, $needle, $offset, $encoding);
49 49
         } else {
50
-            return \strpos( $haystack, $needle, $offset );
50
+            return \strpos($haystack, $needle, $offset);
51 51
         }
52 52
     }
53 53
 
@@ -62,12 +62,12 @@  discard block
 block discarded – undo
62 62
      * @return int Number of charcters or bytes in given $string
63 63
      *             (characters if/when supported, bytes otherwise).
64 64
      */
65
-    public static function strlen( $string, $encoding = null )
65
+    public static function strlen($string, $encoding = null)
66 66
     {
67
-        if ( self::mbstring_available() ) {
68
-            return ( null === $encoding ) ? \mb_strlen( $string ) : \mb_strlen( $string, $encoding );
67
+        if (self::mbstring_available()) {
68
+            return (null === $encoding) ? \mb_strlen($string) : \mb_strlen($string, $encoding);
69 69
         } else {
70
-            return \strlen( $string );
70
+            return \strlen($string);
71 71
         }
72 72
     }
73 73
 
@@ -85,44 +85,44 @@  discard block
 block discarded – undo
85 85
      *
86 86
      * @return string
87 87
      */
88
-    public static function substr_replace( $string, $replacement, $start, $length = null, $encoding = null )
88
+    public static function substr_replace($string, $replacement, $start, $length = null, $encoding = null)
89 89
     {
90
-        if ( self::mbstring_available() ) {
91
-            $strlen = self::strlen( $string, $encoding );
90
+        if (self::mbstring_available()) {
91
+            $strlen = self::strlen($string, $encoding);
92 92
 
93
-            if ( $start < 0 ) {
93
+            if ($start < 0) {
94 94
                 if ( -$start < $strlen ) {
95 95
                     $start = $strlen + $start;
96 96
                 } else {
97 97
                     $start = 0;
98 98
                 }
99
-            } elseif ( $start > $strlen ) {
99
+            } elseif ($start > $strlen) {
100 100
                 $start = $strlen;
101 101
             }
102 102
 
103
-            if ( null === $length || '' === $length ) {
103
+            if (null === $length || '' === $length) {
104 104
                 $start2 = $strlen;
105
-            } elseif ( $length < 0 ) {
105
+            } elseif ($length < 0) {
106 106
                 $start2 = $strlen + $length;
107
-                if ( $start2 < $start ) {
107
+                if ($start2 < $start) {
108 108
                     $start2 = $start;
109 109
                 }
110 110
             } else {
111 111
                 $start2 = $start + $length;
112 112
             }
113 113
 
114
-            if ( null === $encoding ) {
115
-                $leader  = $start ? \mb_substr( $string, 0, $start ) : '';
116
-                $trailer = ( $start2 < $strlen ) ? \mb_substr( $string, $start2, null ) : '';
114
+            if (null === $encoding) {
115
+                $leader  = $start ? \mb_substr($string, 0, $start) : '';
116
+                $trailer = ($start2 < $strlen) ? \mb_substr($string, $start2, null) : '';
117 117
             } else {
118
-                $leader  = $start ? \mb_substr( $string, 0, $start, $encoding ) : '';
119
-                $trailer = ( $start2 < $strlen ) ? \mb_substr( $string, $start2, null, $encoding ) : '';
118
+                $leader  = $start ? \mb_substr($string, 0, $start, $encoding) : '';
119
+                $trailer = ($start2 < $strlen) ? \mb_substr($string, $start2, null, $encoding) : '';
120 120
             }
121 121
 
122 122
             return "{$leader}{$replacement}{$trailer}";
123 123
         }
124 124
 
125
-        return ( null === $length ) ? \substr_replace( $string, $replacement, $start ) : \substr_replace( $string, $replacement, $start, $length );
125
+        return (null === $length) ? \substr_replace($string, $replacement, $start) : \substr_replace($string, $replacement, $start, $length);
126 126
     }
127 127
 
128 128
     /**
@@ -132,16 +132,16 @@  discard block
 block discarded – undo
132 132
      *
133 133
      * @return bool
134 134
      */
135
-    public static function siteurl_not_root( $override = null )
135
+    public static function siteurl_not_root($override = null)
136 136
     {
137 137
         static $subdir = null;
138 138
 
139
-        if ( null === $subdir ) {
139
+        if (null === $subdir) {
140 140
             $parts  = self::get_ao_wp_site_url_parts();
141
-            $subdir = ( isset( $parts['path'] ) && ( '/' !== $parts['path'] ) );
141
+            $subdir = (isset($parts['path']) && ('/' !== $parts['path']));
142 142
         }
143 143
 
144
-        if ( null !== $override ) {
144
+        if (null !== $override) {
145 145
             $subdir = $override;
146 146
         }
147 147
 
@@ -158,8 +158,8 @@  discard block
 block discarded – undo
158 158
     {
159 159
         static $parts = array();
160 160
 
161
-        if ( empty( $parts ) ) {
162
-            $parts = \parse_url( AUTOPTIMIZE_WP_SITE_URL );
161
+        if (empty($parts)) {
162
+            $parts = \parse_url(AUTOPTIMIZE_WP_SITE_URL);
163 163
         }
164 164
 
165 165
         return $parts;
@@ -174,33 +174,33 @@  discard block
 block discarded – undo
174 174
      *
175 175
      * @return string
176 176
      */
177
-    public static function tweak_cdn_url_if_needed( $cdn_url, $force_cache_miss = false )
177
+    public static function tweak_cdn_url_if_needed($cdn_url, $force_cache_miss = false)
178 178
     {
179 179
         static $results = array();
180 180
 
181
-        if ( ! isset( $results[ $cdn_url ] ) || $force_cache_miss ) {
181
+        if (!isset($results[$cdn_url]) || $force_cache_miss) {
182 182
 
183 183
             // In order to return unmodified input when there's no need to tweak.
184
-            $results[ $cdn_url ] = $cdn_url;
184
+            $results[$cdn_url] = $cdn_url;
185 185
 
186 186
             // Behind a default true filter for backcompat, and only for sites
187 187
             // in a subfolder/subdirectory, but still easily turned off if
188 188
             // not wanted/needed...
189
-            if ( autoptimizeUtils::siteurl_not_root() ) {
190
-                $check = apply_filters( 'autoptimize_filter_cdn_magic_path_check', true, $cdn_url );
191
-                if ( $check ) {
189
+            if (autoptimizeUtils::siteurl_not_root()) {
190
+                $check = apply_filters('autoptimize_filter_cdn_magic_path_check', true, $cdn_url);
191
+                if ($check) {
192 192
                     $site_url_parts = autoptimizeUtils::get_ao_wp_site_url_parts();
193
-                    $cdn_url_parts  = \parse_url( $cdn_url );
194
-                    $schemeless     = self::is_protocol_relative( $cdn_url );
195
-                    $cdn_url_parts  = self::maybe_replace_cdn_path( $site_url_parts, $cdn_url_parts );
196
-                    if ( false !== $cdn_url_parts ) {
197
-                        $results[ $cdn_url ] = self::assemble_parsed_url( $cdn_url_parts, $schemeless );
193
+                    $cdn_url_parts  = \parse_url($cdn_url);
194
+                    $schemeless     = self::is_protocol_relative($cdn_url);
195
+                    $cdn_url_parts  = self::maybe_replace_cdn_path($site_url_parts, $cdn_url_parts);
196
+                    if (false !== $cdn_url_parts) {
197
+                        $results[$cdn_url] = self::assemble_parsed_url($cdn_url_parts, $schemeless);
198 198
                     }
199 199
                 }
200 200
             }
201 201
         }
202 202
 
203
-        return $results[ $cdn_url ];
203
+        return $results[$cdn_url];
204 204
     }
205 205
 
206 206
     /**
@@ -214,10 +214,10 @@  discard block
 block discarded – undo
214 214
      *
215 215
      * @return array|false
216 216
      */
217
-    public static function maybe_replace_cdn_path( array $site_url_parts, array $cdn_url_parts )
217
+    public static function maybe_replace_cdn_path(array $site_url_parts, array $cdn_url_parts)
218 218
     {
219
-        if ( isset( $site_url_parts['path'] ) && '/' !== $site_url_parts['path'] ) {
220
-            if ( ! isset( $cdn_url_parts['path'] ) || '/' === $cdn_url_parts['path'] ) {
219
+        if (isset($site_url_parts['path']) && '/' !== $site_url_parts['path']) {
220
+            if (!isset($cdn_url_parts['path']) || '/' === $cdn_url_parts['path']) {
221 221
                 $cdn_url_parts['path'] = $site_url_parts['path'];
222 222
                 return $cdn_url_parts;
223 223
             }
@@ -237,20 +237,20 @@  discard block
 block discarded – undo
237 237
      *
238 238
      * @return string
239 239
      */
240
-    public static function assemble_parsed_url( array $parsed_url, $schemeless = false )
240
+    public static function assemble_parsed_url(array $parsed_url, $schemeless = false)
241 241
     {
242
-        $scheme = isset( $parsed_url['scheme'] ) ? $parsed_url['scheme'] . '://' : '';
243
-        if ( $schemeless ) {
242
+        $scheme = isset($parsed_url['scheme']) ? $parsed_url['scheme'].'://' : '';
243
+        if ($schemeless) {
244 244
             $scheme = '//';
245 245
         }
246
-        $host     = isset( $parsed_url['host'] ) ? $parsed_url['host'] : '';
247
-        $port     = isset( $parsed_url['port'] ) ? ':' . $parsed_url['port'] : '';
248
-        $user     = isset( $parsed_url['user'] ) ? $parsed_url['user'] : '';
249
-        $pass     = isset( $parsed_url['pass'] ) ? ':' . $parsed_url['pass'] : '';
250
-        $pass     = ( $user || $pass ) ? "$pass@" : '';
251
-        $path     = isset( $parsed_url['path'] ) ? $parsed_url['path'] : '';
252
-        $query    = isset( $parsed_url['query'] ) ? '?' . $parsed_url['query'] : '';
253
-        $fragment = isset( $parsed_url['fragment'] ) ? '#' . $parsed_url['fragment'] : '';
246
+        $host     = isset($parsed_url['host']) ? $parsed_url['host'] : '';
247
+        $port     = isset($parsed_url['port']) ? ':'.$parsed_url['port'] : '';
248
+        $user     = isset($parsed_url['user']) ? $parsed_url['user'] : '';
249
+        $pass     = isset($parsed_url['pass']) ? ':'.$parsed_url['pass'] : '';
250
+        $pass     = ($user || $pass) ? "$pass@" : '';
251
+        $path     = isset($parsed_url['path']) ? $parsed_url['path'] : '';
252
+        $query    = isset($parsed_url['query']) ? '?'.$parsed_url['query'] : '';
253
+        $fragment = isset($parsed_url['fragment']) ? '#'.$parsed_url['fragment'] : '';
254 254
 
255 255
         return "$scheme$user$pass$host$port$path$query$fragment";
256 256
     }
@@ -262,12 +262,12 @@  discard block
 block discarded – undo
262 262
      *
263 263
      * @return bool
264 264
      */
265
-    public static function is_protocol_relative( $url )
265
+    public static function is_protocol_relative($url)
266 266
     {
267 267
         $result = false;
268 268
 
269
-        if ( ! empty( $url ) ) {
270
-            $result = ( '/' === $url{1} ); // second char is `/`.
269
+        if (!empty($url)) {
270
+            $result = ('/' === $url{1} ); // second char is `/`.
271 271
         }
272 272
 
273 273
         return $result;
@@ -280,9 +280,9 @@  discard block
 block discarded – undo
280 280
      *
281 281
      * @return string
282 282
      */
283
-    public static function path_canonicalize( $path )
283
+    public static function path_canonicalize($path)
284 284
     {
285
-        $patterns     = array(
285
+        $patterns = array(
286 286
             '~/{2,}~',
287 287
             '~/(\./)+~',
288 288
             '~([^/\.]+/(?R)*\.{2,}/)~',
@@ -295,7 +295,7 @@  discard block
 block discarded – undo
295 295
             '',
296 296
         );
297 297
 
298
-        return preg_replace( $patterns, $replacements, $path );
298
+        return preg_replace($patterns, $replacements, $path);
299 299
     }
300 300
 
301 301
     /**
@@ -305,15 +305,15 @@  discard block
 block discarded – undo
305 305
      *
306 306
      * @return none if $return_result is false (default), array if $return_result is true.
307 307
      */
308
-    public static function check_service_availability( $return_result = false )
308
+    public static function check_service_availability($return_result = false)
309 309
     {
310
-        $service_availability_resp = wp_remote_get( 'https://misc.optimizingmatters.com/api/autoptimize_service_availablity.json?from=aomain&ver=' . AUTOPTIMIZE_PLUGIN_VERSION );
311
-        if ( ! is_wp_error( $service_availability_resp ) ) {
312
-            if ( '200' == wp_remote_retrieve_response_code( $service_availability_resp ) ) {
313
-                $availabilities = json_decode( wp_remote_retrieve_body( $service_availability_resp ), true );
314
-                if ( is_array( $availabilities ) ) {
315
-                    update_option( 'autoptimize_service_availablity', $availabilities );
316
-                    if ( $return_result ) {
310
+        $service_availability_resp = wp_remote_get('https://misc.optimizingmatters.com/api/autoptimize_service_availablity.json?from=aomain&ver='.AUTOPTIMIZE_PLUGIN_VERSION);
311
+        if (!is_wp_error($service_availability_resp)) {
312
+            if ('200' == wp_remote_retrieve_response_code($service_availability_resp)) {
313
+                $availabilities = json_decode(wp_remote_retrieve_body($service_availability_resp), true);
314
+                if (is_array($availabilities)) {
315
+                    update_option('autoptimize_service_availablity', $availabilities);
316
+                    if ($return_result) {
317 317
                         return $availabilities;
318 318
                     }
319 319
                 }
@@ -328,10 +328,10 @@  discard block
 block discarded – undo
328 328
      *
329 329
      * @return bool
330 330
      */
331
-    public static function str_is_valid_regex( $string )
331
+    public static function str_is_valid_regex($string)
332 332
     {
333
-        set_error_handler( function() {}, E_WARNING );
334
-        $is_regex = ( false !== preg_match( $string, '' ) );
333
+        set_error_handler(function() {}, E_WARNING);
334
+        $is_regex = (false !== preg_match($string, ''));
335 335
         restore_error_handler();
336 336
 
337 337
         return $is_regex;
Please login to merge, or discard this patch.
classes/autoptimizeBase.php 1 patch
Spacing   +151 added lines, -151 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * Base class other (more-specific) classes inherit from.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
      */
24 24
     public $debug_log = false;
25 25
 
26
-    public function __construct( $content )
26
+    public function __construct($content)
27 27
     {
28 28
         $this->content = $content;
29 29
     }
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
      *
36 36
      * @return bool
37 37
      */
38
-    abstract public function read( $options );
38
+    abstract public function read($options);
39 39
 
40 40
     /**
41 41
      * Joins and optimizes collected things.
@@ -66,40 +66,40 @@  discard block
 block discarded – undo
66 66
      *
67 67
      * @return bool|string
68 68
      */
69
-    public function getpath( $url )
69
+    public function getpath($url)
70 70
     {
71
-        $url = apply_filters( 'autoptimize_filter_cssjs_alter_url', $url );
71
+        $url = apply_filters('autoptimize_filter_cssjs_alter_url', $url);
72 72
 
73
-        if ( false !== strpos( $url, '%' ) ) {
74
-            $url = urldecode( $url );
73
+        if (false !== strpos($url, '%')) {
74
+            $url = urldecode($url);
75 75
         }
76 76
 
77
-        $site_host    = parse_url( AUTOPTIMIZE_WP_SITE_URL, PHP_URL_HOST );
78
-        $content_host = parse_url( AUTOPTIMIZE_WP_ROOT_URL, PHP_URL_HOST );
77
+        $site_host    = parse_url(AUTOPTIMIZE_WP_SITE_URL, PHP_URL_HOST);
78
+        $content_host = parse_url(AUTOPTIMIZE_WP_ROOT_URL, PHP_URL_HOST);
79 79
 
80 80
         // Normalizing attempts...
81
-        $double_slash_position = strpos( $url, '//' );
82
-        if ( 0 === $double_slash_position ) {
83
-            if ( is_ssl() ) {
84
-                $url = 'https:' . $url;
81
+        $double_slash_position = strpos($url, '//');
82
+        if (0 === $double_slash_position) {
83
+            if (is_ssl()) {
84
+                $url = 'https:'.$url;
85 85
             } else {
86
-                $url = 'http:' . $url;
86
+                $url = 'http:'.$url;
87 87
             }
88
-        } elseif ( ( false === $double_slash_position ) && ( false === strpos( $url, $site_host ) ) ) {
89
-            if ( AUTOPTIMIZE_WP_SITE_URL === $site_host ) {
90
-                $url = AUTOPTIMIZE_WP_SITE_URL . $url;
88
+        } elseif ((false === $double_slash_position) && (false === strpos($url, $site_host))) {
89
+            if (AUTOPTIMIZE_WP_SITE_URL === $site_host) {
90
+                $url = AUTOPTIMIZE_WP_SITE_URL.$url;
91 91
             } else {
92
-                $url = AUTOPTIMIZE_WP_SITE_URL . autoptimizeUtils::path_canonicalize( $url );
92
+                $url = AUTOPTIMIZE_WP_SITE_URL.autoptimizeUtils::path_canonicalize($url);
93 93
             }
94 94
         }
95 95
 
96
-        if ( $site_host !== $content_host ) {
97
-            $url = str_replace( AUTOPTIMIZE_WP_CONTENT_URL, AUTOPTIMIZE_WP_SITE_URL . AUTOPTIMIZE_WP_CONTENT_NAME, $url );
96
+        if ($site_host !== $content_host) {
97
+            $url = str_replace(AUTOPTIMIZE_WP_CONTENT_URL, AUTOPTIMIZE_WP_SITE_URL.AUTOPTIMIZE_WP_CONTENT_NAME, $url);
98 98
         }
99 99
 
100 100
         // First check; hostname wp site should be hostname of url!
101
-        $url_host = @parse_url( $url, PHP_URL_HOST ); // @codingStandardsIgnoreLine
102
-        if ( $url_host !== $site_host ) {
101
+        $url_host = @parse_url($url, PHP_URL_HOST); // @codingStandardsIgnoreLine
102
+        if ($url_host !== $site_host) {
103 103
             /**
104 104
              * First try to get all domains from WPML (if available)
105 105
              * then explicitely declare $this->cdn_url as OK as well
@@ -108,20 +108,20 @@  discard block
 block discarded – undo
108 108
              */
109 109
             $multidomains = array();
110 110
 
111
-            $multidomains_wpml = apply_filters( 'wpml_setting', array(), 'language_domains' );
112
-            if ( ! empty( $multidomains_wpml ) ) {
113
-                $multidomains = array_map( array( $this, 'get_url_hostname' ), $multidomains_wpml );
111
+            $multidomains_wpml = apply_filters('wpml_setting', array(), 'language_domains');
112
+            if (!empty($multidomains_wpml)) {
113
+                $multidomains = array_map(array($this, 'get_url_hostname'), $multidomains_wpml);
114 114
             }
115 115
 
116
-            if ( ! empty( $this->cdn_url ) ) {
117
-                $multidomains[] = parse_url( $this->cdn_url, PHP_URL_HOST );
116
+            if (!empty($this->cdn_url)) {
117
+                $multidomains[] = parse_url($this->cdn_url, PHP_URL_HOST);
118 118
             }
119 119
 
120
-            $multidomains = apply_filters( 'autoptimize_filter_cssjs_multidomain', $multidomains );
120
+            $multidomains = apply_filters('autoptimize_filter_cssjs_multidomain', $multidomains);
121 121
 
122
-            if ( ! empty( $multidomains ) ) {
123
-                if ( in_array( $url_host, $multidomains ) ) {
124
-                    $url = str_replace( $url_host, $site_host, $url );
122
+            if (!empty($multidomains)) {
123
+                if (in_array($url_host, $multidomains)) {
124
+                    $url = str_replace($url_host, $site_host, $url);
125 125
                 } else {
126 126
                     return false;
127 127
                 }
@@ -131,28 +131,28 @@  discard block
 block discarded – undo
131 131
         }
132 132
 
133 133
         // Try to remove "wp root url" from url while not minding http<>https.
134
-        $tmp_ao_root = preg_replace( '/https?:/', '', AUTOPTIMIZE_WP_ROOT_URL );
134
+        $tmp_ao_root = preg_replace('/https?:/', '', AUTOPTIMIZE_WP_ROOT_URL);
135 135
 
136
-        if ( $site_host !== $content_host ) {
136
+        if ($site_host !== $content_host) {
137 137
             // As we replaced the content-domain with the site-domain, we should match against that.
138
-            $tmp_ao_root = preg_replace( '/https?:/', '', AUTOPTIMIZE_WP_SITE_URL );
138
+            $tmp_ao_root = preg_replace('/https?:/', '', AUTOPTIMIZE_WP_SITE_URL);
139 139
         }
140 140
 
141
-        $tmp_url = preg_replace( '/https?:/', '', $url );
142
-        $path    = str_replace( $tmp_ao_root, '', $tmp_url );
141
+        $tmp_url = preg_replace('/https?:/', '', $url);
142
+        $path    = str_replace($tmp_ao_root, '', $tmp_url);
143 143
 
144 144
         // If path starts with :// or //, this is not a URL in the WP context and
145 145
         // we have to assume we can't aggregate.
146
-        if ( preg_match( '#^:?//#', $path ) ) {
146
+        if (preg_match('#^:?//#', $path)) {
147 147
             // External script/css (adsense, etc).
148 148
             return false;
149 149
         }
150 150
 
151 151
         // Prepend with WP_ROOT_DIR to have full path to file.
152
-        $path = str_replace( '//', '/', WP_ROOT_DIR . $path );
152
+        $path = str_replace('//', '/', WP_ROOT_DIR.$path);
153 153
 
154 154
         // Final check: does file exist and is it readable?
155
-        if ( file_exists( $path ) && is_file( $path ) && is_readable( $path ) ) {
155
+        if (file_exists($path) && is_file($path) && is_readable($path)) {
156 156
             return $path;
157 157
         } else {
158 158
             return false;
@@ -169,18 +169,18 @@  discard block
 block discarded – undo
169 169
      *
170 170
      * @return string
171 171
      */
172
-    protected function get_url_hostname( $url )
172
+    protected function get_url_hostname($url)
173 173
     {
174 174
         // Checking that the url starts with something vaguely resembling a protocol.
175
-        if ( ( 0 !== strpos( $url, 'http' ) ) && ( 0 !== strpos( $url, '//' ) ) ) {
176
-            $url = 'http://' . $url;
175
+        if ((0 !== strpos($url, 'http')) && (0 !== strpos($url, '//'))) {
176
+            $url = 'http://'.$url;
177 177
         }
178 178
 
179 179
         // Grab the hostname.
180
-        $hostname = parse_url( $url, PHP_URL_HOST );
180
+        $hostname = parse_url($url, PHP_URL_HOST);
181 181
 
182 182
         // Fallback when parse_url() fails.
183
-        if ( empty( $hostname ) ) {
183
+        if (empty($hostname)) {
184 184
             $hostname = $url;
185 185
         }
186 186
 
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
      *
195 195
      * @return string
196 196
      */
197
-    protected function hide_noptimize( $markup )
197
+    protected function hide_noptimize($markup)
198 198
     {
199 199
         return $this->replace_contents_with_marker_if_exists(
200 200
             'NOPTIMIZE',
@@ -211,9 +211,9 @@  discard block
 block discarded – undo
211 211
      *
212 212
      * @return string
213 213
      */
214
-    protected function restore_noptimize( $markup )
214
+    protected function restore_noptimize($markup)
215 215
     {
216
-        return $this->restore_marked_content( 'NOPTIMIZE', $markup );
216
+        return $this->restore_marked_content('NOPTIMIZE', $markup);
217 217
     }
218 218
 
219 219
     /**
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
      *
224 224
      * @return string
225 225
      */
226
-    protected function hide_iehacks( $markup )
226
+    protected function hide_iehacks($markup)
227 227
     {
228 228
         return $this->replace_contents_with_marker_if_exists(
229 229
             'IEHACK', // Marker name...
@@ -240,9 +240,9 @@  discard block
 block discarded – undo
240 240
      *
241 241
      * @return string
242 242
      */
243
-    protected function restore_iehacks( $markup )
243
+    protected function restore_iehacks($markup)
244 244
     {
245
-        return $this->restore_marked_content( 'IEHACK', $markup );
245
+        return $this->restore_marked_content('IEHACK', $markup);
246 246
     }
247 247
 
248 248
     /**
@@ -254,7 +254,7 @@  discard block
 block discarded – undo
254 254
      *
255 255
      * @return string
256 256
      */
257
-    protected function hide_comments( $markup )
257
+    protected function hide_comments($markup)
258 258
     {
259 259
         return $this->replace_contents_with_marker_if_exists(
260 260
             'COMMENTS',
@@ -272,9 +272,9 @@  discard block
 block discarded – undo
272 272
      *
273 273
      * @return string
274 274
      */
275
-    protected function restore_comments( $markup )
275
+    protected function restore_comments($markup)
276 276
     {
277
-        return $this->restore_marked_content( 'COMMENTS', $markup );
277
+        return $this->restore_marked_content('COMMENTS', $markup);
278 278
     }
279 279
 
280 280
     /**
@@ -285,44 +285,44 @@  discard block
 block discarded – undo
285 285
      *
286 286
      * @return string
287 287
      */
288
-    public function url_replace_cdn( $url )
288
+    public function url_replace_cdn($url)
289 289
     {
290 290
         // For 2.3 back-compat in which cdn-ing appeared to be automatically
291 291
         // including WP subfolder/subdirectory into account as part of cdn-ing,
292 292
         // even though it might've caused serious troubles in certain edge-cases.
293
-        $cdn_url = autoptimizeUtils::tweak_cdn_url_if_needed( $this->cdn_url );
293
+        $cdn_url = autoptimizeUtils::tweak_cdn_url_if_needed($this->cdn_url);
294 294
 
295 295
         // Allows API/filter to further tweak the cdn url...
296
-        $cdn_url = apply_filters( 'autoptimize_filter_base_cdnurl', $cdn_url );
297
-        if ( ! empty( $cdn_url ) ) {
298
-            $this->debug_log( 'before=' . $url );
296
+        $cdn_url = apply_filters('autoptimize_filter_base_cdnurl', $cdn_url);
297
+        if (!empty($cdn_url)) {
298
+            $this->debug_log('before='.$url);
299 299
 
300 300
             // Simple str_replace-based approach fails when $url is protocol-or-host-relative.
301
-            $is_protocol_relative = autoptimizeUtils::is_protocol_relative( $url );
302
-            $is_host_relative     = ( ! $is_protocol_relative && ( '/' === $url{0} ) );
303
-            $cdn_url              = rtrim( $cdn_url, '/' );
301
+            $is_protocol_relative = autoptimizeUtils::is_protocol_relative($url);
302
+            $is_host_relative     = (!$is_protocol_relative && ('/' === $url{0} ));
303
+            $cdn_url              = rtrim($cdn_url, '/');
304 304
 
305
-            if ( $is_host_relative ) {
305
+            if ($is_host_relative) {
306 306
                 // Prepending host-relative urls with the cdn url.
307
-                $url = $cdn_url . $url;
307
+                $url = $cdn_url.$url;
308 308
             } else {
309 309
                 // Either a protocol-relative or "regular" url, replacing it either way.
310
-                if ( $is_protocol_relative ) {
310
+                if ($is_protocol_relative) {
311 311
                     // Massage $site_url so that simple str_replace() still "works" by
312 312
                     // searching for the protocol-relative version of AUTOPTIMIZE_WP_SITE_URL.
313
-                    $site_url = str_replace( array( 'http:', 'https:' ), '', AUTOPTIMIZE_WP_SITE_URL );
313
+                    $site_url = str_replace(array('http:', 'https:'), '', AUTOPTIMIZE_WP_SITE_URL);
314 314
                 } else {
315 315
                     $site_url = AUTOPTIMIZE_WP_SITE_URL;
316 316
                 }
317
-                $this->debug_log( '`' . $site_url . '` -> `' . $cdn_url . '` in `' . $url . '`' );
318
-                $url = str_replace( $site_url, $cdn_url, $url );
317
+                $this->debug_log('`'.$site_url.'` -> `'.$cdn_url.'` in `'.$url.'`');
318
+                $url = str_replace($site_url, $cdn_url, $url);
319 319
             }
320 320
 
321
-            $this->debug_log( 'after=' . $url );
321
+            $this->debug_log('after='.$url);
322 322
         }
323 323
 
324 324
         // Allow API filter to take further care of CDN replacement.
325
-        $url = apply_filters( 'autoptimize_filter_base_replace_cdn', $url );
325
+        $url = apply_filters('autoptimize_filter_base_replace_cdn', $url);
326 326
 
327 327
         return $url;
328 328
     }
@@ -340,18 +340,18 @@  discard block
 block discarded – undo
340 340
      *
341 341
      * @return void
342 342
      */
343
-    protected function inject_in_html( $payload, $where )
343
+    protected function inject_in_html($payload, $where)
344 344
     {
345 345
         $warned   = false;
346
-        $position = autoptimizeUtils::strpos( $this->content, $where[0] );
347
-        if ( false !== $position ) {
346
+        $position = autoptimizeUtils::strpos($this->content, $where[0]);
347
+        if (false !== $position) {
348 348
             // Found the tag, setup content/injection as specified.
349
-            if ( 'after' === $where[1] ) {
350
-                $content = $where[0] . $payload;
351
-            } elseif ( 'replace' === $where[1] ) {
349
+            if ('after' === $where[1]) {
350
+                $content = $where[0].$payload;
351
+            } elseif ('replace' === $where[1]) {
352 352
                 $content = $payload;
353 353
             } else {
354
-                $content = $payload . $where[0];
354
+                $content = $payload.$where[0];
355 355
             }
356 356
             // Place where specified.
357 357
             $this->content = autoptimizeUtils::substr_replace(
@@ -360,14 +360,14 @@  discard block
 block discarded – undo
360 360
                 $position,
361 361
                 // Using plain strlen() should be safe here for now, since
362 362
                 // we're not searching for multibyte chars here still...
363
-                strlen( $where[0] )
363
+                strlen($where[0])
364 364
             );
365 365
         } else {
366 366
             // Couldn't find what was specified, just append and add a warning.
367 367
             $this->content .= $payload;
368
-            if ( ! $warned ) {
369
-                $tag_display    = str_replace( array( '<', '>' ), '', $where[0] );
370
-                $this->content .= '<!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `' . $tag_display . '` missing --><!--/noptimize-->';
368
+            if (!$warned) {
369
+                $tag_display    = str_replace(array('<', '>'), '', $where[0]);
370
+                $this->content .= '<!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `'.$tag_display.'` missing --><!--/noptimize-->';
371 371
                 $warned         = true;
372 372
             }
373 373
         }
@@ -381,10 +381,10 @@  discard block
 block discarded – undo
381 381
      *
382 382
      * @return bool
383 383
      */
384
-    protected function isremovable( $tag, $removables )
384
+    protected function isremovable($tag, $removables)
385 385
     {
386
-        foreach ( $removables as $match ) {
387
-            if ( false !== strpos( $tag, $match ) ) {
386
+        foreach ($removables as $match) {
387
+            if (false !== strpos($tag, $match)) {
388 388
                 return true;
389 389
             }
390 390
         }
@@ -399,10 +399,10 @@  discard block
 block discarded – undo
399 399
      *
400 400
      * @return string
401 401
      */
402
-    public function inject_minified_callback( $matches )
402
+    public function inject_minified_callback($matches)
403 403
     {
404 404
         static $conf = null;
405
-        if ( null === $conf ) {
405
+        if (null === $conf) {
406 406
             $conf = autoptimizeConfig::instance();
407 407
         }
408 408
 
@@ -417,62 +417,62 @@  discard block
 block discarded – undo
417 417
         $filehash = null;
418 418
 
419 419
         // Grab the parts we need.
420
-        $parts = explode( '|', $matches[1] );
421
-        if ( ! empty( $parts ) ) {
422
-            $filepath = isset( $parts[0] ) ? base64_decode($parts[0]) : null;
423
-            $filehash = isset( $parts[1] ) ? $parts[1] : null;
420
+        $parts = explode('|', $matches[1]);
421
+        if (!empty($parts)) {
422
+            $filepath = isset($parts[0]) ? base64_decode($parts[0]) : null;
423
+            $filehash = isset($parts[1]) ? $parts[1] : null;
424 424
         }
425 425
 
426 426
         // Bail early if something's not right...
427
-        if ( ! $filepath || ! $filehash ) {
427
+        if (!$filepath || !$filehash) {
428 428
             return "\n";
429 429
         }
430 430
 
431
-        $filecontent = file_get_contents( $filepath );
431
+        $filecontent = file_get_contents($filepath);
432 432
 
433 433
         // Some things are differently handled for css/js...
434
-        $is_js_file = ( '.js' === substr( $filepath, -3, 3 ) );
434
+        $is_js_file = ('.js' === substr($filepath, -3, 3));
435 435
 
436 436
         $is_css_file = false;
437
-        if ( ! $is_js_file ) {
438
-            $is_css_file = ( '.css' === substr( $filepath, -4, 4 ) );
437
+        if (!$is_js_file) {
438
+            $is_css_file = ('.css' === substr($filepath, -4, 4));
439 439
         }
440 440
 
441 441
         // BOMs being nuked here unconditionally (regardless of where they are)!
442
-        $filecontent = preg_replace( "#\x{EF}\x{BB}\x{BF}#", '', $filecontent );
442
+        $filecontent = preg_replace("#\x{EF}\x{BB}\x{BF}#", '', $filecontent);
443 443
 
444 444
         // Remove comments and blank lines.
445
-        if ( $is_js_file ) {
446
-            $filecontent = preg_replace( '#^\s*\/\/.*$#Um', '', $filecontent );
445
+        if ($is_js_file) {
446
+            $filecontent = preg_replace('#^\s*\/\/.*$#Um', '', $filecontent);
447 447
         }
448 448
 
449 449
         // Nuke un-important comments.
450
-        $filecontent = preg_replace( '#^\s*\/\*[^!].*\*\/\s?#Um', '', $filecontent );
450
+        $filecontent = preg_replace('#^\s*\/\*[^!].*\*\/\s?#Um', '', $filecontent);
451 451
 
452 452
         // Normalize newlines.
453
-        $filecontent = preg_replace( '#(^[\r\n]*|[\r\n]+)[\s\t]*[\r\n]+#', "\n", $filecontent );
453
+        $filecontent = preg_replace('#(^[\r\n]*|[\r\n]+)[\s\t]*[\r\n]+#', "\n", $filecontent);
454 454
 
455 455
         // JS specifics.
456
-        if ( $is_js_file ) {
456
+        if ($is_js_file) {
457 457
             // Append a semicolon at the end of js files if it's missing.
458
-            $last_char = substr( $filecontent, -1, 1 );
459
-            if ( ';' !== $last_char && '}' !== $last_char ) {
458
+            $last_char = substr($filecontent, -1, 1);
459
+            if (';' !== $last_char && '}' !== $last_char) {
460 460
                 $filecontent .= ';';
461 461
             }
462 462
             // Check if try/catch should be used.
463
-            $opt_js_try_catch = $conf->get( 'autoptimize_js_trycatch' );
464
-            if ( 'on' === $opt_js_try_catch ) {
463
+            $opt_js_try_catch = $conf->get('autoptimize_js_trycatch');
464
+            if ('on' === $opt_js_try_catch) {
465 465
                 // It should, wrap in try/catch.
466
-                $filecontent = 'try{' . $filecontent . '}catch(e){}';
466
+                $filecontent = 'try{'.$filecontent.'}catch(e){}';
467 467
             }
468
-        } elseif ( $is_css_file ) {
469
-            $filecontent = autoptimizeStyles::fixurls( $filepath, $filecontent );
468
+        } elseif ($is_css_file) {
469
+            $filecontent = autoptimizeStyles::fixurls($filepath, $filecontent);
470 470
         } else {
471 471
             $filecontent = '';
472 472
         }
473 473
 
474 474
         // Return modified (or empty!) code/content.
475
-        return "\n" . $filecontent;
475
+        return "\n".$filecontent;
476 476
     }
477 477
 
478 478
     /**
@@ -482,13 +482,13 @@  discard block
 block discarded – undo
482 482
      *
483 483
      * @return string
484 484
      */
485
-    protected function inject_minified( $in )
485
+    protected function inject_minified($in)
486 486
     {
487 487
         $out = $in;
488
-        if ( false !== strpos( $in, '%%INJECTLATER%%' ) ) {
488
+        if (false !== strpos($in, '%%INJECTLATER%%')) {
489 489
             $out = preg_replace_callback(
490
-                '#\/\*\!%%INJECTLATER' . AUTOPTIMIZE_HASH . '%%(.*?)%%INJECTLATER%%\*\/#is',
491
-                array( $this, 'inject_minified_callback' ),
490
+                '#\/\*\!%%INJECTLATER'.AUTOPTIMIZE_HASH.'%%(.*?)%%INJECTLATER%%\*\/#is',
491
+                array($this, 'inject_minified_callback'),
492 492
                 $in
493 493
             );
494 494
         }
@@ -508,9 +508,9 @@  discard block
 block discarded – undo
508 508
      *
509 509
      * @return string
510 510
      */
511
-    public static function build_injectlater_marker( $filepath, $hash )
511
+    public static function build_injectlater_marker($filepath, $hash)
512 512
     {
513
-        $contents = '/*!' . self::build_marker( 'INJECTLATER', $filepath, $hash ) . '*/';
513
+        $contents = '/*!'.self::build_marker('INJECTLATER', $filepath, $hash).'*/';
514 514
 
515 515
         return $contents;
516 516
     }
@@ -528,18 +528,18 @@  discard block
 block discarded – undo
528 528
      *
529 529
      * @return string
530 530
      */
531
-    public static function build_marker( $name, $data, $hash = null )
531
+    public static function build_marker($name, $data, $hash = null)
532 532
     {
533 533
         // Start the marker, add the data.
534
-        $marker = '%%' . $name . AUTOPTIMIZE_HASH . '%%' . base64_encode( $data );
534
+        $marker = '%%'.$name.AUTOPTIMIZE_HASH.'%%'.base64_encode($data);
535 535
 
536 536
         // Add the hash if provided.
537
-        if ( null !== $hash ) {
538
-            $marker .= '|' . $hash;
537
+        if (null !== $hash) {
538
+            $marker .= '|'.$hash;
539 539
         }
540 540
 
541 541
         // Close the marker.
542
-        $marker .= '%%' . $name . '%%';
542
+        $marker .= '%%'.$name.'%%';
543 543
 
544 544
         return $marker;
545 545
     }
@@ -559,22 +559,22 @@  discard block
 block discarded – undo
559 559
      *
560 560
      * @return string
561 561
      */
562
-    protected function replace_contents_with_marker_if_exists( $marker, $search, $re_replace_pattern, $content )
562
+    protected function replace_contents_with_marker_if_exists($marker, $search, $re_replace_pattern, $content)
563 563
     {
564 564
         $found = false;
565 565
 
566
-        $is_regex = autoptimizeUtils::str_is_valid_regex( $search );
567
-        if ( $is_regex ) {
568
-            $found = preg_match( $search, $content );
566
+        $is_regex = autoptimizeUtils::str_is_valid_regex($search);
567
+        if ($is_regex) {
568
+            $found = preg_match($search, $content);
569 569
         } else {
570
-            $found = ( false !== strpos( $content, $search ) );
570
+            $found = (false !== strpos($content, $search));
571 571
         }
572 572
 
573
-        if ( $found ) {
573
+        if ($found) {
574 574
             $content = preg_replace_callback(
575 575
                 $re_replace_pattern,
576
-                function( $matches ) use ( $marker ) {
577
-                    return autoptimizeBase::build_marker( $marker, $matches[0] );
576
+                function($matches) use ($marker) {
577
+                    return autoptimizeBase::build_marker($marker, $matches[0]);
578 578
                 },
579 579
                 $content
580 580
             );
@@ -591,13 +591,13 @@  discard block
 block discarded – undo
591 591
      *
592 592
      * @return string
593 593
      */
594
-    protected function restore_marked_content( $marker, $content )
594
+    protected function restore_marked_content($marker, $content)
595 595
     {
596
-        if ( false !== strpos( $content, $marker ) ) {
596
+        if (false !== strpos($content, $marker)) {
597 597
             $content = preg_replace_callback(
598
-                '#%%' . $marker . AUTOPTIMIZE_HASH . '%%(.*?)%%' . $marker . '%%#is',
599
-                function ( $matches ) {
600
-                    return base64_decode( $matches[1] );
598
+                '#%%'.$marker.AUTOPTIMIZE_HASH.'%%(.*?)%%'.$marker.'%%#is',
599
+                function($matches) {
600
+                    return base64_decode($matches[1]);
601 601
                 },
602 602
                 $content
603 603
             );
@@ -613,17 +613,17 @@  discard block
 block discarded – undo
613 613
      *
614 614
      * @return void
615 615
      */
616
-    protected function debug_log( $data )
616
+    protected function debug_log($data)
617 617
     {
618
-        if ( ! isset( $this->debug_log ) || ! $this->debug_log ) {
618
+        if (!isset($this->debug_log) || !$this->debug_log) {
619 619
             return;
620 620
         }
621 621
 
622
-        if ( ! is_string( $data ) && ! is_resource( $data ) ) {
623
-            $data = var_export( $data, true );
622
+        if (!is_string($data) && !is_resource($data)) {
623
+            $data = var_export($data, true);
624 624
         }
625 625
 
626
-        error_log( $data );
626
+        error_log($data);
627 627
     }
628 628
 
629 629
     /**
@@ -633,12 +633,12 @@  discard block
 block discarded – undo
633 633
      *
634 634
      * @return bool|string to be minified code or false.
635 635
      */
636
-    protected function prepare_minify_single( $filepath )
636
+    protected function prepare_minify_single($filepath)
637 637
     {
638 638
         // Decide what we're dealing with, return false if we don't know.
639
-        if ( $this->str_ends_in( $filepath, '.js' ) ) {
639
+        if ($this->str_ends_in($filepath, '.js')) {
640 640
             $type = 'js';
641
-        } elseif ( $this->str_ends_in( $filepath, '.css' ) ) {
641
+        } elseif ($this->str_ends_in($filepath, '.css')) {
642 642
             $type = 'css';
643 643
         } else {
644 644
             return false;
@@ -647,18 +647,18 @@  discard block
 block discarded – undo
647 647
         // Bail if it looks like its already minifed (by having -min or .min
648 648
         // in filename) or if it looks like WP jquery.js (which is minified).
649 649
         $minified_variants = array(
650
-            '-min.' . $type,
651
-            '.min.' . $type,
650
+            '-min.'.$type,
651
+            '.min.'.$type,
652 652
             'js/jquery/jquery.js',
653 653
         );
654
-        foreach ( $minified_variants as $ending ) {
655
-            if ( $this->str_ends_in( $filepath, $ending ) ) {
654
+        foreach ($minified_variants as $ending) {
655
+            if ($this->str_ends_in($filepath, $ending)) {
656 656
                 return false;
657 657
             }
658 658
         }
659 659
 
660 660
         // Get file contents, bail if empty.
661
-        $contents = file_get_contents( $filepath );
661
+        $contents = file_get_contents($filepath);
662 662
 
663 663
         return $contents;
664 664
     }
@@ -671,12 +671,12 @@  discard block
 block discarded – undo
671 671
      *
672 672
      * @return string
673 673
      */
674
-    protected function build_minify_single_url( autoptimizeCache $cache )
674
+    protected function build_minify_single_url(autoptimizeCache $cache)
675 675
     {
676
-        $url = AUTOPTIMIZE_CACHE_URL . $cache->getname();
676
+        $url = AUTOPTIMIZE_CACHE_URL.$cache->getname();
677 677
 
678 678
         // CDN-replace the resulting URL if needed...
679
-        $url = $this->url_replace_cdn( $url );
679
+        $url = $this->url_replace_cdn($url);
680 680
 
681 681
         return $url;
682 682
     }
@@ -689,15 +689,15 @@  discard block
 block discarded – undo
689 689
      *
690 690
      * @return bool
691 691
      */
692
-    protected function str_ends_in( $str, $test )
692
+    protected function str_ends_in($str, $test)
693 693
     {
694 694
         // @codingStandardsIgnoreStart
695 695
         // substr_compare() is bugged on 5.5.11: https://3v4l.org/qGYBH
696 696
         // return ( 0 === substr_compare( $str, $test, -strlen( $test ) ) );
697 697
         // @codingStandardsIgnoreEnd
698 698
 
699
-        $length = strlen( $test );
699
+        $length = strlen($test);
700 700
 
701
-        return ( substr( $str, -$length, $length ) === $test );
701
+        return (substr($str, -$length, $length) === $test);
702 702
     }
703 703
 }
Please login to merge, or discard this patch.
tests/bootstrap.php 1 patch
Spacing   +34 added lines, -34 removed lines patch added patch discarded remove patch
@@ -3,50 +3,50 @@  discard block
 block discarded – undo
3 3
  * WP tests bootstrap.
4 4
  */
5 5
 
6
-$_tests_dir = getenv( 'WP_TESTS_DIR' );
7
-if ( ! $_tests_dir ) {
8
-    $tmp_dir = getenv( 'TMPDIR' );
9
-    if ( ! empty( $tmp_dir ) ) {
10
-        $_tests_dir = rtrim( $tmp_dir, '/' ) . '/wordpress-tests-lib';
11
-        if ( ! is_dir( $_tests_dir ) ) {
6
+$_tests_dir = getenv('WP_TESTS_DIR');
7
+if (!$_tests_dir) {
8
+    $tmp_dir = getenv('TMPDIR');
9
+    if (!empty($tmp_dir)) {
10
+        $_tests_dir = rtrim($tmp_dir, '/').'/wordpress-tests-lib';
11
+        if (!is_dir($_tests_dir)) {
12 12
             $_tests_dir = null;
13 13
         }
14 14
     }
15 15
 }
16 16
 
17
-if ( ! $_tests_dir ) {
17
+if (!$_tests_dir) {
18 18
     $_tests_dir = '/tmp/wordpress-tests-lib';
19 19
 }
20 20
 
21
-require_once $_tests_dir . '/includes/functions.php';
21
+require_once $_tests_dir.'/includes/functions.php';
22 22
 
23 23
 function _manually_load_plugin() {
24
-    define( 'AUTOPTIMIZE_INIT_EARLIER', true );
24
+    define('AUTOPTIMIZE_INIT_EARLIER', true);
25 25
 
26 26
     // For overriding cache dirs and whatnot. Kinda works if you keep a few things in mind.
27
-    if ( getenv('CUSTOM_CONSTANTS' ) ) {
28
-        define( 'AUTOPTIMIZE_CACHE_CHILD_DIR', '/c/ao/' );
29
-        $pathname = WP_CONTENT_DIR . AUTOPTIMIZE_CACHE_CHILD_DIR;
30
-        if ( is_multisite() && apply_filters( 'autoptimize_separate_blog_caches', true ) ) {
27
+    if (getenv('CUSTOM_CONSTANTS')) {
28
+        define('AUTOPTIMIZE_CACHE_CHILD_DIR', '/c/ao/');
29
+        $pathname = WP_CONTENT_DIR.AUTOPTIMIZE_CACHE_CHILD_DIR;
30
+        if (is_multisite() && apply_filters('autoptimize_separate_blog_caches', true)) {
31 31
             $blog_id   = get_current_blog_id();
32
-            $pathname .= $blog_id . '/';
32
+            $pathname .= $blog_id.'/';
33 33
         }
34
-        define( 'AUTOPTIMIZE_CACHE_DIR', $pathname );
34
+        define('AUTOPTIMIZE_CACHE_DIR', $pathname);
35 35
 
36 36
         $custom_site_url = 'http://localhost/wordpress';
37
-        define( 'AUTOPTIMIZE_WP_SITE_URL', $custom_site_url );
38
-        add_filter( 'site_url', function( $url, $path, $scheme, $blog_id ) use ( $custom_site_url ) {
37
+        define('AUTOPTIMIZE_WP_SITE_URL', $custom_site_url);
38
+        add_filter('site_url', function($url, $path, $scheme, $blog_id) use ($custom_site_url) {
39 39
             return $custom_site_url;
40
-        }, 10, 4 );
41
-        add_filter( 'content_url', function( $url, $path ) use ( $custom_site_url ) {
42
-            return $custom_site_url . '/wp-content';
43
-        }, 10, 2 );
44
-        define( 'AO_TEST_SUBFOLDER_INSTALL', true );
40
+        }, 10, 4);
41
+        add_filter('content_url', function($url, $path) use ($custom_site_url) {
42
+            return $custom_site_url.'/wp-content';
43
+        }, 10, 2);
44
+        define('AO_TEST_SUBFOLDER_INSTALL', true);
45 45
 
46
-        define( 'CUSTOM_CONSTANTS_USED', true );
46
+        define('CUSTOM_CONSTANTS_USED', true);
47 47
     } else {
48
-        define( 'CUSTOM_CONSTANTS_USED', false );
49
-        define( 'AO_TEST_SUBFOLDER_INSTALL', false );
48
+        define('CUSTOM_CONSTANTS_USED', false);
49
+        define('AO_TEST_SUBFOLDER_INSTALL', false);
50 50
     }
51 51
 
52 52
     /*
@@ -54,18 +54,18 @@  discard block
 block discarded – undo
54 54
     update_option( 'active_plugins', $active_plugins );
55 55
     */
56 56
 
57
-    update_option( 'autoptimize_js', 1 );
58
-    update_option( 'autoptimize_css', 1 );
59
-    update_option( 'autoptimize_html', 0 );
60
-    update_option( 'autoptimize_cdn_url', 'http://cdn.example.org' );
61
-    update_option( 'autoptimize_cache_nogzip', 1 );
57
+    update_option('autoptimize_js', 1);
58
+    update_option('autoptimize_css', 1);
59
+    update_option('autoptimize_html', 0);
60
+    update_option('autoptimize_cdn_url', 'http://cdn.example.org');
61
+    update_option('autoptimize_cache_nogzip', 1);
62 62
 
63
-    add_filter( 'autoptimize_css_include_inline', function( $include_inline ) {
63
+    add_filter('autoptimize_css_include_inline', function($include_inline) {
64 64
         return true;
65 65
     });
66 66
 
67
-    require dirname( dirname( __FILE__ ) ) . '/autoptimize.php';
67
+    require dirname(dirname(__FILE__)).'/autoptimize.php';
68 68
 }
69
-tests_add_filter( 'muplugins_loaded', '_manually_load_plugin' );
69
+tests_add_filter('muplugins_loaded', '_manually_load_plugin');
70 70
 
71
-require $_tests_dir . '/includes/bootstrap.php';
71
+require $_tests_dir.'/includes/bootstrap.php';
Please login to merge, or discard this patch.
classes/autoptimizeScripts.php 1 patch
Spacing   +151 added lines, -151 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-if ( ! defined( 'ABSPATH' ) ) {
3
+if (!defined('ABSPATH')) {
4 4
     exit;
5 5
 }
6 6
 
@@ -13,18 +13,18 @@  discard block
 block discarded – undo
13 13
     );
14 14
 
15 15
     private $dontmove = array(
16
-        'document.write','html5.js','show_ads.js','google_ad','histats.com/js','statcounter.com/counter/counter.js',
17
-        'ws.amazon.com/widgets','media.fastclick.net','/ads/','comment-form-quicktags/quicktags.php','edToolbar',
18
-        'intensedebate.com','scripts.chitika.net/','_gaq.push','jotform.com/','admin-bar.min.js','GoogleAnalyticsObject',
19
-        'plupload.full.min.js','syntaxhighlighter','adsbygoogle','gist.github.com','_stq','nonce','post_id','data-noptimize'
16
+        'document.write', 'html5.js', 'show_ads.js', 'google_ad', 'histats.com/js', 'statcounter.com/counter/counter.js',
17
+        'ws.amazon.com/widgets', 'media.fastclick.net', '/ads/', 'comment-form-quicktags/quicktags.php', 'edToolbar',
18
+        'intensedebate.com', 'scripts.chitika.net/', '_gaq.push', 'jotform.com/', 'admin-bar.min.js', 'GoogleAnalyticsObject',
19
+        'plupload.full.min.js', 'syntaxhighlighter', 'adsbygoogle', 'gist.github.com', '_stq', 'nonce', 'post_id', 'data-noptimize'
20 20
         ,'logHuman'
21 21
     );
22 22
     private $domove     = array(
23
-        'gaJsHost','load_cmc','jd.gallery.transitions.js','swfobject.embedSWF(','tiny_mce.js','tinyMCEPreInit.go'
23
+        'gaJsHost', 'load_cmc', 'jd.gallery.transitions.js', 'swfobject.embedSWF(', 'tiny_mce.js', 'tinyMCEPreInit.go'
24 24
     );
25 25
     private $domovelast = array(
26
-        'addthis.com','/afsonline/show_afs_search.js','disqus.js','networkedblogs.com/getnetworkwidget','infolinks.com/js/',
27
-        'jd.gallery.js.php','jd.gallery.transitions.js','swfobject.embedSWF(','linkwithin.com/widget.js','tiny_mce.js','tinyMCEPreInit.go'
26
+        'addthis.com', '/afsonline/show_afs_search.js', 'disqus.js', 'networkedblogs.com/getnetworkwidget', 'infolinks.com/js/',
27
+        'jd.gallery.js.php', 'jd.gallery.transitions.js', 'swfobject.embedSWF(', 'linkwithin.com/widget.js', 'tiny_mce.js', 'tinyMCEPreInit.go'
28 28
     );
29 29
 
30 30
     private $aggregate       = true;
@@ -43,84 +43,84 @@  discard block
 block discarded – undo
43 43
     // Reads the page and collects script tags
44 44
     public function read($options)
45 45
     {
46
-        $noptimizeJS = apply_filters( 'autoptimize_filter_js_noptimize', false, $this->content );
47
-        if ( $noptimizeJS ) {
46
+        $noptimizeJS = apply_filters('autoptimize_filter_js_noptimize', false, $this->content);
47
+        if ($noptimizeJS) {
48 48
             return false;
49 49
         }
50 50
 
51 51
         // only optimize known good JS?
52
-        $whitelistJS = apply_filters( 'autoptimize_filter_js_whitelist', '', $this->content );
53
-        if ( ! empty( $whitelistJS ) ) {
54
-            $this->whitelist = array_filter( array_map( 'trim', explode( ',', $whitelistJS ) ) );
52
+        $whitelistJS = apply_filters('autoptimize_filter_js_whitelist', '', $this->content);
53
+        if (!empty($whitelistJS)) {
54
+            $this->whitelist = array_filter(array_map('trim', explode(',', $whitelistJS)));
55 55
         }
56 56
 
57 57
         // is there JS we should simply remove
58
-        $removableJS = apply_filters( 'autoptimize_filter_js_removables', '', $this->content );
58
+        $removableJS = apply_filters('autoptimize_filter_js_removables', '', $this->content);
59 59
         if (!empty($removableJS)) {
60
-            $this->jsremovables = array_filter( array_map( 'trim', explode( ',', $removableJS ) ) );
60
+            $this->jsremovables = array_filter(array_map('trim', explode(',', $removableJS)));
61 61
         }
62 62
 
63 63
         // only header?
64
-        if ( apply_filters( 'autoptimize_filter_js_justhead', $options['justhead'] ) ) {
65
-            $content             = explode( '</head>', $this->content, 2 );
66
-            $this->content       = $content[0] . '</head>';
64
+        if (apply_filters('autoptimize_filter_js_justhead', $options['justhead'])) {
65
+            $content             = explode('</head>', $this->content, 2);
66
+            $this->content       = $content[0].'</head>';
67 67
             $this->restofcontent = $content[1];
68 68
         }
69 69
 
70 70
         // Determine whether we're doing JS-files aggregation or not.
71
-        if ( ! $options['aggregate'] ) {
71
+        if (!$options['aggregate']) {
72 72
             $this->aggregate = false;
73 73
         }
74 74
         // Returning true for "dontaggregate" turns off aggregation.
75
-        if ( $this->aggregate && apply_filters( 'autoptimize_filter_js_dontaggregate', false ) ) {
75
+        if ($this->aggregate && apply_filters('autoptimize_filter_js_dontaggregate', false)) {
76 76
             $this->aggregate = false;
77 77
         }
78 78
 
79 79
         // include inline?
80
-        if ( apply_filters( 'autoptimize_js_include_inline', $options['include_inline'] ) ) {
80
+        if (apply_filters('autoptimize_js_include_inline', $options['include_inline'])) {
81 81
             $this->include_inline = true;
82 82
         }
83 83
 
84 84
         // filter to "late inject minified JS", default to true for now (it is faster)
85
-        $this->inject_min_late = apply_filters( 'autoptimize_filter_js_inject_min_late', true );
85
+        $this->inject_min_late = apply_filters('autoptimize_filter_js_inject_min_late', true);
86 86
 
87 87
         // filters to override hardcoded do(nt)move(last) array contents (array in, array out!)
88
-        $this->dontmove = apply_filters( 'autoptimize_filter_js_dontmove', $this->dontmove );
89
-        $this->domovelast = apply_filters( 'autoptimize_filter_js_movelast', $this->domovelast );
90
-        $this->domove = apply_filters( 'autoptimize_filter_js_domove', $this->domove );
88
+        $this->dontmove = apply_filters('autoptimize_filter_js_dontmove', $this->dontmove);
89
+        $this->domovelast = apply_filters('autoptimize_filter_js_movelast', $this->domovelast);
90
+        $this->domove = apply_filters('autoptimize_filter_js_domove', $this->domove);
91 91
 
92 92
         // get extra exclusions settings or filter
93 93
         $excludeJS = $options['js_exclude'];
94
-        $excludeJS = apply_filters( 'autoptimize_filter_js_exclude', $excludeJS, $this->content );
94
+        $excludeJS = apply_filters('autoptimize_filter_js_exclude', $excludeJS, $this->content);
95 95
 
96
-        if ( '' !== $excludeJS ) {
97
-            if ( is_array( $excludeJS ) ) {
98
-                if ( ( $removeKeys = array_keys( $excludeJS, 'remove' ) ) !== false ) {
99
-                    foreach ( $removeKeys as $removeKey ) {
100
-                        unset( $excludeJS[$removeKey] );
96
+        if ('' !== $excludeJS) {
97
+            if (is_array($excludeJS)) {
98
+                if (($removeKeys = array_keys($excludeJS, 'remove')) !== false) {
99
+                    foreach ($removeKeys as $removeKey) {
100
+                        unset($excludeJS[$removeKey]);
101 101
                         $this->jsremovables[] = $removeKey;
102 102
                     }
103 103
                 }
104
-                $exclJSArr = array_keys( $excludeJS );
104
+                $exclJSArr = array_keys($excludeJS);
105 105
             } else {
106
-                $exclJSArr = array_filter( array_map( 'trim', explode( ',', $excludeJS ) ) );
106
+                $exclJSArr = array_filter(array_map('trim', explode(',', $excludeJS)));
107 107
             }
108
-            $this->dontmove = array_merge( $exclJSArr, $this->dontmove );
108
+            $this->dontmove = array_merge($exclJSArr, $this->dontmove);
109 109
         }
110 110
 
111 111
         // Should we add try-catch?
112
-        if ( $options['trycatch'] ) {
112
+        if ($options['trycatch']) {
113 113
             $this->trycatch = true;
114 114
         }
115 115
 
116 116
         // force js in head?
117
-        if ( $options['forcehead'] ) {
117
+        if ($options['forcehead']) {
118 118
             $this->forcehead = true;
119 119
         } else {
120 120
             $this->forcehead = false;
121 121
         }
122 122
 
123
-        $this->forcehead = apply_filters( 'autoptimize_filter_js_forcehead', $this->forcehead );
123
+        $this->forcehead = apply_filters('autoptimize_filter_js_forcehead', $this->forcehead);
124 124
 
125 125
         // get cdn url
126 126
         $this->cdn_url = $options['cdn_url'];
@@ -135,26 +135,26 @@  discard block
 block discarded – undo
135 135
         $this->content = $this->hide_comments($this->content);
136 136
 
137 137
         // Get script files
138
-        if ( preg_match_all( '#<script.*</script>#Usmi', $this->content, $matches ) ) {
139
-            foreach( $matches[0] as $tag ) {
138
+        if (preg_match_all('#<script.*</script>#Usmi', $this->content, $matches)) {
139
+            foreach ($matches[0] as $tag) {
140 140
                 // only consider script aggregation for types whitelisted in should_aggregate-function
141 141
                 $should_aggregate = $this->should_aggregate($tag);
142
-                if ( ! $should_aggregate ) {
142
+                if (!$should_aggregate) {
143 143
                     $tag = '';
144 144
                     continue;
145 145
                 }
146 146
 
147
-                if ( preg_match( '#<script[^>]*src=("|\')([^>]*)("|\')#Usmi', $tag, $source ) ) {
147
+                if (preg_match('#<script[^>]*src=("|\')([^>]*)("|\')#Usmi', $tag, $source)) {
148 148
                     // non-inline script
149
-                    if ( $this->isremovable($tag, $this->jsremovables) ) {
150
-                        $this->content = str_replace( $tag, '', $this->content );
149
+                    if ($this->isremovable($tag, $this->jsremovables)) {
150
+                        $this->content = str_replace($tag, '', $this->content);
151 151
                         continue;
152 152
                     }
153 153
 
154 154
                     $origTag = null;
155
-                    $url = current( explode( '?', $source[2], 2 ) );
155
+                    $url = current(explode('?', $source[2], 2));
156 156
                     $path = $this->getpath($url);
157
-                    if ( false !== $path && preg_match( '#\.js$#', $path ) && $this->ismergeable($tag) ) {
157
+                    if (false !== $path && preg_match('#\.js$#', $path) && $this->ismergeable($tag)) {
158 158
                         // ok to optimize, add to array
159 159
                         $this->scripts[] = $path;
160 160
                     } else {
@@ -162,35 +162,35 @@  discard block
 block discarded – undo
162 162
                         $newTag  = $tag;
163 163
 
164 164
                         // non-mergeable script (excluded or dynamic or external)
165
-                        if ( is_array( $excludeJS ) ) {
165
+                        if (is_array($excludeJS)) {
166 166
                             // should we add flags?
167
-                            foreach ( $excludeJS as $exclTag => $exclFlags) {
168
-                                if ( false !== strpos( $origTag, $exclTag ) && in_array( $exclFlags, array( 'async', 'defer' ) ) ) {
169
-                                    $newTag = str_replace( '<script ', '<script ' . $exclFlags . ' ', $newTag );
167
+                            foreach ($excludeJS as $exclTag => $exclFlags) {
168
+                                if (false !== strpos($origTag, $exclTag) && in_array($exclFlags, array('async', 'defer'))) {
169
+                                    $newTag = str_replace('<script ', '<script '.$exclFlags.' ', $newTag);
170 170
                                 }
171 171
                             }
172 172
                         }
173 173
 
174 174
                         // Should we minify the non-aggregated script?
175
-                        if ( $path && apply_filters( 'autoptimize_filter_js_minify_excluded', true, $url ) ) {
176
-                            $minified_url = $this->minify_single( $path );
175
+                        if ($path && apply_filters('autoptimize_filter_js_minify_excluded', true, $url)) {
176
+                            $minified_url = $this->minify_single($path);
177 177
                             // replace orig URL with minified URL from cache if so
178
-                            if ( ! empty( $minified_url ) ) {
179
-                                $newTag = str_replace( $url, $minified_url, $newTag );
178
+                            if (!empty($minified_url)) {
179
+                                $newTag = str_replace($url, $minified_url, $newTag);
180 180
                             }
181 181
                         }
182 182
 
183
-                        if ( $this->ismovable($newTag) ) {
183
+                        if ($this->ismovable($newTag)) {
184 184
                             // can be moved, flags and all
185
-                            if ( $this->movetolast($newTag) )  {
185
+                            if ($this->movetolast($newTag)) {
186 186
                                 $this->move['last'][] = $newTag;
187 187
                             } else {
188 188
                                 $this->move['first'][] = $newTag;
189 189
                             }
190 190
                         } else {
191 191
                             // cannot be moved, so if flag was added re-inject altered tag immediately
192
-                            if ( $origTag !== $newTag ) {
193
-                                $this->content = str_replace( $origTag, $newTag, $this->content );
192
+                            if ($origTag !== $newTag) {
193
+                                $this->content = str_replace($origTag, $newTag, $this->content);
194 194
                                 $origTag = '';
195 195
                             }
196 196
                             // and forget about the $tag (not to be touched any more)
@@ -199,23 +199,23 @@  discard block
 block discarded – undo
199 199
                     }
200 200
                 } else {
201 201
                     // Inline script
202
-                    if ( $this->isremovable($tag, $this->jsremovables) ) {
203
-                        $this->content = str_replace( $tag, '', $this->content );
202
+                    if ($this->isremovable($tag, $this->jsremovables)) {
203
+                        $this->content = str_replace($tag, '', $this->content);
204 204
                         continue;
205 205
                     }
206 206
 
207 207
                     // unhide comments, as javascript may be wrapped in comment-tags for old times' sake
208 208
                     $tag = $this->restore_comments($tag);
209
-                    if ( $this->ismergeable($tag) && $this->include_inline ) {
210
-                        preg_match( '#<script.*>(.*)</script>#Usmi', $tag , $code );
211
-                        $code = preg_replace('#.*<!\[CDATA\[(?:\s*\*/)?(.*)(?://|/\*)\s*?\]\]>.*#sm', '$1', $code[1] );
212
-                        $code = preg_replace('/(?:^\\s*<!--\\s*|\\s*(?:\\/\\/)?\\s*-->\\s*$)/', '', $code );
213
-                        $this->scripts[] = 'INLINE;' . $code;
209
+                    if ($this->ismergeable($tag) && $this->include_inline) {
210
+                        preg_match('#<script.*>(.*)</script>#Usmi', $tag, $code);
211
+                        $code = preg_replace('#.*<!\[CDATA\[(?:\s*\*/)?(.*)(?://|/\*)\s*?\]\]>.*#sm', '$1', $code[1]);
212
+                        $code = preg_replace('/(?:^\\s*<!--\\s*|\\s*(?:\\/\\/)?\\s*-->\\s*$)/', '', $code);
213
+                        $this->scripts[] = 'INLINE;'.$code;
214 214
                     } else {
215 215
                         // Can we move this?
216
-                        $autoptimize_js_moveable = apply_filters( 'autoptimize_js_moveable', '', $tag );
217
-                        if ( $this->ismovable($tag) || '' !== $autoptimize_js_moveable ) {
218
-                            if ( $this->movetolast($tag) || 'last' === $autoptimize_js_moveable ) {
216
+                        $autoptimize_js_moveable = apply_filters('autoptimize_js_moveable', '', $tag);
217
+                        if ($this->ismovable($tag) || '' !== $autoptimize_js_moveable) {
218
+                            if ($this->movetolast($tag) || 'last' === $autoptimize_js_moveable) {
219 219
                                 $this->move['last'][] = $tag;
220 220
                             } else {
221 221
                                 $this->move['first'][] = $tag;
@@ -230,7 +230,7 @@  discard block
 block discarded – undo
230 230
                 }
231 231
 
232 232
                 //Remove the original script tag
233
-                $this->content = str_replace( $tag, '', $this->content );
233
+                $this->content = str_replace($tag, '', $this->content);
234 234
             }
235 235
 
236 236
             return true;
@@ -260,21 +260,21 @@  discard block
 block discarded – undo
260 260
         // We're only interested in the type attribute of the <script> tag itself, not any possible
261 261
         // inline code that might just contain the 'type=' string...
262 262
         $tag_parts = array();
263
-        preg_match( '#<(script[^>]*)>#i', $tag, $tag_parts);
263
+        preg_match('#<(script[^>]*)>#i', $tag, $tag_parts);
264 264
         $tag_without_contents = null;
265
-        if ( ! empty( $tag_parts[1] ) ) {
265
+        if (!empty($tag_parts[1])) {
266 266
             $tag_without_contents = $tag_parts[1];
267 267
         }
268 268
 
269
-        $has_type = ( strpos( $tag_without_contents, 'type' ) !== false );
269
+        $has_type = (strpos($tag_without_contents, 'type') !== false);
270 270
 
271 271
         $type_valid = false;
272
-        if ( $has_type ) {
273
-            $type_valid = (bool) preg_match( '/type\s*=\s*[\'"]?(?:text|application)\/(?:javascript|ecmascript)[\'"]?/i', $tag_without_contents );
272
+        if ($has_type) {
273
+            $type_valid = (bool) preg_match('/type\s*=\s*[\'"]?(?:text|application)\/(?:javascript|ecmascript)[\'"]?/i', $tag_without_contents);
274 274
         }
275 275
 
276 276
         $should_aggregate = false;
277
-        if ( ! $has_type || $type_valid ) {
277
+        if (!$has_type || $type_valid) {
278 278
             $should_aggregate = true;
279 279
         }
280 280
 
@@ -284,40 +284,40 @@  discard block
 block discarded – undo
284 284
     //Joins and optimizes JS
285 285
     public function minify()
286 286
     {
287
-        foreach ( $this->scripts as $script ) {
287
+        foreach ($this->scripts as $script) {
288 288
             // TODO/FIXME: some duplicate code here, can be reduced/simplified
289
-            if ( preg_match( '#^INLINE;#', $script ) ) {
289
+            if (preg_match('#^INLINE;#', $script)) {
290 290
                 // Inline script
291
-                $script = preg_replace( '#^INLINE;#', '', $script );
292
-                $script = rtrim( $script, ";\n\t\r" ) . ';';
291
+                $script = preg_replace('#^INLINE;#', '', $script);
292
+                $script = rtrim($script, ";\n\t\r").';';
293 293
                 // Add try-catch?
294
-                if ( $this->trycatch ) {
295
-                    $script = 'try{' . $script . '}catch(e){}';
294
+                if ($this->trycatch) {
295
+                    $script = 'try{'.$script.'}catch(e){}';
296 296
                 }
297
-                $tmpscript = apply_filters( 'autoptimize_js_individual_script', $script, '' );
298
-                if ( has_filter( 'autoptimize_js_individual_script' ) && ! empty( $tmpscript ) ) {
297
+                $tmpscript = apply_filters('autoptimize_js_individual_script', $script, '');
298
+                if (has_filter('autoptimize_js_individual_script') && !empty($tmpscript)) {
299 299
                     $script = $tmpscript;
300 300
                     $this->alreadyminified = true;
301 301
                 }
302
-                $this->jscode .= "\n" . $script;
302
+                $this->jscode .= "\n".$script;
303 303
             } else {
304 304
                 // External script
305
-                if ( false !== $script && file_exists( $script ) && is_readable( $script ) ) {
306
-                    $scriptsrc = file_get_contents( $script );
307
-                    $scriptsrc = preg_replace( '/\x{EF}\x{BB}\x{BF}/', '', $scriptsrc );
308
-                    $scriptsrc = rtrim( $scriptsrc, ";\n\t\r" ) . ';';
305
+                if (false !== $script && file_exists($script) && is_readable($script)) {
306
+                    $scriptsrc = file_get_contents($script);
307
+                    $scriptsrc = preg_replace('/\x{EF}\x{BB}\x{BF}/', '', $scriptsrc);
308
+                    $scriptsrc = rtrim($scriptsrc, ";\n\t\r").';';
309 309
                     // Add try-catch?
310
-                    if ( $this->trycatch ) {
311
-                        $scriptsrc = 'try{' . $scriptsrc . '}catch(e){}';
310
+                    if ($this->trycatch) {
311
+                        $scriptsrc = 'try{'.$scriptsrc.'}catch(e){}';
312 312
                     }
313
-                    $tmpscriptsrc = apply_filters( 'autoptimize_js_individual_script', $scriptsrc, $script );
314
-                    if ( has_filter( 'autoptimize_js_individual_script' ) && ! empty( $tmpscriptsrc ) ) {
313
+                    $tmpscriptsrc = apply_filters('autoptimize_js_individual_script', $scriptsrc, $script);
314
+                    if (has_filter('autoptimize_js_individual_script') && !empty($tmpscriptsrc)) {
315 315
                         $scriptsrc = $tmpscriptsrc;
316 316
                         $this->alreadyminified = true;
317
-                    } else if ( $this->can_inject_late($script) ) {
317
+                    } else if ($this->can_inject_late($script)) {
318 318
                         $scriptsrc = self::build_injectlater_marker($script, md5($scriptsrc));
319 319
                     }
320
-                    $this->jscode .= "\n" . $scriptsrc;
320
+                    $this->jscode .= "\n".$scriptsrc;
321 321
                 }/*else{
322 322
                     //Couldn't read JS. Maybe getpath isn't working?
323 323
                 }*/
@@ -325,32 +325,32 @@  discard block
 block discarded – undo
325 325
         }
326 326
 
327 327
         // Check for already-minified code
328
-        $this->md5hash = md5( $this->jscode );
328
+        $this->md5hash = md5($this->jscode);
329 329
         $ccheck = new autoptimizeCache($this->md5hash, 'js');
330
-        if ( $ccheck->check() ) {
330
+        if ($ccheck->check()) {
331 331
             $this->jscode = $ccheck->retrieve();
332 332
             return true;
333 333
         }
334
-        unset( $ccheck );
334
+        unset($ccheck);
335 335
 
336 336
         // $this->jscode has all the uncompressed code now.
337
-        if ( true !== $this->alreadyminified ) {
338
-            if ( apply_filters( 'autoptimize_js_do_minify', true ) ) {
339
-                $tmp_jscode = trim( JSMin::minify( $this->jscode ) );
340
-                if ( ! empty( $tmp_jscode ) ) {
337
+        if (true !== $this->alreadyminified) {
338
+            if (apply_filters('autoptimize_js_do_minify', true)) {
339
+                $tmp_jscode = trim(JSMin::minify($this->jscode));
340
+                if (!empty($tmp_jscode)) {
341 341
                     $this->jscode = $tmp_jscode;
342
-                    unset( $tmp_jscode );
342
+                    unset($tmp_jscode);
343 343
                 }
344
-                $this->jscode = $this->inject_minified( $this->jscode );
345
-                $this->jscode = apply_filters( 'autoptimize_js_after_minify', $this->jscode );
344
+                $this->jscode = $this->inject_minified($this->jscode);
345
+                $this->jscode = apply_filters('autoptimize_js_after_minify', $this->jscode);
346 346
                 return true;
347 347
             } else {
348
-                $this->jscode = $this->inject_minified( $this->jscode );
348
+                $this->jscode = $this->inject_minified($this->jscode);
349 349
                 return false;
350 350
             }
351 351
         }
352 352
 
353
-        $this->jscode = apply_filters( 'autoptimize_js_after_minify', $this->jscode );
353
+        $this->jscode = apply_filters('autoptimize_js_after_minify', $this->jscode);
354 354
         return true;
355 355
     }
356 356
 
@@ -358,11 +358,11 @@  discard block
 block discarded – undo
358 358
     public function cache()
359 359
     {
360 360
         $cache = new autoptimizeCache($this->md5hash, 'js');
361
-        if ( ! $cache->check() ) {
361
+        if (!$cache->check()) {
362 362
             // Cache our code
363 363
             $cache->cache($this->jscode, 'text/javascript');
364 364
         }
365
-        $this->url = AUTOPTIMIZE_CACHE_URL . $cache->getname();
365
+        $this->url = AUTOPTIMIZE_CACHE_URL.$cache->getname();
366 366
         $this->url = $this->url_replace_cdn($this->url);
367 367
     }
368 368
 
@@ -370,43 +370,43 @@  discard block
 block discarded – undo
370 370
     public function getcontent()
371 371
     {
372 372
         // Restore the full content
373
-        if ( ! empty( $this->restofcontent ) ) {
373
+        if (!empty($this->restofcontent)) {
374 374
             $this->content .= $this->restofcontent;
375 375
             $this->restofcontent = '';
376 376
         }
377 377
 
378 378
         // Add the scripts taking forcehead/ deferred (default) into account
379
-        if ( $this->forcehead ) {
380
-            $replaceTag = array( '</head>', 'before' );
379
+        if ($this->forcehead) {
380
+            $replaceTag = array('</head>', 'before');
381 381
             $defer = '';
382 382
         } else {
383
-            $replaceTag = array( '</body>', 'before' );
383
+            $replaceTag = array('</body>', 'before');
384 384
             $defer = 'defer ';
385 385
         }
386 386
 
387
-        $defer = apply_filters( 'autoptimize_filter_js_defer', $defer );
387
+        $defer = apply_filters('autoptimize_filter_js_defer', $defer);
388 388
 
389
-        $bodyreplacementpayload = '<script type="text/javascript" ' . $defer . 'src="' . $this->url . '"></script>';
390
-        $bodyreplacementpayload = apply_filters( 'autoptimize_filter_js_bodyreplacementpayload', $bodyreplacementpayload );
389
+        $bodyreplacementpayload = '<script type="text/javascript" '.$defer.'src="'.$this->url.'"></script>';
390
+        $bodyreplacementpayload = apply_filters('autoptimize_filter_js_bodyreplacementpayload', $bodyreplacementpayload);
391 391
 
392
-        $bodyreplacement = implode( '', $this->move['first'] );
392
+        $bodyreplacement = implode('', $this->move['first']);
393 393
         $bodyreplacement .= $bodyreplacementpayload;
394
-        $bodyreplacement .= implode( '', $this->move['last'] );
394
+        $bodyreplacement .= implode('', $this->move['last']);
395 395
 
396
-        $replaceTag = apply_filters( 'autoptimize_filter_js_replacetag', $replaceTag );
396
+        $replaceTag = apply_filters('autoptimize_filter_js_replacetag', $replaceTag);
397 397
 
398
-        if ( strlen( $this->jscode ) > 0 ) {
399
-            $this->inject_in_html( $bodyreplacement, $replaceTag );
398
+        if (strlen($this->jscode) > 0) {
399
+            $this->inject_in_html($bodyreplacement, $replaceTag);
400 400
         }
401 401
 
402 402
         // Restore comments.
403
-        $this->content = $this->restore_comments( $this->content );
403
+        $this->content = $this->restore_comments($this->content);
404 404
 
405 405
         // Restore IE hacks.
406
-        $this->content = $this->restore_iehacks( $this->content );
406
+        $this->content = $this->restore_iehacks($this->content);
407 407
 
408 408
         // Restore noptimize.
409
-        $this->content = $this->restore_noptimize( $this->content );
409
+        $this->content = $this->restore_noptimize($this->content);
410 410
 
411 411
         // Return the modified HTML.
412 412
         return $this->content;
@@ -415,32 +415,32 @@  discard block
 block discarded – undo
415 415
     // Checks against the white- and blacklists
416 416
     private function ismergeable($tag)
417 417
     {
418
-        if ( ! $this->aggregate ) {
418
+        if (!$this->aggregate) {
419 419
             return false;
420 420
         }
421 421
 
422
-        if ( ! empty( $this->whitelist ) ) {
423
-            foreach ( $this->whitelist as $match ) {
424
-                if (false !== strpos( $tag, $match ) ) {
422
+        if (!empty($this->whitelist)) {
423
+            foreach ($this->whitelist as $match) {
424
+                if (false !== strpos($tag, $match)) {
425 425
                     return true;
426 426
                 }
427 427
             }
428 428
             // no match with whitelist
429 429
             return false;
430 430
         } else {
431
-            foreach($this->domove as $match) {
432
-                if ( false !== strpos( $tag, $match ) ) {
431
+            foreach ($this->domove as $match) {
432
+                if (false !== strpos($tag, $match)) {
433 433
                     // Matched something
434 434
                     return false;
435 435
                 }
436 436
             }
437 437
 
438
-            if ( $this->movetolast($tag) ) {
438
+            if ($this->movetolast($tag)) {
439 439
                 return false;
440 440
             }
441 441
 
442
-            foreach( $this->dontmove as $match ) {
443
-                if ( false !== strpos( $tag, $match ) ) {
442
+            foreach ($this->dontmove as $match) {
443
+                if (false !== strpos($tag, $match)) {
444 444
                     // Matched something
445 445
                     return false;
446 446
                 }
@@ -454,23 +454,23 @@  discard block
 block discarded – undo
454 454
     // Checks agains the blacklist
455 455
     private function ismovable($tag)
456 456
     {
457
-        if ( true !== $this->include_inline || apply_filters( 'autoptimize_filter_js_unmovable', true ) ) {
457
+        if (true !== $this->include_inline || apply_filters('autoptimize_filter_js_unmovable', true)) {
458 458
             return false;
459 459
         }
460 460
 
461
-        foreach ( $this->domove as $match ) {
462
-            if ( false !== strpos( $tag, $match ) ) {
461
+        foreach ($this->domove as $match) {
462
+            if (false !== strpos($tag, $match)) {
463 463
                 // Matched something
464 464
                 return true;
465 465
             }
466 466
         }
467 467
 
468
-        if ( $this->movetolast($tag) ) {
468
+        if ($this->movetolast($tag)) {
469 469
             return true;
470 470
         }
471 471
 
472
-        foreach ( $this->dontmove as $match ) {
473
-            if ( false !== strpos( $tag, $match ) ) {
472
+        foreach ($this->dontmove as $match) {
473
+            if (false !== strpos($tag, $match)) {
474 474
                 // Matched something
475 475
                 return false;
476 476
             }
@@ -482,8 +482,8 @@  discard block
 block discarded – undo
482 482
 
483 483
     private function movetolast($tag)
484 484
     {
485
-        foreach ( $this->domovelast as $match ) {
486
-            if ( false !== strpos( $tag, $match ) ) {
485
+        foreach ($this->domovelast as $match) {
486
+            if (false !== strpos($tag, $match)) {
487 487
                 // Matched, return true
488 488
                 return true;
489 489
             }
@@ -504,11 +504,11 @@  discard block
 block discarded – undo
504 504
      * @return bool
505 505
      */
506 506
     private function can_inject_late($jsPath) {
507
-        $consider_minified_array = apply_filters( 'autoptimize_filter_js_consider_minified', false );
508
-        if ( true !== $this->inject_min_late ) {
507
+        $consider_minified_array = apply_filters('autoptimize_filter_js_consider_minified', false);
508
+        if (true !== $this->inject_min_late) {
509 509
             // late-inject turned off
510 510
             return false;
511
-        } else if ( ( false === strpos( $jsPath, 'min.js' ) ) && ( false === strpos( $jsPath, 'wp-includes/js/jquery/jquery.js' ) ) && ( str_replace( $consider_minified_array, '', $jsPath ) === $jsPath ) ) {
511
+        } else if ((false === strpos($jsPath, 'min.js')) && (false === strpos($jsPath, 'wp-includes/js/jquery/jquery.js')) && (str_replace($consider_minified_array, '', $jsPath) === $jsPath)) {
512 512
             // file not minified based on filename & filter
513 513
             return false;
514 514
         } else {
@@ -535,26 +535,26 @@  discard block
 block discarded – undo
535 535
      *
536 536
      * @return bool|string Url pointing to the minified js file or false.
537 537
      */
538
-    public function minify_single( $filepath, $cache_miss = false )
538
+    public function minify_single($filepath, $cache_miss = false)
539 539
     {
540
-        $contents = $this->prepare_minify_single( $filepath );
540
+        $contents = $this->prepare_minify_single($filepath);
541 541
 
542
-        if ( empty( $contents ) ) {
542
+        if (empty($contents)) {
543 543
             return false;
544 544
         }
545 545
 
546 546
         // Check cache.
547
-        $hash  = 'single_' . md5( $contents );
548
-        $cache = new autoptimizeCache( $hash, 'js' );
547
+        $hash  = 'single_'.md5($contents);
548
+        $cache = new autoptimizeCache($hash, 'js');
549 549
 
550 550
         // If not in cache already, minify...
551
-        if ( ! $cache->check() || $cache_miss ) {
552
-            $contents = trim( JSMin::minify( $contents ) );
551
+        if (!$cache->check() || $cache_miss) {
552
+            $contents = trim(JSMin::minify($contents));
553 553
             // Store in cache.
554
-            $cache->cache( $contents, 'text/javascript' );
554
+            $cache->cache($contents, 'text/javascript');
555 555
         }
556 556
 
557
-        $url = $this->build_minify_single_url( $cache );
557
+        $url = $this->build_minify_single_url($cache);
558 558
 
559 559
         return $url;
560 560
     }
Please login to merge, or discard this patch.
php/persist-admin-notices-dismissal/persist-admin-notices-dismissal.php 1 patch
Spacing   +30 added lines, -30 removed lines patch added patch discarded remove patch
@@ -28,11 +28,11 @@  discard block
 block discarded – undo
28 28
 /**
29 29
  * Exit if called directly.
30 30
  */
31
-if ( ! defined( 'ABSPATH' ) ) {
31
+if (!defined('ABSPATH')) {
32 32
 	die;
33 33
 }
34 34
 
35
-if ( ! class_exists( 'PAnD' ) ) {
35
+if (!class_exists('PAnD')) {
36 36
 
37 37
 	/**
38 38
 	 * Class PAnD
@@ -43,8 +43,8 @@  discard block
 block discarded – undo
43 43
 		 * Init hooks.
44 44
 		 */
45 45
 		public static function init() {
46
-			add_action( 'admin_enqueue_scripts', array( __CLASS__, 'load_script' ) );
47
-			add_action( 'wp_ajax_dismiss_admin_notice', array( __CLASS__, 'dismiss_admin_notice' ) );
46
+			add_action('admin_enqueue_scripts', array(__CLASS__, 'load_script'));
47
+			add_action('wp_ajax_dismiss_admin_notice', array(__CLASS__, 'dismiss_admin_notice'));
48 48
 		}
49 49
 
50 50
 		/**
@@ -52,14 +52,14 @@  discard block
 block discarded – undo
52 52
 		 */
53 53
 		public static function load_script() {
54 54
 
55
-			if ( is_customize_preview() ) {
55
+			if (is_customize_preview()) {
56 56
 				return;
57 57
 			}
58 58
 
59 59
 			wp_enqueue_script(
60 60
 				'dismissible-notices',
61
-				plugins_url( 'dismiss-notice.js', __FILE__ ),
62
-				array( 'jquery', 'common' ),
61
+				plugins_url('dismiss-notice.js', __FILE__),
62
+				array('jquery', 'common'),
63 63
 				false,
64 64
 				true
65 65
 			);
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
 				'dismissible-notices',
69 69
 				'dismissible_notice',
70 70
 				array(
71
-					'nonce' => wp_create_nonce( 'dismissible-notice' ),
71
+					'nonce' => wp_create_nonce('dismissible-notice'),
72 72
 				)
73 73
 			);
74 74
 		}
@@ -78,17 +78,17 @@  discard block
 block discarded – undo
78 78
 		 * Uses check_ajax_referer to verify nonce.
79 79
 		 */
80 80
 		public static function dismiss_admin_notice() {
81
-			$option_name        = sanitize_text_field( $_POST['option_name'] );
82
-			$dismissible_length = sanitize_text_field( $_POST['dismissible_length'] );
81
+			$option_name        = sanitize_text_field($_POST['option_name']);
82
+			$dismissible_length = sanitize_text_field($_POST['dismissible_length']);
83 83
 
84
-			if ( 'forever' != $dismissible_length ) {
84
+			if ('forever' != $dismissible_length) {
85 85
 				// If $dismissible_length is not an integer default to 1
86
-				$dismissible_length = ( 0 == absint( $dismissible_length ) ) ? 1 : $dismissible_length;
87
-				$dismissible_length = strtotime( absint( $dismissible_length ) . ' days' );
86
+				$dismissible_length = (0 == absint($dismissible_length)) ? 1 : $dismissible_length;
87
+				$dismissible_length = strtotime(absint($dismissible_length).' days');
88 88
 			}
89 89
 
90
-			check_ajax_referer( 'dismissible-notice', 'nonce' );
91
-			self::set_admin_notice_cache( $option_name, $dismissible_length );
90
+			check_ajax_referer('dismissible-notice', 'nonce');
91
+			self::set_admin_notice_cache($option_name, $dismissible_length);
92 92
 			wp_die();
93 93
 		}
94 94
 
@@ -99,14 +99,14 @@  discard block
 block discarded – undo
99 99
 		 *
100 100
 		 * @return bool
101 101
 		 */
102
-		public static function is_admin_notice_active( $arg ) {
103
-			$array       = explode( '-', $arg );
104
-			$length      = array_pop( $array );
105
-			$option_name = implode( '-', $array );
106
-			$db_record   = self::get_admin_notice_cache( $option_name );
107
-			if ( 'forever' == $db_record ) {
102
+		public static function is_admin_notice_active($arg) {
103
+			$array       = explode('-', $arg);
104
+			$length      = array_pop($array);
105
+			$option_name = implode('-', $array);
106
+			$db_record   = self::get_admin_notice_cache($option_name);
107
+			if ('forever' == $db_record) {
108 108
 				return false;
109
-			} elseif ( absint( $db_record ) >= time() ) {
109
+			} elseif (absint($db_record) >= time()) {
110 110
 				return false;
111 111
 			} else {
112 112
 				return true;
@@ -122,15 +122,15 @@  discard block
 block discarded – undo
122 122
 		 *
123 123
 		 * @return array|bool The timeout. False if expired.
124 124
 		 */
125
-		public static function get_admin_notice_cache( $id = false ) {
126
-			if ( ! $id ) {
125
+		public static function get_admin_notice_cache($id = false) {
126
+			if (!$id) {
127 127
 				return false;
128 128
 			}
129
-			$cache_key = 'pand-' . md5( $id );
130
-			$timeout   = get_site_option( $cache_key );
129
+			$cache_key = 'pand-'.md5($id);
130
+			$timeout   = get_site_option($cache_key);
131 131
 			$timeout   = 'forever' === $timeout ? time() + 60 : $timeout;
132 132
 
133
-			if ( empty( $timeout ) || time() > $timeout ) {
133
+			if (empty($timeout) || time() > $timeout) {
134 134
 				return false;
135 135
 			}
136 136
 
@@ -147,9 +147,9 @@  discard block
 block discarded – undo
147 147
 		 *
148 148
 		 * @return bool
149 149
 		 */
150
-		public static function set_admin_notice_cache( $id, $timeout ) {
151
-			$cache_key = 'pand-' . md5( $id );
152
-			update_site_option( $cache_key, $timeout );
150
+		public static function set_admin_notice_cache($id, $timeout) {
151
+			$cache_key = 'pand-'.md5($id);
152
+			update_site_option($cache_key, $timeout);
153 153
 
154 154
 			return true;
155 155
 		}
Please login to merge, or discard this patch.
classes/external/php/jsmin.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
     protected $inputLength = 0;
71 71
     protected $lookAhead   = null;
72 72
     protected $output      = '';
73
-    protected $lastByteOut  = '';
73
+    protected $lastByteOut = '';
74 74
     protected $keptComment = '';
75 75
 
76 76
     /**
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
         }
107 107
 
108 108
         $mbIntEnc = null;
109
-        if (function_exists('mb_strlen') && ((int)ini_get('mbstring.func_overload') & 2)) {
109
+        if (function_exists('mb_strlen') && ((int) ini_get('mbstring.func_overload') & 2)) {
110 110
             $mbIntEnc = mb_internal_encoding();
111 111
             mb_internal_encoding('8bit');
112 112
         }
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
                         && ($this->b === $this->lastByteOut)) {
129 129
                     // Don't delete this space. If we do, the addition/subtraction
130 130
                     // could be parsed as a post-increment
131
-                } elseif (! $this->isAlphaNum($this->b)) {
131
+                } elseif (!$this->isAlphaNum($this->b)) {
132 132
                     $command = self::ACTION_DELETE_A;
133 133
                 }
134 134
             } elseif ($this->a === "\n") {
@@ -139,10 +139,10 @@  discard block
 block discarded – undo
139 139
                     // otherwise mb_strpos will give WARNING
140 140
                 } elseif ($this->b === null
141 141
                           || (false === strpos('{[(+-!~', $this->b)
142
-                              && ! $this->isAlphaNum($this->b))) {
142
+                              && !$this->isAlphaNum($this->b))) {
143 143
                     $command = self::ACTION_DELETE_A;
144 144
                 }
145
-            } elseif (! $this->isAlphaNum($this->a)) {
145
+            } elseif (!$this->isAlphaNum($this->a)) {
146 146
                 if ($this->b === ' '
147 147
                     || ($this->b === "\n"
148 148
                         && (false === strpos('}])+-"\'', $this->a)))) {
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
                 if ($this->a === "'" || $this->a === '"' || $this->a === '`') { // string/template literal
200 200
                     $delimiter = $this->a;
201 201
                     $str = $this->a; // in case needed for exception
202
-                    for(;;) {
202
+                    for (;;) {
203 203
                         $this->output .= $this->a;
204 204
                         $this->lastByteOut = $this->a;
205 205
 
@@ -229,13 +229,13 @@  discard block
 block discarded – undo
229 229
             case self::ACTION_DELETE_A_B: // 3
230 230
                 $this->b = $this->next();
231 231
                 if ($this->b === '/' && $this->isRegexpLiteral()) {
232
-                    $this->output .= $this->a . $this->b;
232
+                    $this->output .= $this->a.$this->b;
233 233
                     $pattern = '/'; // keep entire pattern in case we need to report it in the exception
234
-                    for(;;) {
234
+                    for (;;) {
235 235
                         $this->a = $this->get();
236 236
                         $pattern .= $this->a;
237 237
                         if ($this->a === '[') {
238
-                            for(;;) {
238
+                            for (;;) {
239 239
                                 $this->output .= $this->a;
240 240
                                 $this->a = $this->get();
241 241
                                 $pattern .= $this->a;
@@ -250,7 +250,7 @@  discard block
 block discarded – undo
250 250
                                 if ($this->isEOF($this->a)) {
251 251
                                     throw new JSMin_UnterminatedRegExpException(
252 252
                                         "JSMin: Unterminated set in RegExp at byte "
253
-                                            . $this->inputIndex .": {$pattern}");
253
+                                            . $this->inputIndex.": {$pattern}");
254 254
                                 }
255 255
                             }
256 256
                         }
@@ -295,7 +295,7 @@  discard block
 block discarded – undo
295 295
 
296 296
         // if the "/" follows a keyword, it must be a regexp, otherwise it's best to assume division
297 297
 
298
-        $subject = $this->output . trim($this->a);
298
+        $subject = $this->output.trim($this->a);
299 299
         if (!preg_match('/(?:case|else|in|return|typeof)$/', $subject, $m)) {
300 300
             // not a keyword
301 301
             return false;
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
     {
407 407
         $this->get();
408 408
         $comment = '';
409
-        for(;;) {
409
+        for (;;) {
410 410
             $get = $this->get();
411 411
             if ($get === '*') {
412 412
                 if ($this->peek() === '/') { // end of comment reached
@@ -417,7 +417,7 @@  discard block
 block discarded – undo
417 417
                             // don't prepend a newline if two comments right after one another
418 418
                             $this->keptComment = "\n";
419 419
                         }
420
-                        $this->keptComment .= "/*!" . substr($comment, 1) . "*/\n";
420
+                        $this->keptComment .= "/*!".substr($comment, 1)."*/\n";
421 421
                     } else if (preg_match('/^@(?:cc_on|if|elif|else|end)\\b/', $comment)) {
422 422
                         // IE conditional
423 423
                         $this->keptComment .= "/*{$comment}*/";
Please login to merge, or discard this patch.
tests/test-ao.php 1 patch
Spacing   +378 added lines, -378 removed lines patch added patch discarded remove patch
@@ -10,17 +10,17 @@  discard block
 block discarded – undo
10 10
     protected $ao;
11 11
 
12 12
     protected static $flexible_url_parts_js = [
13
-        'default'          => 'wp-content/cache/autoptimize/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
14
-        'custom'           => 'wp-content/c/ao/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
15
-        'multisite'        => 'wp-content/cache/autoptimize/1/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
16
-        'multisite_custom' => 'wp-content/c/ao/1/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
13
+        'default'          => 'wp-content/cache/autoptimize/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
14
+        'custom'           => 'wp-content/c/ao/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
15
+        'multisite'        => 'wp-content/cache/autoptimize/1/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
16
+        'multisite_custom' => 'wp-content/c/ao/1/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
17 17
     ];
18 18
 
19 19
     protected static $flexible_url_parts_css = [
20
-        'default'          => 'wp-content/cache/autoptimize/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
21
-        'custom'           => 'wp-content/c/ao/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
22
-        'multisite'        => 'wp-content/cache/autoptimize/1/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
23
-        'multisite_custom' => 'wp-content/c/ao/1/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
20
+        'default'          => 'wp-content/cache/autoptimize/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
21
+        'custom'           => 'wp-content/c/ao/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
22
+        'multisite'        => 'wp-content/cache/autoptimize/1/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
23
+        'multisite_custom' => 'wp-content/c/ao/1/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
24 24
     ];
25 25
 
26 26
     protected function getAoStylesDefaultOptions()
@@ -28,16 +28,16 @@  discard block
 block discarded – undo
28 28
         $conf = autoptimizeConfig::instance();
29 29
 
30 30
         return [
31
-            'aggregate'      => $conf->get( 'autoptimize_css_aggregate' ),
32
-            'justhead'       => $conf->get( 'autoptimize_css_justhead' ),
33
-            'datauris'       => $conf->get( 'autoptimize_css_datauris' ),
34
-            'defer'          => $conf->get( 'autoptimize_css_defer' ),
35
-            'defer_inline'   => $conf->get( 'autoptimize_css_defer_inline' ),
36
-            'inline'         => $conf->get( 'autoptimize_css_inline' ),
37
-            'css_exclude'    => $conf->get( 'autoptimize_css_exclude' ),
38
-            'cdn_url'        => $conf->get( 'autoptimize_cdn_url' ),
39
-            'include_inline' => $conf->get( 'autoptimize_css_include_inline' ),
40
-            'nogooglefont'   => $conf->get( 'autoptimize_css_nogooglefont' ),
31
+            'aggregate'      => $conf->get('autoptimize_css_aggregate'),
32
+            'justhead'       => $conf->get('autoptimize_css_justhead'),
33
+            'datauris'       => $conf->get('autoptimize_css_datauris'),
34
+            'defer'          => $conf->get('autoptimize_css_defer'),
35
+            'defer_inline'   => $conf->get('autoptimize_css_defer_inline'),
36
+            'inline'         => $conf->get('autoptimize_css_inline'),
37
+            'css_exclude'    => $conf->get('autoptimize_css_exclude'),
38
+            'cdn_url'        => $conf->get('autoptimize_cdn_url'),
39
+            'include_inline' => $conf->get('autoptimize_css_include_inline'),
40
+            'nogooglefont'   => $conf->get('autoptimize_css_nogooglefont'),
41 41
         ];
42 42
     }
43 43
 
@@ -46,13 +46,13 @@  discard block
 block discarded – undo
46 46
         $conf = autoptimizeConfig::instance();
47 47
 
48 48
         return [
49
-            'aggregate'      => $conf->get( 'autoptimize_js_aggregate' ),
50
-            'justhead'       => $conf->get( 'autoptimize_js_justhead' ),
51
-            'forcehead'      => $conf->get( 'autoptimize_js_forcehead' ),
52
-            'trycatch'       => $conf->get( 'autoptimize_js_trycatch' ),
53
-            'js_exclude'     => $conf->get( 'autoptimize_js_exclude' ),
54
-            'cdn_url'        => $conf->get( 'autoptimize_cdn_url' ),
55
-            'include_inline' => $conf->get( 'autoptimize_js_include_inline' ),
49
+            'aggregate'      => $conf->get('autoptimize_js_aggregate'),
50
+            'justhead'       => $conf->get('autoptimize_js_justhead'),
51
+            'forcehead'      => $conf->get('autoptimize_js_forcehead'),
52
+            'trycatch'       => $conf->get('autoptimize_js_trycatch'),
53
+            'js_exclude'     => $conf->get('autoptimize_js_exclude'),
54
+            'cdn_url'        => $conf->get('autoptimize_cdn_url'),
55
+            'include_inline' => $conf->get('autoptimize_js_include_inline'),
56 56
         ];
57 57
     }
58 58
 
@@ -62,34 +62,34 @@  discard block
 block discarded – undo
62 62
     protected function get_urls()
63 63
     {
64 64
         static $site_url = null;
65
-        if ( null === $site_url ) {
65
+        if (null === $site_url) {
66 66
             $site_url = site_url();
67 67
         }
68 68
 
69 69
         static $cdn_url = null;
70
-        if ( null === $cdn_url ) {
71
-            $cdn_url = get_option( 'autoptimize_cdn_url' );
70
+        if (null === $cdn_url) {
71
+            $cdn_url = get_option('autoptimize_cdn_url');
72 72
         }
73 73
 
74 74
         static $imgopt_host = null;
75
-        if ( null === $imgopt_host ) {
76
-            $imgopt_host = rtrim( autoptimizeExtra::get_imgopt_host_wrapper(), '/' );
75
+        if (null === $imgopt_host) {
76
+            $imgopt_host = rtrim(autoptimizeExtra::get_imgopt_host_wrapper(), '/');
77 77
         }
78 78
 
79 79
         static $urls = [];
80 80
 
81
-        if ( empty( $urls ) ) {
81
+        if (empty($urls)) {
82 82
             $parts = autoptimizeUtils::get_ao_wp_site_url_parts();
83 83
             $urls  = [
84 84
                 'siteurl'    => $site_url,
85
-                'prsiteurl'  => '//' . str_replace( array( 'http://', 'https://' ), '', $site_url ),
86
-                'wwwsiteurl' => $parts['scheme'] . '://www.' . str_replace( 'www.', '', $parts['host'] ),
85
+                'prsiteurl'  => '//'.str_replace(array('http://', 'https://'), '', $site_url),
86
+                'wwwsiteurl' => $parts['scheme'].'://www.'.str_replace('www.', '', $parts['host']),
87 87
                 'cdnurl'     => $cdn_url,
88 88
                 'imgopthost' => $imgopt_host,
89 89
                 'subfolder'  => '',
90 90
             ];
91 91
 
92
-            if ( AO_TEST_SUBFOLDER_INSTALL ) {
92
+            if (AO_TEST_SUBFOLDER_INSTALL) {
93 93
                 $urls['subfolder'] = 'wordpress/';
94 94
             }
95 95
         }
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
      */
103 103
     public function setUp()
104 104
     {
105
-        $this->ao = new autoptimizeMain( AUTOPTIMIZE_PLUGIN_VERSION, AUTOPTIMIZE_PLUGIN_FILE );
105
+        $this->ao = new autoptimizeMain(AUTOPTIMIZE_PLUGIN_VERSION, AUTOPTIMIZE_PLUGIN_FILE);
106 106
 
107 107
         parent::setUp();
108 108
     }
@@ -121,8 +121,8 @@  discard block
 block discarded – undo
121 121
             'autoptimize_filter_css_inlinesize',
122 122
             'autoptimize_filter_css_fonts_cdn',
123 123
         );
124
-        foreach ( $filter_tags as $filter_tag ) {
125
-            remove_all_filters( $filter_tag );
124
+        foreach ($filter_tags as $filter_tag) {
125
+            remove_all_filters($filter_tag);
126 126
         }
127 127
 
128 128
         parent::tearDown();
@@ -246,15 +246,15 @@  discard block
 block discarded – undo
246 246
     protected function get_test_markup_output()
247 247
     {
248 248
         $key = 'default';
249
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
249
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
250 250
             $key = 'custom';
251 251
         }
252 252
 
253
-        $csspart = self::$flexible_url_parts_css[ $key ];
254
-        $jspart  = self::$flexible_url_parts_js[ $key ];
253
+        $csspart = self::$flexible_url_parts_css[$key];
254
+        $jspart  = self::$flexible_url_parts_js[$key];
255 255
 
256 256
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
257
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
257
+        if (AO_TEST_SUBFOLDER_INSTALL) {
258 258
             $csshash = '56398de576d59887e88e3011715250e0';
259 259
         }
260 260
 
@@ -313,15 +313,15 @@  discard block
 block discarded – undo
313 313
     protected function get_test_markup_output_ms()
314 314
     {
315 315
         $key = 'multisite';
316
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
316
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
317 317
             $key = 'multisite_custom';
318 318
         }
319 319
 
320
-        $csspart = self::$flexible_url_parts_css[ $key ];
321
-        $jspart  = self::$flexible_url_parts_js[ $key ];
320
+        $csspart = self::$flexible_url_parts_css[$key];
321
+        $jspart  = self::$flexible_url_parts_js[$key];
322 322
 
323 323
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
324
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
324
+        if (AO_TEST_SUBFOLDER_INSTALL) {
325 325
             $csshash = '56398de576d59887e88e3011715250e0';
326 326
         }
327 327
 
@@ -380,15 +380,15 @@  discard block
 block discarded – undo
380 380
     protected function get_test_markup_output_inline_defer()
381 381
     {
382 382
         $key = 'default';
383
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
383
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
384 384
             $key = 'custom';
385 385
         }
386 386
 
387
-        $csspart = self::$flexible_url_parts_css[ $key ];
388
-        $jspart  = self::$flexible_url_parts_js[ $key ];
387
+        $csspart = self::$flexible_url_parts_css[$key];
388
+        $jspart  = self::$flexible_url_parts_js[$key];
389 389
 
390 390
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
391
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
391
+        if (AO_TEST_SUBFOLDER_INSTALL) {
392 392
             $csshash = '56398de576d59887e88e3011715250e0';
393 393
         }
394 394
 
@@ -447,15 +447,15 @@  discard block
 block discarded – undo
447 447
     protected function get_test_markup_output_inline_defer_ms()
448 448
     {
449 449
         $key = 'multisite';
450
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
450
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
451 451
             $key = 'multisite_custom';
452 452
         }
453 453
 
454
-        $csspart = self::$flexible_url_parts_css[ $key ];
455
-        $jspart  = self::$flexible_url_parts_js[ $key ];
454
+        $csspart = self::$flexible_url_parts_css[$key];
455
+        $jspart  = self::$flexible_url_parts_js[$key];
456 456
 
457 457
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
458
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
458
+        if (AO_TEST_SUBFOLDER_INSTALL) {
459 459
             $csshash = '56398de576d59887e88e3011715250e0';
460 460
         }
461 461
 
@@ -514,11 +514,11 @@  discard block
 block discarded – undo
514 514
     /**
515 515
      * @dataProvider provider_test_rewrite_markup_with_cdn
516 516
      */
517
-    function test_rewrite_markup_with_cdn( $input, $expected )
517
+    function test_rewrite_markup_with_cdn($input, $expected)
518 518
     {
519
-        $actual = $this->ao->end_buffering( $input );
519
+        $actual = $this->ao->end_buffering($input);
520 520
 
521
-        $this->assertEquals( $expected, $actual );
521
+        $this->assertEquals($expected, $actual);
522 522
     }
523 523
 
524 524
     public function provider_test_rewrite_markup_with_cdn()
@@ -530,7 +530,7 @@  discard block
 block discarded – undo
530 530
                 $this->get_test_markup(),
531 531
                 // Expected output.
532 532
                 // TODO/FIXME: This seemed like the fastest way to get MS crude test to pass...
533
-                ( is_multisite() ? $this->get_test_markup_output_ms() : $this->get_test_markup_output() ),
533
+                (is_multisite() ? $this->get_test_markup_output_ms() : $this->get_test_markup_output()),
534 534
             ),
535 535
 
536 536
         );
@@ -590,12 +590,12 @@  discard block
 block discarded – undo
590 590
 }
591 591
 CSS;
592 592
 
593
-        $instance = new autoptimizeStyles( $css_in );
594
-        $instance->setOption( 'cdn_url', $cdnurl );
593
+        $instance = new autoptimizeStyles($css_in);
594
+        $instance->setOption('cdn_url', $cdnurl);
595 595
 
596
-        $css_actual = $instance->rewrite_assets( $css_in );
596
+        $css_actual = $instance->rewrite_assets($css_in);
597 597
 
598
-        $this->assertEquals( $css_expected, $css_actual );
598
+        $this->assertEquals($css_expected, $css_actual);
599 599
     }
600 600
 
601 601
     public function test_default_cssmin_minifier()
@@ -629,21 +629,21 @@  discard block
 block discarded – undo
629 629
 .bg{background:url('img/something.svg')}.bg-no-quote{background:url(img/something.svg)}.bg-double-quotes{background:url("img/something.svg")}.whitespaces{background:url ("../../somewhere-else/svg.svg")}.host-relative{background:url("/img/something.svg")}.protocol-relative{background:url("//something/somewhere/example.png")}@font-face{font-family:'Roboto';font-style:normal;font-weight:100;src:url(../fonts/roboto-v15-latin-ext_latin-100.eot);src:local('Roboto Thin'),local('Roboto-Thin'),url(../fonts/roboto-v15-latin-ext_latin-100.eot?#iefix) format('embedded-opentype'),url(../fonts/roboto-v15-latin-ext_latin-100.woff2) format('woff2'),url(../fonts/roboto-v15-latin-ext_latin-100.woff) format('woff'),url(../fonts/roboto-v15-latin-ext_latin-100.ttf) format('truetype'),url(../fonts/roboto-v15-latin-ext_latin-100.svg#Roboto) format('svg')}
630 630
 CSS;
631 631
 
632
-        $instance = new autoptimizeStyles( $css );
633
-        $minified = $instance->run_minifier_on( $css );
632
+        $instance = new autoptimizeStyles($css);
633
+        $minified = $instance->run_minifier_on($css);
634 634
 
635
-        $this->assertEquals( $expected, $minified );
635
+        $this->assertEquals($expected, $minified);
636 636
     }
637 637
 
638 638
     /**
639 639
      * @dataProvider provider_test_should_aggregate_script_types
640 640
      */
641
-    public function test_should_aggregate_script_types( $input, $expected )
641
+    public function test_should_aggregate_script_types($input, $expected)
642 642
     {
643
-        $instance = new autoptimizeScripts( '' );
644
-        $actual   = $instance->should_aggregate( $input );
643
+        $instance = new autoptimizeScripts('');
644
+        $actual   = $instance->should_aggregate($input);
645 645
 
646
-        $this->assertEquals( $expected, $actual );
646
+        $this->assertEquals($expected, $actual);
647 647
     }
648 648
 
649 649
     public function provider_test_should_aggregate_script_types()
@@ -705,11 +705,11 @@  discard block
 block discarded – undo
705 705
     /**
706 706
      * @dataProvider provider_is_valid_buffer
707 707
      */
708
-    public function test_valid_buffers( $input, $expected )
708
+    public function test_valid_buffers($input, $expected)
709 709
     {
710
-        $actual = $this->ao->is_valid_buffer( $input );
710
+        $actual = $this->ao->is_valid_buffer($input);
711 711
 
712
-        $this->assertEquals( $expected, $actual );
712
+        $this->assertEquals($expected, $actual);
713 713
     }
714 714
 
715 715
     public function provider_is_valid_buffer()
@@ -771,11 +771,11 @@  discard block
 block discarded – undo
771 771
     /**
772 772
      * @dataProvider provider_is_amp_markup
773 773
      */
774
-    public function test_autoptimize_is_amp_markup( $input, $expected )
774
+    public function test_autoptimize_is_amp_markup($input, $expected)
775 775
     {
776
-        $actual = autoptimizeMain::is_amp_markup( $input );
776
+        $actual = autoptimizeMain::is_amp_markup($input);
777 777
 
778
-        $this->assertEquals( $expected, $actual );
778
+        $this->assertEquals($expected, $actual);
779 779
     }
780 780
 
781 781
     public function provider_is_amp_markup()
@@ -806,31 +806,31 @@  discard block
 block discarded – undo
806 806
     public function test_skips_buffering_when_ao_noptimize_filter_is_true()
807 807
     {
808 808
         // True => disable autoptimize.
809
-        add_filter( 'autoptimize_filter_noptimize', '__return_true' );
809
+        add_filter('autoptimize_filter_noptimize', '__return_true');
810 810
 
811 811
         // Buffering should not run due to the above filter.
812 812
         $expected = false;
813
-        $actual   = $this->ao->should_buffer( $doing_tests = true );
813
+        $actual   = $this->ao->should_buffer($doing_tests = true);
814 814
 
815
-        $this->assertEquals( $expected, $actual );
815
+        $this->assertEquals($expected, $actual);
816 816
     }
817 817
 
818 818
     public function test_does_buffering_when_ao_noptimize_filter_is_false()
819 819
     {
820 820
         // False => disable noptimize, aka, run normally (weird, yes...).
821
-        add_filter( 'autoptimize_filter_noptimize', '__return_false' );
821
+        add_filter('autoptimize_filter_noptimize', '__return_false');
822 822
 
823 823
         // Buffering should run because of above.
824 824
         $expected = true;
825
-        $actual   = $this->ao->should_buffer( $doing_tests = true );
825
+        $actual   = $this->ao->should_buffer($doing_tests = true);
826 826
 
827
-        $this->assertEquals( $expected, $actual );
827
+        $this->assertEquals($expected, $actual);
828 828
     }
829 829
 
830 830
     public function test_ignores_ao_noptimize_qs_when_instructed()
831 831
     {
832 832
         // Should skip checking for the qs completely due to filter.
833
-        add_filter( 'autoptimize_filter_honor_qs_noptimize', '__return_false' );
833
+        add_filter('autoptimize_filter_honor_qs_noptimize', '__return_false');
834 834
 
835 835
         /**
836 836
          * The above should then result in the "current" value being `false`
@@ -841,23 +841,23 @@  discard block
 block discarded – undo
841 841
          */
842 842
 
843 843
         $that = $this; // Makes it work on php 5.3!
844
-        add_filter( 'autoptimize_filter_noptimize', function ( $current_value ) use ( $that ) {
844
+        add_filter('autoptimize_filter_noptimize', function($current_value) use ($that) {
845 845
             $expected = false;
846
-            if ( defined( 'DONOTMINIFY' ) && DONOTMINIFY ) {
846
+            if (defined('DONOTMINIFY') && DONOTMINIFY) {
847 847
                 $expected = true;
848 848
             }
849 849
 
850
-            $that->assertEquals( $expected, $current_value );
850
+            $that->assertEquals($expected, $current_value);
851 851
         });
852 852
 
853
-        $this->ao->should_buffer( $doing_tests = true );
853
+        $this->ao->should_buffer($doing_tests = true);
854 854
     }
855 855
 
856 856
     public function test_wpengine_cache_flush()
857 857
     {
858 858
         // Creating a mock so that we can get past class_exists() and method_exists() checks present
859 859
         // in `autoptimizeCache::flushPageCache()`...
860
-        $stub = $this->getMockBuilder( 'WpeCommon' )->disableAutoload()
860
+        $stub = $this->getMockBuilder('WpeCommon')->disableAutoload()
861 861
                 ->disableOriginalConstructor()->setMethods(
862 862
                     array(
863 863
                         'purge_varnish_cache',
@@ -866,9 +866,9 @@  discard block
 block discarded – undo
866 866
                 ->getMock();
867 867
 
868 868
         $that = $this;
869
-        add_filter( 'autoptimize_flush_wpengine_methods', function( $methods ) use ( $that ) {
870
-            $expected_methods = array( 'purge_varnish_cache' );
871
-            $that->assertEquals( $methods, $expected_methods );
869
+        add_filter('autoptimize_flush_wpengine_methods', function($methods) use ($that) {
870
+            $expected_methods = array('purge_varnish_cache');
871
+            $that->assertEquals($methods, $expected_methods);
872 872
 
873 873
             return $methods;
874 874
         });
@@ -882,7 +882,7 @@  discard block
 block discarded – undo
882 882
     public function test_wpengine_cache_flush_agressive()
883 883
     {
884 884
         // Creating a mock so that we can get past class_exists() and method_exists() checks `autoptimize_flush_pagecache()`...
885
-        $stub = $this->getMockBuilder( 'WpeCommon' )->disableAutoload()
885
+        $stub = $this->getMockBuilder('WpeCommon')->disableAutoload()
886 886
                 ->disableOriginalConstructor()->setMethods(
887 887
                     array(
888 888
                         'purge_varnish_cache',
@@ -892,19 +892,19 @@  discard block
 block discarded – undo
892 892
                 )
893 893
                 ->getMock();
894 894
 
895
-        add_filter( 'autoptimize_flush_wpengine_aggressive', function() {
895
+        add_filter('autoptimize_flush_wpengine_aggressive', function() {
896 896
             return true;
897 897
         });
898 898
 
899 899
         $that = $this;
900
-        add_filter( 'autoptimize_flush_wpengine_methods', function( $methods ) use ( $that ) {
900
+        add_filter('autoptimize_flush_wpengine_methods', function($methods) use ($that) {
901 901
             $expected_methods = array(
902 902
                 'purge_varnish_cache',
903 903
                 'purge_memcached',
904 904
                 'clear_maxcdn_cache',
905 905
             );
906 906
 
907
-            $that->assertEquals( $methods, $expected_methods );
907
+            $that->assertEquals($methods, $expected_methods);
908 908
 
909 909
             return $methods;
910 910
         });
@@ -915,15 +915,15 @@  discard block
 block discarded – undo
915 915
     /**
916 916
      * @dataProvider provider_test_url_replace_cdn
917 917
      */
918
-    public function test_url_replace_cdn( $cdn_url, $input, $expected )
918
+    public function test_url_replace_cdn($cdn_url, $input, $expected)
919 919
     {
920
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )
920
+        $mock = $this->getMockBuilder('autoptimizeBase')
921 921
                 ->disableOriginalConstructor()
922 922
                 ->getMockForAbstractClass();
923 923
 
924 924
         $mock->cdn_url = $cdn_url;
925
-        $actual        = $mock->url_replace_cdn( $input );
926
-        $this->assertEquals( $expected, $actual );
925
+        $actual        = $mock->url_replace_cdn($input);
926
+        $this->assertEquals($expected, $actual);
927 927
     }
928 928
 
929 929
     public function provider_test_url_replace_cdn()
@@ -940,13 +940,13 @@  discard block
 block discarded – undo
940 940
                 // CDN base url, url, expected result...
941 941
                 'http://cdn-test.example.org',
942 942
                 '/a.jpg',
943
-                'http://cdn-test.example.org/' . $subfolder . 'a.jpg',
943
+                'http://cdn-test.example.org/'.$subfolder.'a.jpg',
944 944
             ),
945 945
             // Full link with a matching AUTOPTIMIZE_WP_SITE_URL gets properly replaced...
946 946
             array(
947 947
                 'http://cdn-test.example.org',
948
-                $siteurl . '/wp-content/themes/something/example.svg',
949
-                'http://cdn-test.example.org/' . $subfolder . 'wp-content/themes/something/example.svg',
948
+                $siteurl.'/wp-content/themes/something/example.svg',
949
+                'http://cdn-test.example.org/'.$subfolder.'wp-content/themes/something/example.svg',
950 950
             ),
951 951
             // Protocol-relative url with a "local" hostname that doesn't match example.org (AUTOPTIMIZE_WP_SITE_URL)...
952 952
             array(
@@ -957,59 +957,59 @@  discard block
 block discarded – undo
957 957
             // www.example.org does not match example.org (AUTOPTIMIZE_WP_SITE_URL) so it's left alone...
958 958
             array(
959 959
                 'http://cdn-test.example.org',
960
-                $wwwsiteurl . '/wp-content/themes/something/example.svg',
961
-                $wwwsiteurl . '/wp-content/themes/something/example.svg',
960
+                $wwwsiteurl.'/wp-content/themes/something/example.svg',
961
+                $wwwsiteurl.'/wp-content/themes/something/example.svg',
962 962
             ),
963 963
             // SSL cdn url + host-relative link...
964 964
             array(
965 965
                 'https://cdn.example.org',
966 966
                 '/a.jpg',
967
-                'https://cdn.example.org/' . $subfolder . 'a.jpg',
967
+                'https://cdn.example.org/'.$subfolder.'a.jpg',
968 968
             ),
969 969
             // SSL cdn url + http site url that matches AUTOPTIMIZE_WP_SITE_URL is properly replaced...
970 970
             array(
971 971
                 'https://cdn.example.org',
972
-                $siteurl . '/wp-content/themes/something/example.svg',
973
-                'https://cdn.example.org/' . $subfolder . 'wp-content/themes/something/example.svg',
972
+                $siteurl.'/wp-content/themes/something/example.svg',
973
+                'https://cdn.example.org/'.$subfolder.'wp-content/themes/something/example.svg',
974 974
             ),
975 975
             // Protocol-relative cdn url given with protocol relative link that matches AUTOPTIMIZE_WP_SITE_URL host...
976 976
             array(
977 977
                 '//cdn.example.org',
978
-                $protorelativesiteurl . '/something.jpg',
979
-                '//cdn.example.org/' . $subfolder . 'something.jpg',
978
+                $protorelativesiteurl.'/something.jpg',
979
+                '//cdn.example.org/'.$subfolder.'something.jpg',
980 980
             ),
981 981
             // Protocol-relative cdn url given a http link that matches AUTOPTIMIZE_WP_SITE_URL host...
982 982
             array(
983 983
                 '//cdn.example.org',
984
-                $siteurl . '/something.png',
985
-                '//cdn.example.org/' . $subfolder . 'something.png',
984
+                $siteurl.'/something.png',
985
+                '//cdn.example.org/'.$subfolder.'something.png',
986 986
             ),
987 987
             // Protocol-relative cdn url with a host-relative link...
988 988
             array(
989 989
                 '//cdn.example.org',
990 990
                 '/a.jpg',
991
-                '//cdn.example.org/' . $subfolder . 'a.jpg',
991
+                '//cdn.example.org/'.$subfolder.'a.jpg',
992 992
             ),
993 993
             // Testing cdn urls with an explicit port number...
994 994
             array(
995 995
                 'http://cdn.com:8080',
996 996
                 '/a.jpg',
997
-                'http://cdn.com:8080/' . $subfolder . 'a.jpg',
997
+                'http://cdn.com:8080/'.$subfolder.'a.jpg',
998 998
             ),
999 999
             array(
1000 1000
                 '//cdn.com:4433',
1001 1001
                 '/a.jpg',
1002
-                '//cdn.com:4433/' . $subfolder . 'a.jpg',
1002
+                '//cdn.com:4433/'.$subfolder.'a.jpg',
1003 1003
             ),
1004 1004
             array(
1005 1005
                 '//cdn.com:4433',
1006
-                $siteurl . '/something.jpg',
1007
-                '//cdn.com:4433/' . $subfolder . 'something.jpg',
1006
+                $siteurl.'/something.jpg',
1007
+                '//cdn.com:4433/'.$subfolder.'something.jpg',
1008 1008
             ),
1009 1009
             array(
1010 1010
                 '//cdn.com:1234',
1011
-                $protorelativesiteurl . '/something.jpg',
1012
-                '//cdn.com:1234/' . $subfolder . 'something.jpg',
1011
+                $protorelativesiteurl.'/something.jpg',
1012
+                '//cdn.com:1234/'.$subfolder.'something.jpg',
1013 1013
             ),
1014 1014
             // Relative links should not be touched by url_replace_cdn()...
1015 1015
             array(
@@ -1034,8 +1034,8 @@  discard block
 block discarded – undo
1034 1034
             // even though they are really strange...
1035 1035
             array(
1036 1036
                 $siteurl, // example.org or http://localhost or http://localhost/wordpress
1037
-                $siteurl . '/something.jpg',
1038
-                $siteurl . '/something.jpg',
1037
+                $siteurl.'/something.jpg',
1038
+                $siteurl.'/something.jpg',
1039 1039
             ),
1040 1040
             // These shouldn't really be changed, or even if replacements do
1041 1041
             // happen, they shouldn't be destructive...
@@ -1074,31 +1074,31 @@  discard block
 block discarded – undo
1074 1074
         $test_link = '/a.jpg';
1075 1075
         $cdn_url   = '//cdn.example.org';
1076 1076
 
1077
-        $with_ssl = function( $cdn ) {
1077
+        $with_ssl = function($cdn) {
1078 1078
             return '';
1079 1079
         };
1080 1080
         $expected_with_ssl = '/a.jpg';
1081 1081
 
1082
-        $without_ssl = function( $cdn ) {
1082
+        $without_ssl = function($cdn) {
1083 1083
             return $cdn;
1084 1084
         };
1085
-        $expected_without_ssl = '//cdn.example.org/' . $sub . 'a.jpg';
1085
+        $expected_without_ssl = '//cdn.example.org/'.$sub.'a.jpg';
1086 1086
 
1087 1087
         // With a filter that returns something considered "empty", cdn replacement shouldn't occur...
1088
-        add_filter( 'autoptimize_filter_base_cdnurl', $with_ssl );
1089
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )
1088
+        add_filter('autoptimize_filter_base_cdnurl', $with_ssl);
1089
+        $mock = $this->getMockBuilder('autoptimizeBase')
1090 1090
                 ->disableOriginalConstructor()
1091 1091
                 ->getMockForAbstractClass();
1092 1092
 
1093 1093
         $mock->cdn_url   = $cdn_url;
1094
-        $actual_with_ssl = $mock->url_replace_cdn( $test_link );
1095
-        $this->assertEquals( $expected_with_ssl, $actual_with_ssl );
1096
-        remove_filter( 'autoptimize_filter_base_cdnurl', $with_ssl );
1094
+        $actual_with_ssl = $mock->url_replace_cdn($test_link);
1095
+        $this->assertEquals($expected_with_ssl, $actual_with_ssl);
1096
+        remove_filter('autoptimize_filter_base_cdnurl', $with_ssl);
1097 1097
 
1098 1098
         // With a filter that returns an actual cdn url, cdn replacement should occur.
1099
-        add_filter( 'autoptimize_filter_base_cdnurl', $without_ssl );
1100
-        $actual_without_ssl = $mock->url_replace_cdn( $test_link );
1101
-        $this->assertEquals( $expected_without_ssl, $actual_without_ssl );
1099
+        add_filter('autoptimize_filter_base_cdnurl', $without_ssl);
1100
+        $actual_without_ssl = $mock->url_replace_cdn($test_link);
1101
+        $this->assertEquals($expected_without_ssl, $actual_without_ssl);
1102 1102
     }
1103 1103
 
1104 1104
     public function provider_cssmin_issues()
@@ -1137,11 +1137,11 @@  discard block
 block discarded – undo
1137 1137
     /**
1138 1138
      * @dataProvider provider_cssmin_issues
1139 1139
      */
1140
-    public function test_cssmin_issues( $input, $expected )
1140
+    public function test_cssmin_issues($input, $expected)
1141 1141
     {
1142
-        $minifier = new autoptimizeCSSmin( false ); // No need to raise limits for now.
1143
-        $actual   = $minifier->run( $input );
1144
-        $this->assertEquals( $expected, $actual );
1142
+        $minifier = new autoptimizeCSSmin(false); // No need to raise limits for now.
1143
+        $actual   = $minifier->run($input);
1144
+        $this->assertEquals($expected, $actual);
1145 1145
     }
1146 1146
 
1147 1147
     public function provider_getpath()
@@ -1166,7 +1166,7 @@  discard block
 block discarded – undo
1166 1166
             // This file comes with core, so should exist...
1167 1167
             array(
1168 1168
                 '/wp-includes/js/jquery/jquery.js',
1169
-                WP_ROOT_DIR . '/wp-includes/js/jquery/jquery.js',
1169
+                WP_ROOT_DIR.'/wp-includes/js/jquery/jquery.js',
1170 1170
             ),
1171 1171
             // Empty $url should return false.
1172 1172
             array(
@@ -1191,12 +1191,12 @@  discard block
 block discarded – undo
1191 1191
     /**
1192 1192
      * @dataProvider provider_getpath
1193 1193
      */
1194
-    public function test_getpath( $input, $expected )
1194
+    public function test_getpath($input, $expected)
1195 1195
     {
1196
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )->disableOriginalConstructor()->getMockForAbstractClass();
1196
+        $mock = $this->getMockBuilder('autoptimizeBase')->disableOriginalConstructor()->getMockForAbstractClass();
1197 1197
 
1198
-        $actual = $mock->getpath( $input );
1199
-        $this->assertEquals( $expected, $actual );
1198
+        $actual = $mock->getpath($input);
1199
+        $this->assertEquals($expected, $actual);
1200 1200
     }
1201 1201
 
1202 1202
     /**
@@ -1210,7 +1210,7 @@  discard block
 block discarded – undo
1210 1210
 
1211 1211
         $protorelativesiteurl = $urls['prsiteurl'];
1212 1212
 
1213
-        $css_orig     = <<<CSS
1213
+        $css_orig = <<<CSS
1214 1214
 header{background-image:url("data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' width='90px' height='110px' viewBox='0 0 90 110'%3E%3Cstyle%3E.a%7Bstop-color:%23FFF;%7D.b%7Bstop-color:%23B2D235;%7D.c%7Bstop-color:%23BEE7FA;%7D.d%7Bfill:%23590C15;%7D%3C/style%3E%3ClinearGradient id='c' y2='135.4' gradientUnits='userSpaceOnUse' x2='209.1' gradientTransform='rotate(-1.467 -4082.888 7786.794)' y1='205.8' x1='262'%3E%3Cstop class='b' offset='0'/%3E%3Cstop class='b' offset='.48'/%3E%3Cstop stop-color='%23829D25' offset='1'/%3E%3C/linearGradient%3E%3Cpath stroke-width='.3' d='M77.3 45.4c-3-3.5-7.1-6.5-11.6-7.8-5.1-1.5-10-.1-14.9 1.5C52 35.4 54.3 29 60 24l-4.8-5.5c-3.4 3-5.8 6.3-7.5 9.4-1.7-4.3-4.1-8.4-7.5-12C33.4 8.6 24.3 4.7 15.1 4.2c-.2 9.3 3.1 18.6 9.9 25.9 5.2 5.6 11.8 9.2 18.7 10.8-2.5.2-4.9-.1-7.7-.9-5.2-1.4-10.5-2.8-15.8-1C10.6 42.3 4.5 51.9 4 61.7c-.5 11.6 3.8 23.8 9.9 33.5 3.9 6.3 9.6 13.7 17.7 13.4 3.8-.1 7-2.1 10.7-2.7 5.2-.8 9.1 1.2 14.1 1.8 16.4 2 24.4-23.6 26.4-35.9 1.2-9.1.8-19.1-5.5-26.4z' stroke='%233E6D1F' fill='url(%23c)'/%3E%3C/svg%3E")}
1215 1215
 section.clipped.clippedTop {clip-path:url("#clipPolygonTop")}
1216 1216
 section.clipped.clippedBottom {clip-path:url("#clipPolygonBottom")}
@@ -1223,13 +1223,13 @@  discard block
 block discarded – undo
1223 1223
 .myimg {background-image: url($protorelativesiteurl/wp-content/themes/my-theme/images/under-left-leaf.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/over-blue-bird.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/under-top.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/bg-top-grunge.png);}
1224 1224
 CSS;
1225 1225
 
1226
-        $fixurls_result = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $css_orig );
1227
-        $this->assertEquals( $css_expected, $fixurls_result );
1226
+        $fixurls_result = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $css_orig);
1227
+        $this->assertEquals($css_expected, $fixurls_result);
1228 1228
     }
1229 1229
 
1230 1230
     public function test_background_datauri_sprites_with_fixurls()
1231 1231
     {
1232
-        $css_orig     = <<<CSS
1232
+        $css_orig = <<<CSS
1233 1233
 .shadow { background:url(img/1x1.png) top center; }
1234 1234
 .shadow1 { background-image:url(img/1x1.png) 0 -767px repeat-x; }
1235 1235
 .shadow2 {background:url(img/1x1.png) top center}
@@ -1281,27 +1281,27 @@  discard block
 block discarded – undo
1281 1281
 CSS;
1282 1282
 
1283 1283
         // For test purposes, ALL images in the css are being inline with a 1x1 trans png string/datauri.
1284
-        add_filter( 'autoptimize_filter_css_is_datauri_candidate', function( $is_candidate, $path ) {
1284
+        add_filter('autoptimize_filter_css_is_datauri_candidate', function($is_candidate, $path) {
1285 1285
             return true;
1286
-        }, 10, 2 );
1286
+        }, 10, 2);
1287 1287
 
1288 1288
         // For test purposes, ALL images in the css are being inline with a 1x1 trans png string/datauri.
1289
-        add_filter( 'autoptimize_filter_css_datauri_image', function( $base64array, $path ) {
1289
+        add_filter('autoptimize_filter_css_datauri_image', function($base64array, $path) {
1290 1290
             $head = 'data:image/png;base64,';
1291 1291
             $data = 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAQAAAC1HAwCAAAAC0lEQVR42mNkYAAAAAYAAjCB0C8AAAAASUVORK5CYII=';
1292 1292
 
1293
-            $result['full']       = $head . $data;
1293
+            $result['full']       = $head.$data;
1294 1294
             $result['base64data'] = $data;
1295 1295
             return $result;
1296
-        }, 10, 2 );
1296
+        }, 10, 2);
1297 1297
 
1298
-        $instance = new autoptimizeStyles( $css_orig );
1299
-        $instance->setOption( 'datauris', true );
1298
+        $instance = new autoptimizeStyles($css_orig);
1299
+        $instance->setOption('datauris', true);
1300 1300
 
1301
-        $fixurls_result = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $css_orig );
1302
-        $css_actual     = $instance->rewrite_assets( $fixurls_result );
1301
+        $fixurls_result = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $css_orig);
1302
+        $css_actual     = $instance->rewrite_assets($fixurls_result);
1303 1303
 
1304
-        $this->assertEquals( $css_expected, $css_actual );
1304
+        $this->assertEquals($css_expected, $css_actual);
1305 1305
     }
1306 1306
 
1307 1307
     /**
@@ -1321,7 +1321,7 @@  discard block
 block discarded – undo
1321 1321
      */
1322 1322
     public function test_background_datauri_sprites_without_fixurls()
1323 1323
     {
1324
-        $css_orig     = <<<CSS
1324
+        $css_orig = <<<CSS
1325 1325
 .shadow { background:url(img/1x1.png) top center; }
1326 1326
 .shadow1 { background-image:url(img/1x1.png) 0 -767px repeat-x; }
1327 1327
 .shadow2 {background:url(img/1x1.png) top center}
@@ -1365,24 +1365,24 @@  discard block
 block discarded – undo
1365 1365
 CSS;
1366 1366
 
1367 1367
         // For test purposes, ALL images in the css are being inlined with a 1x1 trans png string/datauri.
1368
-        add_filter( 'autoptimize_filter_css_is_datauri_candidate', function( $is_candidate, $path ) {
1368
+        add_filter('autoptimize_filter_css_is_datauri_candidate', function($is_candidate, $path) {
1369 1369
             return true;
1370
-        }, 10, 2 );
1370
+        }, 10, 2);
1371 1371
 
1372 1372
         // For test purposes, ALL images in the css are being inlined with a 1x1 trans png string/datauri.
1373
-        add_filter( 'autoptimize_filter_css_datauri_image', function( $base64array, $path ) {
1373
+        add_filter('autoptimize_filter_css_datauri_image', function($base64array, $path) {
1374 1374
             $head = 'data:image/png;base64,';
1375 1375
             $data = 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAQAAAC1HAwCAAAAC0lEQVR42mNkYAAAAAYAAjCB0C8AAAAASUVORK5CYII=';
1376 1376
 
1377
-            $result['full']       = $head . $data;
1377
+            $result['full']       = $head.$data;
1378 1378
             $result['base64data'] = $data;
1379 1379
             return $result;
1380
-        }, 10, 2 );
1380
+        }, 10, 2);
1381 1381
 
1382
-        $instance = new autoptimizeStyles( $css_orig );
1383
-        $instance->setOption( 'datauris', true );
1384
-        $css_actual = $instance->rewrite_assets( $css_orig );
1385
-        $this->assertEquals( $css_expected, $css_actual );
1382
+        $instance = new autoptimizeStyles($css_orig);
1383
+        $instance->setOption('datauris', true);
1384
+        $css_actual = $instance->rewrite_assets($css_orig);
1385
+        $this->assertEquals($css_expected, $css_actual);
1386 1386
     }
1387 1387
 
1388 1388
     /**
@@ -1502,12 +1502,12 @@  discard block
 block discarded – undo
1502 1502
 CSS;
1503 1503
 
1504 1504
         // Test with fonts pointed to the CDN + cdn option is set.
1505
-        add_filter( 'autoptimize_filter_css_fonts_cdn', '__return_true' );
1506
-        $instance = new autoptimizeStyles( $css_in );
1507
-        $instance->setOption( 'cdn_url', $cdnurl );
1508
-        $css_actual_fonts_cdn = $instance->rewrite_assets( $css_in );
1505
+        add_filter('autoptimize_filter_css_fonts_cdn', '__return_true');
1506
+        $instance = new autoptimizeStyles($css_in);
1507
+        $instance->setOption('cdn_url', $cdnurl);
1508
+        $css_actual_fonts_cdn = $instance->rewrite_assets($css_in);
1509 1509
 
1510
-        $this->assertEquals( $css_expected_fonts_cdn, $css_actual_fonts_cdn );
1510
+        $this->assertEquals($css_expected_fonts_cdn, $css_actual_fonts_cdn);
1511 1511
     }
1512 1512
 
1513 1513
     /**
@@ -1623,10 +1623,10 @@  discard block
 block discarded – undo
1623 1623
 }
1624 1624
 CSS;
1625 1625
         // Test without moving fonts to CDN, but cdn option is set.
1626
-        $instance = new autoptimizeStyles( $css_in );
1627
-        $instance->setOption( 'cdn_url', $urls['cdnurl'] );
1628
-        $css_actual = $instance->rewrite_assets( $css_in );
1629
-        $this->assertEquals( $css_expected, $css_actual );
1626
+        $instance = new autoptimizeStyles($css_in);
1627
+        $instance->setOption('cdn_url', $urls['cdnurl']);
1628
+        $css_actual = $instance->rewrite_assets($css_in);
1629
+        $this->assertEquals($css_expected, $css_actual);
1630 1630
     }
1631 1631
 
1632 1632
     public function test_assets_regex_replaces_multi_bg_images()
@@ -1647,16 +1647,16 @@  discard block
 block discarded – undo
1647 1647
 }
1648 1648
 CSS;
1649 1649
 
1650
-        $instance = new autoptimizeStyles( $in );
1651
-        $instance->setOption( 'cdn_url', $cdnurl );
1652
-        $actual = $instance->rewrite_assets( $in );
1650
+        $instance = new autoptimizeStyles($in);
1651
+        $instance->setOption('cdn_url', $cdnurl);
1652
+        $actual = $instance->rewrite_assets($in);
1653 1653
 
1654
-        $this->assertEquals( $expected, $actual );
1654
+        $this->assertEquals($expected, $actual);
1655 1655
     }
1656 1656
 
1657 1657
     public function test_at_supports_spacing_issue_110()
1658 1658
     {
1659
-        $in       = <<<CSS
1659
+        $in = <<<CSS
1660 1660
 @supports (-webkit-filter: blur(3px)) or (filter: blur(3px)) {
1661 1661
     .blur {
1662 1662
         filter:blur(3px);
@@ -1670,10 +1670,10 @@  discard block
 block discarded – undo
1670 1670
 @supports (-webkit-filter:blur(3px)) or (filter:blur(3px)){.blur{filter:blur(3px)}}@supports((position:-webkit-sticky) or (position:sticky)){.sticky{position:sticky}}
1671 1671
 CSS;
1672 1672
 
1673
-        $instance = new autoptimizeStyles( $in );
1674
-        $actual   = $instance->run_minifier_on( $in );
1673
+        $instance = new autoptimizeStyles($in);
1674
+        $actual   = $instance->run_minifier_on($in);
1675 1675
 
1676
-        $this->assertEquals( $expected, $actual );
1676
+        $this->assertEquals($expected, $actual);
1677 1677
     }
1678 1678
 
1679 1679
     public function test_css_import_semicolon_url_issue_122()
@@ -1689,18 +1689,18 @@  discard block
 block discarded – undo
1689 1689
 </style>
1690 1690
 HTML;
1691 1691
 
1692
-        $expected = '<style type="text/css" media="all">@import url(' . $cdnurl . '/' . $sub . 'foo.css?a&#038;b);@import url(' . $cdnurl . '/' . $sub . 'bar.css);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1692
+        $expected = '<style type="text/css" media="all">@import url('.$cdnurl.'/'.$sub.'foo.css?a&#038;b);@import url('.$cdnurl.'/'.$sub.'bar.css);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1693 1693
 
1694 1694
         $options = [
1695 1695
             'autoptimizeStyles' => $this->getAoStylesDefaultOptions(),
1696 1696
         ];
1697 1697
 
1698
-        $instance = new autoptimizeStyles( $in );
1699
-        $instance->read( $options['autoptimizeStyles'] );
1698
+        $instance = new autoptimizeStyles($in);
1699
+        $instance->read($options['autoptimizeStyles']);
1700 1700
         $instance->minify();
1701 1701
         $instance->cache();
1702 1702
         $actual = $instance->getcontent();
1703
-        $this->assertEquals( $expected, $actual );
1703
+        $this->assertEquals($expected, $actual);
1704 1704
     }
1705 1705
 
1706 1706
     public function test_fixurls_with_at_imports_and_media_queries()
@@ -1710,10 +1710,10 @@  discard block
 block discarded – undo
1710 1710
         $subfolder = $urls['subfolder'];
1711 1711
 
1712 1712
         $in  = '@import "foo.css"; @import "bar.css" (orientation:landscape);';
1713
-        $exp = '@import url(' . $prsiteurl . '/wp-content/themes/my-theme/foo.css); @import url(' . $prsiteurl . '/wp-content/themes/my-theme/bar.css) (orientation:landscape);';
1713
+        $exp = '@import url('.$prsiteurl.'/wp-content/themes/my-theme/foo.css); @import url('.$prsiteurl.'/wp-content/themes/my-theme/bar.css) (orientation:landscape);';
1714 1714
 
1715
-        $actual = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $in );
1716
-        $this->assertEquals( $exp, $actual );
1715
+        $actual = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $in);
1716
+        $this->assertEquals($exp, $actual);
1717 1717
     }
1718 1718
 
1719 1719
     public function test_aostyles_at_imports_with_media_queries()
@@ -1728,43 +1728,43 @@  discard block
 block discarded – undo
1728 1728
 </style>
1729 1729
 HTML;
1730 1730
 
1731
-        $expected = '<style type="text/css" media="all">@import url(' . $cdnurl . '/' . $sub . 'foo.css);@import url(' . $cdnurl . '/' . $sub . 'bar.css) (orientation:landscape);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1731
+        $expected = '<style type="text/css" media="all">@import url('.$cdnurl.'/'.$sub.'foo.css);@import url('.$cdnurl.'/'.$sub.'bar.css) (orientation:landscape);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1732 1732
 
1733 1733
         $options = [
1734 1734
             'autoptimizeStyles' => $this->getAoStylesDefaultOptions(),
1735 1735
         ];
1736 1736
 
1737
-        $instance = new autoptimizeStyles( $in );
1738
-        $instance->read( $options['autoptimizeStyles'] );
1737
+        $instance = new autoptimizeStyles($in);
1738
+        $instance->read($options['autoptimizeStyles']);
1739 1739
         $instance->minify();
1740 1740
         $instance->cache();
1741 1741
 
1742 1742
         $actual = $instance->getcontent();
1743
-        $this->assertEquals( $expected, $actual );
1743
+        $this->assertEquals($expected, $actual);
1744 1744
     }
1745 1745
 
1746 1746
     public function test_cache_size_checker_hooked_by_default()
1747 1747
     {
1748
-        $this->assertNotEmpty( autoptimizeCacheChecker::SCHEDULE_HOOK );
1748
+        $this->assertNotEmpty(autoptimizeCacheChecker::SCHEDULE_HOOK);
1749 1749
 
1750 1750
         // No schedule, because it's only added when is_admin() is true.
1751
-        $this->assertEquals( false, wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1751
+        $this->assertEquals(false, wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1752 1752
 
1753 1753
         // Proving that setup() sets the schedule as needed.
1754 1754
         $checker = new autoptimizeCacheChecker();
1755 1755
         $checker->setup();
1756
-        $this->assertEquals( 'twicedaily', wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1756
+        $this->assertEquals('twicedaily', wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1757 1757
     }
1758 1758
 
1759 1759
     public function test_cache_size_checker_disabled_with_filter()
1760 1760
     {
1761
-        add_filter( 'autoptimize_filter_cachecheck_do', '__return_false' );
1761
+        add_filter('autoptimize_filter_cachecheck_do', '__return_false');
1762 1762
 
1763 1763
         $checker = new autoptimizeCacheChecker();
1764 1764
         $checker->setup();
1765
-        $this->assertEquals( false, wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1765
+        $this->assertEquals(false, wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1766 1766
 
1767
-        remove_all_filters( 'autoptimize_filter_cachecheck_do' );
1767
+        remove_all_filters('autoptimize_filter_cachecheck_do');
1768 1768
     }
1769 1769
 
1770 1770
     public function test_is_start_buffering_hooked_properly()
@@ -1777,12 +1777,12 @@  discard block
 block discarded – undo
1777 1777
          * https://www.theaveragedev.com/mocking-constants-in-tests/.
1778 1778
          */
1779 1779
 
1780
-        if ( defined( 'AUTOPTIMIZE_INIT_EARLIER' ) ) {
1780
+        if (defined('AUTOPTIMIZE_INIT_EARLIER')) {
1781 1781
             $this->assertEquals(
1782 1782
                 autoptimizeMain::INIT_EARLIER_PRIORITY,
1783
-                has_action( 'init', array( $instance, 'start_buffering' ) )
1783
+                has_action('init', array($instance, 'start_buffering'))
1784 1784
             );
1785
-            $this->assertTrue( ! defined( 'AUTOPTIMIZE_HOOK_INTO' ) );
1785
+            $this->assertTrue(!defined('AUTOPTIMIZE_HOOK_INTO'));
1786 1786
         }
1787 1787
 
1788 1788
         /*
@@ -1799,18 +1799,18 @@  discard block
 block discarded – undo
1799 1799
 
1800 1800
     public function test_inline_and_defer_markup()
1801 1801
     {
1802
-        add_filter( 'autoptimize_filter_css_defer', '__return_true' );
1803
-        add_filter( 'autoptimize_filter_css_defer_inline', '__return_true' );
1802
+        add_filter('autoptimize_filter_css_defer', '__return_true');
1803
+        add_filter('autoptimize_filter_css_defer_inline', '__return_true');
1804 1804
 
1805
-        $actual = $this->ao->end_buffering( $this->get_test_markup() );
1806
-        if ( is_multisite() ) {
1807
-            $this->assertEquals( $this->get_test_markup_output_inline_defer_ms(), $actual );
1805
+        $actual = $this->ao->end_buffering($this->get_test_markup());
1806
+        if (is_multisite()) {
1807
+            $this->assertEquals($this->get_test_markup_output_inline_defer_ms(), $actual);
1808 1808
         } else {
1809
-            $this->assertEquals( $this->get_test_markup_output_inline_defer(), $actual );
1809
+            $this->assertEquals($this->get_test_markup_output_inline_defer(), $actual);
1810 1810
         }
1811 1811
 
1812
-        remove_all_filters( 'autoptimize_filter_css_defer' );
1813
-        remove_all_filters( 'autoptimize_filter_css_defer_inline' );
1812
+        remove_all_filters('autoptimize_filter_css_defer');
1813
+        remove_all_filters('autoptimize_filter_css_defer_inline');
1814 1814
     }
1815 1815
 
1816 1816
     public function test_js_aggregation_decision_and_dontaggregate_filter()
@@ -1818,47 +1818,47 @@  discard block
 block discarded – undo
1818 1818
         $opts = $this->getAoScriptsDefaultOptions();
1819 1819
 
1820 1820
         // Aggregating: true by default.
1821
-        $scripts = new autoptimizeScripts( '' );
1822
-        $scripts->read( $opts );
1823
-        $this->assertTrue( $scripts->aggregating() );
1821
+        $scripts = new autoptimizeScripts('');
1822
+        $scripts->read($opts);
1823
+        $this->assertTrue($scripts->aggregating());
1824 1824
 
1825 1825
         // Aggregating: option=true (dontaggregate=false by default).
1826 1826
         $opts['aggregate'] = true;
1827
-        $scripts           = new autoptimizeScripts( '' );
1828
-        $scripts->read( $opts );
1829
-        $this->assertTrue( $scripts->aggregating() );
1827
+        $scripts           = new autoptimizeScripts('');
1828
+        $scripts->read($opts);
1829
+        $this->assertTrue($scripts->aggregating());
1830 1830
 
1831 1831
         // Aggregating: option=true, dontaggregate=false explicit.
1832 1832
         $opts['aggregate'] = true;
1833
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_false' );
1834
-        $scripts = new autoptimizeScripts( '' );
1835
-        $scripts->read( $opts );
1836
-        $this->assertTrue( $scripts->aggregating() );
1837
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1833
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_false');
1834
+        $scripts = new autoptimizeScripts('');
1835
+        $scripts->read($opts);
1836
+        $this->assertTrue($scripts->aggregating());
1837
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1838 1838
 
1839 1839
         // Not aggregating: option=true, dontaggregate=true.
1840 1840
         $opts['aggregate'] = true;
1841
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_true' );
1842
-        $scripts = new autoptimizeScripts( '' );
1843
-        $scripts->read( $opts );
1844
-        $this->assertFalse( $scripts->aggregating() );
1845
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1841
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_true');
1842
+        $scripts = new autoptimizeScripts('');
1843
+        $scripts->read($opts);
1844
+        $this->assertFalse($scripts->aggregating());
1845
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1846 1846
 
1847 1847
         // Not aggregating: option=false, dontaggregate=false.
1848 1848
         $opts['aggregate'] = false;
1849
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_false' );
1850
-        $scripts = new autoptimizeScripts( '' );
1851
-        $scripts->read( $opts );
1852
-        $this->assertFalse( $scripts->aggregating() );
1853
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1849
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_false');
1850
+        $scripts = new autoptimizeScripts('');
1851
+        $scripts->read($opts);
1852
+        $this->assertFalse($scripts->aggregating());
1853
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1854 1854
 
1855 1855
         // Not aggregating: option=false, dontaggregate=true.
1856 1856
         $opts['aggregate'] = false;
1857
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_true' );
1858
-        $scripts = new autoptimizeScripts( '' );
1859
-        $scripts->read( $opts );
1860
-        $this->assertFalse( $scripts->aggregating() );
1861
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1857
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_true');
1858
+        $scripts = new autoptimizeScripts('');
1859
+        $scripts->read($opts);
1860
+        $this->assertFalse($scripts->aggregating());
1861
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1862 1862
     }
1863 1863
 
1864 1864
     public function test_css_aggregation_decision_and_dontaggregate_filter()
@@ -1866,78 +1866,78 @@  discard block
 block discarded – undo
1866 1866
         $opts = $this->getAoStylesDefaultOptions();
1867 1867
 
1868 1868
         // Aggregating: true by default.
1869
-        $styles = new autoptimizeStyles( '' );
1870
-        $this->assertTrue( $styles->aggregating() );
1869
+        $styles = new autoptimizeStyles('');
1870
+        $this->assertTrue($styles->aggregating());
1871 1871
 
1872 1872
         // Aggregating: option=true (dontaggregate=false by default).
1873 1873
         $opts['aggregate'] = true;
1874
-        $styles            = new autoptimizeStyles( '' );
1875
-        $styles->read( $opts );
1876
-        $this->assertTrue( $styles->aggregating() );
1874
+        $styles            = new autoptimizeStyles('');
1875
+        $styles->read($opts);
1876
+        $this->assertTrue($styles->aggregating());
1877 1877
 
1878 1878
         // Aggregating: option=true, dontaggregate=false explicit.
1879 1879
         $opts['aggregate'] = true;
1880
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_false' );
1881
-        $styles = new autoptimizeStyles( '' );
1882
-        $styles->read( $opts );
1883
-        $this->assertTrue( $styles->aggregating() );
1884
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1880
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_false');
1881
+        $styles = new autoptimizeStyles('');
1882
+        $styles->read($opts);
1883
+        $this->assertTrue($styles->aggregating());
1884
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1885 1885
 
1886 1886
         // Not aggregating: option=true, dontaggregate=true.
1887 1887
         $opts['aggregate'] = true;
1888
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_true' );
1889
-        $styles = new autoptimizeStyles( '' );
1890
-        $styles->read( $opts );
1891
-        $this->assertFalse( $styles->aggregating() );
1892
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1888
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_true');
1889
+        $styles = new autoptimizeStyles('');
1890
+        $styles->read($opts);
1891
+        $this->assertFalse($styles->aggregating());
1892
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1893 1893
 
1894 1894
         // Not aggregating: option=false, dontaggregate=false.
1895 1895
         $opts['aggregate'] = false;
1896
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_false' );
1897
-        $styles = new autoptimizeStyles( '' );
1898
-        $styles->read( $opts );
1899
-        $this->assertFalse( $styles->aggregating() );
1900
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1896
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_false');
1897
+        $styles = new autoptimizeStyles('');
1898
+        $styles->read($opts);
1899
+        $this->assertFalse($styles->aggregating());
1900
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1901 1901
 
1902 1902
         // Not aggregating: option=false, dontaggregate=true.
1903 1903
         $opts['aggregate'] = false;
1904
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_true' );
1905
-        $styles = new autoptimizeStyles( '' );
1906
-        $styles->read( $opts );
1907
-        $this->assertFalse( $styles->aggregating() );
1908
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1904
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_true');
1905
+        $styles = new autoptimizeStyles('');
1906
+        $styles->read($opts);
1907
+        $this->assertFalse($styles->aggregating());
1908
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1909 1909
     }
1910 1910
 
1911 1911
     public function test_css_minify_single_with_cdning()
1912 1912
     {
1913
-        $pathname = dirname( __FILE__ ) . '/fixtures/minify-single.css';
1913
+        $pathname = dirname(__FILE__).'/fixtures/minify-single.css';
1914 1914
         $opts     = $this->getAoStylesDefaultOptions();
1915
-        $styles   = new autoptimizeStyles( '' );
1916
-        $styles->read( $opts );
1915
+        $styles   = new autoptimizeStyles('');
1916
+        $styles->read($opts);
1917 1917
 
1918
-        $url = $styles->minify_single( $pathname, $cache_miss = true );
1918
+        $url = $styles->minify_single($pathname, $cache_miss = true);
1919 1919
 
1920 1920
         // Minified url filename + its pointed to cdn.
1921
-        $this->assertContains( AUTOPTIMIZE_CACHE_CHILD_DIR, $url );
1922
-        $this->assertContains( '/autoptimize_single_', $url );
1923
-        $this->assertContains( $styles->cdn_url, $url );
1921
+        $this->assertContains(AUTOPTIMIZE_CACHE_CHILD_DIR, $url);
1922
+        $this->assertContains('/autoptimize_single_', $url);
1923
+        $this->assertContains($styles->cdn_url, $url);
1924 1924
 
1925 1925
         // Actual minified css contents are minified and cdn-ed.
1926
-        $path     = $styles->getpath( $url );
1927
-        $contents = file_get_contents( $path );
1928
-        $this->assertContains( $styles->cdn_url, $contents );
1929
-        $this->assertContains( '.bg{background:url(' . $styles->cdn_url, $contents );
1926
+        $path     = $styles->getpath($url);
1927
+        $contents = file_get_contents($path);
1928
+        $this->assertContains($styles->cdn_url, $contents);
1929
+        $this->assertContains('.bg{background:url('.$styles->cdn_url, $contents);
1930 1930
     }
1931 1931
 
1932 1932
     public function test_ao_partners_instantiation_without_explicit_include()
1933 1933
     {
1934 1934
         $partners = new autoptimizePartners();
1935
-        $this->assertTrue( $partners instanceof autoptimizePartners );
1935
+        $this->assertTrue($partners instanceof autoptimizePartners);
1936 1936
     }
1937 1937
 
1938 1938
     public function test_html_minify_keep_html_comments_inside_script_blocks()
1939 1939
     {
1940
-        $markup   = <<<MARKUP
1940
+        $markup = <<<MARKUP
1941 1941
 <script>
1942 1942
 <!-- End Support AJAX add to cart -->
1943 1943
 var a = "b";
@@ -1967,23 +1967,23 @@  discard block
 block discarded – undo
1967 1967
             ],
1968 1968
         ];
1969 1969
 
1970
-        $instance = new autoptimizeHTML( $markup );
1971
-        $instance->read( $options['autoptimizeHTML'] );
1970
+        $instance = new autoptimizeHTML($markup);
1971
+        $instance->read($options['autoptimizeHTML']);
1972 1972
         $instance->minify();
1973 1973
         $actual = $instance->getcontent();
1974
-        $this->assertEquals( $expected, $actual );
1974
+        $this->assertEquals($expected, $actual);
1975 1975
 
1976
-        $instance = new autoptimizeHTML( $markup2 );
1977
-        $instance->read( $options['autoptimizeHTML'] );
1976
+        $instance = new autoptimizeHTML($markup2);
1977
+        $instance->read($options['autoptimizeHTML']);
1978 1978
         $instance->minify();
1979 1979
         $actual2 = $instance->getcontent();
1980
-        $this->assertEquals( $expected2, $actual2 );
1980
+        $this->assertEquals($expected2, $actual2);
1981 1981
     }
1982 1982
 
1983 1983
     public function test_html_minify_remove_html_comments_inside_script_blocks()
1984 1984
     {
1985 1985
         // Default case, html comments removed (keepcomments = false).
1986
-        $markup1   = <<<MARKUP
1986
+        $markup1 = <<<MARKUP
1987 1987
 <script>
1988 1988
 var a = "b";
1989 1989
 <!-- End Support AJAX add to cart -->
@@ -1994,7 +1994,7 @@  discard block
 block discarded – undo
1994 1994
 <!-- End Support AJAX add to cart</script>
1995 1995
 MARKUP;
1996 1996
 
1997
-        $markup2   = <<<MARKUP
1997
+        $markup2 = <<<MARKUP
1998 1998
 <script>
1999 1999
 <!-- End Support AJAX add to cart -->
2000 2000
 var a = "b";
@@ -2011,17 +2011,17 @@  discard block
 block discarded – undo
2011 2011
             ],
2012 2012
         ];
2013 2013
 
2014
-        $instance = new autoptimizeHTML( $markup1 );
2015
-        $instance->read( $options['autoptimizeHTML'] );
2014
+        $instance = new autoptimizeHTML($markup1);
2015
+        $instance->read($options['autoptimizeHTML']);
2016 2016
         $instance->minify();
2017 2017
         $actual = $instance->getcontent();
2018
-        $this->assertEquals( $expected1, $actual );
2018
+        $this->assertEquals($expected1, $actual);
2019 2019
 
2020
-        $instance = new autoptimizeHTML( $markup2 );
2021
-        $instance->read( $options['autoptimizeHTML'] );
2020
+        $instance = new autoptimizeHTML($markup2);
2021
+        $instance->read($options['autoptimizeHTML']);
2022 2022
         $instance->minify();
2023 2023
         $actual2 = $instance->getcontent();
2024
-        $this->assertEquals( $expected2, $actual2 );
2024
+        $this->assertEquals($expected2, $actual2);
2025 2025
     }
2026 2026
 
2027 2027
     public function test_html_minify_html_comments_inside_script_blocks_old_school_pattern()
@@ -2045,11 +2045,11 @@  discard block
 block discarded – undo
2045 2045
             ],
2046 2046
         ];
2047 2047
 
2048
-        $instance = new autoptimizeHTML( $markup );
2049
-        $instance->read( $options['autoptimizeHTML'] );
2048
+        $instance = new autoptimizeHTML($markup);
2049
+        $instance->read($options['autoptimizeHTML']);
2050 2050
         $instance->minify();
2051 2051
         $actual = $instance->getcontent();
2052
-        $this->assertEquals( $expected, $actual );
2052
+        $this->assertEquals($expected, $actual);
2053 2053
     }
2054 2054
 
2055 2055
     public function test_html_minify_html_comments_inside_script_blocks_old_school_pattern_untouched()
@@ -2074,49 +2074,49 @@  discard block
 block discarded – undo
2074 2074
             ],
2075 2075
         ];
2076 2076
 
2077
-        $instance = new autoptimizeHTML( $markup );
2078
-        $instance->read( $options['autoptimizeHTML'] );
2077
+        $instance = new autoptimizeHTML($markup);
2078
+        $instance->read($options['autoptimizeHTML']);
2079 2079
         $instance->minify();
2080 2080
         $actual = $instance->getcontent();
2081
-        $this->assertEquals( $expected, $actual );
2081
+        $this->assertEquals($expected, $actual);
2082 2082
     }
2083 2083
 
2084 2084
     public function test_utils_mbstring_availabilty_overriding()
2085 2085
     {
2086 2086
         $orig     = autoptimizeUtils::mbstring_available();
2087
-        $opposite = ! $orig;
2087
+        $opposite = !$orig;
2088 2088
 
2089
-        $this->assertSame( $orig, autoptimizeUtils::mbstring_available() );
2089
+        $this->assertSame($orig, autoptimizeUtils::mbstring_available());
2090 2090
         // Override works...
2091
-        $this->assertSame( $opposite, autoptimizeUtils::mbstring_available( $opposite ) );
2091
+        $this->assertSame($opposite, autoptimizeUtils::mbstring_available($opposite));
2092 2092
         // And override remains cached as the last version.
2093
-        $this->assertSame( $opposite, autoptimizeUtils::mbstring_available() );
2093
+        $this->assertSame($opposite, autoptimizeUtils::mbstring_available());
2094 2094
     }
2095 2095
 
2096 2096
     public function test_utils_mbstring_basics()
2097 2097
     {
2098 2098
         // Turn on mbstring usage.
2099
-        autoptimizeUtils::mbstring_available( true );
2099
+        autoptimizeUtils::mbstring_available(true);
2100 2100
 
2101
-        $this->assertSame( 2, autoptimizeUtils::strlen( "\x00\xFF", 'ASCII' ) );
2102
-        $this->assertSame( 2, autoptimizeUtils::strlen( "\x00\xFF", 'CP850' ) );
2103
-        $this->assertSame( 3, autoptimizeUtils::strlen( '한국어' ) );
2101
+        $this->assertSame(2, autoptimizeUtils::strlen("\x00\xFF", 'ASCII'));
2102
+        $this->assertSame(2, autoptimizeUtils::strlen("\x00\xFF", 'CP850'));
2103
+        $this->assertSame(3, autoptimizeUtils::strlen('한국어'));
2104 2104
 
2105
-        $this->assertFalse( @autoptimizeUtils::strpos( 'abc', '' ) );
2106
-        $this->assertFalse( @autoptimizeUtils::strpos( 'abc', 'a', -1 ) );
2107
-        $this->assertFalse( autoptimizeUtils::strpos( 'abc', 'd' ) );
2108
-        $this->assertFalse( autoptimizeUtils::strpos( 'abc', 'a', 3 ) );
2109
-        $this->assertSame( 1, autoptimizeUtils::strpos( '한국어', '국' ) );
2105
+        $this->assertFalse(@autoptimizeUtils::strpos('abc', ''));
2106
+        $this->assertFalse(@autoptimizeUtils::strpos('abc', 'a', -1));
2107
+        $this->assertFalse(autoptimizeUtils::strpos('abc', 'd'));
2108
+        $this->assertFalse(autoptimizeUtils::strpos('abc', 'a', 3));
2109
+        $this->assertSame(1, autoptimizeUtils::strpos('한국어', '국'));
2110 2110
     }
2111 2111
 
2112 2112
     /**
2113 2113
      * @dataProvider provider_utils_substr_replace
2114 2114
      */
2115
-    function test_utils_substr_replace_basics_mbstring( $s, $repl, $start, $len, $expected )
2115
+    function test_utils_substr_replace_basics_mbstring($s, $repl, $start, $len, $expected)
2116 2116
     {
2117 2117
         // Force mbstring code path...
2118
-        autoptimizeUtils::mbstring_available( true );
2119
-        $this->assertEquals( $expected, autoptimizeUtils::substr_replace( $s, $repl, $start, $len ) );
2118
+        autoptimizeUtils::mbstring_available(true);
2119
+        $this->assertEquals($expected, autoptimizeUtils::substr_replace($s, $repl, $start, $len));
2120 2120
     }
2121 2121
 
2122 2122
     public function provider_utils_substr_replace()
@@ -2158,48 +2158,48 @@  discard block
 block discarded – undo
2158 2158
 
2159 2159
     function test_mb_substr_replace_with_ascii_input_string()
2160 2160
     {
2161
-        autoptimizeUtils::mbstring_available( false );
2161
+        autoptimizeUtils::mbstring_available(false);
2162 2162
 
2163 2163
         $str = 'Ascii';
2164 2164
 
2165
-        $this->assertSame( 'Añ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) );
2166
-        $this->assertSame( 'ñcii', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2167
-        $this->assertSame( 'Asñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2168
-        $this->assertSame( 'Asz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) );
2169
-        $this->assertSame( 'Añii', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2165
+        $this->assertSame('Añ', autoptimizeUtils::substr_replace($str, 'ñ', 1));
2166
+        $this->assertSame('ñcii', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2167
+        $this->assertSame('Asñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2168
+        $this->assertSame('Asz', autoptimizeUtils::substr_replace($str, 'z', 2, 10));
2169
+        $this->assertSame('Añii', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2170 2170
     }
2171 2171
 
2172 2172
     function test_mb_substr_replace_with_utf8_input_string()
2173 2173
     {
2174
-        autoptimizeUtils::mbstring_available( true );
2174
+        autoptimizeUtils::mbstring_available(true);
2175 2175
 
2176 2176
         $str = 'âønæë';
2177 2177
 
2178
-        $this->assertSame( 'âñ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) ); // No length.
2179
-        $this->assertSame( 'ñnæë', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2180
-        $this->assertSame( 'âøñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2181
-        $this->assertSame( 'âøz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) ); // Length larger than possible...
2182
-        $this->assertSame( 'âñæë', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2178
+        $this->assertSame('âñ', autoptimizeUtils::substr_replace($str, 'ñ', 1)); // No length.
2179
+        $this->assertSame('ñnæë', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2180
+        $this->assertSame('âøñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2181
+        $this->assertSame('âøz', autoptimizeUtils::substr_replace($str, 'z', 2, 10)); // Length larger than possible...
2182
+        $this->assertSame('âñæë', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2183 2183
     }
2184 2184
 
2185 2185
     function test_default_substr_replace_with_ascii_input_string()
2186 2186
     {
2187 2187
         // Disable mbstring which should fall ack to substr_replace...
2188
-        autoptimizeUtils::mbstring_available( false );
2188
+        autoptimizeUtils::mbstring_available(false);
2189 2189
 
2190 2190
         $str = 'Ascii';
2191 2191
 
2192
-        $this->assertSame( 'Añ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) );
2193
-        $this->assertSame( 'ñcii', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2194
-        $this->assertSame( 'Asñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2195
-        $this->assertSame( 'Asz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) );
2196
-        $this->assertSame( 'Añii', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2192
+        $this->assertSame('Añ', autoptimizeUtils::substr_replace($str, 'ñ', 1));
2193
+        $this->assertSame('ñcii', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2194
+        $this->assertSame('Asñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2195
+        $this->assertSame('Asz', autoptimizeUtils::substr_replace($str, 'z', 2, 10));
2196
+        $this->assertSame('Añii', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2197 2197
     }
2198 2198
 
2199 2199
     function test_default_substr_replace_with_utf8_input_string()
2200 2200
     {
2201 2201
         // Disabling mbstring, falling back to substr_replace...
2202
-        autoptimizeUtils::mbstring_available( false );
2202
+        autoptimizeUtils::mbstring_available(false);
2203 2203
 
2204 2204
         // This is really impossible to make work properly, since
2205 2205
         // any start/len parameters we give are working with bytes instead
@@ -2208,55 +2208,55 @@  discard block
 block discarded – undo
2208 2208
 
2209 2209
         // $this->assertSame( '�ñ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) ); // No length.
2210 2210
         // $this->assertSame( 'ñ�næë', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2211
-        $this->assertSame( 'ñønæë', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2212
-        $this->assertSame( 'âñxæë', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2213
-        $this->assertSame( 'âz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) ); // Length larger than possible...
2211
+        $this->assertSame('ñønæë', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2212
+        $this->assertSame('âñxæë', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2213
+        $this->assertSame('âz', autoptimizeUtils::substr_replace($str, 'z', 2, 10)); // Length larger than possible...
2214 2214
     }
2215 2215
 
2216 2216
     public function test_cache_fast_delete()
2217 2217
     {
2218
-        add_filter( 'autoptimize_filter_cache_clear_advanced', '__return_true' );
2218
+        add_filter('autoptimize_filter_cache_clear_advanced', '__return_true');
2219 2219
 
2220 2220
         autoptimizeCache::clearall_actionless();
2221 2221
 
2222
-        remove_all_filters( 'autoptimize_filter_cache_clear_advanced' );
2222
+        remove_all_filters('autoptimize_filter_cache_clear_advanced');
2223 2223
     }
2224 2224
 
2225 2225
     public function test_delete_advanced_cache_clear_artifacts()
2226 2226
     {
2227
-        $this->assertTrue( autoptimizeCache::delete_advanced_cache_clear_artifacts() );
2227
+        $this->assertTrue(autoptimizeCache::delete_advanced_cache_clear_artifacts());
2228 2228
     }
2229 2229
 
2230 2230
     public function provider_canonicalization()
2231 2231
     {
2232 2232
         return array(
2233
-            array( '../common', 'common' ),
2234
-            array( '../what-does-this-mean/really?/', 'what-does-this-mean/really?/' ),
2235
-            array( '../../what/where/how', 'what/where/how' ),
2236
-            array( '/../more.dots.please/', '/more.dots.please/' ),
2237
-            array( '/../../what/where/how', '/what/where/how' ),
2238
-            array( '/a/b/c/../../../d/e/file.txt', '/d/e/file.txt' ),
2239
-            array( 'a/b/../c', 'a/c' ),
2240
-            array( './../../etc/passwd', './etc/passwd' ),
2241
-            array( '/var/.////./user/./././..//.//../////../././.././test/////', '/test/' ),
2242
-            array( '/var/user/./././.././../.././././test/', '/test/' ),
2243
-            array( '/hello/0//how/../are/../you', '/hello/0/you' ),
2244
-            array( '', '' ),
2245
-            array( '.', '.' ),
2246
-            array( '..', '..' ),
2247
-            array( './..', './..' ),
2248
-            array( '../.', '.' ),
2233
+            array('../common', 'common'),
2234
+            array('../what-does-this-mean/really?/', 'what-does-this-mean/really?/'),
2235
+            array('../../what/where/how', 'what/where/how'),
2236
+            array('/../more.dots.please/', '/more.dots.please/'),
2237
+            array('/../../what/where/how', '/what/where/how'),
2238
+            array('/a/b/c/../../../d/e/file.txt', '/d/e/file.txt'),
2239
+            array('a/b/../c', 'a/c'),
2240
+            array('./../../etc/passwd', './etc/passwd'),
2241
+            array('/var/.////./user/./././..//.//../////../././.././test/////', '/test/'),
2242
+            array('/var/user/./././.././../.././././test/', '/test/'),
2243
+            array('/hello/0//how/../are/../you', '/hello/0/you'),
2244
+            array('', ''),
2245
+            array('.', '.'),
2246
+            array('..', '..'),
2247
+            array('./..', './..'),
2248
+            array('../.', '.'),
2249 2249
             // This might be debatable...
2250
-            array( '../..', '..' ),
2250
+            array('../..', '..'),
2251 2251
         );
2252 2252
     }
2253 2253
 
2254 2254
     /**
2255 2255
      * @dataProvider provider_canonicalization
2256 2256
      */
2257
-    public function test_path_canonicalize( $path, $canonicalized )
2257
+    public function test_path_canonicalize($path, $canonicalized)
2258 2258
     {
2259
-        $this->assertSame( $canonicalized, autoptimizeUtils::path_canonicalize( $path ) );
2259
+        $this->assertSame($canonicalized, autoptimizeUtils::path_canonicalize($path));
2260 2260
     }
2261 2261
 
2262 2262
     /**
@@ -2270,16 +2270,16 @@  discard block
 block discarded – undo
2270 2270
     public function test_autoptimize_filter_cdn_magic_path_check()
2271 2271
     {
2272 2272
         $cdnurl = 'http://cdn.example.org';
2273
-        add_filter( 'autoptimize_filter_cdn_magic_path_check', '__return_false', 10, 2 );
2273
+        add_filter('autoptimize_filter_cdn_magic_path_check', '__return_false', 10, 2);
2274 2274
 
2275 2275
         // Even when site is in a subfolder, the resulting cdn-rewritten url
2276 2276
         // should not magically include it, due to the above filter.
2277 2277
         // The second parameter is here to force a cache miss and re-run
2278 2278
         // the filter since we're using the same cdn url all over the place,
2279 2279
         // but want to prove different things with it.
2280
-        $this->assertSame( $cdnurl, autoptimizeUtils::tweak_cdn_url_if_needed( $cdnurl, true ) );
2280
+        $this->assertSame($cdnurl, autoptimizeUtils::tweak_cdn_url_if_needed($cdnurl, true));
2281 2281
 
2282
-        remove_all_filters( 'autoptimize_filter_cdn_magic_path_check' );
2282
+        remove_all_filters('autoptimize_filter_cdn_magic_path_check');
2283 2283
     }
2284 2284
 
2285 2285
     /**
@@ -2302,8 +2302,8 @@  discard block
 block discarded – undo
2302 2302
 MARKUP;
2303 2303
 
2304 2304
         $instance = new autoptimizeExtra();
2305
-        $actual = $instance->filter_optimize_images( $markup );
2306
-        $this->assertEquals( $expected, $actual );
2305
+        $actual = $instance->filter_optimize_images($markup);
2306
+        $this->assertEquals($expected, $actual);
2307 2307
     }
2308 2308
 
2309 2309
     /**
@@ -2324,8 +2324,8 @@  discard block
 block discarded – undo
2324 2324
 MARKUP;
2325 2325
 
2326 2326
         $instance = new autoptimizeExtra();
2327
-        $actual = $instance->filter_optimize_images( $markup );
2328
-        $this->assertEquals( $expected, $actual );
2327
+        $actual = $instance->filter_optimize_images($markup);
2328
+        $this->assertEquals($expected, $actual);
2329 2329
     }
2330 2330
 
2331 2331
     /**
@@ -2348,8 +2348,8 @@  discard block
 block discarded – undo
2348 2348
 MARKUP;
2349 2349
 
2350 2350
         $instance = new autoptimizeExtra();
2351
-        $actual = $instance->filter_optimize_images( $markup );
2352
-        $this->assertEquals( $expected, $actual );
2351
+        $actual = $instance->filter_optimize_images($markup);
2352
+        $this->assertEquals($expected, $actual);
2353 2353
     }
2354 2354
 
2355 2355
     /**
@@ -2359,7 +2359,7 @@  discard block
 block discarded – undo
2359 2359
     {
2360 2360
         $actual = autoptimizeUtils::str_is_valid_regex($str);
2361 2361
 
2362
-        $this->assertEquals( $expected, $actual );
2362
+        $this->assertEquals($expected, $actual);
2363 2363
     }
2364 2364
 
2365 2365
     public function provider_str_is_valid_regex()
@@ -2375,11 +2375,11 @@  discard block
 block discarded – undo
2375 2375
     /**
2376 2376
      * @dataProvider provider_protocol_relative_tests
2377 2377
      */
2378
-    public function test_is_protocol_relative( $str, $expected )
2378
+    public function test_is_protocol_relative($str, $expected)
2379 2379
     {
2380
-        $actual = autoptimizeUtils::is_protocol_relative( $str );
2380
+        $actual = autoptimizeUtils::is_protocol_relative($str);
2381 2381
 
2382
-        $this->assertEquals( $expected, $actual );
2382
+        $this->assertEquals($expected, $actual);
2383 2383
     }
2384 2384
 
2385 2385
     public function provider_protocol_relative_tests()
@@ -2406,31 +2406,31 @@  discard block
 block discarded – undo
2406 2406
 
2407 2407
         $expected = 'img.centered,.aligncenter{-webkit-box-shadow:none;-moz-box-shadow:none;box-shadow:none}';
2408 2408
 
2409
-        $instance = new autoptimizeStyles( $css );
2410
-        $minified = $instance->run_minifier_on( $css );
2409
+        $instance = new autoptimizeStyles($css);
2410
+        $minified = $instance->run_minifier_on($css);
2411 2411
 
2412
-        $this->assertEquals( $expected, $minified );
2412
+        $this->assertEquals($expected, $minified);
2413 2413
     }
2414 2414
 
2415 2415
     public function test_css_font_names_mangling()
2416 2416
     {
2417 2417
         // Properly quoted font names dont get mangled.
2418 2418
         $css      = 'h2{font-family:"Archivo Black"}';
2419
-        $instance = new autoptimizeStyles( $css );
2420
-        $actual   = $instance->run_minifier_on( $css );
2421
-        $this->assertEquals( $css, $actual );
2419
+        $instance = new autoptimizeStyles($css);
2420
+        $actual   = $instance->run_minifier_on($css);
2421
+        $this->assertEquals($css, $actual);
2422 2422
 
2423 2423
         // When not quoted, 'Black' used to become '#000', but not anymore... :)
2424 2424
         $css_unquoted = 'h2{font-family:Archivo Black;}';
2425 2425
         $expected     = 'h2{font-family:Archivo Black}';
2426
-        $instance     = new autoptimizeStyles( $css_unquoted );
2427
-        $actual       = $instance->run_minifier_on( $css_unquoted );
2428
-        $this->assertEquals( $expected, $actual );
2426
+        $instance     = new autoptimizeStyles($css_unquoted);
2427
+        $actual       = $instance->run_minifier_on($css_unquoted);
2428
+        $this->assertEquals($expected, $actual);
2429 2429
 
2430 2430
         $css_unquoted = 'h1{font:italic 1.2em Fira White,serif}';
2431
-        $instance     = new autoptimizeStyles( $css_unquoted );
2432
-        $actual       = $instance->run_minifier_on( $css_unquoted );
2433
-        $this->assertEquals( $css_unquoted, $actual );
2431
+        $instance     = new autoptimizeStyles($css_unquoted);
2432
+        $actual       = $instance->run_minifier_on($css_unquoted);
2433
+        $this->assertEquals($css_unquoted, $actual);
2434 2434
     }
2435 2435
 
2436 2436
     public function test_jsminphp_string_literal_minification()
@@ -2444,7 +2444,7 @@  discard block
 block discarded – undo
2444 2444
         $expected = '`line
2445 2445
 break`+`he  llo`;foo`hel( \'\');lo`;`he\nl\`lo`;(`he${one + two}`)';
2446 2446
 
2447
-        $actual = JSMin::minify( $js );
2448
-        $this->assertEquals( $expected, $actual );
2447
+        $actual = JSMin::minify($js);
2448
+        $this->assertEquals($expected, $actual);
2449 2449
     }
2450 2450
 }
Please login to merge, or discard this patch.
classes/autoptimizeCache.php 1 patch
Spacing   +135 added lines, -135 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * Handles disk-cache-related operations.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
@@ -38,17 +38,17 @@  discard block
 block discarded – undo
38 38
      * @param string $md5 Hash.
39 39
      * @param string $ext Extension.
40 40
      */
41
-    public function __construct( $md5, $ext = 'php' )
41
+    public function __construct($md5, $ext = 'php')
42 42
     {
43 43
         $this->cachedir = AUTOPTIMIZE_CACHE_DIR;
44 44
         $this->nogzip   = AUTOPTIMIZE_CACHE_NOGZIP;
45
-        if ( ! $this->nogzip ) {
46
-            $this->filename = AUTOPTIMIZE_CACHEFILE_PREFIX . $md5 . '.php';
45
+        if (!$this->nogzip) {
46
+            $this->filename = AUTOPTIMIZE_CACHEFILE_PREFIX.$md5.'.php';
47 47
         } else {
48
-            if ( in_array( $ext, array( 'js', 'css' ) ) ) {
49
-                $this->filename = $ext . '/' . AUTOPTIMIZE_CACHEFILE_PREFIX . $md5 . '.' . $ext;
48
+            if (in_array($ext, array('js', 'css'))) {
49
+                $this->filename = $ext.'/'.AUTOPTIMIZE_CACHEFILE_PREFIX.$md5.'.'.$ext;
50 50
             } else {
51
-                $this->filename = AUTOPTIMIZE_CACHEFILE_PREFIX . $md5 . '.' . $ext;
51
+                $this->filename = AUTOPTIMIZE_CACHEFILE_PREFIX.$md5.'.'.$ext;
52 52
             }
53 53
         }
54 54
     }
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
      */
61 61
     public function check()
62 62
     {
63
-        return file_exists( $this->cachedir . $this->filename );
63
+        return file_exists($this->cachedir.$this->filename);
64 64
     }
65 65
 
66 66
     /**
@@ -70,11 +70,11 @@  discard block
 block discarded – undo
70 70
      */
71 71
     public function retrieve()
72 72
     {
73
-        if ( $this->check() ) {
74
-            if ( false == $this->nogzip ) {
75
-                return file_get_contents( $this->cachedir . $this->filename . '.none' );
73
+        if ($this->check()) {
74
+            if (false == $this->nogzip) {
75
+                return file_get_contents($this->cachedir.$this->filename.'.none');
76 76
             } else {
77
-                return file_get_contents( $this->cachedir . $this->filename );
77
+                return file_get_contents($this->cachedir.$this->filename);
78 78
             }
79 79
         }
80 80
         return false;
@@ -88,22 +88,22 @@  discard block
 block discarded – undo
88 88
      *
89 89
      * @return void
90 90
      */
91
-    public function cache( $data, $mime )
91
+    public function cache($data, $mime)
92 92
     {
93
-        if ( false === $this->nogzip ) {
93
+        if (false === $this->nogzip) {
94 94
             // We handle gzipping ourselves.
95 95
             $file    = 'default.php';
96
-            $phpcode = file_get_contents( AUTOPTIMIZE_PLUGIN_DIR . 'config/' . $file );
97
-            $phpcode = str_replace( array( '%%CONTENT%%', 'exit;' ), array( $mime, '' ), $phpcode );
96
+            $phpcode = file_get_contents(AUTOPTIMIZE_PLUGIN_DIR.'config/'.$file);
97
+            $phpcode = str_replace(array('%%CONTENT%%', 'exit;'), array($mime, ''), $phpcode);
98 98
 
99
-            file_put_contents( $this->cachedir . $this->filename, $phpcode );
100
-            file_put_contents( $this->cachedir . $this->filename . '.none', $data );
99
+            file_put_contents($this->cachedir.$this->filename, $phpcode);
100
+            file_put_contents($this->cachedir.$this->filename.'.none', $data);
101 101
         } else {
102 102
             // Write code to cache without doing anything else.
103
-            file_put_contents( $this->cachedir . $this->filename, $data );
104
-            if ( apply_filters( 'autoptimize_filter_cache_create_static_gzip', false ) ) {
103
+            file_put_contents($this->cachedir.$this->filename, $data);
104
+            if (apply_filters('autoptimize_filter_cache_create_static_gzip', false)) {
105 105
                 // Create an additional cached gzip file.
106
-                file_put_contents( $this->cachedir . $this->filename . '.gz', gzencode( $data, 9, FORCE_GZIP ) );
106
+                file_put_contents($this->cachedir.$this->filename.'.gz', gzencode($data, 9, FORCE_GZIP));
107 107
             }
108 108
         }
109 109
     }
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
         // The original idea here was to provide 3rd party code a hook so that
121 121
         // it can "listen" to all the complete autoptimized-urls that the page
122 122
         // will emit... Or something to that effect I think?
123
-        apply_filters( 'autoptimize_filter_cache_getname', AUTOPTIMIZE_CACHE_URL . $this->filename );
123
+        apply_filters('autoptimize_filter_cache_getname', AUTOPTIMIZE_CACHE_URL.$this->filename);
124 124
 
125 125
         return $this->filename;
126 126
     }
@@ -133,11 +133,11 @@  discard block
 block discarded – undo
133 133
      * @param string $file Filename.
134 134
      * @return bool
135 135
      */
136
-    protected static function is_valid_cache_file( $dir, $file )
136
+    protected static function is_valid_cache_file($dir, $file)
137 137
     {
138
-        if ( '.' !== $file && '..' !== $file &&
139
-            false !== strpos( $file, AUTOPTIMIZE_CACHEFILE_PREFIX ) &&
140
-            is_file( $dir . $file ) ) {
138
+        if ('.' !== $file && '..' !== $file &&
139
+            false !== strpos($file, AUTOPTIMIZE_CACHEFILE_PREFIX) &&
140
+            is_file($dir.$file)) {
141 141
 
142 142
             // It's a valid file!
143 143
             return true;
@@ -155,16 +155,16 @@  discard block
 block discarded – undo
155 155
     protected static function clear_cache_classic()
156 156
     {
157 157
         $contents = self::get_cache_contents();
158
-        foreach ( $contents as $name => $files ) {
159
-            $dir = rtrim( AUTOPTIMIZE_CACHE_DIR . $name, '/' ) . '/';
160
-            foreach ( $files as $file ) {
161
-                if ( self::is_valid_cache_file( $dir, $file ) ) {
162
-                    @unlink( $dir . $file ); // @codingStandardsIgnoreLine
158
+        foreach ($contents as $name => $files) {
159
+            $dir = rtrim(AUTOPTIMIZE_CACHE_DIR.$name, '/').'/';
160
+            foreach ($files as $file) {
161
+                if (self::is_valid_cache_file($dir, $file)) {
162
+                    @unlink($dir.$file); // @codingStandardsIgnoreLine
163 163
                 }
164 164
             }
165 165
         }
166 166
 
167
-        @unlink( AUTOPTIMIZE_CACHE_DIR . '/.htaccess' ); // @codingStandardsIgnoreLine
167
+        @unlink(AUTOPTIMIZE_CACHE_DIR.'/.htaccess'); // @codingStandardsIgnoreLine
168 168
     }
169 169
 
170 170
     /**
@@ -175,19 +175,19 @@  discard block
 block discarded – undo
175 175
      *
176 176
      * @return bool
177 177
      */
178
-    protected static function rmdir( $pathname )
178
+    protected static function rmdir($pathname)
179 179
     {
180
-        $files = self::get_dir_contents( $pathname );
181
-        foreach ( $files as $file ) {
182
-            $path = $pathname . '/' . $file;
183
-            if ( is_dir( $path ) ) {
184
-                self::rmdir( $path );
180
+        $files = self::get_dir_contents($pathname);
181
+        foreach ($files as $file) {
182
+            $path = $pathname.'/'.$file;
183
+            if (is_dir($path)) {
184
+                self::rmdir($path);
185 185
             } else {
186
-                unlink( $path );
186
+                unlink($path);
187 187
             }
188 188
         }
189 189
 
190
-        return rmdir( $pathname );
190
+        return rmdir($pathname);
191 191
     }
192 192
 
193 193
     /**
@@ -204,12 +204,12 @@  discard block
 block discarded – undo
204 204
         $new_name = self::get_unique_name();
205 205
 
206 206
         // Makes sure the new pathname is on the same level...
207
-        $new_pathname = dirname( $dir ) . '/' . $new_name;
208
-        $renamed      = @rename( $dir, $new_pathname ); // @codingStandardsIgnoreLine
207
+        $new_pathname = dirname($dir).'/'.$new_name;
208
+        $renamed      = @rename($dir, $new_pathname); // @codingStandardsIgnoreLine
209 209
 
210 210
         // When renamed, re-create the default cache directory back so it's
211 211
         // available again...
212
-        if ( $renamed ) {
212
+        if ($renamed) {
213 213
             $ok = self::cacheavail();
214 214
         }
215 215
 
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
      */
224 224
     public static function advanced_cache_clear_enabled()
225 225
     {
226
-        return apply_filters( 'autoptimize_filter_cache_clear_advanced', false );
226
+        return apply_filters('autoptimize_filter_cache_clear_advanced', false);
227 227
     }
228 228
 
229 229
     /**
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
     protected static function get_unique_name()
235 235
     {
236 236
         $prefix   = self::get_advanced_cache_clear_prefix();
237
-        $new_name = uniqid( $prefix, true );
237
+        $new_name = uniqid($prefix, true);
238 238
 
239 239
         return $new_name;
240 240
     }
@@ -247,8 +247,8 @@  discard block
 block discarded – undo
247 247
     protected static function get_advanced_cache_clear_prefix()
248 248
     {
249 249
         $pathname = self::get_pathname_base();
250
-        $basename = basename( $pathname );
251
-        $prefix   = $basename . '-';
250
+        $basename = basename($pathname);
251
+        $prefix   = $basename.'-';
252 252
 
253 253
         return $prefix;
254 254
     }
@@ -261,9 +261,9 @@  discard block
 block discarded – undo
261 261
      *
262 262
      * @return array
263 263
      */
264
-    protected static function get_dir_contents( $pathname )
264
+    protected static function get_dir_contents($pathname)
265 265
     {
266
-        return array_slice( scandir( $pathname ), 2 );
266
+        return array_slice(scandir($pathname), 2);
267 267
     }
268 268
 
269 269
     /**
@@ -277,18 +277,18 @@  discard block
 block discarded – undo
277 277
     {
278 278
         $dir    = self::get_pathname_base();
279 279
         $prefix = self::get_advanced_cache_clear_prefix();
280
-        $parent = dirname( $dir );
280
+        $parent = dirname($dir);
281 281
         $ok     = false;
282 282
 
283 283
         // Returns the list of files without '.' and '..' elements.
284
-        $files = self::get_dir_contents( $parent );
285
-        if ( is_array( $files ) && ! empty( $files ) ) {
286
-            foreach ( $files as $file ) {
287
-                $path     = $parent . '/' . $file;
288
-                $prefixed = ( false !== strpos( $path, $prefix ) );
284
+        $files = self::get_dir_contents($parent);
285
+        if (is_array($files) && !empty($files)) {
286
+            foreach ($files as $file) {
287
+                $path     = $parent.'/'.$file;
288
+                $prefixed = (false !== strpos($path, $prefix));
289 289
                 // Removing only our own (prefixed) directories...
290
-                if ( is_dir( $path ) && $prefixed ) {
291
-                    $ok = self::rmdir( $path );
290
+                if (is_dir($path) && $prefixed) {
291
+                    $ok = self::rmdir($path);
292 292
                 }
293 293
             }
294 294
         }
@@ -308,9 +308,9 @@  discard block
 block discarded – undo
308 308
     {
309 309
         $pathname = self::get_pathname_base();
310 310
 
311
-        if ( is_multisite() && apply_filters( 'autoptimize_separate_blog_caches', true ) ) {
311
+        if (is_multisite() && apply_filters('autoptimize_separate_blog_caches', true)) {
312 312
             $blog_id   = get_current_blog_id();
313
-            $pathname .= $blog_id . '/';
313
+            $pathname .= $blog_id.'/';
314 314
         }
315 315
 
316 316
         return $pathname;
@@ -323,7 +323,7 @@  discard block
 block discarded – undo
323 323
      */
324 324
     protected static function get_pathname_base()
325 325
     {
326
-        $pathname = WP_CONTENT_DIR . AUTOPTIMIZE_CACHE_CHILD_DIR;
326
+        $pathname = WP_CONTENT_DIR.AUTOPTIMIZE_CACHE_CHILD_DIR;
327 327
 
328 328
         return $pathname;
329 329
     }
@@ -335,38 +335,38 @@  discard block
 block discarded – undo
335 335
      *
336 336
      * @return bool
337 337
      */
338
-    public static function clearall( $propagate = true )
338
+    public static function clearall($propagate = true)
339 339
     {
340
-        if ( ! self::cacheavail() ) {
340
+        if (!self::cacheavail()) {
341 341
             return false;
342 342
         }
343 343
 
344 344
         // TODO/FIXME: If cache is big, switch to advanced/new cache clearing automatically?
345
-        if ( self::advanced_cache_clear_enabled() ) {
345
+        if (self::advanced_cache_clear_enabled()) {
346 346
             self::clear_cache_via_rename();
347 347
         } else {
348 348
             self::clear_cache_classic();
349 349
         }
350 350
 
351 351
         // Remove the transient so it gets regenerated...
352
-        delete_transient( 'autoptimize_stats' );
352
+        delete_transient('autoptimize_stats');
353 353
 
354 354
         // Cache was just purged, clear page cache and allow others to hook into our purging...
355
-        if ( true === $propagate ) {
356
-            if ( ! function_exists( 'autoptimize_do_cachepurged_action' ) ) {
355
+        if (true === $propagate) {
356
+            if (!function_exists('autoptimize_do_cachepurged_action')) {
357 357
                 function autoptimize_do_cachepurged_action() {
358
-                    do_action( 'autoptimize_action_cachepurged' );
358
+                    do_action('autoptimize_action_cachepurged');
359 359
                 }
360 360
             }
361
-            add_action( 'shutdown', 'autoptimize_do_cachepurged_action', 11 );
362
-            add_action( 'autoptimize_action_cachepurged', array( 'autoptimizeCache', 'flushPageCache' ), 10, 0 );
361
+            add_action('shutdown', 'autoptimize_do_cachepurged_action', 11);
362
+            add_action('autoptimize_action_cachepurged', array('autoptimizeCache', 'flushPageCache'), 10, 0);
363 363
         }
364 364
 
365 365
         // Warm cache (part of speedupper)!
366
-        if ( apply_filters( 'autoptimize_filter_speedupper', true ) ) {
367
-            $url   = site_url() . '/?ao_speedup_cachebuster=' . rand( 1, 100000 );
368
-            $cache = @wp_remote_get( $url ); // @codingStandardsIgnoreLine
369
-            unset( $cache );
366
+        if (apply_filters('autoptimize_filter_speedupper', true)) {
367
+            $url   = site_url().'/?ao_speedup_cachebuster='.rand(1, 100000);
368
+            $cache = @wp_remote_get($url); // @codingStandardsIgnoreLine
369
+            unset($cache);
370 370
         }
371 371
 
372 372
         return true;
@@ -381,7 +381,7 @@  discard block
 block discarded – undo
381 381
      */
382 382
     public static function clearall_actionless()
383 383
     {
384
-        return self::clearall( false );
384
+        return self::clearall(false);
385 385
     }
386 386
 
387 387
     /**
@@ -393,8 +393,8 @@  discard block
 block discarded – undo
393 393
     {
394 394
         $contents = array();
395 395
 
396
-        foreach ( array( '', 'js', 'css' ) as $dir ) {
397
-            $contents[ $dir ] = scandir( AUTOPTIMIZE_CACHE_DIR . $dir );
396
+        foreach (array('', 'js', 'css') as $dir) {
397
+            $contents[$dir] = scandir(AUTOPTIMIZE_CACHE_DIR.$dir);
398 398
         }
399 399
 
400 400
         return $contents;
@@ -407,21 +407,21 @@  discard block
 block discarded – undo
407 407
      */
408 408
     public static function stats()
409 409
     {
410
-        $stats = get_transient( 'autoptimize_stats' );
410
+        $stats = get_transient('autoptimize_stats');
411 411
 
412 412
         // If no transient, do the actual scan!
413
-        if ( ! is_array( $stats ) ) {
414
-            if ( ! self::cacheavail() ) {
413
+        if (!is_array($stats)) {
414
+            if (!self::cacheavail()) {
415 415
                 return 0;
416 416
             }
417 417
             $stats = self::stats_scan();
418 418
             $count = $stats[0];
419
-            if ( $count > 100 ) {
419
+            if ($count > 100) {
420 420
                 // Store results in transient.
421 421
                 set_transient(
422 422
                     'autoptimize_stats',
423 423
                     $stats,
424
-                    apply_filters( 'autoptimize_filter_cache_statsexpiry', HOUR_IN_SECONDS )
424
+                    apply_filters('autoptimize_filter_cache_statsexpiry', HOUR_IN_SECONDS)
425 425
                 );
426 426
             }
427 427
         }
@@ -444,30 +444,30 @@  discard block
 block discarded – undo
444 444
         $size  = 0;
445 445
 
446 446
         // Scan everything in our cache directories.
447
-        foreach ( self::get_cache_contents() as $name => $files ) {
448
-            $dir = rtrim( AUTOPTIMIZE_CACHE_DIR . $name, '/' ) . '/';
449
-            foreach ( $files as $file ) {
450
-                if ( self::is_valid_cache_file( $dir, $file ) ) {
451
-                    if ( AUTOPTIMIZE_CACHE_NOGZIP &&
447
+        foreach (self::get_cache_contents() as $name => $files) {
448
+            $dir = rtrim(AUTOPTIMIZE_CACHE_DIR.$name, '/').'/';
449
+            foreach ($files as $file) {
450
+                if (self::is_valid_cache_file($dir, $file)) {
451
+                    if (AUTOPTIMIZE_CACHE_NOGZIP &&
452 452
                         (
453
-                            false !== strpos( $file, '.js' ) ||
454
-                            false !== strpos( $file, '.css' ) ||
455
-                            false !== strpos( $file, '.img' ) ||
456
-                            false !== strpos( $file, '.txt' )
453
+                            false !== strpos($file, '.js') ||
454
+                            false !== strpos($file, '.css') ||
455
+                            false !== strpos($file, '.img') ||
456
+                            false !== strpos($file, '.txt')
457 457
                         )
458 458
                     ) {
459 459
                         // Web server is gzipping, we count .js|.css|.img|.txt files.
460 460
                         $count++;
461
-                    } elseif ( ! AUTOPTIMIZE_CACHE_NOGZIP && false !== strpos( $file, '.none' ) ) {
461
+                    } elseif (!AUTOPTIMIZE_CACHE_NOGZIP && false !== strpos($file, '.none')) {
462 462
                         // We are gzipping ourselves via php, counting only .none files.
463 463
                         $count++;
464 464
                     }
465
-                    $size += filesize( $dir . $file );
465
+                    $size += filesize($dir.$file);
466 466
                 }
467 467
             }
468 468
         }
469 469
 
470
-        $stats = array( $count, $size, time() );
470
+        $stats = array($count, $size, time());
471 471
 
472 472
         return $stats;
473 473
     }
@@ -481,29 +481,29 @@  discard block
 block discarded – undo
481 481
      */
482 482
     public static function cacheavail()
483 483
     {
484
-        if ( ! defined( 'AUTOPTIMIZE_CACHE_DIR' ) ) {
484
+        if (!defined('AUTOPTIMIZE_CACHE_DIR')) {
485 485
             // We didn't set a cache.
486 486
             return false;
487 487
         }
488 488
 
489
-        foreach ( array( '', 'js', 'css' ) as $dir ) {
490
-            if ( ! self::check_cache_dir( AUTOPTIMIZE_CACHE_DIR . $dir ) ) {
489
+        foreach (array('', 'js', 'css') as $dir) {
490
+            if (!self::check_cache_dir(AUTOPTIMIZE_CACHE_DIR.$dir)) {
491 491
                 return false;
492 492
             }
493 493
         }
494 494
 
495 495
         // Using .htaccess inside our cache folder to overrule wp-super-cache.
496
-        $htaccess = AUTOPTIMIZE_CACHE_DIR . '/.htaccess';
497
-        if ( ! is_file( $htaccess ) ) {
496
+        $htaccess = AUTOPTIMIZE_CACHE_DIR.'/.htaccess';
497
+        if (!is_file($htaccess)) {
498 498
             /**
499 499
              * Create `wp-content/AO_htaccess_tmpl` file with
500 500
              * whatever htaccess rules you might need
501 501
              * if you want to override default AO htaccess
502 502
              */
503
-            $htaccess_tmpl = WP_CONTENT_DIR . '/AO_htaccess_tmpl';
504
-            if ( is_file( $htaccess_tmpl ) ) {
505
-                $content = file_get_contents( $htaccess_tmpl );
506
-            } elseif ( is_multisite() || ! AUTOPTIMIZE_CACHE_NOGZIP ) {
503
+            $htaccess_tmpl = WP_CONTENT_DIR.'/AO_htaccess_tmpl';
504
+            if (is_file($htaccess_tmpl)) {
505
+                $content = file_get_contents($htaccess_tmpl);
506
+            } elseif (is_multisite() || !AUTOPTIMIZE_CACHE_NOGZIP) {
507 507
                 $content = '<IfModule mod_expires.c>
508 508
         ExpiresActive On
509 509
         ExpiresByType text/css A30672000
@@ -556,7 +556,7 @@  discard block
 block discarded – undo
556 556
     </Files>
557 557
 </IfModule>';
558 558
             }
559
-            @file_put_contents( $htaccess, $content ); // @codingStandardsIgnoreLine
559
+            @file_put_contents($htaccess, $content); // @codingStandardsIgnoreLine
560 560
         }
561 561
 
562 562
         // All OK!
@@ -571,25 +571,25 @@  discard block
 block discarded – undo
571 571
      *
572 572
      * @return bool
573 573
      */
574
-    protected static function check_cache_dir( $dir )
574
+    protected static function check_cache_dir($dir)
575 575
     {
576 576
         // Try creating the dir if it doesn't exist.
577
-        if ( ! file_exists( $dir ) ) {
578
-            @mkdir( $dir, 0775, true ); // @codingStandardsIgnoreLine
579
-            if ( ! file_exists( $dir ) ) {
577
+        if (!file_exists($dir)) {
578
+            @mkdir($dir, 0775, true); // @codingStandardsIgnoreLine
579
+            if (!file_exists($dir)) {
580 580
                 return false;
581 581
             }
582 582
         }
583 583
 
584 584
         // If we still cannot write, bail.
585
-        if ( ! is_writable( $dir ) ) {
585
+        if (!is_writable($dir)) {
586 586
             return false;
587 587
         }
588 588
 
589 589
         // Create an index.html in there to avoid prying eyes!
590
-        $idx_file = rtrim( $dir, '/\\' ) . '/index.html';
591
-        if ( ! is_file( $idx_file ) ) {
592
-            @file_put_contents( $idx_file, '<html><head><meta name="robots" content="noindex, nofollow"></head><body>Generated by <a href="http://wordpress.org/extend/plugins/autoptimize/" rel="nofollow">Autoptimize</a></body></html>' ); // @codingStandardsIgnoreLine
590
+        $idx_file = rtrim($dir, '/\\').'/index.html';
591
+        if (!is_file($idx_file)) {
592
+            @file_put_contents($idx_file, '<html><head><meta name="robots" content="noindex, nofollow"></head><body>Generated by <a href="http://wordpress.org/extend/plugins/autoptimize/" rel="nofollow">Autoptimize</a></body></html>'); // @codingStandardsIgnoreLine
593 593
         }
594 594
 
595 595
         return true;
@@ -603,59 +603,59 @@  discard block
 block discarded – undo
603 603
     // @codingStandardsIgnoreStart
604 604
     public static function flushPageCache()
605 605
     {
606
-        if ( function_exists( 'wp_cache_clear_cache' ) ) {
607
-            if ( is_multisite() ) {
606
+        if (function_exists('wp_cache_clear_cache')) {
607
+            if (is_multisite()) {
608 608
                 $blog_id = get_current_blog_id();
609
-                wp_cache_clear_cache( $blog_id );
609
+                wp_cache_clear_cache($blog_id);
610 610
             } else {
611 611
                 wp_cache_clear_cache();
612 612
             }
613
-        } elseif ( has_action( 'cachify_flush_cache' ) ) {
614
-            do_action( 'cachify_flush_cache' );
615
-        } elseif ( function_exists( 'w3tc_pgcache_flush' ) ) {
613
+        } elseif (has_action('cachify_flush_cache')) {
614
+            do_action('cachify_flush_cache');
615
+        } elseif (function_exists('w3tc_pgcache_flush')) {
616 616
             w3tc_pgcache_flush();
617
-        } elseif ( function_exists( 'wp_fast_cache_bulk_delete_all' ) ) {
617
+        } elseif (function_exists('wp_fast_cache_bulk_delete_all')) {
618 618
             wp_fast_cache_bulk_delete_all();
619
-        } elseif ( class_exists( 'WpFastestCache' ) ) {
619
+        } elseif (class_exists('WpFastestCache')) {
620 620
             $wpfc = new WpFastestCache();
621 621
             $wpfc->deleteCache();
622
-        } elseif ( class_exists( 'c_ws_plugin__qcache_purging_routines' ) ) {
622
+        } elseif (class_exists('c_ws_plugin__qcache_purging_routines')) {
623 623
             c_ws_plugin__qcache_purging_routines::purge_cache_dir(); // quick cache
624
-        } elseif ( class_exists( 'zencache' ) ) {
624
+        } elseif (class_exists('zencache')) {
625 625
             zencache::clear();
626
-        } elseif ( class_exists( 'comet_cache' ) ) {
626
+        } elseif (class_exists('comet_cache')) {
627 627
             comet_cache::clear();
628
-        } elseif ( class_exists( 'WpeCommon' ) ) {
628
+        } elseif (class_exists('WpeCommon')) {
629 629
             // WPEngine cache purge/flush methods to call by default
630 630
             $wpe_methods = array(
631 631
                 'purge_varnish_cache',
632 632
             );
633 633
 
634 634
             // More agressive clear/flush/purge behind a filter
635
-            if ( apply_filters( 'autoptimize_flush_wpengine_aggressive', false ) ) {
636
-                $wpe_methods = array_merge( $wpe_methods, array( 'purge_memcached', 'clear_maxcdn_cache' ) );
635
+            if (apply_filters('autoptimize_flush_wpengine_aggressive', false)) {
636
+                $wpe_methods = array_merge($wpe_methods, array('purge_memcached', 'clear_maxcdn_cache'));
637 637
             }
638 638
 
639 639
             // Filtering the entire list of WpeCommon methods to be called (for advanced usage + easier testing)
640
-            $wpe_methods = apply_filters( 'autoptimize_flush_wpengine_methods', $wpe_methods );
640
+            $wpe_methods = apply_filters('autoptimize_flush_wpengine_methods', $wpe_methods);
641 641
 
642
-            foreach ( $wpe_methods as $wpe_method ) {
643
-                if ( method_exists( 'WpeCommon', $wpe_method ) ) {
642
+            foreach ($wpe_methods as $wpe_method) {
643
+                if (method_exists('WpeCommon', $wpe_method)) {
644 644
                     WpeCommon::$wpe_method();
645 645
                 }
646 646
             }
647
-        } elseif ( function_exists( 'sg_cachepress_purge_cache' ) ) {
647
+        } elseif (function_exists('sg_cachepress_purge_cache')) {
648 648
             sg_cachepress_purge_cache();
649
-        } elseif ( file_exists( WP_CONTENT_DIR . '/wp-cache-config.php' ) && function_exists( 'prune_super_cache' ) ) {
649
+        } elseif (file_exists(WP_CONTENT_DIR.'/wp-cache-config.php') && function_exists('prune_super_cache')) {
650 650
             // fallback for WP-Super-Cache
651 651
             global $cache_path;
652
-            if ( is_multisite() ) {
652
+            if (is_multisite()) {
653 653
                 $blog_id = get_current_blog_id();
654
-                prune_super_cache( get_supercache_dir( $blog_id ), true );
655
-                prune_super_cache( $cache_path . 'blogs/', true );
654
+                prune_super_cache(get_supercache_dir($blog_id), true);
655
+                prune_super_cache($cache_path.'blogs/', true);
656 656
             } else {
657
-                prune_super_cache( $cache_path . 'supercache/', true );
658
-                prune_super_cache( $cache_path, true );
657
+                prune_super_cache($cache_path.'supercache/', true);
658
+                prune_super_cache($cache_path, true);
659 659
             }
660 660
         }
661 661
     }
Please login to merge, or discard this patch.