Completed
Push — master ( 6ca290...afc802 )
by frank
02:41
created

autoptimizeStyles::can_inject_late()   B

Complexity

Conditions 10
Paths 5

Size

Total Lines 18
Code Lines 11

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 10
eloc 11
nc 5
nop 2
dl 0
loc 18
rs 7.2765
c 0
b 0
f 0

How to fix   Complexity   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
<?php
2
if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
3
4
class autoptimizeStyles extends autoptimizeBase {
5
6
    const ASSETS_REGEX = '/url\s*\(\s*(?!["\']?data:)(?![\'|\"]?[\#|\%|])([^)]+)\s*\)([^;},]*)/i';
7
8
    private $css = array();
9
    private $csscode = array();
10
    private $url = array();
11
    private $restofcontent = '';
12
    private $datauris = false;
13
    private $hashmap = array();
14
    private $alreadyminified = false;
15
    private $inline = false;
16
    private $defer = false;
17
    private $defer_inline = false;
18
    private $whitelist = '';
19
    private $cssinlinesize = '';
20
    private $cssremovables = array();
21
    private $include_inline = false;
22
    private $inject_min_late = '';
23
24
    //Reads the page and collects style tags
25
    public function read($options) {
26
        $noptimizeCSS = apply_filters( 'autoptimize_filter_css_noptimize', false, $this->content );
27
        if ($noptimizeCSS) return false;
28
29
        $whitelistCSS = apply_filters( 'autoptimize_filter_css_whitelist', '' );
30
        if (!empty($whitelistCSS)) {
31
            $this->whitelist = array_filter(array_map('trim',explode(",",$whitelistCSS)));
0 ignored issues
show
Documentation Bug introduced by
It seems like array_filter(array_map('...e(',', $whitelistCSS))) of type array is incompatible with the declared type string of property $whitelist.

Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property.

Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property..

Loading history...
32
        }
33
        
34
        if ($options['nogooglefont'] == true) {
35
            $removableCSS = "fonts.googleapis.com";
36
        } else {
37
            $removableCSS = "";
38
        }
39
        $removableCSS = apply_filters( 'autoptimize_filter_css_removables', $removableCSS);
40 View Code Duplication
        if (!empty($removableCSS)) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
41
            $this->cssremovables = array_filter(array_map('trim',explode(",",$removableCSS)));
42
        }
43
44
        $this->cssinlinesize = apply_filters('autoptimize_filter_css_inlinesize',256);
45
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);
48
49
        // Remove everything that's not the header
50 View Code Duplication
        if ( apply_filters('autoptimize_filter_css_justhead',$options['justhead']) == true ) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
51
            $content = explode('</head>',$this->content,2);
52
            $this->content = $content[0].'</head>';
53
            $this->restofcontent = $content[1];
54
        }
55
56
        // include inline?
57
        if( apply_filters('autoptimize_css_include_inline',$options['include_inline']) == true ) {
58
            $this->include_inline = true;
59
        }
60
        
61
        // what CSS shouldn't be autoptimized
62
        $excludeCSS = $options['css_exclude'];
63
        $excludeCSS = apply_filters( 'autoptimize_filter_css_exclude', $excludeCSS );
64 View Code Duplication
        if ($excludeCSS!=="") {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
65
            $this->dontmove = array_filter(array_map('trim',explode(",",$excludeCSS)));
0 ignored issues
show
Bug introduced by
The property dontmove does not exist. Did you maybe forget to declare it?

In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code:

class MyClass { }

$x = new MyClass();
$x->foo = true;

Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion:

class MyClass {
    public $foo;
}

$x = new MyClass();
$x->foo = true;
Loading history...
66
        } else {
67
            $this->dontmove = "";
68
        }
69
70
        // should we defer css?
71
        // value: true/ false
72
        $this->defer = $options['defer'];
73
        $this->defer = apply_filters( 'autoptimize_filter_css_defer', $this->defer );
74
75
        // should we inline while deferring?
76
        // value: inlined CSS
77
        $this->defer_inline = $options['defer_inline'];
78
        $this->defer_inline = apply_filters( 'autoptimize_filter_css_defer_inline', $this->defer_inline );
79
80
        // should we inline?
81
        // value: true/ false
82
        $this->inline = $options['inline'];
83
        $this->inline = apply_filters( 'autoptimize_filter_css_inline', $this->inline );
84
        
85
        // get cdn url
86
        $this->cdn_url = $options['cdn_url'];
0 ignored issues
show
Bug introduced by
The property cdn_url does not exist. Did you maybe forget to declare it?

In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code:

class MyClass { }

$x = new MyClass();
$x->foo = true;

Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion:

class MyClass {
    public $foo;
}

$x = new MyClass();
$x->foo = true;
Loading history...
87
        
88
        // Store data: URIs setting for later use
89
        $this->datauris = $options['datauris'];
90
        
91
        // noptimize me
92
        $this->content = $this->hide_noptimize($this->content);
93
        
94
        // exclude (no)script, as those may contain CSS which should be left as is
95 View Code Duplication
        if ( strpos( $this->content, '<script' ) !== false ) { 
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
96
            $this->content = preg_replace_callback(
97
                '#<(?:no)?script.*?<\/(?:no)?script>#is',
98
                create_function(
0 ignored issues
show
Security Best Practice introduced by
The use of create_function is highly discouraged, better use a closure.

create_function can pose a great security vulnerability as it is similar to eval, and could be used for arbitrary code execution. We highly recommend to use a closure instead.

// Instead of
$function = create_function('$a, $b', 'return $a + $b');

// Better use
$function = function($a, $b) { return $a + $b; }
Loading history...
99
                    '$matches',
100
                    'return "%%SCRIPT%%".base64_encode($matches[0])."%%SCRIPT%%";'
101
                ),
102
                $this->content
103
            );
104
        }
105
106
        // Save IE hacks
107
        $this->content = $this->hide_iehacks($this->content);
108
109
        // hide comments
110
        $this->content = $this->hide_comments($this->content);
111
        
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);
117
                } else if ($this->ismovable($tag)) {
118
                    // Get the media
119
                    if(strpos($tag,'media=')!==false) {
120
                        preg_match('#media=(?:"|\')([^>]*)(?:"|\')#Ui',$tag,$medias);
121
                        $medias = explode(',',$medias[1]);
122
                        $media = array();
123
                        foreach($medias as $elem) {
124
                            if (empty($elem)) { $elem="all"; }
125
                            $media[] = $elem;
126
                        }
127
                    } else {
128
                        // No media specified - applies to all
129
                        $media = array('all');
130
                    }
131
                    $media = apply_filters( 'autoptimize_filter_css_tagmedia',$media,$tag );
132
                
133
                    if(preg_match('#<link.*href=("|\')(.*)("|\')#Usmi',$tag,$source)) {
134
                        // <link>
135
                        $explUrl = explode('?',$source[2],2);
136
                        $url = $explUrl[0];
137
                        $path = $this->getpath($url);
138
                        
139
                        if($path!==false && preg_match('#\.css$#',$path)) {
140
                            // Good link
141
                            $this->css[] = array($media,$path);
142
                        }else{
143
                            // Link is dynamic (.php etc)
144
                            $tag = '';
145
                        }
146
                    } else {
147
                        // inline css in style tags can be wrapped in comment tags, so restore comments
148
                        $tag = $this->restore_comments($tag);
149
                        preg_match('#<style.*>(.*)</style>#Usmi',$tag,$code);
150
151
                        // and re-hide them to be able to to the removal based on tag
152
                        $tag = $this->hide_comments($tag);
153
154
                        if ( $this->include_inline ) {
155
                            $code = preg_replace('#^.*<!\[CDATA\[(?:\s*\*/)?(.*)(?://|/\*)\s*?\]\]>.*$#sm','$1',$code[1]);
156
                            $this->css[] = array($media,'INLINE;'.$code);
157
                        } else {
158
                            $tag = '';
159
                        }
160
                    }
161
                    
162
                    // Remove the original style tag
163
                    $this->content = str_replace($tag,'',$this->content);
164
                } else {
165
					// excluded CSS, minify if getpath 
166
					if (preg_match('#<link.*href=("|\')(.*)("|\')#Usmi',$tag,$source)) {
167
						$explUrl = explode('?',$source[2],2);
168
                        $url = $explUrl[0];
169
                        $path = $this->getpath($url);
170
 					
171
						if ($path && apply_filters('autoptimize_filter_css_minify_excluded',false)) {
172
							$_CachedMinifiedUrl = $this->minify_single($path);
173
174
							if (!empty($_CachedMinifiedUrl)) {
175
								// replace orig URL with URL to cache
176
								$newTag = str_replace($url, $_CachedMinifiedUrl, $tag);
177
							} else {
178
								$newTag = $tag;
179
							}
180
							
181
							// remove querystring from URL
182 View Code Duplication
							if ( !empty($explUrl[1]) ) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
183
								$newTag = str_replace("?".$explUrl[1],"",$newTag);
184
							}
185
186
							// and replace
187
							$this->content = str_replace($tag,$newTag,$this->content);
188
						}
189
					}					
