Completed
Pull Request — master (#244)
by
unknown
02:21
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   +311 added lines, -311 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,88 +798,88 @@  discard block
 block discarded – undo
798 798
     public function getcontent()
799 799
     {
800 800
         // restore comments.
801
-        $this->content = $this->restore_comments( $this->content );
801
+        $this->content = $this->restore_comments($this->content);
802 802
 
803 803
         // restore IE hacks.
804
-        $this->content = $this->restore_iehacks( $this->content );
804
+        $this->content = $this->restore_iehacks($this->content);
805 805
 
806 806
         // restore (no)script.
807
-        $this->content = $this->restore_marked_content( 'SCRIPT', $this->content );
807
+        $this->content = $this->restore_marked_content('SCRIPT', $this->content);
808 808
 
809 809
         // Restore noptimize.
810
-        $this->content = $this->restore_noptimize( $this->content );
810
+        $this->content = $this->restore_noptimize($this->content);
811 811
 
812 812
         // Restore the full content.
813
-        if ( ! empty( $this->restofcontent ) ) {
813
+        if (!empty($this->restofcontent)) {
814 814
             $this->content .= $this->restofcontent;
815 815
             $this->restofcontent = '';
816 816
         }
817 817
 
818 818
         // Inject the new stylesheets.
819
-        $replaceTag = array( '<title', 'before' );
820
-        $replaceTag = apply_filters( 'autoptimize_filter_css_replacetag', $replaceTag, $this->content );
819
+        $replaceTag = array('<title', 'before');
820
+        $replaceTag = apply_filters('autoptimize_filter_css_replacetag', $replaceTag, $this->content);
821 821
 
822
-        if ( $this->inline ) {
823
-            foreach ( $this->csscode as $media => $code ) {
824
-                $this->inject_in_html( '<style type="text/css" media="' . $media . '">' . $code . '</style>', $replaceTag );
822
+        if ($this->inline) {
823
+            foreach ($this->csscode as $media => $code) {
824
+                $this->inject_in_html('<style type="text/css" media="'.$media.'">'.$code.'</style>', $replaceTag);
825 825
             }
826 826
         } else {
827
-            if ( $this->defer ) {
827
+            if ($this->defer) {
828 828
                 $preloadCssBlock = '';
829 829
                 $noScriptCssBlock = "<noscript id=\"aonoscrcss\">";
830 830
 
831 831
                 $defer_inline_code = $this->defer_inline;
832
-                if ( ! empty( $defer_inline_code ) ) {
833
-                    if ( apply_filters( 'autoptimize_filter_css_critcss_minify', true ) ) {
834
-                        $iCssHash = md5( $defer_inline_code );
835
-                        $iCssCache = new autoptimizeCache( $iCssHash, 'css' );
836
-                        if ( $iCssCache->check() ) {
832
+                if (!empty($defer_inline_code)) {
833
+                    if (apply_filters('autoptimize_filter_css_critcss_minify', true)) {
834
+                        $iCssHash = md5($defer_inline_code);
835
+                        $iCssCache = new autoptimizeCache($iCssHash, 'css');
836
+                        if ($iCssCache->check()) {
837 837
                             // we have the optimized inline CSS in cache.
838 838
                             $defer_inline_code = $iCssCache->retrieve();
839 839
                         } else {
840 840
                             $cssmin   = new autoptimizeCSSmin();
841
-                            $tmp_code = trim( $cssmin->run( $defer_inline_code ) );
841
+                            $tmp_code = trim($cssmin->run($defer_inline_code));
842 842
 
843
-                            if ( ! empty( $tmp_code ) ) {
843
+                            if (!empty($tmp_code)) {
844 844
                                 $defer_inline_code = $tmp_code;
845
-                                $iCssCache->cache( $defer_inline_code, 'text/css' );
846
-                                unset( $tmp_code );
845
+                                $iCssCache->cache($defer_inline_code, 'text/css');
846
+                                unset($tmp_code);
847 847
                             }
848 848
                         }
849 849
                     }
850
-                    $code_out = '<style type="text/css" id="aoatfcss" media="all">' . $defer_inline_code . '</style>';
851
-                    $this->inject_in_html( $code_out, $replaceTag );
850
+                    $code_out = '<style type="text/css" id="aoatfcss" media="all">'.$defer_inline_code.'</style>';
851
+                    $this->inject_in_html($code_out, $replaceTag);
852 852
                 }
853 853
             }
854 854
 
855
-            foreach ( $this->url as $media => $url ) {
856
-                $url = $this->url_replace_cdn( $url );
855
+            foreach ($this->url as $media => $url) {
856
+                $url = $this->url_replace_cdn($url);
857 857
 
858 858
                 // Add the stylesheet either deferred (import at bottom) or normal links in head.
859
-                if ( $this->defer ) {
859
+                if ($this->defer) {
860 860
                     $preloadOnLoad = autoptimizeConfig::get_ao_css_preload_onload();
861 861
 
862
-                    $preloadCssBlock .= '<link rel="preload" as="style" media="' . $media . '" href="' . $url . '" onload="' . $preloadOnLoad . '" />';
863
-                    $noScriptCssBlock .= '<link type="text/css" media="' . $media . '" href="' . $url . '" rel="stylesheet" />';
862
+                    $preloadCssBlock .= '<link rel="preload" as="style" media="'.$media.'" href="'.$url.'" onload="'.$preloadOnLoad.'" />';
863
+                    $noScriptCssBlock .= '<link type="text/css" media="'.$media.'" href="'.$url.'" rel="stylesheet" />';
864 864
                 } else {
865 865
                     // $this->inject_in_html('<link type="text/css" media="' . $media . '" href="' . $url . '" rel="stylesheet" />', $replaceTag);
866
-                    if ( strlen( $this->csscode[$media] ) > $this->cssinlinesize ) {
867
-                        $this->inject_in_html( '<link type="text/css" media="' . $media . '" href="' . $url . '" rel="stylesheet" />', $replaceTag );
868
-                    } elseif ( strlen( $this->csscode[$media] ) > 0 ) {
869
-                        $this->inject_in_html( '<style type="text/css" media="' . $media . '">' . $this->csscode[$media] . '</style>', $replaceTag );
866
+                    if (strlen($this->csscode[$media]) > $this->cssinlinesize) {
867
+                        $this->inject_in_html('<link type="text/css" media="'.$media.'" href="'.$url.'" rel="stylesheet" />', $replaceTag);
868
+                    } elseif (strlen($this->csscode[$media]) > 0) {
869
+                        $this->inject_in_html('<style type="text/css" media="'.$media.'">'.$this->csscode[$media].'</style>', $replaceTag);
870 870
                     }
871 871
                 }
872 872
             }
873 873
 
874
-            if ( $this->defer ) {
874
+            if ($this->defer) {
875 875
                 $preload_polyfill = autoptimizeConfig::get_ao_css_preload_polyfill();
876 876
                 $noScriptCssBlock .= '</noscript>';
877
-                $this->inject_in_html( $preloadCssBlock . $noScriptCssBlock, $replaceTag );
877
+                $this->inject_in_html($preloadCssBlock.$noScriptCssBlock, $replaceTag);
878 878
 
879 879
                 // Adds preload polyfill at end of body tag.
880 880
                 $this->inject_in_html(
881
-                    apply_filters( 'autoptimize_css_preload_polyfill', $preload_polyfill ),
882
-                    array( '</body>', 'before' )
881
+                    apply_filters('autoptimize_css_preload_polyfill', $preload_polyfill),
882
+                    array('</body>', 'before')
883 883
                 );
884 884
             }
885 885
         }
@@ -888,44 +888,44 @@  discard block
 block discarded – undo
888 888
         return $this->content;
889 889
     }
890 890
 
891
-    static function fixurls( $file, $code )
891
+    static function fixurls($file, $code)
892 892
     {
893 893
         // Switch all imports to the url() syntax.
894
-        $code = preg_replace( '#@import ("|\')(.+?)\.css.*?("|\')#', '@import url("${2}.css")', $code );
894
+        $code = preg_replace('#@import ("|\')(.+?)\.css.*?("|\')#', '@import url("${2}.css")', $code);
895 895
 
896
-        if ( preg_match_all( self::ASSETS_REGEX, $code, $matches ) ) {
897
-            $file = str_replace( WP_ROOT_DIR, '/', $file );
896
+        if (preg_match_all(self::ASSETS_REGEX, $code, $matches)) {
897
+            $file = str_replace(WP_ROOT_DIR, '/', $file);
898 898
             /**
899 899
              * rollback as per https://github.com/futtta/autoptimize/issues/94
900 900
              * $file = str_replace( AUTOPTIMIZE_WP_CONTENT_NAME, '', $file );
901 901
              */
902
-            $dir = dirname( $file ); // Like /themes/expound/css.
902
+            $dir = dirname($file); // Like /themes/expound/css.
903 903
 
904 904
             /**
905 905
              * $dir should not contain backslashes, since it's used to replace
906 906
              * urls, but it can contain them when running on Windows because
907 907
              * fixurls() is sometimes called with `ABSPATH . 'index.php'`
908 908
              */
909
-            $dir = str_replace( '\\', '/', $dir );
910
-            unset( $file ); // not used below at all.
909
+            $dir = str_replace('\\', '/', $dir);
910
+            unset($file); // not used below at all.
911 911
 
912 912
             $replace = array();
913
-            foreach ( $matches[1] as $k => $url ) {
913
+            foreach ($matches[1] as $k => $url) {
914 914
                 // Remove quotes.
915
-                $url    = trim( $url, " \t\n\r\0\x0B\"'" );
916
-                $noQurl = trim( $url, "\"'" );
917
-                if ( $url !== $noQurl ) {
915
+                $url    = trim($url, " \t\n\r\0\x0B\"'");
916
+                $noQurl = trim($url, "\"'");
917
+                if ($url !== $noQurl) {
918 918
                     $removedQuotes = true;
919 919
                 } else {
920 920
                     $removedQuotes = false;
921 921
                 }
922 922
 
923
-                if ( '' === $noQurl ) {
923
+                if ('' === $noQurl) {
924 924
                     continue;
925 925
                 }
926 926
 
927 927
                 $url = $noQurl;
928
-                if ( '/' === $url{0} || preg_match( '#^(https?://|ftp://|data:)#i', $url ) ) {
928
+                if ('/' === $url{0} || preg_match('#^(https?://|ftp://|data:)#i', $url)) {
929 929
                     // URL is protocol-relative, host-relative or something we don't touch.
930 930
                     continue;
931 931
                 } else {
@@ -934,49 +934,49 @@  discard block
 block discarded – undo
934 934
                      * rollback as per https://github.com/futtta/autoptimize/issues/94
935 935
                      * $newurl = preg_replace( '/https?:/', '', str_replace( ' ', '%20', AUTOPTIMIZE_WP_CONTENT_URL . str_replace( '//', '/', $dir . '/' . $url ) ) );
936 936
                      */
937
-                    $newurl = preg_replace( '/https?:/', '', str_replace( ' ', '%20', AUTOPTIMIZE_WP_ROOT_URL . str_replace( '//', '/', $dir . '/' . $url ) ) );
938
-                    $newurl = apply_filters( 'autoptimize_filter_css_fixurl_newurl', $newurl );
937
+                    $newurl = preg_replace('/https?:/', '', str_replace(' ', '%20', AUTOPTIMIZE_WP_ROOT_URL.str_replace('//', '/', $dir.'/'.$url)));
938
+                    $newurl = apply_filters('autoptimize_filter_css_fixurl_newurl', $newurl);
939 939
 
940 940
                     /**
941 941
                      * Hash the url + whatever was behind potentially for replacement
942 942
                      * We must do this, or different css classes referencing the same bg image (but
943 943
                      * different parts of it, say, in sprites and such) loose their stuff...
944 944
                      */
945
-                    $hash = md5( $url . $matches[2][$k] );
946
-                    $code = str_replace( $matches[0][$k], $hash, $code );
945
+                    $hash = md5($url.$matches[2][$k]);
946
+                    $code = str_replace($matches[0][$k], $hash, $code);
947 947
 
948
-                    if ( $removedQuotes ) {
949
-                        $replace[$hash] = "url('" . $newurl . "')" . $matches[2][$k];
948
+                    if ($removedQuotes) {
949
+                        $replace[$hash] = "url('".$newurl."')".$matches[2][$k];
950 950
                     } else {
951
-                        $replace[$hash] = 'url(' . $newurl . ')' . $matches[2][$k];
951
+                        $replace[$hash] = 'url('.$newurl.')'.$matches[2][$k];
952 952
                     }
953 953
                 }
954 954
             }
955 955
 
956
-            $code = self::replace_longest_matches_first( $code, $replace );
956
+            $code = self::replace_longest_matches_first($code, $replace);
957 957
         }
958 958
 
959 959
         return $code;
960 960
     }
961 961
 
962
-    private function ismovable( $tag )
962
+    private function ismovable($tag)
963 963
     {
964
-        if ( ! $this->aggregate ) {
964
+        if (!$this->aggregate) {
965 965
             return false;
966 966
         }
967 967
 
968
-        if ( ! empty( $this->whitelist ) ) {
969
-            foreach ( $this->whitelist as $match ) {
970
-                if ( false !== strpos( $tag, $match ) ) {
968
+        if (!empty($this->whitelist)) {
969
+            foreach ($this->whitelist as $match) {
970
+                if (false !== strpos($tag, $match)) {
971 971
                     return true;
972 972
                 }
973 973
             }
974 974
             // no match with whitelist.
975 975
             return false;
976 976
         } else {
977
-            if ( is_array( $this->dontmove ) && ! empty( $this->dontmove ) ) {
978
-                foreach ( $this->dontmove as $match ) {
979
-                    if ( false !== strpos( $tag, $match ) ) {
977
+            if (is_array($this->dontmove) && !empty($this->dontmove)) {
978
+                foreach ($this->dontmove as $match) {
979
+                    if (false !== strpos($tag, $match)) {
980 980
                         // Matched something.
981 981
                         return false;
982 982
                     }
@@ -988,22 +988,22 @@  discard block
 block discarded – undo
988 988
         }
989 989
     }
990 990
 
991
-    private function can_inject_late( $cssPath, $css )
991
+    private function can_inject_late($cssPath, $css)
992 992
     {
993
-        $consider_minified_array = apply_filters( 'autoptimize_filter_css_consider_minified', false, $cssPath );
994
-        if ( true !== $this->inject_min_late ) {
993
+        $consider_minified_array = apply_filters('autoptimize_filter_css_consider_minified', false, $cssPath);
994
+        if (true !== $this->inject_min_late) {
995 995
             // late-inject turned off.
996 996
             return false;
997
-        } elseif ( ( false === strpos( $cssPath, 'min.css' ) ) && ( str_replace( $consider_minified_array, '', $cssPath ) === $cssPath ) ) {
997
+        } elseif ((false === strpos($cssPath, 'min.css')) && (str_replace($consider_minified_array, '', $cssPath) === $cssPath)) {
998 998
             // file not minified based on filename & filter.
999 999
             return false;
1000
-        } elseif ( false !== strpos( $css, '@import' ) ) {
1000
+        } elseif (false !== strpos($css, '@import')) {
1001 1001
             // can't late-inject files with imports as those need to be aggregated.
1002 1002
             return false;
1003
-        } elseif ( ( false !== strpos( $css, '@font-face' ) ) && ( apply_filters( 'autoptimize_filter_css_fonts_cdn', false ) === true ) && ( ! empty( $this->cdn_url ) ) ) {
1003
+        } elseif ((false !== strpos($css, '@font-face')) && (apply_filters('autoptimize_filter_css_fonts_cdn', false) === true) && (!empty($this->cdn_url))) {
1004 1004
             // don't late-inject CSS with font-src's if fonts are set to be CDN'ed.
1005 1005
             return false;
1006
-        } elseif ( ( ( $this->datauris == true ) || ( ! empty( $this->cdn_url ) ) ) && preg_match( '#background[^;}]*url\(#Ui', $css ) ) {
1006
+        } elseif ((($this->datauris == true) || (!empty($this->cdn_url))) && preg_match('#background[^;}]*url\(#Ui', $css)) {
1007 1007
             // don't late-inject CSS with images if CDN is set OR if image inlining is on.
1008 1008
             return false;
1009 1009
         } else {
@@ -1021,32 +1021,32 @@  discard block
 block discarded – undo
1021 1021
      *
1022 1022
      * @return bool|string Url pointing to the minified css file or false.
1023 1023
      */
1024
-    public function minify_single( $filepath, $cache_miss = false )
1024
+    public function minify_single($filepath, $cache_miss = false)
1025 1025
     {
1026
-        $contents = $this->prepare_minify_single( $filepath );
1026
+        $contents = $this->prepare_minify_single($filepath);
1027 1027
 
1028
-        if ( empty( $contents ) ) {
1028
+        if (empty($contents)) {
1029 1029
             return false;
1030 1030
         }
1031 1031
 
1032 1032
         // Check cache.
1033
-        $hash  = 'single_' . md5( $contents );
1034
-        $cache = new autoptimizeCache( $hash, 'css' );
1033
+        $hash  = 'single_'.md5($contents);
1034
+        $cache = new autoptimizeCache($hash, 'css');
1035 1035
 
1036 1036
         // If not in cache already, minify...
1037
-        if ( ! $cache->check() || $cache_miss ) {
1037
+        if (!$cache->check() || $cache_miss) {
1038 1038
             // Fixurls...
1039
-            $contents = self::fixurls( $filepath, $contents );
1039
+            $contents = self::fixurls($filepath, $contents);
1040 1040
             // CDN-replace any referenced assets if needed...
1041
-            $contents = $this->replace_urls( $contents );
1041
+            $contents = $this->replace_urls($contents);
1042 1042
             // Now minify...
1043 1043
             $cssmin   = new autoptimizeCSSmin();
1044
-            $contents = trim( $cssmin->run( $contents ) );
1044
+            $contents = trim($cssmin->run($contents));
1045 1045
             // Store in cache.
1046
-            $cache->cache( $contents, 'text/css' );
1046
+            $cache->cache($contents, 'text/css');
1047 1047
         }
1048 1048
 
1049
-        $url = $this->build_minify_single_url( $cache );
1049
+        $url = $this->build_minify_single_url($cache);
1050 1050
 
1051 1051
         return $url;
1052 1052
     }
@@ -1066,18 +1066,18 @@  discard block
 block discarded – undo
1066 1066
         return $this->options;
1067 1067
     }
1068 1068
 
1069
-    public function replaceOptions( $options )
1069
+    public function replaceOptions($options)
1070 1070
     {
1071 1071
         $this->options = $options;
1072 1072
     }
1073 1073
 
1074
-    public function setOption( $name, $value )
1074
+    public function setOption($name, $value)
1075 1075
     {
1076 1076
         $this->options[$name] = $value;
1077 1077
         $this->$name = $value;
1078 1078
     }
1079 1079
 
1080
-    public function getOption( $name )
1080
+    public function getOption($name)
1081 1081
     {
1082 1082
         return $this->options[$name];
1083 1083
     }
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   +76 added lines, -76 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * General helpers.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
@@ -16,15 +16,15 @@  discard block
 block discarded – undo
16 16
      *
17 17
      * @return bool
18 18
      */
19
-    public static function mbstring_available( $override = null )
19
+    public static function mbstring_available($override = null)
20 20
     {
21 21
         static $available = null;
22 22
 
23
-        if ( null === $available ) {
24
-            $available = \extension_loaded( 'mbstring' );
23
+        if (null === $available) {
24
+            $available = \extension_loaded('mbstring');
25 25
         }
26 26
 
27
-        if ( null !== $override ) {
27
+        if (null !== $override) {
28 28
             $available = $override;
29 29
         }
30 30
 
@@ -42,12 +42,12 @@  discard block
 block discarded – undo
42 42
      *
43 43
      * @return int|false
44 44
      */
45
-    public static function strpos( $haystack, $needle, $offset = 0, $encoding = null )
45
+    public static function strpos($haystack, $needle, $offset = 0, $encoding = null)
46 46
     {
47
-        if ( self::mbstring_available() ) {
48
-            return ( null === $encoding ) ? \mb_strpos( $haystack, $needle, $offset ) : \mb_strlen( $haystack, $needle, $offset, $encoding );
47
+        if (self::mbstring_available()) {
48
+            return (null === $encoding) ? \mb_strpos($haystack, $needle, $offset) : \mb_strlen($haystack, $needle, $offset, $encoding);
49 49
         } else {
50
-            return \strpos( $haystack, $needle, $offset );
50
+            return \strpos($haystack, $needle, $offset);
51 51
         }
52 52
     }
53 53
 
@@ -62,12 +62,12 @@  discard block
 block discarded – undo
62 62
      * @return int Number of charcters or bytes in given $string
63 63
      *             (characters if/when supported, bytes otherwise).
64 64
      */
65
-    public static function strlen( $string, $encoding = null )
65
+    public static function strlen($string, $encoding = null)
66 66
     {
67
-        if ( self::mbstring_available() ) {
68
-            return ( null === $encoding ) ? \mb_strlen( $string ) : \mb_strlen( $string, $encoding );
67
+        if (self::mbstring_available()) {
68
+            return (null === $encoding) ? \mb_strlen($string) : \mb_strlen($string, $encoding);
69 69
         } else {
70
-            return \strlen( $string );
70
+            return \strlen($string);
71 71
         }
72 72
     }
73 73
 
@@ -85,44 +85,44 @@  discard block
 block discarded – undo
85 85
      *
86 86
      * @return string
87 87
      */
88
-    public static function substr_replace( $string, $replacement, $start, $length = null, $encoding = null )
88
+    public static function substr_replace($string, $replacement, $start, $length = null, $encoding = null)
89 89
     {
90
-        if ( self::mbstring_available() ) {
91
-            $strlen = self::strlen( $string, $encoding );
90
+        if (self::mbstring_available()) {
91
+            $strlen = self::strlen($string, $encoding);
92 92
 
93
-            if ( $start < 0 ) {
93
+            if ($start < 0) {
94 94
                 if ( -$start < $strlen ) {
95 95
                     $start = $strlen + $start;
96 96
                 } else {
97 97
                     $start = 0;
98 98
                 }
99
-            } elseif ( $start > $strlen ) {
99
+            } elseif ($start > $strlen) {
100 100
                 $start = $strlen;
101 101
             }
102 102
 
103
-            if ( null === $length || '' === $length ) {
103
+            if (null === $length || '' === $length) {
104 104
                 $start2 = $strlen;
105
-            } elseif ( $length < 0 ) {
105
+            } elseif ($length < 0) {
106 106
                 $start2 = $strlen + $length;
107
-                if ( $start2 < $start ) {
107
+                if ($start2 < $start) {
108 108
                     $start2 = $start;
109 109
                 }
110 110
             } else {
111 111
                 $start2 = $start + $length;
112 112
             }
113 113
 
114
-            if ( null === $encoding ) {
115
-                $leader  = $start ? \mb_substr( $string, 0, $start ) : '';
116
-                $trailer = ( $start2 < $strlen ) ? \mb_substr( $string, $start2, null ) : '';
114
+            if (null === $encoding) {
115
+                $leader  = $start ? \mb_substr($string, 0, $start) : '';
116
+                $trailer = ($start2 < $strlen) ? \mb_substr($string, $start2, null) : '';
117 117
             } else {
118
-                $leader  = $start ? \mb_substr( $string, 0, $start, $encoding ) : '';
119
-                $trailer = ( $start2 < $strlen ) ? \mb_substr( $string, $start2, null, $encoding ) : '';
118
+                $leader  = $start ? \mb_substr($string, 0, $start, $encoding) : '';
119
+                $trailer = ($start2 < $strlen) ? \mb_substr($string, $start2, null, $encoding) : '';
120 120
             }
121 121
 
122 122
             return "{$leader}{$replacement}{$trailer}";
123 123
         }
124 124
 
125
-        return ( null === $length ) ? \substr_replace( $string, $replacement, $start ) : \substr_replace( $string, $replacement, $start, $length );
125
+        return (null === $length) ? \substr_replace($string, $replacement, $start) : \substr_replace($string, $replacement, $start, $length);
126 126
     }
127 127
 
128 128
     /**
@@ -132,16 +132,16 @@  discard block
 block discarded – undo
132 132
      *
133 133
      * @return bool
134 134
      */
135
-    public static function siteurl_not_root( $override = null )
135
+    public static function siteurl_not_root($override = null)
136 136
     {
137 137
         static $subdir = null;
138 138
 
139
-        if ( null === $subdir ) {
139
+        if (null === $subdir) {
140 140
             $parts  = self::get_ao_wp_site_url_parts();
141
-            $subdir = ( isset( $parts['path'] ) && ( '/' !== $parts['path'] ) );
141
+            $subdir = (isset($parts['path']) && ('/' !== $parts['path']));
142 142
         }
143 143
 
144
-        if ( null !== $override ) {
144
+        if (null !== $override) {
145 145
             $subdir = $override;
146 146
         }
147 147
 
@@ -158,8 +158,8 @@  discard block
 block discarded – undo
158 158
     {
159 159
         static $parts = array();
160 160
 
161
-        if ( empty( $parts ) ) {
162
-            $parts = \parse_url( AUTOPTIMIZE_WP_SITE_URL );
161
+        if (empty($parts)) {
162
+            $parts = \parse_url(AUTOPTIMIZE_WP_SITE_URL);
163 163
         }
164 164
 
165 165
         return $parts;
@@ -174,33 +174,33 @@  discard block
 block discarded – undo
174 174
      *
175 175
      * @return string
176 176
      */
177
-    public static function tweak_cdn_url_if_needed( $cdn_url, $force_cache_miss = false )
177
+    public static function tweak_cdn_url_if_needed($cdn_url, $force_cache_miss = false)
178 178
     {
179 179
         static $results = array();
180 180
 
181
-        if ( ! isset( $results[ $cdn_url ] ) || $force_cache_miss ) {
181
+        if (!isset($results[$cdn_url]) || $force_cache_miss) {
182 182
 
183 183
             // In order to return unmodified input when there's no need to tweak.
184
-            $results[ $cdn_url ] = $cdn_url;
184
+            $results[$cdn_url] = $cdn_url;
185 185
 
186 186
             // Behind a default true filter for backcompat, and only for sites
187 187
             // in a subfolder/subdirectory, but still easily turned off if
188 188
             // not wanted/needed...
189
-            if ( autoptimizeUtils::siteurl_not_root() ) {
190
-                $check = apply_filters( 'autoptimize_filter_cdn_magic_path_check', true, $cdn_url );
191
-                if ( $check ) {
189
+            if (autoptimizeUtils::siteurl_not_root()) {
190
+                $check = apply_filters('autoptimize_filter_cdn_magic_path_check', true, $cdn_url);
191
+                if ($check) {
192 192
                     $site_url_parts = autoptimizeUtils::get_ao_wp_site_url_parts();
193
-                    $cdn_url_parts  = \parse_url( $cdn_url );
194
-                    $schemeless     = self::is_protocol_relative( $cdn_url );
195
-                    $cdn_url_parts  = self::maybe_replace_cdn_path( $site_url_parts, $cdn_url_parts );
196
-                    if ( false !== $cdn_url_parts ) {
197
-                        $results[ $cdn_url ] = self::assemble_parsed_url( $cdn_url_parts, $schemeless );
193
+                    $cdn_url_parts  = \parse_url($cdn_url);
194
+                    $schemeless     = self::is_protocol_relative($cdn_url);
195
+                    $cdn_url_parts  = self::maybe_replace_cdn_path($site_url_parts, $cdn_url_parts);
196
+                    if (false !== $cdn_url_parts) {
197
+                        $results[$cdn_url] = self::assemble_parsed_url($cdn_url_parts, $schemeless);
198 198
                     }
199 199
                 }
200 200
             }
201 201
         }
202 202
 
203
-        return $results[ $cdn_url ];
203
+        return $results[$cdn_url];
204 204
     }
205 205
 
206 206
     /**
@@ -214,10 +214,10 @@  discard block
 block discarded – undo
214 214
      *
215 215
      * @return array|false
216 216
      */
217
-    public static function maybe_replace_cdn_path( array $site_url_parts, array $cdn_url_parts )
217
+    public static function maybe_replace_cdn_path(array $site_url_parts, array $cdn_url_parts)
218 218
     {
219
-        if ( isset( $site_url_parts['path'] ) && '/' !== $site_url_parts['path'] ) {
220
-            if ( ! isset( $cdn_url_parts['path'] ) || '/' === $cdn_url_parts['path'] ) {
219
+        if (isset($site_url_parts['path']) && '/' !== $site_url_parts['path']) {
220
+            if (!isset($cdn_url_parts['path']) || '/' === $cdn_url_parts['path']) {
221 221
                 $cdn_url_parts['path'] = $site_url_parts['path'];
222 222
                 return $cdn_url_parts;
223 223
             }
@@ -237,20 +237,20 @@  discard block
 block discarded – undo
237 237
      *
238 238
      * @return string
239 239
      */
240
-    public static function assemble_parsed_url( array $parsed_url, $schemeless = false )
240
+    public static function assemble_parsed_url(array $parsed_url, $schemeless = false)
241 241
     {
242
-        $scheme = isset( $parsed_url['scheme'] ) ? $parsed_url['scheme'] . '://' : '';
243
-        if ( $schemeless ) {
242
+        $scheme = isset($parsed_url['scheme']) ? $parsed_url['scheme'].'://' : '';
243
+        if ($schemeless) {
244 244
             $scheme = '//';
245 245
         }
246
-        $host     = isset( $parsed_url['host'] ) ? $parsed_url['host'] : '';
247
-        $port     = isset( $parsed_url['port'] ) ? ':' . $parsed_url['port'] : '';
248
-        $user     = isset( $parsed_url['user'] ) ? $parsed_url['user'] : '';
249
-        $pass     = isset( $parsed_url['pass'] ) ? ':' . $parsed_url['pass'] : '';
250
-        $pass     = ( $user || $pass ) ? "$pass@" : '';
251
-        $path     = isset( $parsed_url['path'] ) ? $parsed_url['path'] : '';
252
-        $query    = isset( $parsed_url['query'] ) ? '?' . $parsed_url['query'] : '';
253
-        $fragment = isset( $parsed_url['fragment'] ) ? '#' . $parsed_url['fragment'] : '';
246
+        $host     = isset($parsed_url['host']) ? $parsed_url['host'] : '';
247
+        $port     = isset($parsed_url['port']) ? ':'.$parsed_url['port'] : '';
248
+        $user     = isset($parsed_url['user']) ? $parsed_url['user'] : '';
249
+        $pass     = isset($parsed_url['pass']) ? ':'.$parsed_url['pass'] : '';
250
+        $pass     = ($user || $pass) ? "$pass@" : '';
251
+        $path     = isset($parsed_url['path']) ? $parsed_url['path'] : '';
252
+        $query    = isset($parsed_url['query']) ? '?'.$parsed_url['query'] : '';
253
+        $fragment = isset($parsed_url['fragment']) ? '#'.$parsed_url['fragment'] : '';
254 254
 
255 255
         return "$scheme$user$pass$host$port$path$query$fragment";
256 256
     }
@@ -262,12 +262,12 @@  discard block
 block discarded – undo
262 262
      *
263 263
      * @return bool
264 264
      */
265
-    public static function is_protocol_relative( $url )
265
+    public static function is_protocol_relative($url)
266 266
     {
267 267
         $result = false;
268 268
 
269
-        if ( ! empty( $url ) ) {
270
-            $result = ( '/' === $url{1} ); // second char is `/`.
269
+        if (!empty($url)) {
270
+            $result = ('/' === $url{1} ); // second char is `/`.
271 271
         }
272 272
 
273 273
         return $result;
@@ -280,9 +280,9 @@  discard block
 block discarded – undo
280 280
      *
281 281
      * @return string
282 282
      */
283
-    public static function path_canonicalize( $path )
283
+    public static function path_canonicalize($path)
284 284
     {
285
-        $patterns     = array(
285
+        $patterns = array(
286 286
             '~/{2,}~',
287 287
             '~/(\./)+~',
288 288
             '~([^/\.]+/(?R)*\.{2,}/)~',
@@ -295,7 +295,7 @@  discard block
 block discarded – undo
295 295
             '',
296 296
         );
297 297
 
298
-        return preg_replace( $patterns, $replacements, $path );
298
+        return preg_replace($patterns, $replacements, $path);
299 299
     }
300 300
 
301 301
     /**
@@ -305,15 +305,15 @@  discard block
 block discarded – undo
305 305
      *
306 306
      * @return none if $return_result is false (default), array if $return_result is true.
307 307
      */
308
-    public static function check_service_availability( $return_result = false )
308
+    public static function check_service_availability($return_result = false)
309 309
     {
310
-        $service_availability_resp = wp_remote_get( 'https://misc.optimizingmatters.com/api/autoptimize_service_availablity.json?from=aomain&ver=' . AUTOPTIMIZE_PLUGIN_VERSION );
311
-        if ( ! is_wp_error( $service_availability_resp ) ) {
312
-            if ( '200' == wp_remote_retrieve_response_code( $service_availability_resp ) ) {
313
-                $availabilities = json_decode( wp_remote_retrieve_body( $service_availability_resp ), true );
314
-                if ( is_array( $availabilities ) ) {
315
-                    update_option( 'autoptimize_service_availablity', $availabilities );
316
-                    if ( $return_result ) {
310
+        $service_availability_resp = wp_remote_get('https://misc.optimizingmatters.com/api/autoptimize_service_availablity.json?from=aomain&ver='.AUTOPTIMIZE_PLUGIN_VERSION);
311
+        if (!is_wp_error($service_availability_resp)) {
312
+            if ('200' == wp_remote_retrieve_response_code($service_availability_resp)) {
313
+                $availabilities = json_decode(wp_remote_retrieve_body($service_availability_resp), true);
314
+                if (is_array($availabilities)) {
315
+                    update_option('autoptimize_service_availablity', $availabilities);
316
+                    if ($return_result) {
317 317
                         return $availabilities;
318 318
                     }
319 319
                 }
@@ -328,10 +328,10 @@  discard block
 block discarded – undo
328 328
      *
329 329
      * @return bool
330 330
      */
331
-    public static function str_is_valid_regex( $string )
331
+    public static function str_is_valid_regex($string)
332 332
     {
333
-        set_error_handler( function() {}, E_WARNING );
334
-        $is_regex = ( false !== preg_match( $string, '' ) );
333
+        set_error_handler(function() {}, E_WARNING);
334
+        $is_regex = (false !== preg_match($string, ''));
335 335
         restore_error_handler();
336 336
 
337 337
         return $is_regex;
Please login to merge, or discard this patch.
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   +378 added lines, -378 removed lines patch added patch discarded remove patch
@@ -10,17 +10,17 @@  discard block
 block discarded – undo
10 10
     protected $ao;
11 11
 
12 12
     protected static $flexible_url_parts_js = [
13
-        'default'          => 'wp-content/cache/autoptimize/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
14
-        'custom'           => 'wp-content/c/ao/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
15
-        'multisite'        => 'wp-content/cache/autoptimize/1/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
16
-        'multisite_custom' => 'wp-content/c/ao/1/js/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
13
+        'default'          => 'wp-content/cache/autoptimize/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
14
+        'custom'           => 'wp-content/c/ao/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
15
+        'multisite'        => 'wp-content/cache/autoptimize/1/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
16
+        'multisite_custom' => 'wp-content/c/ao/1/js/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
17 17
     ];
18 18
 
19 19
     protected static $flexible_url_parts_css = [
20
-        'default'          => 'wp-content/cache/autoptimize/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
21
-        'custom'           => 'wp-content/c/ao/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
22
-        'multisite'        => 'wp-content/cache/autoptimize/1/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
23
-        'multisite_custom' => 'wp-content/c/ao/1/css/' . AUTOPTIMIZE_CACHEFILE_PREFIX,
20
+        'default'          => 'wp-content/cache/autoptimize/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
21
+        'custom'           => 'wp-content/c/ao/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
22
+        'multisite'        => 'wp-content/cache/autoptimize/1/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
23
+        'multisite_custom' => 'wp-content/c/ao/1/css/'.AUTOPTIMIZE_CACHEFILE_PREFIX,
24 24
     ];
25 25
 
26 26
     protected function getAoStylesDefaultOptions()
@@ -28,16 +28,16 @@  discard block
 block discarded – undo
28 28
         $conf = autoptimizeConfig::instance();
29 29
 
30 30
         return [
31
-            'aggregate'      => $conf->get( 'autoptimize_css_aggregate' ),
32
-            'justhead'       => $conf->get( 'autoptimize_css_justhead' ),
33
-            'datauris'       => $conf->get( 'autoptimize_css_datauris' ),
34
-            'defer'          => $conf->get( 'autoptimize_css_defer' ),
35
-            'defer_inline'   => $conf->get( 'autoptimize_css_defer_inline' ),
36
-            'inline'         => $conf->get( 'autoptimize_css_inline' ),
37
-            'css_exclude'    => $conf->get( 'autoptimize_css_exclude' ),
38
-            'cdn_url'        => $conf->get( 'autoptimize_cdn_url' ),
39
-            'include_inline' => $conf->get( 'autoptimize_css_include_inline' ),
40
-            'nogooglefont'   => $conf->get( 'autoptimize_css_nogooglefont' ),
31
+            'aggregate'      => $conf->get('autoptimize_css_aggregate'),
32
+            'justhead'       => $conf->get('autoptimize_css_justhead'),
33
+            'datauris'       => $conf->get('autoptimize_css_datauris'),
34
+            'defer'          => $conf->get('autoptimize_css_defer'),
35
+            'defer_inline'   => $conf->get('autoptimize_css_defer_inline'),
36
+            'inline'         => $conf->get('autoptimize_css_inline'),
37
+            'css_exclude'    => $conf->get('autoptimize_css_exclude'),
38
+            'cdn_url'        => $conf->get('autoptimize_cdn_url'),
39
+            'include_inline' => $conf->get('autoptimize_css_include_inline'),
40
+            'nogooglefont'   => $conf->get('autoptimize_css_nogooglefont'),
41 41
         ];
42 42
     }
43 43
 
@@ -46,13 +46,13 @@  discard block
 block discarded – undo
46 46
         $conf = autoptimizeConfig::instance();
47 47
 
48 48
         return [
49
-            'aggregate'      => $conf->get( 'autoptimize_js_aggregate' ),
50
-            'justhead'       => $conf->get( 'autoptimize_js_justhead' ),
51
-            'forcehead'      => $conf->get( 'autoptimize_js_forcehead' ),
52
-            'trycatch'       => $conf->get( 'autoptimize_js_trycatch' ),
53
-            'js_exclude'     => $conf->get( 'autoptimize_js_exclude' ),
54
-            'cdn_url'        => $conf->get( 'autoptimize_cdn_url' ),
55
-            'include_inline' => $conf->get( 'autoptimize_js_include_inline' ),
49
+            'aggregate'      => $conf->get('autoptimize_js_aggregate'),
50
+            'justhead'       => $conf->get('autoptimize_js_justhead'),
51
+            'forcehead'      => $conf->get('autoptimize_js_forcehead'),
52
+            'trycatch'       => $conf->get('autoptimize_js_trycatch'),
53
+            'js_exclude'     => $conf->get('autoptimize_js_exclude'),
54
+            'cdn_url'        => $conf->get('autoptimize_cdn_url'),
55
+            'include_inline' => $conf->get('autoptimize_js_include_inline'),
56 56
         ];
57 57
     }
58 58
 
@@ -62,34 +62,34 @@  discard block
 block discarded – undo
62 62
     protected function get_urls()
63 63
     {
64 64
         static $site_url = null;
65
-        if ( null === $site_url ) {
65
+        if (null === $site_url) {
66 66
             $site_url = site_url();
67 67
         }
68 68
 
69 69
         static $cdn_url = null;
70
-        if ( null === $cdn_url ) {
71
-            $cdn_url = get_option( 'autoptimize_cdn_url' );
70
+        if (null === $cdn_url) {
71
+            $cdn_url = get_option('autoptimize_cdn_url');
72 72
         }
73 73
 
74 74
         static $imgopt_host = null;
75
-        if ( null === $imgopt_host ) {
76
-            $imgopt_host = rtrim( autoptimizeExtra::get_imgopt_host_wrapper(), '/' );
75
+        if (null === $imgopt_host) {
76
+            $imgopt_host = rtrim(autoptimizeExtra::get_imgopt_host_wrapper(), '/');
77 77
         }
78 78
 
79 79
         static $urls = [];
80 80
 
81
-        if ( empty( $urls ) ) {
81
+        if (empty($urls)) {
82 82
             $parts = autoptimizeUtils::get_ao_wp_site_url_parts();
83 83
             $urls  = [
84 84
                 'siteurl'    => $site_url,
85
-                'prsiteurl'  => '//' . str_replace( array( 'http://', 'https://' ), '', $site_url ),
86
-                'wwwsiteurl' => $parts['scheme'] . '://www.' . str_replace( 'www.', '', $parts['host'] ),
85
+                'prsiteurl'  => '//'.str_replace(array('http://', 'https://'), '', $site_url),
86
+                'wwwsiteurl' => $parts['scheme'].'://www.'.str_replace('www.', '', $parts['host']),
87 87
                 'cdnurl'     => $cdn_url,
88 88
                 'imgopthost' => $imgopt_host,
89 89
                 'subfolder'  => '',
90 90
             ];
91 91
 
92
-            if ( AO_TEST_SUBFOLDER_INSTALL ) {
92
+            if (AO_TEST_SUBFOLDER_INSTALL) {
93 93
                 $urls['subfolder'] = 'wordpress/';
94 94
             }
95 95
         }
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
      */
103 103
     public function setUp()
104 104
     {
105
-        $this->ao = new autoptimizeMain( AUTOPTIMIZE_PLUGIN_VERSION, AUTOPTIMIZE_PLUGIN_FILE );
105
+        $this->ao = new autoptimizeMain(AUTOPTIMIZE_PLUGIN_VERSION, AUTOPTIMIZE_PLUGIN_FILE);
106 106
 
107 107
         parent::setUp();
108 108
     }
@@ -121,8 +121,8 @@  discard block
 block discarded – undo
121 121
             'autoptimize_filter_css_inlinesize',
122 122
             'autoptimize_filter_css_fonts_cdn',
123 123
         );
124
-        foreach ( $filter_tags as $filter_tag ) {
125
-            remove_all_filters( $filter_tag );
124
+        foreach ($filter_tags as $filter_tag) {
125
+            remove_all_filters($filter_tag);
126 126
         }
127 127
 
128 128
         parent::tearDown();
@@ -246,15 +246,15 @@  discard block
 block discarded – undo
246 246
     protected function get_test_markup_output()
247 247
     {
248 248
         $key = 'default';
249
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
249
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
250 250
             $key = 'custom';
251 251
         }
252 252
 
253
-        $csspart = self::$flexible_url_parts_css[ $key ];
254
-        $jspart  = self::$flexible_url_parts_js[ $key ];
253
+        $csspart = self::$flexible_url_parts_css[$key];
254
+        $jspart  = self::$flexible_url_parts_js[$key];
255 255
 
256 256
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
257
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
257
+        if (AO_TEST_SUBFOLDER_INSTALL) {
258 258
             $csshash = '56398de576d59887e88e3011715250e0';
259 259
         }
260 260
 
@@ -313,15 +313,15 @@  discard block
 block discarded – undo
313 313
     protected function get_test_markup_output_ms()
314 314
     {
315 315
         $key = 'multisite';
316
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
316
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
317 317
             $key = 'multisite_custom';
318 318
         }
319 319
 
320
-        $csspart = self::$flexible_url_parts_css[ $key ];
321
-        $jspart  = self::$flexible_url_parts_js[ $key ];
320
+        $csspart = self::$flexible_url_parts_css[$key];
321
+        $jspart  = self::$flexible_url_parts_js[$key];
322 322
 
323 323
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
324
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
324
+        if (AO_TEST_SUBFOLDER_INSTALL) {
325 325
             $csshash = '56398de576d59887e88e3011715250e0';
326 326
         }
327 327
 
@@ -380,15 +380,15 @@  discard block
 block discarded – undo
380 380
     protected function get_test_markup_output_inline_defer()
381 381
     {
382 382
         $key = 'default';
383
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
383
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
384 384
             $key = 'custom';
385 385
         }
386 386
 
387
-        $csspart = self::$flexible_url_parts_css[ $key ];
388
-        $jspart  = self::$flexible_url_parts_js[ $key ];
387
+        $csspart = self::$flexible_url_parts_css[$key];
388
+        $jspart  = self::$flexible_url_parts_js[$key];
389 389
 
390 390
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
391
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
391
+        if (AO_TEST_SUBFOLDER_INSTALL) {
392 392
             $csshash = '56398de576d59887e88e3011715250e0';
393 393
         }
394 394
 
@@ -447,15 +447,15 @@  discard block
 block discarded – undo
447 447
     protected function get_test_markup_output_inline_defer_ms()
448 448
     {
449 449
         $key = 'multisite';
450
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
450
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
451 451
             $key = 'multisite_custom';
452 452
         }
453 453
 
454
-        $csspart = self::$flexible_url_parts_css[ $key ];
455
-        $jspart  = self::$flexible_url_parts_js[ $key ];
454
+        $csspart = self::$flexible_url_parts_css[$key];
455
+        $jspart  = self::$flexible_url_parts_js[$key];
456 456
 
457 457
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
458
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
458
+        if (AO_TEST_SUBFOLDER_INSTALL) {
459 459
             $csshash = '56398de576d59887e88e3011715250e0';
460 460
         }
461 461
 
@@ -514,11 +514,11 @@  discard block
 block discarded – undo
514 514
     /**
515 515
      * @dataProvider provider_test_rewrite_markup_with_cdn
516 516
      */
517
-    function test_rewrite_markup_with_cdn( $input, $expected )
517
+    function test_rewrite_markup_with_cdn($input, $expected)
518 518
     {
519
-        $actual = $this->ao->end_buffering( $input );
519
+        $actual = $this->ao->end_buffering($input);
520 520
 
521
-        $this->assertEquals( $expected, $actual );
521
+        $this->assertEquals($expected, $actual);
522 522
     }
523 523
 
524 524
     public function provider_test_rewrite_markup_with_cdn()
@@ -530,7 +530,7 @@  discard block
 block discarded – undo
530 530
                 $this->get_test_markup(),
531 531
                 // Expected output.
532 532
                 // TODO/FIXME: This seemed like the fastest way to get MS crude test to pass...
533
-                ( is_multisite() ? $this->get_test_markup_output_ms() : $this->get_test_markup_output() ),
533
+                (is_multisite() ? $this->get_test_markup_output_ms() : $this->get_test_markup_output()),
534 534
             ),
535 535
 
536 536
         );
@@ -590,12 +590,12 @@  discard block
 block discarded – undo
590 590
 }
591 591
 CSS;
592 592
 
593
-        $instance = new autoptimizeStyles( $css_in );
594
-        $instance->setOption( 'cdn_url', $cdnurl );
593
+        $instance = new autoptimizeStyles($css_in);
594
+        $instance->setOption('cdn_url', $cdnurl);
595 595
 
596
-        $css_actual = $instance->rewrite_assets( $css_in );
596
+        $css_actual = $instance->rewrite_assets($css_in);
597 597
 
598
-        $this->assertEquals( $css_expected, $css_actual );
598
+        $this->assertEquals($css_expected, $css_actual);
599 599
     }
600 600
 
601 601
     public function test_default_cssmin_minifier()
@@ -629,21 +629,21 @@  discard block
 block discarded – undo
629 629
 .bg{background:url('img/something.svg')}.bg-no-quote{background:url(img/something.svg)}.bg-double-quotes{background:url("img/something.svg")}.whitespaces{background:url ("../../somewhere-else/svg.svg")}.host-relative{background:url("/img/something.svg")}.protocol-relative{background:url("//something/somewhere/example.png")}@font-face{font-family:'Roboto';font-style:normal;font-weight:100;src:url(../fonts/roboto-v15-latin-ext_latin-100.eot);src:local('Roboto Thin'),local('Roboto-Thin'),url(../fonts/roboto-v15-latin-ext_latin-100.eot?#iefix) format('embedded-opentype'),url(../fonts/roboto-v15-latin-ext_latin-100.woff2) format('woff2'),url(../fonts/roboto-v15-latin-ext_latin-100.woff) format('woff'),url(../fonts/roboto-v15-latin-ext_latin-100.ttf) format('truetype'),url(../fonts/roboto-v15-latin-ext_latin-100.svg#Roboto) format('svg')}
630 630
 CSS;
631 631
 
632
-        $instance = new autoptimizeStyles( $css );
633
-        $minified = $instance->run_minifier_on( $css );
632
+        $instance = new autoptimizeStyles($css);
633
+        $minified = $instance->run_minifier_on($css);
634 634
 
635
-        $this->assertEquals( $expected, $minified );
635
+        $this->assertEquals($expected, $minified);
636 636
     }
637 637
 
638 638
     /**
639 639
      * @dataProvider provider_test_should_aggregate_script_types
640 640
      */
641
-    public function test_should_aggregate_script_types( $input, $expected )
641
+    public function test_should_aggregate_script_types($input, $expected)
642 642
     {
643
-        $instance = new autoptimizeScripts( '' );
644
-        $actual   = $instance->should_aggregate( $input );
643
+        $instance = new autoptimizeScripts('');
644
+        $actual   = $instance->should_aggregate($input);
645 645
 
646
-        $this->assertEquals( $expected, $actual );
646
+        $this->assertEquals($expected, $actual);
647 647
     }
648 648
 
649 649
     public function provider_test_should_aggregate_script_types()
@@ -705,11 +705,11 @@  discard block
 block discarded – undo
705 705
     /**
706 706
      * @dataProvider provider_is_valid_buffer
707 707
      */
708
-    public function test_valid_buffers( $input, $expected )
708
+    public function test_valid_buffers($input, $expected)
709 709
     {
710
-        $actual = $this->ao->is_valid_buffer( $input );
710
+        $actual = $this->ao->is_valid_buffer($input);
711 711
 
712
-        $this->assertEquals( $expected, $actual );
712
+        $this->assertEquals($expected, $actual);
713 713
     }
714 714
 
715 715
     public function provider_is_valid_buffer()
@@ -771,11 +771,11 @@  discard block
 block discarded – undo
771 771
     /**
772 772
      * @dataProvider provider_is_amp_markup
773 773
      */
774
-    public function test_autoptimize_is_amp_markup( $input, $expected )
774
+    public function test_autoptimize_is_amp_markup($input, $expected)
775 775
     {
776
-        $actual = autoptimizeMain::is_amp_markup( $input );
776
+        $actual = autoptimizeMain::is_amp_markup($input);
777 777
 
778
-        $this->assertEquals( $expected, $actual );
778
+        $this->assertEquals($expected, $actual);
779 779
     }
780 780
 
781 781
     public function provider_is_amp_markup()
@@ -806,31 +806,31 @@  discard block
 block discarded – undo
806 806
     public function test_skips_buffering_when_ao_noptimize_filter_is_true()
807 807
     {
808 808
         // True => disable autoptimize.
809
-        add_filter( 'autoptimize_filter_noptimize', '__return_true' );
809
+        add_filter('autoptimize_filter_noptimize', '__return_true');
810 810
 
811 811
         // Buffering should not run due to the above filter.
812 812
         $expected = false;
813
-        $actual   = $this->ao->should_buffer( $doing_tests = true );
813
+        $actual   = $this->ao->should_buffer($doing_tests = true);
814 814
 
815
-        $this->assertEquals( $expected, $actual );
815
+        $this->assertEquals($expected, $actual);
816 816
     }
817 817
 
818 818
     public function test_does_buffering_when_ao_noptimize_filter_is_false()
819 819
     {
820 820
         // False => disable noptimize, aka, run normally (weird, yes...).
821
-        add_filter( 'autoptimize_filter_noptimize', '__return_false' );
821
+        add_filter('autoptimize_filter_noptimize', '__return_false');
822 822
 
823 823
         // Buffering should run because of above.
824 824
         $expected = true;
825
-        $actual   = $this->ao->should_buffer( $doing_tests = true );
825
+        $actual   = $this->ao->should_buffer($doing_tests = true);
826 826
 
827
-        $this->assertEquals( $expected, $actual );
827
+        $this->assertEquals($expected, $actual);
828 828
     }
829 829
 
830 830
     public function test_ignores_ao_noptimize_qs_when_instructed()
831 831
     {
832 832
         // Should skip checking for the qs completely due to filter.
833
-        add_filter( 'autoptimize_filter_honor_qs_noptimize', '__return_false' );
833
+        add_filter('autoptimize_filter_honor_qs_noptimize', '__return_false');
834 834
 
835 835
         /**
836 836
          * The above should then result in the "current" value being `false`
@@ -841,23 +841,23 @@  discard block
 block discarded – undo
841 841
          */
842 842
 
843 843
         $that = $this; // Makes it work on php 5.3!
844
-        add_filter( 'autoptimize_filter_noptimize', function ( $current_value ) use ( $that ) {
844
+        add_filter('autoptimize_filter_noptimize', function($current_value) use ($that) {
845 845
             $expected = false;
846
-            if ( defined( 'DONOTMINIFY' ) && DONOTMINIFY ) {
846
+            if (defined('DONOTMINIFY') && DONOTMINIFY) {
847 847
                 $expected = true;
848 848
             }
849 849
 
850
-            $that->assertEquals( $expected, $current_value );
850
+            $that->assertEquals($expected, $current_value);
851 851
         });
852 852
 
853
-        $this->ao->should_buffer( $doing_tests = true );
853
+        $this->ao->should_buffer($doing_tests = true);
854 854
     }
855 855
 
856 856
     public function test_wpengine_cache_flush()
857 857
     {
858 858
         // Creating a mock so that we can get past class_exists() and method_exists() checks present
859 859
         // in `autoptimizeCache::flushPageCache()`...
860
-        $stub = $this->getMockBuilder( 'WpeCommon' )->disableAutoload()
860
+        $stub = $this->getMockBuilder('WpeCommon')->disableAutoload()
861 861
                 ->disableOriginalConstructor()->setMethods(
862 862
                     array(
863 863
                         'purge_varnish_cache',
@@ -866,9 +866,9 @@  discard block
 block discarded – undo
866 866
                 ->getMock();
867 867
 
868 868
         $that = $this;
869
-        add_filter( 'autoptimize_flush_wpengine_methods', function( $methods ) use ( $that ) {
870
-            $expected_methods = array( 'purge_varnish_cache' );
871
-            $that->assertEquals( $methods, $expected_methods );
869
+        add_filter('autoptimize_flush_wpengine_methods', function($methods) use ($that) {
870
+            $expected_methods = array('purge_varnish_cache');
871
+            $that->assertEquals($methods, $expected_methods);
872 872
 
873 873
             return $methods;
874 874
         });
@@ -882,7 +882,7 @@  discard block
 block discarded – undo
882 882
     public function test_wpengine_cache_flush_agressive()
883 883
     {
884 884
         // Creating a mock so that we can get past class_exists() and method_exists() checks `autoptimize_flush_pagecache()`...
885
-        $stub = $this->getMockBuilder( 'WpeCommon' )->disableAutoload()
885
+        $stub = $this->getMockBuilder('WpeCommon')->disableAutoload()
886 886
                 ->disableOriginalConstructor()->setMethods(
887 887
                     array(
888 888
                         'purge_varnish_cache',
@@ -892,19 +892,19 @@  discard block
 block discarded – undo
892 892
                 )
893 893
                 ->getMock();
894 894
 
895
-        add_filter( 'autoptimize_flush_wpengine_aggressive', function() {
895
+        add_filter('autoptimize_flush_wpengine_aggressive', function() {
896 896
             return true;
897 897
         });
898 898
 
899 899
         $that = $this;
900
-        add_filter( 'autoptimize_flush_wpengine_methods', function( $methods ) use ( $that ) {
900
+        add_filter('autoptimize_flush_wpengine_methods', function($methods) use ($that) {
901 901
             $expected_methods = array(
902 902
                 'purge_varnish_cache',
903 903
                 'purge_memcached',
904 904
                 'clear_maxcdn_cache',
905 905
             );
906 906
 
907
-            $that->assertEquals( $methods, $expected_methods );
907
+            $that->assertEquals($methods, $expected_methods);
908 908
 
909 909
             return $methods;
910 910
         });
@@ -915,15 +915,15 @@  discard block
 block discarded – undo
915 915
     /**
916 916
      * @dataProvider provider_test_url_replace_cdn
917 917
      */
918
-    public function test_url_replace_cdn( $cdn_url, $input, $expected )
918
+    public function test_url_replace_cdn($cdn_url, $input, $expected)
919 919
     {
920
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )
920
+        $mock = $this->getMockBuilder('autoptimizeBase')
921 921
                 ->disableOriginalConstructor()
922 922
                 ->getMockForAbstractClass();
923 923
 
924 924
         $mock->cdn_url = $cdn_url;
925
-        $actual        = $mock->url_replace_cdn( $input );
926
-        $this->assertEquals( $expected, $actual );
925
+        $actual        = $mock->url_replace_cdn($input);
926
+        $this->assertEquals($expected, $actual);
927 927
     }
928 928
 
929 929
     public function provider_test_url_replace_cdn()
@@ -940,13 +940,13 @@  discard block
 block discarded – undo
940 940
                 // CDN base url, url, expected result...
941 941
                 'http://cdn-test.example.org',
942 942
                 '/a.jpg',
943
-                'http://cdn-test.example.org/' . $subfolder . 'a.jpg',
943
+                'http://cdn-test.example.org/'.$subfolder.'a.jpg',
944 944
             ),
945 945
             // Full link with a matching AUTOPTIMIZE_WP_SITE_URL gets properly replaced...
946 946
             array(
947 947
                 'http://cdn-test.example.org',
948
-                $siteurl . '/wp-content/themes/something/example.svg',
949
-                'http://cdn-test.example.org/' . $subfolder . 'wp-content/themes/something/example.svg',
948
+                $siteurl.'/wp-content/themes/something/example.svg',
949
+                'http://cdn-test.example.org/'.$subfolder.'wp-content/themes/something/example.svg',
950 950
             ),
951 951
             // Protocol-relative url with a "local" hostname that doesn't match example.org (AUTOPTIMIZE_WP_SITE_URL)...
952 952
             array(
@@ -957,59 +957,59 @@  discard block
 block discarded – undo
957 957
             // www.example.org does not match example.org (AUTOPTIMIZE_WP_SITE_URL) so it's left alone...
958 958
             array(
959 959
                 'http://cdn-test.example.org',
960
-                $wwwsiteurl . '/wp-content/themes/something/example.svg',
961
-                $wwwsiteurl . '/wp-content/themes/something/example.svg',
960
+                $wwwsiteurl.'/wp-content/themes/something/example.svg',
961
+                $wwwsiteurl.'/wp-content/themes/something/example.svg',
962 962
             ),
963 963
             // SSL cdn url + host-relative link...
964 964
             array(
965 965
                 'https://cdn.example.org',
966 966
                 '/a.jpg',
967
-                'https://cdn.example.org/' . $subfolder . 'a.jpg',
967
+                'https://cdn.example.org/'.$subfolder.'a.jpg',
968 968
             ),
969 969
             // SSL cdn url + http site url that matches AUTOPTIMIZE_WP_SITE_URL is properly replaced...
970 970
             array(
971 971
                 'https://cdn.example.org',
972
-                $siteurl . '/wp-content/themes/something/example.svg',
973
-                'https://cdn.example.org/' . $subfolder . 'wp-content/themes/something/example.svg',
972
+                $siteurl.'/wp-content/themes/something/example.svg',
973
+                'https://cdn.example.org/'.$subfolder.'wp-content/themes/something/example.svg',
974 974
             ),
975 975
             // Protocol-relative cdn url given with protocol relative link that matches AUTOPTIMIZE_WP_SITE_URL host...
976 976
             array(
977 977
                 '//cdn.example.org',
978
-                $protorelativesiteurl . '/something.jpg',
979
-                '//cdn.example.org/' . $subfolder . 'something.jpg',
978
+                $protorelativesiteurl.'/something.jpg',
979
+                '//cdn.example.org/'.$subfolder.'something.jpg',
980 980
             ),
981 981
             // Protocol-relative cdn url given a http link that matches AUTOPTIMIZE_WP_SITE_URL host...
982 982
             array(
983 983
                 '//cdn.example.org',
984
-                $siteurl . '/something.png',
985
-                '//cdn.example.org/' . $subfolder . 'something.png',
984
+                $siteurl.'/something.png',
985
+                '//cdn.example.org/'.$subfolder.'something.png',
986 986
             ),
987 987
             // Protocol-relative cdn url with a host-relative link...
988 988
             array(
989 989
                 '//cdn.example.org',
990 990
                 '/a.jpg',
991
-                '//cdn.example.org/' . $subfolder . 'a.jpg',
991
+                '//cdn.example.org/'.$subfolder.'a.jpg',
992 992
             ),
993 993
             // Testing cdn urls with an explicit port number...
994 994
             array(
995 995
                 'http://cdn.com:8080',
996 996
                 '/a.jpg',
997
-                'http://cdn.com:8080/' . $subfolder . 'a.jpg',
997
+                'http://cdn.com:8080/'.$subfolder.'a.jpg',
998 998
             ),
999 999
             array(
1000 1000
                 '//cdn.com:4433',
1001 1001
                 '/a.jpg',
1002
-                '//cdn.com:4433/' . $subfolder . 'a.jpg',
1002
+                '//cdn.com:4433/'.$subfolder.'a.jpg',
1003 1003
             ),
1004 1004
             array(
1005 1005
                 '//cdn.com:4433',
1006
-                $siteurl . '/something.jpg',
1007
-                '//cdn.com:4433/' . $subfolder . 'something.jpg',
1006
+                $siteurl.'/something.jpg',
1007
+                '//cdn.com:4433/'.$subfolder.'something.jpg',
1008 1008
             ),
1009 1009
             array(
1010 1010
                 '//cdn.com:1234',
1011
-                $protorelativesiteurl . '/something.jpg',
1012
-                '//cdn.com:1234/' . $subfolder . 'something.jpg',
1011
+                $protorelativesiteurl.'/something.jpg',
1012
+                '//cdn.com:1234/'.$subfolder.'something.jpg',
1013 1013
             ),
1014 1014
             // Relative links should not be touched by url_replace_cdn()...
1015 1015
             array(
@@ -1034,8 +1034,8 @@  discard block
 block discarded – undo
1034 1034
             // even though they are really strange...
1035 1035
             array(
1036 1036
                 $siteurl, // example.org or http://localhost or http://localhost/wordpress
1037
-                $siteurl . '/something.jpg',
1038
-                $siteurl . '/something.jpg',
1037
+                $siteurl.'/something.jpg',
1038
+                $siteurl.'/something.jpg',
1039 1039
             ),
1040 1040
             // These shouldn't really be changed, or even if replacements do
1041 1041
             // happen, they shouldn't be destructive...
@@ -1074,31 +1074,31 @@  discard block
 block discarded – undo
1074 1074
         $test_link = '/a.jpg';
1075 1075
         $cdn_url   = '//cdn.example.org';
1076 1076
 
1077
-        $with_ssl = function( $cdn ) {
1077
+        $with_ssl = function($cdn) {
1078 1078
             return '';
1079 1079
         };
1080 1080
         $expected_with_ssl = '/a.jpg';
1081 1081
 
1082
-        $without_ssl = function( $cdn ) {
1082
+        $without_ssl = function($cdn) {
1083 1083
             return $cdn;
1084 1084
         };
1085
-        $expected_without_ssl = '//cdn.example.org/' . $sub . 'a.jpg';
1085
+        $expected_without_ssl = '//cdn.example.org/'.$sub.'a.jpg';
1086 1086
 
1087 1087
         // With a filter that returns something considered "empty", cdn replacement shouldn't occur...
1088
-        add_filter( 'autoptimize_filter_base_cdnurl', $with_ssl );
1089
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )
1088
+        add_filter('autoptimize_filter_base_cdnurl', $with_ssl);
1089
+        $mock = $this->getMockBuilder('autoptimizeBase')
1090 1090
                 ->disableOriginalConstructor()
1091 1091
                 ->getMockForAbstractClass();
1092 1092
 
1093 1093
         $mock->cdn_url   = $cdn_url;
1094
-        $actual_with_ssl = $mock->url_replace_cdn( $test_link );
1095
-        $this->assertEquals( $expected_with_ssl, $actual_with_ssl );
1096
-        remove_filter( 'autoptimize_filter_base_cdnurl', $with_ssl );
1094
+        $actual_with_ssl = $mock->url_replace_cdn($test_link);
1095
+        $this->assertEquals($expected_with_ssl, $actual_with_ssl);
1096
+        remove_filter('autoptimize_filter_base_cdnurl', $with_ssl);
1097 1097
 
1098 1098
         // With a filter that returns an actual cdn url, cdn replacement should occur.
1099
-        add_filter( 'autoptimize_filter_base_cdnurl', $without_ssl );
1100
-        $actual_without_ssl = $mock->url_replace_cdn( $test_link );
1101
-        $this->assertEquals( $expected_without_ssl, $actual_without_ssl );
1099
+        add_filter('autoptimize_filter_base_cdnurl', $without_ssl);
1100
+        $actual_without_ssl = $mock->url_replace_cdn($test_link);
1101
+        $this->assertEquals($expected_without_ssl, $actual_without_ssl);
1102 1102
     }
1103 1103
 
1104 1104
     public function provider_cssmin_issues()
@@ -1137,11 +1137,11 @@  discard block
 block discarded – undo
1137 1137
     /**
1138 1138
      * @dataProvider provider_cssmin_issues
1139 1139
      */
1140
-    public function test_cssmin_issues( $input, $expected )
1140
+    public function test_cssmin_issues($input, $expected)
1141 1141
     {
1142
-        $minifier = new autoptimizeCSSmin( false ); // No need to raise limits for now.
1143
-        $actual   = $minifier->run( $input );
1144
-        $this->assertEquals( $expected, $actual );
1142
+        $minifier = new autoptimizeCSSmin(false); // No need to raise limits for now.
1143
+        $actual   = $minifier->run($input);
1144
+        $this->assertEquals($expected, $actual);
1145 1145
     }
1146 1146
 
1147 1147
     public function provider_getpath()
@@ -1166,7 +1166,7 @@  discard block
 block discarded – undo
1166 1166
             // This file comes with core, so should exist...
1167 1167
             array(
1168 1168
                 '/wp-includes/js/jquery/jquery.js',
1169
-                WP_ROOT_DIR . '/wp-includes/js/jquery/jquery.js',
1169
+                WP_ROOT_DIR.'/wp-includes/js/jquery/jquery.js',
1170 1170
             ),
1171 1171
             // Empty $url should return false.
1172 1172
             array(
@@ -1191,12 +1191,12 @@  discard block
 block discarded – undo
1191 1191
     /**
1192 1192
      * @dataProvider provider_getpath
1193 1193
      */
1194
-    public function test_getpath( $input, $expected )
1194
+    public function test_getpath($input, $expected)
1195 1195
     {
1196
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )->disableOriginalConstructor()->getMockForAbstractClass();
1196
+        $mock = $this->getMockBuilder('autoptimizeBase')->disableOriginalConstructor()->getMockForAbstractClass();
1197 1197
 
1198
-        $actual = $mock->getpath( $input );
1199
-        $this->assertEquals( $expected, $actual );
1198
+        $actual = $mock->getpath($input);
1199
+        $this->assertEquals($expected, $actual);
1200 1200
     }
1201 1201
 
1202 1202
     /**
@@ -1210,7 +1210,7 @@  discard block
 block discarded – undo
1210 1210
 
1211 1211
         $protorelativesiteurl = $urls['prsiteurl'];
1212 1212
 
1213
-        $css_orig     = <<<CSS
1213
+        $css_orig = <<<CSS
1214 1214
 header{background-image:url("data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' width='90px' height='110px' viewBox='0 0 90 110'%3E%3Cstyle%3E.a%7Bstop-color:%23FFF;%7D.b%7Bstop-color:%23B2D235;%7D.c%7Bstop-color:%23BEE7FA;%7D.d%7Bfill:%23590C15;%7D%3C/style%3E%3ClinearGradient id='c' y2='135.4' gradientUnits='userSpaceOnUse' x2='209.1' gradientTransform='rotate(-1.467 -4082.888 7786.794)' y1='205.8' x1='262'%3E%3Cstop class='b' offset='0'/%3E%3Cstop class='b' offset='.48'/%3E%3Cstop stop-color='%23829D25' offset='1'/%3E%3C/linearGradient%3E%3Cpath stroke-width='.3' d='M77.3 45.4c-3-3.5-7.1-6.5-11.6-7.8-5.1-1.5-10-.1-14.9 1.5C52 35.4 54.3 29 60 24l-4.8-5.5c-3.4 3-5.8 6.3-7.5 9.4-1.7-4.3-4.1-8.4-7.5-12C33.4 8.6 24.3 4.7 15.1 4.2c-.2 9.3 3.1 18.6 9.9 25.9 5.2 5.6 11.8 9.2 18.7 10.8-2.5.2-4.9-.1-7.7-.9-5.2-1.4-10.5-2.8-15.8-1C10.6 42.3 4.5 51.9 4 61.7c-.5 11.6 3.8 23.8 9.9 33.5 3.9 6.3 9.6 13.7 17.7 13.4 3.8-.1 7-2.1 10.7-2.7 5.2-.8 9.1 1.2 14.1 1.8 16.4 2 24.4-23.6 26.4-35.9 1.2-9.1.8-19.1-5.5-26.4z' stroke='%233E6D1F' fill='url(%23c)'/%3E%3C/svg%3E")}
1215 1215
 section.clipped.clippedTop {clip-path:url("#clipPolygonTop")}
1216 1216
 section.clipped.clippedBottom {clip-path:url("#clipPolygonBottom")}
@@ -1223,13 +1223,13 @@  discard block
 block discarded – undo
1223 1223
 .myimg {background-image: url($protorelativesiteurl/wp-content/themes/my-theme/images/under-left-leaf.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/over-blue-bird.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/under-top.png), url($protorelativesiteurl/wp-content/themes/my-theme/images/bg-top-grunge.png);}
1224 1224
 CSS;
1225 1225
 
1226
-        $fixurls_result = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $css_orig );
1227
-        $this->assertEquals( $css_expected, $fixurls_result );
1226
+        $fixurls_result = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $css_orig);
1227
+        $this->assertEquals($css_expected, $fixurls_result);
1228 1228
     }
1229 1229
 
1230 1230
     public function test_background_datauri_sprites_with_fixurls()
1231 1231
     {
1232
-        $css_orig     = <<<CSS
1232
+        $css_orig = <<<CSS
1233 1233
 .shadow { background:url(img/1x1.png) top center; }
1234 1234
 .shadow1 { background-image:url(img/1x1.png) 0 -767px repeat-x; }
1235 1235
 .shadow2 {background:url(img/1x1.png) top center}
@@ -1281,27 +1281,27 @@  discard block
 block discarded – undo
1281 1281
 CSS;
1282 1282
 
1283 1283
         // For test purposes, ALL images in the css are being inline with a 1x1 trans png string/datauri.
1284
-        add_filter( 'autoptimize_filter_css_is_datauri_candidate', function( $is_candidate, $path ) {
1284
+        add_filter('autoptimize_filter_css_is_datauri_candidate', function($is_candidate, $path) {
1285 1285
             return true;
1286
-        }, 10, 2 );
1286
+        }, 10, 2);
1287 1287
 
1288 1288
         // For test purposes, ALL images in the css are being inline with a 1x1 trans png string/datauri.
1289
-        add_filter( 'autoptimize_filter_css_datauri_image', function( $base64array, $path ) {
1289
+        add_filter('autoptimize_filter_css_datauri_image', function($base64array, $path) {
1290 1290
             $head = 'data:image/png;base64,';
1291 1291
             $data = 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAQAAAC1HAwCAAAAC0lEQVR42mNkYAAAAAYAAjCB0C8AAAAASUVORK5CYII=';
1292 1292
 
1293
-            $result['full']       = $head . $data;
1293
+            $result['full']       = $head.$data;
1294 1294
             $result['base64data'] = $data;
1295 1295
             return $result;
1296
-        }, 10, 2 );
1296
+        }, 10, 2);
1297 1297
 
1298
-        $instance = new autoptimizeStyles( $css_orig );
1299
-        $instance->setOption( 'datauris', true );
1298
+        $instance = new autoptimizeStyles($css_orig);
1299
+        $instance->setOption('datauris', true);
1300 1300
 
1301
-        $fixurls_result = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $css_orig );
1302
-        $css_actual     = $instance->rewrite_assets( $fixurls_result );
1301
+        $fixurls_result = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $css_orig);
1302
+        $css_actual     = $instance->rewrite_assets($fixurls_result);
1303 1303
 
1304
-        $this->assertEquals( $css_expected, $css_actual );
1304
+        $this->assertEquals($css_expected, $css_actual);
1305 1305
     }
1306 1306
 
1307 1307
     /**
@@ -1321,7 +1321,7 @@  discard block
 block discarded – undo
1321 1321
      */
1322 1322
     public function test_background_datauri_sprites_without_fixurls()
1323 1323
     {
1324
-        $css_orig     = <<<CSS
1324
+        $css_orig = <<<CSS
1325 1325
 .shadow { background:url(img/1x1.png) top center; }
1326 1326
 .shadow1 { background-image:url(img/1x1.png) 0 -767px repeat-x; }
1327 1327
 .shadow2 {background:url(img/1x1.png) top center}
@@ -1365,24 +1365,24 @@  discard block
 block discarded – undo
1365 1365
 CSS;
1366 1366
 
1367 1367
         // For test purposes, ALL images in the css are being inlined with a 1x1 trans png string/datauri.
1368
-        add_filter( 'autoptimize_filter_css_is_datauri_candidate', function( $is_candidate, $path ) {
1368
+        add_filter('autoptimize_filter_css_is_datauri_candidate', function($is_candidate, $path) {
1369 1369
             return true;
1370
-        }, 10, 2 );
1370
+        }, 10, 2);
1371 1371
 
1372 1372
         // For test purposes, ALL images in the css are being inlined with a 1x1 trans png string/datauri.
1373
-        add_filter( 'autoptimize_filter_css_datauri_image', function( $base64array, $path ) {
1373
+        add_filter('autoptimize_filter_css_datauri_image', function($base64array, $path) {
1374 1374
             $head = 'data:image/png;base64,';
1375 1375
             $data = 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAQAAAC1HAwCAAAAC0lEQVR42mNkYAAAAAYAAjCB0C8AAAAASUVORK5CYII=';
1376 1376
 
1377
-            $result['full']       = $head . $data;
1377
+            $result['full']       = $head.$data;
1378 1378
             $result['base64data'] = $data;
1379 1379
             return $result;
1380
-        }, 10, 2 );
1380
+        }, 10, 2);
1381 1381
 
1382
-        $instance = new autoptimizeStyles( $css_orig );
1383
-        $instance->setOption( 'datauris', true );
1384
-        $css_actual = $instance->rewrite_assets( $css_orig );
1385
-        $this->assertEquals( $css_expected, $css_actual );
1382
+        $instance = new autoptimizeStyles($css_orig);
1383
+        $instance->setOption('datauris', true);
1384
+        $css_actual = $instance->rewrite_assets($css_orig);
1385
+        $this->assertEquals($css_expected, $css_actual);
1386 1386
     }
1387 1387
 
1388 1388
     /**
@@ -1502,12 +1502,12 @@  discard block
 block discarded – undo
1502 1502
 CSS;
1503 1503
 
1504 1504
         // Test with fonts pointed to the CDN + cdn option is set.
1505
-        add_filter( 'autoptimize_filter_css_fonts_cdn', '__return_true' );
1506
-        $instance = new autoptimizeStyles( $css_in );
1507
-        $instance->setOption( 'cdn_url', $cdnurl );
1508
-        $css_actual_fonts_cdn = $instance->rewrite_assets( $css_in );
1505
+        add_filter('autoptimize_filter_css_fonts_cdn', '__return_true');
1506
+        $instance = new autoptimizeStyles($css_in);
1507
+        $instance->setOption('cdn_url', $cdnurl);
1508
+        $css_actual_fonts_cdn = $instance->rewrite_assets($css_in);
1509 1509
 
1510
-        $this->assertEquals( $css_expected_fonts_cdn, $css_actual_fonts_cdn );
1510
+        $this->assertEquals($css_expected_fonts_cdn, $css_actual_fonts_cdn);
1511 1511
     }
1512 1512
 
1513 1513
     /**
@@ -1623,10 +1623,10 @@  discard block
 block discarded – undo
1623 1623
 }
1624 1624
 CSS;
1625 1625
         // Test without moving fonts to CDN, but cdn option is set.
1626
-        $instance = new autoptimizeStyles( $css_in );
1627
-        $instance->setOption( 'cdn_url', $urls['cdnurl'] );
1628
-        $css_actual = $instance->rewrite_assets( $css_in );
1629
-        $this->assertEquals( $css_expected, $css_actual );
1626
+        $instance = new autoptimizeStyles($css_in);
1627
+        $instance->setOption('cdn_url', $urls['cdnurl']);
1628
+        $css_actual = $instance->rewrite_assets($css_in);
1629
+        $this->assertEquals($css_expected, $css_actual);
1630 1630
     }
1631 1631
 
1632 1632
     public function test_assets_regex_replaces_multi_bg_images()
@@ -1647,16 +1647,16 @@  discard block
 block discarded – undo
1647 1647
 }
1648 1648
 CSS;
1649 1649
 
1650
-        $instance = new autoptimizeStyles( $in );
1651
-        $instance->setOption( 'cdn_url', $cdnurl );
1652
-        $actual = $instance->rewrite_assets( $in );
1650
+        $instance = new autoptimizeStyles($in);
1651
+        $instance->setOption('cdn_url', $cdnurl);
1652
+        $actual = $instance->rewrite_assets($in);
1653 1653
 
1654
-        $this->assertEquals( $expected, $actual );
1654
+        $this->assertEquals($expected, $actual);
1655 1655
     }
1656 1656
 
1657 1657
     public function test_at_supports_spacing_issue_110()
1658 1658
     {
1659
-        $in       = <<<CSS
1659
+        $in = <<<CSS
1660 1660
 @supports (-webkit-filter: blur(3px)) or (filter: blur(3px)) {
1661 1661
     .blur {
1662 1662
         filter:blur(3px);
@@ -1670,10 +1670,10 @@  discard block
 block discarded – undo
1670 1670
 @supports (-webkit-filter:blur(3px)) or (filter:blur(3px)){.blur{filter:blur(3px)}}@supports((position:-webkit-sticky) or (position:sticky)){.sticky{position:sticky}}
1671 1671
 CSS;
1672 1672
 
1673
-        $instance = new autoptimizeStyles( $in );
1674
-        $actual   = $instance->run_minifier_on( $in );
1673
+        $instance = new autoptimizeStyles($in);
1674
+        $actual   = $instance->run_minifier_on($in);
1675 1675
 
1676
-        $this->assertEquals( $expected, $actual );
1676
+        $this->assertEquals($expected, $actual);
1677 1677
     }
1678 1678
 
1679 1679
     public function test_css_import_semicolon_url_issue_122()
@@ -1689,18 +1689,18 @@  discard block
 block discarded – undo
1689 1689
 </style>
1690 1690
 HTML;
1691 1691
 
1692
-        $expected = '<style type="text/css" media="all">@import url(' . $cdnurl . '/' . $sub . 'foo.css?a&#038;b);@import url(' . $cdnurl . '/' . $sub . 'bar.css);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1692
+        $expected = '<style type="text/css" media="all">@import url('.$cdnurl.'/'.$sub.'foo.css?a&#038;b);@import url('.$cdnurl.'/'.$sub.'bar.css);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1693 1693
 
1694 1694
         $options = [
1695 1695
             'autoptimizeStyles' => $this->getAoStylesDefaultOptions(),
1696 1696
         ];
1697 1697
 
1698
-        $instance = new autoptimizeStyles( $in );
1699
-        $instance->read( $options['autoptimizeStyles'] );
1698
+        $instance = new autoptimizeStyles($in);
1699
+        $instance->read($options['autoptimizeStyles']);
1700 1700
         $instance->minify();
1701 1701
         $instance->cache();
1702 1702
         $actual = $instance->getcontent();
1703
-        $this->assertEquals( $expected, $actual );
1703
+        $this->assertEquals($expected, $actual);
1704 1704
     }
1705 1705
 
1706 1706
     public function test_fixurls_with_at_imports_and_media_queries()
@@ -1710,10 +1710,10 @@  discard block
 block discarded – undo
1710 1710
         $subfolder = $urls['subfolder'];
1711 1711
 
1712 1712
         $in  = '@import "foo.css"; @import "bar.css" (orientation:landscape);';
1713
-        $exp = '@import url(' . $prsiteurl . '/wp-content/themes/my-theme/foo.css); @import url(' . $prsiteurl . '/wp-content/themes/my-theme/bar.css) (orientation:landscape);';
1713
+        $exp = '@import url('.$prsiteurl.'/wp-content/themes/my-theme/foo.css); @import url('.$prsiteurl.'/wp-content/themes/my-theme/bar.css) (orientation:landscape);';
1714 1714
 
1715
-        $actual = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $in );
1716
-        $this->assertEquals( $exp, $actual );
1715
+        $actual = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $in);
1716
+        $this->assertEquals($exp, $actual);
1717 1717
     }
1718 1718
 
1719 1719
     public function test_aostyles_at_imports_with_media_queries()
@@ -1728,43 +1728,43 @@  discard block
 block discarded – undo
1728 1728
 </style>
1729 1729
 HTML;
1730 1730
 
1731
-        $expected = '<style type="text/css" media="all">@import url(' . $cdnurl . '/' . $sub . 'foo.css);@import url(' . $cdnurl . '/' . $sub . 'bar.css) (orientation:landscape);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1731
+        $expected = '<style type="text/css" media="all">@import url('.$cdnurl.'/'.$sub.'foo.css);@import url('.$cdnurl.'/'.$sub.'bar.css) (orientation:landscape);</style><!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `title` missing --><!--/noptimize-->';
1732 1732
 
1733 1733
         $options = [
1734 1734
             'autoptimizeStyles' => $this->getAoStylesDefaultOptions(),
1735 1735
         ];
1736 1736
 
1737
-        $instance = new autoptimizeStyles( $in );
1738
-        $instance->read( $options['autoptimizeStyles'] );
1737
+        $instance = new autoptimizeStyles($in);
1738
+        $instance->read($options['autoptimizeStyles']);
1739 1739
         $instance->minify();
1740 1740
         $instance->cache();
1741 1741
 
1742 1742
         $actual = $instance->getcontent();
1743
-        $this->assertEquals( $expected, $actual );
1743
+        $this->assertEquals($expected, $actual);
1744 1744
     }
1745 1745
 
1746 1746
     public function test_cache_size_checker_hooked_by_default()
1747 1747
     {
1748
-        $this->assertNotEmpty( autoptimizeCacheChecker::SCHEDULE_HOOK );
1748
+        $this->assertNotEmpty(autoptimizeCacheChecker::SCHEDULE_HOOK);
1749 1749
 
1750 1750
         // No schedule, because it's only added when is_admin() is true.
1751
-        $this->assertEquals( false, wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1751
+        $this->assertEquals(false, wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1752 1752
 
1753 1753
         // Proving that setup() sets the schedule as needed.
1754 1754
         $checker = new autoptimizeCacheChecker();
1755 1755
         $checker->setup();
1756
-        $this->assertEquals( 'twicedaily', wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1756
+        $this->assertEquals('twicedaily', wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1757 1757
     }
1758 1758
 
1759 1759
     public function test_cache_size_checker_disabled_with_filter()
1760 1760
     {
1761
-        add_filter( 'autoptimize_filter_cachecheck_do', '__return_false' );
1761
+        add_filter('autoptimize_filter_cachecheck_do', '__return_false');
1762 1762
 
1763 1763
         $checker = new autoptimizeCacheChecker();
1764 1764
         $checker->setup();
1765
-        $this->assertEquals( false, wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1765
+        $this->assertEquals(false, wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1766 1766
 
1767
-        remove_all_filters( 'autoptimize_filter_cachecheck_do' );
1767
+        remove_all_filters('autoptimize_filter_cachecheck_do');
1768 1768
     }
1769 1769
 
1770 1770
     public function test_is_start_buffering_hooked_properly()
@@ -1777,12 +1777,12 @@  discard block
 block discarded – undo
1777 1777
          * https://www.theaveragedev.com/mocking-constants-in-tests/.
1778 1778
          */
1779 1779
 
1780
-        if ( defined( 'AUTOPTIMIZE_INIT_EARLIER' ) ) {
1780
+        if (defined('AUTOPTIMIZE_INIT_EARLIER')) {
1781 1781
             $this->assertEquals(
1782 1782
                 autoptimizeMain::INIT_EARLIER_PRIORITY,
1783
-                has_action( 'init', array( $instance, 'start_buffering' ) )
1783
+                has_action('init', array($instance, 'start_buffering'))
1784 1784
             );
1785
-            $this->assertTrue( ! defined( 'AUTOPTIMIZE_HOOK_INTO' ) );
1785
+            $this->assertTrue(!defined('AUTOPTIMIZE_HOOK_INTO'));
1786 1786
         }
1787 1787
 
1788 1788
         /*
@@ -1799,18 +1799,18 @@  discard block
 block discarded – undo
1799 1799
 
1800 1800
     public function test_inline_and_defer_markup()
1801 1801
     {
1802
-        add_filter( 'autoptimize_filter_css_defer', '__return_true' );
1803
-        add_filter( 'autoptimize_filter_css_defer_inline', '__return_true' );
1802
+        add_filter('autoptimize_filter_css_defer', '__return_true');
1803
+        add_filter('autoptimize_filter_css_defer_inline', '__return_true');
1804 1804
 
1805
-        $actual = $this->ao->end_buffering( $this->get_test_markup() );
1806
-        if ( is_multisite() ) {
1807
-            $this->assertEquals( $this->get_test_markup_output_inline_defer_ms(), $actual );
1805
+        $actual = $this->ao->end_buffering($this->get_test_markup());
1806
+        if (is_multisite()) {
1807
+            $this->assertEquals($this->get_test_markup_output_inline_defer_ms(), $actual);
1808 1808
         } else {
1809
-            $this->assertEquals( $this->get_test_markup_output_inline_defer(), $actual );
1809
+            $this->assertEquals($this->get_test_markup_output_inline_defer(), $actual);
1810 1810
         }
1811 1811
 
1812
-        remove_all_filters( 'autoptimize_filter_css_defer' );
1813
-        remove_all_filters( 'autoptimize_filter_css_defer_inline' );
1812
+        remove_all_filters('autoptimize_filter_css_defer');
1813
+        remove_all_filters('autoptimize_filter_css_defer_inline');
1814 1814
     }
1815 1815
 
1816 1816
     public function test_js_aggregation_decision_and_dontaggregate_filter()
@@ -1818,47 +1818,47 @@  discard block
 block discarded – undo
1818 1818
         $opts = $this->getAoScriptsDefaultOptions();
1819 1819
 
1820 1820
         // Aggregating: true by default.
1821
-        $scripts = new autoptimizeScripts( '' );
1822
-        $scripts->read( $opts );
1823
-        $this->assertTrue( $scripts->aggregating() );
1821
+        $scripts = new autoptimizeScripts('');
1822
+        $scripts->read($opts);
1823
+        $this->assertTrue($scripts->aggregating());
1824 1824
 
1825 1825
         // Aggregating: option=true (dontaggregate=false by default).
1826 1826
         $opts['aggregate'] = true;
1827
-        $scripts           = new autoptimizeScripts( '' );
1828
-        $scripts->read( $opts );
1829
-        $this->assertTrue( $scripts->aggregating() );
1827
+        $scripts           = new autoptimizeScripts('');
1828
+        $scripts->read($opts);
1829
+        $this->assertTrue($scripts->aggregating());
1830 1830
 
1831 1831
         // Aggregating: option=true, dontaggregate=false explicit.
1832 1832
         $opts['aggregate'] = true;
1833
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_false' );
1834
-        $scripts = new autoptimizeScripts( '' );
1835
-        $scripts->read( $opts );
1836
-        $this->assertTrue( $scripts->aggregating() );
1837
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1833
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_false');
1834
+        $scripts = new autoptimizeScripts('');
1835
+        $scripts->read($opts);
1836
+        $this->assertTrue($scripts->aggregating());
1837
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1838 1838
 
1839 1839
         // Not aggregating: option=true, dontaggregate=true.
1840 1840
         $opts['aggregate'] = true;
1841
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_true' );
1842
-        $scripts = new autoptimizeScripts( '' );
1843
-        $scripts->read( $opts );
1844
-        $this->assertFalse( $scripts->aggregating() );
1845
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1841
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_true');
1842
+        $scripts = new autoptimizeScripts('');
1843
+        $scripts->read($opts);
1844
+        $this->assertFalse($scripts->aggregating());
1845
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1846 1846
 
1847 1847
         // Not aggregating: option=false, dontaggregate=false.
1848 1848
         $opts['aggregate'] = false;
1849
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_false' );
1850
-        $scripts = new autoptimizeScripts( '' );
1851
-        $scripts->read( $opts );
1852
-        $this->assertFalse( $scripts->aggregating() );
1853
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1849
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_false');
1850
+        $scripts = new autoptimizeScripts('');
1851
+        $scripts->read($opts);
1852
+        $this->assertFalse($scripts->aggregating());
1853
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1854 1854
 
1855 1855
         // Not aggregating: option=false, dontaggregate=true.
1856 1856
         $opts['aggregate'] = false;
1857
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_true' );
1858
-        $scripts = new autoptimizeScripts( '' );
1859
-        $scripts->read( $opts );
1860
-        $this->assertFalse( $scripts->aggregating() );
1861
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1857
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_true');
1858
+        $scripts = new autoptimizeScripts('');
1859
+        $scripts->read($opts);
1860
+        $this->assertFalse($scripts->aggregating());
1861
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1862 1862
     }
1863 1863
 
1864 1864
     public function test_css_aggregation_decision_and_dontaggregate_filter()
@@ -1866,78 +1866,78 @@  discard block
 block discarded – undo
1866 1866
         $opts = $this->getAoStylesDefaultOptions();
1867 1867
 
1868 1868
         // Aggregating: true by default.
1869
-        $styles = new autoptimizeStyles( '' );
1870
-        $this->assertTrue( $styles->aggregating() );
1869
+        $styles = new autoptimizeStyles('');
1870
+        $this->assertTrue($styles->aggregating());
1871 1871
 
1872 1872
         // Aggregating: option=true (dontaggregate=false by default).
1873 1873
         $opts['aggregate'] = true;
1874
-        $styles            = new autoptimizeStyles( '' );
1875
-        $styles->read( $opts );
1876
-        $this->assertTrue( $styles->aggregating() );
1874
+        $styles            = new autoptimizeStyles('');
1875
+        $styles->read($opts);
1876
+        $this->assertTrue($styles->aggregating());
1877 1877
 
1878 1878
         // Aggregating: option=true, dontaggregate=false explicit.
1879 1879
         $opts['aggregate'] = true;
1880
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_false' );
1881
-        $styles = new autoptimizeStyles( '' );
1882
-        $styles->read( $opts );
1883
-        $this->assertTrue( $styles->aggregating() );
1884
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1880
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_false');
1881
+        $styles = new autoptimizeStyles('');
1882
+        $styles->read($opts);
1883
+        $this->assertTrue($styles->aggregating());
1884
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1885 1885
 
1886 1886
         // Not aggregating: option=true, dontaggregate=true.
1887 1887
         $opts['aggregate'] = true;
1888
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_true' );
1889
-        $styles = new autoptimizeStyles( '' );
1890
-        $styles->read( $opts );
1891
-        $this->assertFalse( $styles->aggregating() );
1892
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1888
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_true');
1889
+        $styles = new autoptimizeStyles('');
1890
+        $styles->read($opts);
1891
+        $this->assertFalse($styles->aggregating());
1892
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1893 1893
 
1894 1894
         // Not aggregating: option=false, dontaggregate=false.
1895 1895
         $opts['aggregate'] = false;
1896
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_false' );
1897
-        $styles = new autoptimizeStyles( '' );
1898
-        $styles->read( $opts );
1899
-        $this->assertFalse( $styles->aggregating() );
1900
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1896
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_false');
1897
+        $styles = new autoptimizeStyles('');
1898
+        $styles->read($opts);
1899
+        $this->assertFalse($styles->aggregating());
1900
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1901 1901
 
1902 1902
         // Not aggregating: option=false, dontaggregate=true.
1903 1903
         $opts['aggregate'] = false;
1904
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_true' );
1905
-        $styles = new autoptimizeStyles( '' );
1906
-        $styles->read( $opts );
1907
-        $this->assertFalse( $styles->aggregating() );
1908
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1904
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_true');
1905
+        $styles = new autoptimizeStyles('');
1906
+        $styles->read($opts);
1907
+        $this->assertFalse($styles->aggregating());
1908
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1909 1909
     }
1910 1910
 
1911 1911
     public function test_css_minify_single_with_cdning()
1912 1912
     {
1913
-        $pathname = dirname( __FILE__ ) . '/fixtures/minify-single.css';
1913
+        $pathname = dirname(__FILE__).'/fixtures/minify-single.css';
1914 1914
         $opts     = $this->getAoStylesDefaultOptions();
1915
-        $styles   = new autoptimizeStyles( '' );
1916
-        $styles->read( $opts );
1915
+        $styles   = new autoptimizeStyles('');
1916
+        $styles->read($opts);
1917 1917
 
1918
-        $url = $styles->minify_single( $pathname, $cache_miss = true );
1918
+        $url = $styles->minify_single($pathname, $cache_miss = true);
1919 1919
 
1920 1920
         // Minified url filename + its pointed to cdn.
1921
-        $this->assertContains( AUTOPTIMIZE_CACHE_CHILD_DIR, $url );
1922
-        $this->assertContains( '/autoptimize_single_', $url );
1923
-        $this->assertContains( $styles->cdn_url, $url );
1921
+        $this->assertContains(AUTOPTIMIZE_CACHE_CHILD_DIR, $url);
1922
+        $this->assertContains('/autoptimize_single_', $url);
1923
+        $this->assertContains($styles->cdn_url, $url);
1924 1924
 
1925 1925
         // Actual minified css contents are minified and cdn-ed.
1926
-        $path     = $styles->getpath( $url );
1927
-        $contents = file_get_contents( $path );
1928
-        $this->assertContains( $styles->cdn_url, $contents );
1929
-        $this->assertContains( '.bg{background:url(' . $styles->cdn_url, $contents );
1926
+        $path     = $styles->getpath($url);
1927
+        $contents = file_get_contents($path);
1928
+        $this->assertContains($styles->cdn_url, $contents);
1929
+        $this->assertContains('.bg{background:url('.$styles->cdn_url, $contents);
1930 1930
     }
1931 1931
 
1932 1932
     public function test_ao_partners_instantiation_without_explicit_include()
1933 1933
     {
1934 1934
         $partners = new autoptimizePartners();
1935
-        $this->assertTrue( $partners instanceof autoptimizePartners );
1935
+        $this->assertTrue($partners instanceof autoptimizePartners);
1936 1936
     }
1937 1937
 
1938 1938
     public function test_html_minify_keep_html_comments_inside_script_blocks()
1939 1939
     {
1940
-        $markup   = <<<MARKUP
1940
+        $markup = <<<MARKUP
1941 1941
 <script>
1942 1942
 <!-- End Support AJAX add to cart -->
1943 1943
 var a = "b";
@@ -1967,23 +1967,23 @@  discard block
 block discarded – undo
1967 1967
             ],
1968 1968
         ];
1969 1969
 
1970
-        $instance = new autoptimizeHTML( $markup );
1971
-        $instance->read( $options['autoptimizeHTML'] );
1970
+        $instance = new autoptimizeHTML($markup);
1971
+        $instance->read($options['autoptimizeHTML']);
1972 1972
         $instance->minify();
1973 1973
         $actual = $instance->getcontent();
1974
-        $this->assertEquals( $expected, $actual );
1974
+        $this->assertEquals($expected, $actual);
1975 1975
 
1976
-        $instance = new autoptimizeHTML( $markup2 );
1977
-        $instance->read( $options['autoptimizeHTML'] );
1976
+        $instance = new autoptimizeHTML($markup2);
1977
+        $instance->read($options['autoptimizeHTML']);
1978 1978
         $instance->minify();
1979 1979
         $actual2 = $instance->getcontent();
1980
-        $this->assertEquals( $expected2, $actual2 );
1980
+        $this->assertEquals($expected2, $actual2);
1981 1981
     }
1982 1982
 
1983 1983
     public function test_html_minify_remove_html_comments_inside_script_blocks()
1984 1984
     {
1985 1985
         // Default case, html comments removed (keepcomments = false).
1986
-        $markup1   = <<<MARKUP
1986
+        $markup1 = <<<MARKUP
1987 1987
 <script>
1988 1988
 var a = "b";
1989 1989
 <!-- End Support AJAX add to cart -->
@@ -1994,7 +1994,7 @@  discard block
 block discarded – undo
1994 1994
 <!-- End Support AJAX add to cart</script>
1995 1995
 MARKUP;
1996 1996
 
1997
-        $markup2   = <<<MARKUP
1997
+        $markup2 = <<<MARKUP
1998 1998
 <script>
1999 1999
 <!-- End Support AJAX add to cart -->
2000 2000
 var a = "b";
@@ -2011,17 +2011,17 @@  discard block
 block discarded – undo
2011 2011
             ],
2012 2012
         ];
2013 2013
 
2014
-        $instance = new autoptimizeHTML( $markup1 );
2015
-        $instance->read( $options['autoptimizeHTML'] );
2014
+        $instance = new autoptimizeHTML($markup1);
2015
+        $instance->read($options['autoptimizeHTML']);
2016 2016
         $instance->minify();
2017 2017
         $actual = $instance->getcontent();
2018
-        $this->assertEquals( $expected1, $actual );
2018
+        $this->assertEquals($expected1, $actual);
2019 2019
 
2020
-        $instance = new autoptimizeHTML( $markup2 );
2021
-        $instance->read( $options['autoptimizeHTML'] );
2020
+        $instance = new autoptimizeHTML($markup2);
2021
+        $instance->read($options['autoptimizeHTML']);
2022 2022
         $instance->minify();
2023 2023
         $actual2 = $instance->getcontent();
2024
-        $this->assertEquals( $expected2, $actual2 );
2024
+        $this->assertEquals($expected2, $actual2);
2025 2025
     }
2026 2026
 
2027 2027
     public function test_html_minify_html_comments_inside_script_blocks_old_school_pattern()
@@ -2045,11 +2045,11 @@  discard block
 block discarded – undo
2045 2045
             ],
2046 2046
         ];
2047 2047
 
2048
-        $instance = new autoptimizeHTML( $markup );
2049
-        $instance->read( $options['autoptimizeHTML'] );
2048
+        $instance = new autoptimizeHTML($markup);
2049
+        $instance->read($options['autoptimizeHTML']);
2050 2050
         $instance->minify();
2051 2051
         $actual = $instance->getcontent();
2052
-        $this->assertEquals( $expected, $actual );
2052
+        $this->assertEquals($expected, $actual);
2053 2053
     }
2054 2054
 
2055 2055
     public function test_html_minify_html_comments_inside_script_blocks_old_school_pattern_untouched()
@@ -2074,49 +2074,49 @@  discard block
 block discarded – undo
2074 2074
             ],
2075 2075
         ];
2076 2076
 
2077
-        $instance = new autoptimizeHTML( $markup );
2078
-        $instance->read( $options['autoptimizeHTML'] );
2077
+        $instance = new autoptimizeHTML($markup);
2078
+        $instance->read($options['autoptimizeHTML']);
2079 2079
         $instance->minify();
2080 2080
         $actual = $instance->getcontent();
2081
-        $this->assertEquals( $expected, $actual );
2081
+        $this->assertEquals($expected, $actual);
2082 2082
     }
2083 2083
 
2084 2084
     public function test_utils_mbstring_availabilty_overriding()
2085 2085
     {
2086 2086
         $orig     = autoptimizeUtils::mbstring_available();
2087
-        $opposite = ! $orig;
2087
+        $opposite = !$orig;
2088 2088
 
2089
-        $this->assertSame( $orig, autoptimizeUtils::mbstring_available() );
2089
+        $this->assertSame($orig, autoptimizeUtils::mbstring_available());
2090 2090
         // Override works...
2091
-        $this->assertSame( $opposite, autoptimizeUtils::mbstring_available( $opposite ) );
2091
+        $this->assertSame($opposite, autoptimizeUtils::mbstring_available($opposite));
2092 2092
         // And override remains cached as the last version.
2093
-        $this->assertSame( $opposite, autoptimizeUtils::mbstring_available() );
2093
+        $this->assertSame($opposite, autoptimizeUtils::mbstring_available());
2094 2094
     }
2095 2095
 
2096 2096
     public function test_utils_mbstring_basics()
2097 2097
     {
2098 2098
         // Turn on mbstring usage.
2099
-        autoptimizeUtils::mbstring_available( true );
2099
+        autoptimizeUtils::mbstring_available(true);
2100 2100
 
2101
-        $this->assertSame( 2, autoptimizeUtils::strlen( "\x00\xFF", 'ASCII' ) );
2102
-        $this->assertSame( 2, autoptimizeUtils::strlen( "\x00\xFF", 'CP850' ) );
2103
-        $this->assertSame( 3, autoptimizeUtils::strlen( '한국어' ) );
2101
+        $this->assertSame(2, autoptimizeUtils::strlen("\x00\xFF", 'ASCII'));
2102
+        $this->assertSame(2, autoptimizeUtils::strlen("\x00\xFF", 'CP850'));
2103
+        $this->assertSame(3, autoptimizeUtils::strlen('한국어'));
2104 2104
 
2105
-        $this->assertFalse( @autoptimizeUtils::strpos( 'abc', '' ) );
2106
-        $this->assertFalse( @autoptimizeUtils::strpos( 'abc', 'a', -1 ) );
2107
-        $this->assertFalse( autoptimizeUtils::strpos( 'abc', 'd' ) );
2108
-        $this->assertFalse( autoptimizeUtils::strpos( 'abc', 'a', 3 ) );
2109
-        $this->assertSame( 1, autoptimizeUtils::strpos( '한국어', '국' ) );
2105
+        $this->assertFalse(@autoptimizeUtils::strpos('abc', ''));
2106
+        $this->assertFalse(@autoptimizeUtils::strpos('abc', 'a', -1));
2107
+        $this->assertFalse(autoptimizeUtils::strpos('abc', 'd'));
2108
+        $this->assertFalse(autoptimizeUtils::strpos('abc', 'a', 3));
2109
+        $this->assertSame(1, autoptimizeUtils::strpos('한국어', '국'));
2110 2110
     }
2111 2111
 
2112 2112
     /**
2113 2113
      * @dataProvider provider_utils_substr_replace
2114 2114
      */
2115
-    function test_utils_substr_replace_basics_mbstring( $s, $repl, $start, $len, $expected )
2115
+    function test_utils_substr_replace_basics_mbstring($s, $repl, $start, $len, $expected)
2116 2116
     {
2117 2117
         // Force mbstring code path...
2118
-        autoptimizeUtils::mbstring_available( true );
2119
-        $this->assertEquals( $expected, autoptimizeUtils::substr_replace( $s, $repl, $start, $len ) );
2118
+        autoptimizeUtils::mbstring_available(true);
2119
+        $this->assertEquals($expected, autoptimizeUtils::substr_replace($s, $repl, $start, $len));
2120 2120
     }
2121 2121
 
2122 2122
     public function provider_utils_substr_replace()
@@ -2158,48 +2158,48 @@  discard block
 block discarded – undo
2158 2158
 
2159 2159
     function test_mb_substr_replace_with_ascii_input_string()
2160 2160
     {
2161
-        autoptimizeUtils::mbstring_available( false );
2161
+        autoptimizeUtils::mbstring_available(false);
2162 2162
 
2163 2163
         $str = 'Ascii';
2164 2164
 
2165
-        $this->assertSame( 'Añ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) );
2166
-        $this->assertSame( 'ñcii', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2167
-        $this->assertSame( 'Asñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2168
-        $this->assertSame( 'Asz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) );
2169
-        $this->assertSame( 'Añii', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2165
+        $this->assertSame('Añ', autoptimizeUtils::substr_replace($str, 'ñ', 1));
2166
+        $this->assertSame('ñcii', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2167
+        $this->assertSame('Asñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2168
+        $this->assertSame('Asz', autoptimizeUtils::substr_replace($str, 'z', 2, 10));
2169
+        $this->assertSame('Añii', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2170 2170
     }
2171 2171
 
2172 2172
     function test_mb_substr_replace_with_utf8_input_string()
2173 2173
     {
2174
-        autoptimizeUtils::mbstring_available( true );
2174
+        autoptimizeUtils::mbstring_available(true);
2175 2175
 
2176 2176
         $str = 'âønæë';
2177 2177
 
2178
-        $this->assertSame( 'âñ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) ); // No length.
2179
-        $this->assertSame( 'ñnæë', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2180
-        $this->assertSame( 'âøñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2181
-        $this->assertSame( 'âøz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) ); // Length larger than possible...
2182
-        $this->assertSame( 'âñæë', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2178
+        $this->assertSame('âñ', autoptimizeUtils::substr_replace($str, 'ñ', 1)); // No length.
2179
+        $this->assertSame('ñnæë', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2180
+        $this->assertSame('âøñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2181
+        $this->assertSame('âøz', autoptimizeUtils::substr_replace($str, 'z', 2, 10)); // Length larger than possible...
2182
+        $this->assertSame('âñæë', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2183 2183
     }
2184 2184
 
2185 2185
     function test_default_substr_replace_with_ascii_input_string()
2186 2186
     {
2187 2187
         // Disable mbstring which should fall ack to substr_replace...
2188
-        autoptimizeUtils::mbstring_available( false );
2188
+        autoptimizeUtils::mbstring_available(false);
2189 2189
 
2190 2190
         $str = 'Ascii';
2191 2191
 
2192
-        $this->assertSame( 'Añ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) );
2193
-        $this->assertSame( 'ñcii', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2194
-        $this->assertSame( 'Asñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2195
-        $this->assertSame( 'Asz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) );
2196
-        $this->assertSame( 'Añii', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2192
+        $this->assertSame('Añ', autoptimizeUtils::substr_replace($str, 'ñ', 1));
2193
+        $this->assertSame('ñcii', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2194
+        $this->assertSame('Asñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2195
+        $this->assertSame('Asz', autoptimizeUtils::substr_replace($str, 'z', 2, 10));
2196
+        $this->assertSame('Añii', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2197 2197
     }
2198 2198
 
2199 2199
     function test_default_substr_replace_with_utf8_input_string()
2200 2200
     {
2201 2201
         // Disabling mbstring, falling back to substr_replace...
2202
-        autoptimizeUtils::mbstring_available( false );
2202
+        autoptimizeUtils::mbstring_available(false);
2203 2203
 
2204 2204
         // This is really impossible to make work properly, since
2205 2205
         // any start/len parameters we give are working with bytes instead
@@ -2208,55 +2208,55 @@  discard block
 block discarded – undo
2208 2208
 
2209 2209
         // $this->assertSame( '�ñ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) ); // No length.
2210 2210
         // $this->assertSame( 'ñ�næë', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2211
-        $this->assertSame( 'ñønæë', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2212
-        $this->assertSame( 'âñxæë', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2213
-        $this->assertSame( 'âz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) ); // Length larger than possible...
2211
+        $this->assertSame('ñønæë', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2212
+        $this->assertSame('âñxæë', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2213
+        $this->assertSame('âz', autoptimizeUtils::substr_replace($str, 'z', 2, 10)); // Length larger than possible...
2214 2214
     }
2215 2215
 
2216 2216
     public function test_cache_fast_delete()
2217 2217
     {
2218
-        add_filter( 'autoptimize_filter_cache_clear_advanced', '__return_true' );
2218
+        add_filter('autoptimize_filter_cache_clear_advanced', '__return_true');
2219 2219
 
2220 2220
         autoptimizeCache::clearall_actionless();
2221 2221
 
2222
-        remove_all_filters( 'autoptimize_filter_cache_clear_advanced' );
2222
+        remove_all_filters('autoptimize_filter_cache_clear_advanced');
2223 2223
     }
2224 2224
 
2225 2225
     public function test_delete_advanced_cache_clear_artifacts()
2226 2226
     {
2227
-        $this->assertTrue( autoptimizeCache::delete_advanced_cache_clear_artifacts() );
2227
+        $this->assertTrue(autoptimizeCache::delete_advanced_cache_clear_artifacts());
2228 2228
     }
2229 2229
 
2230 2230
     public function provider_canonicalization()
2231 2231
     {
2232 2232
         return array(
2233
-            array( '../common', 'common' ),
2234
-            array( '../what-does-this-mean/really?/', 'what-does-this-mean/really?/' ),
2235
-            array( '../../what/where/how', 'what/where/how' ),
2236
-            array( '/../more.dots.please/', '/more.dots.please/' ),
2237
-            array( '/../../what/where/how', '/what/where/how' ),
2238
-            array( '/a/b/c/../../../d/e/file.txt', '/d/e/file.txt' ),
2239
-            array( 'a/b/../c', 'a/c' ),
2240
-            array( './../../etc/passwd', './etc/passwd' ),
2241
-            array( '/var/.////./user/./././..//.//../////../././.././test/////', '/test/' ),
2242
-            array( '/var/user/./././.././../.././././test/', '/test/' ),
2243
-            array( '/hello/0//how/../are/../you', '/hello/0/you' ),
2244
-            array( '', '' ),
2245
-            array( '.', '.' ),
2246
-            array( '..', '..' ),
2247
-            array( './..', './..' ),
2248
-            array( '../.', '.' ),
2233
+            array('../common', 'common'),
2234
+            array('../what-does-this-mean/really?/', 'what-does-this-mean/really?/'),
2235
+            array('../../what/where/how', 'what/where/how'),
2236
+            array('/../more.dots.please/', '/more.dots.please/'),
2237
+            array('/../../what/where/how', '/what/where/how'),
2238
+            array('/a/b/c/../../../d/e/file.txt', '/d/e/file.txt'),
2239
+            array('a/b/../c', 'a/c'),
2240
+            array('./../../etc/passwd', './etc/passwd'),
2241
+            array('/var/.////./user/./././..//.//../////../././.././test/////', '/test/'),
2242
+            array('/var/user/./././.././../.././././test/', '/test/'),
2243
+            array('/hello/0//how/../are/../you', '/hello/0/you'),
2244
+            array('', ''),
2245
+            array('.', '.'),
2246
+            array('..', '..'),
2247
+            array('./..', './..'),
2248
+            array('../.', '.'),
2249 2249
             // This might be debatable...
2250
-            array( '../..', '..' ),
2250
+            array('../..', '..'),
2251 2251
         );
2252 2252
     }
2253 2253
 
2254 2254
     /**
2255 2255
      * @dataProvider provider_canonicalization
2256 2256
      */
2257
-    public function test_path_canonicalize( $path, $canonicalized )
2257
+    public function test_path_canonicalize($path, $canonicalized)
2258 2258
     {
2259
-        $this->assertSame( $canonicalized, autoptimizeUtils::path_canonicalize( $path ) );
2259
+        $this->assertSame($canonicalized, autoptimizeUtils::path_canonicalize($path));
2260 2260
     }
2261 2261
 
2262 2262
     /**
@@ -2270,16 +2270,16 @@  discard block
 block discarded – undo
2270 2270
     public function test_autoptimize_filter_cdn_magic_path_check()
2271 2271
     {
2272 2272
         $cdnurl = 'http://cdn.example.org';
2273
-        add_filter( 'autoptimize_filter_cdn_magic_path_check', '__return_false', 10, 2 );
2273
+        add_filter('autoptimize_filter_cdn_magic_path_check', '__return_false', 10, 2);
2274 2274
 
2275 2275
         // Even when site is in a subfolder, the resulting cdn-rewritten url
2276 2276
         // should not magically include it, due to the above filter.
2277 2277
         // The second parameter is here to force a cache miss and re-run
2278 2278
         // the filter since we're using the same cdn url all over the place,
2279 2279
         // but want to prove different things with it.
2280
-        $this->assertSame( $cdnurl, autoptimizeUtils::tweak_cdn_url_if_needed( $cdnurl, true ) );
2280
+        $this->assertSame($cdnurl, autoptimizeUtils::tweak_cdn_url_if_needed($cdnurl, true));
2281 2281
 
2282
-        remove_all_filters( 'autoptimize_filter_cdn_magic_path_check' );
2282
+        remove_all_filters('autoptimize_filter_cdn_magic_path_check');
2283 2283
     }
2284 2284
 
2285 2285
     /**
@@ -2302,8 +2302,8 @@  discard block
 block discarded – undo
2302 2302
 MARKUP;
2303 2303
 
2304 2304
         $instance = new autoptimizeExtra();
2305
-        $actual = $instance->filter_optimize_images( $markup );
2306
-        $this->assertEquals( $expected, $actual );
2305
+        $actual = $instance->filter_optimize_images($markup);
2306
+        $this->assertEquals($expected, $actual);
2307 2307
     }
2308 2308
 
2309 2309
     /**
@@ -2324,8 +2324,8 @@  discard block
 block discarded – undo
2324 2324
 MARKUP;
2325 2325
 
2326 2326
         $instance = new autoptimizeExtra();
2327
-        $actual = $instance->filter_optimize_images( $markup );
2328
-        $this->assertEquals( $expected, $actual );
2327
+        $actual = $instance->filter_optimize_images($markup);
2328
+        $this->assertEquals($expected, $actual);
2329 2329
     }
2330 2330
 
2331 2331
     /**
@@ -2348,8 +2348,8 @@  discard block
 block discarded – undo
2348 2348
 MARKUP;
2349 2349
 
2350 2350
         $instance = new autoptimizeExtra();
2351
-        $actual = $instance->filter_optimize_images( $markup );
2352
-        $this->assertEquals( $expected, $actual );
2351
+        $actual = $instance->filter_optimize_images($markup);
2352
+        $this->assertEquals($expected, $actual);
2353 2353
     }
2354 2354
 
2355 2355
     /**
@@ -2359,7 +2359,7 @@  discard block
 block discarded – undo
2359 2359
     {
2360 2360
         $actual = autoptimizeUtils::str_is_valid_regex($str);
2361 2361
 
2362
-        $this->assertEquals( $expected, $actual );
2362
+        $this->assertEquals($expected, $actual);
2363 2363
     }
2364 2364
 
2365 2365
     public function provider_str_is_valid_regex()
@@ -2375,11 +2375,11 @@  discard block
 block discarded – undo
2375 2375
     /**
2376 2376
      * @dataProvider provider_protocol_relative_tests
2377 2377
      */
2378
-    public function test_is_protocol_relative( $str, $expected )
2378
+    public function test_is_protocol_relative($str, $expected)
2379 2379
     {
2380
-        $actual = autoptimizeUtils::is_protocol_relative( $str );
2380
+        $actual = autoptimizeUtils::is_protocol_relative($str);
2381 2381
 
2382
-        $this->assertEquals( $expected, $actual );
2382
+        $this->assertEquals($expected, $actual);
2383 2383
     }
2384 2384
 
2385 2385
     public function provider_protocol_relative_tests()
@@ -2406,31 +2406,31 @@  discard block
 block discarded – undo
2406 2406
 
2407 2407
         $expected = 'img.centered,.aligncenter{-webkit-box-shadow:none;-moz-box-shadow:none;box-shadow:none}';
2408 2408
 
2409
-        $instance = new autoptimizeStyles( $css );
2410
-        $minified = $instance->run_minifier_on( $css );
2409
+        $instance = new autoptimizeStyles($css);
2410
+        $minified = $instance->run_minifier_on($css);
2411 2411
 
2412
-        $this->assertEquals( $expected, $minified );
2412
+        $this->assertEquals($expected, $minified);
2413 2413
     }
2414 2414
 
2415 2415
     public function test_css_font_names_mangling()
2416 2416
     {
2417 2417
         // Properly quoted font names dont get mangled.
2418 2418
         $css      = 'h2{font-family:"Archivo Black"}';
2419
-        $instance = new autoptimizeStyles( $css );
2420
-        $actual   = $instance->run_minifier_on( $css );
2421
-        $this->assertEquals( $css, $actual );
2419
+        $instance = new autoptimizeStyles($css);
2420
+        $actual   = $instance->run_minifier_on($css);
2421
+        $this->assertEquals($css, $actual);
2422 2422
 
2423 2423
         // When not quoted, 'Black' used to become '#000', but not anymore... :)
2424 2424
         $css_unquoted = 'h2{font-family:Archivo Black;}';
2425 2425
         $expected     = 'h2{font-family:Archivo Black}';
2426
-        $instance     = new autoptimizeStyles( $css_unquoted );
2427
-        $actual       = $instance->run_minifier_on( $css_unquoted );
2428
-        $this->assertEquals( $expected, $actual );
2426
+        $instance     = new autoptimizeStyles($css_unquoted);
2427
+        $actual       = $instance->run_minifier_on($css_unquoted);
2428
+        $this->assertEquals($expected, $actual);
2429 2429
 
2430 2430
         $css_unquoted = 'h1{font:italic 1.2em Fira White,serif}';
2431
-        $instance     = new autoptimizeStyles( $css_unquoted );
2432
-        $actual       = $instance->run_minifier_on( $css_unquoted );
2433
-        $this->assertEquals( $css_unquoted, $actual );
2431
+        $instance     = new autoptimizeStyles($css_unquoted);
2432
+        $actual       = $instance->run_minifier_on($css_unquoted);
2433
+        $this->assertEquals($css_unquoted, $actual);
2434 2434
     }
2435 2435
 
2436 2436
     public function test_jsminphp_string_literal_minification()
@@ -2444,7 +2444,7 @@  discard block
 block discarded – undo
2444 2444
         $expected = '`line
2445 2445
 break`+`he  llo`;foo`hel( \'\');lo`;`he\nl\`lo`;(`he${one + two}`)';
2446 2446
 
2447
-        $actual = JSMin::minify( $js );
2448
-        $this->assertEquals( $expected, $actual );
2447
+        $actual = JSMin::minify($js);
2448
+        $this->assertEquals($expected, $actual);
2449 2449
     }
2450 2450
 }
Please login to merge, or discard this patch.
classes/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.