Completed
Push — master ( 2ef6f7...6fcb0f )
by frank
01:45
created
classes/external/php/jsmin.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
     protected $inputLength = 0;
71 71
     protected $lookAhead   = null;
72 72
     protected $output      = '';
73
-    protected $lastByteOut  = '';
73
+    protected $lastByteOut = '';
74 74
     protected $keptComment = '';
75 75
 
76 76
     /**
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
         }
107 107
 
108 108
         $mbIntEnc = null;
109
-        if (function_exists('mb_strlen') && ((int)ini_get('mbstring.func_overload') & 2)) {
109
+        if (function_exists('mb_strlen') && ((int) ini_get('mbstring.func_overload') & 2)) {
110 110
             $mbIntEnc = mb_internal_encoding();
111 111
             mb_internal_encoding('8bit');
112 112
         }
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
                         && ($this->b === $this->lastByteOut)) {
129 129
                     // Don't delete this space. If we do, the addition/subtraction
130 130
                     // could be parsed as a post-increment
131
-                } elseif (! $this->isAlphaNum($this->b)) {
131
+                } elseif (!$this->isAlphaNum($this->b)) {
132 132
                     $command = self::ACTION_DELETE_A;
133 133
                 }
134 134
             } elseif ($this->a === "\n") {
@@ -139,10 +139,10 @@  discard block
 block discarded – undo
139 139
                     // otherwise mb_strpos will give WARNING
140 140
                 } elseif ($this->b === null
141 141
                           || (false === strpos('{[(+-!~', $this->b)
142
-                              && ! $this->isAlphaNum($this->b))) {
142
+                              && !$this->isAlphaNum($this->b))) {
143 143
                     $command = self::ACTION_DELETE_A;
144 144
                 }
145
-            } elseif (! $this->isAlphaNum($this->a)) {
145
+            } elseif (!$this->isAlphaNum($this->a)) {
146 146
                 if ($this->b === ' '
147 147
                     || ($this->b === "\n"
148 148
                         && (false === strpos('}])+-"\'', $this->a)))) {
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
                 if ($this->a === "'" || $this->a === '"' || $this->a === '`') { // string/template literal
200 200
                     $delimiter = $this->a;
201 201
                     $str = $this->a; // in case needed for exception
202
-                    for(;;) {
202
+                    for (;;) {
203 203
                         $this->output .= $this->a;
204 204
                         $this->lastByteOut = $this->a;
205 205
 
@@ -229,13 +229,13 @@  discard block
 block discarded – undo
229 229
             case self::ACTION_DELETE_A_B: // 3
230 230
                 $this->b = $this->next();
231 231
                 if ($this->b === '/' && $this->isRegexpLiteral()) {
232
-                    $this->output .= $this->a . $this->b;
232
+                    $this->output .= $this->a.$this->b;
233 233
                     $pattern = '/'; // keep entire pattern in case we need to report it in the exception
234
-                    for(;;) {
234
+                    for (;;) {
235 235
                         $this->a = $this->get();
236 236
                         $pattern .= $this->a;
237 237
                         if ($this->a === '[') {
238
-                            for(;;) {
238
+                            for (;;) {
239 239
                                 $this->output .= $this->a;
240 240
                                 $this->a = $this->get();
241 241
                                 $pattern .= $this->a;
@@ -250,7 +250,7 @@  discard block
 block discarded – undo
250 250
                                 if ($this->isEOF($this->a)) {
251 251
                                     throw new JSMin_UnterminatedRegExpException(
252 252
                                         "JSMin: Unterminated set in RegExp at byte "
253
-                                            . $this->inputIndex .": {$pattern}");
253
+                                            . $this->inputIndex.": {$pattern}");
254 254
                                 }
255 255
                             }
256 256
                         }
@@ -295,7 +295,7 @@  discard block
 block discarded – undo
295 295
 
296 296
         // if the "/" follows a keyword, it must be a regexp, otherwise it's best to assume division
297 297
 
298
-        $subject = $this->output . trim($this->a);
298
+        $subject = $this->output.trim($this->a);
299 299
         if (!preg_match('/(?:case|else|in|return|typeof)$/', $subject, $m)) {
300 300
             // not a keyword
301 301
             return false;
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
     {
407 407
         $this->get();
408 408
         $comment = '';
409
-        for(;;) {
409
+        for (;;) {
410 410
             $get = $this->get();
411 411
             if ($get === '*') {
412 412
                 if ($this->peek() === '/') { // end of comment reached
@@ -417,7 +417,7 @@  discard block
 block discarded – undo
417 417
                             // don't prepend a newline if two comments right after one another
418 418
                             $this->keptComment = "\n";
419 419
                         }
420
-                        $this->keptComment .= "/*!" . substr($comment, 1) . "*/\n";
420
+                        $this->keptComment .= "/*!".substr($comment, 1)."*/\n";
421 421
                     } else if (preg_match('/^@(?:cc_on|if|elif|else|end)\\b/', $comment)) {
422 422
                         // IE conditional
423 423
                         $this->keptComment .= "/*{$comment}*/";
Please login to merge, or discard this patch.
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,82 +70,82 @@  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' ),
134
+        wp_localize_script('autoptimize-toolbar', 'autoptimize_ajax_object', array(
135
+            'ajaxurl'     => admin_url('admin-ajax.php'),
136 136
             // translators: links to the Autoptimize settings page.
137
-            '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;"' ),
138
-            'dismiss_msg' => __( 'Dismiss this notice.' ),
139
-            'nonce'       => wp_create_nonce( 'ao_delcache_nonce' ),
140
-        ) );
137
+            '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;"'),
138
+            'dismiss_msg' => __('Dismiss this notice.'),
139
+            'nonce'       => wp_create_nonce('ao_delcache_nonce'),
140
+        ));
141 141
     }
142 142
 
143
-    public function format_filesize( $bytes, $decimals = 2 )
143
+    public function format_filesize($bytes, $decimals = 2)
144 144
     {
145
-        $units = array( 'B', 'KB', 'MB', 'GB', 'TB', 'PB', 'EB', 'ZB', 'YB' );
145
+        $units = array('B', 'KB', 'MB', 'GB', 'TB', 'PB', 'EB', 'ZB', 'YB');
146 146
 
147
-        for ( $i = 0; ( $bytes / 1024) > 0.9; $i++, $bytes /= 1024 ) {} // @codingStandardsIgnoreLine
147
+        for ($i = 0; ($bytes/1024) > 0.9; $i++, $bytes /= 1024) {} // @codingStandardsIgnoreLine
148 148
 
149
-        return sprintf( "%1.{$decimals}f %s", round( $bytes, $decimals ), $units[ $i ] );
149
+        return sprintf("%1.{$decimals}f %s", round($bytes, $decimals), $units[$i]);
150 150
     }
151 151
 }
Please login to merge, or discard this patch.
classes/autoptimizeVersionUpdatesHandler.php 1 patch
Spacing   +62 added lines, -62 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * Handles version updates and should only be instantiated in autoptimize.php if/when needed.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
@@ -16,9 +16,9 @@  discard block
 block discarded – undo
16 16
      */
17 17
     protected $current_major_version = null;
18 18
 
19
-    public function __construct( $current_version )
19
+    public function __construct($current_version)
20 20
     {
21
-        $this->current_major_version = substr( $current_version, 0, 3 );
21
+        $this->current_major_version = substr($current_version, 0, 3);
22 22
     }
23 23
 
24 24
     /**
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
     {
30 30
         $major_update = false;
31 31
 
32
-        switch ( $this->current_major_version ) {
32
+        switch ($this->current_major_version) {
33 33
             case '1.6':
34 34
                 $this->upgrade_from_1_6();
35 35
                 $major_update = true;
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
                 $major_update = true;
48 48
                 // No break, intentionally, so all upgrades are ran during a single request...
49 49
             case '2.4':
50
-                if ( autoptimizeOptionWrapper::get_option( 'autoptimize_version', 'none' ) == '2.4.2' ) {
50
+                if (autoptimizeOptionWrapper::get_option('autoptimize_version', 'none') == '2.4.2') {
51 51
                     $this->upgrade_from_2_4_2();
52 52
                 }
53 53
                 $this->upgrade_from_2_4();
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
                 // No break, intentionally, so all upgrades are ran during a single request...
56 56
         }
57 57
 
58
-        if ( true === $major_update ) {
58
+        if (true === $major_update) {
59 59
             $this->on_major_version_update();
60 60
         }
61 61
     }
@@ -67,19 +67,19 @@  discard block
 block discarded – undo
67 67
      *
68 68
      * @param string $target Target version to check against (ie., the currently running one).
69 69
      */