190
				}
191
            }
192
            return true;
193
        }
194
        // Really, no styles?
195
        return false;
196
    }
197
    
198
    // Joins and optimizes CSS
199
    public function minify() {
200
        foreach($this->css as $group) {
201
            list($media,$css) = $group;
202
            if(preg_match('#^INLINE;#',$css)) {
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;
210
                }
211
            } else {
212
                //<link>
213
                if($css !== false && file_exists($css) && is_readable($css)) {
214
                    $cssPath = $css;
215
                    $css = $this->fixurls($cssPath,file_get_contents($cssPath));
216
                    $css = preg_replace('/\x{EF}\x{BB}\x{BF}/','',$css);
217
                    $tmpstyle = apply_filters( 'autoptimize_css_individual_style', $css, $cssPath );
218 View Code Duplication
                    if (has_filter('autoptimize_css_individual_style') && !empty($tmpstyle)) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
219
                        $css=$tmpstyle;
220
                        $this->alreadyminified=true;
221
                    } else if ($this->can_inject_late($cssPath,$css)) {
222
                        $css="%%INJECTLATER%%".base64_encode($cssPath)."|".md5($css)."%%INJECTLATER%%";
223
                    }
224
                } else {
225
                    // Couldn't read CSS. Maybe getpath isn't working?
226
                    $css = '';
227
                }
228
            }
