Completed
Pull Request — master (#223)
by
unknown
01:56
created
classes/autoptimizeCLI.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -3,12 +3,12 @@  discard block
 block discarded – undo
3 3
  * WP-CLI commands for Autoptimize.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
10 10
 // This is a WP-CLI command, so bail if it's not available.
11
-if ( ! defined( 'WP_CLI' ) ) {
11
+if (!defined('WP_CLI')) {
12 12
     return;
13 13
 }
14 14
 
@@ -24,11 +24,11 @@  discard block
 block discarded – undo
24 24
      *
25 25
      * @return void
26 26
      */
27
-    public function clear( $args, $args_assoc ) {
28
-        WP_CLI::line( esc_html__( 'Flushing the cache...', 'autoptimize' ) );
27
+    public function clear($args, $args_assoc) {
28
+        WP_CLI::line(esc_html__('Flushing the cache...', 'autoptimize'));
29 29
         autoptimizeCache::clearall();
30
-        WP_CLI::success( esc_html__( 'Cache flushed.', 'autoptimize' ) );
30
+        WP_CLI::success(esc_html__('Cache flushed.', 'autoptimize'));
31 31
     }
32 32
 }
33 33
 
34
-WP_CLI::add_command( 'autoptimize', 'autoptimizeCLI' );
34
+WP_CLI::add_command('autoptimize', 'autoptimizeCLI');
Please login to merge, or discard this patch.
classes/autoptimizePartners.php 1 patch
Spacing   +28 added lines, -28 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
  * addons and/or affiliate services.
5 5
  */
6 6
 
7
-if ( ! defined( 'ABSPATH' ) ) {
7
+if (!defined('ABSPATH')) {
8 8
     exit;
9 9
 }
10 10
 
@@ -17,64 +17,64 @@  discard block
 block discarded – undo
17 17
 
18 18
     public function run()
19 19
     {
20
-        if ( $this->enabled() ) {
21
-            add_filter( 'autoptimize_filter_settingsscreen_tabs', array( $this, 'add_partner_tabs' ), 10, 1 );
20
+        if ($this->enabled()) {
21
+            add_filter('autoptimize_filter_settingsscreen_tabs', array($this, 'add_partner_tabs'), 10, 1);
22 22
         }
23
-        add_action( 'admin_menu', array( $this, 'add_admin_menu' ) );
23
+        add_action('admin_menu', array($this, 'add_admin_menu'));
24 24
     }
25 25
 
26 26
     protected function enabled()
27 27
     {
28
-        return apply_filters( 'autoptimize_filter_show_partner_tabs', true );
28
+        return apply_filters('autoptimize_filter_show_partner_tabs', true);
29 29
     }
30 30
 
31
-    public function add_partner_tabs( $in )
31
+    public function add_partner_tabs($in)
32 32
     {
33
-        $in = array_merge( $in, array(
34
-            'ao_partners' => __( 'Optimize More!', 'autoptimize' ),
35
-        ) );
33
+        $in = array_merge($in, array(
34
+            'ao_partners' => __('Optimize More!', 'autoptimize'),
35
+        ));
36 36
 
37 37
         return $in;
38 38
     }
39 39
 
40 40
     public function add_admin_menu()
41 41
     {
42
-        if ( $this->enabled() ) {
43
-            add_submenu_page( null, 'AO partner', 'AO partner', 'manage_options', 'ao_partners', array( $this, 'ao_partners_page' ) );
42
+        if ($this->enabled()) {
43
+            add_submenu_page(null, 'AO partner', 'AO partner', 'manage_options', 'ao_partners', array($this, 'ao_partners_page'));
44 44
         }
45 45
     }
46 46
 
47 47
     protected function get_ao_partner_feed_markup()
48 48
     {
49
-        $no_feed_text = __( 'Have a look at <a href="http://optimizingmatters.com/">optimizingmatters.com</a> for Autoptimize power-ups!', 'autoptimize' );
49
+        $no_feed_text = __('Have a look at <a href="http://optimizingmatters.com/">optimizingmatters.com</a> for Autoptimize power-ups!', 'autoptimize');
50 50
         $output       = '';
51
-        if ( apply_filters( 'autoptimize_settingsscreen_remotehttp', true ) ) {
52
-            $rss      = fetch_feed( 'http://feeds.feedburner.com/OptimizingMattersDownloads' );
51
+        if (apply_filters('autoptimize_settingsscreen_remotehttp', true)) {
52
+            $rss      = fetch_feed('http://feeds.feedburner.com/OptimizingMattersDownloads');
53 53
             $maxitems = 0;
54 54
 
55
-            if ( ! is_wp_error( $rss ) ) {
56
-                $maxitems  = $rss->get_item_quantity( 20 );
57
-                $rss_items = $rss->get_items( 0, $maxitems );
55
+            if (!is_wp_error($rss)) {
56
+                $maxitems  = $rss->get_item_quantity(20);
57
+                $rss_items = $rss->get_items(0, $maxitems);
58 58
             }
59 59
 
60
-            if ( 0 == $maxitems ) {
60
+            if (0 == $maxitems) {
61 61
                 $output .= $no_feed_text;
62 62
             } else {
63 63
                 $output .= '<ul>';
64
-                foreach ( $rss_items as $item ) {
65
-                    $item_url  = esc_url( $item->get_permalink() );
64
+                foreach ($rss_items as $item) {
65
+                    $item_url  = esc_url($item->get_permalink());
66 66
                     $enclosure = $item->get_enclosure();
67 67
 
68 68
                     $output .= '<li class="itemDetail">';
69
-                    $output .= '<h3 class="itemTitle"><a href="' . $item_url . '" target="_blank">' . esc_html( $item->get_title() ) . '</a></h3>';
69
+                    $output .= '<h3 class="itemTitle"><a href="'.$item_url.'" target="_blank">'.esc_html($item->get_title()).'</a></h3>';
70 70
 
71
-                    if ( $enclosure && ( false !== strpos( $enclosure->get_type(), 'image' ) ) ) {
72
-                        $img_url = esc_url( $enclosure->get_link() );
73
-                        $output .= '<div class="itemImage"><a href="' . $item_url . '" target="_blank"><img src="' . $img_url . '"></a></div>';
71
+                    if ($enclosure && (false !== strpos($enclosure->get_type(), 'image'))) {
72
+                        $img_url = esc_url($enclosure->get_link());
73
+                        $output .= '<div class="itemImage"><a href="'.$item_url.'" target="_blank"><img src="'.$img_url.'"></a></div>';
74 74
                     }
75 75
 
76
-                    $output .= '<div class="itemDescription">' . wp_kses_post( $item->get_description() ) . '</div>';
77
-                    $output .= '<div class="itemButtonRow"><div class="itemButton button-secondary"><a href="' . $item_url . '" target="_blank">' . __( 'More info', 'autoptimize' ) . '</a></div></div>';
76
+                    $output .= '<div class="itemDescription">'.wp_kses_post($item->get_description()).'</div>';
77
+                    $output .= '<div class="itemButtonRow"><div class="itemButton button-secondary"><a href="'.$item_url.'" target="_blank">'.__('More info', 'autoptimize').'</a></div></div>';
78 78
                     $output .= '</li>';
79 79
                 }
80 80
                 $output .= '</ul>';
@@ -133,9 +133,9 @@  discard block
 block discarded – undo
133 133
     }
134 134
     </style>
135 135
     <div class="wrap">
136
-        <h1><?php _e( 'Autoptimize Settings', 'autoptimize' ); ?></h1>
136
+        <h1><?php _e('Autoptimize Settings', 'autoptimize'); ?></h1>
137 137
         <?php echo autoptimizeConfig::ao_admin_tabs(); ?>
138
-        <?php echo '<h2>' . __( "These Autoptimize power-ups and related services will improve your site's performance even more!", 'autoptimize' ) . '</h2>'; ?>
138
+        <?php echo '<h2>'.__("These Autoptimize power-ups and related services will improve your site's performance even more!", 'autoptimize').'</h2>'; ?>
139 139
         <div>
140 140
             <?php echo $this->get_ao_partner_feed_markup(); ?>
141 141
         </div>
Please login to merge, or discard this patch.
classes/autoptimizeCSSmin.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * Thin wrapper around css minifiers to avoid rewriting a bunch of existing code.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
@@ -21,9 +21,9 @@  discard block
 block discarded – undo
21 21
      *
22 22
      * @param bool $raise_limits Whether to raise memory limits or not. Default true.
23 23
      */
24
-    public function __construct( $raise_limits = true )
24
+    public function __construct($raise_limits = true)
25 25
     {
26
-        $this->minifier = new Autoptimize\tubalmartin\CssMin\Minifier( $raise_limits );
26
+        $this->minifier = new Autoptimize\tubalmartin\CssMin\Minifier($raise_limits);
27 27
     }
28 28
 
29 29
     /**
@@ -34,9 +34,9 @@  discard block
 block discarded – undo
34 34
      *
35 35
      * @return string
36 36
      */
37
-    public function run( $css )
37
+    public function run($css)
38 38
     {
39
-        $result = $this->minifier->run( $css );
39
+        $result = $this->minifier->run($css);
40 40
 
41 41
         return $result;
42 42
     }
@@ -48,10 +48,10 @@  discard block
 block discarded – undo
48 48
      *
49 49
      * @return string
50 50
      */
51
-    public static function minify( $css )
51
+    public static function minify($css)
52 52
     {
53 53
         $minifier = new self();
54 54
 
55
-        return $minifier->run( $css );
55
+        return $minifier->run($css);
56 56
     }
57 57
 }
Please login to merge, or discard this patch.
classes/autoptimizeSpeedupper.php 1 patch
Spacing   +41 added lines, -41 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
  * new in Autoptimize 2.2
5 5
  */
6 6
 
7
-if ( ! defined( 'ABSPATH' ) ) {
7
+if (!defined('ABSPATH')) {
8 8
     exit;
9 9
 }
10 10
 
@@ -17,94 +17,94 @@  discard block
 block discarded – undo
17 17
 
18 18
     public function add_hooks()
19 19
     {
20
-        if ( apply_filters( 'autoptimize_js_do_minify', true ) ) {
21
-            add_filter( 'autoptimize_js_individual_script', array( $this, 'js_snippetcacher' ), 10, 2 );
22
-            add_filter( 'autoptimize_js_after_minify', array( $this, 'js_cleanup' ), 10, 1 );
20
+        if (apply_filters('autoptimize_js_do_minify', true)) {
21
+            add_filter('autoptimize_js_individual_script', array($this, 'js_snippetcacher'), 10, 2);
22
+            add_filter('autoptimize_js_after_minify', array($this, 'js_cleanup'), 10, 1);
23 23
         }
24
-        if ( apply_filters( 'autoptimize_css_do_minify', true ) ) {
25
-            add_filter( 'autoptimize_css_individual_style', array( $this, 'css_snippetcacher' ), 10, 2 );
26
-            add_filter( 'autoptimize_css_after_minify', array( $this, 'css_cleanup' ), 10, 1 );
24
+        if (apply_filters('autoptimize_css_do_minify', true)) {
25
+            add_filter('autoptimize_css_individual_style', array($this, 'css_snippetcacher'), 10, 2);
26
+            add_filter('autoptimize_css_after_minify', array($this, 'css_cleanup'), 10, 1);
27 27
         }
28 28
     }
29 29
 
30
-    public function js_snippetcacher( $jsin, $jsfilename )
30
+    public function js_snippetcacher($jsin, $jsfilename)
31 31
     {
32
-        $md5hash = 'snippet_' . md5( $jsin );
33
-        $ccheck  = new autoptimizeCache( $md5hash, 'js' );
34
-        if ( $ccheck->check() ) {
32
+        $md5hash = 'snippet_'.md5($jsin);
33
+        $ccheck  = new autoptimizeCache($md5hash, 'js');
34
+        if ($ccheck->check()) {
35 35
             $scriptsrc = $ccheck->retrieve();
36 36
         } else {
37
-            if ( false === ( strpos( $jsfilename, 'min.js' ) ) && ( false === strpos( $jsfilename, 'js/jquery/jquery.js' ) ) && ( str_replace( apply_filters( 'autoptimize_filter_js_consider_minified', false ), '', $jsfilename ) === $jsfilename ) ) {
38
-                $tmp_jscode = trim( JSMin::minify( $jsin ) );
39
-                if ( ! empty( $tmp_jscode ) ) {
37
+            if (false === (strpos($jsfilename, 'min.js')) && (false === strpos($jsfilename, 'js/jquery/jquery.js')) && (str_replace(apply_filters('autoptimize_filter_js_consider_minified', false), '', $jsfilename) === $jsfilename)) {
38
+                $tmp_jscode = trim(JSMin::minify($jsin));
39
+                if (!empty($tmp_jscode)) {
40 40
                     $scriptsrc = $tmp_jscode;
41
-                    unset( $tmp_jscode );
41
+                    unset($tmp_jscode);
42 42
                 } else {
43 43
                     $scriptsrc = $jsin;
44 44
                 }
45 45
             } else {
46 46
                 // Removing comments, linebreaks and stuff!
47
-                $scriptsrc = preg_replace( '#^\s*\/\/.*$#Um', '', $jsin );
48
-                $scriptsrc = preg_replace( '#^\s*\/\*[^!].*\*\/\s?#Us', '', $scriptsrc );
49
-                $scriptsrc = preg_replace( "#(^[\r\n]*|[\r\n]+)[\s\t]*[\r\n]+#", "\n", $scriptsrc );
47
+                $scriptsrc = preg_replace('#^\s*\/\/.*$#Um', '', $jsin);
48
+                $scriptsrc = preg_replace('#^\s*\/\*[^!].*\*\/\s?#Us', '', $scriptsrc);
49
+                $scriptsrc = preg_replace("#(^[\r\n]*|[\r\n]+)[\s\t]*[\r\n]+#", "\n", $scriptsrc);
50 50
             }
51 51
 
52
-            $last_char = substr( $scriptsrc, -1, 1 );
53
-            if ( ';' !== $last_char && '}' !== $last_char ) {
52
+            $last_char = substr($scriptsrc, -1, 1);
53
+            if (';' !== $last_char && '}' !== $last_char) {
54 54
                 $scriptsrc .= ';';
55 55
             }
56 56
 
57
-            if ( ! empty( $jsfilename ) && str_replace( apply_filters( 'autoptimize_filter_js_speedup_cache', false ), '', $jsfilename ) === $jsfilename ) {
57
+            if (!empty($jsfilename) && str_replace(apply_filters('autoptimize_filter_js_speedup_cache', false), '', $jsfilename) === $jsfilename) {
58 58
                 // Don't cache inline CSS or if filter says no!
59
-                $ccheck->cache( $scriptsrc, 'text/javascript' );
59
+                $ccheck->cache($scriptsrc, 'text/javascript');
60 60
             }
61 61
         }
62
-        unset( $ccheck );
62
+        unset($ccheck);
63 63
 
64 64
         return $scriptsrc;
65 65
     }
66 66
 
67
-    public function css_snippetcacher( $cssin, $cssfilename )
67
+    public function css_snippetcacher($cssin, $cssfilename)
68 68
     {
69
-        $md5hash = 'snippet_' . md5( $cssin );
70
-        $ccheck  = new autoptimizeCache( $md5hash, 'css' );
71
-        if ( $ccheck->check() ) {
69
+        $md5hash = 'snippet_'.md5($cssin);
70
+        $ccheck  = new autoptimizeCache($md5hash, 'css');
71
+        if ($ccheck->check()) {
72 72
             $stylesrc = $ccheck->retrieve();
73 73
         } else {
74
-            if ( ( false === strpos( $cssfilename, 'min.css' ) ) && ( str_replace( apply_filters( 'autoptimize_filter_css_consider_minified', false ), '', $cssfilename ) === $cssfilename ) ) {
74
+            if ((false === strpos($cssfilename, 'min.css')) && (str_replace(apply_filters('autoptimize_filter_css_consider_minified', false), '', $cssfilename) === $cssfilename)) {
75 75
                 $cssmin   = new autoptimizeCSSmin();
76
-                $tmp_code = trim( $cssmin->run( $cssin ) );
76
+                $tmp_code = trim($cssmin->run($cssin));
77 77
 
78
-                if ( ! empty( $tmp_code ) ) {
78
+                if (!empty($tmp_code)) {
79 79
                     $stylesrc = $tmp_code;
80
-                    unset( $tmp_code );
80
+                    unset($tmp_code);
81 81
                 } else {
82 82
                     $stylesrc = $cssin;
83 83
                 }
84 84
             } else {
85 85
                 // .min.css -> no heavy-lifting, just some cleanup!
86
-                $stylesrc = preg_replace( '#^\s*\/\*[^!].*\*\/\s?#Us', '', $cssin );
87
-                $stylesrc = preg_replace( "#(^[\r\n]*|[\r\n]+)[\s\t]*[\r\n]+#", "\n", $stylesrc );
88
-                $stylesrc = autoptimizeStyles::fixurls( $cssfilename, $stylesrc );
86
+                $stylesrc = preg_replace('#^\s*\/\*[^!].*\*\/\s?#Us', '', $cssin);
87
+                $stylesrc = preg_replace("#(^[\r\n]*|[\r\n]+)[\s\t]*[\r\n]+#", "\n", $stylesrc);
88
+                $stylesrc = autoptimizeStyles::fixurls($cssfilename, $stylesrc);
89 89
             }
90
-            if ( ! empty( $cssfilename ) && ( str_replace( apply_filters( 'autoptimize_filter_css_speedup_cache', false ), '', $cssfilename ) === $cssfilename ) ) {
90
+            if (!empty($cssfilename) && (str_replace(apply_filters('autoptimize_filter_css_speedup_cache', false), '', $cssfilename) === $cssfilename)) {
91 91
                 // Only caching CSS if it's not inline and is allowed by filter!
92
-                $ccheck->cache( $stylesrc, 'text/css' );
92
+                $ccheck->cache($stylesrc, 'text/css');
93 93
             }
94 94
         }
95
-        unset( $ccheck );
95
+        unset($ccheck);
96 96
 
97 97
         return $stylesrc;
98 98
     }
99 99
 
100
-    public function css_cleanup( $cssin )
100
+    public function css_cleanup($cssin)
101 101
     {
102 102
         // Speedupper results in aggregated CSS not being minified, so the filestart-marker AO adds when aggregating needs to be removed.
103
-        return trim( str_replace( array( '/*FILESTART*/', '/*FILESTART2*/' ), '', $cssin ) );
103
+        return trim(str_replace(array('/*FILESTART*/', '/*FILESTART2*/'), '', $cssin));
104 104
     }
105 105
 
106
-    public function js_cleanup( $jsin )
106
+    public function js_cleanup($jsin)
107 107
     {
108
-        return trim( $jsin );
108
+        return trim($jsin);
109 109
     }
110 110
 }
Please login to merge, or discard this patch.
classes/autoptimizeToolbar.php 1 patch
Spacing   +49 added lines, -49 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * Handles toolbar-related stuff.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
@@ -12,32 +12,32 @@  discard block
 block discarded – undo
12 12
     public function __construct()
13 13
     {
14 14
         // If Cache is not available we don't add the toolbar.
15
-        if ( ! autoptimizeCache::cacheavail() ) {
15
+        if (!autoptimizeCache::cacheavail()) {
16 16
             return;
17 17
         }
18 18
 
19 19
         // Load admin toolbar feature once WordPress, all plugins, and the theme are fully loaded and instantiated.
20
-        add_action( 'wp_loaded', array( $this, 'load_toolbar' ) );
20
+        add_action('wp_loaded', array($this, 'load_toolbar'));
21 21
     }
22 22
 
23 23
     public function load_toolbar()
24 24
     {
25 25
         // Check permissions and that toolbar is not hidden via filter.
26
-        if ( current_user_can( 'manage_options' ) && apply_filters( 'autoptimize_filter_toolbar_show', true ) ) {
26
+        if (current_user_can('manage_options') && apply_filters('autoptimize_filter_toolbar_show', true)) {
27 27
 
28 28
             // Create a handler for the AJAX toolbar requests.
29
-            add_action( 'wp_ajax_autoptimize_delete_cache', array( $this, 'delete_cache' ) );
29
+            add_action('wp_ajax_autoptimize_delete_cache', array($this, 'delete_cache'));
30 30
 
31 31
             // Load custom styles, scripts and menu only when needed.
32
-            if ( is_admin_bar_showing() ) {
33
-                if ( is_admin() ) {
34
-                    add_action( 'admin_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
32
+            if (is_admin_bar_showing()) {
33
+                if (is_admin()) {
34
+                    add_action('admin_enqueue_scripts', array($this, 'enqueue_scripts'));
35 35
                 } else {
36
-                    add_action( 'wp_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
36
+                    add_action('wp_enqueue_scripts', array($this, 'enqueue_scripts'));
37 37
                 }
38 38
 
39 39
                 // Add the Autoptimize Toolbar to the Admin bar.
40
-                add_action( 'admin_bar_menu', array( $this, 'add_toolbar' ), 100 );
40
+                add_action('admin_bar_menu', array($this, 'add_toolbar'), 100);
41 41
             }
42 42
         }
43 43
     }
@@ -50,17 +50,17 @@  discard block
 block discarded – undo
50 50
         $stats = autoptimizeCache::stats();
51 51
 
52 52
         // Set the Max Size recommended for cache files.
53
-        $max_size = apply_filters( 'autoptimize_filter_cachecheck_maxsize', 512 * 1024 * 1024 );
53
+        $max_size = apply_filters('autoptimize_filter_cachecheck_maxsize', 512*1024*1024);
54 54
 
55 55
         // Retrieve the current Total Files in cache.
56 56
         $files = $stats[0];
57 57
         // Retrieve the current Total Size of the cache.
58 58
         $bytes = $stats[1];
59
-        $size  = $this->format_filesize( $bytes );
59
+        $size  = $this->format_filesize($bytes);
60 60
 
61 61
         // Calculate the percentage of cache used.
62
-        $percentage = ceil( $bytes / $max_size * 100 );
63
-        if ( $percentage > 100 ) {
62
+        $percentage = ceil($bytes/$max_size*100);
63
+        if ($percentage > 100) {
64 64
             $percentage = 100;
65 65
         }
66 66
 
@@ -70,81 +70,81 @@  discard block
 block discarded – undo
70 70
          * - "orange" if over 80%.
71 71
          * - "red" if over 100%.
72 72
          */
73
-        $color = ( 100 == $percentage ) ? 'red' : ( ( $percentage > 80 ) ? 'orange' : 'green' );
73
+        $color = (100 == $percentage) ? 'red' : (($percentage > 80) ? 'orange' : 'green');
74 74
 
75 75
         // Create or add new items into the Admin Toolbar.
76 76
         // Main "Autoptimize" node.
77
-        $wp_admin_bar->add_node( array(
77
+        $wp_admin_bar->add_node(array(
78 78
             'id'    => 'autoptimize',
79
-            'title' => '<span class="ab-icon"></span><span class="ab-label">' . __( 'Autoptimize', 'autoptimize' ) . '</span>',
80
-            'href'  => admin_url( 'options-general.php?page=autoptimize' ),
81
-            'meta'  => array( 'class' => 'bullet-' . $color ),
79
+            'title' => '<span class="ab-icon"></span><span class="ab-label">'.__('Autoptimize', 'autoptimize').'</span>',
80
+            'href'  => admin_url('options-general.php?page=autoptimize'),
81
+            'meta'  => array('class' => 'bullet-'.$color),
82 82
         ));
83 83
 
84 84
         // "Cache Info" node.
85
-        $wp_admin_bar->add_node( array(
85
+        $wp_admin_bar->add_node(array(
86 86
             'id'     => 'autoptimize-cache-info',
87
-            'title'  => '<p>' . __( 'Cache Info', 'autoptimize' ) . '</p>' .
88
-                        '<div class="autoptimize-radial-bar" percentage="' . $percentage . '">' .
89
-                        '<div class="autoptimize-circle">' .
90
-                        '<div class="mask full"><div class="fill bg-' . $color . '"></div></div>' .
91
-                        '<div class="mask half"><div class="fill bg-' . $color . '"></div></div>' .
92
-                        '<div class="shadow"></div>' .
93
-                        '</div>' .
94
-                        '<div class="inset"><div class="percentage"><div class="numbers ' . $color . '">' . $percentage . '%</div></div></div>' .
95
-                        '</div>' .
96
-                        '<table>' .
97
-                        '<tr><td>' . __( 'Size', 'autoptimize' ) . ':</td><td class="size ' . $color . '">' . $size . '</td></tr>' .
98
-                        '<tr><td>' . __( 'Files', 'autoptimize' ) . ':</td><td class="files white">' . $files . '</td></tr>' .
87
+            'title'  => '<p>'.__('Cache Info', 'autoptimize').'</p>'.
88
+                        '<div class="autoptimize-radial-bar" percentage="'.$percentage.'">'.
89
+                        '<div class="autoptimize-circle">'.
90
+                        '<div class="mask full"><div class="fill bg-'.$color.'"></div></div>'.
91
+                        '<div class="mask half"><div class="fill bg-'.$color.'"></div></div>'.
92
+                        '<div class="shadow"></div>'.
93
+                        '</div>'.
94
+                        '<div class="inset"><div class="percentage"><div class="numbers '.$color.'">'.$percentage.'%</div></div></div>'.
95
+                        '</div>'.
96
+                        '<table>'.
97
+                        '<tr><td>'.__('Size', 'autoptimize').':</td><td class="size '.$color.'">'.$size.'</td></tr>'.
98
+                        '<tr><td>'.__('Files', 'autoptimize').':</td><td class="files white">'.$files.'</td></tr>'.
99 99
                         '</table>',
100 100
             'parent' => 'autoptimize',
101 101
         ));
102 102
 
103 103
         // "Delete Cache" node.
104
-        $wp_admin_bar->add_node( array(
104
+        $wp_admin_bar->add_node(array(
105 105
             'id'     => 'autoptimize-delete-cache',
106
-            'title'  => __( 'Delete Cache', 'autoptimize' ),
106
+            'title'  => __('Delete Cache', 'autoptimize'),
107 107
             'parent' => 'autoptimize',
108 108
         ));
109 109
     }
110 110
 
111 111
     public function delete_cache()
112 112
     {
113
-        check_ajax_referer( 'ao_delcache_nonce', 'nonce' );
113
+        check_ajax_referer('ao_delcache_nonce', 'nonce');
114 114
 
115 115
         $result = false;
116
-        if ( current_user_can( 'manage_options' ) ) {
116
+        if (current_user_can('manage_options')) {
117 117
             // We call the function for cleaning the Autoptimize cache.
118 118
             $result = autoptimizeCache::clearall();
119 119
         }
120 120
 
121
-        wp_send_json( $result );
121
+        wp_send_json($result);
122 122
     }
123 123
 
124 124
     public function enqueue_scripts()
125 125
     {
126 126
         // Autoptimize Toolbar Styles.
127
-        wp_enqueue_style( 'autoptimize-toolbar', plugins_url( '/static/toolbar.css', __FILE__ ), array(), AUTOPTIMIZE_PLUGIN_VERSION, 'all' );
127
+        wp_enqueue_style('autoptimize-toolbar', plugins_url('/static/toolbar.css', __FILE__), array(), AUTOPTIMIZE_PLUGIN_VERSION, 'all');
128 128
 
129 129
         // Autoptimize Toolbar Javascript.
130
-        wp_enqueue_script( 'autoptimize-toolbar', plugins_url( '/static/toolbar.js', __FILE__ ), array( 'jquery' ), AUTOPTIMIZE_PLUGIN_VERSION, true );
130
+        wp_enqueue_script('autoptimize-toolbar', plugins_url('/static/toolbar.js', __FILE__), array('jquery'), AUTOPTIMIZE_PLUGIN_VERSION, true);
131 131
 
132 132
         // Localizes a registered script with data for a JavaScript variable.
133 133
         // Needed for the AJAX to work properly on the frontend.
134
-        wp_localize_script( 'autoptimize-toolbar', 'autoptimize_ajax_object', array(
135
-            'ajaxurl'     => admin_url( 'admin-ajax.php' ),
136
-            'error_msg'   => sprintf( __( 'Your Autoptimize cache might not have been purged successfully, please check on the <a href=%s>Autoptimize settings page</a>.', 'autoptimize' ), admin_url( 'options-general.php?page=autoptimize' ) . ' style="white-space:nowrap;"' ),
137
-            'dismiss_msg' => __( 'Dismiss this notice.' ),
138
-            'nonce'       => wp_create_nonce( 'ao_delcache_nonce' ),
139
-        ) );
134
+        wp_localize_script('autoptimize-toolbar', 'autoptimize_ajax_object', array(
135
+            'ajaxurl'     => admin_url('admin-ajax.php'),
136
+            'error_msg'   => sprintf(__('Your Autoptimize cache might not have been purged successfully, please check on the <a href=%s>Autoptimize settings page</a>.', 'autoptimize'), admin_url('options-general.php?page=autoptimize').' style="white-space:nowrap;"'),
137
+            'dismiss_msg' => __('Dismiss this notice.'),
138
+            'nonce'       => wp_create_nonce('ao_delcache_nonce'),
139
+        ));
140 140
     }
141 141
 
142
-    public function format_filesize( $bytes, $decimals = 2 )
142
+    public function format_filesize($bytes, $decimals = 2)
143 143
     {
144
-        $units = array( 'B', 'KB', 'MB', 'GB', 'TB', 'PB', 'EB', 'ZB', 'YB' );
144
+        $units = array('B', 'KB', 'MB', 'GB', 'TB', 'PB', 'EB', 'ZB', 'YB');
145 145
 
146
-        for ( $i = 0; ( $bytes / 1024) > 0.9; $i++, $bytes /= 1024 ) {} // @codingStandardsIgnoreLine
146
+        for ($i = 0; ($bytes/1024) > 0.9; $i++, $bytes /= 1024) {} // @codingStandardsIgnoreLine
147 147
 
148
-        return sprintf( "%1.{$decimals}f %s", round( $bytes, $decimals ), $units[ $i ] );
148
+        return sprintf("%1.{$decimals}f %s", round($bytes, $decimals), $units[$i]);
149 149
     }
150 150
 }
Please login to merge, or discard this patch.
classes/autoptimizeStyles.php 2 patches
Doc Comments   +19 added lines, -2 removed lines patch added patch discarded remove patch
@@ -282,7 +282,7 @@  discard block
 block discarded – undo
282 282
      * be inlined into a data: URI. Defaults to 4096, passed through
283 283
      * `autoptimize_filter_css_datauri_maxsize` filter.
284 284
      *
285
-     * @return mixed
285
+     * @return integer|null
286 286
      */
287 287
     private function get_datauri_maxsize()
288 288
     {
@@ -303,6 +303,9 @@  discard block
 block discarded – undo
303 303
         return $max_size;
304 304
     }
305 305
 
306
+    /**
307
+     * @param false|string $url
308
+     */
306 309
     private function check_datauri_exclude_list( $url )
307 310
     {
308 311
         static $exclude_list = null;
@@ -328,6 +331,9 @@  discard block
 block discarded – undo
328 331
         return $matched;
329 332
     }
330 333
 
334
+    /**
335
+     * @param false|string $path
336
+     */
331 337
     private function build_or_get_datauri_image( $path )
332 338
     {
333 339
         /**
@@ -746,6 +752,11 @@  discard block
 block discarded – undo
746 752
         return true;
747 753
     }
748 754
 
755
+    /**
756
+     * @param string $code
757
+     *
758
+     * @return string
759
+     */
749 760
     public function run_minifier_on( $code )
750 761
     {
751 762
         if ( ! $this->alreadyminified ) {
@@ -945,6 +956,9 @@  discard block
 block discarded – undo
945 956
         return $code;
946 957
     }
947 958
 
959
+    /**
960
+     * @param string $tag
961
+     */
948 962
     private function ismovable( $tag )
949 963
     {
950 964
         if ( ! $this->aggregate ) {
@@ -1005,7 +1019,7 @@  discard block
 block discarded – undo
1005 1019
      * @param string $filepath Filepath.
1006 1020
      * @param bool   $cache_miss Optional. Force a cache miss. Default false.
1007 1021
      *
1008
-     * @return bool|string Url pointing to the minified css file or false.
1022
+     * @return false|string Url pointing to the minified css file or false.
1009 1023
      */
1010 1024
     public function minify_single( $filepath, $cache_miss = false )
1011 1025
     {
@@ -1057,6 +1071,9 @@  discard block
 block discarded – undo
1057 1071
         $this->options = $options;
1058 1072
     }
1059 1073
 
1074
+    /**
1075
+     * @param string $name
1076
+     */
1060 1077
     public function setOption( $name, $value )
1061 1078
     {
1062 1079
         $this->options[$name] = $value;
Please login to merge, or discard this patch.
Spacing   +310 added lines, -310 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * Class for CSS optimization.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
@@ -48,53 +48,53 @@  discard block
 block discarded – undo
48 48
     // public $cdn_url; // Used all over the place implicitly, so will have to be either public or protected :/ .
49 49
 
50 50
     // Reads the page and collects style tags.
51
-    public function read( $options )
51
+    public function read($options)
52 52
     {
53
-        $noptimizeCSS = apply_filters( 'autoptimize_filter_css_noptimize', false, $this->content );
54
-        if ( $noptimizeCSS ) {
53
+        $noptimizeCSS = apply_filters('autoptimize_filter_css_noptimize', false, $this->content);
54
+        if ($noptimizeCSS) {
55 55
             return false;
56 56
         }
57 57
 
58
-        $whitelistCSS = apply_filters( 'autoptimize_filter_css_whitelist', '', $this->content );
59
-        if ( ! empty( $whitelistCSS ) ) {
60
-            $this->whitelist = array_filter( array_map( 'trim', explode( ',', $whitelistCSS ) ) );
58
+        $whitelistCSS = apply_filters('autoptimize_filter_css_whitelist', '', $this->content);
59
+        if (!empty($whitelistCSS)) {
60
+            $this->whitelist = array_filter(array_map('trim', explode(',', $whitelistCSS)));
61 61
         }
62 62
 
63
-        $removableCSS = apply_filters( 'autoptimize_filter_css_removables', '' );
64
-        if ( ! empty( $removableCSS ) ) {
65
-            $this->cssremovables = array_filter( array_map( 'trim', explode( ',', $removableCSS ) ) );
63
+        $removableCSS = apply_filters('autoptimize_filter_css_removables', '');
64
+        if (!empty($removableCSS)) {
65
+            $this->cssremovables = array_filter(array_map('trim', explode(',', $removableCSS)));
66 66
         }
67 67
 
68
-        $this->cssinlinesize = apply_filters( 'autoptimize_filter_css_inlinesize', 256 );
68
+        $this->cssinlinesize = apply_filters('autoptimize_filter_css_inlinesize', 256);
69 69
 
70 70
         // filter to "late inject minified CSS", default to true for now (it is faster).
71
-        $this->inject_min_late = apply_filters( 'autoptimize_filter_css_inject_min_late', true );
71
+        $this->inject_min_late = apply_filters('autoptimize_filter_css_inject_min_late', true);
72 72
 
73 73
         // Remove everything that's not the header.
74
-        if ( apply_filters( 'autoptimize_filter_css_justhead', $options['justhead'] ) ) {
75
-            $content             = explode( '</head>', $this->content, 2 );
76
-            $this->content       = $content[0] . '</head>';
74
+        if (apply_filters('autoptimize_filter_css_justhead', $options['justhead'])) {
75
+            $content             = explode('</head>', $this->content, 2);
76
+            $this->content       = $content[0].'</head>';
77 77
             $this->restofcontent = $content[1];
78 78
         }
79 79
 
80 80
         // Determine whether we're doing CSS-files aggregation or not.
81
-        if ( isset( $options['aggregate'] ) && ! $options['aggregate'] ) {
81
+        if (isset($options['aggregate']) && !$options['aggregate']) {
82 82
             $this->aggregate = false;
83 83
         }
84 84
         // Returning true for "dontaggregate" turns off aggregation.
85
-        if ( $this->aggregate && apply_filters( 'autoptimize_filter_css_dontaggregate', false ) ) {
85
+        if ($this->aggregate && apply_filters('autoptimize_filter_css_dontaggregate', false)) {
86 86
             $this->aggregate = false;
87 87
         }
88 88
 
89 89
         // include inline?
90
-        if ( apply_filters( 'autoptimize_css_include_inline', $options['include_inline'] ) ) {
90
+        if (apply_filters('autoptimize_css_include_inline', $options['include_inline'])) {
91 91
             $this->include_inline = true;
92 92
         }
93 93
 
94 94
         // List of CSS strings which are excluded from autoptimization.
95
-        $excludeCSS = apply_filters( 'autoptimize_filter_css_exclude', $options['css_exclude'], $this->content );
96
-        if ( '' !== $excludeCSS ) {
97
-            $this->dontmove = array_filter( array_map( 'trim', explode( ',', $excludeCSS ) ) );
95
+        $excludeCSS = apply_filters('autoptimize_filter_css_exclude', $options['css_exclude'], $this->content);
96
+        if ('' !== $excludeCSS) {
97
+            $this->dontmove = array_filter(array_map('trim', explode(',', $excludeCSS)));
98 98
         } else {
99 99
             $this->dontmove = array();
100 100
         }
@@ -105,16 +105,16 @@  discard block
 block discarded – undo
105 105
         // Should we defer css?
106 106
         // value: true / false.
107 107
         $this->defer = $options['defer'];
108
-        $this->defer = apply_filters( 'autoptimize_filter_css_defer', $this->defer, $this->content );
108
+        $this->defer = apply_filters('autoptimize_filter_css_defer', $this->defer, $this->content);
109 109
 
110 110
         // Should we inline while deferring?
111 111
         // value: inlined CSS.
112
-        $this->defer_inline = apply_filters( 'autoptimize_filter_css_defer_inline', $options['defer_inline'], $this->content );
112
+        $this->defer_inline = apply_filters('autoptimize_filter_css_defer_inline', $options['defer_inline'], $this->content);
113 113
 
114 114
         // Should we inline?
115 115
         // value: true / false.
116 116
         $this->inline = $options['inline'];
117
-        $this->inline = apply_filters( 'autoptimize_filter_css_inline', $this->inline, $this->content );
117
+        $this->inline = apply_filters('autoptimize_filter_css_inline', $this->inline, $this->content);
118 118
 
119 119
         // Store cdn url.
120 120
         $this->cdn_url = $options['cdn_url'];
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
         $this->datauris = $options['datauris'];
124 124
 
125 125
         // noptimize me.
126
-        $this->content = $this->hide_noptimize( $this->content );
126
+        $this->content = $this->hide_noptimize($this->content);
127 127
 
128 128
         // Exclude (no)script, as those may contain CSS which should be left as is.
129 129
         $this->content = $this->replace_contents_with_marker_if_exists(
@@ -134,26 +134,26 @@  discard block
 block discarded – undo
134 134
         );
135 135
 
136 136
         // Save IE hacks.
137
-        $this->content = $this->hide_iehacks( $this->content );
137
+        $this->content = $this->hide_iehacks($this->content);
138 138
 
139 139
         // Hide HTML comments.
140
-        $this->content = $this->hide_comments( $this->content );
140
+        $this->content = $this->hide_comments($this->content);
141 141
 
142 142
         // Get <style> and <link>.
143
-        if ( preg_match_all( '#(<style[^>]*>.*</style>)|(<link[^>]*stylesheet[^>]*>)#Usmi', $this->content, $matches ) ) {
143
+        if (preg_match_all('#(<style[^>]*>.*</style>)|(<link[^>]*stylesheet[^>]*>)#Usmi', $this->content, $matches)) {
144 144
 
145
-            foreach ( $matches[0] as $tag ) {
146
-                if ( $this->isremovable( $tag, $this->cssremovables ) ) {
147
-                    $this->content = str_replace( $tag, '', $this->content );
148
-                } elseif ( $this->ismovable( $tag ) ) {
145
+            foreach ($matches[0] as $tag) {
146
+                if ($this->isremovable($tag, $this->cssremovables)) {
147
+                    $this->content = str_replace($tag, '', $this->content);
148
+                } elseif ($this->ismovable($tag)) {
149 149
                     // Get the media.
150
-                    if ( false !== strpos( $tag, 'media=' ) ) {
151
-                        preg_match( '#media=(?:"|\')([^>]*)(?:"|\')#Ui', $tag, $medias );
152
-                        $medias = explode( ',', $medias[1] );
150
+                    if (false !== strpos($tag, 'media=')) {
151
+                        preg_match('#media=(?:"|\')([^>]*)(?:"|\')#Ui', $tag, $medias);
152
+                        $medias = explode(',', $medias[1]);
153 153
                         $media = array();
154
-                        foreach ( $medias as $elem ) {
154
+                        foreach ($medias as $elem) {
155 155
                             /* $media[] = current(explode(' ',trim($elem),2)); */
156
-                            if ( empty( $elem ) ) {
156
+                            if (empty($elem)) {
157 157
                                 $elem = 'all';
158 158
                             }
159 159
 
@@ -161,65 +161,65 @@  discard block
 block discarded – undo
161 161
                         }
162 162
                     } else {
163 163
                         // No media specified - applies to all.
164
-                        $media = array( 'all' );
164
+                        $media = array('all');
165 165
                     }
166 166
 
167
-                    $media = apply_filters( 'autoptimize_filter_css_tagmedia', $media, $tag );
167
+                    $media = apply_filters('autoptimize_filter_css_tagmedia', $media, $tag);
168 168
 
169
-                    if ( preg_match( '#<link.*href=("|\')(.*)("|\')#Usmi', $tag, $source ) ) {
169
+                    if (preg_match('#<link.*href=("|\')(.*)("|\')#Usmi', $tag, $source)) {
170 170
                         // <link>.
171
-                        $url  = current( explode( '?', $source[2], 2 ) );
172
-                        $path = $this->getpath( $url );
171
+                        $url  = current(explode('?', $source[2], 2));
172
+                        $path = $this->getpath($url);
173 173
 
174
-                        if ( false !== $path && preg_match( '#\.css$#', $path ) ) {
174
+                        if (false !== $path && preg_match('#\.css$#', $path)) {
175 175
                             // Good link.
176
-                            $this->css[] = array( $media, $path );
176
+                            $this->css[] = array($media, $path);
177 177
                         } else {
178 178
                             // Link is dynamic (.php etc).
179
-                            $new_tag = $this->optionally_defer_excluded( $tag, 'none' );
180
-                            if ( $new_tag !== $tag ) {
181
-                                $this->content = str_replace( $tag, $new_tag, $this->content );
179
+                            $new_tag = $this->optionally_defer_excluded($tag, 'none');
180
+                            if ($new_tag !== $tag) {
181
+                                $this->content = str_replace($tag, $new_tag, $this->content);
182 182
                             }
183 183
                             $tag = '';
184 184
                         }
185 185
                     } else {
186 186
                         // Inline css in style tags can be wrapped in comment tags, so restore comments.
187
-                        $tag = $this->restore_comments( $tag );
188
-                        preg_match( '#<style.*>(.*)</style>#Usmi', $tag, $code );
187
+                        $tag = $this->restore_comments($tag);
188
+                        preg_match('#<style.*>(.*)</style>#Usmi', $tag, $code);
189 189
 
190 190
                         // And re-hide them to be able to to the removal based on tag.
191
-                        $tag = $this->hide_comments( $tag );
191
+                        $tag = $this->hide_comments($tag);
192 192
 
193
-                        if ( $this->include_inline ) {
194
-                            $code = preg_replace( '#^.*<!\[CDATA\[(?:\s*\*/)?(.*)(?://|/\*)\s*?\]\]>.*$#sm', '$1', $code[1] );
195
-                            $this->css[] = array( $media, 'INLINE;' . $code );
193
+                        if ($this->include_inline) {
194
+                            $code = preg_replace('#^.*<!\[CDATA\[(?:\s*\*/)?(.*)(?://|/\*)\s*?\]\]>.*$#sm', '$1', $code[1]);
195
+                            $this->css[] = array($media, 'INLINE;'.$code);
196 196
                         } else {
197 197
                             $tag = '';
198 198
                         }
199 199
                     }
200 200
 
201 201
                     // Remove the original style tag.
202
-                    $this->content = str_replace( $tag, '', $this->content );
202
+                    $this->content = str_replace($tag, '', $this->content);
203 203
                 } else {
204 204
                     // Excluded CSS, minify if getpath and filter says so...
205
-                    if ( preg_match( '#<link.*href=("|\')(.*)("|\')#Usmi', $tag, $source ) ) {
206
-                        $exploded_url = explode( '?', $source[2], 2 );
205
+                    if (preg_match('#<link.*href=("|\')(.*)("|\')#Usmi', $tag, $source)) {
206
+                        $exploded_url = explode('?', $source[2], 2);
207 207
                         $url          = $exploded_url[0];
208
-                        $path         = $this->getpath( $url );
208
+                        $path         = $this->getpath($url);
209 209
 
210
-                        if ( $path && apply_filters( 'autoptimize_filter_css_minify_excluded', true, $url ) ) {
211
-                            $minified_url = $this->minify_single( $path );
212
-                            if ( ! empty( $minified_url ) ) {
210
+                        if ($path && apply_filters('autoptimize_filter_css_minify_excluded', true, $url)) {
211
+                            $minified_url = $this->minify_single($path);
212
+                            if (!empty($minified_url)) {
213 213
                                 // Replace orig URL with cached minified URL.
214
-                                $new_tag = str_replace( $url, $minified_url, $tag );
214
+                                $new_tag = str_replace($url, $minified_url, $tag);
215 215
                             } else {
216 216
                                 $new_tag = $tag;
217 217
                             }
218 218
 
219
-                            $new_tag = $this->optionally_defer_excluded( $new_tag, $url );
219
+                            $new_tag = $this->optionally_defer_excluded($new_tag, $url);
220 220
 
221 221
                             // And replace!
222
-                            $this->content = str_replace( $tag, $new_tag, $this->content );
222
+                            $this->content = str_replace($tag, $new_tag, $this->content);
223 223
                         }
224 224
                     }
225 225
                 }
@@ -240,10 +240,10 @@  discard block
 block discarded – undo
240 240
      *
241 241
      * @return string $new_tag
242 242
      */
243
-    private function optionally_defer_excluded( $new_tag, $url = '' )
243
+    private function optionally_defer_excluded($new_tag, $url = '')
244 244
     {
245 245
         // Defer single CSS if "inline & defer" is ON and there is inline CSS.
246
-        if ( $this->defer && ! empty( $this->defer_inline ) ) {
246
+        if ($this->defer && !empty($this->defer_inline)) {
247 247
             // Get/ set (via filter) the JS to be triggers onload of the preloaded CSS.
248 248
             $_preload_onload = apply_filters(
249 249
                 'autoptimize_filter_css_preload_onload',
@@ -251,12 +251,12 @@  discard block
 block discarded – undo
251 251
                 $url
252 252
             );
253 253
             // Adapt original <link> element for CSS to be preloaded and add <noscript>-version for fallback.
254
-            $new_tag = '<noscript>' . $new_tag . '</noscript>' . str_replace(
254
+            $new_tag = '<noscript>'.$new_tag.'</noscript>'.str_replace(
255 255
                 array(
256 256
                     "rel='stylesheet'",
257 257
                     'rel="stylesheet"',
258 258
                 ),
259
-                "rel='preload' as='style' onload=\"" . $_preload_onload . "\"",
259
+                "rel='preload' as='style' onload=\"".$_preload_onload."\"",
260 260
                 $new_tag
261 261
             );
262 262
         }
@@ -270,22 +270,22 @@  discard block
 block discarded – undo
270 270
      * @param string $path
271 271
      * @return boolean
272 272
      */
273
-    private function is_datauri_candidate( $path )
273
+    private function is_datauri_candidate($path)
274 274
     {
275 275
         // Call only once since it's called from a loop.
276 276
         static $max_size = null;
277
-        if ( null === $max_size ) {
277
+        if (null === $max_size) {
278 278
             $max_size = $this->get_datauri_maxsize();
279 279
         }
280 280
 
281
-        if ( $path && preg_match( '#\.(jpe?g|png|gif|webp|bmp)$#i', $path ) &&
282
-            file_exists( $path ) && is_readable( $path ) && filesize( $path ) <= $max_size ) {
281
+        if ($path && preg_match('#\.(jpe?g|png|gif|webp|bmp)$#i', $path) &&
282
+            file_exists($path) && is_readable($path) && filesize($path) <= $max_size) {
283 283
 
284 284
             // Seems we have a candidate.
285 285
             $is_candidate = true;
286 286
         } else {
287 287
             // Filter allows overriding default decision (which checks for local file existence).
288
-            $is_candidate = apply_filters( 'autoptimize_filter_css_is_datauri_candidate', false, $path );
288
+            $is_candidate = apply_filters('autoptimize_filter_css_is_datauri_candidate', false, $path);
289 289
         }
290 290
 
291 291
         return $is_candidate;
@@ -310,29 +310,29 @@  discard block
 block discarded – undo
310 310
          * of thing you're probably better of building assets completely
311 311
          * outside of WordPress anyway.
312 312
          */
313
-        if ( null === $max_size ) {
314
-            $max_size = (int) apply_filters( 'autoptimize_filter_css_datauri_maxsize', 4096 );
313
+        if (null === $max_size) {
314
+            $max_size = (int) apply_filters('autoptimize_filter_css_datauri_maxsize', 4096);
315 315
         }
316 316
 
317 317
         return $max_size;
318 318
     }
319 319
 
320
-    private function check_datauri_exclude_list( $url )
320
+    private function check_datauri_exclude_list($url)
321 321
     {
322 322
         static $exclude_list = null;
323 323
         $no_datauris         = array();
324 324
 
325 325
         // Again, skip doing certain stuff repeatedly when loop-called.
326
-        if ( null === $exclude_list ) {
327
-            $exclude_list = apply_filters( 'autoptimize_filter_css_datauri_exclude', '' );
328
-            $no_datauris = array_filter( array_map( 'trim', explode( ',', $exclude_list ) ) );
326
+        if (null === $exclude_list) {
327
+            $exclude_list = apply_filters('autoptimize_filter_css_datauri_exclude', '');
328
+            $no_datauris = array_filter(array_map('trim', explode(',', $exclude_list)));
329 329
         }
330 330
 
331 331
         $matched = false;
332 332
 
333
-        if ( ! empty( $exclude_list ) ) {
334
-            foreach ( $no_datauris as $no_datauri ) {
335
-                if ( false !== strpos( $url, $no_datauri ) ) {
333
+        if (!empty($exclude_list)) {
334
+            foreach ($no_datauris as $no_datauri) {
335
+                if (false !== strpos($url, $no_datauri)) {
336 336
                     $matched = true;
337 337
                     break;
338 338
                 }
@@ -342,7 +342,7 @@  discard block
 block discarded – undo
342 342
         return $matched;
343 343
     }
344 344
 
345
-    private function build_or_get_datauri_image( $path )
345
+    private function build_or_get_datauri_image($path)
346 346
     {
347 347
         /**
348 348
          * TODO/FIXME: document the required return array format, or better yet,
@@ -351,27 +351,27 @@  discard block
 block discarded – undo
351 351
          */
352 352
 
353 353
         // Allows short-circuiting datauri generation for an image.
354
-        $result = apply_filters( 'autoptimize_filter_css_datauri_image', array(), $path );
355
-        if ( ! empty( $result ) ) {
356
-            if ( is_array( $result ) && isset( $result['full'] ) && isset( $result['base64data'] ) ) {
354
+        $result = apply_filters('autoptimize_filter_css_datauri_image', array(), $path);
355
+        if (!empty($result)) {
356
+            if (is_array($result) && isset($result['full']) && isset($result['base64data'])) {
357 357
                 return $result;
358 358
             }
359 359
         }
360 360
 
361
-        $hash = md5( $path );
362
-        $check = new autoptimizeCache( $hash, 'img' );
363
-        if ( $check->check() ) {
361
+        $hash = md5($path);
362
+        $check = new autoptimizeCache($hash, 'img');
363
+        if ($check->check()) {
364 364
             // we have the base64 image in cache.
365 365
             $headAndData = $check->retrieve();
366
-            $_base64data = explode( ';base64,', $headAndData );
366
+            $_base64data = explode(';base64,', $headAndData);
367 367
             $base64data  = $_base64data[1];
368
-            unset( $_base64data );
368
+            unset($_base64data);
369 369
         } else {
370 370
             // It's an image and we don't have it in cache, get the type by extension.
371
-            $exploded_path = explode( '.', $path );
372
-            $type = end( $exploded_path );
371
+            $exploded_path = explode('.', $path);
372
+            $type = end($exploded_path);
373 373
 
374
-            switch ( $type ) {
374
+            switch ($type) {
375 375
                 case 'jpg':
376 376
                 case 'jpeg':
377 377
                     $dataurihead = 'data:image/jpeg;base64,';
@@ -393,15 +393,15 @@  discard block
 block discarded – undo
393 393
             }
394 394
 
395 395
             // Encode the data.
396
-            $base64data  = base64_encode( file_get_contents( $path ) );
397
-            $headAndData = $dataurihead . $base64data;
396
+            $base64data  = base64_encode(file_get_contents($path));
397
+            $headAndData = $dataurihead.$base64data;
398 398
 
399 399
             // Save in cache.
400
-            $check->cache( $headAndData, 'text/plain' );
400
+            $check->cache($headAndData, 'text/plain');
401 401
         }
402
-        unset( $check );
402
+        unset($check);
403 403
 
404
-        return array( 'full' => $headAndData, 'base64data' => $base64data );
404
+        return array('full' => $headAndData, 'base64data' => $base64data);
405 405
     }
406 406
 
407 407
     /**
@@ -413,13 +413,13 @@  discard block
 block discarded – undo
413 413
      *
414 414
      * @return string
415 415
      */
416
-    protected static function replace_longest_matches_first( $string, $replacements = array() )
416
+    protected static function replace_longest_matches_first($string, $replacements = array())
417 417
     {
418
-        if ( ! empty( $replacements ) ) {
418
+        if (!empty($replacements)) {
419 419
             // Sort the replacements array by key length in desc order (so that the longest strings are replaced first).
420
-            $keys = array_map( 'strlen', array_keys( $replacements ) );
421
-            array_multisort( $keys, SORT_DESC, $replacements );
422
-            $string = str_replace( array_keys( $replacements ), array_values( $replacements ), $string );
420
+            $keys = array_map('strlen', array_keys($replacements));
421
+            array_multisort($keys, SORT_DESC, $replacements);
422
+            $string = str_replace(array_keys($replacements), array_values($replacements), $string);
423 423
         }
424 424
 
425 425
         return $string;
@@ -435,15 +435,15 @@  discard block
 block discarded – undo
435 435
      *
436 436
      * @return string
437 437
      */
438
-    public function replace_urls( $code = '' )
438
+    public function replace_urls($code = '')
439 439
     {
440 440
         $replacements = array();
441 441
 
442
-        preg_match_all( self::ASSETS_REGEX, $code, $url_src_matches );
443
-        if ( is_array( $url_src_matches ) && ! empty( $url_src_matches ) ) {
444
-            foreach ( $url_src_matches[1] as $count => $original_url ) {
442
+        preg_match_all(self::ASSETS_REGEX, $code, $url_src_matches);
443
+        if (is_array($url_src_matches) && !empty($url_src_matches)) {
444
+            foreach ($url_src_matches[1] as $count => $original_url) {
445 445
                 // Removes quotes and other cruft.
446
-                $url = trim( $original_url, " \t\n\r\0\x0B\"'" );
446
+                $url = trim($original_url, " \t\n\r\0\x0B\"'");
447 447
 
448 448
                 /**
449 449
                  * TODO/FIXME: Add a way for other code / callable to be called here
@@ -461,17 +461,17 @@  discard block
 block discarded – undo
461 461
                  */
462 462
 
463 463
                 // Do CDN replacement if needed.
464
-                if ( ! empty( $this->cdn_url ) ) {
465
-                    $replacement_url = $this->url_replace_cdn( $url );
464
+                if (!empty($this->cdn_url)) {
465
+                    $replacement_url = $this->url_replace_cdn($url);
466 466
                     // Prepare replacements array.
467
-                    $replacements[ $url_src_matches[1][ $count ] ] = str_replace(
467
+                    $replacements[$url_src_matches[1][$count]] = str_replace(
468 468
                         $original_url, $replacement_url, $url_src_matches[1][$count]
469 469
                     );
470 470
                 }
471 471
             }
472 472
         }
473 473
 
474
-        $code = self::replace_longest_matches_first( $code, $replacements );
474
+        $code = self::replace_longest_matches_first($code, $replacements);
475 475
 
476 476
         return $code;
477 477
     }
@@ -484,26 +484,26 @@  discard block
 block discarded – undo
484 484
      * @param string $code
485 485
      * @return string
486 486
      */
487
-    public function hide_fontface_and_maybe_cdn( $code )
487
+    public function hide_fontface_and_maybe_cdn($code)
488 488
     {
489 489
         // Proceed only if @font-face declarations exist within $code.
490
-        preg_match_all( self::FONT_FACE_REGEX, $code, $fontfaces );
491
-        if ( isset( $fontfaces[0] ) ) {
490
+        preg_match_all(self::FONT_FACE_REGEX, $code, $fontfaces);
491
+        if (isset($fontfaces[0])) {
492 492
             // Check if we need to cdn fonts or not.
493
-            $do_font_cdn = apply_filters( 'autoptimize_filter_css_fonts_cdn', false );
493
+            $do_font_cdn = apply_filters('autoptimize_filter_css_fonts_cdn', false);
494 494
 
495
-            foreach ( $fontfaces[0] as $full_match ) {
495
+            foreach ($fontfaces[0] as $full_match) {
496 496
                 // Keep original match so we can search/replace it.
497 497
                 $match_search = $full_match;
498 498
 
499 499
                 // Do font cdn if needed.
500
-                if ( $do_font_cdn ) {
501
-                    $full_match = $this->replace_urls( $full_match );
500
+                if ($do_font_cdn) {
501
+                    $full_match = $this->replace_urls($full_match);
502 502
                 }
503 503
 
504 504
                 // Replace declaration with its base64 encoded string.
505
-                $replacement = self::build_marker( 'FONTFACE', $full_match );
506
-                $code = str_replace( $match_search, $replacement, $code );
505
+                $replacement = self::build_marker('FONTFACE', $full_match);
506
+                $code = str_replace($match_search, $replacement, $code);
507 507
             }
508 508
         }
509 509
 
@@ -517,16 +517,16 @@  discard block
 block discarded – undo
517 517
      * @param string $code
518 518
      * @return string
519 519
      */
520
-    public function restore_fontface( $code )
520
+    public function restore_fontface($code)
521 521
     {
522
-        return $this->restore_marked_content( 'FONTFACE', $code );
522
+        return $this->restore_marked_content('FONTFACE', $code);
523 523
     }
524 524
 
525 525
     // Re-write (and/or inline) referenced assets.
526
-    public function rewrite_assets( $code )
526
+    public function rewrite_assets($code)
527 527
     {
528 528
         // Handle @font-face rules by hiding and processing them separately.
529
-        $code = $this->hide_fontface_and_maybe_cdn( $code );
529
+        $code = $this->hide_fontface_and_maybe_cdn($code);
530 530
 
531 531
         /**
532 532
          * TODO/FIXME:
@@ -542,30 +542,30 @@  discard block
 block discarded – undo
542 542
         $url_src_matches = array();
543 543
         $imgreplace = array();
544 544
         // Matches and captures anything specified within the literal `url()` and excludes those containing data: URIs.
545
-        preg_match_all( self::ASSETS_REGEX, $code, $url_src_matches );
546
-        if ( is_array( $url_src_matches ) && ! empty( $url_src_matches ) ) {
547
-            foreach ( $url_src_matches[1] as $count => $original_url ) {
545
+        preg_match_all(self::ASSETS_REGEX, $code, $url_src_matches);
546
+        if (is_array($url_src_matches) && !empty($url_src_matches)) {
547
+            foreach ($url_src_matches[1] as $count => $original_url) {
548 548
                 // Removes quotes and other cruft.
549
-                $url = trim( $original_url, " \t\n\r\0\x0B\"'" );
549
+                $url = trim($original_url, " \t\n\r\0\x0B\"'");
550 550
 
551 551
                 // If datauri inlining is turned on, do it.
552 552
                 $inlined = false;
553
-                if ( $this->datauris ) {
553
+                if ($this->datauris) {
554 554
                     $iurl = $url;
555
-                    if ( false !== strpos( $iurl, '?' ) ) {
556
-                        $iurl = strtok( $iurl, '?' );
555
+                    if (false !== strpos($iurl, '?')) {
556
+                        $iurl = strtok($iurl, '?');
557 557
                     }
558 558
 
559
-                    $ipath = $this->getpath( $iurl );
559
+                    $ipath = $this->getpath($iurl);
560 560
 
561
-                    $excluded = $this->check_datauri_exclude_list( $ipath );
562
-                    if ( ! $excluded ) {
563
-                        $is_datauri_candidate = $this->is_datauri_candidate( $ipath );
564
-                        if ( $is_datauri_candidate ) {
565
-                            $datauri     = $this->build_or_get_datauri_image( $ipath );
561
+                    $excluded = $this->check_datauri_exclude_list($ipath);
562
+                    if (!$excluded) {
563
+                        $is_datauri_candidate = $this->is_datauri_candidate($ipath);
564
+                        if ($is_datauri_candidate) {
565
+                            $datauri     = $this->build_or_get_datauri_image($ipath);
566 566
                             $base64data  = $datauri['base64data'];
567 567
                             // Add it to the list for replacement.
568
-                            $imgreplace[ $url_src_matches[1][ $count ] ] = str_replace(
568
+                            $imgreplace[$url_src_matches[1][$count]] = str_replace(
569 569
                                     $original_url,
570 570
                                     $datauri['full'],
571 571
                                     $url_src_matches[1][$count]
@@ -581,20 +581,20 @@  discard block
 block discarded – undo
581 581
                  * inlining isn't turned on, or if a resource is skipped from
582 582
                  * being inlined for whatever reason above.
583 583
                  */
584
-                if ( ! $inlined && ( ! empty( $this->cdn_url ) || has_filter( 'autoptimize_filter_base_replace_cdn' ) ) ) {
584
+                if (!$inlined && (!empty($this->cdn_url) || has_filter('autoptimize_filter_base_replace_cdn'))) {
585 585
                     // Just do the "simple" CDN replacement.
586
-                    $replacement_url = $this->url_replace_cdn( $url );
587
-                    $imgreplace[ $url_src_matches[1][ $count ] ] = str_replace(
586
+                    $replacement_url = $this->url_replace_cdn($url);
587
+                    $imgreplace[$url_src_matches[1][$count]] = str_replace(
588 588
                         $original_url, $replacement_url, $url_src_matches[1][$count]
589 589
                     );
590 590
                 }
591 591
             }
592 592
         }
593 593
 
594
-        $code = self::replace_longest_matches_first( $code, $imgreplace );
594
+        $code = self::replace_longest_matches_first($code, $imgreplace);
595 595
 
596 596
         // Replace back font-face markers with actual font-face declarations.
597
-        $code = $this->restore_fontface( $code );
597
+        $code = $this->restore_fontface($code);
598 598
 
599 599
         return $code;
600 600
     }
@@ -602,29 +602,29 @@  discard block
 block discarded – undo
602 602
     // Joins and optimizes CSS.
603 603
     public function minify()
604 604
     {
605
-        foreach ( $this->css as $group ) {
606
-            list( $media, $css ) = $group;
607
-            if ( preg_match( '#^INLINE;#', $css ) ) {
605
+        foreach ($this->css as $group) {
606
+            list($media, $css) = $group;
607
+            if (preg_match('#^INLINE;#', $css)) {
608 608
                 // <style>.
609
-                $css = preg_replace( '#^INLINE;#', '', $css );
610
-                $css = self::fixurls( ABSPATH . 'index.php', $css ); // ABSPATH already contains a trailing slash.
611
-                $tmpstyle = apply_filters( 'autoptimize_css_individual_style', $css, '' );
612
-                if ( has_filter( 'autoptimize_css_individual_style' ) && ! empty( $tmpstyle ) ) {
609
+                $css = preg_replace('#^INLINE;#', '', $css);
610
+                $css = self::fixurls(ABSPATH.'index.php', $css); // ABSPATH already contains a trailing slash.
611
+                $tmpstyle = apply_filters('autoptimize_css_individual_style', $css, '');
612
+                if (has_filter('autoptimize_css_individual_style') && !empty($tmpstyle)) {
613 613
                     $css = $tmpstyle;
614 614
                     $this->alreadyminified = true;
615 615
                 }
616 616
             } else {
617 617
                 // <link>
618
-                if ( false !== $css && file_exists( $css ) && is_readable( $css ) ) {
618
+                if (false !== $css && file_exists($css) && is_readable($css)) {
619 619
                     $cssPath = $css;
620
-                    $css = self::fixurls( $cssPath, file_get_contents( $cssPath ) );
621
-                    $css = preg_replace( '/\x{EF}\x{BB}\x{BF}/', '', $css );
622
-                    $tmpstyle = apply_filters( 'autoptimize_css_individual_style', $css, $cssPath );
623
-                    if ( has_filter( 'autoptimize_css_individual_style' ) && ! empty( $tmpstyle ) ) {
620
+                    $css = self::fixurls($cssPath, file_get_contents($cssPath));
621
+                    $css = preg_replace('/\x{EF}\x{BB}\x{BF}/', '', $css);
622
+                    $tmpstyle = apply_filters('autoptimize_css_individual_style', $css, $cssPath);
623
+                    if (has_filter('autoptimize_css_individual_style') && !empty($tmpstyle)) {
624 624
                         $css = $tmpstyle;
625 625
                         $this->alreadyminified = true;
626
-                    } elseif ( $this->can_inject_late( $cssPath, $css ) ) {
627
-                        $css = self::build_injectlater_marker( $cssPath, md5( $css ) );
626
+                    } elseif ($this->can_inject_late($cssPath, $css)) {
627
+                        $css = self::build_injectlater_marker($cssPath, md5($css));
628 628
                     }
629 629
                 } else {
630 630
                     // Couldn't read CSS. Maybe getpath isn't working?
@@ -632,12 +632,12 @@  discard block
 block discarded – undo
632 632
                 }
633 633
             }
634 634
 
635
-            foreach ( $media as $elem ) {
636
-                if ( ! empty( $css ) ) {
637
-                    if ( ! isset( $this->csscode[$elem] ) ) {
635
+            foreach ($media as $elem) {
636
+                if (!empty($css)) {
637
+                    if (!isset($this->csscode[$elem])) {
638 638
                         $this->csscode[$elem] = '';
639 639
                     }
640
-                    $this->csscode[$elem] .= "\n/*FILESTART*/" . $css;
640
+                    $this->csscode[$elem] .= "\n/*FILESTART*/".$css;
641 641
                 }
642 642
             }
643 643
         }
@@ -645,133 +645,133 @@  discard block
 block discarded – undo
645 645
         // Check for duplicate code.
646 646
         $md5list = array();
647 647
         $tmpcss  = $this->csscode;
648
-        foreach ( $tmpcss as $media => $code ) {
649
-            $md5sum    = md5( $code );
648
+        foreach ($tmpcss as $media => $code) {
649
+            $md5sum    = md5($code);
650 650
             $medianame = $media;
651
-            foreach ( $md5list as $med => $sum ) {
651
+            foreach ($md5list as $med => $sum) {
652 652
                 // If same code.
653
-                if ( $sum === $md5sum ) {
653
+                if ($sum === $md5sum) {
654 654
                     // Add the merged code.
655
-                    $medianame                 = $med . ', ' . $media;
655
+                    $medianame                 = $med.', '.$media;
656 656
                     $this->csscode[$medianame] = $code;
657 657
                     $md5list[$medianame]       = $md5list[$med];
658
-                    unset( $this->csscode[$med], $this->csscode[$media], $md5list[$med] );
658
+                    unset($this->csscode[$med], $this->csscode[$media], $md5list[$med]);
659 659
                 }
660 660
             }
661 661
             $md5list[$medianame] = $md5sum;
662 662
         }
663
-        unset( $tmpcss );
663
+        unset($tmpcss);
664 664
 
665 665
         // Manage @imports, while is for recursive import management.
666
-        foreach ( $this->csscode as &$thiscss ) {
666
+        foreach ($this->csscode as &$thiscss) {
667 667
             // Flag to trigger import reconstitution and var to hold external imports.
668 668
             $fiximports       = false;
669 669
             $external_imports = '';
670 670
 
671 671
             // remove comments to avoid importing commented-out imports.
672
-            $thiscss_nocomments = preg_replace( '#/\*.*\*/#Us', '', $thiscss );
673
-            while ( preg_match_all( '#@import +(?:url)?(?:(?:\((["\']?)(?:[^"\')]+)\1\)|(["\'])(?:[^"\']+)\2)(?:[^,;"\']+(?:,[^,;"\']+)*)?)(?:;)#mi', $thiscss_nocomments, $matches ) ) {
674
-                foreach ( $matches[0] as $import ) {
675
-                    if ( $this->isremovable( $import, $this->cssremovables ) ) {
676
-                        $thiscss = str_replace( $import, '', $thiscss );
672
+            $thiscss_nocomments = preg_replace('#/\*.*\*/#Us', '', $thiscss);
673
+            while (preg_match_all('#@import +(?:url)?(?:(?:\((["\']?)(?:[^"\')]+)\1\)|(["\'])(?:[^"\']+)\2)(?:[^,;"\']+(?:,[^,;"\']+)*)?)(?:;)#mi', $thiscss_nocomments, $matches)) {
674
+                foreach ($matches[0] as $import) {
675
+                    if ($this->isremovable($import, $this->cssremovables)) {
676
+                        $thiscss = str_replace($import, '', $thiscss);
677 677
                         $import_ok = true;
678 678
                     } else {
679
-                        $url = trim( preg_replace( '#^.*((?:https?:|ftp:)?//.*\.css).*$#', '$1', trim( $import ) ), " \t\n\r\0\x0B\"'" );
680
-                        $path = $this->getpath( $url );
679
+                        $url = trim(preg_replace('#^.*((?:https?:|ftp:)?//.*\.css).*$#', '$1', trim($import)), " \t\n\r\0\x0B\"'");
680
+                        $path = $this->getpath($url);
681 681
                         $import_ok = false;
682
-                        if ( file_exists( $path ) && is_readable( $path ) ) {
683
-                            $code = addcslashes( self::fixurls( $path, file_get_contents( $path ) ), "\\" );
684
-                            $code = preg_replace( '/\x{EF}\x{BB}\x{BF}/', '', $code );
685
-                            $tmpstyle = apply_filters( 'autoptimize_css_individual_style', $code, '' );
686
-                            if ( has_filter( 'autoptimize_css_individual_style' ) && ! empty( $tmpstyle ) ) {
682
+                        if (file_exists($path) && is_readable($path)) {
683
+                            $code = addcslashes(self::fixurls($path, file_get_contents($path)), "\\");
684
+                            $code = preg_replace('/\x{EF}\x{BB}\x{BF}/', '', $code);
685
+                            $tmpstyle = apply_filters('autoptimize_css_individual_style', $code, '');
686
+                            if (has_filter('autoptimize_css_individual_style') && !empty($tmpstyle)) {
687 687
                                 $code = $tmpstyle;
688 688
                                 $this->alreadyminified = true;
689
-                            } elseif ( $this->can_inject_late( $path, $code ) ) {
690
-                                $code = self::build_injectlater_marker( $path, md5( $code ) );
689
+                            } elseif ($this->can_inject_late($path, $code)) {
690
+                                $code = self::build_injectlater_marker($path, md5($code));
691 691
                             }
692 692
 
693
-                            if ( ! empty( $code ) ) {
694
-                                $tmp_thiscss = preg_replace( '#(/\*FILESTART\*/.*)' . preg_quote( $import, '#' ) . '#Us', '/*FILESTART2*/' . $code . '$1', $thiscss );
695
-                                if ( ! empty( $tmp_thiscss ) ) {
693
+                            if (!empty($code)) {
694
+                                $tmp_thiscss = preg_replace('#(/\*FILESTART\*/.*)'.preg_quote($import, '#').'#Us', '/*FILESTART2*/'.$code.'$1', $thiscss);
695
+                                if (!empty($tmp_thiscss)) {
696 696
                                     $thiscss = $tmp_thiscss;
697 697
                                     $import_ok = true;
698
-                                    unset( $tmp_thiscss );
698
+                                    unset($tmp_thiscss);
699 699
                                 }
700 700
                             }
701
-                            unset( $code );
701
+                            unset($code);
702 702
                         }
703 703
                     }
704
-                    if ( ! $import_ok ) {
704
+                    if (!$import_ok) {
705 705
                         // External imports and general fall-back.
706 706
                         $external_imports .= $import;
707 707
 
708
-                        $thiscss    = str_replace( $import, '', $thiscss );
708
+                        $thiscss    = str_replace($import, '', $thiscss);
709 709
                         $fiximports = true;
710 710
                     }
711 711
                 }
712
-                $thiscss = preg_replace( '#/\*FILESTART\*/#', '', $thiscss );
713
-                $thiscss = preg_replace( '#/\*FILESTART2\*/#', '/*FILESTART*/', $thiscss );
712
+                $thiscss = preg_replace('#/\*FILESTART\*/#', '', $thiscss);
713
+                $thiscss = preg_replace('#/\*FILESTART2\*/#', '/*FILESTART*/', $thiscss);
714 714
 
715 715
                 // and update $thiscss_nocomments before going into next iteration in while loop.
716
-                $thiscss_nocomments = preg_replace( '#/\*.*\*/#Us', '', $thiscss );
716
+                $thiscss_nocomments = preg_replace('#/\*.*\*/#Us', '', $thiscss);
717 717
             }
718
-            unset( $thiscss_nocomments );
718
+            unset($thiscss_nocomments);
719 719
 
720 720
             // Add external imports to top of aggregated CSS.
721
-            if ( $fiximports ) {
722
-                $thiscss = $external_imports . $thiscss;
721
+            if ($fiximports) {
722
+                $thiscss = $external_imports.$thiscss;
723 723
             }
724 724
         }
725
-        unset( $thiscss );
725
+        unset($thiscss);
726 726
 
727 727
         // $this->csscode has all the uncompressed code now.
728
-        foreach ( $this->csscode as &$code ) {
728
+        foreach ($this->csscode as &$code) {
729 729
             // Check for already-minified code.
730
-            $hash = md5( $code );
731
-            do_action( 'autoptimize_action_css_hash', $hash );
732
-            $ccheck = new autoptimizeCache( $hash, 'css' );
733
-            if ( $ccheck->check() ) {
730
+            $hash = md5($code);
731
+            do_action('autoptimize_action_css_hash', $hash);
732
+            $ccheck = new autoptimizeCache($hash, 'css');
733
+            if ($ccheck->check()) {
734 734
                 $code = $ccheck->retrieve();
735
-                $this->hashmap[md5( $code )] = $hash;
735
+                $this->hashmap[md5($code)] = $hash;
736 736
                 continue;
737 737
             }
738
-            unset( $ccheck );
738
+            unset($ccheck);
739 739
 
740 740
             // Rewrite and/or inline referenced assets.
741
-            $code = $this->rewrite_assets( $code );
741
+            $code = $this->rewrite_assets($code);
742 742
 
743 743
             // Minify.
744
-            $code = $this->run_minifier_on( $code );
744
+            $code = $this->run_minifier_on($code);
745 745
 
746 746
             // Bring back INJECTLATER stuff.
747
-            $code = $this->inject_minified( $code );
747
+            $code = $this->inject_minified($code);
748 748
 
749 749
             // Filter results.
750
-            $tmp_code = apply_filters( 'autoptimize_css_after_minify', $code );
751
-            if ( ! empty( $tmp_code ) ) {
750
+            $tmp_code = apply_filters('autoptimize_css_after_minify', $code);
751
+            if (!empty($tmp_code)) {
752 752
                 $code = $tmp_code;
753
-                unset( $tmp_code );
753
+                unset($tmp_code);
754 754
             }
755 755
 
756
-            $this->hashmap[md5( $code )] = $hash;
756
+            $this->hashmap[md5($code)] = $hash;
757 757
         }
758 758
 
759
-        unset( $code );
759
+        unset($code);
760 760
         return true;
761 761
     }
762 762
 
763
-    public function run_minifier_on( $code )
763
+    public function run_minifier_on($code)
764 764
     {
765
-        if ( ! $this->alreadyminified ) {
766
-            $do_minify = apply_filters( 'autoptimize_css_do_minify', true );
765
+        if (!$this->alreadyminified) {
766
+            $do_minify = apply_filters('autoptimize_css_do_minify', true);
767 767
 
768
-            if ( $do_minify ) {
768
+            if ($do_minify) {
769 769
                 $cssmin   = new autoptimizeCSSmin();
770
-                $tmp_code = trim( $cssmin->run( $code ) );
770
+                $tmp_code = trim($cssmin->run($code));
771 771
 
772
-                if ( ! empty( $tmp_code ) ) {
772
+                if (!empty($tmp_code)) {
773 773
                     $code = $tmp_code;
774
-                    unset( $tmp_code );
774
+                    unset($tmp_code);
775 775
                 }
776 776
             }
777 777
         }
@@ -783,14 +783,14 @@  discard block
 block discarded – undo
783 783
     public function cache()
784 784
     {
785 785
         // CSS cache.
786
-        foreach ( $this->csscode as $media => $code ) {
787
-            $md5 = $this->hashmap[md5( $code )];
788
-            $cache = new autoptimizeCache( $md5, 'css' );
789
-            if ( ! $cache->check() ) {
786
+        foreach ($this->csscode as $media => $code) {
787
+            $md5 = $this->hashmap[md5($code)];
788
+            $cache = new autoptimizeCache($md5, 'css');
789
+            if (!$cache->check()) {
790 790
                 // Cache our code.
791
-                $cache->cache( $code, 'text/css' );
791
+                $cache->cache($code, 'text/css');
792 792
             }
793
-            $this->url[$media] = AUTOPTIMIZE_CACHE_URL . $cache->getname();
793
+            $this->url[$media] = AUTOPTIMIZE_CACHE_URL.$cache->getname();
794 794
         }
795 795
     }
796 796
 
@@ -798,136 +798,136 @@  discard block
 block discarded – undo
798 798
     public function getcontent()
799 799
     {
800 800
         // Restore the full content (only applies when "autoptimize_filter_css_justhead" filter is true).
801
-        if ( ! empty( $this->restofcontent ) ) {
801
+        if (!empty($this->restofcontent)) {
802 802
             $this->content .= $this->restofcontent;
803 803
             $this->restofcontent = '';
804 804
         }
805 805
 
806 806
         // Inject the new stylesheets.
807
-        $replaceTag = array( '<title', 'before' );
808
-        $replaceTag = apply_filters( 'autoptimize_filter_css_replacetag', $replaceTag, $this->content );
807
+        $replaceTag = array('<title', 'before');
808
+        $replaceTag = apply_filters('autoptimize_filter_css_replacetag', $replaceTag, $this->content);
809 809
 
810
-        if ( $this->inline ) {
811
-            foreach ( $this->csscode as $media => $code ) {
812
-                $this->inject_in_html( '<style type="text/css" media="' . $media . '">' . $code . '</style>', $replaceTag );
810
+        if ($this->inline) {
811
+            foreach ($this->csscode as $media => $code) {
812
+                $this->inject_in_html('<style type="text/css" media="'.$media.'">'.$code.'</style>', $replaceTag);
813 813
             }
814 814
         } else {
815
-            if ( $this->defer ) {
815
+            if ($this->defer) {
816 816
                 $preloadCssBlock = '';
817 817
                 $noScriptCssBlock = "<noscript id=\"aonoscrcss\">";
818 818
 
819 819
                 $defer_inline_code = $this->defer_inline;
820
-                if ( ! empty( $defer_inline_code ) ) {
821
-                    if ( apply_filters( 'autoptimize_filter_css_critcss_minify', true ) ) {
822
-                        $iCssHash = md5( $defer_inline_code );
823
-                        $iCssCache = new autoptimizeCache( $iCssHash, 'css' );
824
-                        if ( $iCssCache->check() ) {
820
+                if (!empty($defer_inline_code)) {
821
+                    if (apply_filters('autoptimize_filter_css_critcss_minify', true)) {
822
+                        $iCssHash = md5($defer_inline_code);
823
+                        $iCssCache = new autoptimizeCache($iCssHash, 'css');
824
+                        if ($iCssCache->check()) {
825 825
                             // we have the optimized inline CSS in cache.
826 826
                             $defer_inline_code = $iCssCache->retrieve();
827 827
                         } else {
828 828
                             $cssmin   = new autoptimizeCSSmin();
829
-                            $tmp_code = trim( $cssmin->run( $defer_inline_code ) );
829
+                            $tmp_code = trim($cssmin->run($defer_inline_code));
830 830
 
831
-                            if ( ! empty( $tmp_code ) ) {
831
+                            if (!empty($tmp_code)) {
832 832
                                 $defer_inline_code = $tmp_code;
833
-                                $iCssCache->cache( $defer_inline_code, 'text/css' );
834
-                                unset( $tmp_code );
833
+                                $iCssCache->cache($defer_inline_code, 'text/css');
834
+                                unset($tmp_code);
835 835
                             }
836 836
                         }
837 837
                     }
838 838
                     // inlined critical css set here, but injected when full CSS is injected
839 839
                     // to avoid CSS containing SVG with <title tag receiving the full CSS link.
840
-                    $inlined_ccss_block = '<style type="text/css" id="aoatfcss" media="all">' . $defer_inline_code . '</style>';
840
+                    $inlined_ccss_block = '<style type="text/css" id="aoatfcss" media="all">'.$defer_inline_code.'</style>';
841 841
                 }
842 842
             }
843 843
 
844
-            foreach ( $this->url as $media => $url ) {
845
-                $url = $this->url_replace_cdn( $url );
844
+            foreach ($this->url as $media => $url) {
845
+                $url = $this->url_replace_cdn($url);
846 846
 
847 847
                 // Add the stylesheet either deferred (import at bottom) or normal links in head.
848
-                if ( $this->defer ) {
848
+                if ($this->defer) {
849 849
                     $preloadOnLoad = autoptimizeConfig::get_ao_css_preload_onload();
850 850
 
851
-                    $preloadCssBlock .= '<link rel="preload" as="style" media="' . $media . '" href="' . $url . '" onload="' . $preloadOnLoad . '" />';
852
-                    $noScriptCssBlock .= '<link type="text/css" media="' . $media . '" href="' . $url . '" rel="stylesheet" />';
851
+                    $preloadCssBlock .= '<link rel="preload" as="style" media="'.$media.'" href="'.$url.'" onload="'.$preloadOnLoad.'" />';
852
+                    $noScriptCssBlock .= '<link type="text/css" media="'.$media.'" href="'.$url.'" rel="stylesheet" />';
853 853
                 } else {
854 854
                     // $this->inject_in_html('<link type="text/css" media="' . $media . '" href="' . $url . '" rel="stylesheet" />', $replaceTag);
855
-                    if ( strlen( $this->csscode[$media] ) > $this->cssinlinesize ) {
856
-                        $this->inject_in_html( '<link type="text/css" media="' . $media . '" href="' . $url . '" rel="stylesheet" />', $replaceTag );
857
-                    } elseif ( strlen( $this->csscode[$media] ) > 0 ) {
858
-                        $this->inject_in_html( '<style type="text/css" media="' . $media . '">' . $this->csscode[$media] . '</style>', $replaceTag );
855
+                    if (strlen($this->csscode[$media]) > $this->cssinlinesize) {
856
+                        $this->inject_in_html('<link type="text/css" media="'.$media.'" href="'.$url.'" rel="stylesheet" />', $replaceTag);
857
+                    } elseif (strlen($this->csscode[$media]) > 0) {
858
+                        $this->inject_in_html('<style type="text/css" media="'.$media.'">'.$this->csscode[$media].'</style>', $replaceTag);
859 859
                     }
860 860
                 }
861 861
             }
862 862
 
863
-            if ( $this->defer ) {
863
+            if ($this->defer) {
864 864
                 $preload_polyfill = autoptimizeConfig::get_ao_css_preload_polyfill();
865 865
                 $noScriptCssBlock .= '</noscript>';
866 866
                 // Inject inline critical CSS, the preloaded full CSS and the noscript-CSS.
867
-                $this->inject_in_html( $inlined_ccss_block . $preloadCssBlock . $noScriptCssBlock, $replaceTag );
867
+                $this->inject_in_html($inlined_ccss_block.$preloadCssBlock.$noScriptCssBlock, $replaceTag);
868 868
 
869 869
                 // Adds preload polyfill at end of body tag.
870 870
                 $this->inject_in_html(
871
-                    apply_filters( 'autoptimize_css_preload_polyfill', $preload_polyfill ),
872
-                    array( '</body>', 'before' )
871
+                    apply_filters('autoptimize_css_preload_polyfill', $preload_polyfill),
872
+                    array('</body>', 'before')
873 873
                 );
874 874
             }
875 875
         }
876 876
 
877 877
         // restore comments.
878
-        $this->content = $this->restore_comments( $this->content );
878
+        $this->content = $this->restore_comments($this->content);
879 879
 
880 880
         // restore IE hacks.
881
-        $this->content = $this->restore_iehacks( $this->content );
881
+        $this->content = $this->restore_iehacks($this->content);
882 882
 
883 883
         // restore (no)script.
884
-        $this->content = $this->restore_marked_content( 'SCRIPT', $this->content );
884
+        $this->content = $this->restore_marked_content('SCRIPT', $this->content);
885 885
 
886 886
         // Restore noptimize.
887
-        $this->content = $this->restore_noptimize( $this->content );
887
+        $this->content = $this->restore_noptimize($this->content);
888 888
 
889 889
         // Return the modified stylesheet.
890 890
         return $this->content;
891 891
     }
892 892
 
893
-    static function fixurls( $file, $code )
893
+    static function fixurls($file, $code)
894 894
     {
895 895
         // Switch all imports to the url() syntax.
896
-        $code = preg_replace( '#@import ("|\')(.+?)\.css.*?("|\')#', '@import url("${2}.css")', $code );
896
+        $code = preg_replace('#@import ("|\')(.+?)\.css.*?("|\')#', '@import url("${2}.css")', $code);
897 897
 
898
-        if ( preg_match_all( self::ASSETS_REGEX, $code, $matches ) ) {
899
-            $file = str_replace( WP_ROOT_DIR, '/', $file );
898
+        if (preg_match_all(self::ASSETS_REGEX, $code, $matches)) {
899
+            $file = str_replace(WP_ROOT_DIR, '/', $file);
900 900
             /**
901 901
              * rollback as per https://github.com/futtta/autoptimize/issues/94
902 902
              * $file = str_replace( AUTOPTIMIZE_WP_CONTENT_NAME, '', $file );
903 903
              */
904
-            $dir = dirname( $file ); // Like /themes/expound/css.
904
+            $dir = dirname($file); // Like /themes/expound/css.
905 905
 
906 906
             /**
907 907
              * $dir should not contain backslashes, since it's used to replace
908 908
              * urls, but it can contain them when running on Windows because
909 909
              * fixurls() is sometimes called with `ABSPATH . 'index.php'`
910 910
              */
911
-            $dir = str_replace( '\\', '/', $dir );
912
-            unset( $file ); // not used below at all.
911
+            $dir = str_replace('\\', '/', $dir);
912
+            unset($file); // not used below at all.
913 913
 
914 914
             $replace = array();
915
-            foreach ( $matches[1] as $k => $url ) {
915
+            foreach ($matches[1] as $k => $url) {
916 916
                 // Remove quotes.
917
-                $url    = trim( $url, " \t\n\r\0\x0B\"'" );
918
-                $noQurl = trim( $url, "\"'" );
919
-                if ( $url !== $noQurl ) {
917
+                $url    = trim($url, " \t\n\r\0\x0B\"'");
918
+                $noQurl = trim($url, "\"'");
919
+                if ($url !== $noQurl) {
920 920
                     $removedQuotes = true;
921 921
                 } else {
922 922
                     $removedQuotes = false;
923 923
                 }
924 924
 
925
-                if ( '' === $noQurl ) {
925
+                if ('' === $noQurl) {
926 926
                     continue;
927 927
                 }
928 928
 
929 929
                 $url = $noQurl;
930
-                if ( '/' === $url{0} || preg_match( '#^(https?://|ftp://|data:)#i', $url ) ) {
930
+                if ('/' === $url{0} || preg_match('#^(https?://|ftp://|data:)#i', $url)) {
931 931
                     // URL is protocol-relative, host-relative or something we don't touch.
932 932
                     continue;
933 933
                 } else {
@@ -936,49 +936,49 @@  discard block
 block discarded – undo
936 936
                      * rollback as per https://github.com/futtta/autoptimize/issues/94
937 937
                      * $newurl = preg_replace( '/https?:/', '', str_replace( ' ', '%20', AUTOPTIMIZE_WP_CONTENT_URL . str_replace( '//', '/', $dir . '/' . $url ) ) );
938 938
                      */
939
-                    $newurl = preg_replace( '/https?:/', '', str_replace( ' ', '%20', AUTOPTIMIZE_WP_ROOT_URL . str_replace( '//', '/', $dir . '/' . $url ) ) );
940
-                    $newurl = apply_filters( 'autoptimize_filter_css_fixurl_newurl', $newurl );
939
+                    $newurl = preg_replace('/https?:/', '', str_replace(' ', '%20', AUTOPTIMIZE_WP_ROOT_URL.str_replace('//', '/', $dir.'/'.$url)));
940
+                    $newurl = apply_filters('autoptimize_filter_css_fixurl_newurl', $newurl);
941 941
 
942 942
                     /**
943 943
                      * Hash the url + whatever was behind potentially for replacement
944 944
                      * We must do this, or different css classes referencing the same bg image (but
945 945
                      * different parts of it, say, in sprites and such) loose their stuff...
946 946
                      */
947
-                    $hash = md5( $url . $matches[2][$k] );
948
-                    $code = str_replace( $matches[0][$k], $hash, $code );
947
+                    $hash = md5($url.$matches[2][$k]);
948
+                    $code = str_replace($matches[0][$k], $hash, $code);
949 949
 
950
-                    if ( $removedQuotes ) {
951
-                        $replace[$hash] = "url('" . $newurl . "')" . $matches[2][$k];
950
+                    if ($removedQuotes) {
951
+                        $replace[$hash] = "url('".$newurl."')".$matches[2][$k];
952 952
                     } else {
953
-                        $replace[$hash] = 'url(' . $newurl . ')' . $matches[2][$k];
953
+                        $replace[$hash] = 'url('.$newurl.')'.$matches[2][$k];
954 954
                     }
955 955
                 }
956 956
             }
957 957
 
958
-            $code = self::replace_longest_matches_first( $code, $replace );
958
+            $code = self::replace_longest_matches_first($code, $replace);
959 959
         }
960 960
 
961 961
         return $code;
962 962
     }
963 963
 
964
-    private function ismovable( $tag )
964
+    private function ismovable($tag)
965 965
     {
966
-        if ( ! $this->aggregate ) {
966
+        if (!$this->aggregate) {
967 967
             return false;
968 968
         }
969 969
 
970
-        if ( ! empty( $this->whitelist ) ) {
971
-            foreach ( $this->whitelist as $match ) {
972
-                if ( false !== strpos( $tag, $match ) ) {
970
+        if (!empty($this->whitelist)) {
971
+            foreach ($this->whitelist as $match) {
972
+                if (false !== strpos($tag, $match)) {
973 973
                     return true;
974 974
                 }
975 975
             }
976 976
             // no match with whitelist.
977 977
             return false;
978 978
         } else {
979
-            if ( is_array( $this->dontmove ) && ! empty( $this->dontmove ) ) {
980
-                foreach ( $this->dontmove as $match ) {
981
-                    if ( false !== strpos( $tag, $match ) ) {
979
+            if (is_array($this->dontmove) && !empty($this->dontmove)) {
980
+                foreach ($this->dontmove as $match) {
981
+                    if (false !== strpos($tag, $match)) {
982 982
                         // Matched something.
983 983
                         return false;
984 984
                     }
@@ -990,22 +990,22 @@  discard block
 block discarded – undo
990 990
         }
991 991
     }
992 992
 
993
-    private function can_inject_late( $cssPath, $css )
993
+    private function can_inject_late($cssPath, $css)
994 994
     {
995
-        $consider_minified_array = apply_filters( 'autoptimize_filter_css_consider_minified', false, $cssPath );
996
-        if ( true !== $this->inject_min_late ) {
995
+        $consider_minified_array = apply_filters('autoptimize_filter_css_consider_minified', false, $cssPath);
996
+        if (true !== $this->inject_min_late) {
997 997
             // late-inject turned off.
998 998
             return false;
999
-        } elseif ( ( false === strpos( $cssPath, 'min.css' ) ) && ( str_replace( $consider_minified_array, '', $cssPath ) === $cssPath ) ) {
999
+        } elseif ((false === strpos($cssPath, 'min.css')) && (str_replace($consider_minified_array, '', $cssPath) === $cssPath)) {
1000 1000
             // file not minified based on filename & filter.
1001 1001
             return false;
1002
-        } elseif ( false !== strpos( $css, '@import' ) ) {
1002
+        } elseif (false !== strpos($css, '@import')) {
1003 1003
             // can't late-inject files with imports as those need to be aggregated.
1004 1004
             return false;
1005
-        } elseif ( ( false !== strpos( $css, '@font-face' ) ) && ( apply_filters( 'autoptimize_filter_css_fonts_cdn', false ) === true ) && ( ! empty( $this->cdn_url ) ) ) {
1005
+        } elseif ((false !== strpos($css, '@font-face')) && (apply_filters('autoptimize_filter_css_fonts_cdn', false) === true) && (!empty($this->cdn_url))) {
1006 1006
             // don't late-inject CSS with font-src's if fonts are set to be CDN'ed.
1007 1007
             return false;
1008
-        } elseif ( ( ( $this->datauris == true ) || ( ! empty( $this->cdn_url ) ) ) && preg_match( '#background[^;}]*url\(#Ui', $css ) ) {
1008
+        } elseif ((($this->datauris == true) || (!empty($this->cdn_url))) && preg_match('#background[^;}]*url\(#Ui', $css)) {
1009 1009
             // don't late-inject CSS with images if CDN is set OR if image inlining is on.
1010 1010
             return false;
1011 1011
         } else {
@@ -1023,32 +1023,32 @@  discard block
 block discarded – undo
1023 1023
      *
1024 1024
      * @return bool|string Url pointing to the minified css file or false.
1025 1025
      */
1026
-    public function minify_single( $filepath, $cache_miss = false )
1026
+    public function minify_single($filepath, $cache_miss = false)
1027 1027
     {
1028
-        $contents = $this->prepare_minify_single( $filepath );
1028
+        $contents = $this->prepare_minify_single($filepath);
1029 1029
 
1030
-        if ( empty( $contents ) ) {
1030
+        if (empty($contents)) {
1031 1031
             return false;
1032 1032
         }
1033 1033
 
1034 1034
         // Check cache.
1035
-        $hash  = 'single_' . md5( $contents );
1036
-        $cache = new autoptimizeCache( $hash, 'css' );
1035
+        $hash  = 'single_'.md5($contents);
1036
+        $cache = new autoptimizeCache($hash, 'css');
1037 1037
 
1038 1038
         // If not in cache already, minify...
1039
-        if ( ! $cache->check() || $cache_miss ) {
1039
+        if (!$cache->check() || $cache_miss) {
1040 1040
             // Fixurls...
1041
-            $contents = self::fixurls( $filepath, $contents );
1041
+            $contents = self::fixurls($filepath, $contents);
1042 1042
             // CDN-replace any referenced assets if needed...
1043
-            $contents = $this->replace_urls( $contents );
1043
+            $contents = $this->replace_urls($contents);
1044 1044
             // Now minify...
1045 1045
             $cssmin   = new autoptimizeCSSmin();
1046
-            $contents = trim( $cssmin->run( $contents ) );
1046
+            $contents = trim($cssmin->run($contents));
1047 1047
             // Store in cache.
1048
-            $cache->cache( $contents, 'text/css' );
1048
+            $cache->cache($contents, 'text/css');
1049 1049
         }
1050 1050
 
1051
-        $url = $this->build_minify_single_url( $cache );
1051
+        $url = $this->build_minify_single_url($cache);
1052 1052
 
1053 1053
         return $url;
1054 1054
     }
@@ -1068,18 +1068,18 @@  discard block
 block discarded – undo
1068 1068
         return $this->options;
1069 1069
     }
1070 1070
 
1071
-    public function replaceOptions( $options )
1071
+    public function replaceOptions($options)
1072 1072
     {
1073 1073
         $this->options = $options;
1074 1074
     }
1075 1075
 
1076
-    public function setOption( $name, $value )
1076
+    public function setOption($name, $value)
1077 1077
     {
1078 1078
         $this->options[$name] = $value;
1079 1079
         $this->$name = $value;
1080 1080
     }
1081 1081
 
1082
-    public function getOption( $name )
1082
+    public function getOption($name)
1083 1083
     {
1084 1084
         return $this->options[$name];
1085 1085
     }
Please login to merge, or discard this patch.
classes/autoptimizeUtils.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
      *
15 15
      * @param bool|null $override Allows overriding the decision.
16 16
      *
17
-     * @return bool
17
+     * @return boolean|null
18 18
      */
19 19
     public static function mbstring_available( $override = null )
20 20
     {
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
      *
131 131
      * @param bool $override Allows overriding the decision when needed.
132 132
      *
133
-     * @return bool
133
+     * @return boolean|null
134 134
      */
135 135
     public static function siteurl_not_root( $override = null )
136 136
     {
Please login to merge, or discard this patch.
Spacing   +82 added lines, -82 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * General helpers.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
@@ -16,15 +16,15 @@  discard block
 block discarded – undo
16 16
      *
17 17
      * @return bool
18 18
      */
19
-    public static function mbstring_available( $override = null )
19
+    public static function mbstring_available($override = null)
20 20
     {
21 21
         static $available = null;
22 22
 
23
-        if ( null === $available ) {
24
-            $available = \extension_loaded( 'mbstring' );
23
+        if (null === $available) {
24
+            $available = \extension_loaded('mbstring');
25 25
         }
26 26
 
27
-        if ( null !== $override ) {
27
+        if (null !== $override) {
28 28
             $available = $override;
29 29
         }
30 30
 
@@ -42,12 +42,12 @@  discard block
 block discarded – undo
42 42
      *
43 43
      * @return int|false
44 44
      */
45
-    public static function strpos( $haystack, $needle, $offset = 0, $encoding = null )
45
+    public static function strpos($haystack, $needle, $offset = 0, $encoding = null)
46 46
     {
47
-        if ( self::mbstring_available() ) {
48
-            return ( null === $encoding ) ? \mb_strpos( $haystack, $needle, $offset ) : \mb_strlen( $haystack, $needle, $offset, $encoding );
47
+        if (self::mbstring_available()) {
48
+            return (null === $encoding) ? \mb_strpos($haystack, $needle, $offset) : \mb_strlen($haystack, $needle, $offset, $encoding);
49 49
         } else {
50
-            return \strpos( $haystack, $needle, $offset );
50
+            return \strpos($haystack, $needle, $offset);
51 51
         }
52 52
     }
53 53
 
@@ -62,12 +62,12 @@  discard block
 block discarded – undo
62 62
      * @return int Number of charcters or bytes in given $string
63 63
      *             (characters if/when supported, bytes otherwise).
64 64
      */
65
-    public static function strlen( $string, $encoding = null )
65
+    public static function strlen($string, $encoding = null)
66 66
     {
67
-        if ( self::mbstring_available() ) {
68
-            return ( null === $encoding ) ? \mb_strlen( $string ) : \mb_strlen( $string, $encoding );
67
+        if (self::mbstring_available()) {
68
+            return (null === $encoding) ? \mb_strlen($string) : \mb_strlen($string, $encoding);
69 69
         } else {
70
-            return \strlen( $string );
70
+            return \strlen($string);
71 71
         }
72 72
     }
73 73
 
@@ -85,44 +85,44 @@  discard block
 block discarded – undo
85 85
      *
86 86
      * @return string
87 87
      */
88
-    public static function substr_replace( $string, $replacement, $start, $length = null, $encoding = null )
88
+    public static function substr_replace($string, $replacement, $start, $length = null, $encoding = null)
89 89
     {
90
-        if ( self::mbstring_available() ) {
91
-            $strlen = self::strlen( $string, $encoding );
90
+        if (self::mbstring_available()) {
91
+            $strlen = self::strlen($string, $encoding);
92 92
 
93
-            if ( $start < 0 ) {
93
+            if ($start < 0) {
94 94
                 if ( -$start < $strlen ) {
95 95
                     $start = $strlen + $start;
96 96
                 } else {
97 97
                     $start = 0;
98 98
                 }
99
-            } elseif ( $start > $strlen ) {
99
+            } elseif ($start > $strlen) {
100 100
                 $start = $strlen;
101 101
             }
102 102
 
103
-            if ( null === $length || '' === $length ) {
103
+            if (null === $length || '' === $length) {
104 104
                 $start2 = $strlen;
105
-            } elseif ( $length < 0 ) {
105
+            } elseif ($length < 0) {
106 106
                 $start2 = $strlen + $length;
107
-                if ( $start2 < $start ) {
107
+                if ($start2 < $start) {
108 108
                     $start2 = $start;
109 109
                 }
110 110
             } else {
111 111
                 $start2 = $start + $length;
112 112
             }
113 113
 
114
-            if ( null === $encoding ) {
115
-                $leader  = $start ? \mb_substr( $string, 0, $start ) : '';
116
-                $trailer = ( $start2 < $strlen ) ? \mb_substr( $string, $start2, null ) : '';
114
+            if (null === $encoding) {
115
+                $leader  = $start ? \mb_substr($string, 0, $start) : '';
116
+                $trailer = ($start2 < $strlen) ? \mb_substr($string, $start2, null) : '';
117 117
             } else {
118
-                $leader  = $start ? \mb_substr( $string, 0, $start, $encoding ) : '';
119
-                $trailer = ( $start2 < $strlen ) ? \mb_substr( $string, $start2, null, $encoding ) : '';
118
+                $leader  = $start ? \mb_substr($string, 0, $start, $encoding) : '';
119
+                $trailer = ($start2 < $strlen) ? \mb_substr($string, $start2, null, $encoding) : '';
120 120
             }
121 121
 
122 122
             return "{$leader}{$replacement}{$trailer}";
123 123
         }
124 124
 
125
-        return ( null === $length ) ? \substr_replace( $string, $replacement, $start ) : \substr_replace( $string, $replacement, $start, $length );
125
+        return (null === $length) ? \substr_replace($string, $replacement, $start) : \substr_replace($string, $replacement, $start, $length);
126 126
     }
127 127
 
128 128
     /**
@@ -132,16 +132,16 @@  discard block
 block discarded – undo
132 132
      *
133 133
      * @return bool
134 134
      */
135
-    public static function siteurl_not_root( $override = null )
135
+    public static function siteurl_not_root($override = null)
136 136
     {
137 137
         static $subdir = null;
138 138
 
139
-        if ( null === $subdir ) {
139
+        if (null === $subdir) {
140 140
             $parts  = self::get_ao_wp_site_url_parts();
141
-            $subdir = ( isset( $parts['path'] ) && ( '/' !== $parts['path'] ) );
141
+            $subdir = (isset($parts['path']) && ('/' !== $parts['path']));
142 142
         }
143 143
 
144
-        if ( null !== $override ) {
144
+        if (null !== $override) {
145 145
             $subdir = $override;
146 146
         }
147 147
 
@@ -158,8 +158,8 @@  discard block
 block discarded – undo
158 158
     {
159 159
         static $parts = array();
160 160
 
161
-        if ( empty( $parts ) ) {
162
-            $parts = \parse_url( AUTOPTIMIZE_WP_SITE_URL );
161
+        if (empty($parts)) {
162
+            $parts = \parse_url(AUTOPTIMIZE_WP_SITE_URL);
163 163
         }
164 164
 
165 165
         return $parts;
@@ -174,33 +174,33 @@  discard block
 block discarded – undo
174 174
      *
175 175
      * @return string
176 176
      */
177
-    public static function tweak_cdn_url_if_needed( $cdn_url, $force_cache_miss = false )
177
+    public static function tweak_cdn_url_if_needed($cdn_url, $force_cache_miss = false)
178 178
     {
179 179
         static $results = array();
180 180
 
181
-        if ( ! isset( $results[ $cdn_url ] ) || $force_cache_miss ) {
181
+        if (!isset($results[$cdn_url]) || $force_cache_miss) {
182 182
 
183 183
             // In order to return unmodified input when there's no need to tweak.
184
-            $results[ $cdn_url ] = $cdn_url;
184
+            $results[$cdn_url] = $cdn_url;
185 185
 
186 186
             // Behind a default true filter for backcompat, and only for sites
187 187
             // in a subfolder/subdirectory, but still easily turned off if
188 188
             // not wanted/needed...
189
-            if ( autoptimizeUtils::siteurl_not_root() ) {
190
-                $check = apply_filters( 'autoptimize_filter_cdn_magic_path_check', true, $cdn_url );
191
-                if ( $check ) {
189
+            if (autoptimizeUtils::siteurl_not_root()) {
190
+                $check = apply_filters('autoptimize_filter_cdn_magic_path_check', true, $cdn_url);
191
+                if ($check) {
192 192
                     $site_url_parts = autoptimizeUtils::get_ao_wp_site_url_parts();
193
-                    $cdn_url_parts  = \parse_url( $cdn_url );
194
-                    $schemeless     = self::is_protocol_relative( $cdn_url );
195
-                    $cdn_url_parts  = self::maybe_replace_cdn_path( $site_url_parts, $cdn_url_parts );
196
-                    if ( false !== $cdn_url_parts ) {
197
-                        $results[ $cdn_url ] = self::assemble_parsed_url( $cdn_url_parts, $schemeless );
193
+                    $cdn_url_parts  = \parse_url($cdn_url);
194
+                    $schemeless     = self::is_protocol_relative($cdn_url);
195
+                    $cdn_url_parts  = self::maybe_replace_cdn_path($site_url_parts, $cdn_url_parts);
196
+                    if (false !== $cdn_url_parts) {
197
+                        $results[$cdn_url] = self::assemble_parsed_url($cdn_url_parts, $schemeless);
198 198
                     }
199 199
                 }
200 200
             }
201 201
         }
202 202
 
203
-        return $results[ $cdn_url ];
203
+        return $results[$cdn_url];
204 204
     }
205 205
 
206 206
     /**
@@ -214,10 +214,10 @@  discard block
 block discarded – undo
214 214
      *
215 215
      * @return array|false
216 216
      */
217
-    public static function maybe_replace_cdn_path( array $site_url_parts, array $cdn_url_parts )
217
+    public static function maybe_replace_cdn_path(array $site_url_parts, array $cdn_url_parts)
218 218
     {
219
-        if ( isset( $site_url_parts['path'] ) && '/' !== $site_url_parts['path'] ) {
220
-            if ( ! isset( $cdn_url_parts['path'] ) || '/' === $cdn_url_parts['path'] ) {
219
+        if (isset($site_url_parts['path']) && '/' !== $site_url_parts['path']) {
220
+            if (!isset($cdn_url_parts['path']) || '/' === $cdn_url_parts['path']) {
221 221
                 $cdn_url_parts['path'] = $site_url_parts['path'];
222 222
                 return $cdn_url_parts;
223 223
             }
@@ -237,20 +237,20 @@  discard block
 block discarded – undo
237 237
      *
238 238
      * @return string
239 239
      */
240
-    public static function assemble_parsed_url( array $parsed_url, $schemeless = false )
240
+    public static function assemble_parsed_url(array $parsed_url, $schemeless = false)
241 241
     {
242
-        $scheme = isset( $parsed_url['scheme'] ) ? $parsed_url['scheme'] . '://' : '';
243
-        if ( $schemeless ) {
242
+        $scheme = isset($parsed_url['scheme']) ? $parsed_url['scheme'].'://' : '';
243
+        if ($schemeless) {
244 244
             $scheme = '//';
245 245
         }
246
-        $host     = isset( $parsed_url['host'] ) ? $parsed_url['host'] : '';
247
-        $port     = isset( $parsed_url['port'] ) ? ':' . $parsed_url['port'] : '';
248
-        $user     = isset( $parsed_url['user'] ) ? $parsed_url['user'] : '';
249
-        $pass     = isset( $parsed_url['pass'] ) ? ':' . $parsed_url['pass'] : '';
250
-        $pass     = ( $user || $pass ) ? "$pass@" : '';
251
-        $path     = isset( $parsed_url['path'] ) ? $parsed_url['path'] : '';
252
-        $query    = isset( $parsed_url['query'] ) ? '?' . $parsed_url['query'] : '';
253
-        $fragment = isset( $parsed_url['fragment'] ) ? '#' . $parsed_url['fragment'] : '';
246
+        $host     = isset($parsed_url['host']) ? $parsed_url['host'] : '';
247
+        $port     = isset($parsed_url['port']) ? ':'.$parsed_url['port'] : '';
248
+        $user     = isset($parsed_url['user']) ? $parsed_url['user'] : '';
249
+        $pass     = isset($parsed_url['pass']) ? ':'.$parsed_url['pass'] : '';
250
+        $pass     = ($user || $pass) ? "$pass@" : '';
251
+        $path     = isset($parsed_url['path']) ? $parsed_url['path'] : '';
252
+        $query    = isset($parsed_url['query']) ? '?'.$parsed_url['query'] : '';
253
+        $fragment = isset($parsed_url['fragment']) ? '#'.$parsed_url['fragment'] : '';
254 254
 
255 255
         return "$scheme$user$pass$host$port$path$query$fragment";
256 256
     }
@@ -262,12 +262,12 @@  discard block
 block discarded – undo
262 262
      *
263 263
      * @return bool
264 264
      */
265
-    public static function is_protocol_relative( $url )
265
+    public static function is_protocol_relative($url)
266 266
     {
267 267
         $result = false;
268 268
 
269
-        if ( ! empty( $url ) ) {
270
-            $result = ( 0 === strpos( $url, '//' ) );
269
+        if (!empty($url)) {
270
+            $result = (0 === strpos($url, '//'));
271 271
         }
272 272
 
273 273
         return $result;
@@ -280,9 +280,9 @@  discard block
 block discarded – undo
280 280
      *
281 281
      * @return string
282 282
      */
283
-    public static function path_canonicalize( $path )
283
+    public static function path_canonicalize($path)
284 284
     {
285
-        $patterns     = array(
285
+        $patterns = array(
286 286
             '~/{2,}~',
287 287
             '~/(\./)+~',
288 288
             '~([^/\.]+/(?R)*\.{2,}/)~',
@@ -295,7 +295,7 @@  discard block
 block discarded – undo
295 295
             '',
296 296
         );
297 297
 
298
-        return preg_replace( $patterns, $replacements, $path );
298
+        return preg_replace($patterns, $replacements, $path);
299 299
     }
300 300
 
301 301
     /**
@@ -305,15 +305,15 @@  discard block
 block discarded – undo
305 305
      *
306 306
      * @return none if $return_result is false (default), array if $return_result is true.
307 307
      */
308
-    public static function check_service_availability( $return_result = false )
308
+    public static function check_service_availability($return_result = false)
309 309
     {
310
-        $service_availability_resp = wp_remote_get( 'https://misc.optimizingmatters.com/api/autoptimize_service_availablity.json?from=aomain&ver=' . AUTOPTIMIZE_PLUGIN_VERSION );
311
-        if ( ! is_wp_error( $service_availability_resp ) ) {
312
-            if ( '200' == wp_remote_retrieve_response_code( $service_availability_resp ) ) {
313
-                $availabilities = json_decode( wp_remote_retrieve_body( $service_availability_resp ), true );
314
-                if ( is_array( $availabilities ) ) {
315
-                    update_option( 'autoptimize_service_availablity', $availabilities );
316
-                    if ( $return_result ) {
310
+        $service_availability_resp = wp_remote_get('https://misc.optimizingmatters.com/api/autoptimize_service_availablity.json?from=aomain&ver='.AUTOPTIMIZE_PLUGIN_VERSION);
311
+        if (!is_wp_error($service_availability_resp)) {
312
+            if ('200' == wp_remote_retrieve_response_code($service_availability_resp)) {
313
+                $availabilities = json_decode(wp_remote_retrieve_body($service_availability_resp), true);
314
+                if (is_array($availabilities)) {
315
+                    update_option('autoptimize_service_availablity', $availabilities);
316
+                    if ($return_result) {
317 317
                         return $availabilities;
318 318
                     }
319 319
                 }
@@ -328,10 +328,10 @@  discard block
 block discarded – undo
328 328
      *
329 329
      * @return bool
330 330
      */
331
-    public static function str_is_valid_regex( $string )
331
+    public static function str_is_valid_regex($string)
332 332
     {
333
-        set_error_handler( function() {}, E_WARNING );
334
-        $is_regex = ( false !== preg_match( $string, '' ) );
333
+        set_error_handler(function() {}, E_WARNING);
334
+        $is_regex = (false !== preg_match($string, ''));
335 335
         restore_error_handler();
336 336
 
337 337
         return $is_regex;
@@ -344,16 +344,16 @@  discard block
 block discarded – undo
344 344
      *
345 345
      * @return bool
346 346
      */
347
-    public static function is_plugin_active( $plugin_file )
347
+    public static function is_plugin_active($plugin_file)
348 348
     {
349 349
         static $ipa_exists = null;
350
-        if ( null === $ipa_exists ) {
351
-            if ( ! function_exists( '\is_plugin_active' ) ) {
352
-                require_once ABSPATH . 'wp-admin/includes/plugin.php';
350
+        if (null === $ipa_exists) {
351
+            if (!function_exists('\is_plugin_active')) {
352
+                require_once ABSPATH.'wp-admin/includes/plugin.php';
353 353
             }
354
-            $ipa_exists = function_exists( '\is_plugin_active' );
354
+            $ipa_exists = function_exists('\is_plugin_active');
355 355
         }
356 356
 
357
-        return $ipa_exists && \is_plugin_active( $plugin_file );
357
+        return $ipa_exists && \is_plugin_active($plugin_file);
358 358
     }
359 359
 }
Please login to merge, or discard this patch.
tests/test-ao.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -128,6 +128,9 @@
 block discarded – undo
128 128
         parent::tearDown();
129 129
     }
130 130
 
131
+    /**
132
+     * @return string
133
+     */
131 134
     protected function get_test_markup()
132 135
     {
133 136
         $siteurl = $this->get_urls()['siteurl'];
Please login to merge, or discard this patch.
Spacing   +385 added lines, -385 removed lines patch added patch discarded remove patch
@@ -10,17 +10,17 @@  discard block
 block discarded – undo
10 10
     protected $ao;
11 11
 
12 12
     protected static $flexible_url_parts_js = [
13
-        'default'          => 'wp-content/cache/autoptimize/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
14
-        'custom'           => 'wp-content/c/ao/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
15
-        'multisite'        => 'wp-content/cache/autoptimize/1/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
16
-        'multisite_custom' => 'wp-content/c/ao/1/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
13
+        'default'          => 'wp-content/cache/autoptimize/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
14
+        'custom'           => 'wp-content/c/ao/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
15
+        'multisite'        => 'wp-content/cache/autoptimize/1/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
16
+        'multisite_custom' => 'wp-content/c/ao/1/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
17 17
     ];
18 18
 
19 19
     protected static $flexible_url_parts_css = [
20
-        'default'          => 'wp-content/cache/autoptimize/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
21
-        'custom'           => 'wp-content/c/ao/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
22
-        'multisite'        => 'wp-content/cache/autoptimize/1/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
23
-        'multisite_custom' => 'wp-content/c/ao/1/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
20
+        'default'          => 'wp-content/cache/autoptimize/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
21
+        'custom'           => 'wp-content/c/ao/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
22
+        'multisite'        => 'wp-content/cache/autoptimize/1/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
23
+        'multisite_custom' => 'wp-content/c/ao/1/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
24 24
     ];
25 25
 
26 26
     protected function getAoStylesDefaultOptions()
@@ -28,16 +28,16 @@  discard block
 block discarded – undo
28 28
         $conf = autoptimizeConfig::instance();
29 29
 
30 30
         return [
31
-            'aggregate'      => $conf->get( 'autoptimize_css_aggregate' ),
32
-            'justhead'       => $conf->get( 'autoptimize_css_justhead' ),
33
-            'datauris'       => $conf->get( 'autoptimize_css_datauris' ),
34
-            'defer'          => $conf->get( 'autoptimize_css_defer' ),
35
-            'defer_inline'   => $conf->get( 'autoptimize_css_defer_inline' ),
36
-            'inline'         => $conf->get( 'autoptimize_css_inline' ),
37
-            'css_exclude'    => $conf->get( 'autoptimize_css_exclude' ),
38
-            'cdn_url'        => $conf->get( 'autoptimize_cdn_url' ),
39
-            'include_inline' => $conf->get( 'autoptimize_css_include_inline' ),
40
-            'nogooglefont'   => $conf->get( 'autoptimize_css_nogooglefont' ),
31
+            'aggregate'      => $conf->get('autoptimize_css_aggregate'),
32
+            'justhead'       => $conf->get('autoptimize_css_justhead'),
33
+            'datauris'       => $conf->get('autoptimize_css_datauris'),
34
+            'defer'          => $conf->get('autoptimize_css_defer'),
35
+            'defer_inline'   => $conf->get('autoptimize_css_defer_inline'),
36
+            'inline'         => $conf->get('autoptimize_css_inline'),
37
+            'css_exclude'    => $conf->get('autoptimize_css_exclude'),
38
+            'cdn_url'        => $conf->get('autoptimize_cdn_url'),
39
+            'include_inline' => $conf->get('autoptimize_css_include_inline'),
40
+            'nogooglefont'   => $conf->get('autoptimize_css_nogooglefont'),
41 41
         ];
42 42
     }
43 43
 
@@ -46,13 +46,13 @@  discard block
 block discarded – undo
46 46
         $conf = autoptimizeConfig::instance();
47 47
 
48 48
         return [
49
-            'aggregate'      => $conf->get( 'autoptimize_js_aggregate' ),
50
-            'justhead'       => $conf->get( 'autoptimize_js_justhead' ),
51
-            'forcehead'      => $conf->get( 'autoptimize_js_forcehead' ),
52
-            'trycatch'       => $conf->get( 'autoptimize_js_trycatch' ),
53
-            'js_exclude'     => $conf->get( 'autoptimize_js_exclude' ),
54
-            'cdn_url'        => $conf->get( 'autoptimize_cdn_url' ),
55
-            'include_inline' => $conf->get( 'autoptimize_js_include_inline' ),
49
+            'aggregate'      => $conf->get('autoptimize_js_aggregate'),
50
+            'justhead'       => $conf->get('autoptimize_js_justhead'),
51
+            'forcehead'      => $conf->get('autoptimize_js_forcehead'),
52
+            'trycatch'       => $conf->get('autoptimize_js_trycatch'),
53
+            'js_exclude'     => $conf->get('autoptimize_js_exclude'),
54
+            'cdn_url'        => $conf->get('autoptimize_cdn_url'),
55
+            'include_inline' => $conf->get('autoptimize_js_include_inline'),
56 56
         ];
57 57
     }
58 58
 
@@ -62,35 +62,35 @@  discard block
 block discarded – undo
62 62
     protected function get_urls()
63 63
     {
64 64
         static $site_url = null;
65
-        if ( null === $site_url ) {
65
+        if (null === $site_url) {
66 66
             $site_url = site_url();
67 67
         }
68 68
 
69 69
         static $cdn_url = null;
70
-        if ( null === $cdn_url ) {
71
-            $cdn_url = get_option( 'autoptimize_cdn_url' );
70
+        if (null === $cdn_url) {
71
+            $cdn_url = get_option('autoptimize_cdn_url');
72 72
         }
73 73
 
74 74
         static $imgopt_host = null;
75
-        if ( null === $imgopt_host ) {
76
-            $optimizer   = new autoptimizeImages( autoptimizeExtra::fetch_options() );
77
-            $imgopt_host = rtrim( $optimizer->get_imgopt_host(), '/' );
75
+        if (null === $imgopt_host) {
76
+            $optimizer   = new autoptimizeImages(autoptimizeExtra::fetch_options());
77
+            $imgopt_host = rtrim($optimizer->get_imgopt_host(), '/');
78 78
         }
79 79
 
80 80
         static $urls = [];
81 81
 
82
-        if ( empty( $urls ) ) {
82
+        if (empty($urls)) {
83 83
             $parts = autoptimizeUtils::get_ao_wp_site_url_parts();
84 84
             $urls  = [
85 85
                 'siteurl'    => $site_url,
86
-                'prsiteurl'  => '//' . str_replace( array( 'http://', 'https://' ), '', $site_url ),
87
-                'wwwsiteurl' => $parts['scheme'] . '://www.' . str_replace( 'www.', '', $parts['host'] ),
86
+                'prsiteurl'  => '//'.str_replace(array('http://', 'https://'), '', $site_url),
87
+                'wwwsiteurl' => $parts['scheme'].'://www.'.str_replace('www.', '', $parts['host']),
88 88
                 'cdnurl'     => $cdn_url,
89 89
                 'imgopthost' => $imgopt_host,
90 90
                 'subfolder'  => '',
91 91
             ];
92 92
 
93
-            if ( AO_TEST_SUBFOLDER_INSTALL ) {
93
+            if (AO_TEST_SUBFOLDER_INSTALL) {
94 94
                 $urls['subfolder'] = 'wordpress/';
95 95
             }
96 96
         }
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
      */
104 104
     public function setUp()
105 105
     {
106
-        $this->ao = new autoptimizeMain( AUTOPTIMIZE_PLUGIN_VERSION, AUTOPTIMIZE_PLUGIN_FILE );
106
+        $this->ao = new autoptimizeMain(AUTOPTIMIZE_PLUGIN_VERSION, AUTOPTIMIZE_PLUGIN_FILE);
107 107
 
108 108
         parent::setUp();
109 109
     }
@@ -122,8 +122,8 @@  discard block
 block discarded – undo
122 122
             'autoptimize_filter_css_inlinesize',
123 123
             'autoptimize_filter_css_fonts_cdn',
124 124
         );
125
-        foreach ( $filter_tags as $filter_tag ) {
126
-            remove_all_filters( $filter_tag );
125
+        foreach ($filter_tags as $filter_tag) {
126
+            remove_all_filters($filter_tag);
127 127
         }
128 128
 
129 129
         parent::tearDown();
@@ -247,15 +247,15 @@  discard block
 block discarded – undo
247 247
     protected function get_test_markup_output()
248 248
     {
249 249
         $key = 'default';
250
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
250
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
251 251
             $key = 'custom';
252 252
         }
253 253
 
254
-        $csspart = self::$flexible_url_parts_css[ $key ];
255
-        $jspart  = self::$flexible_url_parts_js[ $key ];
254
+        $csspart = self::$flexible_url_parts_css[$key];
255
+        $jspart  = self::$flexible_url_parts_js[$key];
256 256
 
257 257
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
258
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
258
+        if (AO_TEST_SUBFOLDER_INSTALL) {
259 259
             $csshash = '56398de576d59887e88e3011715250e0';
260 260
         }
261 261
 
@@ -314,15 +314,15 @@  discard block
 block discarded – undo
314 314
     protected function get_test_markup_output_ms()
315 315
     {
316 316
         $key = 'multisite';
317
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
317
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
318 318
             $key = 'multisite_custom';
319 319
         }
320 320
 
321
-        $csspart = self::$flexible_url_parts_css[ $key ];
322
-        $jspart  = self::$flexible_url_parts_js[ $key ];
321
+        $csspart = self::$flexible_url_parts_css[$key];
322
+        $jspart  = self::$flexible_url_parts_js[$key];
323 323
 
324 324
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
325
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
325
+        if (AO_TEST_SUBFOLDER_INSTALL) {
326 326
             $csshash = '56398de576d59887e88e3011715250e0';
327 327
         }
328 328
 
@@ -381,15 +381,15 @@  discard block
 block discarded – undo
381 381
     protected function get_test_markup_output_inline_defer()
382 382
     {
383 383
         $key = 'default';
384
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
384
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
385 385
             $key = 'custom';
386 386
         }
387 387
 
388
-        $csspart = self::$flexible_url_parts_css[ $key ];
389
-        $jspart  = self::$flexible_url_parts_js[ $key ];
388
+        $csspart = self::$flexible_url_parts_css[$key];
389
+        $jspart  = self::$flexible_url_parts_js[$key];
390 390
 
391 391
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
392
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
392
+        if (AO_TEST_SUBFOLDER_INSTALL) {
393 393
             $csshash = '56398de576d59887e88e3011715250e0';
394 394
         }
395 395
 
@@ -448,15 +448,15 @@  discard block
 block discarded – undo
448 448
     protected function get_test_markup_output_inline_defer_ms()
449 449
     {
450 450
         $key = 'multisite';
451
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
451
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
452 452
             $key = 'multisite_custom';
453 453
         }
454 454
 
455
-        $csspart = self::$flexible_url_parts_css[ $key ];
456
-        $jspart  = self::$flexible_url_parts_js[ $key ];
455
+        $csspart = self::$flexible_url_parts_css[$key];
456
+        $jspart  = self::$flexible_url_parts_js[$key];
457 457
 
458 458
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
459
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
459
+        if (AO_TEST_SUBFOLDER_INSTALL) {
460 460
             $csshash = '56398de576d59887e88e3011715250e0';
461 461
         }
462 462
 
@@ -515,11 +515,11 @@  discard block
 block discarded – undo
515 515
     /**
516 516
      * @dataProvider provider_test_rewrite_markup_with_cdn
517 517
      */
518
-    function test_rewrite_markup_with_cdn( $input, $expected )
518
+    function test_rewrite_markup_with_cdn($input, $expected)
519 519
     {
520
-        $actual = $this->ao->end_buffering( $input );
520
+        $actual = $this->ao->end_buffering($input);
521 521
 
522
-        $this->assertEquals( $expected, $actual );
522
+        $this->assertEquals($expected, $actual);
523 523
     }
524 524
 
525 525
     public function provider_test_rewrite_markup_with_cdn()
@@ -531,7 +531,7 @@  discard block
 block discarded – undo
531 531
                 $this->get_test_markup(),
532 532
                 // Expected output.
533 533
                 // TODO/FIXME: This seemed like the fastest way to get MS crude test to pass...
534
-                ( is_multisite() ? $this->get_test_markup_output_ms() : $this->get_test_markup_output() ),
534
+                (is_multisite() ? $this->get_test_markup_output_ms() : $this->get_test_markup_output()),
535 535
             ),
536 536
 
537 537
         );
@@ -591,12 +591,12 @@  discard block
 block discarded – undo
591 591
 }
592 592
 CSS;
593 593
 
594
-        $instance = new autoptimizeStyles( $css_in );
595
-        $instance->setOption( 'cdn_url', $cdnurl );
594
+        $instance = new autoptimizeStyles($css_in);
595
+        $instance->setOption('cdn_url', $cdnurl);
596 596
 
597
-        $css_actual = $instance->rewrite_assets( $css_in );
597
+        $css_actual = $instance->rewrite_assets($css_in);
598 598
 
599
-        $this->assertEquals( $css_expected, $css_actual );
599
+        $this->assertEquals($css_expected, $css_actual);
600 600
     }
601 601
 
602 602
     public function test_default_cssmin_minifier()
@@ -630,21 +630,21 @@  discard block
 block discarded – undo
630 630
 .bg{background:url('img/something.svg')}.bg-no-quote{background:url(img/something.svg)}.bg-double-quotes{background:url("img/something.svg")}.whitespaces{background:url ("../../somewhere-else/svg.svg")}.host-relative{background:url("/img/something.svg")}.protocol-relative{background:url("//something/somewhere/example.png")}@font-face{font-family:'Roboto';font-style:normal;font-weight:100;src:url(../fonts/roboto-v15-latin-ext_latin-100.eot);src:local('Roboto Thin'),local('Roboto-Thin'),url(../fonts/roboto-v15-latin-ext_latin-100.eot?#iefix) format('embedded-opentype'),url(../fonts/roboto-v15-latin-ext_latin-100.woff2) format('woff2'),url(../fonts/roboto-v15-latin-ext_latin-100.woff) format('woff'),url(../fonts/roboto-v15-latin-ext_latin-100.ttf) format('truetype'),url(../fonts/roboto-v15-latin-ext_latin-100.svg#Roboto) format('svg')}
631 631
 CSS;
632 632
 
633
-        $instance = new autoptimizeStyles( $css );
634
-        $minified = $instance->run_minifier_on( $css );
633
+        $instance = new autoptimizeStyles($css);
634
+        $minified = $instance->run_minifier_on($css);
635 635
 
636
-        $this->assertEquals( $expected, $minified );
636
+        $this->assertEquals($expected, $minified);
637 637
     }
638 638
 
639 639
     /**
640 640
      * @dataProvider provider_test_should_aggregate_script_types
641 641
      */
642
-    public function test_should_aggregate_script_types( $input, $expected )
642
+    public function test_should_aggregate_script_types($input, $expected)
643 643
     {
644
-        $instance = new autoptimizeScripts( '' );
645
-        $actual   = $instance->should_aggregate( $input );
644
+        $instance = new autoptimizeScripts('');
645
+        $actual   = $instance->should_aggregate($input);
646 646
 
647
-        $this->assertEquals( $expected, $actual );
647
+        $this->assertEquals($expected, $actual);
648 648
     }
649 649
 
650 650
     public function provider_test_should_aggregate_script_types()
@@ -706,11 +706,11 @@  discard block
 block discarded – undo
706 706
     /**
707 707
      * @dataProvider provider_is_valid_buffer
708 708
      */
709
-    public function test_valid_buffers( $input, $expected )
709
+    public function test_valid_buffers($input, $expected)
710 710
     {
711
-        $actual = $this->ao->is_valid_buffer( $input );
711
+        $actual = $this->ao->is_valid_buffer($input);
712 712
 
713
-        $this->assertEquals( $expected, $actual );
713
+        $this->assertEquals($expected, $actual);
714 714
     }
715 715
 
716 716
     public function provider_is_valid_buffer()
@@ -772,11 +772,11 @@  discard block
 block discarded – undo
772 772
     /**
773 773
      * @dataProvider provider_is_amp_markup
774 774
      */
775
-    public function test_autoptimize_is_amp_markup( $input, $expected )
775
+    public function test_autoptimize_is_amp_markup($input, $expected)
776 776
     {
777
-        $actual = autoptimizeMain::is_amp_markup( $input );
777
+        $actual = autoptimizeMain::is_amp_markup($input);
778 778
 
779
-        $this->assertEquals( $expected, $actual );
779
+        $this->assertEquals($expected, $actual);
780 780
     }
781 781
 
782 782
     public function provider_is_amp_markup()
@@ -807,31 +807,31 @@  discard block
 block discarded – undo
807 807
     public function test_skips_buffering_when_ao_noptimize_filter_is_true()
808 808
     {
809 809
         // True => disable autoptimize.
810
-        add_filter( 'autoptimize_filter_noptimize', '__return_true' );
810
+        add_filter('autoptimize_filter_noptimize', '__return_true');
811 811
 
812 812
         // Buffering should not run due to the above filter.
813 813
         $expected = false;
814
-        $actual   = $this->ao->should_buffer( $doing_tests = true );
814
+        $actual   = $this->ao->should_buffer($doing_tests = true);
815 815
 
816
-        $this->assertEquals( $expected, $actual );
816
+        $this->assertEquals($expected, $actual);
817 817
     }
818 818
 
819 819
     public function test_does_buffering_when_ao_noptimize_filter_is_false()
820 820
     {
821 821
         // False => disable noptimize, aka, run normally (weird, yes...).
822
-        add_filter( 'autoptimize_filter_noptimize', '__return_false' );
822
+        add_filter('autoptimize_filter_noptimize', '__return_false');
823 823
 
824 824
         // Buffering should run because of above.
825 825
         $expected = true;
826
-        $actual   = $this->ao->should_buffer( $doing_tests = true );
826
+        $actual   = $this->ao->should_buffer($doing_tests = true);
827 827
 
828
-        $this->assertEquals( $expected, $actual );
828
+        $this->assertEquals($expected, $actual);
829 829
     }
830 830
 
831 831
     public function test_ignores_ao_noptimize_qs_when_instructed()
832 832
     {
833 833
         // Should skip checking for the qs completely due to filter.
834
-        add_filter( 'autoptimize_filter_honor_qs_noptimize', '__return_false' );
834
+        add_filter('autoptimize_filter_honor_qs_noptimize', '__return_false');
835 835
 
836 836
         /**
837 837
          * The above should then result in the "current" value being `false`
@@ -842,23 +842,23 @@  discard block
 block discarded – undo
842 842
          */
843 843
 
844 844
         $that = $this; // Makes it work on php 5.3!
845
-        add_filter( 'autoptimize_filter_noptimize', function ( $current_value ) use ( $that ) {
845
+        add_filter('autoptimize_filter_noptimize', function($current_value) use ($that) {
846 846
             $expected = false;
847
-            if ( defined( 'DONOTMINIFY' ) && DONOTMINIFY ) {
847
+            if (defined('DONOTMINIFY') && DONOTMINIFY) {
848 848
                 $expected = true;
849 849
             }
850 850
 
851
-            $that->assertEquals( $expected, $current_value );
851
+            $that->assertEquals($expected, $current_value);
852 852
         });
853 853
 
854
-        $this->ao->should_buffer( $doing_tests = true );
854
+        $this->ao->should_buffer($doing_tests = true);
855 855
     }
856 856
 
857 857
     public function test_wpengine_cache_flush()
858 858
     {
859 859
         // Creating a mock so that we can get past class_exists() and method_exists() checks present
860 860
         // in `autoptimizeCache::flushPageCache()`...
861
-        $stub = $this->getMockBuilder( 'WpeCommon' )->disableAutoload()
861
+        $stub = $this->getMockBuilder('WpeCommon')->disableAutoload()
862 862
                 ->disableOriginalConstructor()->setMethods(
863 863
                     array(
864 864
                         'purge_varnish_cache',
@@ -867,9 +867,9 @@  discard block
 block discarded – undo
867 867
                 ->getMock();
868 868
 
869 869
         $that = $this;
870
-        add_filter( 'autoptimize_flush_wpengine_methods', function( $methods ) use ( $that ) {
871
-            $expected_methods = array( 'purge_varnish_cache' );
872
-            $that->assertEquals( $methods, $expected_methods );
870
+        add_filter('autoptimize_flush_wpengine_methods', function($methods) use ($that) {
871
+            $expected_methods = array('purge_varnish_cache');
872
+            $that->assertEquals($methods, $expected_methods);
873 873
 
874 874
             return $methods;
875 875
         });
@@ -883,7 +883,7 @@  discard block
 block discarded – undo
883 883
     public function test_wpengine_cache_flush_agressive()
884 884
     {
885 885
         // Creating a mock so that we can get past class_exists() and method_exists() checks `autoptimize_flush_pagecache()`...
886
-        $stub = $this->getMockBuilder( 'WpeCommon' )->disableAutoload()
886
+        $stub = $this->getMockBuilder('WpeCommon')->disableAutoload()
887 887
                 ->disableOriginalConstructor()->setMethods(
888 888
                     array(
889 889
                         'purge_varnish_cache',
@@ -893,19 +893,19 @@  discard block
 block discarded – undo
893 893
                 )
894 894
                 ->getMock();
895 895
 
896
-        add_filter( 'autoptimize_flush_wpengine_aggressive', function() {
896
+        add_filter('autoptimize_flush_wpengine_aggressive', function() {
897 897
             return true;
898 898
         });
899 899
 
900 900
         $that = $this;
901
-        add_filter( 'autoptimize_flush_wpengine_methods', function( $methods ) use ( $that ) {
901
+        add_filter('autoptimize_flush_wpengine_methods', function($methods) use ($that) {
902 902
             $expected_methods = array(
903 903
                 'purge_varnish_cache',
904 904
                 'purge_memcached',
905 905
                 'clear_maxcdn_cache',
906 906
             );
907 907
 
908
-            $that->assertEquals( $methods, $expected_methods );
908
+            $that->assertEquals($methods, $expected_methods);
909 909
 
910 910
             return $methods;
911 911
         });
@@ -916,15 +916,15 @@  discard block
 block discarded – undo
916 916
     /**
917 917
      * @dataProvider provider_test_url_replace_cdn
918 918
      */
919
-    public function test_url_replace_cdn( $cdn_url, $input, $expected )
919
+    public function test_url_replace_cdn($cdn_url, $input, $expected)
920 920
     {
921
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )
921
+        $mock = $this->getMockBuilder('autoptimizeBase')
922 922
                 ->disableOriginalConstructor()
923 923
                 ->getMockForAbstractClass();
924 924
 
925 925
         $mock->cdn_url = $cdn_url;
926
-        $actual        = $mock->url_replace_cdn( $input );
927
-        $this->assertEquals( $expected, $actual );
926
+        $actual        = $mock->url_replace_cdn($input);
927
+        $this->assertEquals($expected, $actual);
928 928
     }
929 929
 
930 930
     public function provider_test_url_replace_cdn()
@@ -941,13 +941,13 @@  discard block
 block discarded – undo
941 941
                 // CDN base url, url, expected result...
942 942
                 'http://cdn-test.example.org',
943 943
                 '/a.jpg',
944
-                'http://cdn-test.example.org/' . $subfolder . 'a.jpg',
944
+                'http://cdn-test.example.org/'.$subfolder.'a.jpg',
945 945
             ),
946 946
             // Full link with a matching AUTOPTIMIZE_WP_SITE_URL gets properly replaced...
947 947
             array(
948 948
                 'http://cdn-test.example.org',
949
-                $siteurl . '/wp-content/themes/something/example.svg',
950
-                'http://cdn-test.example.org/' . $subfolder . 'wp-content/themes/something/example.svg',
949
+                $siteurl.'/wp-content/themes/something/example.svg',
950
+                'http://cdn-test.example.org/'.$subfolder.'wp-content/themes/something/example.svg',
951 951
             ),
952 952
             // Protocol-relative url with a "local" hostname that doesn't match example.org (AUTOPTIMIZE_WP_SITE_URL)...
953 953
             array(
@@ -958,59 +958,59 @@  discard block
 block discarded – undo
958 958
             // www.example.org does not match example.org (AUTOPTIMIZE_WP_SITE_URL) so it's left alone...
959 959
             array(
960 960
                 'http://cdn-test.example.org',
961
-                $wwwsiteurl . '/wp-content/themes/something/example.svg',
962
-                $wwwsiteurl . '/wp-content/themes/something/example.svg',
961
+                $wwwsiteurl.'/wp-content/themes/something/example.svg',
962
+                $wwwsiteurl.'/wp-content/themes/something/example.svg',
963 963
             ),
964 964
             // SSL cdn url + host-relative link...
965 965
             array(
966 966
                 'https://cdn.example.org',
967 967
                 '/a.jpg',
968
-                'https://cdn.example.org/' . $subfolder . 'a.jpg',
968
+                'https://cdn.example.org/'.$subfolder.'a.jpg',
969 969
             ),
970 970
             // SSL cdn url + http site url that matches AUTOPTIMIZE_WP_SITE_URL is properly replaced...
971 971
             array(
972 972
                 'https://cdn.example.org',
973
-                $siteurl . '/wp-content/themes/something/example.svg',
974
-                'https://cdn.example.org/' . $subfolder . 'wp-content/themes/something/example.svg',
973
+                $siteurl.'/wp-content/themes/something/example.svg',
974
+                'https://cdn.example.org/'.$subfolder.'wp-content/themes/something/example.svg',
975 975
             ),
976 976
             // Protocol-relative cdn url given with protocol relative link that matches AUTOPTIMIZE_WP_SITE_URL host...
977 977
             array(
978 978
                 '//cdn.example.org',
979
-                $protorelativesiteurl . '/something.jpg',
980
-                '//cdn.example.org/' . $subfolder . 'something.jpg',
979
+                $protorelativesiteurl.'/something.jpg',
980
+                '//cdn.example.org/'.$subfolder.'something.jpg',
981 981
             ),
982 982
             // Protocol-relative cdn url given a http link that matches AUTOPTIMIZE_WP_SITE_URL host...
983 983
             array(
984 984
                 '//cdn.example.org',
985
-                $siteurl . '/something.png',
986
-                '//cdn.example.org/' . $subfolder . 'something.png',
985
+                $siteurl.'/something.png',
986
+                '//cdn.example.org/'.$subfolder.'something.png',
987 987
             ),
988 988
             // Protocol-relative cdn url with a host-relative link...
989 989
             array(
990 990
                 '//cdn.example.org',
991 991
                 '/a.jpg',
992
-                '//cdn.example.org/' . $subfolder . 'a.jpg',
992
+                '//cdn.example.org/'.$subfolder.'a.jpg',
993 993
             ),
994 994
             // Testing cdn urls with an explicit port number...
995 995
             array(
996 996
                 'http://cdn.com:8080',
997 997
                 '/a.jpg',
998
-                'http://cdn.com:8080/' . $subfolder . 'a.jpg',
998
+                'http://cdn.com:8080/'.$subfolder.'a.jpg',
999 999
             ),
1000 1000
             array(
1001 1001
                 '//cdn.com:4433',
1002 1002
                 '/a.jpg',
1003
-                '//cdn.com:4433/' . $subfolder . 'a.jpg',
1003
+                '//cdn.com:4433/'.$subfolder.'a.jpg',
1004 1004
             ),
1005 1005
             array(
1006 1006
                 '//cdn.com:4433',
1007
-                $siteurl . '/something.jpg',
1008
-                '//cdn.com:4433/' . $subfolder . 'something.jpg',
1007
+                $siteurl.'/something.jpg',
1008
+                '//cdn.com:4433/'.$subfolder.'something.jpg',
1009 1009
             ),
1010 1010
             array(
1011 1011
                 '//cdn.com:1234',
1012
-                $protorelativesiteurl . '/something.jpg',
1013
-                '//cdn.com:1234/' . $subfolder . 'something.jpg',
1012
+                $protorelativesiteurl.'/something.jpg',
1013
+                '//cdn.com:1234/'.$subfolder.'something.jpg',
1014 1014
             ),
1015 1015
             // Relative links should not be touched by url_replace_cdn()...
1016 1016
             array(
@@ -1035,8 +1035,8 @@  discard block
 block discarded – undo
1035 1035
             // even though they are really strange...
1036 1036
             array(
1037 1037
                 $siteurl, // example.org or http://localhost or http://localhost/wordpress
1038
-                $siteurl . '/something.jpg',
1039
-                $siteurl . '/something.jpg',
1038
+                $siteurl.'/something.jpg',
1039
+                $siteurl.'/something.jpg',
1040 1040
             ),
1041 1041
             // These shouldn't really be changed, or even if replacements do
1042 1042
             // happen, they shouldn't be destructive...
@@ -1075,31 +1075,31 @@  discard block
 block discarded – undo
1075 1075
         $test_link = '/a.jpg';
1076 1076
         $cdn_url   = '//cdn.example.org';
1077 1077
 
1078
-        $with_ssl = function( $cdn ) {
1078
+        $with_ssl = function($cdn) {
1079 1079
             return '';
1080 1080
         };
1081 1081
         $expected_with_ssl = '/a.jpg';
1082 1082
 
1083
-        $without_ssl = function( $cdn ) {
1083
+        $without_ssl = function($cdn) {
1084 1084
             return $cdn;
1085 1085
         };
1086
-        $expected_without_ssl = '//cdn.example.org/' . $sub . 'a.jpg';
1086
+        $expected_without_ssl = '//cdn.example.org/'.$sub.'a.jpg';
1087 1087
 
1088 1088
         // With a filter that returns something considered "empty", cdn replacement shouldn't occur...
1089
-        add_filter( 'autoptimize_filter_base_cdnurl', $with_ssl );
1090
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )
1089
+        add_filter('autoptimize_filter_base_cdnurl', $with_ssl);
1090
+        $mock = $this->getMockBuilder('autoptimizeBase')
1091 1091
                 ->disableOriginalConstructor()
1092 1092
                 ->getMockForAbstractClass();
1093 1093
 
1094 1094
         $mock->cdn_url   = $cdn_url;
1095
-        $actual_with_ssl = $mock->url_replace_cdn( $test_link );
1096
-        $this->assertEquals( $expected_with_ssl, $actual_with_ssl );
1097
-        remove_filter( 'autoptimize_filter_base_cdnurl', $with_ssl );
1095
+        $actual_with_ssl = $mock->url_replace_cdn($test_link);
1096
+        $this->assertEquals($expected_with_ssl, $actual_with_ssl);
1097
+        remove_filter('autoptimize_filter_base_cdnurl', $with_ssl);
1098 1098
 
1099 1099
         // With a filter that returns an actual cdn url, cdn replacement should occur.
1100
-        add_filter( 'autoptimize_filter_base_cdnurl', $without_ssl );
1101
-        $actual_without_ssl = $mock->url_replace_cdn( $test_link );
1102
-        $this->assertEquals( $expected_without_ssl, $actual_without_ssl );
1100
+        add_filter('autoptimize_filter_base_cdnurl', $without_ssl);
1101
+        $actual_without_ssl = $mock->url_replace_cdn($test_link);
1102
+        $this->assertEquals($expected_without_ssl, $actual_without_ssl);
1103 1103
     }
1104 1104
 
1105 1105
     public function provider_cssmin_issues()
@@ -1138,11 +1138,11 @@  discard block
 block discarded – undo
1138 1138
     /**
1139 1139
      * @dataProvider provider_cssmin_issues
1140 1140
      */
1141
-    public function test_cssmin_issues( $input, $expected )
1141
+    public function test_cssmin_issues($input, $expected)
1142 1142
     {
1143
-        $minifier = new autoptimizeCSSmin( false ); // No need to raise limits for now.
1144
-        $actual   = $minifier->run( $input );
1145
-        $this->assertEquals( $expected, $actual );
1143
+        $minifier = new autoptimizeCSSmin(false); // No need to raise limits for now.
1144
+        $actual   = $minifier->run($input);
1145
+        $this->assertEquals($expected, $actual);
1146 1146
     }
1147 1147
 
1148 1148
     public function provider_getpath()
@@ -1167,7 +1167,7 @@  discard block
 block discarded – undo
1167 1167
             // This file comes with core, so should exist...
1168 1168
             array(
1169 1169
                 '/wp-includes/js/jquery/jquery.js',
1170
-                WP_ROOT_DIR . '/wp-includes/js/jquery/jquery.js',
1170
+                WP_ROOT_DIR.'/wp-includes/js/jquery/jquery.js',
1171 1171
             ),
1172 1172
             // Empty $url should return false.
1173 1173
             array(
@@ -1192,12 +1192,12 @@  discard block
 block discarded – undo
1192 1192
     /**
1193 1193
      * @dataProvider provider_getpath
1194 1194
      */
1195
-    public function test_getpath( $input, $expected )
1195
+    public function test_getpath($input, $expected)
1196 1196
     {
1197
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )->disableOriginalConstructor()->getMockForAbstractClass();
1197
+        $mock = $this->getMockBuilder('autoptimizeBase')->disableOriginalConstructor()->getMockForAbstractClass();
1198 1198
 
1199
-        $actual = $mock->getpath( $input );
1200
-        $this->assertEquals( $expected, $actual );
1199
+        $actual = $mock->getpath($input);
1200
+        $this->assertEquals($expected, $actual);
1201 1201
     }
1202 1202
 
1203 1203
     /**
@@ -1211,7 +1211,7 @@  discard block
 block discarded – undo
1211 1211
 
1212 1212
         $protorelativesiteurl = $urls['prsiteurl'];
1213 1213
 
1214
-        $css_orig     = <<<CSS
1214
+        $css_orig = <<<CSS
1215 1215
 header{background-image:url("data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' width='90px' height='110px' viewBox='0 0 90 110'%3E%3Cstyle%3E.a%7Bstop-color:%23FFF;%7D.b%7Bstop-color:%23B2D235;%7D.c%7Bstop-color:%23BEE7FA;%7D.d%7Bfill:%23590C15;%7D%3C/style%3E%3ClinearGradient id='c' y2='135.4' gradientUnits='userSpaceOnUse' x2='209.1' gradientTransform='rotate(-1.467 -4082.888 7786.794)' y1='205.8' x1='262'%3E%3Cstop class='b' offset='0'/%3E%3Cstop class='b' offset='.48'/%3E%3Cstop stop-color='%23829D25' offset='1'/%3E%3C/linearGradient%3E%3Cpath stroke-width='.3' d='M77.3 45.4c-3-3.5-7.1-6.5-11.6-7.8-5.1-1.5-10-.1-14.9 1.5C52 35.4 54.3 29 60 24l-4.8-5.5c-3.4 3-5.8 6.3-7.5 9.4-1.7-4.3-4.1-8.4-7.5-12C33.4 8.6 24.3 4.7 15.1 4.2c-.2 9.3 3.1 18.6 9.9 25.9 5.2 5.6 11.8 9.2 18.7 10.8-2.5.2-4.9-.1-7.7-.9-5.2-1.4-10.5-2.8-15.8-1C10.6 42.3 4.5 51.9 4 61.7c-.5 11.6 3.8 23.8 9.9 33.5 3.9 6.3 9.6 13.7 17.7 13.4 3.8-.1 7-2.1 10.7-2.7 5.2-.8 9.1 1.2 14.1 1.8 16.4 2 24.4-23.6 26.4-35.9 1.2-9.1.8-19.1-5.5-26.4z' stroke='%233E6D1F' fill='url(%23c)'/%3E%3C/svg%3E")}
1216 1216
 section.clipped.clippedTop {clip-path:url("#clipPolygonTop")}
1217 1217
 section.clipped.clippedBottom {clip-path:url("#clipPolygonBottom")}
@@ -1224,13 +1224,13 @@  discard block
 block discarded – undo
1224 1224
 .myimg {background-image: url($protorelativesiteurl/wp-content/themes/my-theme/images/under-left-leaf.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/over-blue-bird.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/under-top.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/bg-top-grunge.png);}
1225 1225
 CSS;
1226 1226
 
1227
-        $fixurls_result = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $css_orig );
1228
-        $this->assertEquals( $css_expected, $fixurls_result );
1227
+        $fixurls_result = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $css_orig);
1228
+        $this->assertEquals($css_expected, $fixurls_result);
1229 1229
     }
1230 1230
 
1231 1231
     public function test_background_datauri_sprites_with_fixurls()
1232 1232
     {
1233
-        $css_orig     = <<<CSS
1233
+        $css_orig = <<<CSS
1234 1234
 .shadow { background:url(img/1x1.png) top center; }
1235 1235
 .shadow1 { background-image:url(img/1x1.png) 0 -767px repeat-x; }
1236 1236
 .shadow2 {background:url(img/1x1.png) top center}
@@ -1282,27 +1282,27 @@  discard block
 block discarded – undo
1282 1282
 CSS;
1283 1283
 
1284 1284
         // For test purposes, ALL images in the css are being inline with a 1x1 trans png string/datauri.
1285
-        add_filter( 'autoptimize_filter_css_is_datauri_candidate', function( $is_candidate, $path ) {
1285
+        add_filter('autoptimize_filter_css_is_datauri_candidate', function($is_candidate, $path) {
1286 1286
             return true;
1287
-        }, 10, 2 );
1287
+        }, 10, 2);
1288 1288
 
1289 1289
         // For test purposes, ALL images in the css are being inline with a 1x1 trans png string/datauri.
1290
-        add_filter( 'autoptimize_filter_css_datauri_image', function( $base64array, $path ) {
1290
+        add_filter('autoptimize_filter_css_datauri_image', function($base64array, $path) {
1291 1291
             $head = 'data:image/png;base64,';
1292 1292
             $data = 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAQAAAC1HAwCAAAAC0lEQVR42mNkYAAAAAYAAjCB0C8AAAAASUVORK5CYII=';
1293 1293
 
1294
-            $result['full']       = $head . $data;
1294
+            $result['full']       = $head.$data;
1295 1295
             $result['base64data'] = $data;
1296 1296
             return $result;
1297
-        }, 10, 2 );
1297
+        }, 10, 2);
1298 1298
 
1299
-        $instance = new autoptimizeStyles( $css_orig );
1300
-        $instance->setOption( 'datauris', true );
1299
+        $instance = new autoptimizeStyles($css_orig);
1300
+        $instance->setOption('datauris', true);
1301 1301
 
1302
-        $fixurls_result = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $css_orig );
1303
-        $css_actual     = $instance->rewrite_assets( $fixurls_result );
1302
+        $fixurls_result = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $css_orig);
1303
+        $css_actual     = $instance->rewrite_assets($fixurls_result);
1304 1304
 
1305
-        $this->assertEquals( $css_expected, $css_actual );
1305
+        $this->assertEquals($css_expected, $css_actual);
1306 1306
     }
1307 1307
 
1308 1308
     /**
@@ -1322,7 +1322,7 @@  discard block
 block discarded – undo
1322 1322
      */
1323 1323
     public function test_background_datauri_sprites_without_fixurls()
1324 1324
     {
1325
-        $css_orig     = <<<CSS
1325
+        $css_orig = <<<CSS
1326 1326
 .shadow { background:url(img/1x1.png) top center; }
1327 1327
 .shadow1 { background-image:url(img/1x1.png) 0 -767px repeat-x; }
1328 1328
 .shadow2 {background:url(img/1x1.png) top center}
@@ -1366,24 +1366,24 @@  discard block
 block discarded – undo
1366 1366
 CSS;
1367 1367
 
1368 1368
         // For test purposes, ALL images in the css are being inlined with a 1x1 trans png string/datauri.
1369
-        add_filter( 'autoptimize_filter_css_is_datauri_candidate', function( $is_candidate, $path ) {
1369
+        add_filter('autoptimize_filter_css_is_datauri_candidate', function($is_candidate, $path) {
1370 1370
             return true;
1371
-        }, 10, 2 );
1371
+        }, 10, 2);
1372 1372
 
1373 1373
         // For test purposes, ALL images in the css are being inlined with a 1x1 trans png string/datauri.
1374
-        add_filter( 'autoptimize_filter_css_datauri_image', function( $base64array, $path ) {
1374
+        add_filter('autoptimize_filter_css_datauri_image', function($base64array, $path) {
1375 1375
             $head = 'data:image/png;base64,';
1376 1376
             $data = 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAQAAAC1HAwCAAAAC0lEQVR42mNkYAAAAAYAAjCB0C8AAAAASUVORK5CYII=';
1377 1377
 
1378
-            $result['full']       = $head . $data;
1378
+            $result['full']       = $head.$data;
1379 1379
             $result['base64data'] = $data;
1380 1380
             return $result;
1381
-        }, 10, 2 );
1381
+        }, 10, 2);
1382 1382
 
1383
-        $instance = new autoptimizeStyles( $css_orig );
1384
-        $instance->setOption( 'datauris', true );
1385
-        $css_actual = $instance->rewrite_assets( $css_orig );
1386
-        $this->assertEquals( $css_expected, $css_actual );
1383
+        $instance = new autoptimizeStyles($css_orig);
1384
+        $instance->setOption('datauris', true);
1385
+        $css_actual = $instance->rewrite_assets($css_orig);
1386
+        $this->assertEquals($css_expected, $css_actual);
1387 1387
     }
1388 1388
 
1389 1389
     /**
@@ -1503,12 +1503,12 @@  discard block
 block discarded – undo
1503 1503
 CSS;
1504 1504
 
1505 1505
         // Test with fonts pointed to the CDN + cdn option is set.
1506
-        add_filter( 'autoptimize_filter_css_fonts_cdn', '__return_true' );
1507
-        $instance = new autoptimizeStyles( $css_in );
1508
-        $instance->setOption( 'cdn_url', $cdnurl );
1509
-        $css_actual_fonts_cdn = $instance->rewrite_assets( $css_in );
1506
+        add_filter('autoptimize_filter_css_fonts_cdn', '__return_true');
1507
+        $instance = new autoptimizeStyles($css_in);
1508
+        $instance->setOption('cdn_url', $cdnurl);
1509
+        $css_actual_fonts_cdn = $instance->rewrite_assets($css_in);
1510 1510
 
1511
-        $this->assertEquals( $css_expected_fonts_cdn, $css_actual_fonts_cdn );
1511
+        $this->assertEquals($css_expected_fonts_cdn, $css_actual_fonts_cdn);
1512 1512
     }
1513 1513
 
1514 1514
     /**
@@ -1624,10 +1624,10 @@  discard block
 block discarded – undo
1624 1624
 }
1625 1625
 CSS;
1626 1626
         // Test without moving fonts to CDN, but cdn option is set.
1627
-        $instance = new autoptimizeStyles( $css_in );
1628
-        $instance->setOption( 'cdn_url', $urls['cdnurl'] );
1629
-        $css_actual = $instance->rewrite_assets( $css_in );
1630
-        $this->assertEquals( $css_expected, $css_actual );
1627
+        $instance = new autoptimizeStyles($css_in);
1628
+        $instance->setOption('cdn_url', $urls['cdnurl']);
1629
+        $css_actual = $instance->rewrite_assets($css_in);
1630
+        $this->assertEquals($css_expected, $css_actual);
1631 1631
     }
1632 1632
 
1633 1633
     public function test_assets_regex_replaces_multi_bg_images()
@@ -1648,16 +1648,16 @@  discard block
 block discarded – undo
1648 1648
 }
1649 1649
 CSS;
1650 1650
 
1651
-        $instance = new autoptimizeStyles( $in );
1652
-        $instance->setOption( 'cdn_url', $cdnurl );
1653
-        $actual = $instance->rewrite_assets( $in );
1651
+        $instance = new autoptimizeStyles($in);
1652
+        $instance->setOption('cdn_url', $cdnurl);
1653
+        $actual = $instance->rewrite_assets($in);
1654 1654
 
1655
-        $this->assertEquals( $expected, $actual );
1655
+        $this->assertEquals($expected, $actual);
1656 1656
     }
1657 1657
 
1658 1658
     public function test_at_supports_spacing_issue_110()
1659 1659
     {
1660
-        $in       = <<<CSS
1660
+        $in = <<<CSS
1661 1661
 @supports (-webkit-filter: blur(3px)) or (filter: blur(3px)) {
1662 1662
     .blur {
1663 1663
         filter:blur(3px);
@@ -1671,10 +1671,10 @@  discard block
 block discarded – undo
1671 1671
 @supports (-webkit-filter:blur(3px)) or (filter:blur(3px)){.blur{filter:blur(3px)}}@supports((position:-webkit-sticky) or (position:sticky)){.sticky{position:sticky}}
1672 1672
 CSS;
1673 1673
 
1674
-        $instance = new autoptimizeStyles( $in );
1675
-        $actual   = $instance->run_minifier_on( $in );
1674
+        $instance = new autoptimizeStyles($in);
1675
+        $actual   = $instance->run_minifier_on($in);
1676 1676
 
1677
-        $this->assertEquals( $expected, $actual );
1677
+        $this->assertEquals($expected, $actual);
1678 1678
     }
1679 1679
 
1680 1680
     public function test_css_import_semicolon_url_issue_122()
@@ -1690,18 +1690,18 @@  discard block
 block discarded – undo
1690 1690
 </style>
1691 1691
 HTML;
1692 1692
 
1693
-        $expected = '<style type="text/css" media="all">@import url(' . $cdnurl . '/' . $sub . 'foo.css?a&#038;b);@import url(' . $cdnurl . '/' . $sub . 'bar.css);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1693
+        $expected = '<style type="text/css" media="all">@import url('.$cdnurl.'/'.$sub.'foo.css?a&#038;b);@import url('.$cdnurl.'/'.$sub.'bar.css);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1694 1694
 
1695 1695
         $options = [
1696 1696
             'autoptimizeStyles' => $this->getAoStylesDefaultOptions(),
1697 1697
         ];
1698 1698
 
1699
-        $instance = new autoptimizeStyles( $in );
1700
-        $instance->read( $options['autoptimizeStyles'] );
1699
+        $instance = new autoptimizeStyles($in);
1700
+        $instance->read($options['autoptimizeStyles']);
1701 1701
         $instance->minify();
1702 1702
         $instance->cache();
1703 1703
         $actual = $instance->getcontent();
1704
-        $this->assertEquals( $expected, $actual );
1704
+        $this->assertEquals($expected, $actual);
1705 1705
     }
1706 1706
 
1707 1707
     public function test_fixurls_with_at_imports_and_media_queries()
@@ -1711,10 +1711,10 @@  discard block
 block discarded – undo
1711 1711
         $subfolder = $urls['subfolder'];
1712 1712
 
1713 1713
         $in  = '@import "foo.css"; @import "bar.css" (orientation:landscape);';
1714
-        $exp = '@import url(' . $prsiteurl . '/wp-content/themes/my-theme/foo.css); @import url(' . $prsiteurl . '/wp-content/themes/my-theme/bar.css) (orientation:landscape);';
1714
+        $exp = '@import url('.$prsiteurl.'/wp-content/themes/my-theme/foo.css); @import url('.$prsiteurl.'/wp-content/themes/my-theme/bar.css) (orientation:landscape);';
1715 1715
 
1716
-        $actual = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $in );
1717
-        $this->assertEquals( $exp, $actual );
1716
+        $actual = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $in);
1717
+        $this->assertEquals($exp, $actual);
1718 1718
     }
1719 1719
 
1720 1720
     public function test_aostyles_at_imports_with_media_queries()
@@ -1729,43 +1729,43 @@  discard block
 block discarded – undo
1729 1729
 </style>
1730 1730
 HTML;
1731 1731
 
1732
-        $expected = '<style type="text/css" media="all">@import url(' . $cdnurl . '/' . $sub . 'foo.css);@import url(' . $cdnurl . '/' . $sub . 'bar.css) (orientation:landscape);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1732
+        $expected = '<style type="text/css" media="all">@import url('.$cdnurl.'/'.$sub.'foo.css);@import url('.$cdnurl.'/'.$sub.'bar.css) (orientation:landscape);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1733 1733
 
1734 1734
         $options = [
1735 1735
             'autoptimizeStyles' => $this->getAoStylesDefaultOptions(),
1736 1736
         ];
1737 1737
 
1738
-        $instance = new autoptimizeStyles( $in );
1739
-        $instance->read( $options['autoptimizeStyles'] );
1738
+        $instance = new autoptimizeStyles($in);
1739
+        $instance->read($options['autoptimizeStyles']);
1740 1740
         $instance->minify();
1741 1741
         $instance->cache();
1742 1742
 
1743 1743
         $actual = $instance->getcontent();
1744
-        $this->assertEquals( $expected, $actual );
1744
+        $this->assertEquals($expected, $actual);
1745 1745
     }
1746 1746
 
1747 1747
     public function test_cache_size_checker_hooked_by_default()
1748 1748
     {
1749
-        $this->assertNotEmpty( autoptimizeCacheChecker::SCHEDULE_HOOK );
1749
+        $this->assertNotEmpty(autoptimizeCacheChecker::SCHEDULE_HOOK);
1750 1750
 
1751 1751
         // No schedule, because it's only added when is_admin() is true.
1752
-        $this->assertEquals( false, wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1752
+        $this->assertEquals(false, wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1753 1753
 
1754 1754
         // Proving that setup() sets the schedule as needed.
1755 1755
         $checker = new autoptimizeCacheChecker();
1756 1756
         $checker->setup();
1757
-        $this->assertEquals( 'twicedaily', wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1757
+        $this->assertEquals('twicedaily', wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1758 1758
     }
1759 1759
 
1760 1760
     public function test_cache_size_checker_disabled_with_filter()
1761 1761
     {
1762
-        add_filter( 'autoptimize_filter_cachecheck_do', '__return_false' );
1762
+        add_filter('autoptimize_filter_cachecheck_do', '__return_false');
1763 1763
 
1764 1764
         $checker = new autoptimizeCacheChecker();
1765 1765
         $checker->setup();
1766
-        $this->assertEquals( false, wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1766
+        $this->assertEquals(false, wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1767 1767
 
1768
-        remove_all_filters( 'autoptimize_filter_cachecheck_do' );
1768
+        remove_all_filters('autoptimize_filter_cachecheck_do');
1769 1769
     }
1770 1770
 
1771 1771
     public function test_is_start_buffering_hooked_properly()
@@ -1778,12 +1778,12 @@  discard block
 block discarded – undo
1778 1778
          * https://www.theaveragedev.com/mocking-constants-in-tests/.
1779 1779
          */
1780 1780
 
1781
-        if ( defined( 'AUTOPTIMIZE_INIT_EARLIER' ) ) {
1781
+        if (defined('AUTOPTIMIZE_INIT_EARLIER')) {
1782 1782
             $this->assertEquals(
1783 1783
                 autoptimizeMain::INIT_EARLIER_PRIORITY,
1784
-                has_action( 'init', array( $instance, 'start_buffering' ) )
1784
+                has_action('init', array($instance, 'start_buffering'))
1785 1785
             );
1786
-            $this->assertTrue( ! defined( 'AUTOPTIMIZE_HOOK_INTO' ) );
1786
+            $this->assertTrue(!defined('AUTOPTIMIZE_HOOK_INTO'));
1787 1787
         }
1788 1788
 
1789 1789
         /*
@@ -1800,18 +1800,18 @@  discard block
 block discarded – undo
1800 1800
 
1801 1801
     public function test_inline_and_defer_markup()
1802 1802
     {
1803
-        add_filter( 'autoptimize_filter_css_defer', '__return_true' );
1804
-        add_filter( 'autoptimize_filter_css_defer_inline', '__return_true' );
1803
+        add_filter('autoptimize_filter_css_defer', '__return_true');
1804
+        add_filter('autoptimize_filter_css_defer_inline', '__return_true');
1805 1805
 
1806
-        $actual = $this->ao->end_buffering( $this->get_test_markup() );
1807
-        if ( is_multisite() ) {
1808
-            $this->assertEquals( $this->get_test_markup_output_inline_defer_ms(), $actual );
1806
+        $actual = $this->ao->end_buffering($this->get_test_markup());
1807
+        if (is_multisite()) {
1808
+            $this->assertEquals($this->get_test_markup_output_inline_defer_ms(), $actual);
1809 1809
         } else {
1810
-            $this->assertEquals( $this->get_test_markup_output_inline_defer(), $actual );
1810
+            $this->assertEquals($this->get_test_markup_output_inline_defer(), $actual);
1811 1811
         }
1812 1812
 
1813
-        remove_all_filters( 'autoptimize_filter_css_defer' );
1814
-        remove_all_filters( 'autoptimize_filter_css_defer_inline' );
1813
+        remove_all_filters('autoptimize_filter_css_defer');
1814
+        remove_all_filters('autoptimize_filter_css_defer_inline');
1815 1815
     }
1816 1816
 
1817 1817
     public function test_js_aggregation_decision_and_dontaggregate_filter()
@@ -1819,47 +1819,47 @@  discard block
 block discarded – undo
1819 1819
         $opts = $this->getAoScriptsDefaultOptions();
1820 1820
 
1821 1821
         // Aggregating: true by default.
1822
-        $scripts = new autoptimizeScripts( '' );
1823
-        $scripts->read( $opts );
1824
-        $this->assertTrue( $scripts->aggregating() );
1822
+        $scripts = new autoptimizeScripts('');
1823
+        $scripts->read($opts);
1824
+        $this->assertTrue($scripts->aggregating());
1825 1825
 
1826 1826
         // Aggregating: option=true (dontaggregate=false by default).
1827 1827
         $opts['aggregate'] = true;
1828
-        $scripts           = new autoptimizeScripts( '' );
1829
-        $scripts->read( $opts );
1830
-        $this->assertTrue( $scripts->aggregating() );
1828
+        $scripts           = new autoptimizeScripts('');
1829
+        $scripts->read($opts);
1830
+        $this->assertTrue($scripts->aggregating());
1831 1831
 
1832 1832
         // Aggregating: option=true, dontaggregate=false explicit.
1833 1833
         $opts['aggregate'] = true;
1834
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_false' );
1835
-        $scripts = new autoptimizeScripts( '' );
1836
-        $scripts->read( $opts );
1837
-        $this->assertTrue( $scripts->aggregating() );
1838
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1834
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_false');
1835
+        $scripts = new autoptimizeScripts('');
1836
+        $scripts->read($opts);
1837
+        $this->assertTrue($scripts->aggregating());
1838
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1839 1839
 
1840 1840
         // Not aggregating: option=true, dontaggregate=true.
1841 1841
         $opts['aggregate'] = true;
1842
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_true' );
1843
-        $scripts = new autoptimizeScripts( '' );
1844
-        $scripts->read( $opts );
1845
-        $this->assertFalse( $scripts->aggregating() );
1846
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1842
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_true');
1843
+        $scripts = new autoptimizeScripts('');
1844
+        $scripts->read($opts);
1845
+        $this->assertFalse($scripts->aggregating());
1846
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1847 1847
 
1848 1848
         // Not aggregating: option=false, dontaggregate=false.
1849 1849
         $opts['aggregate'] = false;
1850
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_false' );
1851
-        $scripts = new autoptimizeScripts( '' );
1852
-        $scripts->read( $opts );
1853
-        $this->assertFalse( $scripts->aggregating() );
1854
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1850
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_false');
1851
+        $scripts = new autoptimizeScripts('');
1852
+        $scripts->read($opts);
1853
+        $this->assertFalse($scripts->aggregating());
1854
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1855 1855
 
1856 1856
         // Not aggregating: option=false, dontaggregate=true.
1857 1857
         $opts['aggregate'] = false;
1858
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_true' );
1859
-        $scripts = new autoptimizeScripts( '' );
1860
-        $scripts->read( $opts );
1861
-        $this->assertFalse( $scripts->aggregating() );
1862
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1858
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_true');
1859
+        $scripts = new autoptimizeScripts('');
1860
+        $scripts->read($opts);
1861
+        $this->assertFalse($scripts->aggregating());
1862
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1863 1863
     }
1864 1864
 
1865 1865
     public function test_css_aggregation_decision_and_dontaggregate_filter()
@@ -1867,78 +1867,78 @@  discard block
 block discarded – undo
1867 1867
         $opts = $this->getAoStylesDefaultOptions();
1868 1868
 
1869 1869
         // Aggregating: true by default.
1870
-        $styles = new autoptimizeStyles( '' );
1871
-        $this->assertTrue( $styles->aggregating() );
1870
+        $styles = new autoptimizeStyles('');
1871
+        $this->assertTrue($styles->aggregating());
1872 1872
 
1873 1873
         // Aggregating: option=true (dontaggregate=false by default).
1874 1874
         $opts['aggregate'] = true;
1875
-        $styles            = new autoptimizeStyles( '' );
1876
-        $styles->read( $opts );
1877
-        $this->assertTrue( $styles->aggregating() );
1875
+        $styles            = new autoptimizeStyles('');
1876
+        $styles->read($opts);
1877
+        $this->assertTrue($styles->aggregating());
1878 1878
 
1879 1879
         // Aggregating: option=true, dontaggregate=false explicit.
1880 1880
         $opts['aggregate'] = true;
1881
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_false' );
1882
-        $styles = new autoptimizeStyles( '' );
1883
-        $styles->read( $opts );
1884
-        $this->assertTrue( $styles->aggregating() );
1885
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1881
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_false');
1882
+        $styles = new autoptimizeStyles('');
1883
+        $styles->read($opts);
1884
+        $this->assertTrue($styles->aggregating());
1885
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1886 1886
 
1887 1887
         // Not aggregating: option=true, dontaggregate=true.
1888 1888
         $opts['aggregate'] = true;
1889
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_true' );
1890
-        $styles = new autoptimizeStyles( '' );
1891
-        $styles->read( $opts );
1892
-        $this->assertFalse( $styles->aggregating() );
1893
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1889
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_true');
1890
+        $styles = new autoptimizeStyles('');
1891
+        $styles->read($opts);
1892
+        $this->assertFalse($styles->aggregating());
1893
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1894 1894
 
1895 1895
         // Not aggregating: option=false, dontaggregate=false.
1896 1896
         $opts['aggregate'] = false;
1897
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_false' );
1898
-        $styles = new autoptimizeStyles( '' );
1899
-        $styles->read( $opts );
1900
-        $this->assertFalse( $styles->aggregating() );
1901
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1897
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_false');
1898
+        $styles = new autoptimizeStyles('');
1899
+        $styles->read($opts);
1900
+        $this->assertFalse($styles->aggregating());
1901
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1902 1902
 
1903 1903
         // Not aggregating: option=false, dontaggregate=true.
1904 1904
         $opts['aggregate'] = false;
1905
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_true' );
1906
-        $styles = new autoptimizeStyles( '' );
1907
-        $styles->read( $opts );
1908
-        $this->assertFalse( $styles->aggregating() );
1909
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1905
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_true');
1906
+        $styles = new autoptimizeStyles('');
1907
+        $styles->read($opts);
1908
+        $this->assertFalse($styles->aggregating());
1909
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1910 1910
     }
1911 1911
 
1912 1912
     public function test_css_minify_single_with_cdning()
1913 1913
     {
1914
-        $pathname = dirname( __FILE__ ) . '/fixtures/minify-single.css';
1914
+        $pathname = dirname(__FILE__).'/fixtures/minify-single.css';
1915 1915
         $opts     = $this->getAoStylesDefaultOptions();
1916
-        $styles   = new autoptimizeStyles( '' );
1917
-        $styles->read( $opts );
1916
+        $styles   = new autoptimizeStyles('');
1917
+        $styles->read($opts);
1918 1918
 
1919
-        $url = $styles->minify_single( $pathname, $cache_miss = true );
1919
+        $url = $styles->minify_single($pathname, $cache_miss = true);
1920 1920
 
1921 1921
         // Minified url filename + its pointed to cdn.
1922
-        $this->assertContains( AUTOPTIMIZE_CACHE_CHILD_DIR, $url );
1923
-        $this->assertContains( '/autoptimize_single_', $url );
1924
-        $this->assertContains( $styles->cdn_url, $url );
1922
+        $this->assertContains(AUTOPTIMIZE_CACHE_CHILD_DIR, $url);
1923
+        $this->assertContains('/autoptimize_single_', $url);
1924
+        $this->assertContains($styles->cdn_url, $url);
1925 1925
 
1926 1926
         // Actual minified css contents are minified and cdn-ed.
1927
-        $path     = $styles->getpath( $url );
1928
-        $contents = file_get_contents( $path );
1929
-        $this->assertContains( $styles->cdn_url, $contents );
1930
-        $this->assertContains( '.bg{background:url(' . $styles->cdn_url, $contents );
1927
+        $path     = $styles->getpath($url);
1928
+        $contents = file_get_contents($path);
1929
+        $this->assertContains($styles->cdn_url, $contents);
1930
+        $this->assertContains('.bg{background:url('.$styles->cdn_url, $contents);
1931 1931
     }
1932 1932
 
1933 1933
     public function test_ao_partners_instantiation_without_explicit_include()
1934 1934
     {
1935 1935
         $partners = new autoptimizePartners();
1936
-        $this->assertTrue( $partners instanceof autoptimizePartners );
1936
+        $this->assertTrue($partners instanceof autoptimizePartners);
1937 1937
     }
1938 1938
 
1939 1939
     public function test_html_minify_keep_html_comments_inside_script_blocks()
1940 1940
     {
1941
-        $markup   = <<<MARKUP
1941
+        $markup = <<<MARKUP
1942 1942
 <script>
1943 1943
 <!-- End Support AJAX add to cart -->
1944 1944
 var a = "b";
@@ -1968,23 +1968,23 @@  discard block
 block discarded – undo
1968 1968
             ],
1969 1969
         ];
1970 1970
 
1971
-        $instance = new autoptimizeHTML( $markup );
1972
-        $instance->read( $options['autoptimizeHTML'] );
1971
+        $instance = new autoptimizeHTML($markup);
1972
+        $instance->read($options['autoptimizeHTML']);
1973 1973
         $instance->minify();
1974 1974
         $actual = $instance->getcontent();
1975
-        $this->assertEquals( $expected, $actual );
1975
+        $this->assertEquals($expected, $actual);
1976 1976
 
1977
-        $instance = new autoptimizeHTML( $markup2 );
1978
-        $instance->read( $options['autoptimizeHTML'] );
1977
+        $instance = new autoptimizeHTML($markup2);
1978
+        $instance->read($options['autoptimizeHTML']);
1979 1979
         $instance->minify();
1980 1980
         $actual2 = $instance->getcontent();
1981
-        $this->assertEquals( $expected2, $actual2 );
1981
+        $this->assertEquals($expected2, $actual2);
1982 1982
     }
1983 1983
 
1984 1984
     public function test_html_minify_remove_html_comments_inside_script_blocks()
1985 1985
     {
1986 1986
         // Default case, html comments removed (keepcomments = false).
1987
-        $markup1   = <<<MARKUP
1987
+        $markup1 = <<<MARKUP
1988 1988
 <script>
1989 1989
 var a = "b";
1990 1990
 <!-- End Support AJAX add to cart -->
@@ -1995,7 +1995,7 @@  discard block
 block discarded – undo
1995 1995
 <!-- End Support AJAX add to cart</script>
1996 1996
 MARKUP;
1997 1997
 
1998
-        $markup2   = <<<MARKUP
1998
+        $markup2 = <<<MARKUP
1999 1999
 <script>
2000 2000
 <!-- End Support AJAX add to cart -->
2001 2001
 var a = "b";
@@ -2012,17 +2012,17 @@  discard block
 block discarded – undo
2012 2012
             ],
2013 2013
         ];
2014 2014
 
2015
-        $instance = new autoptimizeHTML( $markup1 );
2016
-        $instance->read( $options['autoptimizeHTML'] );
2015
+        $instance = new autoptimizeHTML($markup1);
2016
+        $instance->read($options['autoptimizeHTML']);
2017 2017
         $instance->minify();
2018 2018
         $actual = $instance->getcontent();
2019
-        $this->assertEquals( $expected1, $actual );
2019
+        $this->assertEquals($expected1, $actual);
2020 2020
 
2021
-        $instance = new autoptimizeHTML( $markup2 );
2022
-        $instance->read( $options['autoptimizeHTML'] );
2021
+        $instance = new autoptimizeHTML($markup2);
2022
+        $instance->read($options['autoptimizeHTML']);
2023 2023
         $instance->minify();
2024 2024
         $actual2 = $instance->getcontent();
2025
-        $this->assertEquals( $expected2, $actual2 );
2025
+        $this->assertEquals($expected2, $actual2);
2026 2026
     }
2027 2027
 
2028 2028
     public function test_html_minify_html_comments_inside_script_blocks_old_school_pattern()
@@ -2046,11 +2046,11 @@  discard block
 block discarded – undo
2046 2046
             ],
2047 2047
         ];
2048 2048
 
2049
-        $instance = new autoptimizeHTML( $markup );
2050
-        $instance->read( $options['autoptimizeHTML'] );
2049
+        $instance = new autoptimizeHTML($markup);
2050
+        $instance->read($options['autoptimizeHTML']);
2051 2051
         $instance->minify();
2052 2052
         $actual = $instance->getcontent();
2053
-        $this->assertEquals( $expected, $actual );
2053
+        $this->assertEquals($expected, $actual);
2054 2054
     }
2055 2055
 
2056 2056
     public function test_html_minify_html_comments_inside_script_blocks_old_school_pattern_untouched()
@@ -2075,49 +2075,49 @@  discard block
 block discarded – undo
2075 2075
             ],
2076 2076
         ];
2077 2077
 
2078
-        $instance = new autoptimizeHTML( $markup );
2079
-        $instance->read( $options['autoptimizeHTML'] );
2078
+        $instance = new autoptimizeHTML($markup);
2079
+        $instance->read($options['autoptimizeHTML']);
2080 2080
         $instance->minify();
2081 2081
         $actual = $instance->getcontent();
2082
-        $this->assertEquals( $expected, $actual );
2082
+        $this->assertEquals($expected, $actual);
2083 2083
     }
2084 2084
 
2085 2085
     public function test_utils_mbstring_availabilty_overriding()
2086 2086
     {
2087 2087
         $orig     = autoptimizeUtils::mbstring_available();
2088
-        $opposite = ! $orig;
2088
+        $opposite = !$orig;
2089 2089
 
2090
-        $this->assertSame( $orig, autoptimizeUtils::mbstring_available() );
2090
+        $this->assertSame($orig, autoptimizeUtils::mbstring_available());
2091 2091
         // Override works...
2092
-        $this->assertSame( $opposite, autoptimizeUtils::mbstring_available( $opposite ) );
2092
+        $this->assertSame($opposite, autoptimizeUtils::mbstring_available($opposite));
2093 2093
         // And override remains cached as the last version.
2094
-        $this->assertSame( $opposite, autoptimizeUtils::mbstring_available() );
2094
+        $this->assertSame($opposite, autoptimizeUtils::mbstring_available());
2095 2095
     }
2096 2096
 
2097 2097
     public function test_utils_mbstring_basics()
2098 2098
     {
2099 2099
         // Turn on mbstring usage.
2100
-        autoptimizeUtils::mbstring_available( true );
2100
+        autoptimizeUtils::mbstring_available(true);
2101 2101
 
2102
-        $this->assertSame( 2, autoptimizeUtils::strlen( "\x00\xFF", 'ASCII' ) );
2103
-        $this->assertSame( 2, autoptimizeUtils::strlen( "\x00\xFF", 'CP850' ) );
2104
-        $this->assertSame( 3, autoptimizeUtils::strlen( '한국어' ) );
2102
+        $this->assertSame(2, autoptimizeUtils::strlen("\x00\xFF", 'ASCII'));
2103
+        $this->assertSame(2, autoptimizeUtils::strlen("\x00\xFF", 'CP850'));
2104
+        $this->assertSame(3, autoptimizeUtils::strlen('한국어'));
2105 2105
 
2106
-        $this->assertFalse( @autoptimizeUtils::strpos( 'abc', '' ) );
2107
-        $this->assertFalse( @autoptimizeUtils::strpos( 'abc', 'a', -1 ) );
2108
-        $this->assertFalse( autoptimizeUtils::strpos( 'abc', 'd' ) );
2109
-        $this->assertFalse( autoptimizeUtils::strpos( 'abc', 'a', 3 ) );
2110
-        $this->assertSame( 1, autoptimizeUtils::strpos( '한국어', '국' ) );
2106
+        $this->assertFalse(@autoptimizeUtils::strpos('abc', ''));
2107
+        $this->assertFalse(@autoptimizeUtils::strpos('abc', 'a', -1));
2108
+        $this->assertFalse(autoptimizeUtils::strpos('abc', 'd'));
2109
+        $this->assertFalse(autoptimizeUtils::strpos('abc', 'a', 3));
2110
+        $this->assertSame(1, autoptimizeUtils::strpos('한국어', '국'));
2111 2111
     }
2112 2112
 
2113 2113
     /**
2114 2114
      * @dataProvider provider_utils_substr_replace
2115 2115
      */
2116
-    function test_utils_substr_replace_basics_mbstring( $s, $repl, $start, $len, $expected )
2116
+    function test_utils_substr_replace_basics_mbstring($s, $repl, $start, $len, $expected)
2117 2117
     {
2118 2118
         // Force mbstring code path...
2119
-        autoptimizeUtils::mbstring_available( true );
2120
-        $this->assertEquals( $expected, autoptimizeUtils::substr_replace( $s, $repl, $start, $len ) );
2119
+        autoptimizeUtils::mbstring_available(true);
2120
+        $this->assertEquals($expected, autoptimizeUtils::substr_replace($s, $repl, $start, $len));
2121 2121
     }
2122 2122
 
2123 2123
     public function provider_utils_substr_replace()
@@ -2159,48 +2159,48 @@  discard block
 block discarded – undo
2159 2159
 
2160 2160
     function test_mb_substr_replace_with_ascii_input_string()
2161 2161
     {
2162
-        autoptimizeUtils::mbstring_available( false );
2162
+        autoptimizeUtils::mbstring_available(false);
2163 2163
 
2164 2164
         $str = 'Ascii';
2165 2165
 
2166
-        $this->assertSame( 'Añ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) );
2167
-        $this->assertSame( 'ñcii', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2168
-        $this->assertSame( 'Asñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2169
-        $this->assertSame( 'Asz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) );
2170
-        $this->assertSame( 'Añii', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2166
+        $this->assertSame('Añ', autoptimizeUtils::substr_replace($str, 'ñ', 1));
2167
+        $this->assertSame('ñcii', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2168
+        $this->assertSame('Asñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2169
+        $this->assertSame('Asz', autoptimizeUtils::substr_replace($str, 'z', 2, 10));
2170
+        $this->assertSame('Añii', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2171 2171
     }
2172 2172
 
2173 2173
     function test_mb_substr_replace_with_utf8_input_string()
2174 2174
     {
2175
-        autoptimizeUtils::mbstring_available( true );
2175
+        autoptimizeUtils::mbstring_available(true);
2176 2176
 
2177 2177
         $str = 'âønæë';
2178 2178
 
2179
-        $this->assertSame( 'âñ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) ); // No length.
2180
-        $this->assertSame( 'ñnæë', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2181
-        $this->assertSame( 'âøñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2182
-        $this->assertSame( 'âøz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) ); // Length larger than possible...
2183
-        $this->assertSame( 'âñæë', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2179
+        $this->assertSame('âñ', autoptimizeUtils::substr_replace($str, 'ñ', 1)); // No length.
2180
+        $this->assertSame('ñnæë', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2181
+        $this->assertSame('âøñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2182
+        $this->assertSame('âøz', autoptimizeUtils::substr_replace($str, 'z', 2, 10)); // Length larger than possible...
2183
+        $this->assertSame('âñæë', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2184 2184
     }
2185 2185
 
2186 2186
     function test_default_substr_replace_with_ascii_input_string()
2187 2187
     {
2188 2188
         // Disable mbstring which should fall ack to substr_replace...
2189
-        autoptimizeUtils::mbstring_available( false );
2189
+        autoptimizeUtils::mbstring_available(false);
2190 2190
 
2191 2191
         $str = 'Ascii';
2192 2192
 
2193
-        $this->assertSame( 'Añ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) );
2194
-        $this->assertSame( 'ñcii', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2195
-        $this->assertSame( 'Asñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2196
-        $this->assertSame( 'Asz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) );
2197
-        $this->assertSame( 'Añii', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2193
+        $this->assertSame('Añ', autoptimizeUtils::substr_replace($str, 'ñ', 1));
2194
+        $this->assertSame('ñcii', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2195
+        $this->assertSame('Asñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2196
+        $this->assertSame('Asz', autoptimizeUtils::substr_replace($str, 'z', 2, 10));
2197
+        $this->assertSame('Añii', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2198 2198
     }
2199 2199
 
2200 2200
     function test_default_substr_replace_with_utf8_input_string()
2201 2201
     {
2202 2202
         // Disabling mbstring, falling back to substr_replace...
2203
-        autoptimizeUtils::mbstring_available( false );
2203
+        autoptimizeUtils::mbstring_available(false);
2204 2204
 
2205 2205
         // This is really impossible to make work properly, since
2206 2206
         // any start/len parameters we give are working with bytes instead
@@ -2209,55 +2209,55 @@  discard block
 block discarded – undo
2209 2209
 
2210 2210
         // $this->assertSame( '�ñ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) ); // No length.
2211 2211
         // $this->assertSame( 'ñ�næë', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2212
-        $this->assertSame( 'ñønæë', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2213
-        $this->assertSame( 'âñxæë', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2214
-        $this->assertSame( 'âz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) ); // Length larger than possible...
2212
+        $this->assertSame('ñønæë', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2213
+        $this->assertSame('âñxæë', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2214
+        $this->assertSame('âz', autoptimizeUtils::substr_replace($str, 'z', 2, 10)); // Length larger than possible...
2215 2215
     }
2216 2216
 
2217 2217
     public function test_cache_fast_delete()
2218 2218
     {
2219
-        add_filter( 'autoptimize_filter_cache_clear_advanced', '__return_true' );
2219
+        add_filter('autoptimize_filter_cache_clear_advanced', '__return_true');
2220 2220
 
2221 2221
         autoptimizeCache::clearall_actionless();
2222 2222
 
2223
-        remove_all_filters( 'autoptimize_filter_cache_clear_advanced' );
2223
+        remove_all_filters('autoptimize_filter_cache_clear_advanced');
2224 2224
     }
2225 2225
 
2226 2226
     public function test_delete_advanced_cache_clear_artifacts()
2227 2227
     {
2228
-        $this->assertTrue( autoptimizeCache::delete_advanced_cache_clear_artifacts() );
2228
+        $this->assertTrue(autoptimizeCache::delete_advanced_cache_clear_artifacts());
2229 2229
     }
2230 2230
 
2231 2231
     public function provider_canonicalization()
2232 2232
     {
2233 2233
         return array(
2234
-            array( '../common', 'common' ),
2235
-            array( '../what-does-this-mean/really?/', 'what-does-this-mean/really?/' ),
2236
-            array( '../../what/where/how', 'what/where/how' ),
2237
-            array( '/../more.dots.please/', '/more.dots.please/' ),
2238
-            array( '/../../what/where/how', '/what/where/how' ),
2239
-            array( '/a/b/c/../../../d/e/file.txt', '/d/e/file.txt' ),
2240
-            array( 'a/b/../c', 'a/c' ),
2241
-            array( './../../etc/passwd', './etc/passwd' ),
2242
-            array( '/var/.////./user/./././..//.//../////../././.././test/////', '/test/' ),
2243
-            array( '/var/user/./././.././../.././././test/', '/test/' ),
2244
-            array( '/hello/0//how/../are/../you', '/hello/0/you' ),
2245
-            array( '', '' ),
2246
-            array( '.', '.' ),
2247
-            array( '..', '..' ),
2248
-            array( './..', './..' ),
2249
-            array( '../.', '.' ),
2234
+            array('../common', 'common'),
2235
+            array('../what-does-this-mean/really?/', 'what-does-this-mean/really?/'),
2236
+            array('../../what/where/how', 'what/where/how'),
2237
+            array('/../more.dots.please/', '/more.dots.please/'),
2238
+            array('/../../what/where/how', '/what/where/how'),
2239
+            array('/a/b/c/../../../d/e/file.txt', '/d/e/file.txt'),
2240
+            array('a/b/../c', 'a/c'),
2241
+            array('./../../etc/passwd', './etc/passwd'),
2242
+            array('/var/.////./user/./././..//.//../////../././.././test/////', '/test/'),
2243
+            array('/var/user/./././.././../.././././test/', '/test/'),
2244
+            array('/hello/0//how/../are/../you', '/hello/0/you'),
2245
+            array('', ''),
2246
+            array('.', '.'),
2247
+            array('..', '..'),
2248
+            array('./..', './..'),
2249
+            array('../.', '.'),
2250 2250
             // This might be debatable...
2251
-            array( '../..', '..' ),
2251
+            array('../..', '..'),
2252 2252
         );
2253 2253
     }
2254 2254
 
2255 2255
     /**
2256 2256
      * @dataProvider provider_canonicalization
2257 2257
      */
2258
-    public function test_path_canonicalize( $path, $canonicalized )
2258
+    public function test_path_canonicalize($path, $canonicalized)
2259 2259
     {
2260
-        $this->assertSame( $canonicalized, autoptimizeUtils::path_canonicalize( $path ) );
2260
+        $this->assertSame($canonicalized, autoptimizeUtils::path_canonicalize($path));
2261 2261
     }
2262 2262
 
2263 2263
     /**
@@ -2271,16 +2271,16 @@  discard block
 block discarded – undo
2271 2271
     public function test_autoptimize_filter_cdn_magic_path_check()
2272 2272
     {
2273 2273
         $cdnurl = 'http://cdn.example.org';
2274
-        add_filter( 'autoptimize_filter_cdn_magic_path_check', '__return_false', 10, 2 );
2274
+        add_filter('autoptimize_filter_cdn_magic_path_check', '__return_false', 10, 2);
2275 2275
 
2276 2276
         // Even when site is in a subfolder, the resulting cdn-rewritten url
2277 2277
         // should not magically include it, due to the above filter.
2278 2278
         // The second parameter is here to force a cache miss and re-run
2279 2279
         // the filter since we're using the same cdn url all over the place,
2280 2280
         // but want to prove different things with it.
2281
-        $this->assertSame( $cdnurl, autoptimizeUtils::tweak_cdn_url_if_needed( $cdnurl, true ) );
2281
+        $this->assertSame($cdnurl, autoptimizeUtils::tweak_cdn_url_if_needed($cdnurl, true));
2282 2282
 
2283
-        remove_all_filters( 'autoptimize_filter_cdn_magic_path_check' );
2283
+        remove_all_filters('autoptimize_filter_cdn_magic_path_check');
2284 2284
     }
2285 2285
 
2286 2286
     /**
@@ -2302,8 +2302,8 @@  discard block
 block discarded – undo
2302 2302
 <img src='$imgopthost/client/q_glossy,ret_img,w_400,h_200/$siteurl/wp-content/image.jpg' width='400' height='200' srcset="$imgopthost/client/q_glossy,ret_img,w_300/$siteurl/wp-content/image-300X150.jpg 300w, $imgopthost/client/q_glossy,ret_img,w_600/$siteurl/wp-content/image-600X300.jpg 600w" sizes="(max-width: 300px) 100vw, 300px" />
2303 2303
 MARKUP;
2304 2304
 
2305
-        $actual = autoptimizeImages::instance()->filter_optimize_images( $markup );
2306
-        $this->assertEquals( $expected, $actual );
2305
+        $actual = autoptimizeImages::instance()->filter_optimize_images($markup);
2306
+        $this->assertEquals($expected, $actual);
2307 2307
     }
2308 2308
 
2309 2309
     /**
@@ -2317,9 +2317,9 @@  discard block
 block discarded – undo
2317 2317
 <img src='/wp-content/plugins/imageplugin/image.php?id=16' width='400' height='200'>
2318 2318
 MARKUP;
2319 2319
 
2320
-        $actual = autoptimizeImages::instance()->filter_optimize_images( $markup );
2320
+        $actual = autoptimizeImages::instance()->filter_optimize_images($markup);
2321 2321
         // Expecting $markup since replacement should not happen.
2322
-        $this->assertEquals( $markup, $actual );
2322
+        $this->assertEquals($markup, $actual);
2323 2323
     }
2324 2324
 
2325 2325
     /**
@@ -2341,8 +2341,8 @@  discard block
 block discarded – undo
2341 2341
 <img src="data:image/gif;base64,R0lGODdhAQABAPAAAP///wAAACwAAAAAAQABAEACAkQBADs=" data-lazy-src='$imgopthost/client/q_glossy,ret_img,w_400,h_200/$siteurl/wp-content/image.jpg' width='400' height='200' data-lazy-srcset="$imgopthost/client/q_glossy,ret_img,w_300/$siteurl/wp-content/image-300X150.jpg 300w, $imgopthost/client/q_glossy,ret_img,w_600/$siteurl/wp-content/image-600X300.jpg 600w" sizes="(max-width: 300px) 100vw, 300px" />
2342 2342
 MARKUP;
2343 2343
 
2344
-        $actual = autoptimizeImages::instance()->filter_optimize_images( $markup );
2345
-        $this->assertEquals( $expected, $actual );
2344
+        $actual = autoptimizeImages::instance()->filter_optimize_images($markup);
2345
+        $this->assertEquals($expected, $actual);
2346 2346
     }
2347 2347
 
2348 2348
     /**
@@ -2362,8 +2362,8 @@  discard block
 block discarded – undo
2362 2362
 <img src='$imgopthost/client/q_glossy,ret_img,w_400,h_200/$siteurl/wp-content/image.jpg' width='400' height='200' srcset="$imgopthost/client/q_glossy,ret_img,w_300/$siteurl/wp-content/image-300X150.jpg 300w, $imgopthost/client/q_glossy,ret_img,w_600/$siteurl/wp-content/image-600X300.jpg 600w" sizes="(max-width: 300px) 100vw, 300px" />
2363 2363
 MARKUP;
2364 2364
 
2365
-        $actual = autoptimizeImages::instance()->filter_optimize_images( $markup );
2366
-        $this->assertEquals( $expected, $actual );
2365
+        $actual = autoptimizeImages::instance()->filter_optimize_images($markup);
2366
+        $this->assertEquals($expected, $actual);
2367 2367
     }
2368 2368
 
2369 2369
     /**
@@ -2384,8 +2384,8 @@  discard block
 block discarded – undo
2384 2384
 <img src='$imgopthost/client/q_glossy,ret_img,w_400,h_200/$siteurl/wp-content/image.jpg' width='400' height='200' srcset="$imgopthost/client/q_glossy,ret_img,w_300/$siteurl/wp-content/image-300X150.jpg 300w, $imgopthost/client/q_glossy,ret_img,w_600/$siteurl/wp-content/image-600X300.jpg 600w" sizes="(max-width: 300px) 100vw, 300px" />
2385 2385
 MARKUP;
2386 2386
 
2387
-        $actual = autoptimizeImages::instance()->filter_optimize_images( $markup );
2388
-        $this->assertEquals( $expected, $actual );
2387
+        $actual = autoptimizeImages::instance()->filter_optimize_images($markup);
2388
+        $this->assertEquals($expected, $actual);
2389 2389
     }
2390 2390
 
2391 2391
     /**
@@ -2395,7 +2395,7 @@  discard block
 block discarded – undo
2395 2395
     {
2396 2396
         $actual = autoptimizeUtils::str_is_valid_regex($str);
2397 2397
 
2398
-        $this->assertEquals( $expected, $actual );
2398
+        $this->assertEquals($expected, $actual);
2399 2399
     }
2400 2400
 
2401 2401
     public function provider_str_is_valid_regex()
@@ -2411,11 +2411,11 @@  discard block
 block discarded – undo
2411 2411
     /**
2412 2412
      * @dataProvider provider_protocol_relative_tests
2413 2413
      */
2414
-    public function test_is_protocol_relative( $str, $expected )
2414
+    public function test_is_protocol_relative($str, $expected)
2415 2415
     {
2416
-        $actual = autoptimizeUtils::is_protocol_relative( $str );
2416
+        $actual = autoptimizeUtils::is_protocol_relative($str);
2417 2417
 
2418
-        $this->assertEquals( $expected, $actual );
2418
+        $this->assertEquals($expected, $actual);
2419 2419
     }
2420 2420
 
2421 2421
     public function provider_protocol_relative_tests()
@@ -2443,38 +2443,38 @@  discard block
 block discarded – undo
2443 2443
 
2444 2444
         $expected = 'img.centered,.aligncenter{-webkit-box-shadow:none;-moz-box-shadow:none;box-shadow:none}';
2445 2445
 
2446
-        $instance = new autoptimizeStyles( $css );
2447
-        $minified = $instance->run_minifier_on( $css );
2446
+        $instance = new autoptimizeStyles($css);
2447
+        $minified = $instance->run_minifier_on($css);
2448 2448
 
2449
-        $this->assertEquals( $expected, $minified );
2449
+        $this->assertEquals($expected, $minified);
2450 2450
     }
2451 2451
 
2452 2452
     public function test_css_font_names_mangling()
2453 2453
     {
2454 2454
         // Properly quoted font names dont get mangled.
2455 2455
         $css      = 'h2{font-family:"Archivo Black"}';
2456
-        $instance = new autoptimizeStyles( $css );
2457
-        $actual   = $instance->run_minifier_on( $css );
2458
-        $this->assertEquals( $css, $actual );
2456
+        $instance = new autoptimizeStyles($css);
2457
+        $actual   = $instance->run_minifier_on($css);
2458
+        $this->assertEquals($css, $actual);
2459 2459
 
2460 2460
         // When not quoted, 'Black' used to become '#000', but not anymore... :)
2461 2461
         $css_unquoted = 'h2{font-family:Archivo Black;}';
2462 2462
         $expected     = 'h2{font-family:Archivo Black}';
2463
-        $instance     = new autoptimizeStyles( $css_unquoted );
2464
-        $actual       = $instance->run_minifier_on( $css_unquoted );
2465
-        $this->assertEquals( $expected, $actual );
2463
+        $instance     = new autoptimizeStyles($css_unquoted);
2464
+        $actual       = $instance->run_minifier_on($css_unquoted);
2465
+        $this->assertEquals($expected, $actual);
2466 2466
 
2467 2467
         $css_unquoted = 'h1{font:italic 1.2em Fira White,serif}';
2468
-        $instance     = new autoptimizeStyles( $css_unquoted );
2469
-        $actual       = $instance->run_minifier_on( $css_unquoted );
2470
-        $this->assertEquals( $css_unquoted, $actual );
2468
+        $instance     = new autoptimizeStyles($css_unquoted);
2469
+        $actual       = $instance->run_minifier_on($css_unquoted);
2470
+        $this->assertEquals($css_unquoted, $actual);
2471 2471
     }
2472 2472
 
2473 2473
     public function test_is_plugin_active_utils_wrapper()
2474 2474
     {
2475 2475
         // Our plugin is loaded via "muplugins_loaded" filter in tests/bootstrap.php
2476
-        $this->assertFalse( autoptimizeUtils::is_plugin_active( 'autoptimize/autoptimize.php' ) );
2477
-        $this->assertFalse( autoptimizeUtils::is_plugin_active( 'async-javascript/async-javascript.php' ) );
2476
+        $this->assertFalse(autoptimizeUtils::is_plugin_active('autoptimize/autoptimize.php'));
2477
+        $this->assertFalse(autoptimizeUtils::is_plugin_active('async-javascript/async-javascript.php'));
2478 2478
     }
2479 2479
     
2480 2480
     public function test_jsminphp_string_literal_minification()
@@ -2488,7 +2488,7 @@  discard block
 block discarded – undo
2488 2488
         $expected = '`line
2489 2489
 break`+`he  llo`;foo`hel( \'\');lo`;`he\nl\`lo`;(`he${one + two}`)';
2490 2490
 
2491
-        $actual = JSMin::minify( $js );
2492
-        $this->assertEquals( $expected, $actual );
2491
+        $actual = JSMin::minify($js);
2492
+        $this->assertEquals($expected, $actual);
2493 2493
     }
2494 2494
 }
2495 2495
\ No newline at end of file
Please login to merge, or discard this patch.
classes/external/php/yui-php-cssmin-bundled/Minifier.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -72,9 +72,9 @@  discard block
 block discarded – undo
72 72
     public function __construct($raisePhpLimits = true)
73 73
     {
74 74
         $this->raisePhpLimits = (bool) $raisePhpLimits;
75
-        $this->memoryLimit = 128 * 1048576; // 128MB in bytes
76
-        $this->pcreBacktrackLimit = 1000 * 1000;
77
-        $this->pcreRecursionLimit = 500 * 1000;
75
+        $this->memoryLimit = 128*1048576; // 128MB in bytes
76
+        $this->pcreBacktrackLimit = 1000*1000;
77
+        $this->pcreRecursionLimit = 500*1000;
78 78
         $this->hexToNamedColorsMap = Colors::getHexToNamedMap();
79 79
         $this->namedToHexColorsMap = Colors::getNamedToHexMap();
80 80
         $this->namedToHexColorsRegex = sprintf(
@@ -176,8 +176,8 @@  discard block
 block discarded – undo
176 176
      */
177 177
     private function setShortenZeroValuesRegexes()
178 178
     {
179
-        $zeroRegex = '0'. $this->unitsGroupRegex;
180
-        $numOrPosRegex = '('. $this->numRegex .'|top|left|bottom|right|center) ';
179
+        $zeroRegex = '0'.$this->unitsGroupRegex;
180
+        $numOrPosRegex = '('.$this->numRegex.'|top|left|bottom|right|center) ';
181 181
         $oneZeroSafeProperties = array(
182 182
             '(?:line-)?height',
183 183
             '(?:(?:min|max)-)?width',
@@ -195,14 +195,14 @@  discard block
 block discarded – undo
195 195
         );
196 196
 
197 197
         // First zero regex
198
-        $regex = '/(^|;)('. implode('|', $oneZeroSafeProperties) .'):%s/Si';
198
+        $regex = '/(^|;)('.implode('|', $oneZeroSafeProperties).'):%s/Si';
199 199
         $this->shortenOneZeroesRegex = sprintf($regex, $zeroRegex);
200 200
 
201 201
         // Multiple zeroes regexes
202 202
         $regex = '/(^|;)(margin|padding|border-(?:width|radius)|background-position):%s/Si';
203
-        $this->shortenTwoZeroesRegex = sprintf($regex, $numOrPosRegex . $zeroRegex);
204
-        $this->shortenThreeZeroesRegex = sprintf($regex, $numOrPosRegex . $numOrPosRegex . $zeroRegex);
205
-        $this->shortenFourZeroesRegex = sprintf($regex, $numOrPosRegex . $numOrPosRegex . $numOrPosRegex . $zeroRegex);
203
+        $this->shortenTwoZeroesRegex = sprintf($regex, $numOrPosRegex.$zeroRegex);
204
+        $this->shortenThreeZeroesRegex = sprintf($regex, $numOrPosRegex.$numOrPosRegex.$zeroRegex);
205
+        $this->shortenFourZeroesRegex = sprintf($regex, $numOrPosRegex.$numOrPosRegex.$numOrPosRegex.$zeroRegex);
206 206
     }
207 207
 
208 208
     /**
@@ -389,7 +389,7 @@  discard block
 block discarded – undo
389 389
             $dataStartIndex = $matchStartIndex + 4; // url( length
390 390
             $searchOffset = $matchStartIndex + strlen($m[0][0]);
391 391
             $terminator = $m[1][0]; // ', " or empty (not quoted)
392
-            $terminatorRegex = '/(?<!\\\\)'. (strlen($terminator) === 0 ? '' : $terminator.'\s*') .'(\))/S';
392
+            $terminatorRegex = '/(?<!\\\\)'.(strlen($terminator) === 0 ? '' : $terminator.'\s*').'(\))/S';
393 393
 
394 394
             $ret .= substr($css, $substrOffset, $matchStartIndex - $substrOffset);
395 395
 
@@ -404,7 +404,7 @@  discard block
 block discarded – undo
404 404
                     $token = preg_replace('/\s+/S', '', $token);
405 405
                 }
406 406
 
407
-                $ret .= 'url('. $this->registerPreservedToken(trim($token)) .')';
407
+                $ret .= 'url('.$this->registerPreservedToken(trim($token)).')';
408 408
             // No end terminator found, re-add the whole match. Should we throw/warn here?
409 409
             } else {
410 410
                 $ret .= substr($css, $matchStartIndex, $searchOffset - $matchStartIndex);
@@ -425,7 +425,7 @@  discard block
 block discarded – undo
425 425
      */
426 426
     private function processCommentsCallback($matches)
427 427
     {
428
-        return '/*'. $this->registerCommentToken($matches[1]) .'*/';
428
+        return '/*'.$this->registerCommentToken($matches[1]).'*/';
429 429
     }
430 430
 
431 431
     /**
@@ -435,7 +435,7 @@  discard block
 block discarded – undo
435 435
      */
436 436
     private function processOldIeSpecificMatrixDefinitionCallback($matches)
437 437
     {
438
-        return 'filter:progid:DXImageTransform.Microsoft.Matrix('. $this->registerPreservedToken($matches[1]) .')';
438
+        return 'filter:progid:DXImageTransform.Microsoft.Matrix('.$this->registerPreservedToken($matches[1]).')';
439 439
     }
440 440
 
441 441
     /**
@@ -458,7 +458,7 @@  discard block
 block discarded – undo
458 458
         // minify alpha opacity in filter strings
459 459
         $match = str_ireplace('progid:DXImageTransform.Microsoft.Alpha(Opacity=', 'alpha(opacity=', $match);
460 460
 
461
-        return $quote . $this->registerPreservedToken($match) . $quote;
461
+        return $quote.$this->registerPreservedToken($match).$quote;
462 462
     }
463 463
 
464 464
     /**
@@ -469,7 +469,7 @@  discard block
 block discarded – undo
469 469
      */
470 470
     private function processImportUnquotedUrlAtRulesCallback($matches)
471 471
     {
472
-        return '@import url('. $this->registerPreservedToken($matches[1]) .')'. $matches[2];
472
+        return '@import url('.$this->registerPreservedToken($matches[1]).')'.$matches[2];
473 473
     }
474 474
 
475 475
     /**
@@ -480,7 +480,7 @@  discard block
 block discarded – undo
480 480
     private function processComments($css)
481 481
     {
482 482
         foreach ($this->comments as $commentId => $comment) {
483
-            $commentIdString = '/*'. $commentId .'*/';
483
+            $commentIdString = '/*'.$commentId.'*/';
484 484
 
485 485
             // ! in the first position of the comment means preserve
486 486
             // so push to the preserved tokens keeping the !
@@ -531,7 +531,7 @@  discard block
 block discarded – undo
531 531
             $blockEndPos = strpos($css, '}', $blockStartPos);
532 532
             // When ending curly brace is missing, let's
533 533
             // behave like there was one at the end of the block...
534
-            if ( false === $blockEndPos ) {
534
+            if (false === $blockEndPos) {
535 535
                 $blockEndPos = strlen($css) - 1;
536 536
             }
537 537
             $nextBlockStartPos = strpos($css, '{', $blockStartPos + 1);
@@ -543,7 +543,7 @@  discard block
 block discarded – undo
543 543
             } else {
544 544
                 $ruleBody = substr($css, $blockStartPos + 1, $blockEndPos - $blockStartPos - 1);
545 545
                 $ruleBodyToken = $this->registerRuleBodyToken($this->processRuleBody($ruleBody));
546
-                $ret .= '{'. $ruleBodyToken .'}';
546
+                $ret .= '{'.$ruleBodyToken.'}';
547 547
                 $searchOffset = $blockEndPos + 1;
548 548
             }
549 549
 
@@ -643,8 +643,8 @@  discard block
 block discarded – undo
643 643
         $body = preg_replace('/([ :,(])\+(\.?\d+)/S', '$1$2', $body);
644 644
 
645 645
         // shorten ms to s
646
-        $body = preg_replace_callback('/([ :,(])(-?)(\d{3,})ms/Si', function ($matches) {
647
-            return $matches[1] . $matches[2] . ((int) $matches[3] / 1000) .'s';
646
+        $body = preg_replace_callback('/([ :,(])(-?)(\d{3,})ms/Si', function($matches) {
647
+            return $matches[1].$matches[2].((int) $matches[3]/1000).'s';
648 648
         }, $body);
649 649
 
650 650
         // Remove leading zeros from integer and float numbers.
@@ -756,13 +756,13 @@  discard block
 block discarded – undo
756 756
         $css = preg_replace('/::(before|after|first-(?:line|letter))(\{|,)/Si', ':$1$2', $css);
757 757
 
758 758
         // Retain space for special IE6 cases
759
-        $css = preg_replace_callback('/:first-(line|letter)(\{|,)/Si', function ($matches) {
760
-            return ':first-'. strtolower($matches[1]) .' '. $matches[2];
759
+        $css = preg_replace_callback('/:first-(line|letter)(\{|,)/Si', function($matches) {
760
+            return ':first-'.strtolower($matches[1]).' '.$matches[2];
761 761
         }, $css);
762 762
 
763 763
         // Find a fraction that may used in some @media queries such as: (min-aspect-ratio: 1/1)
764 764
         // Add token to add the "/" back in later
765
-        $css = preg_replace('/\(([a-z-]+):([0-9]+)\/([0-9]+)\)/Si', '($1:$2'. self::QUERY_FRACTION .'$3)', $css);
765
+        $css = preg_replace('/\(([a-z-]+):([0-9]+)\/([0-9]+)\)/Si', '($1:$2'.self::QUERY_FRACTION.'$3)', $css);
766 766
 
767 767
         // Remove empty rule blocks up to 2 levels deep.
768 768
         $css = preg_replace(array_fill(0, 2, '/(\{)[^{};\/\n]+\{\}/S'), '$1', $css);
@@ -810,7 +810,7 @@  discard block
 block discarded – undo
810 810
         }
811 811
 
812 812
         // @import handling
813
-        $css = preg_replace_callback($this->importRegex, function ($matches) use (&$imports) {
813
+        $css = preg_replace_callback($this->importRegex, function($matches) use (&$imports) {
814 814
             // Keep all @import at-rules found for later
815 815
             $imports .= $matches[0];
816 816
             // Delete all @import at-rules
@@ -818,7 +818,7 @@  discard block
 block discarded – undo
818 818
         }, $css);
819 819
 
820 820
         // @namespace handling
821
-        $css = preg_replace_callback($this->namespaceRegex, function ($matches) use (&$namespaces) {
821
+        $css = preg_replace_callback($this->namespaceRegex, function($matches) use (&$namespaces) {
822 822
             // Keep all @namespace at-rules found for later
823 823
             $namespaces .= $matches[0];
824 824
             // Delete all @namespace at-rules
@@ -829,7 +829,7 @@  discard block
 block discarded – undo
829 829
         // 1. @charset first
830 830
         // 2. @imports below @charset
831 831
         // 3. @namespaces below @imports
832
-        $css = $charset . $imports . $namespaces . $css;
832
+        $css = $charset.$imports.$namespaces.$css;
833 833
 
834 834
         return $css;
835 835
     }
@@ -883,10 +883,10 @@  discard block
 block discarded – undo
883 883
         // Restore space after rgb() or hsl() function in some cases such as:
884 884
         // background-image: linear-gradient(to bottom, rgb(210,180,140) 10%, rgb(255,0,0) 90%);
885 885
         if (!empty($terminator) && !preg_match('/[ ,);]/S', $terminator)) {
886
-            $terminator = ' '. $terminator;
886
+            $terminator = ' '.$terminator;
887 887
         }
888 888
 
889
-        return '#'. implode('', $hexColors) . $terminator;
889
+        return '#'.implode('', $hexColors).$terminator;
890 890
     }
891 891
 
892 892
     /**
@@ -900,16 +900,16 @@  discard block
 block discarded – undo
900 900
 
901 901
         // Shorten suitable 6 chars HEX colors
902 902
         if (strlen($hex) === 6 && preg_match('/^([0-9a-f])\1([0-9a-f])\2([0-9a-f])\3$/Si', $hex, $m)) {
903
-            $hex = $m[1] . $m[2] . $m[3];
903
+            $hex = $m[1].$m[2].$m[3];
904 904
         }
905 905
 
906 906
         // Lowercase
907
-        $hex = '#'. strtolower($hex);
907
+        $hex = '#'.strtolower($hex);
908 908
 
909 909
         // Replace Hex colors with shorter color names
910 910
         $color = array_key_exists($hex, $this->hexToNamedColorsMap) ? $this->hexToNamedColorsMap[$hex] : $hex;
911 911
 
912
-        return $color . $matches[2];
912
+        return $color.$matches[2];
913 913
     }
914 914
 
915 915
     /**
@@ -920,7 +920,7 @@  discard block
 block discarded – undo
920 920
      */
921 921
     private function shortenNamedColorsCallback($matches)
922 922
     {
923
-        return $matches[1] . $this->namedToHexColorsMap[strtolower($matches[2])] . $matches[3];
923
+        return $matches[1].$this->namedToHexColorsMap[strtolower($matches[2])].$matches[3];
924 924
     }
925 925
 
926 926
     /**
Please login to merge, or discard this patch.