70
-    public static function check_installed_and_update( $target )
70
+    public static function check_installed_and_update($target)
71 71
     {
72
-        $db_version = autoptimizeOptionWrapper::get_option( 'autoptimize_version', 'none' );
73
-        if ( $db_version !== $target ) {
74
-            if ( 'none' === $db_version ) {
75
-                add_action( 'admin_notices', 'autoptimizeMain::notice_installed' );
72
+        $db_version = autoptimizeOptionWrapper::get_option('autoptimize_version', 'none');
73
+        if ($db_version !== $target) {
74
+            if ('none' === $db_version) {
75
+                add_action('admin_notices', 'autoptimizeMain::notice_installed');
76 76
             } else {
77
-                $updater = new self( $db_version );
77
+                $updater = new self($db_version);
78 78
                 $updater->run_needed_major_upgrades();
79 79
             }
80 80
 
81 81
             // Versions differed, upgrades happened if needed, store the new version.
82
-            autoptimizeOptionWrapper::update_option( 'autoptimize_version', $target );
82
+            autoptimizeOptionWrapper::update_option('autoptimize_version', $target);
83 83
         }
84 84
     }
85 85
 
@@ -90,10 +90,10 @@  discard block
 block discarded – undo
90 90
     protected function on_major_version_update()
91 91
     {
92 92
         // The transients guard here prevents stale object caches from busting the cache on every request.
93
-        if ( false == get_transient( 'autoptimize_stale_option_buster' ) ) {
94
-            set_transient( 'autoptimize_stale_option_buster', 'Mamsie & Liessie zehhe: ZWIJH!', HOUR_IN_SECONDS );
93
+        if (false == get_transient('autoptimize_stale_option_buster')) {
94
+            set_transient('autoptimize_stale_option_buster', 'Mamsie & Liessie zehhe: ZWIJH!', HOUR_IN_SECONDS);
95 95
             autoptimizeCache::clearall();
96
-            add_action( 'admin_notices', 'autoptimizeMain::notice_updated' );
96
+            add_action('admin_notices', 'autoptimizeMain::notice_updated');
97 97
         }
98 98
     }
99 99
 
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
     private function upgrade_from_1_6()
104 104
     {
105 105
         // If user was on version 1.6.x, force advanced options to be shown by default.
106
-        autoptimizeOptionWrapper::update_option( 'autoptimize_show_adv', '1' );
106
+        autoptimizeOptionWrapper::update_option('autoptimize_show_adv', '1');
107 107
 
108 108
         // And remove old options.
109 109
         $to_delete_options = array(
@@ -115,8 +115,8 @@  discard block
 block discarded – undo
115 115
             'autoptimize_cdn_img_url',
116 116
             'autoptimize_css_yui',
117 117
         );
118
-        foreach ( $to_delete_options as $del_opt ) {
119
-            delete_option( $del_opt );
118
+        foreach ($to_delete_options as $del_opt) {
119
+            delete_option($del_opt);
120 120
         }
121 121
     }
122 122
 
@@ -127,29 +127,29 @@  discard block
 block discarded – undo
127 127
      */
128 128
     private function upgrade_from_1_7()
129 129
     {
130
-        if ( ! is_multisite() ) {
131
-            $css_exclude = autoptimizeOptionWrapper::get_option( 'autoptimize_css_exclude' );
132
-            if ( empty( $css_exclude ) ) {
130
+        if (!is_multisite()) {
131
+            $css_exclude = autoptimizeOptionWrapper::get_option('autoptimize_css_exclude');
132
+            if (empty($css_exclude)) {
133 133
                 $css_exclude = 'admin-bar.min.css, dashicons.min.css';
134
-            } elseif ( false === strpos( $css_exclude, 'dashicons.min.css' ) ) {
134
+            } elseif (false === strpos($css_exclude, 'dashicons.min.css')) {
135 135
                 $css_exclude .= ', dashicons.min.css';
136 136
             }
137
-            autoptimizeOptionWrapper::update_option( 'autoptimize_css_exclude', $css_exclude );
137
+            autoptimizeOptionWrapper::update_option('autoptimize_css_exclude', $css_exclude);
138 138
         } else {
139 139
             global $wpdb;
140
-            $blog_ids         = $wpdb->get_col( "SELECT blog_id FROM $wpdb->blogs" );
140
+            $blog_ids         = $wpdb->get_col("SELECT blog_id FROM $wpdb->blogs");
141 141
             $original_blog_id = get_current_blog_id();
142
-            foreach ( $blog_ids as $blog_id ) {
143
-                switch_to_blog( $blog_id );
144
-                $css_exclude = autoptimizeOptionWrapper::get_option( 'autoptimize_css_exclude' );
145
-                if ( empty( $css_exclude ) ) {
142
+            foreach ($blog_ids as $blog_id) {
143
+                switch_to_blog($blog_id);
144
+                $css_exclude = autoptimizeOptionWrapper::get_option('autoptimize_css_exclude');
145
+                if (empty($css_exclude)) {
146 146
                     $css_exclude = 'admin-bar.min.css, dashicons.min.css';
147
-                } elseif ( false === strpos( $css_exclude, 'dashicons.min.css' ) ) {
147
+                } elseif (false === strpos($css_exclude, 'dashicons.min.css')) {
148 148
                     $css_exclude .= ', dashicons.min.css';
149 149
                 }
150
-                autoptimizeOptionWrapper::update_option( 'autoptimize_css_exclude', $css_exclude );
150
+                autoptimizeOptionWrapper::update_option('autoptimize_css_exclude', $css_exclude);
151 151
             }
152
-            switch_to_blog( $original_blog_id );
152
+            switch_to_blog($original_blog_id);
153 153
         }
154 154
     }
155 155
 
@@ -161,19 +161,19 @@  discard block
 block discarded – undo
161 161
      */
162 162
     private function upgrade_from_1_9()
163 163
     {
164
-        if ( ! is_multisite() ) {
165
-            autoptimizeOptionWrapper::update_option( 'autoptimize_css_include_inline', 'on' );
166
-            autoptimizeOptionWrapper::update_option( 'autoptimize_js_include_inline', 'on' );
164
+        if (!is_multisite()) {
165
+            autoptimizeOptionWrapper::update_option('autoptimize_css_include_inline', 'on');
166
+            autoptimizeOptionWrapper::update_option('autoptimize_js_include_inline', 'on');
167 167
         } else {
168 168
             global $wpdb;
169
-            $blog_ids         = $wpdb->get_col( "SELECT blog_id FROM $wpdb->blogs" );
169
+            $blog_ids         = $wpdb->get_col("SELECT blog_id FROM $wpdb->blogs");
170 170
             $original_blog_id = get_current_blog_id();
171
-            foreach ( $blog_ids as $blog_id ) {
172
-                switch_to_blog( $blog_id );
173
-                autoptimizeOptionWrapper::update_option( 'autoptimize_css_include_inline', 'on' );
174
-                autoptimizeOptionWrapper::update_option( 'autoptimize_js_include_inline', 'on' );
171
+            foreach ($blog_ids as $blog_id) {
172
+                switch_to_blog($blog_id);
173
+                autoptimizeOptionWrapper::update_option('autoptimize_css_include_inline', 'on');
174
+                autoptimizeOptionWrapper::update_option('autoptimize_js_include_inline', 'on');
175 175
             }
176
-            switch_to_blog( $original_blog_id );
176
+            switch_to_blog($original_blog_id);
177 177
         }
178 178
     }
179 179
 
@@ -184,17 +184,17 @@  discard block
 block discarded – undo
184 184
      */
185 185
     private function upgrade_from_2_2()
186 186
     {
187
-        if ( ! is_multisite() ) {
187
+        if (!is_multisite()) {
188 188
             $this->do_2_2_settings_update();
189 189
         } else {
190 190
             global $wpdb;
191
-            $blog_ids         = $wpdb->get_col( "SELECT blog_id FROM $wpdb->blogs" );
191
+            $blog_ids         = $wpdb->get_col("SELECT blog_id FROM $wpdb->blogs");
192 192
             $original_blog_id = get_current_blog_id();
193
-            foreach ( $blog_ids as $blog_id ) {
194
-                switch_to_blog( $blog_id );
193
+            foreach ($blog_ids as $blog_id) {
194
+                switch_to_blog($blog_id);
195 195
                 $this->do_2_2_settings_update();
196 196
             }
197
-            switch_to_blog( $original_blog_id );
197
+            switch_to_blog($original_blog_id);
198 198
         }
199 199
     }
200 200
 
@@ -203,12 +203,12 @@  discard block
 block discarded – undo
203 203
      */
204 204
     private function do_2_2_settings_update()
205 205
     {
206
-        $nogooglefont    = autoptimizeOptionWrapper::get_option( 'autoptimize_css_nogooglefont', '' );
207
-        $ao_extrasetting = autoptimizeOptionWrapper::get_option( 'autoptimize_extra_settings', '' );
208
-        if ( ( $nogooglefont ) && ( empty( $ao_extrasetting ) ) ) {
209
-            autoptimizeOptionWrapper::update_option( 'autoptimize_extra_settings', autoptimizeConfig::get_ao_extra_default_options() );
206
+        $nogooglefont    = autoptimizeOptionWrapper::get_option('autoptimize_css_nogooglefont', '');
207
+        $ao_extrasetting = autoptimizeOptionWrapper::get_option('autoptimize_extra_settings', '');
208
+        if (($nogooglefont) && (empty($ao_extrasetting))) {
209
+            autoptimizeOptionWrapper::update_option('autoptimize_extra_settings', autoptimizeConfig::get_ao_extra_default_options());
210 210
         }
211
-        delete_option( 'autoptimize_css_nogooglefont' );
211
+        delete_option('autoptimize_css_nogooglefont');
212 212
     }
213 213
 
214 214
     /**
@@ -220,33 +220,33 @@  discard block
 block discarded – undo
220 220
         $jobs = _get_cron_array();
221 221
 
222 222
         // Remove all ao_cachechecker cron jobs (for now).
223
-        foreach ( $jobs as $when => $job ) {
224
-            $name = key( $job );
223
+        foreach ($jobs as $when => $job) {
224
+            $name = key($job);
225 225
 
226
-            if ( false !== strpos( $name, 'ao_cachechecker' ) ) {
227
-                unset( $jobs[ $when ] );
226
+            if (false !== strpos($name, 'ao_cachechecker')) {
227
+                unset($jobs[$when]);
228 228
             }
229 229
         }
230 230
 
231 231
         // Save the data.
232
-        _set_cron_array( $jobs );
232
+        _set_cron_array($jobs);
233 233
     }
234 234
 
235 235
     /**
236 236
      * Migrate imgopt options from autoptimize_extra_settings to autoptimize_imgopt_settings
237 237
      */
238 238
     private function upgrade_from_2_4() {
239
-        $extra_settings  = autoptimizeOptionWrapper::get_option( 'autoptimize_extra_settings', '' );
240
-        $imgopt_settings = autoptimizeOptionWrapper::get_option( 'autoptimize_imgopt_settings', '' );
241
-        if ( empty( $imgopt_settings ) && ! empty( $extra_settings ) ) {
239
+        $extra_settings  = autoptimizeOptionWrapper::get_option('autoptimize_extra_settings', '');
240
+        $imgopt_settings = autoptimizeOptionWrapper::get_option('autoptimize_imgopt_settings', '');
241
+        if (empty($imgopt_settings) && !empty($extra_settings)) {
242 242
             $imgopt_settings = autoptimizeConfig::get_ao_imgopt_default_options();
243
-            if ( array_key_exists( 'autoptimize_extra_checkbox_field_5', $extra_settings ) ) {
243
+            if (array_key_exists('autoptimize_extra_checkbox_field_5', $extra_settings)) {
244 244
                 $imgopt_settings['autoptimize_imgopt_checkbox_field_1'] = $extra_settings['autoptimize_extra_checkbox_field_5'];
245 245
             }
246
-            if ( array_key_exists( 'autoptimize_extra_select_field_6', $extra_settings ) ) {
246
+            if (array_key_exists('autoptimize_extra_select_field_6', $extra_settings)) {
247 247
                 $imgopt_settings['autoptimize_imgopt_select_field_2'] = $extra_settings['autoptimize_extra_select_field_6'];
248 248
             }
249
-            autoptimizeOptionWrapper::update_option( 'autoptimize_imgopt_settings', $imgopt_settings );
249
+            autoptimizeOptionWrapper::update_option('autoptimize_imgopt_settings', $imgopt_settings);
250 250
         }
251 251
     }
252 252
 }
Please login to merge, or discard this patch.
classes/autoptimizeBase.php 1 patch
Spacing   +148 added lines, -148 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * Base class other (more-specific) classes inherit from.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
      */
31 31
     public $cdn_url = '';
32 32
 
33
-    public function __construct( $content )
33
+    public function __construct($content)
34 34
     {
35 35
         $this->content = $content;
36 36
     }
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
      *
43 43
      * @return bool
44 44
      */
45
-    abstract public function read( $options );
45
+    abstract public function read($options);
46 46
 
47 47
     /**
48 48
      * Joins and optimizes collected things.
@@ -73,40 +73,40 @@  discard block
 block discarded – undo
73 73
      *
74 74
      * @return bool|string
75 75
      */
76
-    public function getpath( $url )
76
+    public function getpath($url)
77 77
     {
78
-        $url = apply_filters( 'autoptimize_filter_cssjs_alter_url', $url );
78
+        $url = apply_filters('autoptimize_filter_cssjs_alter_url', $url);
79 79
 
80
-        if ( false !== strpos( $url, '%' ) ) {
81
-            $url = urldecode( $url );
80
+        if (false !== strpos($url, '%')) {
81
+            $url = urldecode($url);
82 82
         }
83 83
 
84
-        $site_host    = parse_url( AUTOPTIMIZE_WP_SITE_URL, PHP_URL_HOST );
85
-        $content_host = parse_url( AUTOPTIMIZE_WP_ROOT_URL, PHP_URL_HOST );
84
+        $site_host    = parse_url(AUTOPTIMIZE_WP_SITE_URL, PHP_URL_HOST);
85
+        $content_host = parse_url(AUTOPTIMIZE_WP_ROOT_URL, PHP_URL_HOST);
86 86
 
87 87
         // Normalizing attempts...
88
-        $double_slash_position = strpos( $url, '//' );
89
-        if ( 0 === $double_slash_position ) {
90
-            if ( is_ssl() ) {
91
-                $url = 'https:' . $url;
88
+        $double_slash_position = strpos($url, '//');
89
+        if (0 === $double_slash_position) {
90
+            if (is_ssl()) {
91
+                $url = 'https:'.$url;
92 92
             } else {
93
-                $url = 'http:' . $url;
93
+                $url = 'http:'.$url;
94 94
             }
95
-        } elseif ( ( false === $double_slash_position ) && ( false === strpos( $url, $site_host ) ) ) {
96
-            if ( AUTOPTIMIZE_WP_SITE_URL === $site_host ) {
97
-                $url = AUTOPTIMIZE_WP_SITE_URL . $url;
95
+        } elseif ((false === $double_slash_position) && (false === strpos($url, $site_host))) {
96
+            if (AUTOPTIMIZE_WP_SITE_URL === $site_host) {
97
+                $url = AUTOPTIMIZE_WP_SITE_URL.$url;
98 98
             } else {
99
-                $url = AUTOPTIMIZE_WP_SITE_URL . autoptimizeUtils::path_canonicalize( $url );
99
+                $url = AUTOPTIMIZE_WP_SITE_URL.autoptimizeUtils::path_canonicalize($url);
100 100
             }
101 101
         }
102 102
 
103
-        if ( $site_host !== $content_host ) {
104
-            $url = str_replace( AUTOPTIMIZE_WP_CONTENT_URL, AUTOPTIMIZE_WP_SITE_URL . AUTOPTIMIZE_WP_CONTENT_NAME, $url );
103
+        if ($site_host !== $content_host) {
104
+            $url = str_replace(AUTOPTIMIZE_WP_CONTENT_URL, AUTOPTIMIZE_WP_SITE_URL.AUTOPTIMIZE_WP_CONTENT_NAME, $url);
105 105
         }
106 106
 
107 107
         // First check; hostname wp site should be hostname of url!
108
-        $url_host = @parse_url( $url, PHP_URL_HOST ); // @codingStandardsIgnoreLine
109
-        if ( $url_host !== $site_host ) {
108
+        $url_host = @parse_url($url, PHP_URL_HOST); // @codingStandardsIgnoreLine
109
+        if ($url_host !== $site_host) {
110 110
             /**
111 111
              * First try to get all domains from WPML (if available)
112 112
              * then explicitely declare $this->cdn_url as OK as well
@@ -115,20 +115,20 @@  discard block
 block discarded – undo
115 115
              */
116 116
             $multidomains = array();
117 117
 
118
-            $multidomains_wpml = apply_filters( 'wpml_setting', array(), 'language_domains' );
119
-            if ( ! empty( $multidomains_wpml ) ) {
120
-                $multidomains = array_map( array( $this, 'get_url_hostname' ), $multidomains_wpml );
118
+            $multidomains_wpml = apply_filters('wpml_setting', array(), 'language_domains');
119
+            if (!empty($multidomains_wpml)) {
120
+                $multidomains = array_map(array($this, 'get_url_hostname'), $multidomains_wpml);
121 121
             }
122 122
 
123
-            if ( ! empty( $this->cdn_url ) ) {
124
-                $multidomains[] = parse_url( $this->cdn_url, PHP_URL_HOST );
123
+            if (!empty($this->cdn_url)) {
124
+                $multidomains[] = parse_url($this->cdn_url, PHP_URL_HOST);
125 125
             }
126 126
 
127
-            $multidomains = apply_filters( 'autoptimize_filter_cssjs_multidomain', $multidomains );
127
+            $multidomains = apply_filters('autoptimize_filter_cssjs_multidomain', $multidomains);
128 128
 
129
-            if ( ! empty( $multidomains ) ) {
130
-                if ( in_array( $url_host, $multidomains ) ) {
131
-                    $url = str_replace( $url_host, $site_host, $url );
129
+            if (!empty($multidomains)) {
130
+                if (in_array($url_host, $multidomains)) {
131
+                    $url = str_replace($url_host, $site_host, $url);
132 132
                 } else {
133 133
                     return false;
134 134
                 }
@@ -138,28 +138,28 @@  discard block
 block discarded – undo
138 138
         }
139 139
 
140 140
         // Try to remove "wp root url" from url while not minding http<>https.
141
-        $tmp_ao_root = preg_replace( '/https?:/', '', AUTOPTIMIZE_WP_ROOT_URL );
141
+        $tmp_ao_root = preg_replace('/https?:/', '', AUTOPTIMIZE_WP_ROOT_URL);
142 142
 
143
-        if ( $site_host !== $content_host ) {
143
+        if ($site_host !== $content_host) {
144 144
             // As we replaced the content-domain with the site-domain, we should match against that.
145
-            $tmp_ao_root = preg_replace( '/https?:/', '', AUTOPTIMIZE_WP_SITE_URL );
145
+            $tmp_ao_root = preg_replace('/https?:/', '', AUTOPTIMIZE_WP_SITE_URL);
146 146
         }
147 147
 
148
-        $tmp_url = preg_replace( '/https?:/', '', $url );
149
-        $path    = str_replace( $tmp_ao_root, '', $tmp_url );
148
+        $tmp_url = preg_replace('/https?:/', '', $url);
149
+        $path    = str_replace($tmp_ao_root, '', $tmp_url);
150 150
 
151 151
         // If path starts with :// or //, this is not a URL in the WP context and
152 152
         // we have to assume we can't aggregate.
153
-        if ( preg_match( '#^:?//#', $path ) ) {
153
+        if (preg_match('#^:?//#', $path)) {
154 154
             // External script/css (adsense, etc).
155 155
             return false;
156 156
         }
157 157
 
158 158
         // Prepend with WP_ROOT_DIR to have full path to file.
159
-        $path = str_replace( '//', '/', WP_ROOT_DIR . $path );
159
+        $path = str_replace('//', '/', WP_ROOT_DIR.$path);
160 160
 
161 161
         // Final check: does file exist and is it readable?
162
-        if ( file_exists( $path ) && is_file( $path ) && is_readable( $path ) ) {
162
+        if (file_exists($path) && is_file($path) && is_readable($path)) {
163 163
             return $path;
164 164
         } else {
165 165
             return false;
@@ -176,18 +176,18 @@  discard block
 block discarded – undo
176 176
      *
177 177
      * @return string
178 178
      */
179
-    protected function get_url_hostname( $url )
179
+    protected function get_url_hostname($url)
180 180
     {
181 181
         // Checking that the url starts with something vaguely resembling a protocol.
182
-        if ( ( 0 !== strpos( $url, 'http' ) ) && ( 0 !== strpos( $url, '//' ) ) ) {
183
-            $url = 'http://' . $url;
182
+        if ((0 !== strpos($url, 'http')) && (0 !== strpos($url, '//'))) {
183
+            $url = 'http://'.$url;
184 184
         }
185 185
 
186 186
         // Grab the hostname.
187
-        $hostname = parse_url( $url, PHP_URL_HOST );
187
+        $hostname = parse_url($url, PHP_URL_HOST);
188 188
 
189 189
         // Fallback when parse_url() fails.
190
-        if ( empty( $hostname ) ) {
190
+        if (empty($hostname)) {
191 191
             $hostname = $url;
192 192
         }
193 193
 
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
      *
202 202
      * @return string
203 203
      */
204
-    protected function hide_noptimize( $markup )
204
+    protected function hide_noptimize($markup)
205 205
     {
206 206
         return $this->replace_contents_with_marker_if_exists(
207 207
             'NOPTIMIZE',
@@ -218,9 +218,9 @@  discard block
 block discarded – undo
218 218
      *
219 219
      * @return string
220 220
      */
221
-    protected function restore_noptimize( $markup )
221
+    protected function restore_noptimize($markup)
222 222
     {
223
-        return $this->restore_marked_content( 'NOPTIMIZE', $markup );
223
+        return $this->restore_marked_content('NOPTIMIZE', $markup);
224 224
     }
225 225
 
226 226
     /**
@@ -230,7 +230,7 @@  discard block
 block discarded – undo
230 230
      *
231 231
      * @return string
232 232
      */
233
-    protected function hide_iehacks( $markup )
233
+    protected function hide_iehacks($markup)
234 234
     {
235 235
         return $this->replace_contents_with_marker_if_exists(
236 236
             'IEHACK', // Marker name...
@@ -247,9 +247,9 @@  discard block
 block discarded – undo
247 247
      *
248 248
      * @return string
249 249
      */
250
-    protected function restore_iehacks( $markup )
250
+    protected function restore_iehacks($markup)
251 251
     {
252
-        return $this->restore_marked_content( 'IEHACK', $markup );
252
+        return $this->restore_marked_content('IEHACK', $markup);
253 253
     }
254 254
 
255 255
     /**
@@ -261,7 +261,7 @@  discard block
 block discarded – undo
261 261
      *
262 262
      * @return string
263 263
      */
264
-    protected function hide_comments( $markup )
264
+    protected function hide_comments($markup)
265 265
     {
266 266
         return $this->replace_contents_with_marker_if_exists(
267 267
             'COMMENTS',
@@ -279,9 +279,9 @@  discard block
 block discarded – undo
279 279
      *
280 280
      * @return string
281 281
      */
282
-    protected function restore_comments( $markup )
282
+    protected function restore_comments($markup)
283 283
     {
284
-        return $this->restore_marked_content( 'COMMENTS', $markup );
284
+        return $this->restore_marked_content('COMMENTS', $markup);
285 285
     }
286 286
 
287 287
     /**
@@ -292,44 +292,44 @@  discard block
 block discarded – undo
292 292
      *
293 293
      * @return string
294 294
      */
295
-    public function url_replace_cdn( $url )
295
+    public function url_replace_cdn($url)
296 296
     {
297 297
         // For 2.3 back-compat in which cdn-ing appeared to be automatically
298 298
         // including WP subfolder/subdirectory into account as part of cdn-ing,
299 299
         // even though it might've caused serious troubles in certain edge-cases.
300
-        $cdn_url = autoptimizeUtils::tweak_cdn_url_if_needed( $this->cdn_url );
300
+        $cdn_url = autoptimizeUtils::tweak_cdn_url_if_needed($this->cdn_url);
301 301
 
302 302
         // Allows API/filter to further tweak the cdn url...
303
-        $cdn_url = apply_filters( 'autoptimize_filter_base_cdnurl', $cdn_url );
304
-        if ( ! empty( $cdn_url ) ) {
305
-            $this->debug_log( 'before=' . $url );
303
+        $cdn_url = apply_filters('autoptimize_filter_base_cdnurl', $cdn_url);
304
+        if (!empty($cdn_url)) {
305
+            $this->debug_log('before='.$url);
306 306
 
307 307
             // Simple str_replace-based approach fails when $url is protocol-or-host-relative.
308
-            $is_protocol_relative = autoptimizeUtils::is_protocol_relative( $url );
309
-            $is_host_relative     = ( ! $is_protocol_relative && ( '/' === $url[0] ) );
310
-            $cdn_url              = rtrim( $cdn_url, '/' );
308
+            $is_protocol_relative = autoptimizeUtils::is_protocol_relative($url);
309
+            $is_host_relative     = (!$is_protocol_relative && ('/' === $url[0]));
310
+            $cdn_url              = rtrim($cdn_url, '/');
311 311
 
312
-            if ( $is_host_relative ) {
312
+            if ($is_host_relative) {
313 313
                 // Prepending host-relative urls with the cdn url.
314
-                $url = $cdn_url . $url;
314
+                $url = $cdn_url.$url;
315 315
             } else {
316 316
                 // Either a protocol-relative or "regular" url, replacing it either way.
317
-                if ( $is_protocol_relative ) {
317
+                if ($is_protocol_relative) {
318 318
                     // Massage $site_url so that simple str_replace() still "works" by
319 319
                     // searching for the protocol-relative version of AUTOPTIMIZE_WP_SITE_URL.
320
-                    $site_url = str_replace( array( 'http:', 'https:' ), '', AUTOPTIMIZE_WP_SITE_URL );
320
+                    $site_url = str_replace(array('http:', 'https:'), '', AUTOPTIMIZE_WP_SITE_URL);
321 321
                 } else {
322 322
                     $site_url = AUTOPTIMIZE_WP_SITE_URL;
323 323
                 }
324
-                $this->debug_log( '`' . $site_url . '` -> `' . $cdn_url . '` in `' . $url . '`' );
325
-                $url = str_replace( $site_url, $cdn_url, $url );
324
+                $this->debug_log('`'.$site_url.'` -> `'.$cdn_url.'` in `'.$url.'`');
325
+                $url = str_replace($site_url, $cdn_url, $url);
326 326
             }
327 327
 
328
-            $this->debug_log( 'after=' . $url );
328
+            $this->debug_log('after='.$url);
329 329
         }
330 330
 
331 331
         // Allow API filter to take further care of CDN replacement.
332
-        $url = apply_filters( 'autoptimize_filter_base_replace_cdn', $url );
332
+        $url = apply_filters('autoptimize_filter_base_replace_cdn', $url);
333 333
 
334 334
         return $url;
335 335
     }
@@ -347,18 +347,18 @@  discard block
 block discarded – undo
347 347
      *
348 348
      * @return void
349 349
      */
350
-    protected function inject_in_html( $payload, $where )
350
+    protected function inject_in_html($payload, $where)
351 351
     {
352 352
         $warned   = false;
353
-        $position = autoptimizeUtils::strpos( $this->content, $where[0] );
354
-        if ( false !== $position ) {
353
+        $position = autoptimizeUtils::strpos($this->content, $where[0]);
354
+        if (false !== $position) {
355 355
             // Found the tag, setup content/injection as specified.
356
-            if ( 'after' === $where[1] ) {
357
-                $content = $where[0] . $payload;
358
-            } elseif ( 'replace' === $where[1] ) {
356
+            if ('after' === $where[1]) {
357
+                $content = $where[0].$payload;
358
+            } elseif ('replace' === $where[1]) {
359 359
                 $content = $payload;
360 360
             } else {
361
-                $content = $payload . $where[0];
361
+                $content = $payload.$where[0];
362 362
             }
363 363
             // Place where specified.
364 364
             $this->content = autoptimizeUtils::substr_replace(
@@ -367,14 +367,14 @@  discard block
 block discarded – undo
367 367
                 $position,
368 368
                 // Using plain strlen() should be safe here for now, since
369 369
                 // we're not searching for multibyte chars here still...
370
-                strlen( $where[0] )
370
+                strlen($where[0])
371 371
             );
372 372
         } else {
373 373
             // Couldn't find what was specified, just append and add a warning.
374 374
             $this->content .= $payload;
375
-            if ( ! $warned ) {
376
-                $tag_display    = str_replace( array( '<', '>' ), '', $where[0] );
377
-                $this->content .= '<!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `' . $tag_display . '` missing --><!--/noptimize-->';
375
+            if (!$warned) {
376
+                $tag_display    = str_replace(array('<', '>'), '', $where[0]);
377
+                $this->content .= '<!--noptimize--><!-- Autoptimize found a problem with the HTML in your Theme, tag `'.$tag_display.'` missing --><!--/noptimize-->';
378 378
                 $warned         = true;
379 379
             }
380 380
         }
@@ -388,10 +388,10 @@  discard block
 block discarded – undo
388 388
      *
389 389
      * @return bool
390 390
      */
391
-    protected function isremovable( $tag, $removables )
391
+    protected function isremovable($tag, $removables)
392 392
     {
393
-        foreach ( $removables as $match ) {
394
-            if ( false !== strpos( $tag, $match ) ) {
393
+        foreach ($removables as $match) {
394
+            if (false !== strpos($tag, $match)) {
395 395
                 return true;
396 396
             }
397 397
         }
@@ -406,10 +406,10 @@  discard block
 block discarded – undo
406 406
      *
407 407
      * @return string
408 408
      */
409
-    public function inject_minified_callback( $matches )
409
+    public function inject_minified_callback($matches)
410 410
     {
411 411
         static $conf = null;
412
-        if ( null === $conf ) {
412
+        if (null === $conf) {
413 413
             $conf = autoptimizeConfig::instance();
414 414
         }
415 415
 
@@ -424,62 +424,62 @@  discard block
 block discarded – undo
424 424
         $filehash = null;
425 425
 
426 426
         // Grab the parts we need.
427
-        $parts = explode( '|', $matches[1] );
428
-        if ( ! empty( $parts ) ) {
429
-            $filepath = isset( $parts[0] ) ? base64_decode( $parts[0] ) : null;
430
-            $filehash = isset( $parts[1] ) ? $parts[1] : null;
427
+        $parts = explode('|', $matches[1]);
428
+        if (!empty($parts)) {
429
+            $filepath = isset($parts[0]) ? base64_decode($parts[0]) : null;
430
+            $filehash = isset($parts[1]) ? $parts[1] : null;
431 431
         }
432 432
 
433 433
         // Bail early if something's not right...
434
-        if ( ! $filepath || ! $filehash ) {
434
+        if (!$filepath || !$filehash) {
435 435
             return "\n";
436 436
         }
437 437
 
438
-        $filecontent = file_get_contents( $filepath );
438
+        $filecontent = file_get_contents($filepath);
439 439
 
440 440
         // Some things are differently handled for css/js...
441
-        $is_js_file = ( '.js' === substr( $filepath, -3, 3 ) );
441
+        $is_js_file = ('.js' === substr($filepath, -3, 3));
442 442
 
443 443
         $is_css_file = false;
444
-        if ( ! $is_js_file ) {
445
-            $is_css_file = ( '.css' === substr( $filepath, -4, 4 ) );
444
+        if (!$is_js_file) {
445
+            $is_css_file = ('.css' === substr($filepath, -4, 4));
446 446
         }
447 447
 
448 448
         // BOMs being nuked here unconditionally (regardless of where they are)!
449
-        $filecontent = preg_replace( "#\x{EF}\x{BB}\x{BF}#", '', $filecontent );
449
+        $filecontent = preg_replace("#\x{EF}\x{BB}\x{BF}#", '', $filecontent);
450 450
 
451 451
         // Remove comments and blank lines.
452
-        if ( $is_js_file ) {
453
-            $filecontent = preg_replace( '#^\s*\/\/.*$#Um', '', $filecontent );
452
+        if ($is_js_file) {
453
+            $filecontent = preg_replace('#^\s*\/\/.*$#Um', '', $filecontent);
454 454
         }
455 455
 
456 456
         // Nuke un-important comments.
457
-        $filecontent = preg_replace( '#^\s*\/\*[^!].*\*\/\s?#Um', '', $filecontent );
457
+        $filecontent = preg_replace('#^\s*\/\*[^!].*\*\/\s?#Um', '', $filecontent);
458 458
 
459 459
         // Normalize newlines.
460
-        $filecontent = preg_replace( '#(^[\r\n]*|[\r\n]+)[\s\t]*[\r\n]+#', "\n", $filecontent );
460
+        $filecontent = preg_replace('#(^[\r\n]*|[\r\n]+)[\s\t]*[\r\n]+#', "\n", $filecontent);
461 461
 
462 462
         // JS specifics.
463
-        if ( $is_js_file ) {
463
+        if ($is_js_file) {
464 464
             // Append a semicolon at the end of js files if it's missing.
465
-            $last_char = substr( $filecontent, -1, 1 );
466
-            if ( ';' !== $last_char && '}' !== $last_char ) {
465
+            $last_char = substr($filecontent, -1, 1);
466
+            if (';' !== $last_char && '}' !== $last_char) {
467 467
                 $filecontent .= ';';
468 468
             }
469 469
             // Check if try/catch should be used.
470
-            $opt_js_try_catch = $conf->get( 'autoptimize_js_trycatch' );
471
-            if ( 'on' === $opt_js_try_catch ) {
470
+            $opt_js_try_catch = $conf->get('autoptimize_js_trycatch');
471
+            if ('on' === $opt_js_try_catch) {
472 472
                 // It should, wrap in try/catch.
473
-                $filecontent = 'try{' . $filecontent . '}catch(e){}';
473
+                $filecontent = 'try{'.$filecontent.'}catch(e){}';
474 474
             }
475
-        } elseif ( $is_css_file ) {
476
-            $filecontent = autoptimizeStyles::fixurls( $filepath, $filecontent );
475
+        } elseif ($is_css_file) {
476
+            $filecontent = autoptimizeStyles::fixurls($filepath, $filecontent);
477 477
         } else {
478 478
             $filecontent = '';
479 479
         }
480 480
 
481 481
         // Return modified (or empty!) code/content.
482
-        return "\n" . $filecontent;
482
+        return "\n".$filecontent;
483 483
     }
484 484
 
485 485
     /**
@@ -489,13 +489,13 @@  discard block
 block discarded – undo
489 489
      *
490 490
      * @return string
491 491
      */
492
-    protected function inject_minified( $in )
492
+    protected function inject_minified($in)
493 493
     {
494 494
         $out = $in;
495
-        if ( false !== strpos( $in, '%%INJECTLATER%%' ) ) {
495
+        if (false !== strpos($in, '%%INJECTLATER%%')) {
496 496
             $out = preg_replace_callback(
497
-                '#\/\*\!%%INJECTLATER' . AUTOPTIMIZE_HASH . '%%(.*?)%%INJECTLATER%%\*\/#is',
498
-                array( $this, 'inject_minified_callback' ),
497
+                '#\/\*\!%%INJECTLATER'.AUTOPTIMIZE_HASH.'%%(.*?)%%INJECTLATER%%\*\/#is',
498
+                array($this, 'inject_minified_callback'),
499 499
                 $in
500 500
             );
501 501
         }
@@ -515,9 +515,9 @@  discard block
 block discarded – undo
515 515
      *
516 516
      * @return string
517 517
      */
518
-    public static function build_injectlater_marker( $filepath, $hash )
518
+    public static function build_injectlater_marker($filepath, $hash)
519 519
     {
520
-        $contents = '/*!' . self::build_marker( 'INJECTLATER', $filepath, $hash ) . '*/';
520
+        $contents = '/*!'.self::build_marker('INJECTLATER', $filepath, $hash).'*/';
521 521
 
522 522
         return $contents;
523 523
     }
@@ -535,18 +535,18 @@  discard block
 block discarded – undo
535 535
      *
536 536
      * @return string
537 537
      */
538
-    public static function build_marker( $name, $data, $hash = null )
538
+    public static function build_marker($name, $data, $hash = null)
539 539
     {
540 540
         // Start the marker, add the data.
541
-        $marker = '%%' . $name . AUTOPTIMIZE_HASH . '%%' . base64_encode( $data );
541
+        $marker = '%%'.$name.AUTOPTIMIZE_HASH.'%%'.base64_encode($data);
542 542
 
543 543
         // Add the hash if provided.
544
-        if ( null !== $hash ) {
545
-            $marker .= '|' . $hash;
544
+        if (null !== $hash) {
545
+            $marker .= '|'.$hash;
546 546
         }
547 547
 
548 548
         // Close the marker.
549
-        $marker .= '%%' . $name . '%%';
549
+        $marker .= '%%'.$name.'%%';
550 550
 
551 551
         return $marker;
552 552
     }
@@ -566,22 +566,22 @@  discard block
 block discarded – undo
566 566
      *
567 567
      * @return string
568 568
      */
569
-    public static function replace_contents_with_marker_if_exists( $marker, $search, $re_replace_pattern, $content )
569
+    public static function replace_contents_with_marker_if_exists($marker, $search, $re_replace_pattern, $content)
570 570
     {
571 571
         $found = false;
572 572
 
573
-        $is_regex = autoptimizeUtils::str_is_valid_regex( $search );
574
-        if ( $is_regex ) {
575
-            $found = preg_match( $search, $content );
573
+        $is_regex = autoptimizeUtils::str_is_valid_regex($search);
574
+        if ($is_regex) {
575
+            $found = preg_match($search, $content);
576 576
         } else {
577
-            $found = ( false !== strpos( $content, $search ) );
577
+            $found = (false !== strpos($content, $search));
578 578
         }
579 579
 
580
-        if ( $found ) {
580
+        if ($found) {
581 581
             $content = preg_replace_callback(
582 582
                 $re_replace_pattern,
583
-                function( $matches ) use ( $marker ) {
584
-                    return autoptimizeBase::build_marker( $marker, $matches[0] );
583
+                function($matches) use ($marker) {
584
+                    return autoptimizeBase::build_marker($marker, $matches[0]);
585 585
                 },
586 586
                 $content
587 587
             );
@@ -598,13 +598,13 @@  discard block
 block discarded – undo
598 598
      *
599 599
      * @return string
600 600
      */
601
-    public static function restore_marked_content( $marker, $content )
601
+    public static function restore_marked_content($marker, $content)
602 602
     {
603
-        if ( false !== strpos( $content, $marker ) ) {
603
+        if (false !== strpos($content, $marker)) {
604 604
             $content = preg_replace_callback(
605
-                '#%%' . $marker . AUTOPTIMIZE_HASH . '%%(.*?)%%' . $marker . '%%#is',
606
-                function ( $matches ) {
607
-                    return base64_decode( $matches[1] );
605
+                '#%%'.$marker.AUTOPTIMIZE_HASH.'%%(.*?)%%'.$marker.'%%#is',
606
+                function($matches) {
607
+                    return base64_decode($matches[1]);
608 608
                 },
609 609
                 $content
610 610
             );
@@ -620,17 +620,17 @@  discard block
 block discarded – undo
620 620
      *
621 621
      * @return void
622 622
      */
623
-    protected function debug_log( $data )
623
+    protected function debug_log($data)
624 624
     {
625
-        if ( ! isset( $this->debug_log ) || ! $this->debug_log ) {
625
+        if (!isset($this->debug_log) || !$this->debug_log) {
626 626
             return;
627 627
         }
628 628
 
629
-        if ( ! is_string( $data ) && ! is_resource( $data ) ) {
630
-            $data = var_export( $data, true );
629
+        if (!is_string($data) && !is_resource($data)) {
630
+            $data = var_export($data, true);
631 631
         }
632 632
 
633
-        error_log( $data );
633
+        error_log($data);
634 634
     }
635 635
 
636 636
     /**
@@ -640,12 +640,12 @@  discard block
 block discarded – undo
640 640
      *
641 641
      * @return bool|string to be minified code or false.
642 642
      */
643
-    protected function prepare_minify_single( $filepath )
643
+    protected function prepare_minify_single($filepath)
644 644
     {
645 645
         // Decide what we're dealing with, return false if we don't know.
646
-        if ( autoptimizeUtils::str_ends_in( $filepath, '.js' ) ) {
646
+        if (autoptimizeUtils::str_ends_in($filepath, '.js')) {
647 647
             $type = 'js';
648
-        } elseif ( autoptimizeUtils::str_ends_in( $filepath, '.css' ) ) {
648
+        } elseif (autoptimizeUtils::str_ends_in($filepath, '.css')) {
649 649
             $type = 'css';
650 650
         } else {
651 651
             return false;
@@ -654,18 +654,18 @@  discard block
 block discarded – undo
654 654
         // Bail if it looks like its already minifed (by having -min or .min
655 655
         // in filename) or if it looks like WP jquery.js (which is minified).
656 656
         $minified_variants = array(
657
-            '-min.' . $type,
658
-            '.min.' . $type,
657
+            '-min.'.$type,
658
+            '.min.'.$type,
659 659
             'js/jquery/jquery.js',
660 660
         );
661
-        foreach ( $minified_variants as $ending ) {
662
-            if ( autoptimizeUtils::str_ends_in( $filepath, $ending ) ) {
661
+        foreach ($minified_variants as $ending) {
662
+            if (autoptimizeUtils::str_ends_in($filepath, $ending)) {
663 663
                 return false;
664 664
             }
665 665
         }
666 666
 
667 667
         // Get file contents, bail if empty.
668
-        $contents = file_get_contents( $filepath );
668
+        $contents = file_get_contents($filepath);
669 669
 
670 670
         return $contents;
671 671
     }
@@ -678,12 +678,12 @@  discard block
 block discarded – undo
678 678
      *
679 679
      * @return string
680 680
      */
681
-    protected function build_minify_single_url( autoptimizeCache $cache )
681
+    protected function build_minify_single_url(autoptimizeCache $cache)
682 682
     {
683
-        $url = AUTOPTIMIZE_CACHE_URL . $cache->getname();
683
+        $url = AUTOPTIMIZE_CACHE_URL.$cache->getname();
684 684
 
685 685
         // CDN-replace the resulting URL if needed...
686
-        $url = $this->url_replace_cdn( $url );
686
+        $url = $this->url_replace_cdn($url);
687 687
 
688 688
         return $url;
689 689
     }
Please login to merge, or discard this patch.
classes/autoptimizeCacheChecker.php 1 patch
Spacing   +38 added lines, -38 removed lines patch added patch discarded remove patch
@@ -6,7 +6,7 @@  discard block
 block discarded – undo
6 6
  * Checks if cachesize is > 0.5GB (size is filterable), if so, an option is set which controls showing an admin notice.
7 7
  */
8 8
 
9
-if ( ! defined( 'ABSPATH' ) ) {
9
+if (!defined('ABSPATH')) {
10 10
     exit;
11 11
 }
12 12
 
@@ -25,51 +25,51 @@  discard block
 block discarded – undo
25 25
 
26 26
     public function add_hooks()
27 27
     {
28
-        if ( is_admin() ) {
29
-            add_action( 'plugins_loaded', array( $this, 'setup' ) );
28
+        if (is_admin()) {
29
+            add_action('plugins_loaded', array($this, 'setup'));
30 30
         }
31
-        add_action( self::SCHEDULE_HOOK, array( $this, 'cronjob' ) );
32
-        add_action( 'admin_notices', array( $this, 'show_admin_notice' ) );
31
+        add_action(self::SCHEDULE_HOOK, array($this, 'cronjob'));
32
+        add_action('admin_notices', array($this, 'show_admin_notice'));
33 33
     }
34 34
 
35 35
     public function setup()
36 36
     {
37
-        $do_cache_check = (bool) apply_filters( 'autoptimize_filter_cachecheck_do', true );
38
-        $schedule       = wp_get_schedule( self::SCHEDULE_HOOK );
39
-        $frequency      = apply_filters( 'autoptimize_filter_cachecheck_frequency', 'twicedaily' );
40
-        if ( ! in_array( $frequency, array( 'hourly', 'twicedaily', 'daily', 'weekly', 'monthly' ) ) ) {
37
+        $do_cache_check = (bool) apply_filters('autoptimize_filter_cachecheck_do', true);
38
+        $schedule       = wp_get_schedule(self::SCHEDULE_HOOK);
39
+        $frequency      = apply_filters('autoptimize_filter_cachecheck_frequency', 'twicedaily');
40
+        if (!in_array($frequency, array('hourly', 'twicedaily', 'daily', 'weekly', 'monthly'))) {
41 41
             $frequency = 'twicedaily';
42 42
         }
43
-        if ( $do_cache_check && ( ! $schedule || $schedule !== $frequency ) ) {
44
-            if ( $schedule ) {
45
-                wp_clear_scheduled_hook( self::SCHEDULE_HOOK );
43
+        if ($do_cache_check && (!$schedule || $schedule !== $frequency)) {
44
+            if ($schedule) {
45
+                wp_clear_scheduled_hook(self::SCHEDULE_HOOK);
46 46
             }
47
-            wp_schedule_event( time(), $frequency, self::SCHEDULE_HOOK );
48
-        } elseif ( $schedule && ! $do_cache_check ) {
49
-            wp_clear_scheduled_hook( self::SCHEDULE_HOOK );
47
+            wp_schedule_event(time(), $frequency, self::SCHEDULE_HOOK);
48
+        } elseif ($schedule && !$do_cache_check) {
49
+            wp_clear_scheduled_hook(self::SCHEDULE_HOOK);
50 50
         }
51 51
     }
52 52
 
53 53
     public function cronjob()
54 54
     {
55 55
         // Check cachesize and act accordingly.
56
-        $max_size       = (int) apply_filters( 'autoptimize_filter_cachecheck_maxsize', 536870912 );
57
-        $do_cache_check = (bool) apply_filters( 'autoptimize_filter_cachecheck_do', true );
56
+        $max_size       = (int) apply_filters('autoptimize_filter_cachecheck_maxsize', 536870912);
57
+        $do_cache_check = (bool) apply_filters('autoptimize_filter_cachecheck_do', true);
58 58
         $stat_array     = autoptimizeCache::stats();
59
-        $cache_size     = round( $stat_array[1] );
60
-        if ( ( $cache_size > $max_size ) && ( $do_cache_check ) ) {
61
-            autoptimizeOptionWrapper::update_option( 'autoptimize_cachesize_notice', true );
62
-            if ( apply_filters( 'autoptimize_filter_cachecheck_sendmail', true ) ) {
63
-                $home_url  = esc_url( home_url() );
64
-                $ao_mailto = apply_filters( 'autoptimize_filter_cachecheck_mailto', autoptimizeOptionWrapper::get_option( 'admin_email', '' ) );
65
-
66
-                $ao_mailsubject = __( 'Autoptimize cache size warning', 'autoptimize' ) . ' (' . $home_url . ')';
67
-                $ao_mailbody    = __( 'Autoptimize\'s cache size is getting big, consider purging the cache. Have a look at https://wordpress.org/plugins/autoptimize/faq/ to see how you can keep the cache size under control.', 'autoptimize' ) . ' (site: ' . $home_url . ')';
68
-
69
-                if ( ! empty( $ao_mailto ) ) {
70
-                    $ao_mailresult = wp_mail( $ao_mailto, $ao_mailsubject, $ao_mailbody );
71
-                    if ( ! $ao_mailresult ) {
72
-                        error_log( 'Autoptimize could not send cache size warning mail.' );
59
+        $cache_size     = round($stat_array[1]);
60
+        if (($cache_size > $max_size) && ($do_cache_check)) {
61
+            autoptimizeOptionWrapper::update_option('autoptimize_cachesize_notice', true);
62
+            if (apply_filters('autoptimize_filter_cachecheck_sendmail', true)) {
63
+                $home_url  = esc_url(home_url());
64
+                $ao_mailto = apply_filters('autoptimize_filter_cachecheck_mailto', autoptimizeOptionWrapper::get_option('admin_email', ''));
65
+
66
+                $ao_mailsubject = __('Autoptimize cache size warning', 'autoptimize').' ('.$home_url.')';
67
+                $ao_mailbody    = __('Autoptimize\'s cache size is getting big, consider purging the cache. Have a look at https://wordpress.org/plugins/autoptimize/faq/ to see how you can keep the cache size under control.', 'autoptimize').' (site: '.$home_url.')';
68
+
69
+                if (!empty($ao_mailto)) {
70
+                    $ao_mailresult = wp_mail($ao_mailto, $ao_mailsubject, $ao_mailbody);
71
+                    if (!$ao_mailresult) {
72
+                        error_log('Autoptimize could not send cache size warning mail.');
73 73
                     }
74 74
                 }
75 75
             }
@@ -87,16 +87,16 @@  discard block
 block discarded – undo
87 87
 
88 88
     public function show_admin_notice()
89 89
     {
90
-        if ( (bool) autoptimizeOptionWrapper::get_option( 'autoptimize_cachesize_notice', false ) && current_user_can( 'manage_options' ) ) {
90
+        if ((bool) autoptimizeOptionWrapper::get_option('autoptimize_cachesize_notice', false) && current_user_can('manage_options')) {
91 91
             echo '<div class="notice notice-warning"><p>';
92
-            _e( '<strong>Autoptimize\'s cache size is getting big</strong>, consider purging the cache. Have a look at <a href="https://wordpress.org/plugins/autoptimize/faq/" target="_blank" rel="noopener noreferrer">the Autoptimize FAQ</a> to see how you can keep the cache size under control.', 'autoptimize' );
92
+            _e('<strong>Autoptimize\'s cache size is getting big</strong>, consider purging the cache. Have a look at <a href="https://wordpress.org/plugins/autoptimize/faq/" target="_blank" rel="noopener noreferrer">the Autoptimize FAQ</a> to see how you can keep the cache size under control.', 'autoptimize');
93 93
             echo '</p></div>';
94
-            autoptimizeOptionWrapper::update_option( 'autoptimize_cachesize_notice', false );
94
+            autoptimizeOptionWrapper::update_option('autoptimize_cachesize_notice', false);
95 95
         }
96 96
 
97 97
         // Notice for image proxy usage.
98 98
         $_imgopt_notice = autoptimizeImages::instance()->get_imgopt_status_notice_wrapper();
99
-        if ( current_user_can( 'manage_options' ) && is_array( $_imgopt_notice ) && array_key_exists( 'status', $_imgopt_notice ) && in_array( $_imgopt_notice['status'], array( 1, -1, -2, -3 ) ) ) {
99
+        if (current_user_can('manage_options') && is_array($_imgopt_notice) && array_key_exists('status', $_imgopt_notice) && in_array($_imgopt_notice['status'], array(1, -1, -2, -3))) {
100 100
             $_dismissible = 'ao-img-opt-notice-';
101 101
             $_hide_notice = '7';
102 102
 
@@ -104,10 +104,10 @@  discard block
 block discarded – undo
104 104
                 $_hide_notice = '1';
105 105
             }
106 106
 
107
-            $_imgopt_notice_dismissible = apply_filters( 'autoptimize_filter_imgopt_notice_dismissable', $_dismissible . $_hide_notice );
107
+            $_imgopt_notice_dismissible = apply_filters('autoptimize_filter_imgopt_notice_dismissable', $_dismissible.$_hide_notice);
108 108
 
109
-            if ( $_imgopt_notice && PAnD::is_admin_notice_active( $_imgopt_notice_dismissible ) ) {
110
-                echo '<div class="notice notice-warning is-dismissible" data-dismissible="' . $_imgopt_notice_dismissible . '"><p><strong>' . __( 'Autoptimize', 'autoptimize' ) . '</strong>: ' . $_imgopt_notice['notice'] . '</p></div>';
109
+            if ($_imgopt_notice && PAnD::is_admin_notice_active($_imgopt_notice_dismissible)) {
110
+                echo '<div class="notice notice-warning is-dismissible" data-dismissible="'.$_imgopt_notice_dismissible.'"><p><strong>'.__('Autoptimize', 'autoptimize').'</strong>: '.$_imgopt_notice['notice'].'</p></div>';
111 111
             }
112 112
         }
113 113
     }
Please login to merge, or discard this patch.
classes/autoptimizePartners.php 1 patch
Spacing   +31 added lines, -31 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,68 +17,68 @@  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
-        if ( is_multisite() && is_network_admin() && autoptimizeOptionWrapper::is_ao_active_for_network() ) {
24
-            add_action( 'network_admin_menu', array( $this, 'add_admin_menu' ) );
23
+        if (is_multisite() && is_network_admin() && autoptimizeOptionWrapper::is_ao_active_for_network()) {
24
+            add_action('network_admin_menu', array($this, 'add_admin_menu'));
25 25
         } else {
26
-            add_action( 'admin_menu', array( $this, 'add_admin_menu' ) );
26
+            add_action('admin_menu', array($this, 'add_admin_menu'));
27 27
         }
28 28
     }
29 29
 
30 30
     protected function enabled()
31 31
     {
32
-        return apply_filters( 'autoptimize_filter_show_partner_tabs', true );
32
+        return apply_filters('autoptimize_filter_show_partner_tabs', true);
33 33
     }
34 34
 
35
-    public function add_partner_tabs( $in )
35
+    public function add_partner_tabs($in)
36 36
     {
37
-        $in = array_merge( $in, array(
38
-            'ao_partners' => __( 'Optimize More!', 'autoptimize' ),
39
-        ) );
37
+        $in = array_merge($in, array(
38
+            'ao_partners' => __('Optimize More!', 'autoptimize'),
39
+        ));
40 40
 
41 41
         return $in;
42 42
     }
43 43
 
44 44
     public function add_admin_menu()
45 45
     {
46
-        if ( $this->enabled() ) {
47
-            add_submenu_page( null, 'AO partner', 'AO partner', 'manage_options', 'ao_partners', array( $this, 'ao_partners_page' ) );
46
+        if ($this->enabled()) {
47
+            add_submenu_page(null, 'AO partner', 'AO partner', 'manage_options', 'ao_partners', array($this, 'ao_partners_page'));
48 48
         }
49 49
     }
50 50
 
51 51
     protected function get_ao_partner_feed_markup()
52 52
     {
53
-        $no_feed_text = __( 'Have a look at <a href="http://optimizingmatters.com/">optimizingmatters.com</a> for Autoptimize power-ups!', 'autoptimize' );
53
+        $no_feed_text = __('Have a look at <a href="http://optimizingmatters.com/">optimizingmatters.com</a> for Autoptimize power-ups!', 'autoptimize');
54 54
         $output       = '';
55
-        if ( apply_filters( 'autoptimize_settingsscreen_remotehttp', true ) ) {
56
-            $rss      = fetch_feed( 'http://feeds.feedburner.com/OptimizingMattersDownloads' );
55
+        if (apply_filters('autoptimize_settingsscreen_remotehttp', true)) {
56
+            $rss      = fetch_feed('http://feeds.feedburner.com/OptimizingMattersDownloads');
57 57
             $maxitems = 0;
58 58
 
59
-            if ( ! is_wp_error( $rss ) ) {
60
-                $maxitems  = $rss->get_item_quantity( 20 );
61
-                $rss_items = $rss->get_items( 0, $maxitems );
59
+            if (!is_wp_error($rss)) {
60
+                $maxitems  = $rss->get_item_quantity(20);
61
+                $rss_items = $rss->get_items(0, $maxitems);
62 62
             }
63 63
 
64
-            if ( 0 == $maxitems ) {
64
+            if (0 == $maxitems) {
65 65
                 $output .= $no_feed_text;
66 66
             } else {
67 67
                 $output .= '<ul>';
68
-                foreach ( $rss_items as $item ) {
69
-                    $item_url  = esc_url( $item->get_permalink() );
68
+                foreach ($rss_items as $item) {
69
+                    $item_url  = esc_url($item->get_permalink());
70 70
                     $enclosure = $item->get_enclosure();
71 71
 
72 72
                     $output .= '<li class="itemDetail">';
73
-                    $output .= '<h3 class="itemTitle"><a href="' . $item_url . '" target="_blank">' . esc_html( $item->get_title() ) . '</a></h3>';
73
+                    $output .= '<h3 class="itemTitle"><a href="'.$item_url.'" target="_blank">'.esc_html($item->get_title()).'</a></h3>';
74 74
 
75
-                    if ( $enclosure && ( false !== strpos( $enclosure->get_type(), 'image' ) ) ) {
76
-                        $img_url = esc_url( $enclosure->get_link() );
77
-                        $output .= '<div class="itemImage"><a href="' . $item_url . '" target="_blank"><img src="' . $img_url . '"></a></div>';
75
+                    if ($enclosure && (false !== strpos($enclosure->get_type(), 'image'))) {
76
+                        $img_url = esc_url($enclosure->get_link());
77
+                        $output .= '<div class="itemImage"><a href="'.$item_url.'" target="_blank"><img src="'.$img_url.'"></a></div>';
78 78
                     }
79 79
 
80
-                    $output .= '<div class="itemDescription">' . wp_kses_post( $item->get_description() ) . '</div>';
81
-                    $output .= '<div class="itemButtonRow"><div class="itemButton button-secondary"><a href="' . $item_url . '" target="_blank">' . __( 'More info', 'autoptimize' ) . '</a></div></div>';
80
+                    $output .= '<div class="itemDescription">'.wp_kses_post($item->get_description()).'</div>';
81
+                    $output .= '<div class="itemButtonRow"><div class="itemButton button-secondary"><a href="'.$item_url.'" target="_blank">'.__('More info', 'autoptimize').'</a></div></div>';
82 82
                     $output .= '</li>';
83 83
                 }
84 84
                 $output .= '</ul>';
@@ -136,11 +136,11 @@  discard block
 block discarded – undo
136 136
         color: #23282d;
137 137
     }
138 138
     </style>
139
-    <script>document.title = "Autoptimize: <?php _e( 'Optimize More!', 'autoptimize' ); ?> " + document.title;</script>
139
+    <script>document.title = "Autoptimize: <?php _e('Optimize More!', 'autoptimize'); ?> " + document.title;</script>
140 140
     <div class="wrap">
141
-        <h1><?php _e( 'Autoptimize Settings', 'autoptimize' ); ?></h1>
141
+        <h1><?php _e('Autoptimize Settings', 'autoptimize'); ?></h1>
142 142
         <?php echo autoptimizeConfig::ao_admin_tabs(); ?>
143
-        <?php echo '<h2>' . __( "These Autoptimize power-ups and related services will improve your site's performance even more!", 'autoptimize' ) . '</h2>'; ?>
143
+        <?php echo '<h2>'.__("These Autoptimize power-ups and related services will improve your site's performance even more!", 'autoptimize').'</h2>'; ?>
144 144
         <div>
145 145
             <?php echo $this->get_ao_partner_feed_markup(); ?>
146 146
         </div>
Please login to merge, or discard this patch.
config/autoptimize_404_handler.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -20,19 +20,19 @@
 block discarded – undo
20 20
  * to files that don't exist in AO's cache.
21 21
  */
22 22
 
23
-$original_request = strtok( $_SERVER['REQUEST_URI'], '?' );
24
-$fallback_target  = preg_replace( '/(.*)_(?:[a-z0-9]{32})\.(js|css)$/', '${1}_fallback.${2}', $original_request );
23
+$original_request = strtok($_SERVER['REQUEST_URI'], '?');
24
+$fallback_target  = preg_replace('/(.*)_(?:[a-z0-9]{32})\.(js|css)$/', '${1}_fallback.${2}', $original_request);
25 25
 $ao_cache_dir     = '<!--ao-cache-dir-->';
26
-$js_or_css        = pathinfo( $original_request, PATHINFO_EXTENSION );
27
-$fallback_path    = $ao_cache_dir . $js_or_css . '/autoptimize_fallback.' . $js_or_css;
26
+$js_or_css        = pathinfo($original_request, PATHINFO_EXTENSION);
27
+$fallback_path    = $ao_cache_dir.$js_or_css.'/autoptimize_fallback.'.$js_or_css;
28 28
 
29
-if ( $original_request !== $fallback_target && file_exists( $fallback_path ) ) {
29
+if ($original_request !== $fallback_target && file_exists($fallback_path)) {
30 30
     // error_log( 'Autoptimize file ' . $original_request . ' not found, using fallback instead.' );
31
-    header( 'HTTP/1.1 301 Moved Permanently' );
32
-    header( 'Location: ' . $fallback_target );
31
+    header('HTTP/1.1 301 Moved Permanently');
32
+    header('Location: '.$fallback_target);
33 33
 } else {
34 34
     // error_log( 'Autoptimize file ' . $original_request . ' not found, sending 410 gone response.' );
35
-    header( 'HTTP/1.1 410 Gone' );
35
+    header('HTTP/1.1 410 Gone');
36 36
 }
37 37
 
38 38
 exit();
Please login to merge, or discard this patch.
classes/autoptimizeOptionWrapper.php 1 patch
Spacing   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * Autoptimize options handler.
4 4
  */
5 5
 
6
-if ( ! defined( 'ABSPATH' ) ) {
6
+if (!defined('ABSPATH')) {
7 7
     exit;
8 8
 }
9 9
 
@@ -21,8 +21,8 @@  discard block
 block discarded – undo
21 21
      * Ensure that is_plugin_active_for_network function is declared.
22 22
      */
23 23
     public static function maybe_include_plugin_functions() {
24
-        if ( ! function_exists( 'is_plugin_active_for_network' ) ) {
25
-            include_once ABSPATH . 'wp-admin/includes/plugin.php';
24
+        if (!function_exists('is_plugin_active_for_network')) {
25
+            include_once ABSPATH.'wp-admin/includes/plugin.php';
26 26
         }
27 27
     }
28 28
 
@@ -33,19 +33,19 @@  discard block
 block discarded – undo
33 33
      * @param mixed  $default Optional. Default value to return if the option does not exist.
34 34
      * @return mixed Value set for the option.
35 35
      */
36
-    public static function get_option( $option, $default = false ) {
36
+    public static function get_option($option, $default = false) {
37 37
         // This is always a network setting, it is on by default to ensure settings are available at site level unless explicitly turned off.
38
-        if ( 'autoptimize_enable_site_config' === $option ) {
39
-            return get_network_option( get_main_network_id(), $option, 'on' );
38
+        if ('autoptimize_enable_site_config' === $option) {
39
+            return get_network_option(get_main_network_id(), $option, 'on');
40 40
         }
41 41
 
42 42
         // If the plugin is network activated and our per site setting is not on, use the network configuration.
43
-        $configuration_per_site = get_network_option( get_main_network_id(), 'autoptimize_enable_site_config', 'on' );
44
-        if ( self::is_ao_active_for_network() && ( 'on' !== $configuration_per_site || is_network_admin() ) ) {
45
-            return get_network_option( get_main_network_id(), $option, $default );
43
+        $configuration_per_site = get_network_option(get_main_network_id(), 'autoptimize_enable_site_config', 'on');
44
+        if (self::is_ao_active_for_network() && ('on' !== $configuration_per_site || is_network_admin())) {
45
+            return get_network_option(get_main_network_id(), $option, $default);
46 46
         }
47 47
 
48
-        return get_option( $option, $default );
48
+        return get_option($option, $default);
49 49
     }
50 50
 
51 51
     /**
@@ -59,11 +59,11 @@  discard block
 block discarded – undo
59 59
      *                              the default value is 'yes'. Default null.
60 60
      * @return bool False if value was not updated and true if value was updated.
61 61
      */
62
-    public static function update_option( $option, $value, $autoload = null ) {
63
-        if ( self::is_ao_active_for_network() && is_network_admin() ) {
64
-            return update_network_option( get_main_network_id(), $option, $value );
65
-        } elseif ( 'autoptimize_enable_site_config' !== $option ) {
66
-            return update_option( $option, $value, $autoload );
62
+    public static function update_option($option, $value, $autoload = null) {
63
+        if (self::is_ao_active_for_network() && is_network_admin()) {
64
+            return update_network_option(get_main_network_id(), $option, $value);
65
+        } elseif ('autoptimize_enable_site_config' !== $option) {
66
+            return update_option($option, $value, $autoload);
67 67
         }
68 68
     }
69 69
 
@@ -72,8 +72,8 @@  discard block
 block discarded – undo
72 72
      * in that case, take care of multisite case.
73 73
      */
74 74
     public static function check_multisite_on_saving_options() {
75
-        if ( self::is_ao_active_for_network() ) {
76
-            add_filter( 'pre_update_option', 'autoptimizeOptionWrapper::update_autoptimize_option_on_network', 10, 3 );
75
+        if (self::is_ao_active_for_network()) {
76
+            add_filter('pre_update_option', 'autoptimizeOptionWrapper::update_autoptimize_option_on_network', 10, 3);
77 77
         }
78 78
     }
79 79
 
@@ -84,16 +84,16 @@  discard block
 block discarded – undo
84 84
      * @param string $option    Option name.
85 85
      * @param string $old_value Old value.
86 86
      */
87
-    public static function update_autoptimize_option_on_network( $value, $option, $old_value ) {
88
-        if ( strpos( $option, 'autoptimize_' ) === 0 && self::is_options_from_network_admin() ) {
89
-            if ( self::is_ao_active_for_network() ) {
90
-                update_network_option( get_main_network_id(), $option, $value );
87
+    public static function update_autoptimize_option_on_network($value, $option, $old_value) {
88
+        if (strpos($option, 'autoptimize_') === 0 && self::is_options_from_network_admin()) {
89
+            if (self::is_ao_active_for_network()) {
90
+                update_network_option(get_main_network_id(), $option, $value);
91 91
                 // Return old value, to stop update_option logic.
92 92
                 return $old_value;
93 93
             }
94
-            if ( apply_filters( 'autoptimize_filter_optionwrapper_wp_cache_delete', false ) ) {
94
+            if (apply_filters('autoptimize_filter_optionwrapper_wp_cache_delete', false)) {
95 95
                 // in some (rare) cases options seem to get stuck in WP's Object cache, this should clear it there.
96
-                wp_cache_delete( $option );
96
+                wp_cache_delete($option);
97 97
             }
98 98
         }
99 99
         return $value;
@@ -107,8 +107,8 @@  discard block
 block discarded – undo
107 107
     public static function is_options_from_network_admin() {
108 108
         static $_really_is_network_admin = null;
109 109
 
110
-        if ( null === $_really_is_network_admin ) {
111
-            if ( array_key_exists( '_wp_http_referer', $_POST ) && strpos( network_admin_url( 'settings.php' ), strtok( $_POST['_wp_http_referer'], '?' ) ) !== false ) {
110
+        if (null === $_really_is_network_admin) {
111
+            if (array_key_exists('_wp_http_referer', $_POST) && strpos(network_admin_url('settings.php'), strtok($_POST['_wp_http_referer'], '?')) !== false) {
112 112
                 $_really_is_network_admin = true;
113 113
             } else {
114 114
                 $_really_is_network_admin = false;
@@ -123,9 +123,9 @@  discard block
 block discarded – undo
123 123
      */
124 124
     public static function is_ao_active_for_network() {
125 125
         static $_is_ao_active_for_network = null;
126
-        if ( null === $_is_ao_active_for_network || defined( 'TEST_MULTISITE_FORCE_AO_ON_NETWORK' ) ) {
126
+        if (null === $_is_ao_active_for_network || defined('TEST_MULTISITE_FORCE_AO_ON_NETWORK')) {
127 127
             self::maybe_include_plugin_functions();
128
-            if ( is_plugin_active_for_network( 'autoptimize/autoptimize.php' ) || is_plugin_active_for_network( 'autoptimize-beta/autoptimize.php' ) || defined( 'TEST_MULTISITE_FORCE_AO_ON_NETWORK' ) ) {
128
+            if (is_plugin_active_for_network('autoptimize/autoptimize.php') || is_plugin_active_for_network('autoptimize-beta/autoptimize.php') || defined('TEST_MULTISITE_FORCE_AO_ON_NETWORK')) {
129 129
                 $_is_ao_active_for_network = true;
130 130
             } else {
131 131
                 $_is_ao_active_for_network = false;
Please login to merge, or discard this patch.
tests/test-ao.php 1 patch
Spacing   +450 added lines, -450 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,17 +28,17 @@  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' ),
41
-            'minify_excluded' => $conf->get( 'autoptimize_minify_excluded' ),
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
+            'minify_excluded' => $conf->get('autoptimize_minify_excluded'),
42 42
         ];
43 43
     }
44 44
 
@@ -47,14 +47,14 @@  discard block
 block discarded – undo
47 47
         $conf = autoptimizeConfig::instance();
48 48
 
49 49
         return [
50
-            'aggregate'       => $conf->get( 'autoptimize_js_aggregate' ),
51
-            'justhead'        => $conf->get( 'autoptimize_js_justhead' ),
52
-            'forcehead'       => $conf->get( 'autoptimize_js_forcehead' ),
53
-            'trycatch'        => $conf->get( 'autoptimize_js_trycatch' ),
54
-            'js_exclude'      => $conf->get( 'autoptimize_js_exclude' ),
55
-            'cdn_url'         => $conf->get( 'autoptimize_cdn_url' ),
56
-            'include_inline'  => $conf->get( 'autoptimize_js_include_inline' ),
57
-            'minify_excluded' => $conf->get( 'autoptimize_minify_excluded' ),
50
+            'aggregate'       => $conf->get('autoptimize_js_aggregate'),
51
+            'justhead'        => $conf->get('autoptimize_js_justhead'),
52
+            'forcehead'       => $conf->get('autoptimize_js_forcehead'),
53
+            'trycatch'        => $conf->get('autoptimize_js_trycatch'),
54
+            'js_exclude'      => $conf->get('autoptimize_js_exclude'),
55
+            'cdn_url'         => $conf->get('autoptimize_cdn_url'),
56
+            'include_inline'  => $conf->get('autoptimize_js_include_inline'),
57
+            'minify_excluded' => $conf->get('autoptimize_minify_excluded'),
58 58
         ];
59 59
     }
60 60
 
@@ -64,35 +64,35 @@  discard block
 block discarded – undo
64 64
     protected function get_urls()
65 65
     {
66 66
         static $site_url = null;
67
-        if ( null === $site_url ) {
67
+        if (null === $site_url) {
68 68
             $site_url = site_url();
69 69
         }
70 70
 
71 71
         static $cdn_url = null;
72
-        if ( null === $cdn_url ) {
73
-            $cdn_url = autoptimizeOptionWrapper::get_option( 'autoptimize_cdn_url' );
72
+        if (null === $cdn_url) {
73
+            $cdn_url = autoptimizeOptionWrapper::get_option('autoptimize_cdn_url');
74 74
         }
75 75
 
76 76
         static $imgopt_host = null;
77
-        if ( null === $imgopt_host ) {
77
+        if (null === $imgopt_host) {
78 78
             $optimizer   = new autoptimizeImages();
79
-            $imgopt_host = rtrim( $optimizer->get_imgopt_host(), '/' );
79
+            $imgopt_host = rtrim($optimizer->get_imgopt_host(), '/');
80 80
         }
81 81
 
82 82
         static $urls = [];
83 83
 
84
-        if ( empty( $urls ) ) {
84
+        if (empty($urls)) {
85 85
             $parts = autoptimizeUtils::get_ao_wp_site_url_parts();
86 86
             $urls  = [
87 87
                 'siteurl'    => $site_url,
88
-                'prsiteurl'  => '//' . str_replace( array( 'http://', 'https://' ), '', $site_url ),
89
-                'wwwsiteurl' => $parts['scheme'] . '://www.' . str_replace( 'www.', '', $parts['host'] ),
88
+                'prsiteurl'  => '//'.str_replace(array('http://', 'https://'), '', $site_url),
89
+                'wwwsiteurl' => $parts['scheme'].'://www.'.str_replace('www.', '', $parts['host']),
90 90
                 'cdnurl'     => $cdn_url,
91 91
                 'imgopthost' => $imgopt_host,
92 92
                 'subfolder'  => '',
93 93
             ];
94 94
 
95
-            if ( AO_TEST_SUBFOLDER_INSTALL ) {
95
+            if (AO_TEST_SUBFOLDER_INSTALL) {
96 96
                 $urls['subfolder'] = 'wordpress/';
97 97
             }
98 98
         }
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
      */
106 106
     public function setUp()
107 107
     {
108
-        $this->ao = new autoptimizeMain( AUTOPTIMIZE_PLUGIN_VERSION, AUTOPTIMIZE_PLUGIN_FILE );
108
+        $this->ao = new autoptimizeMain(AUTOPTIMIZE_PLUGIN_VERSION, AUTOPTIMIZE_PLUGIN_FILE);
109 109
 
110 110
         parent::setUp();
111 111
     }
@@ -124,8 +124,8 @@  discard block
 block discarded – undo
124 124
             'autoptimize_filter_css_inlinesize',
125 125
             'autoptimize_filter_css_fonts_cdn',
126 126
         );
127
-        foreach ( $filter_tags as $filter_tag ) {
128
-            remove_all_filters( $filter_tag );
127
+        foreach ($filter_tags as $filter_tag) {
128
+            remove_all_filters($filter_tag);
129 129
         }
130 130
 
131 131
         parent::tearDown();
@@ -249,20 +249,20 @@  discard block
 block discarded – undo
249 249
     protected function get_test_markup_output()
250 250
     {
251 251
         $key = 'default';
252
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
252
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
253 253
             $key = 'custom';
254 254
         }
255 255
 
256
-        $csspart = self::$flexible_url_parts_css[ $key ];
257
-        $jspart  = self::$flexible_url_parts_js[ $key ];
256
+        $csspart = self::$flexible_url_parts_css[$key];
257
+        $jspart  = self::$flexible_url_parts_js[$key];
258 258
 
259 259
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
260
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
260
+        if (AO_TEST_SUBFOLDER_INSTALL) {
261 261
             $csshash = '56398de576d59887e88e3011715250e0';
262 262
         }
263 263
 
264 264
         $urls      = $this->get_urls();
265
-            $siteurl   = $urls['siteurl'];
265
+            $siteurl = $urls['siteurl'];
266 266
         $cdnurl    = $urls['cdnurl'];
267 267
         $subfolder = $urls['subfolder'];
268 268
 
@@ -316,15 +316,15 @@  discard block
 block discarded – undo
316 316
     protected function get_test_markup_output_ms()
317 317
     {
318 318
         $key = 'multisite';
319
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
319
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
320 320
             $key = 'multisite_custom';
321 321
         }
322 322
 
323
-        $csspart = self::$flexible_url_parts_css[ $key ];
324
-        $jspart  = self::$flexible_url_parts_js[ $key ];
323
+        $csspart = self::$flexible_url_parts_css[$key];
324
+        $jspart  = self::$flexible_url_parts_js[$key];
325 325
 
326 326
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
327
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
327
+        if (AO_TEST_SUBFOLDER_INSTALL) {
328 328
             $csshash = '56398de576d59887e88e3011715250e0';
329 329
         }
330 330
 
@@ -383,15 +383,15 @@  discard block
 block discarded – undo
383 383
     protected function get_test_markup_output_inline_defer()
384 384
     {
385 385
         $key = 'default';
386
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
386
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
387 387
             $key = 'custom';
388 388
         }
389 389
 
390
-        $csspart = self::$flexible_url_parts_css[ $key ];
391
-        $jspart  = self::$flexible_url_parts_js[ $key ];
390
+        $csspart = self::$flexible_url_parts_css[$key];
391
+        $jspart  = self::$flexible_url_parts_js[$key];
392 392
 
393 393
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
394
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
394
+        if (AO_TEST_SUBFOLDER_INSTALL) {
395 395
             $csshash = '56398de576d59887e88e3011715250e0';
396 396
         }
397 397
 
@@ -450,15 +450,15 @@  discard block
 block discarded – undo
450 450
     protected function get_test_markup_output_inline_defer_ms()
451 451
     {
452 452
         $key = 'multisite';
453
-        if ( defined( 'CUSTOM_CONSTANTS_USED' ) && CUSTOM_CONSTANTS_USED ) {
453
+        if (defined('CUSTOM_CONSTANTS_USED') && CUSTOM_CONSTANTS_USED) {
454 454
             $key = 'multisite_custom';
455 455
         }
456 456
 
457
-        $csspart = self::$flexible_url_parts_css[ $key ];
458
-        $jspart  = self::$flexible_url_parts_js[ $key ];
457
+        $csspart = self::$flexible_url_parts_css[$key];
458
+        $jspart  = self::$flexible_url_parts_js[$key];
459 459
 
460 460
         $csshash = '863f587e89f100b0223ddccc0dabc57a';
461
-        if ( AO_TEST_SUBFOLDER_INSTALL ) {
461
+        if (AO_TEST_SUBFOLDER_INSTALL) {
462 462
             $csshash = '56398de576d59887e88e3011715250e0';
463 463
         }
464 464
 
@@ -517,11 +517,11 @@  discard block
 block discarded – undo
517 517
     /**
518 518
      * @dataProvider provider_test_rewrite_markup_with_cdn
519 519
      */
520
-    function test_rewrite_markup_with_cdn( $input, $expected )
520
+    function test_rewrite_markup_with_cdn($input, $expected)
521 521
     {
522
-        $actual = $this->ao->end_buffering( $input );
522
+        $actual = $this->ao->end_buffering($input);
523 523
 
524
-        $this->assertEquals( $expected, $actual );
524
+        $this->assertEquals($expected, $actual);
525 525
     }
526 526
 
527 527
     public function provider_test_rewrite_markup_with_cdn()
@@ -533,7 +533,7 @@  discard block
 block discarded – undo
533 533
                 $this->get_test_markup(),
534 534
                 // Expected output.
535 535
                 // TODO/FIXME: This seemed like the fastest way to get MS crude test to pass...
536
-                ( is_multisite() ? $this->get_test_markup_output_ms() : $this->get_test_markup_output() ),
536
+                (is_multisite() ? $this->get_test_markup_output_ms() : $this->get_test_markup_output()),
537 537
             ),
538 538
 
539 539
         );
@@ -593,12 +593,12 @@  discard block
 block discarded – undo
593 593
 }
594 594
 CSS;
595 595
 
596
-        $instance = new autoptimizeStyles( $css_in );
597
-        $instance->setOption( 'cdn_url', $cdnurl );
596
+        $instance = new autoptimizeStyles($css_in);
597
+        $instance->setOption('cdn_url', $cdnurl);
598 598
 
599
-        $css_actual = $instance->rewrite_assets( $css_in );
599
+        $css_actual = $instance->rewrite_assets($css_in);
600 600
 
601
-        $this->assertEquals( $css_expected, $css_actual );
601
+        $this->assertEquals($css_expected, $css_actual);
602 602
     }
603 603
 
604 604
     public function test_default_cssmin_minifier()
@@ -632,21 +632,21 @@  discard block
 block discarded – undo
632 632
 .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')}
633 633
 CSS;
634 634
 
635
-        $instance = new autoptimizeStyles( $css );
636
-        $minified = $instance->run_minifier_on( $css );
635
+        $instance = new autoptimizeStyles($css);
636
+        $minified = $instance->run_minifier_on($css);
637 637
 
638
-        $this->assertEquals( $expected, $minified );
638
+        $this->assertEquals($expected, $minified);
639 639
     }
640 640
 
641 641
     /**
642 642
      * @dataProvider provider_test_should_aggregate_script_types
643 643
      */
644
-    public function test_should_aggregate_script_types( $input, $expected )
644
+    public function test_should_aggregate_script_types($input, $expected)
645 645
     {
646
-        $instance = new autoptimizeScripts( '' );
647
-        $actual   = $instance->should_aggregate( $input );
646
+        $instance = new autoptimizeScripts('');
647
+        $actual   = $instance->should_aggregate($input);
648 648
 
649
-        $this->assertEquals( $expected, $actual );
649
+        $this->assertEquals($expected, $actual);
650 650
     }
651 651
 
652 652
     public function provider_test_should_aggregate_script_types()
@@ -708,11 +708,11 @@  discard block
 block discarded – undo
708 708
     /**
709 709
      * @dataProvider provider_is_valid_buffer
710 710
      */
711
-    public function test_valid_buffers( $input, $expected )
711
+    public function test_valid_buffers($input, $expected)
712 712
     {
713
-        $actual = $this->ao->is_valid_buffer( $input );
713
+        $actual = $this->ao->is_valid_buffer($input);
714 714
 
715
-        $this->assertEquals( $expected, $actual );
715
+        $this->assertEquals($expected, $actual);
716 716
     }
717 717
 
718 718
     public function provider_is_valid_buffer()
@@ -781,11 +781,11 @@  discard block
 block discarded – undo
781 781
     /**
782 782
      * @dataProvider provider_is_amp_markup
783 783
      */
784
-    public function test_autoptimize_is_amp_markup( $input, $expected )
784
+    public function test_autoptimize_is_amp_markup($input, $expected)
785 785
     {
786
-        $actual = autoptimizeMain::is_amp_markup( $input );
786
+        $actual = autoptimizeMain::is_amp_markup($input);
787 787
 
788
-        $this->assertEquals( $expected, $actual );
788
+        $this->assertEquals($expected, $actual);
789 789
     }
790 790
 
791 791
     public function provider_is_amp_markup()
@@ -816,31 +816,31 @@  discard block
 block discarded – undo
816 816
     public function test_skips_buffering_when_ao_noptimize_filter_is_true()
817 817
     {
818 818
         // True => disable autoptimize.
819
-        add_filter( 'autoptimize_filter_noptimize', '__return_true' );
819
+        add_filter('autoptimize_filter_noptimize', '__return_true');
820 820
 
821 821
         // Buffering should not run due to the above filter.
822 822
         $expected = false;
823
-        $actual   = $this->ao->should_buffer( $doing_tests = true );
823
+        $actual   = $this->ao->should_buffer($doing_tests = true);
824 824
 
825
-        $this->assertEquals( $expected, $actual );
825
+        $this->assertEquals($expected, $actual);
826 826
     }
827 827
 
828 828
     public function test_does_buffering_when_ao_noptimize_filter_is_false()
829 829
     {
830 830
         // False => disable noptimize, aka, run normally (weird, yes...).
831
-        add_filter( 'autoptimize_filter_noptimize', '__return_false' );
831
+        add_filter('autoptimize_filter_noptimize', '__return_false');
832 832
 
833 833
         // Buffering should run because of above.
834 834
         $expected = true;
835
-        $actual   = $this->ao->should_buffer( $doing_tests = true );
835
+        $actual   = $this->ao->should_buffer($doing_tests = true);
836 836
 
837
-        $this->assertEquals( $expected, $actual );
837
+        $this->assertEquals($expected, $actual);
838 838
     }
839 839
 
840 840
     public function test_ignores_ao_noptimize_qs_when_instructed()
841 841
     {
842 842
         // Should skip checking for the qs completely due to filter.
843
-        add_filter( 'autoptimize_filter_honor_qs_noptimize', '__return_false' );
843
+        add_filter('autoptimize_filter_honor_qs_noptimize', '__return_false');
844 844
 
845 845
         /**
846 846
          * The above should then result in the "current" value being `false`
@@ -851,23 +851,23 @@  discard block
 block discarded – undo
851 851
          */
852 852
 
853 853
         $that = $this; // Makes it work on php 5.3!
854
-        add_filter( 'autoptimize_filter_noptimize', function ( $current_value ) use ( $that ) {
854
+        add_filter('autoptimize_filter_noptimize', function($current_value) use ($that) {
855 855
             $expected = false;
856
-            if ( defined( 'DONOTMINIFY' ) && DONOTMINIFY ) {
856
+            if (defined('DONOTMINIFY') && DONOTMINIFY) {
857 857
                 $expected = true;
858 858
             }
859 859
 
860
-            $that->assertEquals( $expected, $current_value );
860
+            $that->assertEquals($expected, $current_value);
861 861
         });
862 862
 
863
-        $this->ao->should_buffer( $doing_tests = true );
863
+        $this->ao->should_buffer($doing_tests = true);
864 864
     }
865 865
 
866 866
     public function test_wpengine_cache_flush()
867 867
     {
868 868
         // Creating a mock so that we can get past class_exists() and method_exists() checks present
869 869
         // in `autoptimizeCache::flushPageCache()`...
870
-        $stub = $this->getMockBuilder( 'WpeCommon' )->disableAutoload()
870
+        $stub = $this->getMockBuilder('WpeCommon')->disableAutoload()
871 871
                 ->disableOriginalConstructor()->setMethods(
872 872
                     array(
873 873
                         'purge_varnish_cache',
@@ -876,9 +876,9 @@  discard block
 block discarded – undo
876 876
                 ->getMock();
877 877
 
878 878
         $that = $this;
879
-        add_filter( 'autoptimize_flush_wpengine_methods', function( $methods ) use ( $that ) {
880
-            $expected_methods = array( 'purge_varnish_cache' );
881
-            $that->assertEquals( $methods, $expected_methods );
879
+        add_filter('autoptimize_flush_wpengine_methods', function($methods) use ($that) {
880
+            $expected_methods = array('purge_varnish_cache');
881
+            $that->assertEquals($methods, $expected_methods);
882 882
 
883 883
             return $methods;
884 884
         });
@@ -892,7 +892,7 @@  discard block
 block discarded – undo
892 892
     public function test_wpengine_cache_flush_agressive()
893 893
     {
894 894
         // Creating a mock so that we can get past class_exists() and method_exists() checks `autoptimize_flush_pagecache()`...
895
-        $stub = $this->getMockBuilder( 'WpeCommon' )->disableAutoload()
895
+        $stub = $this->getMockBuilder('WpeCommon')->disableAutoload()
896 896
                 ->disableOriginalConstructor()->setMethods(
897 897
                     array(
898 898
                         'purge_varnish_cache',
@@ -902,19 +902,19 @@  discard block
 block discarded – undo
902 902
                 )
903 903
                 ->getMock();
904 904
 
905
-        add_filter( 'autoptimize_flush_wpengine_aggressive', function() {
905
+        add_filter('autoptimize_flush_wpengine_aggressive', function() {
906 906
             return true;
907 907
         });
908 908
 
909 909
         $that = $this;
910
-        add_filter( 'autoptimize_flush_wpengine_methods', function( $methods ) use ( $that ) {
910
+        add_filter('autoptimize_flush_wpengine_methods', function($methods) use ($that) {
911 911
             $expected_methods = array(
912 912
                 'purge_varnish_cache',
913 913
                 'purge_memcached',
914 914
                 'clear_maxcdn_cache',
915 915
             );
916 916
 
917
-            $that->assertEquals( $methods, $expected_methods );
917
+            $that->assertEquals($methods, $expected_methods);
918 918
 
919 919
             return $methods;
920 920
         });
@@ -925,15 +925,15 @@  discard block
 block discarded – undo
925 925
     /**
926 926
      * @dataProvider provider_test_url_replace_cdn
927 927
      */
928
-    public function test_url_replace_cdn( $cdn_url, $input, $expected )
928
+    public function test_url_replace_cdn($cdn_url, $input, $expected)
929 929
     {
930
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )
930
+        $mock = $this->getMockBuilder('autoptimizeBase')
931 931
                 ->disableOriginalConstructor()
932 932
                 ->getMockForAbstractClass();
933 933
 
934 934
         $mock->cdn_url = $cdn_url;
935
-        $actual        = $mock->url_replace_cdn( $input );
936
-        $this->assertEquals( $expected, $actual );
935
+        $actual        = $mock->url_replace_cdn($input);
936
+        $this->assertEquals($expected, $actual);
937 937
     }
938 938
 
939 939
     public function provider_test_url_replace_cdn()
@@ -950,13 +950,13 @@  discard block
 block discarded – undo
950 950
                 // CDN base url, url, expected result...
951 951
                 'http://cdn-test.example.org',
952 952
                 '/a.jpg',
953
-                'http://cdn-test.example.org/' . $subfolder . 'a.jpg',
953
+                'http://cdn-test.example.org/'.$subfolder.'a.jpg',
954 954
             ),
955 955
             // Full link with a matching AUTOPTIMIZE_WP_SITE_URL gets properly replaced...
956 956
             array(
957 957
                 'http://cdn-test.example.org',
958
-                $siteurl . '/wp-content/themes/something/example.svg',
959
-                'http://cdn-test.example.org/' . $subfolder . 'wp-content/themes/something/example.svg',
958
+                $siteurl.'/wp-content/themes/something/example.svg',
959
+                'http://cdn-test.example.org/'.$subfolder.'wp-content/themes/something/example.svg',
960 960
             ),
961 961
             // Protocol-relative url with a "local" hostname that doesn't match example.org (AUTOPTIMIZE_WP_SITE_URL)...
962 962
             array(
@@ -967,59 +967,59 @@  discard block
 block discarded – undo
967 967
             // www.example.org does not match example.org (AUTOPTIMIZE_WP_SITE_URL) so it's left alone...
968 968
             array(
969 969
                 'http://cdn-test.example.org',
970
-                $wwwsiteurl . '/wp-content/themes/something/example.svg',
971
-                $wwwsiteurl . '/wp-content/themes/something/example.svg',
970
+                $wwwsiteurl.'/wp-content/themes/something/example.svg',
971
+                $wwwsiteurl.'/wp-content/themes/something/example.svg',
972 972
             ),
973 973
             // SSL cdn url + host-relative link...
974 974
             array(
975 975
                 'https://cdn.example.org',
976 976
                 '/a.jpg',
977
-                'https://cdn.example.org/' . $subfolder . 'a.jpg',
977
+                'https://cdn.example.org/'.$subfolder.'a.jpg',
978 978
             ),
979 979
             // SSL cdn url + http site url that matches AUTOPTIMIZE_WP_SITE_URL is properly replaced...
980 980
             array(
981 981
                 'https://cdn.example.org',
982
-                $siteurl . '/wp-content/themes/something/example.svg',
983
-                'https://cdn.example.org/' . $subfolder . 'wp-content/themes/something/example.svg',
982
+                $siteurl.'/wp-content/themes/something/example.svg',
983
+                'https://cdn.example.org/'.$subfolder.'wp-content/themes/something/example.svg',
984 984
             ),
985 985
             // Protocol-relative cdn url given with protocol relative link that matches AUTOPTIMIZE_WP_SITE_URL host...
986 986
             array(
987 987
                 '//cdn.example.org',
988
-                $protorelativesiteurl . '/something.jpg',
989
-                '//cdn.example.org/' . $subfolder . 'something.jpg',
988
+                $protorelativesiteurl.'/something.jpg',
989
+                '//cdn.example.org/'.$subfolder.'something.jpg',
990 990
             ),
991 991
             // Protocol-relative cdn url given a http link that matches AUTOPTIMIZE_WP_SITE_URL host...
992 992
             array(
993 993
                 '//cdn.example.org',
994
-                $siteurl . '/something.png',
995
-                '//cdn.example.org/' . $subfolder . 'something.png',
994
+                $siteurl.'/something.png',
995
+                '//cdn.example.org/'.$subfolder.'something.png',
996 996
             ),
997 997
             // Protocol-relative cdn url with a host-relative link...
998 998
             array(
999 999
                 '//cdn.example.org',
1000 1000
                 '/a.jpg',
1001
-                '//cdn.example.org/' . $subfolder . 'a.jpg',
1001
+                '//cdn.example.org/'.$subfolder.'a.jpg',
1002 1002
             ),
1003 1003
             // Testing cdn urls with an explicit port number...
1004 1004
             array(
1005 1005
                 'http://cdn.com:8080',
1006 1006
                 '/a.jpg',
1007
-                'http://cdn.com:8080/' . $subfolder . 'a.jpg',
1007
+                'http://cdn.com:8080/'.$subfolder.'a.jpg',
1008 1008
             ),
1009 1009
             array(
1010 1010
                 '//cdn.com:4433',
1011 1011
                 '/a.jpg',
1012
-                '//cdn.com:4433/' . $subfolder . 'a.jpg',
1012
+                '//cdn.com:4433/'.$subfolder.'a.jpg',
1013 1013
             ),
1014 1014
             array(
1015 1015
                 '//cdn.com:4433',
1016
-                $siteurl . '/something.jpg',
1017
-                '//cdn.com:4433/' . $subfolder . 'something.jpg',
1016
+                $siteurl.'/something.jpg',
1017
+                '//cdn.com:4433/'.$subfolder.'something.jpg',
1018 1018
             ),
1019 1019
             array(
1020 1020
                 '//cdn.com:1234',
1021
-                $protorelativesiteurl . '/something.jpg',
1022
-                '//cdn.com:1234/' . $subfolder . 'something.jpg',
1021
+                $protorelativesiteurl.'/something.jpg',
1022
+                '//cdn.com:1234/'.$subfolder.'something.jpg',
1023 1023
             ),
1024 1024
             // Relative links should not be touched by url_replace_cdn()...
1025 1025
             array(
@@ -1044,8 +1044,8 @@  discard block
 block discarded – undo
1044 1044
             // even though they are really strange...
1045 1045
             array(
1046 1046
                 $siteurl, // example.org or http://localhost or http://localhost/wordpress
1047
-                $siteurl . '/something.jpg',
1048
-                $siteurl . '/something.jpg',
1047
+                $siteurl.'/something.jpg',
1048
+                $siteurl.'/something.jpg',
1049 1049
             ),
1050 1050
             // These shouldn't really be changed, or even if replacements do
1051 1051
             // happen, they shouldn't be destructive...
@@ -1084,31 +1084,31 @@  discard block
 block discarded – undo
1084 1084
         $test_link = '/a.jpg';
1085 1085
         $cdn_url   = '//cdn.example.org';
1086 1086
 
1087
-        $with_ssl = function( $cdn ) {
1087
+        $with_ssl = function($cdn) {
1088 1088
             return '';
1089 1089
         };
1090 1090
         $expected_with_ssl = '/a.jpg';
1091 1091
 
1092
-        $without_ssl = function( $cdn ) {
1092
+        $without_ssl = function($cdn) {
1093 1093
             return $cdn;
1094 1094
         };
1095
-        $expected_without_ssl = '//cdn.example.org/' . $sub . 'a.jpg';
1095
+        $expected_without_ssl = '//cdn.example.org/'.$sub.'a.jpg';
1096 1096
 
1097 1097
         // With a filter that returns something considered "empty", cdn replacement shouldn't occur...
1098
-        add_filter( 'autoptimize_filter_base_cdnurl', $with_ssl );
1099
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )
1098
+        add_filter('autoptimize_filter_base_cdnurl', $with_ssl);
1099
+        $mock = $this->getMockBuilder('autoptimizeBase')
1100 1100
                 ->disableOriginalConstructor()
1101 1101
                 ->getMockForAbstractClass();
1102 1102
 
1103 1103
         $mock->cdn_url   = $cdn_url;
1104
-        $actual_with_ssl = $mock->url_replace_cdn( $test_link );
1105
-        $this->assertEquals( $expected_with_ssl, $actual_with_ssl );
1106
-        remove_filter( 'autoptimize_filter_base_cdnurl', $with_ssl );
1104
+        $actual_with_ssl = $mock->url_replace_cdn($test_link);
1105
+        $this->assertEquals($expected_with_ssl, $actual_with_ssl);
1106
+        remove_filter('autoptimize_filter_base_cdnurl', $with_ssl);
1107 1107
 
1108 1108
         // With a filter that returns an actual cdn url, cdn replacement should occur.
1109
-        add_filter( 'autoptimize_filter_base_cdnurl', $without_ssl );
1110
-        $actual_without_ssl = $mock->url_replace_cdn( $test_link );
1111
-        $this->assertEquals( $expected_without_ssl, $actual_without_ssl );
1109
+        add_filter('autoptimize_filter_base_cdnurl', $without_ssl);
1110
+        $actual_without_ssl = $mock->url_replace_cdn($test_link);
1111
+        $this->assertEquals($expected_without_ssl, $actual_without_ssl);
1112 1112
     }
1113 1113
 
1114 1114
     public function provider_cssmin_issues()
@@ -1147,11 +1147,11 @@  discard block
 block discarded – undo
1147 1147
     /**
1148 1148
      * @dataProvider provider_cssmin_issues
1149 1149
      */
1150
-    public function test_cssmin_issues( $input, $expected )
1150
+    public function test_cssmin_issues($input, $expected)
1151 1151
     {
1152
-        $minifier = new autoptimizeCSSmin( false ); // No need to raise limits for now.
1153
-        $actual   = $minifier->run( $input );
1154
-        $this->assertEquals( $expected, $actual );
1152
+        $minifier = new autoptimizeCSSmin(false); // No need to raise limits for now.
1153
+        $actual   = $minifier->run($input);
1154
+        $this->assertEquals($expected, $actual);
1155 1155
     }
1156 1156
 
1157 1157
     public function provider_getpath()
@@ -1176,7 +1176,7 @@  discard block
 block discarded – undo
1176 1176
             // This file comes with core, so should exist...
1177 1177
             array(
1178 1178
                 '/wp-includes/js/jquery/jquery.js',
1179
-                WP_ROOT_DIR . '/wp-includes/js/jquery/jquery.js',
1179
+                WP_ROOT_DIR.'/wp-includes/js/jquery/jquery.js',
1180 1180
             ),
1181 1181
             // Empty $url should return false.
1182 1182
             array(
@@ -1201,12 +1201,12 @@  discard block
 block discarded – undo
1201 1201
     /**
1202 1202
      * @dataProvider provider_getpath
1203 1203
      */
1204
-    public function test_getpath( $input, $expected )
1204
+    public function test_getpath($input, $expected)
1205 1205
     {
1206
-        $mock = $this->getMockBuilder( 'autoptimizeBase' )->disableOriginalConstructor()->getMockForAbstractClass();
1206
+        $mock = $this->getMockBuilder('autoptimizeBase')->disableOriginalConstructor()->getMockForAbstractClass();
1207 1207
 
1208
-        $actual = $mock->getpath( $input );
1209
-        $this->assertEquals( $expected, $actual );
1208
+        $actual = $mock->getpath($input);
1209
+        $this->assertEquals($expected, $actual);
1210 1210
     }
1211 1211
 
1212 1212
     /**
@@ -1220,7 +1220,7 @@  discard block
 block discarded – undo
1220 1220
 
1221 1221
         $protorelativesiteurl = $urls['prsiteurl'];
1222 1222
 
1223
-        $css_orig     = <<<CSS
1223
+        $css_orig = <<<CSS
1224 1224
 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")}
1225 1225
 section.clipped.clippedTop {clip-path:url("#clipPolygonTop")}
1226 1226
 section.clipped.clippedBottom {clip-path:url("#clipPolygonBottom")}
@@ -1233,13 +1233,13 @@  discard block
 block discarded – undo
1233 1233
 .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);}
1234 1234
 CSS;
1235 1235
 
1236
-        $fixurls_result = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $css_orig );
1237
-        $this->assertEquals( $css_expected, $fixurls_result );
1236
+        $fixurls_result = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $css_orig);
1237
+        $this->assertEquals($css_expected, $fixurls_result);
1238 1238
     }
1239 1239
 
1240 1240
     public function test_background_datauri_sprites_with_fixurls()
1241 1241
     {
1242
-        $css_orig     = <<<CSS
1242
+        $css_orig = <<<CSS
1243 1243
 .shadow { background:url(img/1x1.png) top center; }
1244 1244
 .shadow1 { background-image:url(img/1x1.png) 0 -767px repeat-x; }
1245 1245
 .shadow2 {background:url(img/1x1.png) top center}
@@ -1291,27 +1291,27 @@  discard block
 block discarded – undo
1291 1291
 CSS;
1292 1292
 
1293 1293
         // For test purposes, ALL images in the css are being inline with a 1x1 trans png string/datauri.
1294
-        add_filter( 'autoptimize_filter_css_is_datauri_candidate', function( $is_candidate, $path ) {
1294
+        add_filter('autoptimize_filter_css_is_datauri_candidate', function($is_candidate, $path) {
1295 1295
             return true;
1296
-        }, 10, 2 );
1296
+        }, 10, 2);
1297 1297
 
1298 1298
         // For test purposes, ALL images in the css are being inline with a 1x1 trans png string/datauri.
1299
-        add_filter( 'autoptimize_filter_css_datauri_image', function( $base64array, $path ) {
1299
+        add_filter('autoptimize_filter_css_datauri_image', function($base64array, $path) {
1300 1300
             $head = 'data:image/png;base64,';
1301 1301
             $data = 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAQAAAC1HAwCAAAAC0lEQVR42mNkYAAAAAYAAjCB0C8AAAAASUVORK5CYII=';
1302 1302
 
1303
-            $result['full']       = $head . $data;
1303
+            $result['full']       = $head.$data;
1304 1304
             $result['base64data'] = $data;
1305 1305
             return $result;
1306
-        }, 10, 2 );
1306
+        }, 10, 2);
1307 1307
 
1308
-        $instance = new autoptimizeStyles( $css_orig );
1309
-        $instance->setOption( 'datauris', true );
1308
+        $instance = new autoptimizeStyles($css_orig);
1309
+        $instance->setOption('datauris', true);
1310 1310
 
1311
-        $fixurls_result = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $css_orig );
1312
-        $css_actual     = $instance->rewrite_assets( $fixurls_result );
1311
+        $fixurls_result = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $css_orig);
1312
+        $css_actual     = $instance->rewrite_assets($fixurls_result);
1313 1313
 
1314
-        $this->assertEquals( $css_expected, $css_actual );
1314
+        $this->assertEquals($css_expected, $css_actual);
1315 1315
     }
1316 1316
 
1317 1317
     /**
@@ -1331,7 +1331,7 @@  discard block
 block discarded – undo
1331 1331
      */
1332 1332
     public function test_background_datauri_sprites_without_fixurls()
1333 1333
     {
1334
-        $css_orig     = <<<CSS
1334
+        $css_orig = <<<CSS
1335 1335
 .shadow { background:url(img/1x1.png) top center; }
1336 1336
 .shadow1 { background-image:url(img/1x1.png) 0 -767px repeat-x; }
1337 1337
 .shadow2 {background:url(img/1x1.png) top center}
@@ -1375,24 +1375,24 @@  discard block
 block discarded – undo
1375 1375
 CSS;
1376 1376
 
1377 1377
         // For test purposes, ALL images in the css are being inlined with a 1x1 trans png string/datauri.
1378
-        add_filter( 'autoptimize_filter_css_is_datauri_candidate', function( $is_candidate, $path ) {
1378
+        add_filter('autoptimize_filter_css_is_datauri_candidate', function($is_candidate, $path) {
1379 1379
             return true;
1380
-        }, 10, 2 );
1380
+        }, 10, 2);
1381 1381
 
1382 1382
         // For test purposes, ALL images in the css are being inlined with a 1x1 trans png string/datauri.
1383
-        add_filter( 'autoptimize_filter_css_datauri_image', function( $base64array, $path ) {
1383
+        add_filter('autoptimize_filter_css_datauri_image', function($base64array, $path) {
1384 1384
             $head = 'data:image/png;base64,';
1385 1385
             $data = 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAQAAAC1HAwCAAAAC0lEQVR42mNkYAAAAAYAAjCB0C8AAAAASUVORK5CYII=';
1386 1386
 
1387
-            $result['full']       = $head . $data;
1387
+            $result['full']       = $head.$data;
1388 1388
             $result['base64data'] = $data;
1389 1389
             return $result;
1390
-        }, 10, 2 );
1390
+        }, 10, 2);
1391 1391
 
1392
-        $instance = new autoptimizeStyles( $css_orig );
1393
-        $instance->setOption( 'datauris', true );
1394
-        $css_actual = $instance->rewrite_assets( $css_orig );
1395
-        $this->assertEquals( $css_expected, $css_actual );
1392
+        $instance = new autoptimizeStyles($css_orig);
1393
+        $instance->setOption('datauris', true);
1394
+        $css_actual = $instance->rewrite_assets($css_orig);
1395
+        $this->assertEquals($css_expected, $css_actual);
1396 1396
     }
1397 1397
 
1398 1398
     /**
@@ -1512,12 +1512,12 @@  discard block
 block discarded – undo
1512 1512
 CSS;
1513 1513
 
1514 1514
         // Test with fonts pointed to the CDN + cdn option is set.
1515
-        add_filter( 'autoptimize_filter_css_fonts_cdn', '__return_true' );
1516
-        $instance = new autoptimizeStyles( $css_in );
1517
-        $instance->setOption( 'cdn_url', $cdnurl );
1518
-        $css_actual_fonts_cdn = $instance->rewrite_assets( $css_in );
1515
+        add_filter('autoptimize_filter_css_fonts_cdn', '__return_true');
1516
+        $instance = new autoptimizeStyles($css_in);
1517
+        $instance->setOption('cdn_url', $cdnurl);
1518
+        $css_actual_fonts_cdn = $instance->rewrite_assets($css_in);
1519 1519
 
1520
-        $this->assertEquals( $css_expected_fonts_cdn, $css_actual_fonts_cdn );
1520
+        $this->assertEquals($css_expected_fonts_cdn, $css_actual_fonts_cdn);
1521 1521
     }
1522 1522
 
1523 1523
     /**
@@ -1633,10 +1633,10 @@  discard block
 block discarded – undo
1633 1633
 }
1634 1634
 CSS;
1635 1635
         // Test without moving fonts to CDN, but cdn option is set.
1636
-        $instance = new autoptimizeStyles( $css_in );
1637
-        $instance->setOption( 'cdn_url', $urls['cdnurl'] );
1638
-        $css_actual = $instance->rewrite_assets( $css_in );
1639
-        $this->assertEquals( $css_expected, $css_actual );
1636
+        $instance = new autoptimizeStyles($css_in);
1637
+        $instance->setOption('cdn_url', $urls['cdnurl']);
1638
+        $css_actual = $instance->rewrite_assets($css_in);
1639
+        $this->assertEquals($css_expected, $css_actual);
1640 1640
     }
1641 1641
 
1642 1642
     public function test_assets_regex_replaces_multi_bg_images()
@@ -1657,16 +1657,16 @@  discard block
 block discarded – undo
1657 1657
 }
1658 1658
 CSS;
1659 1659
 
1660
-        $instance = new autoptimizeStyles( $in );
1661
-        $instance->setOption( 'cdn_url', $cdnurl );
1662
-        $actual = $instance->rewrite_assets( $in );
1660
+        $instance = new autoptimizeStyles($in);
1661
+        $instance->setOption('cdn_url', $cdnurl);
1662
+        $actual = $instance->rewrite_assets($in);
1663 1663
 
1664
-        $this->assertEquals( $expected, $actual );
1664
+        $this->assertEquals($expected, $actual);
1665 1665
     }
1666 1666
 
1667 1667
     public function test_at_supports_spacing_issue_110()
1668 1668
     {
1669
-        $in       = <<<CSS
1669
+        $in = <<<CSS
1670 1670
 @supports (-webkit-filter: blur(3px)) or (filter: blur(3px)) {
1671 1671
     .blur {
1672 1672
         filter:blur(3px);
@@ -1680,10 +1680,10 @@  discard block
 block discarded – undo
1680 1680
 @supports (-webkit-filter:blur(3px)) or (filter:blur(3px)){.blur{filter:blur(3px)}}@supports((position:-webkit-sticky) or (position:sticky)){.sticky{position:sticky}}
1681 1681
 CSS;
1682 1682
 
1683
-        $instance = new autoptimizeStyles( $in );
1684
-        $actual   = $instance->run_minifier_on( $in );
1683
+        $instance = new autoptimizeStyles($in);
1684
+        $actual   = $instance->run_minifier_on($in);
1685 1685
 
1686
-        $this->assertEquals( $expected, $actual );
1686
+        $this->assertEquals($expected, $actual);
1687 1687
     }
1688 1688
 
1689 1689
     public function test_css_import_semicolon_url_issue_122()
@@ -1699,18 +1699,18 @@  discard block
 block discarded – undo
1699 1699
 </style>
1700 1700
 HTML;
1701 1701
 
1702
-        $expected = '<style 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-->';
1702
+        $expected = '<style 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-->';
1703 1703
 
1704 1704
         $options = [
1705 1705
             'autoptimizeStyles' => $this->getAoStylesDefaultOptions(),
1706 1706
         ];
1707 1707
 
1708
-        $instance = new autoptimizeStyles( $in );
1709
-        $instance->read( $options['autoptimizeStyles'] );
1708
+        $instance = new autoptimizeStyles($in);
1709
+        $instance->read($options['autoptimizeStyles']);
1710 1710
         $instance->minify();
1711 1711
         $instance->cache();
1712 1712
         $actual = $instance->getcontent();
1713
-        $this->assertEquals( $expected, $actual );
1713
+        $this->assertEquals($expected, $actual);
1714 1714
     }
1715 1715
 
1716 1716
     public function test_fixurls_with_at_imports_and_media_queries()
@@ -1720,10 +1720,10 @@  discard block
 block discarded – undo
1720 1720
         $subfolder = $urls['subfolder'];
1721 1721
 
1722 1722
         $in  = '@import "foo.css"; @import "bar.css" (orientation:landscape);';
1723
-        $exp = '@import url(' . $prsiteurl . '/wp-content/themes/my-theme/foo.css); @import url(' . $prsiteurl . '/wp-content/themes/my-theme/bar.css) (orientation:landscape);';
1723
+        $exp = '@import url('.$prsiteurl.'/wp-content/themes/my-theme/foo.css); @import url('.$prsiteurl.'/wp-content/themes/my-theme/bar.css) (orientation:landscape);';
1724 1724
 
1725
-        $actual = autoptimizeStyles::fixurls( ABSPATH . 'wp-content/themes/my-theme/style.css', $in );
1726
-        $this->assertEquals( $exp, $actual );
1725
+        $actual = autoptimizeStyles::fixurls(ABSPATH.'wp-content/themes/my-theme/style.css', $in);
1726
+        $this->assertEquals($exp, $actual);
1727 1727
     }
1728 1728
 
1729 1729
     public function test_aostyles_at_imports_with_media_queries()
@@ -1738,43 +1738,43 @@  discard block
 block discarded – undo
1738 1738
 </style>
1739 1739
 HTML;
1740 1740
 
1741
-        $expected = '<style 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-->';
1741
+        $expected = '<style 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-->';
1742 1742
 
1743 1743
         $options = [
1744 1744
             'autoptimizeStyles' => $this->getAoStylesDefaultOptions(),
1745 1745
         ];
1746 1746
 
1747
-        $instance = new autoptimizeStyles( $in );
1748
-        $instance->read( $options['autoptimizeStyles'] );
1747
+        $instance = new autoptimizeStyles($in);
1748
+        $instance->read($options['autoptimizeStyles']);
1749 1749
         $instance->minify();
1750 1750
         $instance->cache();
1751 1751
 
1752 1752
         $actual = $instance->getcontent();
1753
-        $this->assertEquals( $expected, $actual );
1753
+        $this->assertEquals($expected, $actual);
1754 1754
     }
1755 1755
 
1756 1756
     public function test_cache_size_checker_hooked_by_default()
1757 1757
     {
1758
-        $this->assertNotEmpty( autoptimizeCacheChecker::SCHEDULE_HOOK );
1758
+        $this->assertNotEmpty(autoptimizeCacheChecker::SCHEDULE_HOOK);
1759 1759
 
1760 1760
         // No schedule, because it's only added when is_admin() is true.
1761
-        $this->assertEquals( false, wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1761
+        $this->assertEquals(false, wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1762 1762
 
1763 1763
         // Proving that setup() sets the schedule as needed.
1764 1764
         $checker = new autoptimizeCacheChecker();
1765 1765
         $checker->setup();
1766
-        $this->assertEquals( 'twicedaily', wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1766
+        $this->assertEquals('twicedaily', wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1767 1767
     }
1768 1768
 
1769 1769
     public function test_cache_size_checker_disabled_with_filter()
1770 1770
     {
1771
-        add_filter( 'autoptimize_filter_cachecheck_do', '__return_false' );
1771
+        add_filter('autoptimize_filter_cachecheck_do', '__return_false');
1772 1772
 
1773 1773
         $checker = new autoptimizeCacheChecker();
1774 1774
         $checker->setup();
1775
-        $this->assertEquals( false, wp_get_schedule( autoptimizeCacheChecker::SCHEDULE_HOOK ) );
1775
+        $this->assertEquals(false, wp_get_schedule(autoptimizeCacheChecker::SCHEDULE_HOOK));
1776 1776
 
1777
-        remove_all_filters( 'autoptimize_filter_cachecheck_do' );
1777
+        remove_all_filters('autoptimize_filter_cachecheck_do');
1778 1778
     }
1779 1779
 
1780 1780
     public function test_is_start_buffering_hooked_properly()
@@ -1787,12 +1787,12 @@  discard block
 block discarded – undo
1787 1787
          * https://www.theaveragedev.com/mocking-constants-in-tests/.
1788 1788
          */
1789 1789
 
1790
-        if ( defined( 'AUTOPTIMIZE_INIT_EARLIER' ) ) {
1790
+        if (defined('AUTOPTIMIZE_INIT_EARLIER')) {
1791 1791
             $this->assertEquals(
1792 1792
                 autoptimizeMain::INIT_EARLIER_PRIORITY,
1793
-                has_action( 'init', array( $instance, 'start_buffering' ) )
1793
+                has_action('init', array($instance, 'start_buffering'))
1794 1794
             );
1795
-            $this->assertTrue( ! defined( 'AUTOPTIMIZE_HOOK_INTO' ) );
1795
+            $this->assertTrue(!defined('AUTOPTIMIZE_HOOK_INTO'));
1796 1796
         }
1797 1797
 
1798 1798
         /*
@@ -1809,18 +1809,18 @@  discard block
 block discarded – undo
1809 1809
 
1810 1810
     public function test_inline_and_defer_markup()
1811 1811
     {
1812
-        add_filter( 'autoptimize_filter_css_defer', '__return_true' );
1813
-        add_filter( 'autoptimize_filter_css_defer_inline', '__return_true' );
1812
+        add_filter('autoptimize_filter_css_defer', '__return_true');
1813
+        add_filter('autoptimize_filter_css_defer_inline', '__return_true');
1814 1814
 
1815
-        $actual = $this->ao->end_buffering( $this->get_test_markup() );
1816
-        if ( is_multisite() ) {
1817
-            $this->assertEquals( $this->get_test_markup_output_inline_defer_ms(), $actual );
1815
+        $actual = $this->ao->end_buffering($this->get_test_markup());
1816
+        if (is_multisite()) {
1817
+            $this->assertEquals($this->get_test_markup_output_inline_defer_ms(), $actual);
1818 1818
         } else {
1819
-            $this->assertEquals( $this->get_test_markup_output_inline_defer(), $actual );
1819
+            $this->assertEquals($this->get_test_markup_output_inline_defer(), $actual);
1820 1820
         }
1821 1821
 
1822
-        remove_all_filters( 'autoptimize_filter_css_defer' );
1823
-        remove_all_filters( 'autoptimize_filter_css_defer_inline' );
1822
+        remove_all_filters('autoptimize_filter_css_defer');
1823
+        remove_all_filters('autoptimize_filter_css_defer_inline');
1824 1824
     }
1825 1825
 
1826 1826
     public function test_js_aggregation_decision_and_dontaggregate_filter()
@@ -1828,47 +1828,47 @@  discard block
 block discarded – undo
1828 1828
         $opts = $this->getAoScriptsDefaultOptions();
1829 1829
 
1830 1830
         // Aggregating: true by default.
1831
-        $scripts = new autoptimizeScripts( '' );
1832
-        $scripts->read( $opts );
1833
-        $this->assertTrue( $scripts->aggregating() );
1831
+        $scripts = new autoptimizeScripts('');
1832
+        $scripts->read($opts);
1833
+        $this->assertTrue($scripts->aggregating());
1834 1834
 
1835 1835
         // Aggregating: option=true (dontaggregate=false by default).
1836 1836
         $opts['aggregate'] = true;
1837
-        $scripts           = new autoptimizeScripts( '' );
1838
-        $scripts->read( $opts );
1839
-        $this->assertTrue( $scripts->aggregating() );
1837
+        $scripts           = new autoptimizeScripts('');
1838
+        $scripts->read($opts);
1839
+        $this->assertTrue($scripts->aggregating());
1840 1840
 
1841 1841
         // Aggregating: option=true, dontaggregate=false explicit.
1842 1842
         $opts['aggregate'] = true;
1843
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_false' );
1844
-        $scripts = new autoptimizeScripts( '' );
1845
-        $scripts->read( $opts );
1846
-        $this->assertTrue( $scripts->aggregating() );
1847
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1843
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_false');
1844
+        $scripts = new autoptimizeScripts('');
1845
+        $scripts->read($opts);
1846
+        $this->assertTrue($scripts->aggregating());
1847
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1848 1848
 
1849 1849
         // Not aggregating: option=true, dontaggregate=true.
1850 1850
         $opts['aggregate'] = true;
1851
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_true' );
1852
-        $scripts = new autoptimizeScripts( '' );
1853
-        $scripts->read( $opts );
1854
-        $this->assertFalse( $scripts->aggregating() );
1855
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1851
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_true');
1852
+        $scripts = new autoptimizeScripts('');
1853
+        $scripts->read($opts);
1854
+        $this->assertFalse($scripts->aggregating());
1855
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1856 1856
 
1857 1857
         // Not aggregating: option=false, dontaggregate=false.
1858 1858
         $opts['aggregate'] = false;
1859
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_false' );
1860
-        $scripts = new autoptimizeScripts( '' );
1861
-        $scripts->read( $opts );
1862
-        $this->assertFalse( $scripts->aggregating() );
1863
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1859
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_false');
1860
+        $scripts = new autoptimizeScripts('');
1861
+        $scripts->read($opts);
1862
+        $this->assertFalse($scripts->aggregating());
1863
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1864 1864
 
1865 1865
         // Not aggregating: option=false, dontaggregate=true.
1866 1866
         $opts['aggregate'] = false;
1867
-        add_filter( 'autoptimize_filter_js_dontaggregate', '__return_true' );
1868
-        $scripts = new autoptimizeScripts( '' );
1869
-        $scripts->read( $opts );
1870
-        $this->assertFalse( $scripts->aggregating() );
1871
-        remove_all_filters( 'autoptimize_filter_js_dontaggregate' );
1867
+        add_filter('autoptimize_filter_js_dontaggregate', '__return_true');
1868
+        $scripts = new autoptimizeScripts('');
1869
+        $scripts->read($opts);
1870
+        $this->assertFalse($scripts->aggregating());
1871
+        remove_all_filters('autoptimize_filter_js_dontaggregate');
1872 1872
     }
1873 1873
 
1874 1874
     public function test_css_aggregation_decision_and_dontaggregate_filter()
@@ -1876,78 +1876,78 @@  discard block
 block discarded – undo
1876 1876
         $opts = $this->getAoStylesDefaultOptions();
1877 1877
 
1878 1878
         // Aggregating: true by default.
1879
-        $styles = new autoptimizeStyles( '' );
1880
-        $this->assertTrue( $styles->aggregating() );
1879
+        $styles = new autoptimizeStyles('');
1880
+        $this->assertTrue($styles->aggregating());
1881 1881
 
1882 1882
         // Aggregating: option=true (dontaggregate=false by default).
1883 1883
         $opts['aggregate'] = true;
1884
-        $styles            = new autoptimizeStyles( '' );
1885
-        $styles->read( $opts );
1886
-        $this->assertTrue( $styles->aggregating() );
1884
+        $styles            = new autoptimizeStyles('');
1885
+        $styles->read($opts);
1886
+        $this->assertTrue($styles->aggregating());
1887 1887
 
1888 1888
         // Aggregating: option=true, dontaggregate=false explicit.
1889 1889
         $opts['aggregate'] = true;
1890
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_false' );
1891
-        $styles = new autoptimizeStyles( '' );
1892
-        $styles->read( $opts );
1893
-        $this->assertTrue( $styles->aggregating() );
1894
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1890
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_false');
1891
+        $styles = new autoptimizeStyles('');
1892
+        $styles->read($opts);
1893
+        $this->assertTrue($styles->aggregating());
1894
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1895 1895
 
1896 1896
         // Not aggregating: option=true, dontaggregate=true.
1897 1897
         $opts['aggregate'] = true;
1898
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_true' );
1899
-        $styles = new autoptimizeStyles( '' );
1900
-        $styles->read( $opts );
1901
-        $this->assertFalse( $styles->aggregating() );
1902
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1898
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_true');
1899
+        $styles = new autoptimizeStyles('');
1900
+        $styles->read($opts);
1901
+        $this->assertFalse($styles->aggregating());
1902
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1903 1903
 
1904 1904
         // Not aggregating: option=false, dontaggregate=false.
1905 1905
         $opts['aggregate'] = false;
1906
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_false' );
1907
-        $styles = new autoptimizeStyles( '' );
1908
-        $styles->read( $opts );
1909
-        $this->assertFalse( $styles->aggregating() );
1910
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1906
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_false');
1907
+        $styles = new autoptimizeStyles('');
1908
+        $styles->read($opts);
1909
+        $this->assertFalse($styles->aggregating());
1910
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1911 1911
 
1912 1912
         // Not aggregating: option=false, dontaggregate=true.
1913 1913
         $opts['aggregate'] = false;
1914
-        add_filter( 'autoptimize_filter_css_dontaggregate', '__return_true' );
1915
-        $styles = new autoptimizeStyles( '' );
1916
-        $styles->read( $opts );
1917
-        $this->assertFalse( $styles->aggregating() );
1918
-        remove_all_filters( 'autoptimize_filter_css_dontaggregate' );
1914
+        add_filter('autoptimize_filter_css_dontaggregate', '__return_true');
1915
+        $styles = new autoptimizeStyles('');
1916
+        $styles->read($opts);
1917
+        $this->assertFalse($styles->aggregating());
1918
+        remove_all_filters('autoptimize_filter_css_dontaggregate');
1919 1919
     }
1920 1920
 
1921 1921
     public function test_css_minify_single_with_cdning()
1922 1922
     {
1923
-        $pathname = dirname( __FILE__ ) . '/fixtures/minify-single.css';
1923
+        $pathname = dirname(__FILE__).'/fixtures/minify-single.css';
1924 1924
         $opts     = $this->getAoStylesDefaultOptions();
1925
-        $styles   = new autoptimizeStyles( '' );
1926
-        $styles->read( $opts );
1925
+        $styles   = new autoptimizeStyles('');
1926
+        $styles->read($opts);
1927 1927
 
1928
-        $url = $styles->minify_single( $pathname, $cache_miss = true );
1928
+        $url = $styles->minify_single($pathname, $cache_miss = true);
1929 1929
 
1930 1930
         // Minified url filename + its pointed to cdn.
1931
-        $this->assertContains( AUTOPTIMIZE_CACHE_CHILD_DIR, $url );
1932
-        $this->assertContains( '/autoptimize_single_', $url );
1933
-        $this->assertContains( $styles->cdn_url, $url );
1931
+        $this->assertContains(AUTOPTIMIZE_CACHE_CHILD_DIR, $url);
1932
+        $this->assertContains('/autoptimize_single_', $url);
1933
+        $this->assertContains($styles->cdn_url, $url);
1934 1934
 
1935 1935
         // Actual minified css contents are minified and cdn-ed.
1936
-        $path     = $styles->getpath( $url );
1937
-        $contents = file_get_contents( $path );
1938
-        $this->assertContains( $styles->cdn_url, $contents );
1939
-        $this->assertContains( '.bg{background:url(' . $styles->cdn_url, $contents );
1936
+        $path     = $styles->getpath($url);
1937
+        $contents = file_get_contents($path);
1938
+        $this->assertContains($styles->cdn_url, $contents);
1939
+        $this->assertContains('.bg{background:url('.$styles->cdn_url, $contents);
1940 1940
     }
1941 1941
 
1942 1942
     public function test_ao_partners_instantiation_without_explicit_include()
1943 1943
     {
1944 1944
         $partners = new autoptimizePartners();
1945
-        $this->assertTrue( $partners instanceof autoptimizePartners );
1945
+        $this->assertTrue($partners instanceof autoptimizePartners);
1946 1946
     }
1947 1947
 
1948 1948
     public function test_html_minify_keep_html_comments_inside_script_blocks()
1949 1949
     {
1950
-        $markup   = <<<MARKUP
1950
+        $markup = <<<MARKUP
1951 1951
 <script>
1952 1952
 <!-- End Support AJAX add to cart -->
1953 1953
 var a = "b";
@@ -1977,23 +1977,23 @@  discard block
 block discarded – undo
1977 1977
             ],
1978 1978
         ];
1979 1979
 
1980
-        $instance = new autoptimizeHTML( $markup );
1981
-        $instance->read( $options['autoptimizeHTML'] );
1980
+        $instance = new autoptimizeHTML($markup);
1981
+        $instance->read($options['autoptimizeHTML']);
1982 1982
         $instance->minify();
1983 1983
         $actual = $instance->getcontent();
1984
-        $this->assertEquals( $expected, $actual );
1984
+        $this->assertEquals($expected, $actual);
1985 1985
 
1986
-        $instance = new autoptimizeHTML( $markup2 );
1987
-        $instance->read( $options['autoptimizeHTML'] );
1986
+        $instance = new autoptimizeHTML($markup2);
1987
+        $instance->read($options['autoptimizeHTML']);
1988 1988
         $instance->minify();
1989 1989
         $actual2 = $instance->getcontent();
1990
-        $this->assertEquals( $expected2, $actual2 );
1990
+        $this->assertEquals($expected2, $actual2);
1991 1991
     }
1992 1992
 
1993 1993
     public function test_html_minify_remove_html_comments_inside_script_blocks()
1994 1994
     {
1995 1995
         // Default case, html comments removed (keepcomments = false).
1996
-        $markup1   = <<<MARKUP
1996
+        $markup1 = <<<MARKUP
1997 1997
 <script>
1998 1998
 var a = "b";
1999 1999
 <!-- End Support AJAX add to cart -->
@@ -2004,7 +2004,7 @@  discard block
 block discarded – undo
2004 2004
 <!-- End Support AJAX add to cart</script>
2005 2005
 MARKUP;
2006 2006
 
2007
-        $markup2   = <<<MARKUP
2007
+        $markup2 = <<<MARKUP
2008 2008
 <script>
2009 2009
 <!-- End Support AJAX add to cart -->
2010 2010
 var a = "b";
@@ -2021,17 +2021,17 @@  discard block
 block discarded – undo
2021 2021
             ],
2022 2022
         ];
2023 2023
 
2024
-        $instance = new autoptimizeHTML( $markup1 );
2025
-        $instance->read( $options['autoptimizeHTML'] );
2024
+        $instance = new autoptimizeHTML($markup1);
2025
+        $instance->read($options['autoptimizeHTML']);
2026 2026
         $instance->minify();
2027 2027
         $actual = $instance->getcontent();
2028
-        $this->assertEquals( $expected1, $actual );
2028
+        $this->assertEquals($expected1, $actual);
2029 2029
 
2030
-        $instance = new autoptimizeHTML( $markup2 );
2031
-        $instance->read( $options['autoptimizeHTML'] );
2030
+        $instance = new autoptimizeHTML($markup2);
2031
+        $instance->read($options['autoptimizeHTML']);
2032 2032
         $instance->minify();
2033 2033
         $actual2 = $instance->getcontent();
2034
-        $this->assertEquals( $expected2, $actual2 );
2034
+        $this->assertEquals($expected2, $actual2);
2035 2035
     }
2036 2036
 
2037 2037
     public function test_html_minify_html_comments_inside_script_blocks_old_school_pattern()
@@ -2055,11 +2055,11 @@  discard block
 block discarded – undo
2055 2055
             ],
2056 2056
         ];
2057 2057
 
2058
-        $instance = new autoptimizeHTML( $markup );
2059
-        $instance->read( $options['autoptimizeHTML'] );
2058
+        $instance = new autoptimizeHTML($markup);
2059
+        $instance->read($options['autoptimizeHTML']);
2060 2060
         $instance->minify();
2061 2061
         $actual = $instance->getcontent();
2062
-        $this->assertEquals( $expected, $actual );
2062
+        $this->assertEquals($expected, $actual);
2063 2063
     }
2064 2064
 
2065 2065
     public function test_html_minify_html_comments_inside_script_blocks_old_school_pattern_untouched()
@@ -2084,49 +2084,49 @@  discard block
 block discarded – undo
2084 2084
             ],
2085 2085
         ];
2086 2086
 
2087
-        $instance = new autoptimizeHTML( $markup );
2088
-        $instance->read( $options['autoptimizeHTML'] );
2087
+        $instance = new autoptimizeHTML($markup);
2088
+        $instance->read($options['autoptimizeHTML']);
2089 2089
         $instance->minify();
2090 2090
         $actual = $instance->getcontent();
2091
-        $this->assertEquals( $expected, $actual );
2091
+        $this->assertEquals($expected, $actual);
2092 2092
     }
2093 2093
 
2094 2094
     public function test_utils_mbstring_availabilty_overriding()
2095 2095
     {
2096 2096
         $orig     = autoptimizeUtils::mbstring_available();
2097
-        $opposite = ! $orig;
2097
+        $opposite = !$orig;
2098 2098
 
2099
-        $this->assertSame( $orig, autoptimizeUtils::mbstring_available() );
2099
+        $this->assertSame($orig, autoptimizeUtils::mbstring_available());
2100 2100
         // Override works...
2101
-        $this->assertSame( $opposite, autoptimizeUtils::mbstring_available( $opposite ) );
2101
+        $this->assertSame($opposite, autoptimizeUtils::mbstring_available($opposite));
2102 2102
         // And override remains cached as the last version.
2103
-        $this->assertSame( $opposite, autoptimizeUtils::mbstring_available() );
2103
+        $this->assertSame($opposite, autoptimizeUtils::mbstring_available());
2104 2104
     }
2105 2105
 
2106 2106
     public function test_utils_mbstring_basics()
2107 2107
     {
2108 2108
         // Turn on mbstring usage.
2109
-        autoptimizeUtils::mbstring_available( true );
2109
+        autoptimizeUtils::mbstring_available(true);
2110 2110
 
2111
-        $this->assertSame( 2, autoptimizeUtils::strlen( "\x00\xFF", 'ASCII' ) );
2112
-        $this->assertSame( 2, autoptimizeUtils::strlen( "\x00\xFF", 'CP850' ) );
2113
-        $this->assertSame( 3, autoptimizeUtils::strlen( '한국어' ) );
2111
+        $this->assertSame(2, autoptimizeUtils::strlen("\x00\xFF", 'ASCII'));
2112
+        $this->assertSame(2, autoptimizeUtils::strlen("\x00\xFF", 'CP850'));
2113
+        $this->assertSame(3, autoptimizeUtils::strlen('한국어'));
2114 2114
 
2115
-        $this->assertFalse( @autoptimizeUtils::strpos( 'abc', '' ) );
2116
-        $this->assertFalse( @autoptimizeUtils::strpos( 'abc', 'a', -1 ) );
2117
-        $this->assertFalse( autoptimizeUtils::strpos( 'abc', 'd' ) );
2118
-        $this->assertFalse( autoptimizeUtils::strpos( 'abc', 'a', 3 ) );
2119
-        $this->assertSame( 1, autoptimizeUtils::strpos( '한국어', '국' ) );
2115
+        $this->assertFalse(@autoptimizeUtils::strpos('abc', ''));
2116
+        $this->assertFalse(@autoptimizeUtils::strpos('abc', 'a', -1));
2117
+        $this->assertFalse(autoptimizeUtils::strpos('abc', 'd'));
2118
+        $this->assertFalse(autoptimizeUtils::strpos('abc', 'a', 3));
2119
+        $this->assertSame(1, autoptimizeUtils::strpos('한국어', '국'));
2120 2120
     }
2121 2121
 
2122 2122
     /**
2123 2123
      * @dataProvider provider_utils_substr_replace
2124 2124
      */
2125
-    function test_utils_substr_replace_basics_mbstring( $s, $repl, $start, $len, $expected )
2125
+    function test_utils_substr_replace_basics_mbstring($s, $repl, $start, $len, $expected)
2126 2126
     {
2127 2127
         // Force mbstring code path...
2128
-        autoptimizeUtils::mbstring_available( true );
2129
-        $this->assertEquals( $expected, autoptimizeUtils::substr_replace( $s, $repl, $start, $len ) );
2128
+        autoptimizeUtils::mbstring_available(true);
2129
+        $this->assertEquals($expected, autoptimizeUtils::substr_replace($s, $repl, $start, $len));
2130 2130
     }
2131 2131
 
2132 2132
     public function provider_utils_substr_replace()
@@ -2168,48 +2168,48 @@  discard block
 block discarded – undo
2168 2168
 
2169 2169
     function test_mb_substr_replace_with_ascii_input_string()
2170 2170
     {
2171
-        autoptimizeUtils::mbstring_available( false );
2171
+        autoptimizeUtils::mbstring_available(false);
2172 2172
 
2173 2173
         $str = 'Ascii';
2174 2174
 
2175
-        $this->assertSame( 'Añ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) );
2176
-        $this->assertSame( 'ñcii', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2177
-        $this->assertSame( 'Asñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2178
-        $this->assertSame( 'Asz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) );
2179
-        $this->assertSame( 'Añii', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2175
+        $this->assertSame('Añ', autoptimizeUtils::substr_replace($str, 'ñ', 1));
2176
+        $this->assertSame('ñcii', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2177
+        $this->assertSame('Asñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2178
+        $this->assertSame('Asz', autoptimizeUtils::substr_replace($str, 'z', 2, 10));
2179
+        $this->assertSame('Añii', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2180 2180
     }
2181 2181
 
2182 2182
     function test_mb_substr_replace_with_utf8_input_string()
2183 2183
     {
2184
-        autoptimizeUtils::mbstring_available( true );
2184
+        autoptimizeUtils::mbstring_available(true);
2185 2185
 
2186 2186
         $str = 'âønæë';
2187 2187
 
2188
-        $this->assertSame( 'âñ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) ); // No length.
2189
-        $this->assertSame( 'ñnæë', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2190
-        $this->assertSame( 'âøñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2191
-        $this->assertSame( 'âøz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) ); // Length larger than possible...
2192
-        $this->assertSame( 'âñæë', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2188
+        $this->assertSame('âñ', autoptimizeUtils::substr_replace($str, 'ñ', 1)); // No length.
2189
+        $this->assertSame('ñnæë', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2190
+        $this->assertSame('âøñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2191
+        $this->assertSame('âøz', autoptimizeUtils::substr_replace($str, 'z', 2, 10)); // Length larger than possible...
2192
+        $this->assertSame('âñæë', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2193 2193
     }
2194 2194
 
2195 2195
     function test_default_substr_replace_with_ascii_input_string()
2196 2196
     {
2197 2197
         // Disable mbstring which should fall ack to substr_replace...
2198
-        autoptimizeUtils::mbstring_available( false );
2198
+        autoptimizeUtils::mbstring_available(false);
2199 2199
 
2200 2200
         $str = 'Ascii';
2201 2201
 
2202
-        $this->assertSame( 'Añ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) );
2203
-        $this->assertSame( 'ñcii', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2204
-        $this->assertSame( 'Asñx', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2205
-        $this->assertSame( 'Asz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) );
2206
-        $this->assertSame( 'Añii', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2202
+        $this->assertSame('Añ', autoptimizeUtils::substr_replace($str, 'ñ', 1));
2203
+        $this->assertSame('ñcii', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2204
+        $this->assertSame('Asñx', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2205
+        $this->assertSame('Asz', autoptimizeUtils::substr_replace($str, 'z', 2, 10));
2206
+        $this->assertSame('Añii', autoptimizeUtils::substr_replace($str, 'ñ', 1, 2));
2207 2207
     }
2208 2208
 
2209 2209
     function test_default_substr_replace_with_utf8_input_string()
2210 2210
     {
2211 2211
         // Disabling mbstring, falling back to substr_replace...
2212
-        autoptimizeUtils::mbstring_available( false );
2212
+        autoptimizeUtils::mbstring_available(false);
2213 2213
 
2214 2214
         // This is really impossible to make work properly, since
2215 2215
         // any start/len parameters we give are working with bytes instead
@@ -2218,59 +2218,59 @@  discard block
 block discarded – undo
2218 2218
 
2219 2219
         // $this->assertSame( '�ñ', autoptimizeUtils::substr_replace( $str, 'ñ', 1 ) ); // No length.
2220 2220
         // $this->assertSame( 'ñ�næë', autoptimizeUtils::substr_replace( $str, 'ñ', 1, 2 ) );
2221
-        $this->assertSame( 'ñønæë', autoptimizeUtils::substr_replace( $str, 'ñ', 0, 2 ) );
2222
-        $this->assertSame( 'âñxæë', autoptimizeUtils::substr_replace( $str, 'ñx', 2, 3 ) );
2223
-        $this->assertSame( 'âz', autoptimizeUtils::substr_replace( $str, 'z', 2, 10 ) ); // Length larger than possible...
2221
+        $this->assertSame('ñønæë', autoptimizeUtils::substr_replace($str, 'ñ', 0, 2));
2222
+        $this->assertSame('âñxæë', autoptimizeUtils::substr_replace($str, 'ñx', 2, 3));
2223
+        $this->assertSame('âz', autoptimizeUtils::substr_replace($str, 'z', 2, 10)); // Length larger than possible...
2224 2224
     }
2225 2225
 
2226 2226
     public function test_cache_fast_delete()
2227 2227
     {
2228
-        add_filter( 'autoptimize_filter_cache_clear_advanced', '__return_true' );
2228
+        add_filter('autoptimize_filter_cache_clear_advanced', '__return_true');
2229 2229
 
2230 2230
         autoptimizeCache::clearall_actionless();
2231 2231
 
2232
-        remove_all_filters( 'autoptimize_filter_cache_clear_advanced' );
2232
+        remove_all_filters('autoptimize_filter_cache_clear_advanced');
2233 2233
     }
2234 2234
 
2235 2235
     public function test_delete_advanced_cache_clear_artifacts()
2236 2236
     {
2237
-        add_filter( 'autoptimize_filter_cache_clear_advanced', '__return_true' );
2237
+        add_filter('autoptimize_filter_cache_clear_advanced', '__return_true');
2238 2238
 
2239
-        $this->assertTrue( autoptimizeCache::delete_advanced_cache_clear_artifacts() );
2239
+        $this->assertTrue(autoptimizeCache::delete_advanced_cache_clear_artifacts());
2240 2240
 
2241
-        remove_all_filters( 'autoptimize_filter_cache_clear_advanced' );
2241
+        remove_all_filters('autoptimize_filter_cache_clear_advanced');
2242 2242
     }
2243 2243
 
2244 2244
     public function provider_canonicalization()
2245 2245
     {
2246 2246
         return array(
2247
-            array( '../common', 'common' ),
2248
-            array( '../what-does-this-mean/really?/', 'what-does-this-mean/really?/' ),
2249
-            array( '../../what/where/how', 'what/where/how' ),
2250
-            array( '/../more.dots.please/', '/more.dots.please/' ),
2251
-            array( '/../../what/where/how', '/what/where/how' ),
2252
-            array( '/a/b/c/../../../d/e/file.txt', '/d/e/file.txt' ),
2253
-            array( 'a/b/../c', 'a/c' ),
2254
-            array( './../../etc/passwd', './etc/passwd' ),
2255
-            array( '/var/.////./user/./././..//.//../////../././.././test/////', '/test/' ),
2256
-            array( '/var/user/./././.././../.././././test/', '/test/' ),
2257
-            array( '/hello/0//how/../are/../you', '/hello/0/you' ),
2258
-            array( '', '' ),
2259
-            array( '.', '.' ),
2260
-            array( '..', '..' ),
2261
-            array( './..', './..' ),
2262
-            array( '../.', '.' ),
2247
+            array('../common', 'common'),
2248
+            array('../what-does-this-mean/really?/', 'what-does-this-mean/really?/'),
2249
+            array('../../what/where/how', 'what/where/how'),
2250
+            array('/../more.dots.please/', '/more.dots.please/'),
2251
+            array('/../../what/where/how', '/what/where/how'),
2252
+            array('/a/b/c/../../../d/e/file.txt', '/d/e/file.txt'),
2253
+            array('a/b/../c', 'a/c'),
2254
+            array('./../../etc/passwd', './etc/passwd'),
2255
+            array('/var/.////./user/./././..//.//../////../././.././test/////', '/test/'),
2256
+            array('/var/user/./././.././../.././././test/', '/test/'),
2257
+            array('/hello/0//how/../are/../you', '/hello/0/you'),
2258
+            array('', ''),
2259
+            array('.', '.'),
2260
+            array('..', '..'),
2261
+            array('./..', './..'),
2262
+            array('../.', '.'),
2263 2263
             // This might be debatable...
2264
-            array( '../..', '..' ),
2264
+            array('../..', '..'),
2265 2265
         );
2266 2266
     }
2267 2267
 
2268 2268
     /**
2269 2269
      * @dataProvider provider_canonicalization
2270 2270
      */
2271
-    public function test_path_canonicalize( $path, $canonicalized )
2271
+    public function test_path_canonicalize($path, $canonicalized)
2272 2272
     {
2273
-        $this->assertSame( $canonicalized, autoptimizeUtils::path_canonicalize( $path ) );
2273
+        $this->assertSame($canonicalized, autoptimizeUtils::path_canonicalize($path));
2274 2274
     }
2275 2275
 
2276 2276
     /**
@@ -2284,16 +2284,16 @@  discard block
 block discarded – undo
2284 2284
     public function test_autoptimize_filter_cdn_magic_path_check()
2285 2285
     {
2286 2286
         $cdnurl = 'http://cdn.example.org';
2287
-        add_filter( 'autoptimize_filter_cdn_magic_path_check', '__return_false', 10, 2 );
2287
+        add_filter('autoptimize_filter_cdn_magic_path_check', '__return_false', 10, 2);
2288 2288
 
2289 2289
         // Even when site is in a subfolder, the resulting cdn-rewritten url
2290 2290
         // should not magically include it, due to the above filter.
2291 2291
         // The second parameter is here to force a cache miss and re-run
2292 2292
         // the filter since we're using the same cdn url all over the place,
2293 2293
         // but want to prove different things with it.
2294
-        $this->assertSame( $cdnurl, autoptimizeUtils::tweak_cdn_url_if_needed( $cdnurl, true ) );
2294
+        $this->assertSame($cdnurl, autoptimizeUtils::tweak_cdn_url_if_needed($cdnurl, true));
2295 2295
 
2296
-        remove_all_filters( 'autoptimize_filter_cdn_magic_path_check' );
2296
+        remove_all_filters('autoptimize_filter_cdn_magic_path_check');
2297 2297
     }
2298 2298
 
2299 2299
     /**
@@ -2318,9 +2318,9 @@  discard block
 block discarded – undo
2318 2318
 <img src='$imgopthost/client/q_glossy,ret_img,w_400,h_200/$siteurl/wp-content/image.jpg' width='400' height='200' srcset="$imgopthost/client/q_glossy,ret_img,w_300/$siteurl/wp-content/image-300X150.jpg 300w, $imgopthost/client/q_glossy,ret_img,w_600/$siteurl/wp-content/image-600X300.jpg 600w" sizes="(max-width: 300px) 100vw, 300px" />
2319 2319
 MARKUP;
2320 2320
         $instance = autoptimizeImages::instance();
2321
-        $instance->set_options( $opts );
2322
-        $actual = $instance->filter_optimize_images( $markup );
2323
-        $this->assertEquals( $expected, $actual );
2321
+        $instance->set_options($opts);
2322
+        $actual = $instance->filter_optimize_images($markup);
2323
+        $this->assertEquals($expected, $actual);
2324 2324
     }
2325 2325
 
2326 2326
     /**
@@ -2345,9 +2345,9 @@  discard block
 block discarded – undo
2345 2345
 <picture><source srcset="$imgopthost/client/q_glossy,ret_img/$siteurl/wp-content/image.jpg" media="(min-width: 800px)"><img src="$imgopthost/client/q_glossy,ret_img/$siteurl/wp-content/image.jpg"/></picture>
2346 2346
 MARKUP;
2347 2347
         $instance = autoptimizeImages::instance();
2348
-        $instance->set_options( $opts );
2349
-        $actual = $instance->filter_optimize_images( $markup );
2350
-        $this->assertEquals( $expected, $actual );
2348
+        $instance->set_options($opts);
2349
+        $actual = $instance->filter_optimize_images($markup);
2350
+        $this->assertEquals($expected, $actual);
2351 2351
     }
2352 2352
 
2353 2353
     /**
@@ -2372,9 +2372,9 @@  discard block
 block discarded – undo
2372 2372
 <picture><source data-srcset="$imgopthost/client/q_glossy,ret_img/$siteurl/wp-content/image.jpg" media="(min-width: 800px)"><noscript><img src="$imgopthost/client/q_glossy,ret_img/$siteurl/wp-content/image.jpg"/></noscript><img class="lazyload" src='$imgopthost/client/q_lqip,ret_wait/$siteurl/wp-content/image.jpg' data-src="https://cdn.shortpixel.ai/client/q_glossy,ret_img/$siteurl/wp-content/image.jpg"/></picture>
2373 2373
 MARKUP;
2374 2374
         $instance = autoptimizeImages::instance();
2375
-        $instance->set_options( $opts );
2376
-        $actual = $instance->filter_optimize_images( $markup );
2377
-        $this->assertEquals( $expected, $actual );
2375
+        $instance->set_options($opts);
2376
+        $actual = $instance->filter_optimize_images($markup);
2377
+        $this->assertEquals($expected, $actual);
2378 2378
     }
2379 2379
 
2380 2380
     /**
@@ -2400,9 +2400,9 @@  discard block
 block discarded – undo
2400 2400
 MARKUP;
2401 2401
 
2402 2402
         $instance = autoptimizeImages::instance();
2403
-        $instance->set_options( $opts );
2404
-        $actual = $instance->filter_optimize_images( $markup );
2405
-        $this->assertEquals( $expected, $actual );
2403
+        $instance->set_options($opts);
2404
+        $actual = $instance->filter_optimize_images($markup);
2405
+        $this->assertEquals($expected, $actual);
2406 2406
     }
2407 2407
 
2408 2408
     /**
@@ -2428,9 +2428,9 @@  discard block
 block discarded – undo
2428 2428
 MARKUP;
2429 2429
 
2430 2430
         $instance = autoptimizeImages::instance();
2431
-        $instance->set_options( $opts );
2432
-        $actual = $instance->filter_optimize_images( $markup );
2433
-        $this->assertEquals( $expected, $actual );
2431
+        $instance->set_options($opts);
2432
+        $actual = $instance->filter_optimize_images($markup);
2433
+        $this->assertEquals($expected, $actual);
2434 2434
     }
2435 2435
 
2436 2436
     /**
@@ -2455,9 +2455,9 @@  discard block
 block discarded – undo
2455 2455
 MARKUP;
2456 2456
 
2457 2457
         $instance = autoptimizeImages::instance();
2458
-        $instance->set_options( $opts );
2459
-        $actual = $instance->filter_lazyload_images( $markup );
2460
-        $this->assertEquals( $expected, $actual );
2458
+        $instance->set_options($opts);
2459
+        $actual = $instance->filter_lazyload_images($markup);
2460
+        $this->assertEquals($expected, $actual);
2461 2461
     }
2462 2462
 
2463 2463
     /**
@@ -2482,9 +2482,9 @@  discard block
 block discarded – undo
2482 2482
 MARKUP;
2483 2483
 
2484 2484
         $instance = autoptimizeImages::instance();
2485
-        $instance->set_options( $opts );
2486
-        $actual = $instance->filter_lazyload_images( $markup );
2487
-        $this->assertEquals( $expected, $actual );
2485
+        $instance->set_options($opts);
2486
+        $actual = $instance->filter_lazyload_images($markup);
2487
+        $this->assertEquals($expected, $actual);
2488 2488
     }
2489 2489
 
2490 2490
     /**
@@ -2509,9 +2509,9 @@  discard block
 block discarded – undo
2509 2509
 MARKUP;
2510 2510
 
2511 2511
         $instance = autoptimizeImages::instance();
2512
-        $instance->set_options( $opts );
2513
-        $actual = $instance->filter_lazyload_images( $markup );
2514
-        $this->assertEquals( $expected, $actual );
2512
+        $instance->set_options($opts);
2513
+        $actual = $instance->filter_lazyload_images($markup);
2514
+        $this->assertEquals($expected, $actual);
2515 2515
     }
2516 2516
 
2517 2517
     /**
@@ -2530,11 +2530,11 @@  discard block
 block discarded – undo
2530 2530
 MARKUP;
2531 2531
 
2532 2532
         $instance = autoptimizeImages::instance();
2533
-        $instance->set_options( $opts );
2534
-        $actual = $instance->filter_optimize_images( $markup );
2533
+        $instance->set_options($opts);
2534
+        $actual = $instance->filter_optimize_images($markup);
2535 2535
 
2536 2536
         // Expecting $markup since replacement should not happen.
2537
-        $this->assertEquals( $markup, $actual );
2537
+        $this->assertEquals($markup, $actual);
2538 2538
     }
2539 2539
 
2540 2540
     /**
@@ -2560,10 +2560,10 @@  discard block
 block discarded – undo
2560 2560
 MARKUP;
2561 2561
 
2562 2562
         $instance = autoptimizeImages::instance();
2563
-        $instance->set_options( $opts );
2564
-        $actual = $instance->filter_optimize_images( $markup );
2563
+        $instance->set_options($opts);
2564
+        $actual = $instance->filter_optimize_images($markup);
2565 2565
 
2566
-        $this->assertEquals( $expected, $actual );
2566
+        $this->assertEquals($expected, $actual);
2567 2567
     }
2568 2568
 
2569 2569
     /**
@@ -2573,8 +2573,8 @@  discard block
 block discarded – undo
2573 2573
     {
2574 2574
         $urls                                        = $this->get_urls();
2575 2575
         $siteurl                                     = $urls['siteurl'];
2576
-        $siteurlparsed                               = parse_url( $siteurl );
2577
-        $sitedomain                                  = $siteurlparsed['scheme'] . '://' . $siteurlparsed['host'];
2576
+        $siteurlparsed                               = parse_url($siteurl);
2577
+        $sitedomain                                  = $siteurlparsed['scheme'].'://'.$siteurlparsed['host'];
2578 2578
         $imgopthost                                  = $urls['imgopthost'];
2579 2579
         $opts                                        = autoptimizeImages::fetch_options();
2580 2580
         $opts['autoptimize_imgopt_checkbox_field_1'] = '1';
@@ -2589,10 +2589,10 @@  discard block
 block discarded – undo
2589 2589
 MARKUP;
2590 2590
 
2591 2591
         $instance = autoptimizeImages::instance();
2592
-        $instance->set_options( $opts );
2593
-        $actual = $instance->filter_optimize_images( $markup );
2592
+        $instance->set_options($opts);
2593
+        $actual = $instance->filter_optimize_images($markup);
2594 2594
 
2595
-        $this->assertEquals( $expected, $actual );
2595
+        $this->assertEquals($expected, $actual);
2596 2596
     }
2597 2597
 
2598 2598
     /**
@@ -2617,53 +2617,53 @@  discard block
 block discarded – undo
2617 2617
 MARKUP;
2618 2618
 
2619 2619
         $instance = autoptimizeImages::instance();
2620
-        $instance->set_options( $opts );
2621
-        $actual = $instance->filter_optimize_images( $markup );
2620
+        $instance->set_options($opts);
2621
+        $actual = $instance->filter_optimize_images($markup);
2622 2622
 
2623
-        $this->assertEquals( $expected, $actual );
2623
+        $this->assertEquals($expected, $actual);
2624 2624
     }
2625 2625
 
2626 2626
     /**
2627 2627
      * @dataProvider provider_str_is_valid_regex
2628 2628
      */
2629
-    public function test_str_is_valid_regex( $str, $expected )
2629
+    public function test_str_is_valid_regex($str, $expected)
2630 2630
     {
2631
-        $actual = autoptimizeUtils::str_is_valid_regex( $str );
2631
+        $actual = autoptimizeUtils::str_is_valid_regex($str);
2632 2632
 
2633
-        $this->assertEquals( $expected, $actual );
2633
+        $this->assertEquals($expected, $actual);
2634 2634
     }
2635 2635
 
2636 2636
     public function provider_str_is_valid_regex()
2637 2637
     {
2638 2638
         return [
2639
-            [ '<!--[if', false ],
2640
-            [ '&lt;--[if', false ],
2641
-            [ '/booya/i', true ],
2642
-            [ '~Valid(Regular)Expression~', true ],
2639
+            ['<!--[if', false],
2640
+            ['&lt;--[if', false],
2641
+            ['/booya/i', true],
2642
+            ['~Valid(Regular)Expression~', true],
2643 2643
         ];
2644 2644
     }
2645 2645
 
2646 2646
     /**
2647 2647
      * @dataProvider provider_protocol_relative_tests
2648 2648
      */
2649
-    public function test_is_protocol_relative( $str, $expected )
2649
+    public function test_is_protocol_relative($str, $expected)
2650 2650
     {
2651
-        $actual = autoptimizeUtils::is_protocol_relative( $str );
2651
+        $actual = autoptimizeUtils::is_protocol_relative($str);
2652 2652
 
2653
-        $this->assertEquals( $expected, $actual );
2653
+        $this->assertEquals($expected, $actual);
2654 2654
     }
2655 2655
 
2656 2656
     public function provider_protocol_relative_tests()
2657 2657
     {
2658 2658
         return [
2659
-            [ '//something-that-might-be-an-url-but-isnt-really', true ],
2660
-            [ '', false ],
2661
-            [ null, false ],
2662
-            [ 'booya/i', false ],
2663
-            [ '/root-relative', false ],
2664
-            [ 'http://what.ever/', false ],
2665
-            [ 'https://booya.kasha', false ],
2666
-            [ '1/', false ],
2659
+            ['//something-that-might-be-an-url-but-isnt-really', true],
2660
+            ['', false],
2661
+            [null, false],
2662
+            ['booya/i', false],
2663
+            ['/root-relative', false],
2664
+            ['http://what.ever/', false],
2665
+            ['https://booya.kasha', false],
2666
+            ['1/', false],
2667 2667
         ];
2668 2668
     }
2669 2669
 
@@ -2678,38 +2678,38 @@  discard block
 block discarded – undo
2678 2678
 
2679 2679
         $expected = 'img.centered,.aligncenter{-webkit-box-shadow:none;-moz-box-shadow:none;box-shadow:none}';
2680 2680
 
2681
-        $instance = new autoptimizeStyles( $css );
2682
-        $minified = $instance->run_minifier_on( $css );
2681
+        $instance = new autoptimizeStyles($css);
2682
+        $minified = $instance->run_minifier_on($css);
2683 2683
 
2684
-        $this->assertEquals( $expected, $minified );
2684
+        $this->assertEquals($expected, $minified);
2685 2685
     }
2686 2686
 
2687 2687
     public function test_css_font_names_mangling()
2688 2688
     {
2689 2689
         // Properly quoted font names dont get mangled.
2690 2690
         $css      = 'h2{font-family:"Archivo Black"}';
2691
-        $instance = new autoptimizeStyles( $css );
2692
-        $actual   = $instance->run_minifier_on( $css );
2693
-        $this->assertEquals( $css, $actual );
2691
+        $instance = new autoptimizeStyles($css);
2692
+        $actual   = $instance->run_minifier_on($css);
2693
+        $this->assertEquals($css, $actual);
2694 2694
 
2695 2695
         // When not quoted, 'Black' used to become '#000', but not anymore :) ...
2696 2696
         $css_unquoted = 'h2{font-family:Archivo Black;}';
2697 2697
         $expected     = 'h2{font-family:Archivo Black}';
2698
-        $instance     = new autoptimizeStyles( $css_unquoted );
2699
-        $actual       = $instance->run_minifier_on( $css_unquoted );
2700
-        $this->assertEquals( $expected, $actual );
2698
+        $instance     = new autoptimizeStyles($css_unquoted);
2699
+        $actual       = $instance->run_minifier_on($css_unquoted);
2700
+        $this->assertEquals($expected, $actual);
2701 2701
 
2702 2702
         $css_unquoted = 'h1{font:italic 1.2em Fira White,serif}';
2703
-        $instance     = new autoptimizeStyles( $css_unquoted );
2704
-        $actual       = $instance->run_minifier_on( $css_unquoted );
2705
-        $this->assertEquals( $css_unquoted, $actual );
2703
+        $instance     = new autoptimizeStyles($css_unquoted);
2704
+        $actual       = $instance->run_minifier_on($css_unquoted);
2705
+        $this->assertEquals($css_unquoted, $actual);
2706 2706
     }
2707 2707
 
2708 2708
     public function test_is_plugin_active_utils_wrapper()
2709 2709
     {
2710 2710
         // Our plugin is loaded via "muplugins_loaded" filter in tests/bootstrap.php.
2711
-        $this->assertFalse( autoptimizeUtils::is_plugin_active( 'autoptimize/autoptimize.php' ) );
2712
-        $this->assertFalse( autoptimizeUtils::is_plugin_active( 'async-javascript/async-javascript.php' ) );
2711
+        $this->assertFalse(autoptimizeUtils::is_plugin_active('autoptimize/autoptimize.php'));
2712
+        $this->assertFalse(autoptimizeUtils::is_plugin_active('async-javascript/async-javascript.php'));
2713 2713
     }
2714 2714
 
2715 2715
     public function test_jsminphp_string_literal_minification()
@@ -2723,8 +2723,8 @@  discard block
 block discarded – undo
2723 2723
         $expected = '`line
2724 2724
 break`+`he  llo`;foo`hel( \'\');lo`;`he\nl\`lo`;(`he${one + two}`)';
2725 2725
 
2726
-        $actual = JSMin::minify( $js );
2727
-        $this->assertEquals( $expected, $actual );
2726
+        $actual = JSMin::minify($js);
2727
+        $this->assertEquals($expected, $actual);
2728 2728
     }
2729 2729
 
2730 2730
     /**
@@ -2744,9 +2744,9 @@  discard block
 block discarded – undo
2744 2744
 MARKUP;
2745 2745
 
2746 2746
         $instance = autoptimizeExtra::instance();
2747
-        $instance->set_options( $opts );
2748
-        $actual = $instance->filter_preload( $markup );
2749
-        $this->assertEquals( $expected, $actual );
2747
+        $instance->set_options($opts);
2748
+        $actual = $instance->filter_preload($markup);
2749
+        $this->assertEquals($expected, $actual);
2750 2750
     }
2751 2751
 
2752 2752
     /**
@@ -2766,9 +2766,9 @@  discard block
 block discarded – undo
2766 2766
 MARKUP;
2767 2767
 
2768 2768
         $instance = autoptimizeExtra::instance();
2769
-        $instance->set_options( $opts );
2770
-        $actual = $instance->filter_preload( $markup );
2771
-        $this->assertEquals( $expected, $actual );
2769
+        $instance->set_options($opts);
2770
+        $actual = $instance->filter_preload($markup);
2771
+        $this->assertEquals($expected, $actual);
2772 2772
     }
2773 2773
 
2774 2774
 
@@ -2777,17 +2777,17 @@  discard block
 block discarded – undo
2777 2777
      */
2778 2778
     public function test_network_no_site_settings()
2779 2779
     {
2780
-        if ( is_multisite() ) {
2781
-            define( 'TEST_MULTISITE_FORCE_AO_ON_NETWORK', true );
2780
+        if (is_multisite()) {
2781
+            define('TEST_MULTISITE_FORCE_AO_ON_NETWORK', true);
2782 2782
             // set options through WordPress core methods.
2783
-            update_option( 'autoptimize_js_exclude', 'site' );
2784
-            update_network_option( get_main_network_id(), 'autoptimize_js_exclude', 'network' );
2785
-            update_network_option( get_main_network_id(), 'autoptimize_enable_site_config', '' );
2783
+            update_option('autoptimize_js_exclude', 'site');
2784
+            update_network_option(get_main_network_id(), 'autoptimize_js_exclude', 'network');
2785
+            update_network_option(get_main_network_id(), 'autoptimize_enable_site_config', '');
2786 2786
 
2787 2787
             // and then try fetching them through optionwrapper.
2788 2788
             $expected = 'network';
2789
-            $actual   = autoptimizeOptionWrapper::get_option( 'autoptimize_js_exclude' );
2790
-            $this->assertEquals( $expected, $actual );
2789
+            $actual   = autoptimizeOptionWrapper::get_option('autoptimize_js_exclude');
2790
+            $this->assertEquals($expected, $actual);
2791 2791
         }
2792 2792
     }
2793 2793
 
@@ -2796,17 +2796,17 @@  discard block
 block discarded – undo
2796 2796
      */
2797 2797
     public function test_network_per_site_settings()
2798 2798
     {
2799
-        if ( is_multisite() ) {
2799
+        if (is_multisite()) {
2800 2800
             // define of TEST_MULTISITE_FORCE_AO_ON_NETWORK not needed, done in previous test.
2801 2801
             // set options through WordPress core methods.
2802
-            update_option( 'autoptimize_js_exclude', 'site' );
2803
-            update_network_option( get_main_network_id(), 'autoptimize_js_exclude', 'network' );
2804
-            update_network_option( get_main_network_id(), 'autoptimize_enable_site_config', 'on' );
2802
+            update_option('autoptimize_js_exclude', 'site');
2803
+            update_network_option(get_main_network_id(), 'autoptimize_js_exclude', 'network');
2804
+            update_network_option(get_main_network_id(), 'autoptimize_enable_site_config', 'on');
2805 2805
 
2806 2806
             // and then try fetching them through optionwrapper.
2807 2807
             $expected = 'site';
2808
-            $actual   = autoptimizeOptionWrapper::get_option( 'autoptimize_js_exclude' );
2809
-            $this->assertEquals( $expected, $actual );
2808
+            $actual   = autoptimizeOptionWrapper::get_option('autoptimize_js_exclude');
2809
+            $this->assertEquals($expected, $actual);
2810 2810
         }
2811 2811
     }
2812 2812
 }
Please login to merge, or discard this patch.