229
            
230
            foreach($media as $elem) {
231
                if(!isset($this->csscode[$elem]))
232
                    $this->csscode[$elem] = '';
233
                $this->csscode[$elem] .= "\n/*FILESTART*/".$css;
234
            }
235
        }
236
        
237
        // Check for duplicate code
238
        $md5list = array();
239
        $tmpcss = $this->csscode;
240
        foreach($tmpcss as $media => $code) {
241
            $md5sum = md5($code);
242
            $medianame = $media;
243
            foreach($md5list as $med => $sum) {
244
                // If same code
245
                if($sum === $md5sum) {
246
                    //Add the merged code
247
                    $medianame = $med.', '.$media;
248
                    $this->csscode[$medianame] = $code;
249
                    $md5list[$medianame] = $md5list[$med];
250
                    unset($this->csscode[$med], $this->csscode[$media]);
251
                    unset($md5list[$med]);
252
                }
253
            }
254
            $md5list[$medianame] = $md5sum;
255
        }
256
        unset($tmpcss);
257
        
258
        // Manage @imports, while is for recursive import management
259
        foreach ($this->csscode as &$thiscss) {
260
            // Flag to trigger import reconstitution and var to hold external imports
261
            $fiximports = false;
262
            $external_imports = "";
263
264
            // remove comments to avoid importing commented-out imports
265
            $thiscss_nocomments = preg_replace('#/\*.*\*/#Us','',$thiscss);
266
267
            while(preg_match_all('#@import.*(?:;|$)#Um',$thiscss_nocomments,$matches)) {
268
                foreach($matches[0] as $import)    {
269
                    if ($this->isremovable($import,$this->cssremovables)) {
270
                        $thiscss = str_replace($import,'',$thiscss);
271
                        $import_ok = true;
272
                    } else {
273
                        $url = trim(preg_replace('#^.*((?:https?:|ftp:)?//.*\.css).*$#','$1',trim($import))," \t\n\r\0\x0B\"'");
274
                        $path = $this->getpath($url);
275
                        $import_ok = false;
276
                        if (file_exists($path) && is_readable($path)) {
277
                            $code = addcslashes($this->fixurls($path,file_get_contents($path)),"\\");
278
                            $code = preg_replace('/\x{EF}\x{BB}\x{BF}/','',$code);
279
                            $tmpstyle = apply_filters( 'autoptimize_css_individual_style', $code, "" );
280 View Code Duplication
                            if ( has_filter('autoptimize_css_individual_style') && !empty($tmpstyle)) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
281
                                $code=$tmpstyle;
282
                                $this->alreadyminified=true;
283
                            } else if ($this->can_inject_late($path,$code)) {
284
                                $code="%%INJECTLATER%%".base64_encode($path)."|".md5($code)."%%INJECTLATER%%";
285
                            }
286
                            
287
                            if(!empty($code)) {
288
                                $tmp_thiscss = preg_replace('#(/\*FILESTART\*/.*)'.preg_quote($import,'#').'#Us','/*FILESTART2*/'.$code.'$1',$thiscss);
289
                                if (!empty($tmp_thiscss)) {
290
                                    $thiscss = $tmp_thiscss;
291
                                    $import_ok = true;
292
                                    unset($tmp_thiscss);
293
                                }
294
                                unset($code);
295
                            }
296
                        }
297
                    }
298
299
                    if (!$import_ok) {
300
                        // external imports and general fall-back
301
                        $external_imports .= $import;
302
                        $thiscss = str_replace($import,'',$thiscss);
303
                        $fiximports = true;
304
                    }
305
                }
306
                $thiscss = preg_replace('#/\*FILESTART\*/#','',$thiscss);
307
                $thiscss = preg_replace('#/\*FILESTART2\*/#','/*FILESTART*/',$thiscss);
308
                
309
                // and update $thiscss_nocomments before going into next iteration in while loop
310
                $thiscss_nocomments=preg_replace('#/\*.*\*/#Us','',$thiscss);
311
            }
