Completed
Push — master ( 4d020d...670e37 )
by frank
02:48
created
classes/autoptimizeScripts.php 1 patch
Spacing   +115 added lines, -115 removed lines patch added patch discarded remove patch
@@ -1,11 +1,11 @@  discard block
 block discarded – undo
1 1
 <?php
2
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
2
+if (!defined('ABSPATH')) exit; // Exit if accessed directly
3 3
 
4 4
 class autoptimizeScripts extends autoptimizeBase {
5 5
     private $scripts = array();
6
-    private $dontmove = array('document.write','html5.js','show_ads.js','google_ad','blogcatalog.com/w','tweetmeme.com/i','mybloglog.com/','histats.com/js','ads.smowtion.com/ad.js','statcounter.com/counter/counter.js','widgets.amung.us','ws.amazon.com/widgets','media.fastclick.net','/ads/','comment-form-quicktags/quicktags.php','edToolbar','intensedebate.com','scripts.chitika.net/','_gaq.push','jotform.com/','admin-bar.min.js','GoogleAnalyticsObject','plupload.full.min.js','syntaxhighlighter','adsbygoogle','gist.github.com','_stq','nonce','post_id','data-noptimize');
7
-    private $domove = array('gaJsHost','load_cmc','jd.gallery.transitions.js','swfobject.embedSWF(','tiny_mce.js','tinyMCEPreInit.go');
8
-    private $domovelast = array('addthis.com','/afsonline/show_afs_search.js','disqus.js','networkedblogs.com/getnetworkwidget','infolinks.com/js/','jd.gallery.js.php','jd.gallery.transitions.js','swfobject.embedSWF(','linkwithin.com/widget.js','tiny_mce.js','tinyMCEPreInit.go');
6
+    private $dontmove = array('document.write', 'html5.js', 'show_ads.js', 'google_ad', 'blogcatalog.com/w', 'tweetmeme.com/i', 'mybloglog.com/', 'histats.com/js', 'ads.smowtion.com/ad.js', 'statcounter.com/counter/counter.js', 'widgets.amung.us', 'ws.amazon.com/widgets', 'media.fastclick.net', '/ads/', 'comment-form-quicktags/quicktags.php', 'edToolbar', 'intensedebate.com', 'scripts.chitika.net/', '_gaq.push', 'jotform.com/', 'admin-bar.min.js', 'GoogleAnalyticsObject', 'plupload.full.min.js', 'syntaxhighlighter', 'adsbygoogle', 'gist.github.com', '_stq', 'nonce', 'post_id', 'data-noptimize');
7
+    private $domove = array('gaJsHost', 'load_cmc', 'jd.gallery.transitions.js', 'swfobject.embedSWF(', 'tiny_mce.js', 'tinyMCEPreInit.go');
8
+    private $domovelast = array('addthis.com', '/afsonline/show_afs_search.js', 'disqus.js', 'networkedblogs.com/getnetworkwidget', 'infolinks.com/js/', 'jd.gallery.js.php', 'jd.gallery.transitions.js', 'swfobject.embedSWF(', 'linkwithin.com/widget.js', 'tiny_mce.js', 'tinyMCEPreInit.go');
9 9
     private $trycatch = false;
10 10
     private $alreadyminified = false;
11 11
     private $forcehead = true;
@@ -21,70 +21,70 @@  discard block
 block discarded – undo
21 21
     
22 22
     //Reads the page and collects script tags
23 23
     public function read($options) {
24
-        $noptimizeJS = apply_filters( 'autoptimize_filter_js_noptimize', false, $this->content );
24
+        $noptimizeJS = apply_filters('autoptimize_filter_js_noptimize', false, $this->content);
25 25
         if ($noptimizeJS) return false;
26 26
 
27 27
         // only optimize known good JS?
28
-        $whitelistJS = apply_filters( 'autoptimize_filter_js_whitelist', '', $this->content );
28
+        $whitelistJS = apply_filters('autoptimize_filter_js_whitelist', '', $this->content);
29 29
         if (!empty($whitelistJS)) {
30
-            $this->whitelist = array_filter(array_map('trim',explode(",",$whitelistJS)));
30
+            $this->whitelist = array_filter(array_map('trim', explode(",", $whitelistJS)));
31 31
         }
32 32
 
33 33
         // is there JS we should simply remove
34
-        $removableJS = apply_filters( 'autoptimize_filter_js_removables', '', $this->content );
34
+        $removableJS = apply_filters('autoptimize_filter_js_removables', '', $this->content);
35 35
         if (!empty($removableJS)) {
36
-            $this->jsremovables = array_filter(array_map('trim',explode(",",$removableJS)));
36
+            $this->jsremovables = array_filter(array_map('trim', explode(",", $removableJS)));
37 37
         }
38 38
 
39 39
         // only header?
40
-        if( apply_filters('autoptimize_filter_js_justhead', $options['justhead']) == true ) {
41
-            $content = explode('</head>',$this->content,2);
40
+        if (apply_filters('autoptimize_filter_js_justhead', $options['justhead']) == true) {
41
+            $content = explode('</head>', $this->content, 2);
42 42
             $this->content = $content[0].'</head>';
43 43
             $this->restofcontent = $content[1];
44 44
         }
45 45
         
46 46
         // include inline?
47
-        if( apply_filters('autoptimize_js_include_inline', $options['include_inline']) == true ) {
47
+        if (apply_filters('autoptimize_js_include_inline', $options['include_inline']) == true) {
48 48
             $this->include_inline = true;
49 49
         }
50 50
 
51 51
         // filter to "late inject minified JS", default to true for now (it is faster)
52
-        $this->inject_min_late = apply_filters('autoptimize_filter_js_inject_min_late',true);
52
+        $this->inject_min_late = apply_filters('autoptimize_filter_js_inject_min_late', true);
53 53
 
54 54
         // filters to override hardcoded do(nt)move(last) array contents (array in, array out!)
55
-        $this->dontmove = apply_filters( 'autoptimize_filter_js_dontmove', $this->dontmove );        
56
-        $this->domovelast = apply_filters( 'autoptimize_filter_js_movelast', $this->domovelast );
57
-        $this->domove = apply_filters( 'autoptimize_filter_js_domove', $this->domove );
55
+        $this->dontmove = apply_filters('autoptimize_filter_js_dontmove', $this->dontmove);        
56
+        $this->domovelast = apply_filters('autoptimize_filter_js_movelast', $this->domovelast);
57
+        $this->domove = apply_filters('autoptimize_filter_js_domove', $this->domove);
58 58
 
59 59
         // get extra exclusions settings or filter
60 60
         $excludeJS = $options['js_exclude'];
61
-        $excludeJS = apply_filters( 'autoptimize_filter_js_exclude', $excludeJS, $this->content );
62
-        if ($excludeJS!=="") {
61
+        $excludeJS = apply_filters('autoptimize_filter_js_exclude', $excludeJS, $this->content);
62
+        if ($excludeJS !== "") {
63 63
             if (is_array($excludeJS)) {
64
-                if(($removeKeys = array_keys($excludeJS,"remove")) !== false) {
64
+                if (($removeKeys = array_keys($excludeJS, "remove")) !== false) {
65 65
                     foreach ($removeKeys as $removeKey) {
66 66
                         unset($excludeJS[$removeKey]);
67
-                        $this->jsremovables[]=$removeKey;
67
+                        $this->jsremovables[] = $removeKey;
68 68
                     }
69 69
                 }
70 70
                 $exclJSArr = array_keys($excludeJS);
71 71
             } else {
72
-                $exclJSArr = array_filter(array_map('trim',explode(",",$excludeJS)));
72
+                $exclJSArr = array_filter(array_map('trim', explode(",", $excludeJS)));
73 73
             }
74
-            $this->dontmove = array_merge($exclJSArr,$this->dontmove);
74
+            $this->dontmove = array_merge($exclJSArr, $this->dontmove);
75 75
         }
76 76
 
77 77
         //Should we add try-catch?
78
-        if($options['trycatch'] == true)
78
+        if ($options['trycatch'] == true)
79 79
             $this->trycatch = true;
80 80
 
81 81
         // force js in head?    
82
-        if($options['forcehead'] == true) {
82
+        if ($options['forcehead'] == true) {
83 83
             $this->forcehead = true;
84 84
         } else {
85 85
             $this->forcehead = false;
86 86
         }
87
-        $this->forcehead = apply_filters( 'autoptimize_filter_js_forcehead', $this->forcehead );
87
+        $this->forcehead = apply_filters('autoptimize_filter_js_forcehead', $this->forcehead);
88 88
 
89 89
         // get cdn url
90 90
         $this->cdn_url = $options['cdn_url'];
@@ -99,24 +99,24 @@  discard block
 block discarded – undo
99 99
         $this->content = $this->hide_comments($this->content);
100 100
 
101 101
         // Get script files
102
-        if (preg_match_all('#<script.*</script>#Usmi',$this->content,$matches)) {
103
-            foreach($matches[0] as $tag) {
102
+        if (preg_match_all('#<script.*</script>#Usmi', $this->content, $matches)) {
103
+            foreach ($matches[0] as $tag) {
104 104
                 // only consider script aggregation for types whitelisted in should_aggregate-function
105
-                if( !$this->should_aggregate($tag) ) {
106
-                    $tag='';
105
+                if (!$this->should_aggregate($tag)) {
106
+                    $tag = '';
107 107
                     continue;
108 108
                 }
109 109
 
110
-                if (preg_match('#<script[^>]*src=("|\')([^>]*)("|\')#Usmi',$tag,$source)) {
110
+                if (preg_match('#<script[^>]*src=("|\')([^>]*)("|\')#Usmi', $tag, $source)) {
111 111
                     // non-inline script
112
-                    if ($this->isremovable($tag,$this->jsremovables)) {
113
-                        $this->content = str_replace($tag,'',$this->content);
112
+                    if ($this->isremovable($tag, $this->jsremovables)) {
113
+                        $this->content = str_replace($tag, '', $this->content);
114 114
                         continue;
115 115
                     }
116
-                    $explUrl = explode('?',$source[2],2);
116
+                    $explUrl = explode('?', $source[2], 2);
117 117
                     $url = $explUrl[0];
118 118
                     $path = $this->getpath($url);
119
-                    if($path !== false && preg_match('#\.js$#',$path) && $this->ismergeable($tag)) {
119
+                    if ($path !== false && preg_match('#\.js$#', $path) && $this->ismergeable($tag)) {
120 120
                         // ok to optimize, add to array
121 121
                         $this->scripts[] = $path;
122 122
                     } else {
@@ -127,14 +127,14 @@  discard block
 block discarded – undo
127 127
                         if (is_array($excludeJS)) {
128 128
                             // should we add flags?
129 129
                             foreach ($excludeJS as $exclTag => $exclFlags) {
130
-                                if ( strpos($origTag,$exclTag)!==false && in_array($exclFlags,array("async","defer")) ) {
131
-                                   $newTag = str_replace('<script ','<script '.$exclFlags.' ',$newTag);
130
+                                if (strpos($origTag, $exclTag) !== false && in_array($exclFlags, array("async", "defer"))) {
131
+                                   $newTag = str_replace('<script ', '<script '.$exclFlags.' ', $newTag);
132 132
                                 }
133 133
                             }
134 134
                         }
135 135
                         
136 136
    						// should we minify the non-aggregated script?
137
-						if ($path && apply_filters('autoptimize_filter_js_minify_excluded',false)) {
137
+						if ($path && apply_filters('autoptimize_filter_js_minify_excluded', false)) {
138 138
 							$_CachedMinifiedUrl = $this->minify_single($path);
139 139
 
140 140
 							// replace orig URL with minified URL from cache if so
@@ -143,23 +143,23 @@  discard block
 block discarded – undo
143 143
 							}
144 144
 							
145 145
 							// remove querystring from URL in newTag
146
-							if ( !empty($explUrl[1]) ) {
147
-								$newTag = str_replace("?".$explUrl[1],"",$newTag);
146
+							if (!empty($explUrl[1])) {
147
+								$newTag = str_replace("?".$explUrl[1], "", $newTag);
148 148
 							}
149 149
 						}
150 150
 
151 151
 						// should we move the non-aggregated script?
152
-                        if( $this->ismovable($newTag) ) {
152
+                        if ($this->ismovable($newTag)) {
153 153
                             // can be moved, flags and all
154
-                            if( $this->movetolast($newTag) )    {
154
+                            if ($this->movetolast($newTag)) {
155 155
                                 $this->move['last'][] = $newTag;
156 156
                             } else {
157 157
                                 $this->move['first'][] = $newTag;
158 158
                             }
159 159
                         } else {
160 160
                             // cannot be moved, so if flag was added re-inject altered tag immediately
161
-                            if ( $origTag !== $newTag ) {
162
-                                $this->content = str_replace($origTag,$newTag,$this->content);
161
+                            if ($origTag !== $newTag) {
162
+                                $this->content = str_replace($origTag, $newTag, $this->content);
163 163
                                 $origTag = '';
164 164
                             }
165 165
                             // and forget about the $tag (not to be touched any more)
@@ -168,23 +168,23 @@  discard block
 block discarded – undo
168 168
                     }
169 169
                 } else {
170 170
                     // Inline script
171
-                    if ($this->isremovable($tag,$this->jsremovables)) {
172
-                        $this->content = str_replace($tag,'',$this->content);
171
+                    if ($this->isremovable($tag, $this->jsremovables)) {
172
+                        $this->content = str_replace($tag, '', $this->content);
173 173
                         continue;
174 174
                     }
175 175
                     
176 176
                     // unhide comments, as javascript may be wrapped in comment-tags for old times' sake
177 177
                     $tag = $this->restore_comments($tag);
178
-                    if($this->ismergeable($tag) && ( $this->include_inline )) {
179
-                        preg_match('#<script.*>(.*)</script>#Usmi',$tag,$code);
180
-                        $code = preg_replace('#.*<!\[CDATA\[(?:\s*\*/)?(.*)(?://|/\*)\s*?\]\]>.*#sm','$1',$code[1]);
181
-                        $code = preg_replace('/(?:^\\s*<!--\\s*|\\s*(?:\\/\\/)?\\s*-->\\s*$)/','',$code);
178
+                    if ($this->ismergeable($tag) && ($this->include_inline)) {
179
+                        preg_match('#<script.*>(.*)</script>#Usmi', $tag, $code);
180
+                        $code = preg_replace('#.*<!\[CDATA\[(?:\s*\*/)?(.*)(?://|/\*)\s*?\]\]>.*#sm', '$1', $code[1]);
181
+                        $code = preg_replace('/(?:^\\s*<!--\\s*|\\s*(?:\\/\\/)?\\s*-->\\s*$)/', '', $code);
182 182
                         $this->scripts[] = 'INLINE;'.$code;
183 183
                     } else {
184 184
                         // Can we move this?
185
-                        $autoptimize_js_moveable = apply_filters( 'autoptimize_js_moveable', '', $tag );
186
-                        if( $this->ismovable($tag) || $autoptimize_js_moveable !== '' ) {
187
-                            if( $this->movetolast($tag) || $autoptimize_js_moveable === 'last' ) {
185
+                        $autoptimize_js_moveable = apply_filters('autoptimize_js_moveable', '', $tag);
186
+                        if ($this->ismovable($tag) || $autoptimize_js_moveable !== '') {
187
+                            if ($this->movetolast($tag) || $autoptimize_js_moveable === 'last') {
188 188
                                 $this->move['last'][] = $tag;
189 189
                             } else {
190 190
                                 $this->move['first'][] = $tag;
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
                 }
200 200
                 
201 201
                 //Remove the original script tag
202
-                $this->content = str_replace($tag,'',$this->content);
202
+                $this->content = str_replace($tag, '', $this->content);
203 203
             }
204 204
             
205 205
             return true;
@@ -211,39 +211,39 @@  discard block
 block discarded – undo
211 211
     
212 212
     //Joins and optimizes JS
213 213
     public function minify() {
214
-        foreach($this->scripts as $script) {
215
-            if(preg_match('#^INLINE;#',$script)) {
214
+        foreach ($this->scripts as $script) {
215
+            if (preg_match('#^INLINE;#', $script)) {
216 216
                 //Inline script
217
-                $script = preg_replace('#^INLINE;#','',$script);
218
-                $script = rtrim( $script, ";\n\t\r" ) . ';';
217
+                $script = preg_replace('#^INLINE;#', '', $script);
218
+                $script = rtrim($script, ";\n\t\r").';';
219 219
                 //Add try-catch?
220
-                if($this->trycatch) {
220
+                if ($this->trycatch) {
221 221
                     $script = 'try{'.$script.'}catch(e){}';
222 222
                 }
223
-                $tmpscript = apply_filters( 'autoptimize_js_individual_script', $script, '' );
224
-                if ( has_filter('autoptimize_js_individual_script') && !empty($tmpscript) ) {
225
-                    $script=$tmpscript;
226
-                    $this->alreadyminified=true;
223
+                $tmpscript = apply_filters('autoptimize_js_individual_script', $script, '');
224
+                if (has_filter('autoptimize_js_individual_script') && !empty($tmpscript)) {
225
+                    $script = $tmpscript;
226
+                    $this->alreadyminified = true;
227 227
                 }
228
-                $this->jscode .= "\n" . $script;
228
+                $this->jscode .= "\n".$script;
229 229
             } else {
230 230
                 //External script
231
-                if($script !== false && file_exists($script) && is_readable($script)) {
231
+                if ($script !== false && file_exists($script) && is_readable($script)) {
232 232
                     $scriptsrc = file_get_contents($script);
233 233
                     $scripthash = md5($scriptsrc);
234
-                    $scriptsrc = preg_replace('/\x{EF}\x{BB}\x{BF}/','',$scriptsrc);
235
-                    $scriptsrc = rtrim($scriptsrc,";\n\t\r").';';
234
+                    $scriptsrc = preg_replace('/\x{EF}\x{BB}\x{BF}/', '', $scriptsrc);
235
+                    $scriptsrc = rtrim($scriptsrc, ";\n\t\r").';';
236 236
 
237 237
                     //Add try-catch?
238
-                    if($this->trycatch) {
238
+                    if ($this->trycatch) {
239 239
                         $scriptsrc = 'try{'.$scriptsrc.'}catch(e){}';
240 240
                     }
241
-                    $tmpscriptsrc = apply_filters( 'autoptimize_js_individual_script', $scriptsrc, $script );
242
-                    if ( has_filter('autoptimize_js_individual_script') && !empty($tmpscriptsrc) ) {
243
-                        $scriptsrc=$tmpscriptsrc;
244
-                        $this->alreadyminified=true;
241
+                    $tmpscriptsrc = apply_filters('autoptimize_js_individual_script', $scriptsrc, $script);
242
+                    if (has_filter('autoptimize_js_individual_script') && !empty($tmpscriptsrc)) {
243
+                        $scriptsrc = $tmpscriptsrc;
244
+                        $this->alreadyminified = true;
245 245
                     } else if ($this->can_inject_late($script)) {
246
-                        $scriptsrc="/*!%%INJECTLATER%%".base64_encode($script)."|".$scripthash."%%INJECTLATER%%*/";
246
+                        $scriptsrc = "/*!%%INJECTLATER%%".base64_encode($script)."|".$scripthash."%%INJECTLATER%%*/";
247 247
                     }
248 248
                     $this->jscode .= "\n".$scriptsrc;
249 249
                 }/*else{
@@ -254,24 +254,24 @@  discard block
 block discarded – undo
254 254
 
255 255
         //Check for already-minified code
256 256
         $this->md5hash = md5($this->jscode);
257
-        $ccheck = new autoptimizeCache($this->md5hash,'js');
258
-        if($ccheck->check()) {
257
+        $ccheck = new autoptimizeCache($this->md5hash, 'js');
258
+        if ($ccheck->check()) {
259 259
             $this->jscode = $ccheck->retrieve();
260 260
             return true;
261 261
         }
262 262
         unset($ccheck);
263 263
         
264 264
         //$this->jscode has all the uncompressed code now.
265
-        if ($this->alreadyminified!==true) {
266
-          if (class_exists('JSMin') && apply_filters( 'autoptimize_js_do_minify' , true)) {
267
-            if (@is_callable(array("JSMin","minify"))) {
265
+        if ($this->alreadyminified !== true) {
266
+          if (class_exists('JSMin') && apply_filters('autoptimize_js_do_minify', true)) {
267
+            if (@is_callable(array("JSMin", "minify"))) {
268 268
                 $tmp_jscode = trim(JSMin::minify($this->jscode));
269 269
                 if (!empty($tmp_jscode)) {
270 270
                     $this->jscode = $tmp_jscode;
271 271
                     unset($tmp_jscode);
272 272
                 }
273 273
                 $this->jscode = $this->inject_minified($this->jscode);
274
-                $this->jscode = apply_filters( 'autoptimize_js_after_minify', $this->jscode );
274
+                $this->jscode = apply_filters('autoptimize_js_after_minify', $this->jscode);
275 275
                 return true;
276 276
             } else {
277 277
                 $this->jscode = $this->inject_minified($this->jscode);
@@ -282,16 +282,16 @@  discard block
 block discarded – undo
282 282
               return false;
283 283
           }
284 284
         }
285
-        $this->jscode = apply_filters( 'autoptimize_js_after_minify', $this->jscode );
285
+        $this->jscode = apply_filters('autoptimize_js_after_minify', $this->jscode);
286 286
         return true;
287 287
     }
288 288
     
289 289
     //Caches the JS in uncompressed, deflated and gzipped form.
290
-    public function cache()    {
291
-        $cache = new autoptimizeCache($this->md5hash,'js');
292
-        if(!$cache->check()) {
290
+    public function cache() {
291
+        $cache = new autoptimizeCache($this->md5hash, 'js');
292
+        if (!$cache->check()) {
293 293
             //Cache our code
294
-            $cache->cache($this->jscode,'text/javascript');
294
+            $cache->cache($this->jscode, 'text/javascript');
295 295
         }
296 296
         $this->url = AUTOPTIMIZE_CACHE_URL.$cache->getname();
297 297
         $this->url = $this->url_replace_cdn($this->url);
@@ -300,32 +300,32 @@  discard block
 block discarded – undo
300 300
     // Returns the content
301 301
     public function getcontent() {
302 302
         // Restore the full content
303
-        if(!empty($this->restofcontent)) {
303
+        if (!empty($this->restofcontent)) {
304 304
             $this->content .= $this->restofcontent;
305 305
             $this->restofcontent = '';
306 306
         }
307 307
         
308 308
         // Add the scripts taking forcehead/ deferred (default) into account
309
-        if($this->forcehead == true) {
310
-            $replaceTag=array("</head>","before");
311
-            $defer="";
309
+        if ($this->forcehead == true) {
310
+            $replaceTag = array("</head>", "before");
311
+            $defer = "";
312 312
         } else {
313
-            $replaceTag=array("</body>","before");
314
-            $defer="defer ";
313
+            $replaceTag = array("</body>", "before");
314
+            $defer = "defer ";
315 315
         }
316 316
         
317
-        $defer = apply_filters( 'autoptimize_filter_js_defer', $defer );
317
+        $defer = apply_filters('autoptimize_filter_js_defer', $defer);
318 318
         $bodyreplacementpayload = '<script type="text/javascript" '.$defer.'src="'.$this->url.'"></script>';
319
-        $bodyreplacementpayload = apply_filters('autoptimize_filter_js_bodyreplacementpayload',$bodyreplacementpayload);
319
+        $bodyreplacementpayload = apply_filters('autoptimize_filter_js_bodyreplacementpayload', $bodyreplacementpayload);
320 320
 
321
-        $bodyreplacement = implode('',$this->move['first']);
321
+        $bodyreplacement = implode('', $this->move['first']);
322 322
         $bodyreplacement .= $bodyreplacementpayload;
323
-        $bodyreplacement .= implode('',$this->move['last']);
323
+        $bodyreplacement .= implode('', $this->move['last']);
324 324
 
325
-        $replaceTag = apply_filters( 'autoptimize_filter_js_replacetag', $replaceTag );
325
+        $replaceTag = apply_filters('autoptimize_filter_js_replacetag', $replaceTag);
326 326
 
327
-        if (strlen($this->jscode)>0) {
328
-            $this->inject_in_html($bodyreplacement,$replaceTag);
327
+        if (strlen($this->jscode) > 0) {
328
+            $this->inject_in_html($bodyreplacement, $replaceTag);
329 329
         }
330 330
         
331 331
         // restore comments
@@ -343,19 +343,19 @@  discard block
 block discarded – undo
343 343
     
344 344
     // Checks against the white- and blacklists
345 345
     private function ismergeable($tag) {
346
-		if (apply_filters('autoptimize_filter_js_dontaggregate',false)) {
346
+		if (apply_filters('autoptimize_filter_js_dontaggregate', false)) {
347 347
 			return false;
348 348
         } else if (!empty($this->whitelist)) {
349 349
             foreach ($this->whitelist as $match) {
350
-                if(strpos($tag,$match)!==false) {
350
+                if (strpos($tag, $match) !== false) {
351 351
                     return true;
352 352
                 }
353 353
             }
354 354
             // no match with whitelist
355 355
             return false;
356 356
         } else {
357
-            foreach($this->domove as $match) {
358
-                if(strpos($tag,$match)!==false)    {
357
+            foreach ($this->domove as $match) {
358
+                if (strpos($tag, $match) !== false) {
359 359
                     // Matched something
360 360
                     return false;
361 361
                 }
@@ -365,8 +365,8 @@  discard block
 block discarded – undo
365 365
                 return false;
366 366
             }
367 367
             
368
-            foreach($this->dontmove as $match) {
369
-                if(strpos($tag,$match)!==false)    {
368
+            foreach ($this->dontmove as $match) {
369
+                if (strpos($tag, $match) !== false) {
370 370
                     //Matched something
371 371
                     return false;
372 372
                 }
@@ -379,12 +379,12 @@  discard block
 block discarded – undo
379 379
     
380 380
     // Checks againstt the blacklist
381 381
     private function ismovable($tag) {
382
-        if ($this->include_inline !== true || apply_filters('autoptimize_filter_js_unmovable',true)) {
382
+        if ($this->include_inline !== true || apply_filters('autoptimize_filter_js_unmovable', true)) {
383 383
             return false;
384 384
         }
385 385
         
386
-        foreach($this->domove as $match) {
387
-            if(strpos($tag,$match)!==false)    {
386
+        foreach ($this->domove as $match) {
387
+            if (strpos($tag, $match) !== false) {
388 388
                 // Matched something
389 389
                 return true;
390 390
             }
@@ -394,8 +394,8 @@  discard block
 block discarded – undo
394 394
             return true;
395 395
         }
396 396
         
397
-        foreach($this->dontmove as $match) {
398
-            if(strpos($tag,$match)!==false) {
397
+        foreach ($this->dontmove as $match) {
398
+            if (strpos($tag, $match) !== false) {
399 399
                 // Matched something
400 400
                 return false;
401 401
             }
@@ -406,8 +406,8 @@  discard block
 block discarded – undo
406 406
     }
407 407
     
408 408
     private function movetolast($tag) {
409
-        foreach($this->domovelast as $match) {
410
-            if(strpos($tag,$match)!==false)    {
409
+        foreach ($this->domovelast as $match) {
410
+            if (strpos($tag, $match) !== false) {
411 411
                 // Matched, return true
412 412
                 return true;
413 413
             }
@@ -433,10 +433,10 @@  discard block
 block discarded – undo
433 433
      * original function by https://github.com/zytzagoo/ on his AO fork, thanks Tomas!
434 434
      */
435 435
     public function should_aggregate($tag) {
436
-        preg_match('#<(script[^>]*)>#i',$tag,$scripttag);
437
-        if ( strpos($scripttag[1], 'type')===false ) {
436
+        preg_match('#<(script[^>]*)>#i', $tag, $scripttag);
437
+        if (strpos($scripttag[1], 'type') === false) {
438 438
             return true;
439
-        } else if ( preg_match('/type\s*=\s*["\']?(?:text|application)\/(?:javascript|ecmascript)["\']?/i', $scripttag[1]) ) {
439
+        } else if (preg_match('/type\s*=\s*["\']?(?:text|application)\/(?:javascript|ecmascript)["\']?/i', $scripttag[1])) {
440 440
             return true;
441 441
         } else {
442 442
             return false;
@@ -454,11 +454,11 @@  discard block
 block discarded – undo
454 454
      * @return bool
455 455
 	 */
456 456
 	private function can_inject_late($jsPath) {
457
-		$consider_minified_array = apply_filters('autoptimize_filter_js_consider_minified',false);
458
-        if ( $this->inject_min_late !== true ) {
457
+		$consider_minified_array = apply_filters('autoptimize_filter_js_consider_minified', false);
458
+        if ($this->inject_min_late !== true) {
459 459
             // late-inject turned off
460 460
             return false;
461
-        } else if ( (strpos($jsPath,"min.js") === false) && ( strpos($jsPath,"wp-includes/js/jquery/jquery.js") === false ) && ( str_replace($consider_minified_array, '', $jsPath) === $jsPath ) ) {
461
+        } else if ((strpos($jsPath, "min.js") === false) && (strpos($jsPath, "wp-includes/js/jquery/jquery.js") === false) && (str_replace($consider_minified_array, '', $jsPath) === $jsPath)) {
462 462
 			// file not minified based on filename & filter
463 463
 			return false;
464 464
         } else {
Please login to merge, or discard this patch.
classes/autoptimizeStyles.php 1 patch
Spacing   +187 added lines, -187 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
2
+if (!defined('ABSPATH')) exit; // Exit if accessed directly
3 3
 
4 4
 class autoptimizeStyles extends autoptimizeBase {
5 5
 
@@ -23,12 +23,12 @@  discard block
 block discarded – undo
23 23
 
24 24
     //Reads the page and collects style tags
25 25
     public function read($options) {
26
-        $noptimizeCSS = apply_filters( 'autoptimize_filter_css_noptimize', false, $this->content );
26
+        $noptimizeCSS = apply_filters('autoptimize_filter_css_noptimize', false, $this->content);
27 27
         if ($noptimizeCSS) return false;
28 28
 
29
-        $whitelistCSS = apply_filters( 'autoptimize_filter_css_whitelist', '', $this->content );
29
+        $whitelistCSS = apply_filters('autoptimize_filter_css_whitelist', '', $this->content);
30 30
         if (!empty($whitelistCSS)) {
31
-            $this->whitelist = array_filter(array_map('trim',explode(",",$whitelistCSS)));
31
+            $this->whitelist = array_filter(array_map('trim', explode(",", $whitelistCSS)));
32 32
         }
33 33
         
34 34
         if ($options['nogooglefont'] == true) {
@@ -36,33 +36,33 @@  discard block
 block discarded – undo
36 36
         } else {
37 37
             $removableCSS = "";
38 38
         }
39
-        $removableCSS = apply_filters( 'autoptimize_filter_css_removables', $removableCSS);
39
+        $removableCSS = apply_filters('autoptimize_filter_css_removables', $removableCSS);
40 40
         if (!empty($removableCSS)) {
41
-            $this->cssremovables = array_filter(array_map('trim',explode(",",$removableCSS)));
41
+            $this->cssremovables = array_filter(array_map('trim', explode(",", $removableCSS)));
42 42
         }
43 43
 
44
-        $this->cssinlinesize = apply_filters('autoptimize_filter_css_inlinesize',256);
44
+        $this->cssinlinesize = apply_filters('autoptimize_filter_css_inlinesize', 256);
45 45
 
46 46
         // filter to "late inject minified CSS", default to true for now (it is faster)
47
-        $this->inject_min_late = apply_filters('autoptimize_filter_css_inject_min_late',true);
47
+        $this->inject_min_late = apply_filters('autoptimize_filter_css_inject_min_late', true);
48 48
 
49 49
         // Remove everything that's not the header
50
-        if ( apply_filters('autoptimize_filter_css_justhead',$options['justhead']) == true ) {
51
-            $content = explode('</head>',$this->content,2);
50
+        if (apply_filters('autoptimize_filter_css_justhead', $options['justhead']) == true) {
51
+            $content = explode('</head>', $this->content, 2);
52 52
             $this->content = $content[0].'</head>';
53 53
             $this->restofcontent = $content[1];
54 54
         }
55 55
 
56 56
         // include inline?
57
-        if( apply_filters('autoptimize_css_include_inline',$options['include_inline']) == true ) {
57
+        if (apply_filters('autoptimize_css_include_inline', $options['include_inline']) == true) {
58 58
             $this->include_inline = true;
59 59
         }
60 60
         
61 61
         // what CSS shouldn't be autoptimized
62 62
         $excludeCSS = $options['css_exclude'];
63
-        $excludeCSS = apply_filters( 'autoptimize_filter_css_exclude', $excludeCSS, $this->content );
64
-        if ($excludeCSS!=="") {
65
-            $this->dontmove = array_filter(array_map('trim',explode(",",$excludeCSS)));
63
+        $excludeCSS = apply_filters('autoptimize_filter_css_exclude', $excludeCSS, $this->content);
64
+        if ($excludeCSS !== "") {
65
+            $this->dontmove = array_filter(array_map('trim', explode(",", $excludeCSS)));
66 66
         } else {
67 67
             $this->dontmove = "";
68 68
         }
@@ -70,17 +70,17 @@  discard block
 block discarded – undo
70 70
         // should we defer css?
71 71
         // value: true/ false
72 72
         $this->defer = $options['defer'];
73
-        $this->defer = apply_filters( 'autoptimize_filter_css_defer', $this->defer, $this->content );
73
+        $this->defer = apply_filters('autoptimize_filter_css_defer', $this->defer, $this->content);
74 74
 
75 75
         // should we inline while deferring?
76 76
         // value: inlined CSS
77 77
         $this->defer_inline = $options['defer_inline'];
78
-        $this->defer_inline = apply_filters( 'autoptimize_filter_css_defer_inline', $this->defer_inline, $this->content );
78
+        $this->defer_inline = apply_filters('autoptimize_filter_css_defer_inline', $this->defer_inline, $this->content);
79 79
 
80 80
         // should we inline?
81 81
         // value: true/ false
82 82
         $this->inline = $options['inline'];
83
-        $this->inline = apply_filters( 'autoptimize_filter_css_inline', $this->inline, $this->content );
83
+        $this->inline = apply_filters('autoptimize_filter_css_inline', $this->inline, $this->content);
84 84
         
85 85
         // get cdn url
86 86
         $this->cdn_url = $options['cdn_url'];
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
         $this->content = $this->hide_noptimize($this->content);
93 93
         
94 94
         // exclude (no)script, as those may contain CSS which should be left as is
95
-        if ( strpos( $this->content, '<script' ) !== false ) { 
95
+        if (strpos($this->content, '<script') !== false) { 
96 96
             $this->content = preg_replace_callback(
97 97
                 '#<(?:no)?script.*?<\/(?:no)?script>#is',
98 98
                 create_function(
@@ -110,65 +110,65 @@  discard block
 block discarded – undo
110 110
         $this->content = $this->hide_comments($this->content);
111 111
         
112 112
         // Get <style> and <link>
113
-        if(preg_match_all('#(<style[^>]*>.*</style>)|(<link[^>]*stylesheet[^>]*>)#Usmi',$this->content,$matches)) {
114
-            foreach($matches[0] as $tag) {
115
-                if ($this->isremovable($tag,$this->cssremovables)) {
116
-                    $this->content = str_replace($tag,'',$this->content);
113
+        if (preg_match_all('#(<style[^>]*>.*</style>)|(<link[^>]*stylesheet[^>]*>)#Usmi', $this->content, $matches)) {
114
+            foreach ($matches[0] as $tag) {
115
+                if ($this->isremovable($tag, $this->cssremovables)) {
116
+                    $this->content = str_replace($tag, '', $this->content);
117 117
                 } else if ($this->ismovable($tag)) {
118 118
                     // Get the media
119
-                    if(strpos($tag,'media=')!==false) {
120
-                        preg_match('#media=(?:"|\')([^>]*)(?:"|\')#Ui',$tag,$medias);
121
-                        $medias = explode(',',$medias[1]);
119
+                    if (strpos($tag, 'media=') !== false) {
120
+                        preg_match('#media=(?:"|\')([^>]*)(?:"|\')#Ui', $tag, $medias);
121
+                        $medias = explode(',', $medias[1]);
122 122
                         $media = array();
123
-                        foreach($medias as $elem) {
124
-                            if (empty($elem)) { $elem="all"; }
123
+                        foreach ($medias as $elem) {
124
+                            if (empty($elem)) { $elem = "all"; }
125 125
                             $media[] = $elem;
126 126
                         }
127 127
                     } else {
128 128
                         // No media specified - applies to all
129 129
                         $media = array('all');
130 130
                     }
131
-                    $media = apply_filters( 'autoptimize_filter_css_tagmedia',$media,$tag );
131
+                    $media = apply_filters('autoptimize_filter_css_tagmedia', $media, $tag);
132 132
                 
133
-                    if(preg_match('#<link.*href=("|\')(.*)("|\')#Usmi',$tag,$source)) {
133
+                    if (preg_match('#<link.*href=("|\')(.*)("|\')#Usmi', $tag, $source)) {
134 134
                         // <link>
135
-                        $explUrl = explode('?',$source[2],2);
135
+                        $explUrl = explode('?', $source[2], 2);
136 136
                         $url = $explUrl[0];
137 137
                         $path = $this->getpath($url);
138 138
                         
139
-                        if($path!==false && preg_match('#\.css$#',$path)) {
139
+                        if ($path !== false && preg_match('#\.css$#', $path)) {
140 140
                             // Good link
141
-                            $this->css[] = array($media,$path);
142
-                        }else{
141
+                            $this->css[] = array($media, $path);
142
+                        } else {
143 143
                             // Link is dynamic (.php etc)
144 144
                             $tag = '';
145 145
                         }
146 146
                     } else {
147 147
                         // inline css in style tags can be wrapped in comment tags, so restore comments
148 148
                         $tag = $this->restore_comments($tag);
149
-                        preg_match('#<style.*>(.*)</style>#Usmi',$tag,$code);
149
+                        preg_match('#<style.*>(.*)</style>#Usmi', $tag, $code);
150 150
 
151 151
                         // and re-hide them to be able to to the removal based on tag
152 152
                         $tag = $this->hide_comments($tag);
153 153
 
154
-                        if ( $this->include_inline ) {
155
-                            $code = preg_replace('#^.*<!\[CDATA\[(?:\s*\*/)?(.*)(?://|/\*)\s*?\]\]>.*$#sm','$1',$code[1]);
156
-                            $this->css[] = array($media,'INLINE;'.$code);
154
+                        if ($this->include_inline) {
155
+                            $code = preg_replace('#^.*<!\[CDATA\[(?:\s*\*/)?(.*)(?://|/\*)\s*?\]\]>.*$#sm', '$1', $code[1]);
156
+                            $this->css[] = array($media, 'INLINE;'.$code);
157 157
                         } else {
158 158
                             $tag = '';
159 159
                         }
160 160
                     }
161 161
                     
162 162
                     // Remove the original style tag
163
-                    $this->content = str_replace($tag,'',$this->content);
163
+                    $this->content = str_replace($tag, '', $this->content);
164 164
                 } else {
165 165
 					// excluded CSS, minify if getpath 
166
-					if (preg_match('#<link.*href=("|\')(.*)("|\')#Usmi',$tag,$source)) {
167
-						$explUrl = explode('?',$source[2],2);
166
+					if (preg_match('#<link.*href=("|\')(.*)("|\')#Usmi', $tag, $source)) {
167
+						$explUrl = explode('?', $source[2], 2);
168 168
                         $url = $explUrl[0];
169 169
                         $path = $this->getpath($url);
170 170
  					
171
-						if ($path && apply_filters('autoptimize_filter_css_minify_excluded',false)) {
171
+						if ($path && apply_filters('autoptimize_filter_css_minify_excluded', false)) {
172 172
 							$_CachedMinifiedUrl = $this->minify_single($path);
173 173
 
174 174
 							if (!empty($_CachedMinifiedUrl)) {
@@ -179,12 +179,12 @@  discard block
 block discarded – undo
179 179
 							}
180 180
 							
181 181
 							// remove querystring from URL
182
-							if ( !empty($explUrl[1]) ) {
183
-								$newTag = str_replace("?".$explUrl[1],"",$newTag);
182
+							if (!empty($explUrl[1])) {
183
+								$newTag = str_replace("?".$explUrl[1], "", $newTag);
184 184
 							}
185 185
 
186 186
 							// and replace
187
-							$this->content = str_replace($tag,$newTag,$this->content);
187
+							$this->content = str_replace($tag, $newTag, $this->content);
188 188
 						}
189 189
 					}					
190 190
 				}
@@ -197,31 +197,31 @@  discard block
 block discarded – undo
197 197
     
198 198
     // Joins and optimizes CSS
199 199
     public function minify() {
200
-        foreach($this->css as $group) {
201
-            list($media,$css) = $group;
202
-            if(preg_match('#^INLINE;#',$css)) {
200
+        foreach ($this->css as $group) {
201
+            list($media, $css) = $group;
202
+            if (preg_match('#^INLINE;#', $css)) {
203 203
                 // <style>
204
-                $css = preg_replace('#^INLINE;#','',$css);
205
-                $css = $this->fixurls(ABSPATH.'/index.php',$css);
206
-                $tmpstyle = apply_filters( 'autoptimize_css_individual_style', $css, "" );
207
-                if ( has_filter('autoptimize_css_individual_style') && !empty($tmpstyle) ) {
208
-                    $css=$tmpstyle;
209
-                    $this->alreadyminified=true;
204
+                $css = preg_replace('#^INLINE;#', '', $css);
205
+                $css = $this->fixurls(ABSPATH.'/index.php', $css);
206
+                $tmpstyle = apply_filters('autoptimize_css_individual_style', $css, "");
207
+                if (has_filter('autoptimize_css_individual_style') && !empty($tmpstyle)) {
208
+                    $css = $tmpstyle;
209
+                    $this->alreadyminified = true;
210 210
                 }
211 211
             } else {
212 212
                 //<link>
213
-                if($css !== false && file_exists($css) && is_readable($css)) {
213
+                if ($css !== false && file_exists($css) && is_readable($css)) {
214 214
                     $cssPath = $css;
215 215
                     $cssContents = file_get_contents($cssPath);
216 216
                     $cssHash = md5($cssContents);
217
-                    $css = $this->fixurls($cssPath,$cssContents);
218
-                    $css = preg_replace('/\x{EF}\x{BB}\x{BF}/','',$css);
219
-                    $tmpstyle = apply_filters( 'autoptimize_css_individual_style', $css, $cssPath );
217
+                    $css = $this->fixurls($cssPath, $cssContents);
218
+                    $css = preg_replace('/\x{EF}\x{BB}\x{BF}/', '', $css);
219
+                    $tmpstyle = apply_filters('autoptimize_css_individual_style', $css, $cssPath);
220 220
                     if (has_filter('autoptimize_css_individual_style') && !empty($tmpstyle)) {
221
-                        $css=$tmpstyle;
222
-                        $this->alreadyminified=true;
223
-                    } else if ($this->can_inject_late($cssPath,$css)) {
224
-                        $css="/*!%%INJECTLATER%%".base64_encode($cssPath)."|".$cssHash."%%INJECTLATER%%*/";
221
+                        $css = $tmpstyle;
222
+                        $this->alreadyminified = true;
223
+                    } else if ($this->can_inject_late($cssPath, $css)) {
224
+                        $css = "/*!%%INJECTLATER%%".base64_encode($cssPath)."|".$cssHash."%%INJECTLATER%%*/";
225 225
                     }
226 226
                 } else {
227 227
                     // Couldn't read CSS. Maybe getpath isn't working?
@@ -229,8 +229,8 @@  discard block
 block discarded – undo
229 229
                 }
230 230
             }
231 231
             
232
-            foreach($media as $elem) {
233
-                if(!isset($this->csscode[$elem]))
232
+            foreach ($media as $elem) {
233
+                if (!isset($this->csscode[$elem]))
234 234
                     $this->csscode[$elem] = '';
235 235
                 $this->csscode[$elem] .= "\n/*FILESTART*/".$css;
236 236
             }
@@ -239,12 +239,12 @@  discard block
 block discarded – undo
239 239
         // Check for duplicate code
240 240
         $md5list = array();
241 241
         $tmpcss = $this->csscode;
242
-        foreach($tmpcss as $media => $code) {
242
+        foreach ($tmpcss as $media => $code) {
243 243
             $md5sum = md5($code);
244 244
             $medianame = $media;
245
-            foreach($md5list as $med => $sum) {
245
+            foreach ($md5list as $med => $sum) {
246 246
                 // If same code
247
-                if($sum === $md5sum) {
247
+                if ($sum === $md5sum) {
248 248
                     //Add the merged code
249 249
                     $medianame = $med.', '.$media;
250 250
                     $this->csscode[$medianame] = $code;
@@ -264,30 +264,30 @@  discard block
 block discarded – undo
264 264
             $external_imports = "";
265 265
 
266 266
             // remove comments to avoid importing commented-out imports
267
-            $thiscss_nocomments = preg_replace('#/\*.*\*/#Us','',$thiscss);
267
+            $thiscss_nocomments = preg_replace('#/\*.*\*/#Us', '', $thiscss);
268 268
 
269
-            while(preg_match_all('#@import.*(?:;|$)#Um',$thiscss_nocomments,$matches)) {
270
-                foreach($matches[0] as $import)    {
271
-                    if ($this->isremovable($import,$this->cssremovables)) {
272
-                        $thiscss = str_replace($import,'',$thiscss);
269
+            while (preg_match_all('#@import.*(?:;|$)#Um', $thiscss_nocomments, $matches)) {
270
+                foreach ($matches[0] as $import) {
271
+                    if ($this->isremovable($import, $this->cssremovables)) {
272
+                        $thiscss = str_replace($import, '', $thiscss);
273 273
                         $import_ok = true;
274 274
                     } else {
275
-                        $url = trim(preg_replace('#^.*((?:https?:|ftp:)?//.*\.css).*$#','$1',trim($import))," \t\n\r\0\x0B\"'");
275
+                        $url = trim(preg_replace('#^.*((?:https?:|ftp:)?//.*\.css).*$#', '$1', trim($import)), " \t\n\r\0\x0B\"'");
276 276
                         $path = $this->getpath($url);
277 277
                         $import_ok = false;
278 278
                         if (file_exists($path) && is_readable($path)) {
279
-                            $code = addcslashes($this->fixurls($path,file_get_contents($path)),"\\");
280
-                            $code = preg_replace('/\x{EF}\x{BB}\x{BF}/','',$code);
281
-                            $tmpstyle = apply_filters( 'autoptimize_css_individual_style', $code, "" );
282
-                            if ( has_filter('autoptimize_css_individual_style') && !empty($tmpstyle)) {
283
-                                $code=$tmpstyle;
284
-                                $this->alreadyminified=true;
285
-                            } else if ($this->can_inject_late($path,$code)) {
286
-                                $code="/*!%%INJECTLATER%%".base64_encode($path)."|".md5($code)."%%INJECTLATER%%*/";
279
+                            $code = addcslashes($this->fixurls($path, file_get_contents($path)), "\\");
280
+                            $code = preg_replace('/\x{EF}\x{BB}\x{BF}/', '', $code);
281
+                            $tmpstyle = apply_filters('autoptimize_css_individual_style', $code, "");
282
+                            if (has_filter('autoptimize_css_individual_style') && !empty($tmpstyle)) {
283
+                                $code = $tmpstyle;
284
+                                $this->alreadyminified = true;
285
+                            } else if ($this->can_inject_late($path, $code)) {
286
+                                $code = "/*!%%INJECTLATER%%".base64_encode($path)."|".md5($code)."%%INJECTLATER%%*/";
287 287
                             }
288 288
                             
289
-                            if(!empty($code)) {
290
-                                $tmp_thiscss = preg_replace('#(/\*FILESTART\*/.*)'.preg_quote($import,'#').'#Us','/*FILESTART2*/'.$code.'$1',$thiscss);
289
+                            if (!empty($code)) {
290
+                                $tmp_thiscss = preg_replace('#(/\*FILESTART\*/.*)'.preg_quote($import, '#').'#Us', '/*FILESTART2*/'.$code.'$1', $thiscss);
291 291
                                 if (!empty($tmp_thiscss)) {
292 292
                                     $thiscss = $tmp_thiscss;
293 293
                                     $import_ok = true;
@@ -301,31 +301,31 @@  discard block
 block discarded – undo
301 301
                     if (!$import_ok) {
302 302
                         // external imports and general fall-back
303 303
                         $external_imports .= $import;
304
-                        $thiscss = str_replace($import,'',$thiscss);
304
+                        $thiscss = str_replace($import, '', $thiscss);
305 305
                         $fiximports = true;
306 306
                     }
307 307
                 }
308
-                $thiscss = preg_replace('#/\*FILESTART\*/#','',$thiscss);
309
-                $thiscss = preg_replace('#/\*FILESTART2\*/#','/*FILESTART*/',$thiscss);
308
+                $thiscss = preg_replace('#/\*FILESTART\*/#', '', $thiscss);
309
+                $thiscss = preg_replace('#/\*FILESTART2\*/#', '/*FILESTART*/', $thiscss);
310 310
                 
311 311
                 // and update $thiscss_nocomments before going into next iteration in while loop
312
-                $thiscss_nocomments=preg_replace('#/\*.*\*/#Us','',$thiscss);
312
+                $thiscss_nocomments = preg_replace('#/\*.*\*/#Us', '', $thiscss);
313 313
             }
314 314
             unset($thiscss_nocomments);
315 315
             
316 316
             // add external imports to top of aggregated CSS
317
-            if($fiximports) {
318
-                $thiscss=$external_imports.$thiscss;
317
+            if ($fiximports) {
318
+                $thiscss = $external_imports.$thiscss;
319 319
             }
320 320
         }
321 321
         unset($thiscss);
322 322
         
323 323
         // $this->csscode has all the uncompressed code now. 
324
-        foreach($this->csscode as &$code) {
324
+        foreach ($this->csscode as &$code) {
325 325
             // Check for already-minified code
326 326
             $hash = md5($code);
327
-            $ccheck = new autoptimizeCache($hash,'css');
328
-            if($ccheck->check()) {
327
+            $ccheck = new autoptimizeCache($hash, 'css');
328
+            if ($ccheck->check()) {
329 329
                 $code = $ccheck->retrieve();
330 330
                 $this->hashmap[md5($code)] = $hash;
331 331
                 continue;
@@ -334,44 +334,44 @@  discard block
 block discarded – undo
334 334
 
335 335
             // Do the imaging!
336 336
             $imgreplace = array();
337
-            preg_match_all( self::ASSETS_REGEX, $code, $matches );
337
+            preg_match_all(self::ASSETS_REGEX, $code, $matches);
338 338
 
339
-            if ( ($this->datauris == true) && (function_exists('base64_encode')) && (is_array($matches)) ) {
340
-                foreach($matches[1] as $count => $quotedurl) {
341
-                    $iurl = trim($quotedurl," \t\n\r\0\x0B\"'");
339
+            if (($this->datauris == true) && (function_exists('base64_encode')) && (is_array($matches))) {
340
+                foreach ($matches[1] as $count => $quotedurl) {
341
+                    $iurl = trim($quotedurl, " \t\n\r\0\x0B\"'");
342 342
 
343 343
                     // if querystring, remove it from url
344
-                    if (strpos($iurl,'?') !== false) { $iurl = strtok($iurl,'?'); }
344
+                    if (strpos($iurl, '?') !== false) { $iurl = strtok($iurl, '?'); }
345 345
                     
346 346
                     $ipath = $this->getpath($iurl);
347 347
 
348 348
                     $datauri_max_size = 4096;
349
-                    $datauri_max_size = (int) apply_filters( 'autoptimize_filter_css_datauri_maxsize', $datauri_max_size );
350
-                    $datauri_exclude = apply_filters( 'autoptimize_filter_css_datauri_exclude', "");
349
+                    $datauri_max_size = (int) apply_filters('autoptimize_filter_css_datauri_maxsize', $datauri_max_size);
350
+                    $datauri_exclude = apply_filters('autoptimize_filter_css_datauri_exclude', "");
351 351
                     if (!empty($datauri_exclude)) {
352
-                        $no_datauris=array_filter(array_map('trim',explode(",",$datauri_exclude)));
352
+                        $no_datauris = array_filter(array_map('trim', explode(",", $datauri_exclude)));
353 353
                         foreach ($no_datauris as $no_datauri) {
354
-                            if (strpos($iurl,$no_datauri)!==false) {
355
-                                $ipath=false;
354
+                            if (strpos($iurl, $no_datauri) !== false) {
355
+                                $ipath = false;
356 356
                                 break;
357 357
                             }
358 358
                         }
359 359
                     }
360 360
 
361
-                    if($ipath != false && preg_match('#\.(jpe?g|png|gif|bmp)$#i',$ipath) && file_exists($ipath) && is_readable($ipath) && filesize($ipath) <= $datauri_max_size) {
362
-                        $ihash=md5($ipath);
363
-                        $icheck = new autoptimizeCache($ihash,'img');
364
-                        if($icheck->check()) {
361
+                    if ($ipath != false && preg_match('#\.(jpe?g|png|gif|bmp)$#i', $ipath) && file_exists($ipath) && is_readable($ipath) && filesize($ipath) <= $datauri_max_size) {
362
+                        $ihash = md5($ipath);
363
+                        $icheck = new autoptimizeCache($ihash, 'img');
364
+                        if ($icheck->check()) {
365 365
                             // we have the base64 image in cache
366
-                            $headAndData=$icheck->retrieve();
367
-                            $_base64data=explode(";base64,",$headAndData);
368
-                            $base64data=$_base64data[1];
366
+                            $headAndData = $icheck->retrieve();
367
+                            $_base64data = explode(";base64,", $headAndData);
368
+                            $base64data = $_base64data[1];
369 369
                         } else {
370 370
                             // It's an image and we don't have it in cache, get the type
371
-                            $explA=explode('.',$ipath);
372
-                            $type=end($explA);
371
+                            $explA = explode('.', $ipath);
372
+                            $type = end($explA);
373 373
 
374
-                            switch($type) {
374
+                            switch ($type) {
375 375
                                 case 'jpeg':
376 376
                                     $dataurihead = 'data:image/jpeg;base64,';
377 377
                                     break;
@@ -393,42 +393,42 @@  discard block
 block discarded – undo
393 393
                         
394 394
                             // Encode the data
395 395
                             $base64data = base64_encode(file_get_contents($ipath));
396
-                            $headAndData=$dataurihead.$base64data;
396
+                            $headAndData = $dataurihead.$base64data;
397 397
 
398 398
                             // Save in cache
399
-                            $icheck->cache($headAndData,"text/plain");
399
+                            $icheck->cache($headAndData, "text/plain");
400 400
                         }
401 401
                         unset($icheck);
402 402
 
403 403
                         // Add it to the list for replacement
404
-                        $imgreplace[$matches[0][$count]] = str_replace($quotedurl,$headAndData,$matches[0][$count]);
404
+                        $imgreplace[$matches[0][$count]] = str_replace($quotedurl, $headAndData, $matches[0][$count]);
405 405
                     } else {
406 406
                         // just cdn the URL if applicable
407 407
                         if (!empty($this->cdn_url)) {
408
-                            $imgreplace[$matches[0][$count]] = str_replace($quotedurl,$this->maybe_cdn_urls($quotedurl),$matches[0][$count]);
408
+                            $imgreplace[$matches[0][$count]] = str_replace($quotedurl, $this->maybe_cdn_urls($quotedurl), $matches[0][$count]);
409 409
 						}
410 410
                     }
411 411
                 }
412 412
             } else if ((is_array($matches)) && (!empty($this->cdn_url))) {
413 413
                 // change urls to cdn-url
414
-                foreach($matches[1] as $count => $quotedurl) {
415
-                    $imgreplace[$matches[0][$count]] = str_replace($quotedurl,$this->maybe_cdn_urls($quotedurl),$matches[0][$count]);
414
+                foreach ($matches[1] as $count => $quotedurl) {
415
+                    $imgreplace[$matches[0][$count]] = str_replace($quotedurl, $this->maybe_cdn_urls($quotedurl), $matches[0][$count]);
416 416
                 }
417 417
             }
418 418
             
419
-            if(!empty($imgreplace)) {
420
-                $code = str_replace(array_keys($imgreplace),array_values($imgreplace),$code);
419
+            if (!empty($imgreplace)) {
420
+                $code = str_replace(array_keys($imgreplace), array_values($imgreplace), $code);
421 421
             }
422 422
             
423 423
             // Minify
424
-            if (($this->alreadyminified!==true) && (apply_filters( "autoptimize_css_do_minify", true))) {
424
+            if (($this->alreadyminified !== true) && (apply_filters("autoptimize_css_do_minify", true))) {
425 425
                 if (class_exists('Minify_CSS_Compressor')) {
426 426
                     $tmp_code = trim(Minify_CSS_Compressor::process($code));
427
-                } else if(class_exists('CSSmin')) {
427
+                } else if (class_exists('CSSmin')) {
428 428
                     $cssmin = new CSSmin();
429
-                    if (method_exists($cssmin,"run")) {
429
+                    if (method_exists($cssmin, "run")) {
430 430
                         $tmp_code = trim($cssmin->run($code));
431
-                    } elseif (@is_callable(array($cssmin,"minify"))) {
431
+                    } elseif (@is_callable(array($cssmin, "minify"))) {
432 432
                         $tmp_code = trim(CssMin::minify($code));
433 433
                     }
434 434
                 }
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
             
441 441
             $code = $this->inject_minified($code);
442 442
             
443
-            $tmp_code = apply_filters( 'autoptimize_css_after_minify', $code );
443
+            $tmp_code = apply_filters('autoptimize_css_after_minify', $code);
444 444
             if (!empty($tmp_code)) {
445 445
                 $code = $tmp_code;
446 446
                 unset($tmp_code);
@@ -455,13 +455,13 @@  discard block
 block discarded – undo
455 455
     //Caches the CSS in uncompressed, deflated and gzipped form.
456 456
     public function cache() {
457 457
         // CSS cache
458
-        foreach($this->csscode as $media => $code) {
458
+        foreach ($this->csscode as $media => $code) {
459 459
             $md5 = $this->hashmap[md5($code)];
460 460
                 
461
-            $cache = new autoptimizeCache($md5,'css');
462
-            if(!$cache->check()) {
461
+            $cache = new autoptimizeCache($md5, 'css');
462
+            if (!$cache->check()) {
463 463
                 // Cache our code
464
-                $cache->cache($code,'text/css');
464
+                $cache->cache($code, 'text/css');
465 465
             }
466 466
             $this->url[$media] = AUTOPTIMIZE_CACHE_URL.$cache->getname();
467 467
         }
@@ -476,7 +476,7 @@  discard block
 block discarded – undo
476 476
         $this->content = $this->restore_comments($this->content);
477 477
         
478 478
         // restore (no)script
479
-        if ( strpos( $this->content, '%%SCRIPT%%' ) !== false ) { 
479
+        if (strpos($this->content, '%%SCRIPT%%') !== false) { 
480 480
             $this->content = preg_replace_callback(
481 481
                 '#%%SCRIPT%%(.*?)%%SCRIPT%%#is',
482 482
                 create_function(
@@ -491,74 +491,74 @@  discard block
 block discarded – undo
491 491
         $this->content = $this->restore_noptimize($this->content);
492 492
         
493 493
         //Restore the full content
494
-        if(!empty($this->restofcontent)) {
494
+        if (!empty($this->restofcontent)) {
495 495
             $this->content .= $this->restofcontent;
496 496
             $this->restofcontent = '';
497 497
         }
498 498
         
499 499
         // Inject the new stylesheets
500
-        $replaceTag = array("<title","before");
501
-        $replaceTag = apply_filters( 'autoptimize_filter_css_replacetag', $replaceTag, $this->content );
500
+        $replaceTag = array("<title", "before");
501
+        $replaceTag = apply_filters('autoptimize_filter_css_replacetag', $replaceTag, $this->content);
502 502
 
503 503
         if ($this->inline == true) {
504
-            foreach($this->csscode as $media => $code) {
505
-                $this->inject_in_html('<style type="text/css" media="'.$media.'">'.$code.'</style>',$replaceTag);
504
+            foreach ($this->csscode as $media => $code) {
505
+                $this->inject_in_html('<style type="text/css" media="'.$media.'">'.$code.'</style>', $replaceTag);
506 506
             }
507 507
         } else {
508 508
             if ($this->defer == true) {
509 509
                 $preloadCssBlock = "";
510 510
                 $noScriptCssBlock = "<noscript id=\"aonoscrcss\">";
511
-                $defer_inline_code=$this->defer_inline;
512
-                if(!empty($defer_inline_code)){
513
-                    if ( apply_filters( 'autoptimize_filter_css_critcss_minify', true ) ) {
511
+                $defer_inline_code = $this->defer_inline;
512
+                if (!empty($defer_inline_code)) {
513
+                    if (apply_filters('autoptimize_filter_css_critcss_minify', true)) {
514 514
                         $iCssHash = md5($defer_inline_code);
515
-                        $iCssCache = new autoptimizeCache($iCssHash,'css');
516
-                        if($iCssCache->check()) { 
515
+                        $iCssCache = new autoptimizeCache($iCssHash, 'css');
516
+                        if ($iCssCache->check()) { 
517 517
                             // we have the optimized inline CSS in cache
518
-                            $defer_inline_code=$iCssCache->retrieve();
518
+                            $defer_inline_code = $iCssCache->retrieve();
519 519
                         } else {
520 520
                             if (class_exists('Minify_CSS_Compressor')) {
521 521
                                 $tmp_code = trim(Minify_CSS_Compressor::process($defer_inline_code));
522
-                            } else if(class_exists('CSSmin')) {
522
+                            } else if (class_exists('CSSmin')) {
523 523
                                 $cssmin = new CSSmin();
524 524
                                 $tmp_code = trim($cssmin->run($defer_inline_code));
525 525
                             }
526 526
                             if (!empty($tmp_code)) {
527 527
                                 $defer_inline_code = $tmp_code;
528
-                                $iCssCache->cache($defer_inline_code,"text/css");
528
+                                $iCssCache->cache($defer_inline_code, "text/css");
529 529
                                 unset($tmp_code);
530 530
                             }
531 531
                         }
532 532
                     }
533
-                    $code_out='<style type="text/css" id="aoatfcss" media="all">'.$defer_inline_code.'</style>';
534
-                    $this->inject_in_html($code_out,$replaceTag);
533
+                    $code_out = '<style type="text/css" id="aoatfcss" media="all">'.$defer_inline_code.'</style>';
534
+                    $this->inject_in_html($code_out, $replaceTag);
535 535
                 }
536 536
             }
537 537
 
538
-            foreach($this->url as $media => $url) {
538
+            foreach ($this->url as $media => $url) {
539 539
                 $url = $this->url_replace_cdn($url);
540 540
                 
541 541
                 //Add the stylesheet either deferred (import at bottom) or normal links in head
542
-                if($this->defer == true) {
542
+                if ($this->defer == true) {
543 543
                     $preloadCssBlock .= '<link rel="preload" as="style" media="'.$media.'" href="'.$url.'" onload="this.rel=\'stylesheet\'" />';
544 544
                     $noScriptCssBlock .= '<link type="text/css" media="'.$media.'" href="'.$url.'" rel="stylesheet" />';
545 545
                 } else {
546 546
                     if (strlen($this->csscode[$media]) > $this->cssinlinesize) {
547
-                        $this->inject_in_html('<link type="text/css" media="'.$media.'" href="'.$url.'" rel="stylesheet" />',$replaceTag);
548
-                    } else if (strlen($this->csscode[$media])>0) {
549
-                        $this->inject_in_html('<style type="text/css" media="'.$media.'">'.$this->csscode[$media].'</style>',$replaceTag);
547
+                        $this->inject_in_html('<link type="text/css" media="'.$media.'" href="'.$url.'" rel="stylesheet" />', $replaceTag);
548
+                    } else if (strlen($this->csscode[$media]) > 0) {
549
+                        $this->inject_in_html('<style type="text/css" media="'.$media.'">'.$this->csscode[$media].'</style>', $replaceTag);
550 550
                     }
551 551
                 }
552 552
             }
553 553
             
554
-            if($this->defer == true) {
554
+            if ($this->defer == true) {
555 555
                 $preloadPolyfill = '<script data-cfasync=\'false\'>/*! loadCSS. [c]2017 Filament Group, Inc. MIT License */
556 556
 !function(a){"use strict";var b=function(b,c,d){function e(a){return h.body?a():void setTimeout(function(){e(a)})}function f(){i.addEventListener&&i.removeEventListener("load",f),i.media=d||"all"}var g,h=a.document,i=h.createElement("link");if(c)g=c;else{var j=(h.body||h.getElementsByTagName("head")[0]).childNodes;g=j[j.length-1]}var k=h.styleSheets;i.rel="stylesheet",i.href=b,i.media="only x",e(function(){g.parentNode.insertBefore(i,c?g:g.nextSibling)});var l=function(a){for(var b=i.href,c=k.length;c--;)if(k[c].href===b)return a();setTimeout(function(){l(a)})};return i.addEventListener&&i.addEventListener("load",f),i.onloadcssdefined=l,l(f),i};"undefined"!=typeof exports?exports.loadCSS=b:a.loadCSS=b}("undefined"!=typeof global?global:this);
557 557
 /*! loadCSS rel=preload polyfill. [c]2017 Filament Group, Inc. MIT License */
558 558
 !function(a){if(a.loadCSS){var b=loadCSS.relpreload={};if(b.support=function(){try{return a.document.createElement("link").relList.supports("preload")}catch(b){return!1}},b.poly=function(){for(var b=a.document.getElementsByTagName("link"),c=0;c<b.length;c++){var d=b[c];"preload"===d.rel&&"style"===d.getAttribute("as")&&(a.loadCSS(d.href,d,d.getAttribute("media")),d.rel=null)}},!b.support()){b.poly();var c=a.setInterval(b.poly,300);a.addEventListener&&a.addEventListener("load",function(){b.poly(),a.clearInterval(c)}),a.attachEvent&&a.attachEvent("onload",function(){a.clearInterval(c)})}}}(this);</script>';
559 559
                 $noScriptCssBlock .= "</noscript>";
560
-                $this->inject_in_html($preloadCssBlock.$noScriptCssBlock,$replaceTag);
561
-                $this->inject_in_html($preloadPolyfill,array('</body>','before'));
560
+                $this->inject_in_html($preloadCssBlock.$noScriptCssBlock, $replaceTag);
561
+                $this->inject_in_html($preloadPolyfill, array('</body>', 'before'));
562 562
             }
563 563
         }
564 564
 
@@ -568,62 +568,62 @@  discard block
 block discarded – undo
568 568
     
569 569
     static function fixurls($file, $code) {
570 570
         // Switch all imports to the url() syntax
571
-        $code = preg_replace( '#@import ("|\')(.+?)\.css.*("|\')#', '@import url("${2}.css")', $code );
571
+        $code = preg_replace('#@import ("|\')(.+?)\.css.*("|\')#', '@import url("${2}.css")', $code);
572 572
 
573
-        if ( preg_match_all( self::ASSETS_REGEX, $code, $matches ) ) {
574
-            $file = str_replace( WP_ROOT_DIR, '/', $file );
575
-            $dir = dirname( $file ); // Like /themes/expound/css
573
+        if (preg_match_all(self::ASSETS_REGEX, $code, $matches)) {
574
+            $file = str_replace(WP_ROOT_DIR, '/', $file);
575
+            $dir = dirname($file); // Like /themes/expound/css
576 576
 
577 577
             // $dir should not contain backslashes, since it's used to replace
578 578
             // urls, but it can contain them when running on Windows because
579 579
             // fixurls() is sometimes called with `ABSPATH . 'index.php'`
580
-            $dir = str_replace( '\\', '/', $dir );
581
-            unset( $file ); // not used below at all
580
+            $dir = str_replace('\\', '/', $dir);
581
+            unset($file); // not used below at all
582 582
 
583 583
             $replace = array();
584
-            foreach ( $matches[1] as $k => $url ) {
584
+            foreach ($matches[1] as $k => $url) {
585 585
                 // Remove quotes
586
-                $url    = trim( $url," \t\n\r\0\x0B\"'" );
587
-                $noQurl = trim( $url, "\"'" );
588
-                if ( $url !== $noQurl ) {
586
+                $url    = trim($url, " \t\n\r\0\x0B\"'");
587
+                $noQurl = trim($url, "\"'");
588
+                if ($url !== $noQurl) {
589 589
                     $removedQuotes = true;
590 590
                 } else {
591 591
                     $removedQuotes = false;
592 592
                 }
593 593
 
594
-                if ( '' === $noQurl ) {
594
+                if ('' === $noQurl) {
595 595
                     continue;
596 596
                 }
597 597
 
598 598
                 $url = $noQurl;
599
-                if ( '/' === $url{0} || preg_match( '#^(https?://|ftp://|data:)#i', $url ) ) {
599
+                if ('/' === $url{0} || preg_match('#^(https?://|ftp://|data:)#i', $url)) {
600 600
                     // URL is protocol-relative, host-relative or something we don't touch
601 601
                     continue;
602 602
                 } else {
603 603
                     // Relative URL
604
-                    $newurl = preg_replace( '/https?:/', '', str_replace( ' ', '%20', AUTOPTIMIZE_WP_ROOT_URL . str_replace( '//', '/', $dir . '/' . $url ) ) );
604
+                    $newurl = preg_replace('/https?:/', '', str_replace(' ', '%20', AUTOPTIMIZE_WP_ROOT_URL.str_replace('//', '/', $dir.'/'.$url)));
605 605
 
606 606
                     // Hash the url + whatever was behind potentially for replacement
607 607
                     // We must do this, or different css classes referencing the same bg image (but
608 608
                     // different parts of it, say, in sprites and such) loose their stuff...
609
-                    $hash = md5( $url . $matches[2][$k] );
610
-                    $code = str_replace( $matches[0][$k], $hash, $code );
609
+                    $hash = md5($url.$matches[2][$k]);
610
+                    $code = str_replace($matches[0][$k], $hash, $code);
611 611
 
612
-                    if ( $removedQuotes ) {
613
-                        $replace[$hash] = "url('" . $newurl . "')" . $matches[2][$k];
612
+                    if ($removedQuotes) {
613
+                        $replace[$hash] = "url('".$newurl."')".$matches[2][$k];
614 614
                     } else {
615
-                        $replace[$hash] = 'url(' . $newurl . ')' . $matches[2][$k];
615
+                        $replace[$hash] = 'url('.$newurl.')'.$matches[2][$k];
616 616
                     }
617 617
                 }
618 618
             }
619 619
 
620
-            if ( ! empty( $replace ) ) {
620
+            if (!empty($replace)) {
621 621
                 // Sort the replacements array by key length in desc order (so that the longest strings are replaced first)
622
-                $keys = array_map( 'strlen', array_keys( $replace ) );
623
-                array_multisort( $keys, SORT_DESC, $replace );
622
+                $keys = array_map('strlen', array_keys($replace));
623
+                array_multisort($keys, SORT_DESC, $replace);
624 624
 
625 625
                 // Replace URLs found within $code
626
-                $code = str_replace( array_keys( $replace ), array_values( $replace ), $code );
626
+                $code = str_replace(array_keys($replace), array_values($replace), $code);
627 627
             }
628 628
         }
629 629
 
@@ -631,11 +631,11 @@  discard block
 block discarded – undo
631 631
     }
632 632
     
633 633
     private function ismovable($tag) {
634
-		if ( apply_filters('autoptimize_filter_css_dontaggregate', false) ) {
634
+		if (apply_filters('autoptimize_filter_css_dontaggregate', false)) {
635 635
 			return false;
636 636
         } else if (!empty($this->whitelist)) {
637 637
             foreach ($this->whitelist as $match) {
638
-                if(strpos($tag,$match)!==false) {
638
+                if (strpos($tag, $match) !== false) {
639 639
                     return true;
640 640
                 }
641 641
             }
@@ -643,8 +643,8 @@  discard block
 block discarded – undo
643 643
             return false;
644 644
         } else {
645 645
             if (is_array($this->dontmove)) {
646
-                foreach($this->dontmove as $match) {
647
-                    if(strpos($tag,$match)!==false) {
646
+                foreach ($this->dontmove as $match) {
647
+                    if (strpos($tag, $match) !== false) {
648 648
                         //Matched something
649 649
                         return false;
650 650
                     }
@@ -656,21 +656,21 @@  discard block
 block discarded – undo
656 656
         }
657 657
     }
658 658
     
659
-    private function can_inject_late($cssPath,$css) {
659
+    private function can_inject_late($cssPath, $css) {
660 660
 		$consider_minified_array = apply_filters('autoptimize_filter_css_consider_minified', false, $cssPath);
661
-        if ( $this->inject_min_late !== true ) {
661
+        if ($this->inject_min_late !== true) {
662 662
             // late-inject turned off
663 663
             return false;
664
-        } else if ( (strpos($cssPath,"min.css") === false) && ( str_replace($consider_minified_array, '', $cssPath) === $cssPath ) ) {
664
+        } else if ((strpos($cssPath, "min.css") === false) && (str_replace($consider_minified_array, '', $cssPath) === $cssPath)) {
665 665
 			// file not minified based on filename & filter
666 666
 			return false;
667
-        } else if ( strpos($css,"@import") !== false ) {
667
+        } else if (strpos($css, "@import") !== false) {
668 668
             // can't late-inject files with imports as those need to be aggregated 
669 669
             return false;
670
-        } else if ( (strpos($css,"@font-face")!==false ) && ( apply_filters("autoptimize_filter_css_fonts_cdn",false)===true) && (!empty($this->cdn_url)) ) {
670
+        } else if ((strpos($css, "@font-face") !== false) && (apply_filters("autoptimize_filter_css_fonts_cdn", false) === true) && (!empty($this->cdn_url))) {
671 671
             // don't late-inject CSS with font-src's if fonts are set to be CDN'ed
672 672
             return false;
673
-        } else if ( (($this->datauris == true) || (!empty($this->cdn_url))) && preg_match("#background[^;}]*url\(#Ui",$css) ) {
673
+        } else if ((($this->datauris == true) || (!empty($this->cdn_url))) && preg_match("#background[^;}]*url\(#Ui", $css)) {
674 674
             // don't late-inject CSS with images if CDN is set OR is image inlining is on
675 675
             return false;
676 676
         } else {
@@ -680,11 +680,11 @@  discard block
 block discarded – undo
680 680
     }
681 681
     
682 682
     private function maybe_cdn_urls($inUrl) {
683
-        $url = trim($inUrl," \t\n\r\0\x0B\"'");
684
-        $urlPath = parse_url($url,PHP_URL_PATH);
683
+        $url = trim($inUrl, " \t\n\r\0\x0B\"'");
684
+        $urlPath = parse_url($url, PHP_URL_PATH);
685 685
 
686 686
         // exclude fonts from CDN except if filter returns true
687
-        if ( !preg_match('#\.(woff2?|eot|ttf|otf)$#i',$urlPath) || apply_filters('autoptimize_filter_css_fonts_cdn',false) ) {
687
+        if (!preg_match('#\.(woff2?|eot|ttf|otf)$#i', $urlPath) || apply_filters('autoptimize_filter_css_fonts_cdn', false)) {
688 688
             $cdn_url = $this->url_replace_cdn($url);
689 689
         } else {
690 690
             $cdn_url = $url;
Please login to merge, or discard this patch.