312
            unset($thiscss_nocomments);
313
            
314
            // add external imports to top of aggregated CSS
315
            if($fiximports) {
316
                $thiscss=$external_imports.$thiscss;
317
            }
318
        }
319
        unset($thiscss);
320
        
321
        // $this->csscode has all the uncompressed code now. 
322
        foreach($this->csscode as &$code) {
323
            // Check for already-minified code
324
            $hash = md5($code);
325
            $ccheck = new autoptimizeCache($hash,'css');
326
            if($ccheck->check()) {
327
                $code = $ccheck->retrieve();
328
                $this->hashmap[md5($code)] = $hash;
329
                continue;
330
            }
331
            unset($ccheck);            
332
333
            // Do the imaging!
334
            $imgreplace = array();
335
            // dtbaker update. Instead of searching for the entire `background: url(), url(), url();` string we just search for the individual url() elements.
336
            // this allows us to find/replace multiple background images with minimal code changes below.
337
            // this is the old regex that searched for the entire background css rule, but it wouldn't match multiple background image url css rules.
338
            // preg_match_all('#(background[^;{}]*url\((?!\s?"?\'?\s?data)(.*)\)[^;}]*)(?:;|$|})#Usm',$code,$matches);
0 ignored issues
show
Unused Code Comprehensibility introduced by
75% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
339
            // this new regex will be slightly faster too:
340
            preg_match_all( self::ASSETS_REGEX, $code, $matches );
341
342
            if(($this->datauris == true) && (function_exists('base64_encode')) && (is_array($matches)))    {
0 ignored issues
show
Coding Style Best Practice introduced by
It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

When comparing two booleans, it is generally considered safer to use the strict comparison operator.

Loading history...
343
                foreach($matches[1] as $count => $quotedurl) {
344
                    $iurl = trim($quotedurl," \t\n\r\0\x0B\"'");
345
346
                    // if querystring, remove it from url
347
                    if (strpos($iurl,'?') !== false) { $iurl = strtok($iurl,'?'); }
348
                    
349
                    $ipath = $this->getpath($iurl);
350
351
                    $datauri_max_size = 4096;
352
                    $datauri_max_size = (int) apply_filters( 'autoptimize_filter_css_datauri_maxsize', $datauri_max_size );
353
                    $datauri_exclude = apply_filters( 'autoptimize_filter_css_datauri_exclude', "");
354
                    if (!empty($datauri_exclude)) {
355
                        $no_datauris=array_filter(array_map('trim',explode(",",$datauri_exclude)));
356
                        foreach ($no_datauris as $no_datauri) {
357
                            if (strpos($iurl,$no_datauri)!==false) {
358
                                $ipath=false;
359
                                break;
360
                            }
361
                        }
362
                    }
363
364
                    if($ipath != false && preg_match('#\.(jpe?g|png|gif|bmp)$#i',$ipath) && file_exists($ipath) && is_readable($ipath) && filesize($ipath) <= $datauri_max_size) {
0 ignored issues
show
Bug introduced by
It seems like you are loosely comparing $ipath of type false|string against false; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.
Loading history...
365
                        $ihash=md5($ipath);
366
                        $icheck = new autoptimizeCache($ihash,'img');
367
                        if($icheck->check()) {
368
                            // we have the base64 image in cache
369
                            $headAndData=$icheck->retrieve();
370
                            $_base64data=explode(";base64,",$headAndData);
371
                            $base64data=$_base64data[1];
0 ignored issues
show
Unused Code introduced by
$base64data is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
372
                        } else {
373
                            // It's an image and we don't have it in cache, get the type
374
                            $explA=explode('.',$ipath);
375
                            $type=end($explA);
376
377
                            switch($type) {
378
                                case 'jpeg':
379
                                    $dataurihead = 'data:image/jpeg;base64,';
380
                                    break;
381
                                case 'jpg':
382
                                    $dataurihead = 'data:image/jpeg;base64,';
383
                                    break;
384
                                case 'gif':
385
                                    $dataurihead = 'data:image/gif;base64,';
386
                                    break;
387
                                case 'png':
388
                                    $dataurihead = 'data:image/png;base64,';
389
                                    break;
390
                                case 'bmp':
391
                                    $dataurihead = 'data:image/bmp;base64,';
392
                                    break;
393
                                default:
394
                                    $dataurihead = 'data:application/octet-stream;base64,';
395
                            }
396
                        
397
                            // Encode the data
398
                            $base64data = base64_encode(file_get_contents($ipath));
399
                            $headAndData=$dataurihead.$base64data;
400
401
                            // Save in cache
402
                            $icheck->cache($headAndData,"text/plain");
403
                        }
404
                        unset($icheck);
405
406
                        // Add it to the list for replacement
407
                        $imgreplace[$matches[0][$count]] = str_replace($quotedurl,$headAndData,$matches[0][$count]).';';
408
                        
409
                    } else {
410
                        // just cdn the URL if applicable
411
                        if (!empty($this->cdn_url)) {
412
                            $url = trim($quotedurl," \t\n\r\0\x0B\"'");
413
                            $cdn_url=$this->url_replace_cdn($url);
414
                            $imgreplace[$matches[0][$count]] = str_replace($quotedurl,$cdn_url,$matches[0][$count]);
415
                        }
416
                    }
417
                }
418
            } else if ((is_array($matches)) && (!empty($this->cdn_url))) {
419
                // change background image urls to cdn-url
420 View Code Duplication
                foreach($matches[1] as $count => $quotedurl) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
421
                    $url = trim($quotedurl," \t\n\r\0\x0B\"'");
422
                    $cdn_url=$this->url_replace_cdn($url);
423
                    $imgreplace[$matches[0][$count]] = str_replace($quotedurl,$cdn_url,$matches[0][$count]);
424
                }
425
            }
426
            
427
            if(!empty($imgreplace)) {
428
                $code = str_replace(array_keys($imgreplace),array_values($imgreplace),$code);
429
            }
430
            
431
            // CDN the fonts!
432
            if ( (!empty($this->cdn_url)) && (apply_filters('autoptimize_filter_css_fonts_cdn',false)) && (version_compare(PHP_VERSION, '5.3.0') >= 0) ) {
433
                $fontreplace = array();
434
                include_once(AUTOPTIMIZE_PLUGIN_DIR.'classlesses/autoptimizeFontRegex.php');
435
                
436
                preg_match_all($fonturl_regex,$code,$matches);
0 ignored issues
show
Bug introduced by
The variable $fonturl_regex does not exist. Did you forget to declare it?

This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug.

Loading history...
437
                if (is_array($matches)) {
438 View Code Duplication
                    foreach($matches[8] as $count => $quotedurl) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
439
                        $url = trim($quotedurl," \t\n\r\0\x0B\"'");
440
                        $cdn_url=$this->url_replace_cdn($url);
441
                        $fontreplace[$matches[8][$count]] = str_replace($quotedurl,$cdn_url,$matches[8][$count]);
442
                    }
443
                    if(!empty($fontreplace)) {
444
                        $code = str_replace(array_keys($fontreplace),array_values($fontreplace),$code);
445
                    }
446
                }
447
            }
448
            
449
            // Minify
450
            if (($this->alreadyminified!==true) && (apply_filters( "autoptimize_css_do_minify", true))) {
451 View Code Duplication
                if (class_exists('Minify_CSS_Compressor')) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
452
                    $tmp_code = trim(Minify_CSS_Compressor::process($code));
453
                } else if(class_exists('CSSmin')) {
454
                    $cssmin = new CSSmin();
455
                    if (method_exists($cssmin,"run")) {
456
                        $tmp_code = trim($cssmin->run($code));
457
                    } elseif (@is_callable(array($cssmin,"minify"))) {
458
                        $tmp_code = trim(CssMin::minify($code));
0 ignored issues
show
Bug introduced by
The method minify() cannot be called from this context as it is declared private in class CSSmin.

This check looks for access to methods that are not accessible from the current context.

If you need to make a method accessible to another context you can raise its visibility level in the defining class.

Loading history...
Bug introduced by
The call to minify() misses a required argument $linebreak_pos.

This check looks for function calls that miss required arguments.

Loading history...
459
                    }
460
                }
461
                if (!empty($tmp_code)) {
462
                    $code = $tmp_code;
463
                    unset($tmp_code);
464
                }
465
            }
466
            
467
            $code = $this->inject_minified($code);
468
            
469
            $tmp_code = apply_filters( 'autoptimize_css_after_minify',$code );
470
            if (!empty($tmp_code)) {
471
                $code = $tmp_code;
472
                unset($tmp_code);
473
            }
474
            
475
            $this->hashmap[md5($code)] = $hash;
476
        }
477
        unset($code);
478
        return true;
479
    }
480
    
481
    //Caches the CSS in uncompressed, deflated and gzipped form.
482
    public function cache() {
483
        // CSS cache
484
        foreach($this->csscode as $media => $code) {
485
            $md5 = $this->hashmap[md5($code)];
486
                
487
            $cache = new autoptimizeCache($md5,'css');
488
            if(!$cache->check()) {
489
                // Cache our code
490
                $cache->cache($code,'text/css');
491
            }
492
            $this->url[$media] = AUTOPTIMIZE_CACHE_URL.$cache->getname();
493
        }
494
    }
495
    
496
    //Returns the content
497
    public function getcontent() {
498
        // restore IE hacks
499
        $this->content = $this->restore_iehacks($this->content);
500
501
        // restore comments
502
        $this->content = $this->restore_comments($this->content);
503
        
504
        // restore (no)script
505 View Code Duplication
        if ( strpos( $this->content, '%%SCRIPT%%' ) !== false ) { 
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
506
            $this->content = preg_replace_callback(
507
                '#%%SCRIPT%%(.*?)%%SCRIPT%%#is',
508
                create_function(
0 ignored issues
show
Security Best Practice introduced by
The use of create_function is highly discouraged, better use a closure.

create_function can pose a great security vulnerability as it is similar to eval, and could be used for arbitrary code execution. We highly recommend to use a closure instead.

// Instead of
$function = create_function('$a, $b', 'return $a + $b');

// Better use
$function = function($a, $b) { return $a + $b; }
Loading history...
509
                    '$matches',
510
                    'return base64_decode($matches[1]);'
511
                ),
512
                $this->content
513
            );
514
        }
515
516
        // restore noptimize
517
        $this->content = $this->restore_noptimize($this->content);
518
        
519
        //Restore the full content
520
        if(!empty($this->restofcontent)) {
521
            $this->content .= $this->restofcontent;
522
            $this->restofcontent = '';
523
        }
524
        
525
        // Inject the new stylesheets
526
        $replaceTag = array("<title","before");
527
        $replaceTag = apply_filters( 'autoptimize_filter_css_replacetag', $replaceTag );
528
529
        if ($this->inline == true) {
0 ignored issues
show
Coding Style Best Practice introduced by
It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

When comparing two booleans, it is generally considered safer to use the strict comparison operator.

Loading history...
530
            foreach($this->csscode as $media => $code) {
531
                $this->inject_in_html('<style type="text/css" media="'.$media.'">'.$code.'</style>',$replaceTag);
532
            }
533
        } else {
534
            if ($this->defer == true) {
0 ignored issues
show
Coding Style Best Practice introduced by
It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

When comparing two booleans, it is generally considered safer to use the strict comparison operator.

Loading history...
535
                $preloadCssBlock = "";
536
                $noScriptCssBlock = "<noscript id=\"aonoscrcss\">";
537
                $defer_inline_code=$this->defer_inline;
538
                if(!empty($defer_inline_code)){
539
                    if ( apply_filters( 'autoptimize_filter_css_critcss_minify',true ) ) {
540
                        $iCssHash=md5($defer_inline_code);
541
                        $iCssCache = new autoptimizeCache($iCssHash,'css');
542
                        if($iCssCache->check()) { 
543
                            // we have the optimized inline CSS in cache
544
                            $defer_inline_code=$iCssCache->retrieve();
545
                        } else {
546
                            if (class_exists('Minify_CSS_Compressor')) {
547
                                $tmp_code = trim(Minify_CSS_Compressor::process($defer_inline_code));
0 ignored issues
show
Documentation introduced by
$defer_inline_code is of type boolean, but the function expects a string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
548
                            } else if(class_exists('CSSmin')) {
549
                                $cssmin = new CSSmin();
550
                                $tmp_code = trim($cssmin->run($defer_inline_code));
0 ignored issues
show
Documentation introduced by
$defer_inline_code is of type boolean, but the function expects a string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
551
                            }
552
                            if (!empty($tmp_code)) {
553
                                $defer_inline_code = $tmp_code;
554
                                $iCssCache->cache($defer_inline_code,"text/css");
555
                                unset($tmp_code);
556
                            }
557
                        }
558
                    }
559
                    $code_out='<style type="text/css" id="aoatfcss" media="all">'.$defer_inline_code.'</style>';
560
                    $this->inject_in_html($code_out,$replaceTag);
561
                }
562
            }
563
564
            foreach($this->url as $media => $url) {
565
                $url = $this->url_replace_cdn($url);
566
                
567
                //Add the stylesheet either deferred (import at bottom) or normal links in head
568
                if($this->defer == true) {
0 ignored issues
show
Coding Style Best Practice introduced by
It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

When comparing two booleans, it is generally considered safer to use the strict comparison operator.

Loading history...
569
                    $preloadCssBlock .= '<link rel="preload" as="style" media="'.$media.'" href="'.$url.'" onload="this.rel=\'stylesheet\'" />';
0 ignored issues
show
Bug introduced by
The variable $preloadCssBlock does not seem to be defined for all execution paths leading up to this point.

If you define a variable conditionally, it can happen that it is not defined for all execution paths.

Let’s take a look at an example:

function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.

Available Fixes

  1. Check for existence of the variable explicitly:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        if (isset($x)) { // Make sure it's always set.
            echo $x;
        }
    }
    
  2. Define a default value for the variable:

    function myFunction($a) {
        $x = ''; // Set a default which gets overridden for certain paths.
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        echo $x;
    }
    
  3. Add a value for the missing path:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
    
            // We add support for the missing case.
            default:
                $x = '';
                break;
        }
    
        echo $x;
    }
    
Loading history...
570
                    $noScriptCssBlock .= '<link type="text/css" media="'.$media.'" href="'.$url.'" rel="stylesheet" />';
0 ignored issues
show
Bug introduced by
The variable $noScriptCssBlock does not seem to be defined for all execution paths leading up to this point.

If you define a variable conditionally, it can happen that it is not defined for all execution paths.

Let’s take a look at an example:

function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.

Available Fixes

  1. Check for existence of the variable explicitly:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        if (isset($x)) { // Make sure it's always set.
            echo $x;
        }
    }
    
  2. Define a default value for the variable:

    function myFunction($a) {
        $x = ''; // Set a default which gets overridden for certain paths.
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        echo $x;
    }
    
  3. Add a value for the missing path:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
    
            // We add support for the missing case.
            default:
                $x = '';
                break;
        }
    
        echo $x;
    }
    
Loading history...
571
                } else {
572
                    if (strlen($this->csscode[$media]) > $this->cssinlinesize) {
573
                        $this->inject_in_html('<link type="text/css" media="'.$media.'" href="'.$url.'" rel="stylesheet" />',$replaceTag);
574
                    } else if (strlen($this->csscode[$media])>0) {
575
                        $this->inject_in_html('<style type="text/css" media="'.$media.'">'.$this->csscode[$media].'</style>',$replaceTag);
576
                    }
577
                }
578
            }
579
            
580
            if($this->defer == true) {
0 ignored issues
show
Coding Style Best Practice introduced by
It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

When comparing two booleans, it is generally considered safer to use the strict comparison operator.

Loading history...
581
                $preloadPolyfill = '<script data-cfasync=\'false\'>/*! loadCSS. [c]2017 Filament Group, Inc. MIT License */
582
!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);
583
/*! loadCSS rel=preload polyfill. [c]2017 Filament Group, Inc. MIT License */
584
!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>';
585
                $noScriptCssBlock .= "</noscript>";
586
                $this->inject_in_html($preloadCssBlock.$noScriptCssBlock,$replaceTag);
587
                $this->inject_in_html($preloadPolyfill,array('</body>','before'));
588
            }
589
        }
590
591
        //Return the modified stylesheet
592
        return $this->content;
593
    }
594
    
595
    static function fixurls($file,$code) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
596
        $file = str_replace(WP_ROOT_DIR,'',$file);
597
        $file = str_replace(AUTOPTIMIZE_WP_CONTENT_NAME,'',$file);
598
        $dir = dirname($file); // Like /themes/expound/css
599
600
        // switch all imports to the url() syntax
601
        $code=preg_replace('#@import ("|\')(.+?)\.css.*("|\')#','@import url("${2}.css")',$code);
602
603
        if( preg_match_all( self::ASSETS_REGEX, $code, $matches ) ) {
604
            $replace = array();
605
            foreach($matches[1] as $k => $url) {
606
                // Remove quotes
607
                $url = trim($url," \t\n\r\0\x0B\"'");
608
                $noQurl = trim($url,"\"'");
609
                
610
                if ($noQurl === '') { continue; }
611
                
612
                if ($url!==$noQurl) {
613
                    $removedQuotes=true;
614
                } else {
615
                    $removedQuotes=false;
616
                }
617
                $url=$noQurl;
618
                if(substr($url,0,1)=='/' || preg_match('#^(https?://|ftp://|data:)#i',$url)) {
619
                    //URL is absolute
620
                    continue;
621
                } else {
622
                    // relative URL
623
                    $newurl = preg_replace('/https?:/','',str_replace(" ","%20",AUTOPTIMIZE_WP_CONTENT_URL.str_replace('//','/',$dir.'/'.$url)));
624
625
                    $hash = md5($url);
626
                    $code = str_replace($matches[0][$k],$hash,$code);
627
628
                    if (!empty($removedQuotes)) {
629
                        $replace[$hash] = 'url(\''.$newurl.'\')';
630
                    } else {
631
                        $replace[$hash] = 'url('.$newurl.')';
632
                    }
633
                }
634
            }    
635
            //Do the replacing here to avoid breaking URLs
636
            $code = str_replace(array_keys($replace),array_values($replace),$code);
637
        }    
638
        return $code;
639
    }
640
    
641
    private function ismovable($tag) {
642
		if (apply_filters('autoptimize_filter_css_dontaggregate',false)) {
643
			return false;
644
        } else if (!empty($this->whitelist)) {
645
            foreach ($this->whitelist as $match) {
0 ignored issues
show
Bug introduced by
The expression $this->whitelist of type string is not traversable.
Loading history...
646
                if(strpos($tag,$match)!==false) {
647
                    return true;
648
                }
649
            }
650
            // no match with whitelist
651
            return false;
652
        } else {
653
            if (is_array($this->dontmove)) {
654
                foreach($this->dontmove as $match) {
655
                    if(strpos($tag,$match)!==false) {
656
                        //Matched something
657
                        return false;
658
                    }
659
                }
660
            }
661
            
662
            //If we're here it's safe to move
663
            return true;
664
        }
665
    }
666
    
667
    private function can_inject_late($cssPath,$css) {
668
        if ((strpos($cssPath,"min.css")===false) || ($this->inject_min_late!==true)) {
669
            // late-inject turned off or file not minified based on filename
670
            return false;
671
        } else if (strpos($css,"@import")!==false) {
672
            // can't late-inject files with imports as those need to be aggregated 
673
            return false;
674
        } else if ( (strpos($css,"@font-face")!==false ) && ( apply_filters("autoptimize_filter_css_fonts_cdn",false)===true) && (!empty($this->cdn_url)) ) {
675
            // don't late-inject CSS with font-src's if fonts are set to be CDN'ed
676
            return false;
677
        } else if ( (($this->datauris == true) || (!empty($this->cdn_url))) && preg_match("#background[^;}]*url\(#Ui",$css) ) {
0 ignored issues
show
Coding Style Best Practice introduced by
It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

When comparing two booleans, it is generally considered safer to use the strict comparison operator.

Loading history...
678
            // don't late-inject CSS with images if CDN is set OR is image inlining is on
679
            return false;
680
        } else {
681
            // phew, all is safe, we can late-inject
682
            return true;
683
        }
684
    }
685
}
686