futtta / autoptimize

config/default.php

Spacing +10 added lines, -10 removed lines

@@ -1,9 +1,9 @@ 
 <?php exit;
 
 //Check everything exists before using it
-if(!isset($_SERVER['HTTP_ACCEPT_ENCODING']))
+if (!isset($_SERVER['HTTP_ACCEPT_ENCODING']))
 	$_SERVER['HTTP_ACCEPT_ENCODING'] = '';
-if(!isset($_SERVER['HTTP_USER_AGENT']))
+if (!isset($_SERVER['HTTP_USER_AGENT']))
 	$_SERVER['HTTP_USER_AGENT'] = '';
 	
 // Determine supported compression method
@@ -27,27 +27,27 @@ 
 }
 
 //Some servers compress the output of PHP - Don't break in those cases
-if(ini_get('output_handler') == 'ob_gzhandler' || ini_get('zlib.output_compression') == 1)
+if (ini_get('output_handler') == 'ob_gzhandler' || ini_get('zlib.output_compression') == 1)
 	$encoding = 'none';
 
 //Get data
 $contents = file_get_contents(__FILE__.'.'.$encoding);
 
 // first check if we have to send 304
-$eTag=md5($contents);
-$modTime=filemtime(__FILE__.'.none');
+$eTag = md5($contents);
+$modTime = filemtime(__FILE__.'.none');
 
-$eTagMatch = (isset($_SERVER['HTTP_IF_NONE_MATCH']) && strpos($_SERVER['HTTP_IF_NONE_MATCH'],$eTag));
+$eTagMatch = (isset($_SERVER['HTTP_IF_NONE_MATCH']) && strpos($_SERVER['HTTP_IF_NONE_MATCH'], $eTag));
 $modTimeMatch = (isset($_SERVER['HTTP_IF_MODIFIED_SINCE']) && strtotime($_SERVER['HTTP_IF_MODIFIED_SINCE']) === $modTime);
 
-if (($modTimeMatch)||($eTagMatch)) {
+if (($modTimeMatch) || ($eTagMatch)) {
 	header('HTTP/1.1 304 Not Modified');
 	header('Connection: close');
 } else {
 	// send all sorts of headers
-	$expireTime=60*60*24*356; // 1y max according to RFC
+	$expireTime = 60*60*24*356; // 1y max according to RFC
 
-	if(isset($encoding) && $encoding != 'none') 
+	if (isset($encoding) && $encoding != 'none') 
 	{
 		header('Content-Encoding: '.$encoding);
 	}
@@ -56,7 +56,7 @@ 
 	header('Content-type: %%CONTENT%%; charset=utf-8');
 	header('Cache-Control: max-age='.$expireTime.', public, must-revalidate');
 	header('Expires: '.gmdate('D, d M Y H:i:s', time() + $expireTime).' GMT'); //10 years
-	header('ETag: ' . $eTag);
+	header('ETag: '.$eTag);
 	header('Last-Modified: '.gmdate('D, d M Y H:i:s', $modTime).' GMT');
 	
 	// send output

Braces +13 added lines, -8 removed lines

@@ -1,10 +1,12 @@ 
 <?php exit;
 
 //Check everything exists before using it
-if(!isset($_SERVER['HTTP_ACCEPT_ENCODING']))
+if(!isset($_SERVER['HTTP_ACCEPT_ENCODING'])) {
 	$_SERVER['HTTP_ACCEPT_ENCODING'] = '';
-if(!isset($_SERVER['HTTP_USER_AGENT']))
+}
+if(!isset($_SERVER['HTTP_USER_AGENT'])) {
 	$_SERVER['HTTP_USER_AGENT'] = '';
+}
 	
 // Determine supported compression method
 $gzip = strstr($_SERVER['HTTP_ACCEPT_ENCODING'], 'gzip');
@@ -19,16 +21,19 @@ 
 {
 	$version = floatval($matches[1]);
 	
-	if ($version < 6)
-		$encoding = 'none';
+	if ($version < 6) {
+			$encoding = 'none';
+	}
 		
-	if ($version == 6 && !strstr($_SERVER['HTTP_USER_AGENT'], 'EV1')) 
-		$encoding = 'none';
-}
+	if ($version == 6 && !strstr($_SERVER['HTTP_USER_AGENT'], 'EV1')) {
+			$encoding = 'none';
+	}
+	}
 
 //Some servers compress the output of PHP - Don't break in those cases
-if(ini_get('output_handler') == 'ob_gzhandler' || ini_get('zlib.output_compression') == 1)
+if(ini_get('output_handler') == 'ob_gzhandler' || ini_get('zlib.output_compression') == 1) {
 	$encoding = 'none';
+}
 
 //Get data
 $contents = file_get_contents(__FILE__.'.'.$encoding);

config/delayed.php

Spacing +16 added lines, -16 removed lines

@@ -1,9 +1,9 @@ 
 <?php exit;
 
 //Check everything exists before using it
-if(!isset($_SERVER['HTTP_ACCEPT_ENCODING']))
+if (!isset($_SERVER['HTTP_ACCEPT_ENCODING']))
 	$_SERVER['HTTP_ACCEPT_ENCODING'] = '';
-if(!isset($_SERVER['HTTP_USER_AGENT']))
+if (!isset($_SERVER['HTTP_USER_AGENT']))
 	$_SERVER['HTTP_USER_AGENT'] = '';
 	
 // Determine supported compression method
@@ -27,16 +27,16 @@ 
 }
 
 //Some servers compress the output of PHP - Don't break in those cases
-if(ini_get('output_handler') == 'ob_gzhandler' || ini_get('zlib.output_compression') == 1)
+if (ini_get('output_handler') == 'ob_gzhandler' || ini_get('zlib.output_compression') == 1)
 	$encoding = 'none';
 
 $iscompressed = file_exists(__FILE__.'.'.$encoding);
-if($encoding != 'none' && $iscompressed == false)
+if ($encoding != 'none' && $iscompressed == false)
 {
 	$flag = ($encoding == 'gzip' ? FORCE_GZIP : FORCE_DEFLATE);
 	$code = file_get_contents(__FILE__.'.none');
-	$contents = gzencode($code,9,$flag);
-}else{
+	$contents = gzencode($code, 9, $flag);
+} else {
 	//Get data
 	$contents = file_get_contents(__FILE__.'.'.$encoding);
 }
@@ -44,18 +44,18 @@ 
 // first check if we have to send 304
 // inspired by http://www.jonasjohn.de/snippets/php/caching.htm
 
-$eTag=md5($contents);
-$modTime=filemtime(__FILE__.'.none');
+$eTag = md5($contents);
+$modTime = filemtime(__FILE__.'.none');
 
-$eTagMatch = (isset($_SERVER['HTTP_IF_NONE_MATCH']) && strpos($_SERVER['HTTP_IF_NONE_MATCH'],$eTag));
+$eTagMatch = (isset($_SERVER['HTTP_IF_NONE_MATCH']) && strpos($_SERVER['HTTP_IF_NONE_MATCH'], $eTag));
 $modTimeMatch = (isset($_SERVER['HTTP_IF_MODIFIED_SINCE']) && strtotime($_SERVER['HTTP_IF_MODIFIED_SINCE']) === $modTime);
 
-if (($modTimeMatch)||($eTagMatch)) {
+if (($modTimeMatch) || ($eTagMatch)) {
 	header('HTTP/1.1 304 Not Modified');
 	header('Connection: close');
 } else {
 	// send all sorts of headers
-	$expireTime=60*60*24*355; // 1y max according to RFC
+	$expireTime = 60*60*24*355; // 1y max according to RFC
 	if ($encoding != 'none') {
 		header('Content-Encoding: '.$encoding);
 	}
@@ -64,22 +64,22 @@ 
 	header('Content-type: %%CONTENT%%; charset=utf-8');
 	header('Cache-Control: max-age='.$expireTime.', public, must-revalidate');
 	header('Expires: '.gmdate('D, d M Y H:i:s', time() + $expireTime).' GMT');
-	header('ETag: ' . $eTag);
+	header('ETag: '.$eTag);
 	header('Last-Modified: '.gmdate('D, d M Y H:i:s', $modTime).' GMT');
 
 	// send output
 	echo $contents;
 
 	//And write to filesystem cache if not done yet
-	if($encoding != 'none' && $iscompressed == false)
+	if ($encoding != 'none' && $iscompressed == false)
 	{
 		//Write the content we sent
-		file_put_contents(__FILE__.'.'.$encoding,$contents);
+		file_put_contents(__FILE__.'.'.$encoding, $contents);
 		
 		//And write the new content
 		$flag = ($encoding == 'gzip' ? FORCE_DEFLATE : FORCE_GZIP);
 		$ext = ($encoding == 'gzip' ? 'deflate' : 'gzip');
-		$contents = gzencode($code,9,$flag);
-		file_put_contents(__FILE__.'.'.$ext,$contents);
+		$contents = gzencode($code, 9, $flag);
+		file_put_contents(__FILE__.'.'.$ext, $contents);
 	}
 }

Braces +14 added lines, -9 removed lines

@@ -1,10 +1,12 @@ 
 <?php exit;
 
 //Check everything exists before using it
-if(!isset($_SERVER['HTTP_ACCEPT_ENCODING']))
+if(!isset($_SERVER['HTTP_ACCEPT_ENCODING'])) {
 	$_SERVER['HTTP_ACCEPT_ENCODING'] = '';
-if(!isset($_SERVER['HTTP_USER_AGENT']))
+}
+if(!isset($_SERVER['HTTP_USER_AGENT'])) {
 	$_SERVER['HTTP_USER_AGENT'] = '';
+}
 	
 // Determine supported compression method
 $gzip = strstr($_SERVER['HTTP_ACCEPT_ENCODING'], 'gzip');
@@ -19,16 +21,19 @@ 
 {
 	$version = floatval($matches[1]);
 	
-	if ($version < 6)
-		$encoding = 'none';
+	if ($version < 6) {
+			$encoding = 'none';
+	}
 		
-	if ($version == 6 && !strstr($_SERVER['HTTP_USER_AGENT'], 'EV1')) 
-		$encoding = 'none';
-}
+	if ($version == 6 && !strstr($_SERVER['HTTP_USER_AGENT'], 'EV1')) {
+			$encoding = 'none';
+	}
+	}
 
 //Some servers compress the output of PHP - Don't break in those cases
-if(ini_get('output_handler') == 'ob_gzhandler' || ini_get('zlib.output_compression') == 1)
+if(ini_get('output_handler') == 'ob_gzhandler' || ini_get('zlib.output_compression') == 1) {
 	$encoding = 'none';
+}
 
 $iscompressed = file_exists(__FILE__.'.'.$encoding);
 if($encoding != 'none' && $iscompressed == false)
@@ -36,7 +41,7 @@ 
 	$flag = ($encoding == 'gzip' ? FORCE_GZIP : FORCE_DEFLATE);
 	$code = file_get_contents(__FILE__.'.none');
 	$contents = gzencode($code,9,$flag);
-}else{
+} else{
 	//Get data
 	$contents = file_get_contents(__FILE__.'.'.$encoding);
 }

classlesses/autoptimizePartners.php

Spacing +21 added lines, -21 removed lines

@@ -5,20 +5,20 @@ 
 
 add_action('admin_init', 'ao_partner_tabs_preinit');
 function ao_partner_tabs_preinit() {
-    if (apply_filters('autoptimize_filter_show_partner_tabs',true)) {
-        add_filter('autoptimize_filter_settingsscreen_tabs','ao_add_partner_tabs');
+    if (apply_filters('autoptimize_filter_show_partner_tabs', true)) {
+        add_filter('autoptimize_filter_settingsscreen_tabs', 'ao_add_partner_tabs');
     }
 }
 
 function ao_add_partner_tabs($in) {
-    $in=array_merge($in,array('ao_partners' => __('Optimize More!','autoptimize')));
+    $in = array_merge($in, array('ao_partners' => __('Optimize More!', 'autoptimize')));
     return $in;
 }
 
-add_action('admin_menu','ao_partners_init');
+add_action('admin_menu', 'ao_partners_init');
 function ao_partners_init() {
-    if (apply_filters('autoptimize_filter_show_partner_tabs',true)) {
-        $hook=add_submenu_page(NULL,'AO partner','AO partner','manage_options','ao_partners','ao_partners');
+    if (apply_filters('autoptimize_filter_show_partner_tabs', true)) {
+        $hook = add_submenu_page(NULL, 'AO partner', 'AO partner', 'manage_options', 'ao_partners', 'ao_partners');
         // register_settings here as well if needed
     }
 }
@@ -69,10 +69,10 @@ 
     }    
     </style>
     <div class="wrap">
-        <h1><?php _e('Autoptimize Settings','autoptimize'); ?></h1>
+        <h1><?php _e('Autoptimize Settings', 'autoptimize'); ?></h1>
         <?php echo autoptimizeConfig::ao_admin_tabs(); ?>
         <?php
-            _e("<h2>These Autoptimize power-ups and related services will improve your site's performance even more!</h2>","autoptimize");
+            _e("<h2>These Autoptimize power-ups and related services will improve your site's performance even more!</h2>", "autoptimize");
         ?>
         <div>
             <?php getAOPartnerFeed(); ?>
@@ -82,32 +82,32 @@ 
 }
 
 function getAOPartnerFeed() {
-    $noFeedText=__( 'Have a look at <a href="http://optimizingmatters.com/">optimizingmatters.com</a> for Autoptimize power-ups!', 'autoptimize' );
+    $noFeedText = __('Have a look at <a href="http://optimizingmatters.com/">optimizingmatters.com</a> for Autoptimize power-ups!', 'autoptimize');
 
-    if (apply_filters('autoptimize_settingsscreen_remotehttp',true)) {
-        $rss = fetch_feed( "http://feeds.feedburner.com/OptimizingMattersDownloads" );
+    if (apply_filters('autoptimize_settingsscreen_remotehttp', true)) {
+        $rss = fetch_feed("http://feeds.feedburner.com/OptimizingMattersDownloads");
         $maxitems = 0;
 
-        if ( ! is_wp_error( $rss ) ) {
-            $maxitems = $rss->get_item_quantity( 20 ); 
-            $rss_items = $rss->get_items( 0, $maxitems );
+        if (!is_wp_error($rss)) {
+            $maxitems = $rss->get_item_quantity(20); 
+            $rss_items = $rss->get_items(0, $maxitems);
         } ?>
         <ul>
             <?php
-            if ( $maxitems == 0 ) {
+            if ($maxitems == 0) {
                 echo $noFeedText;
             } else {
-                foreach ( $rss_items as $item ) : 
-                    $itemURL = esc_url( $item->get_permalink() ); ?>
+                foreach ($rss_items as $item) : 
+                    $itemURL = esc_url($item->get_permalink()); ?>
                     <li class="itemDetail">
-                        <h3 class="itemTitle"><a href="<?php echo $itemURL; ?>" target="_blank"><?php echo esc_html( $item->get_title() ); ?></a></h3>
+                        <h3 class="itemTitle"><a href="<?php echo $itemURL; ?>" target="_blank"><?php echo esc_html($item->get_title()); ?></a></h3>
                         <?php
-                        if (($enclosure = $item->get_enclosure()) && (strpos($enclosure->get_type(),"image")!==false) ) {
-                            $itemImgURL=esc_url($enclosure->get_link());
+                        if (($enclosure = $item->get_enclosure()) && (strpos($enclosure->get_type(), "image") !== false)) {
+                            $itemImgURL = esc_url($enclosure->get_link());
                             echo "<div class=\"itemImage\"><a href=\"".$itemURL."\" target=\"_blank\"><img src=\"".$itemImgURL."\"/></a></div>";
                         }
                         ?>
-                        <div class="itemDescription"><?php echo wp_kses_post($item -> get_description() ); ?></div>
+                        <div class="itemDescription"><?php echo wp_kses_post($item -> get_description()); ?></div>
                         <div class="itemButtonRow"><div class="itemButton button-secondary"><a href="<?php echo $itemURL; ?>" target="_blank">More info</a></div></div>
                     </li>
                 <?php endforeach; ?>

classlesses/autoptimizeUpdateCode.php

Spacing +24 added lines, -24 removed lines

@@ -4,45 +4,45 @@ 
 */
 
 $majorUp = false;        
-$autoptimize_major_version=substr($autoptimize_db_version,0,3);
+$autoptimize_major_version = substr($autoptimize_db_version, 0, 3);
 
-switch($autoptimize_major_version) {
+switch ($autoptimize_major_version) {
     case "1.6":
         // from back in the days when I did not yet consider multisite
         // if user was on version 1.6.x, force advanced options to be shown by default
-        update_option('autoptimize_show_adv','1');
+        update_option('autoptimize_show_adv', '1');
 
         // and remove old options
-        $to_delete_options=array("autoptimize_cdn_css","autoptimize_cdn_css_url","autoptimize_cdn_js","autoptimize_cdn_js_url","autoptimize_cdn_img","autoptimize_cdn_img_url","autoptimize_css_yui","autoptimize_js_yui");
+        $to_delete_options = array("autoptimize_cdn_css", "autoptimize_cdn_css_url", "autoptimize_cdn_js", "autoptimize_cdn_js_url", "autoptimize_cdn_img", "autoptimize_cdn_img_url", "autoptimize_css_yui", "autoptimize_js_yui");
         foreach ($to_delete_options as $del_opt) {
-            delete_option( $del_opt );
+            delete_option($del_opt);
         }
         $majorUp = true;
     case "1.7":
         // force 3.8 dashicons in CSS exclude options when upgrading from 1.7 to 1.8
-        if ( !is_multisite() ) {
+        if (!is_multisite()) {
             $css_exclude = get_option('autoptimize_css_exclude');
             if (empty($css_exclude)) {
                 $css_exclude = "admin-bar.min.css, dashicons.min.css";
-            } else if (strpos($css_exclude,"dashicons.min.css")===false) {
+            } else if (strpos($css_exclude, "dashicons.min.css") === false) {
                 $css_exclude .= ", dashicons.min.css";
             }
-            update_option('autoptimize_css_exclude',$css_exclude);
+            update_option('autoptimize_css_exclude', $css_exclude);
         } else {
             global $wpdb;
-            $blog_ids = $wpdb->get_col( "SELECT blog_id FROM $wpdb->blogs" );
+            $blog_ids = $wpdb->get_col("SELECT blog_id FROM $wpdb->blogs");
             $original_blog_id = get_current_blog_id();
-            foreach ( $blog_ids as $blog_id ) {
-                switch_to_blog( $blog_id );
+            foreach ($blog_ids as $blog_id) {
+                switch_to_blog($blog_id);
                 $css_exclude = get_option('autoptimize_css_exclude');
                 if (empty($css_exclude)) {
                     $css_exclude = "admin-bar.min.css, dashicons.min.css";
-                } else if (strpos($css_exclude,"dashicons.min.css")===false) {
+                } else if (strpos($css_exclude, "dashicons.min.css") === false) {
                     $css_exclude .= ", dashicons.min.css";
                 }
-                update_option('autoptimize_css_exclude',$css_exclude);
+                update_option('autoptimize_css_exclude', $css_exclude);
             }
-            switch_to_blog( $original_blog_id );
+            switch_to_blog($original_blog_id);
         }
         $majorUp = true;
     case "1.9":
@@ -50,24 +50,24 @@ 
         * 2.0 will not aggregate inline CSS/JS by default, but we want users
         * upgrading from 1.9 to keep their inline code aggregated by default. 
         */
-        if ( !is_multisite() ) {
-            update_option('autoptimize_css_include_inline','on');
-            update_option('autoptimize_js_include_inline','on');
+        if (!is_multisite()) {
+            update_option('autoptimize_css_include_inline', 'on');
+            update_option('autoptimize_js_include_inline', 'on');
         } else {
             global $wpdb;
-            $blog_ids = $wpdb->get_col( "SELECT blog_id FROM $wpdb->blogs" );
+            $blog_ids = $wpdb->get_col("SELECT blog_id FROM $wpdb->blogs");
             $original_blog_id = get_current_blog_id();
-            foreach ( $blog_ids as $blog_id ) {
-                switch_to_blog( $blog_id );
-                update_option('autoptimize_css_include_inline','on');
-                update_option('autoptimize_js_include_inline','on');
+            foreach ($blog_ids as $blog_id) {
+                switch_to_blog($blog_id);
+                update_option('autoptimize_css_include_inline', 'on');
+                update_option('autoptimize_js_include_inline', 'on');
             }
-            switch_to_blog( $original_blog_id );    
+            switch_to_blog($original_blog_id);    
         }
         $majorUp = true;
     }
 
-if ( $majorUp === true ) {
+if ($majorUp === true) {
     // clear cache and notify user to check result if major upgrade
     autoptimizeCache::clearall();
     add_action('admin_notices', 'autoptimize_update_config_notice');

classlesses/autoptimizePageCacheFlush.php

Spacing +18 added lines, -18 removed lines

@@ -3,52 +3,52 @@
 // hyper cache and gator cache hook into AO, so we don't need to :-)
 
 function autoptimize_flush_pagecache() {
-    if(function_exists('wp_cache_clear_cache')) {
+    if (function_exists('wp_cache_clear_cache')) {
         if (is_multisite()) {
             $blog_id = get_current_blog_id();
             wp_cache_clear_cache($blog_id);
         } else {
             wp_cache_clear_cache();
         }
-    } else if ( has_action('cachify_flush_cache') ) {
+    } else if (has_action('cachify_flush_cache')) {
         do_action('cachify_flush_cache');
-    } else if ( function_exists('w3tc_pgcache_flush') ) {
+    } else if (function_exists('w3tc_pgcache_flush')) {
         w3tc_pgcache_flush();
-    } else if ( function_exists('wp_fast_cache_bulk_delete_all') ) {
+    } else if (function_exists('wp_fast_cache_bulk_delete_all')) {
         wp_fast_cache_bulk_delete_all(); // still to retest
     } else if (class_exists("WpFastestCache")) {
         $wpfc = new WpFastestCache();
         $wpfc -> deleteCache();
-    } else if ( class_exists("c_ws_plugin__qcache_purging_routines") ) {
+    } else if (class_exists("c_ws_plugin__qcache_purging_routines")) {
         c_ws_plugin__qcache_purging_routines::purge_cache_dir(); // quick cache, still to retest
-    } else if ( class_exists("zencache") ) {
+    } else if (class_exists("zencache")) {
         zencache::clear();
-    } else if ( class_exists("comet_cache") ) {
+    } else if (class_exists("comet_cache")) {
         comet_cache::clear();
-    } else if ( class_exists("WpeCommon") ) {
-        if ( apply_filters('autoptimize_flush_wpengine_aggressive', false) ) {
-            if ( method_exists( "WpeCommon", "purge_memcached" ) ) {
+    } else if (class_exists("WpeCommon")) {
+        if (apply_filters('autoptimize_flush_wpengine_aggressive', false)) {
+            if (method_exists("WpeCommon", "purge_memcached")) {
                 WpeCommon::purge_memcached();
             }
-            if ( method_exists( "WpeCommon", "clear_maxcdn_cache" ) ) {  
+            if (method_exists("WpeCommon", "clear_maxcdn_cache")) {  
                 WpeCommon::clear_maxcdn_cache();
             }
         }
-        if ( method_exists( "WpeCommon", "purge_varnish_cache" ) ) {
+        if (method_exists("WpeCommon", "purge_varnish_cache")) {
             WpeCommon::purge_varnish_cache();   
         }
-    } else if ( function_exists('sg_cachepress_purge_cache') ) {
+    } else if (function_exists('sg_cachepress_purge_cache')) {
         sg_cachepress_purge_cache();
-    } else if(file_exists(WP_CONTENT_DIR.'/wp-cache-config.php') && function_exists('prune_super_cache')){
+    } else if (file_exists(WP_CONTENT_DIR.'/wp-cache-config.php') && function_exists('prune_super_cache')) {
         // fallback for WP-Super-Cache
         global $cache_path;
         if (is_multisite()) {
             $blog_id = get_current_blog_id();
-            prune_super_cache( get_supercache_dir( $blog_id ), true );
-            prune_super_cache( $cache_path . 'blogs/', true );
+            prune_super_cache(get_supercache_dir($blog_id), true);
+            prune_super_cache($cache_path.'blogs/', true);
         } else {
-            prune_super_cache($cache_path.'supercache/',true);
-            prune_super_cache($cache_path,true);
+            prune_super_cache($cache_path.'supercache/', true);
+            prune_super_cache($cache_path, true);
         }
     }
 }
\ No newline at end of file

classlesses/autoptimizeCacheChecker.php

Spacing +25 added lines, -25 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if (!defined('ABSPATH')) exit; // Exit if accessed directly
 
 /* 
  * cachechecker code
@@ -13,39 +13,39 @@ 
  */
 
 if (is_admin()) {
-    add_action('plugins_loaded','ao_cachechecker_setup');
+    add_action('plugins_loaded', 'ao_cachechecker_setup');
 }
 
 function ao_cachechecker_setup() {
-    $doCacheCheck = (bool) apply_filters( 'autoptimize_filter_cachecheck_do', true);
-    $cacheCheckSchedule = wp_get_schedule( 'ao_cachechecker' );
-    $AOCCfreq = apply_filters('autoptimize_filter_cachecheck_frequency','daily');
-    if (!in_array($AOCCfreq,array('hourly','daily','monthly'))) {
-        $AOCCfreq='daily';
+    $doCacheCheck = (bool) apply_filters('autoptimize_filter_cachecheck_do', true);
+    $cacheCheckSchedule = wp_get_schedule('ao_cachechecker');
+    $AOCCfreq = apply_filters('autoptimize_filter_cachecheck_frequency', 'daily');
+    if (!in_array($AOCCfreq, array('hourly', 'daily', 'monthly'))) {
+        $AOCCfreq = 'daily';
     }
-    if ( $doCacheCheck && ( !$cacheCheckSchedule || $cacheCheckSchedule !== $AOCCfreq ) ) {
+    if ($doCacheCheck && (!$cacheCheckSchedule || $cacheCheckSchedule !== $AOCCfreq)) {
         wp_schedule_event(time(), $AOCCfreq, 'ao_cachechecker');
-    } else if ( $cacheCheckSchedule && !$doCacheCheck ) {
-        wp_clear_scheduled_hook( 'ao_cachechecker' );
+    } else if ($cacheCheckSchedule && !$doCacheCheck) {
+        wp_clear_scheduled_hook('ao_cachechecker');
     }
 }
 
 add_action('ao_cachechecker', 'ao_cachechecker_cronjob');
 function ao_cachechecker_cronjob() {
-    $maxSize = (int) apply_filters( "autoptimize_filter_cachecheck_maxsize", 512000);
-    $doCacheCheck = (bool) apply_filters( "autoptimize_filter_cachecheck_do", true);
-    $statArr=autoptimizeCache::stats(); 
-    $cacheSize=round($statArr[1]/1024);
-    if (($cacheSize>$maxSize) && ($doCacheCheck)) {
-        update_option("autoptimize_cachesize_notice",true);
-        if (apply_filters('autoptimize_filter_cachecheck_sendmail',true)) {
-            $saniSiteUrl=esc_url(site_url());
-            $ao_mailto=apply_filters('autoptimize_filter_cachecheck_mailto',get_option('admin_email',''));
-            $ao_mailsubject=__('Autoptimize cache size warning','autoptimize')." (".$saniSiteUrl.")";
-            $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: ".$saniSiteUrl.")";
+    $maxSize = (int) apply_filters("autoptimize_filter_cachecheck_maxsize", 512000);
+    $doCacheCheck = (bool) apply_filters("autoptimize_filter_cachecheck_do", true);
+    $statArr = autoptimizeCache::stats(); 
+    $cacheSize = round($statArr[1]/1024);
+    if (($cacheSize > $maxSize) && ($doCacheCheck)) {
+        update_option("autoptimize_cachesize_notice", true);
+        if (apply_filters('autoptimize_filter_cachecheck_sendmail', true)) {
+            $saniSiteUrl = esc_url(site_url());
+            $ao_mailto = apply_filters('autoptimize_filter_cachecheck_mailto', get_option('admin_email', ''));
+            $ao_mailsubject = __('Autoptimize cache size warning', 'autoptimize')." (".$saniSiteUrl.")";
+            $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: ".$saniSiteUrl.")";
 
             if (!empty($ao_mailto)) {
-                $ao_mailresult=wp_mail($ao_mailto,$ao_mailsubject,$ao_mailbody);
+                $ao_mailresult = wp_mail($ao_mailto, $ao_mailsubject, $ao_mailbody);
                 if (!$ao_mailresult) {
                     error_log("Autoptimize could not send cache size warning mail.");
                 }
@@ -56,10 +56,10 @@ 
 
 add_action('admin_notices', 'autoptimize_cachechecker_notice');
 function autoptimize_cachechecker_notice() {
-    if ((bool) get_option("autoptimize_cachesize_notice",false)) {
+    if ((bool) get_option("autoptimize_cachesize_notice", false)) {
         echo '<div class="notice notice-warning"><p>';
-        _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">the Autoptimize FAQ</a> to see how you can keep the cache size under control.', 'autoptimize' );
+        _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">the Autoptimize FAQ</a> to see how you can keep the cache size under control.', 'autoptimize');
         echo '</p></div>';
-        update_option("autoptimize_cachesize_notice",false);
+        update_option("autoptimize_cachesize_notice", false);
     }
 }

Braces +4 added lines, -1 removed lines

@@ -1,5 +1,8 @@
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+    exit;
+}
+// Exit if accessed directly
 
 class autoptimizeCache {
     private $filename;

classes/autoptimizeCache.php

Spacing +59 added lines, -59 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if (!defined('ABSPATH')) exit; // Exit if accessed directly
 
 class autoptimizeCache {
     private $filename;
@@ -7,14 +7,14 @@ 
     private $cachedir;
     private $delayed;
     
-    public function __construct($md5,$ext='php') {
+    public function __construct($md5, $ext = 'php') {
         $this->cachedir = AUTOPTIMIZE_CACHE_DIR;
         $this->delayed = AUTOPTIMIZE_CACHE_DELAY;
         $this->nogzip = AUTOPTIMIZE_CACHE_NOGZIP;
-        if($this->nogzip == false) {
+        if ($this->nogzip == false) {
             $this->filename = AUTOPTIMIZE_CACHEFILE_PREFIX.$md5.'.php';
         } else {
-            if (in_array($ext, array("js","css")))     {
+            if (in_array($ext, array("js", "css"))) {
                 $this->filename = $ext.'/'.AUTOPTIMIZE_CACHEFILE_PREFIX.$md5.'.'.$ext;
             } else {
                 $this->filename = AUTOPTIMIZE_CACHEFILE_PREFIX.$md5.'.'.$ext;
@@ -23,7 +23,7 @@ 
     }
     
     public function check() {
-        if(!file_exists($this->cachedir.$this->filename)) {
+        if (!file_exists($this->cachedir.$this->filename)) {
             // No cached file, sorry
             return false;
         }
@@ -32,8 +32,8 @@ 
     }
     
     public function retrieve() {
-        if($this->check()) {
-            if($this->nogzip == false) {
+        if ($this->check()) {
+            if ($this->nogzip == false) {
                 return file_get_contents($this->cachedir.$this->filename.'.none');
             } else {
                 return file_get_contents($this->cachedir.$this->filename);
@@ -42,48 +42,48 @@ 
         return false;
     }
     
-    public function cache($code,$mime) {
-        if($this->nogzip == false) {
+    public function cache($code, $mime) {
+        if ($this->nogzip == false) {
             $file = ($this->delayed ? 'delayed.php' : 'default.php');
             $phpcode = file_get_contents(AUTOPTIMIZE_PLUGIN_DIR.'/config/'.$file);
-            $phpcode = str_replace(array('%%CONTENT%%','exit;'),array($mime,''),$phpcode);
-            file_put_contents($this->cachedir.$this->filename,$phpcode, LOCK_EX);
-            file_put_contents($this->cachedir.$this->filename.'.none',$code, LOCK_EX);
-            if(!$this->delayed) {
+            $phpcode = str_replace(array('%%CONTENT%%', 'exit;'), array($mime, ''), $phpcode);
+            file_put_contents($this->cachedir.$this->filename, $phpcode, LOCK_EX);
+            file_put_contents($this->cachedir.$this->filename.'.none', $code, LOCK_EX);
+            if (!$this->delayed) {
                 // Compress now!
-                file_put_contents($this->cachedir.$this->filename.'.deflate',gzencode($code,9,FORCE_DEFLATE), LOCK_EX);
-                file_put_contents($this->cachedir.$this->filename.'.gzip',gzencode($code,9,FORCE_GZIP), LOCK_EX);
+                file_put_contents($this->cachedir.$this->filename.'.deflate', gzencode($code, 9, FORCE_DEFLATE), LOCK_EX);
+                file_put_contents($this->cachedir.$this->filename.'.gzip', gzencode($code, 9, FORCE_GZIP), LOCK_EX);
             }
         } else {
             // Write code to cache without doing anything else
-            file_put_contents($this->cachedir.$this->filename,$code, LOCK_EX);
+            file_put_contents($this->cachedir.$this->filename, $code, LOCK_EX);
             if (apply_filters('autoptimize_filter_cache_create_static_gzip', false)) {
                 // Create an additional cached gzip file
-                file_put_contents($this->cachedir.$this->filename.'.gz', gzencode($code,9,FORCE_GZIP), LOCK_EX);
+                file_put_contents($this->cachedir.$this->filename.'.gz', gzencode($code, 9, FORCE_GZIP), LOCK_EX);
             }
         }
     }
     
     public function getname() {
-        apply_filters('autoptimize_filter_cache_getname',AUTOPTIMIZE_CACHE_URL.$this->filename);
+        apply_filters('autoptimize_filter_cache_getname', AUTOPTIMIZE_CACHE_URL.$this->filename);
         return $this->filename;
     }
     
     static function clearall() {
-        if(!autoptimizeCache::cacheavail()) {
+        if (!autoptimizeCache::cacheavail()) {
             return false;
         }
     
         // scan the cachedirs        
-        foreach (array("","js","css") as $scandirName) {
+        foreach (array("", "js", "css") as $scandirName) {
             $scan[$scandirName] = scandir(AUTOPTIMIZE_CACHE_DIR.$scandirName);
         }
         
         // clear the cachedirs
         foreach ($scan as $scandirName=>$scanneddir) {
-            $thisAoCacheDir=rtrim(AUTOPTIMIZE_CACHE_DIR.$scandirName,"/")."/";
-            foreach($scanneddir as $file) {
-                if(!in_array($file,array('.','..')) && strpos($file,AUTOPTIMIZE_CACHEFILE_PREFIX) !== false && is_file($thisAoCacheDir.$file)) {
+            $thisAoCacheDir = rtrim(AUTOPTIMIZE_CACHE_DIR.$scandirName, "/")."/";
+            foreach ($scanneddir as $file) {
+                if (!in_array($file, array('.', '..')) && strpos($file, AUTOPTIMIZE_CACHEFILE_PREFIX) !== false && is_file($thisAoCacheDir.$file)) {
                     @unlink($thisAoCacheDir.$file);
                 }
             }
@@ -98,21 +98,21 @@ 
                 do_action("autoptimize_action_cachepurged");
             }
         }
-        add_action("shutdown","autoptimize_do_cachepurged_action",11);
+        add_action("shutdown", "autoptimize_do_cachepurged_action", 11);
         
         // try to purge caching plugins cache-files?
         include_once(AUTOPTIMIZE_PLUGIN_DIR.'classlesses/autoptimizePageCacheFlush.php');
-        add_action("autoptimize_action_cachepurged","autoptimize_flush_pagecache",10,0);
+        add_action("autoptimize_action_cachepurged", "autoptimize_flush_pagecache", 10, 0);
 
         return true;
     }
 
-    static function stats()    {
-        $AOstats=get_transient("autoptimize_stats");
+    static function stats() {
+        $AOstats = get_transient("autoptimize_stats");
 
         if (empty($AOstats)) {
             // Cache not available :(
-            if(!autoptimizeCache::cacheavail()) {
+            if (!autoptimizeCache::cacheavail()) {
                 return 0;
             }
             
@@ -121,28 +121,28 @@ 
             $size = 0;
             
             // scan the cachedirs        
-            foreach (array("","js","css") as $scandirName) {
+            foreach (array("", "js", "css") as $scandirName) {
                 $scan[$scandirName] = scandir(AUTOPTIMIZE_CACHE_DIR.$scandirName);
             }
             
             foreach ($scan as $scandirName=>$scanneddir) {
-                $thisAoCacheDir=rtrim(AUTOPTIMIZE_CACHE_DIR.$scandirName,"/")."/";
-                foreach($scanneddir as $file) {
-                    if(!in_array($file,array('.','..')) && strpos($file,AUTOPTIMIZE_CACHEFILE_PREFIX) !== false) {
-                        if(is_file($thisAoCacheDir.$file)) {
-                            if(AUTOPTIMIZE_CACHE_NOGZIP && (strpos($file,'.js') !== false || strpos($file,'.css') !== false || strpos($file,'.img') !== false || strpos($file,'.txt') !== false )) {
+                $thisAoCacheDir = rtrim(AUTOPTIMIZE_CACHE_DIR.$scandirName, "/")."/";
+                foreach ($scanneddir as $file) {
+                    if (!in_array($file, array('.', '..')) && strpos($file, AUTOPTIMIZE_CACHEFILE_PREFIX) !== false) {
+                        if (is_file($thisAoCacheDir.$file)) {
+                            if (AUTOPTIMIZE_CACHE_NOGZIP && (strpos($file, '.js') !== false || strpos($file, '.css') !== false || strpos($file, '.img') !== false || strpos($file, '.txt') !== false)) {
                                 $count++;
-                            } elseif(!AUTOPTIMIZE_CACHE_NOGZIP && strpos($file,'.none') !== false) {
+                            } elseif (!AUTOPTIMIZE_CACHE_NOGZIP && strpos($file, '.none') !== false) {
                                 $count++;
                             }
-                            $size+=filesize($thisAoCacheDir.$file);
+                            $size += filesize($thisAoCacheDir.$file);
                         }
                     }
                 }
             }
-            $AOstats=array($count,$size,time());
-            if ($count>100) {
-                set_transient("autoptimize_stats",$AOstats,HOUR_IN_SECONDS);
+            $AOstats = array($count, $size, time());
+            if ($count > 100) {
+                set_transient("autoptimize_stats", $AOstats, HOUR_IN_SECONDS);
             }
         }
         // print the number of instances
@@ -150,36 +150,36 @@ 
     }    
 
     static function cacheavail() {
-        if(!defined('AUTOPTIMIZE_CACHE_DIR')) {
+        if (!defined('AUTOPTIMIZE_CACHE_DIR')) {
             // We didn't set a cache
             return false;
         }
         
-        foreach (array("","js","css") as $checkDir) {
-            if(!autoptimizeCache::checkCacheDir(AUTOPTIMIZE_CACHE_DIR.$checkDir)) {
+        foreach (array("", "js", "css") as $checkDir) {
+            if (!autoptimizeCache::checkCacheDir(AUTOPTIMIZE_CACHE_DIR.$checkDir)) {
                 return false;
             }
         }
         
         /** write index.html here to avoid prying eyes */
-        $indexFile=AUTOPTIMIZE_CACHE_DIR.'/index.html';
-        if(!is_file($indexFile)) {
-            @file_put_contents($indexFile,'<html><head><meta name="robots" content="noindex, nofollow"></head><body>Generated by <a href="http://wordpress.org/extend/plugins/autoptimize/" rel="nofollow">Autoptimize</a></body></html>');
+        $indexFile = AUTOPTIMIZE_CACHE_DIR.'/index.html';
+        if (!is_file($indexFile)) {
+            @file_put_contents($indexFile, '<html><head><meta name="robots" content="noindex, nofollow"></head><body>Generated by <a href="http://wordpress.org/extend/plugins/autoptimize/" rel="nofollow">Autoptimize</a></body></html>');
         }
 
         /** write .htaccess here to overrule wp_super_cache */
-        $htAccess=AUTOPTIMIZE_CACHE_DIR.'/.htaccess';
-        if(!is_file($htAccess)) {
+        $htAccess = AUTOPTIMIZE_CACHE_DIR.'/.htaccess';
+        if (!is_file($htAccess)) {
             /** 
              * create wp-content/AO_htaccess_tmpl with 
              * whatever htaccess rules you might need
              * if you want to override default AO htaccess
              */
-            $htaccess_tmpl=WP_CONTENT_DIR."/AO_htaccess_tmpl";
+            $htaccess_tmpl = WP_CONTENT_DIR."/AO_htaccess_tmpl";
             if (is_file($htaccess_tmpl)) { 
-                $htAccessContent=file_get_contents($htaccess_tmpl);
+                $htAccessContent = file_get_contents($htaccess_tmpl);
             } else if (is_multisite() || AUTOPTIMIZE_CACHE_NOGZIP == false) {
-                $htAccessContent='<IfModule mod_headers.c>
+                $htAccessContent = '<IfModule mod_headers.c>
         Header set Vary "Accept-Encoding"
         Header set Cache-Control "max-age=10672000, must-revalidate"
 </IfModule>
@@ -206,7 +206,7 @@ 
     </Files>
 </IfModule>';
             } else {
-                $htAccessContent='<IfModule mod_headers.c>
+                $htAccessContent = '<IfModule mod_headers.c>
         Header set Vary "Accept-Encoding"
         Header set Cache-Control "max-age=10672000, must-revalidate"
 </IfModule>
@@ -233,7 +233,7 @@ 
     </Files>
 </IfModule>';
             }
-            @file_put_contents($htAccess,$htAccessContent);
+            @file_put_contents($htAccess, $htAccessContent);
         }
 
         // All OK
@@ -242,22 +242,22 @@ 
 
     static function checkCacheDir($dir) {
         // Check and create if not exists
-        if(!file_exists($dir))    {
-            @mkdir($dir,0775,true);
-            if(!file_exists($dir))    {
+        if (!file_exists($dir)) {
+            @mkdir($dir, 0775, true);
+            if (!file_exists($dir)) {
                 return false;
             }
         }
 
         // check if we can now write
-        if(!is_writable($dir))    {
+        if (!is_writable($dir)) {
             return false;
         }
 
         // and write index.html here to avoid prying eyes
-        $indexFile=$dir.'/index.html';
-        if(!is_file($indexFile)) {
-            @file_put_contents($indexFile,'<html><head><meta name="robots" content="noindex, nofollow"></head><body>Generated by <a href="http://wordpress.org/extend/plugins/autoptimize/" rel="nofollow">Autoptimize</a></body></html>');
+        $indexFile = $dir.'/index.html';
+        if (!is_file($indexFile)) {
+            @file_put_contents($indexFile, '<html><head><meta name="robots" content="noindex, nofollow"></head><body>Generated by <a href="http://wordpress.org/extend/plugins/autoptimize/" rel="nofollow">Autoptimize</a></body></html>');
         }
         
         return true;

Braces +4 added lines, -1 removed lines

@@ -1,5 +1,8 @@
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+    exit;
+}
+// Exit if accessed directly
 
 class autoptimizeCache {
     private $filename;

classes/autoptimizeHTML.php

Spacing +16 added lines, -16 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if (!defined('ABSPATH')) exit; // Exit if accessed directly
 
 class autoptimizeHTML extends autoptimizeBase {
     private $keepcomments = false;
@@ -10,13 +10,13 @@ 
         $this->keepcomments = (bool) $options['keepcomments'];
         
         // filter to force xhtml
-        $this->forcexhtml = (bool) apply_filters( 'autoptimize_filter_html_forcexhtml', false );
+        $this->forcexhtml = (bool) apply_filters('autoptimize_filter_html_forcexhtml', false);
         
         // filter to add strings to be excluded from HTML minification
-        $excludeHTML = apply_filters( 'autoptimize_filter_html_exclude','' );
-        if ($excludeHTML!=="") {
-            $exclHTMLArr = array_filter(array_map('trim',explode(",",$excludeHTML)));
-            $this->exclude = array_merge($exclHTMLArr,$this->exclude);
+        $excludeHTML = apply_filters('autoptimize_filter_html_exclude', '');
+        if ($excludeHTML !== "") {
+            $exclHTMLArr = array_filter(array_map('trim', explode(",", $excludeHTML)));
+            $this->exclude = array_merge($exclHTMLArr, $this->exclude);
         }
         
         // Nothing else for HTML
@@ -25,16 +25,16 @@ 
     
     //Joins and optimizes CSS
     public function minify() {
-        $noptimizeHTML = apply_filters( 'autoptimize_filter_html_noptimize', false, $this->content );
+        $noptimizeHTML = apply_filters('autoptimize_filter_html_noptimize', false, $this->content);
         if ($noptimizeHTML)
             return false;
         
-        if(class_exists('Minify_HTML')) {
+        if (class_exists('Minify_HTML')) {
             // wrap the to-be-excluded strings in noptimize tags
             foreach ($this->exclude as $exclString) {
-                if (strpos($this->content,$exclString)!==false) {
-                    $replString="<!--noptimize-->".$exclString."<!--/noptimize-->";
-                    $this->content=str_replace($exclString,$replString,$this->content);
+                if (strpos($this->content, $exclString) !== false) {
+                    $replString = "<!--noptimize-->".$exclString."<!--/noptimize-->";
+                    $this->content = str_replace($exclString, $replString, $this->content);
                 }
             }
 
@@ -47,8 +47,8 @@ 
                 $options['xhtml'] = true;
             }
 
-            if (@is_callable(array("Minify_HTML","minify"))) {
-                $tmp_content = Minify_HTML::minify($this->content,$options);
+            if (@is_callable(array("Minify_HTML", "minify"))) {
+                $tmp_content = Minify_HTML::minify($this->content, $options);
                 if (!empty($tmp_content)) {
                     $this->content = $tmp_content;
                     unset($tmp_content);
@@ -60,9 +60,9 @@ 
             
             // remove the noptimize-wrapper from around the excluded strings
             foreach ($this->exclude as $exclString) {
-                $replString="<!--noptimize-->".$exclString."<!--/noptimize-->";
-                if (strpos($this->content,$replString)!==false) {
-                    $this->content=str_replace($replString,$exclString,$this->content);
+                $replString = "<!--noptimize-->".$exclString."<!--/noptimize-->";
+                if (strpos($this->content, $replString) !== false) {
+                    $this->content = str_replace($replString, $exclString, $this->content);
                 }
             }
 

Braces +7 added lines, -3 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+    exit;
+}
+// Exit if accessed directly
 
 class autoptimizeHTML extends autoptimizeBase {
     private $keepcomments = false;
@@ -26,8 +29,9 @@ 
     //Joins and optimizes CSS
     public function minify() {
         $noptimizeHTML = apply_filters( 'autoptimize_filter_html_noptimize', false, $this->content );
-        if ($noptimizeHTML)
-            return false;
+        if ($noptimizeHTML) {
+                    return false;
+        }
         
         if(class_exists('Minify_HTML')) {
             // wrap the to-be-excluded strings in noptimize tags

classes/autoptimizeScripts.php

Braces +10 added lines, -4 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+    exit;
+}
+// Exit if accessed directly
 
 class autoptimizeScripts extends autoptimizeBase {
     private $scripts = array();
@@ -22,7 +25,9 @@ 
     //Reads the page and collects script tags
     public function read($options) {
         $noptimizeJS = apply_filters( 'autoptimize_filter_js_noptimize', false, $this->content );
-        if ($noptimizeJS) return false;
+        if ($noptimizeJS) {
+            return false;
+        }
 
         // only optimize known good JS?
         $whitelistJS = apply_filters( 'autoptimize_filter_js_whitelist', "" );
@@ -65,8 +70,9 @@ 
         }
 
         //Should we add try-catch?
-        if($options['trycatch'] == true)
-            $this->trycatch = true;
+        if($options['trycatch'] == true) {
+                    $this->trycatch = true;
+        }
 
         // force js in head?    
         if($options['forcehead'] == true) {

Spacing +107 added lines, -107 removed lines

@@ -1,11 +1,11 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if (!defined('ABSPATH')) exit; // Exit if accessed directly
 
 class autoptimizeScripts extends autoptimizeBase {
     private $scripts = array();
-    private $dontmove = array('document.write','html5.js','show_ads.js','google_ad','blogcatalog.com/w','tweetmeme.com/i','mybloglog.com/','histats.com/js','ads.smowtion.com/ad.js','statcounter.com/counter/counter.js','widgets.amung.us','ws.amazon.com/widgets','media.fastclick.net','/ads/','comment-form-quicktags/quicktags.php','edToolbar','intensedebate.com','scripts.chitika.net/','_gaq.push','jotform.com/','admin-bar.min.js','GoogleAnalyticsObject','plupload.full.min.js','syntaxhighlighter','adsbygoogle','gist.github.com','_stq','nonce','post_id','data-noptimize');
-    private $domove = array('gaJsHost','load_cmc','jd.gallery.transitions.js','swfobject.embedSWF(','tiny_mce.js','tinyMCEPreInit.go');
-    private $domovelast = array('addthis.com','/afsonline/show_afs_search.js','disqus.js','networkedblogs.com/getnetworkwidget','infolinks.com/js/','jd.gallery.js.php','jd.gallery.transitions.js','swfobject.embedSWF(','linkwithin.com/widget.js','tiny_mce.js','tinyMCEPreInit.go');
+    private $dontmove = array('document.write', 'html5.js', 'show_ads.js', 'google_ad', 'blogcatalog.com/w', 'tweetmeme.com/i', 'mybloglog.com/', 'histats.com/js', 'ads.smowtion.com/ad.js', 'statcounter.com/counter/counter.js', 'widgets.amung.us', 'ws.amazon.com/widgets', 'media.fastclick.net', '/ads/', 'comment-form-quicktags/quicktags.php', 'edToolbar', 'intensedebate.com', 'scripts.chitika.net/', '_gaq.push', 'jotform.com/', 'admin-bar.min.js', 'GoogleAnalyticsObject', 'plupload.full.min.js', 'syntaxhighlighter', 'adsbygoogle', 'gist.github.com', '_stq', 'nonce', 'post_id', 'data-noptimize');
+    private $domove = array('gaJsHost', 'load_cmc', 'jd.gallery.transitions.js', 'swfobject.embedSWF(', 'tiny_mce.js', 'tinyMCEPreInit.go');
+    private $domovelast = array('addthis.com', '/afsonline/show_afs_search.js', 'disqus.js', 'networkedblogs.com/getnetworkwidget', 'infolinks.com/js/', 'jd.gallery.js.php', 'jd.gallery.transitions.js', 'swfobject.embedSWF(', 'linkwithin.com/widget.js', 'tiny_mce.js', 'tinyMCEPreInit.go');
     private $trycatch = false;
     private $alreadyminified = false;
     private $forcehead = true;
@@ -21,70 +21,70 @@ 
     
     //Reads the page and collects script tags
     public function read($options) {
-        $noptimizeJS = apply_filters( 'autoptimize_filter_js_noptimize', false, $this->content );
+        $noptimizeJS = apply_filters('autoptimize_filter_js_noptimize', false, $this->content);
         if ($noptimizeJS) return false;
 
         // only optimize known good JS?
-        $whitelistJS = apply_filters( 'autoptimize_filter_js_whitelist', "" );
+        $whitelistJS = apply_filters('autoptimize_filter_js_whitelist', "");
         if (!empty($whitelistJS)) {
-            $this->whitelist = array_filter(array_map('trim',explode(",",$whitelistJS)));
+            $this->whitelist = array_filter(array_map('trim', explode(",", $whitelistJS)));
         }
 
         // is there JS we should simply remove
-        $removableJS = apply_filters( 'autoptimize_filter_js_removables', '');
+        $removableJS = apply_filters('autoptimize_filter_js_removables', '');
         if (!empty($removableJS)) {
-            $this->jsremovables = array_filter(array_map('trim',explode(",",$removableJS)));
+            $this->jsremovables = array_filter(array_map('trim', explode(",", $removableJS)));
         }
 
         // only header?
-        if( apply_filters('autoptimize_filter_js_justhead',$options['justhead']) == true ) {
-            $content = explode('</head>',$this->content,2);
+        if (apply_filters('autoptimize_filter_js_justhead', $options['justhead']) == true) {
+            $content = explode('</head>', $this->content, 2);
             $this->content = $content[0].'</head>';
             $this->restofcontent = $content[1];
         }
         
         // include inline?
-        if( apply_filters('autoptimize_js_include_inline',$options['include_inline']) == true ) {
+        if (apply_filters('autoptimize_js_include_inline', $options['include_inline']) == true) {
             $this->include_inline = true;
         }
 
         // filter to "late inject minified JS", default to true for now (it is faster)
-        $this->inject_min_late = apply_filters('autoptimize_filter_js_inject_min_late',true);
+        $this->inject_min_late = apply_filters('autoptimize_filter_js_inject_min_late', true);
 
         // filters to override hardcoded do(nt)move(last) array contents (array in, array out!)
-        $this->dontmove = apply_filters( 'autoptimize_filter_js_dontmove', $this->dontmove );        
-        $this->domovelast = apply_filters( 'autoptimize_filter_js_movelast', $this->domovelast );
-        $this->domove = apply_filters( 'autoptimize_filter_js_domove', $this->domove );
+        $this->dontmove = apply_filters('autoptimize_filter_js_dontmove', $this->dontmove);        
+        $this->domovelast = apply_filters('autoptimize_filter_js_movelast', $this->domovelast);
+        $this->domove = apply_filters('autoptimize_filter_js_domove', $this->domove);
 
         // get extra exclusions settings or filter
         $excludeJS = $options['js_exclude'];
-        $excludeJS = apply_filters( 'autoptimize_filter_js_exclude', $excludeJS );
-        if ($excludeJS!=="") {
+        $excludeJS = apply_filters('autoptimize_filter_js_exclude', $excludeJS);
+        if ($excludeJS !== "") {
             if (is_array($excludeJS)) {
-                if(($removeKeys = array_keys($excludeJS,"remove")) !== false) {
+                if (($removeKeys = array_keys($excludeJS, "remove")) !== false) {
                     foreach ($removeKeys as $removeKey) {
                         unset($excludeJS[$removeKey]);
-                        $this->jsremovables[]=$removeKey;
+                        $this->jsremovables[] = $removeKey;
                     }
                 }
                 $exclJSArr = array_keys($excludeJS);
             } else {
-                $exclJSArr = array_filter(array_map('trim',explode(",",$excludeJS)));
+                $exclJSArr = array_filter(array_map('trim', explode(",", $excludeJS)));
             }
-            $this->dontmove = array_merge($exclJSArr,$this->dontmove);
+            $this->dontmove = array_merge($exclJSArr, $this->dontmove);
         }
 
         //Should we add try-catch?
-        if($options['trycatch'] == true)
+        if ($options['trycatch'] == true)
             $this->trycatch = true;
 
         // force js in head?    
-        if($options['forcehead'] == true) {
+        if ($options['forcehead'] == true) {
             $this->forcehead = true;
         } else {
             $this->forcehead = false;
         }
-        $this->forcehead = apply_filters( 'autoptimize_filter_js_forcehead', $this->forcehead );
+        $this->forcehead = apply_filters('autoptimize_filter_js_forcehead', $this->forcehead);
 
         // get cdn url
         $this->cdn_url = $options['cdn_url'];
@@ -99,24 +99,24 @@ 
         $this->content = $this->hide_comments($this->content);
 
         // Get script files
-        if (preg_match_all('#<script.*</script>#Usmi',$this->content,$matches)) {
-            foreach($matches[0] as $tag) {
+        if (preg_match_all('#<script.*</script>#Usmi', $this->content, $matches)) {
+            foreach ($matches[0] as $tag) {
                 // only consider script aggregation for types whitelisted in should_aggregate-function
-                if( !$this->should_aggregate($tag) ) {
-                    $tag='';
+                if (!$this->should_aggregate($tag)) {
+                    $tag = '';
                     continue;
                 }
 
-                if (preg_match('#<script[^>]*src=("|\')([^>]*)("|\')#Usmi',$tag,$source)) {
+                if (preg_match('#<script[^>]*src=("|\')([^>]*)("|\')#Usmi', $tag, $source)) {
                     // non-inline script
-                    if ($this->isremovable($tag,$this->jsremovables)) {
-                        $this->content = str_replace($tag,'',$this->content);
+                    if ($this->isremovable($tag, $this->jsremovables)) {
+                        $this->content = str_replace($tag, '', $this->content);
                         continue;
                     }
                     
-                    $url = current(explode('?',$source[2],2));
+                    $url = current(explode('?', $source[2], 2));
                     $path = $this->getpath($url);
-                    if($path !== false && preg_match('#\.js$#',$path) && $this->ismergeable($tag)) {
+                    if ($path !== false && preg_match('#\.js$#', $path) && $this->ismergeable($tag)) {
                         // ok to optimize, add to array
                         $this->scripts[] = $path;
                     } else {
@@ -125,14 +125,14 @@ 
                             // should we add flags?
                             $origTag = $tag;
                             foreach ($excludeJS as $exclTag => $exclFlags) {
-                                if ( strpos($origTag,$exclTag)!==false && in_array($exclFlags,array("async","defer")) ) {
-                                   $tag = str_replace('<script ','<script '.$exclFlags.' ',$tag);
+                                if (strpos($origTag, $exclTag) !== false && in_array($exclFlags, array("async", "defer"))) {
+                                   $tag = str_replace('<script ', '<script '.$exclFlags.' ', $tag);
                                 }
                             }
                         }
-                        if($this->ismovable($tag)) {
+                        if ($this->ismovable($tag)) {
                             // can be moved, flags and all
-                            if($this->movetolast($tag))    {
+                            if ($this->movetolast($tag)) {
                                 $this->move['last'][] = $tag;
                             } else {
                                 $this->move['first'][] = $tag;
@@ -140,7 +140,7 @@ 
                         } else {
                             // cannot be moved, so if flag was added re-inject altered tag immediately
                             if ($origTag && $origTag !== $tag) {
-                                $this->content = str_replace($origTag,$tag,$this->content);
+                                $this->content = str_replace($origTag, $tag, $this->content);
                                 $origTag = '';
                             }
                             // and forget about the $tag (not to be touched any more)
@@ -149,23 +149,23 @@ 
                     }
                 } else {
                     // Inline script
-                    if ($this->isremovable($tag,$this->jsremovables)) {
-                        $this->content = str_replace($tag,'',$this->content);
+                    if ($this->isremovable($tag, $this->jsremovables)) {
+                        $this->content = str_replace($tag, '', $this->content);
                         continue;
                     }
                     
                     // unhide comments, as javascript may be wrapped in comment-tags for old times' sake
                     $tag = $this->restore_comments($tag);
-                    if($this->ismergeable($tag) && ( $this->include_inline )) {
-                        preg_match('#<script.*>(.*)</script>#Usmi',$tag,$code);
-                        $code = preg_replace('#.*<!\[CDATA\[(?:\s*\*/)?(.*)(?://|/\*)\s*?\]\]>.*#sm','$1',$code[1]);
-                        $code = preg_replace('/(?:^\\s*<!--\\s*|\\s*(?:\\/\\/)?\\s*-->\\s*$)/','',$code);
+                    if ($this->ismergeable($tag) && ($this->include_inline)) {
+                        preg_match('#<script.*>(.*)</script>#Usmi', $tag, $code);
+                        $code = preg_replace('#.*<!\[CDATA\[(?:\s*\*/)?(.*)(?://|/\*)\s*?\]\]>.*#sm', '$1', $code[1]);
+                        $code = preg_replace('/(?:^\\s*<!--\\s*|\\s*(?:\\/\\/)?\\s*-->\\s*$)/', '', $code);
                         $this->scripts[] = 'INLINE;'.$code;
                     } else {
                         // Can we move this?
-                        $autoptimize_js_moveable = apply_filters( 'autoptimize_js_moveable', '', $tag );
-                        if( $this->ismovable($tag) || $autoptimize_js_moveable !== '' ) {
-                            if( $this->movetolast($tag) || $autoptimize_js_moveable === 'last' ) {
+                        $autoptimize_js_moveable = apply_filters('autoptimize_js_moveable', '', $tag);
+                        if ($this->ismovable($tag) || $autoptimize_js_moveable !== '') {
+                            if ($this->movetolast($tag) || $autoptimize_js_moveable === 'last') {
                                 $this->move['last'][] = $tag;
                             } else {
                                 $this->move['first'][] = $tag;
@@ -180,7 +180,7 @@ 
                 }
                 
                 //Remove the original script tag
-                $this->content = str_replace($tag,'',$this->content);
+                $this->content = str_replace($tag, '', $this->content);
             }
             
             return true;
@@ -192,37 +192,37 @@ 
     
     //Joins and optimizes JS
     public function minify() {
-        foreach($this->scripts as $script) {
-            if(preg_match('#^INLINE;#',$script)) {
+        foreach ($this->scripts as $script) {
+            if (preg_match('#^INLINE;#', $script)) {
                 //Inline script
-                $script = preg_replace('#^INLINE;#','',$script);
-                $script = rtrim( $script, ";\n\t\r" ) . ';';
+                $script = preg_replace('#^INLINE;#', '', $script);
+                $script = rtrim($script, ";\n\t\r").';';
                 //Add try-catch?
-                if($this->trycatch) {
+                if ($this->trycatch) {
                     $script = 'try{'.$script.'}catch(e){}';
                 }
-                $tmpscript = apply_filters( 'autoptimize_js_individual_script', $script, "" );
-                if ( has_filter('autoptimize_js_individual_script') && !empty($tmpscript) ) {
-                    $script=$tmpscript;
-                    $this->alreadyminified=true;
+                $tmpscript = apply_filters('autoptimize_js_individual_script', $script, "");
+                if (has_filter('autoptimize_js_individual_script') && !empty($tmpscript)) {
+                    $script = $tmpscript;
+                    $this->alreadyminified = true;
                 }
-                $this->jscode .= "\n" . $script;
+                $this->jscode .= "\n".$script;
             } else {
                 //External script
-                if($script !== false && file_exists($script) && is_readable($script)) {
+                if ($script !== false && file_exists($script) && is_readable($script)) {
                     $scriptsrc = file_get_contents($script);
-                    $scriptsrc = preg_replace('/\x{EF}\x{BB}\x{BF}/','',$scriptsrc);
-                    $scriptsrc = rtrim($scriptsrc,";\n\t\r").';';
+                    $scriptsrc = preg_replace('/\x{EF}\x{BB}\x{BF}/', '', $scriptsrc);
+                    $scriptsrc = rtrim($scriptsrc, ";\n\t\r").';';
                     //Add try-catch?
-                    if($this->trycatch) {
+                    if ($this->trycatch) {
                         $scriptsrc = 'try{'.$scriptsrc.'}catch(e){}';
                     }
-                    $tmpscriptsrc = apply_filters( 'autoptimize_js_individual_script', $scriptsrc, $script );
-                    if ( has_filter('autoptimize_js_individual_script') && !empty($tmpscriptsrc) ) {
-                        $scriptsrc=$tmpscriptsrc;
-                        $this->alreadyminified=true;
-                    } else if ((( strpos($script,"min.js") !== false ) || ( strpos($script,"wp-includes/js/jquery/jquery.js") !== false )) && ( $this->inject_min_late === true )) {
-                        $scriptsrc="%%INJECTLATER%%".base64_encode($script)."|".md5($scriptsrc)."%%INJECTLATER%%";
+                    $tmpscriptsrc = apply_filters('autoptimize_js_individual_script', $scriptsrc, $script);
+                    if (has_filter('autoptimize_js_individual_script') && !empty($tmpscriptsrc)) {
+                        $scriptsrc = $tmpscriptsrc;
+                        $this->alreadyminified = true;
+                    } else if (((strpos($script, "min.js") !== false) || (strpos($script, "wp-includes/js/jquery/jquery.js") !== false)) && ($this->inject_min_late === true)) {
+                        $scriptsrc = "%%INJECTLATER%%".base64_encode($script)."|".md5($scriptsrc)."%%INJECTLATER%%";
                     }
                     $this->jscode .= "\n".$scriptsrc;
                 }/*else{
@@ -233,24 +233,24 @@ 
 
         //Check for already-minified code
         $this->md5hash = md5($this->jscode);
-        $ccheck = new autoptimizeCache($this->md5hash,'js');
-        if($ccheck->check()) {
+        $ccheck = new autoptimizeCache($this->md5hash, 'js');
+        if ($ccheck->check()) {
             $this->jscode = $ccheck->retrieve();
             return true;
         }
         unset($ccheck);
         
         //$this->jscode has all the uncompressed code now.
-        if ($this->alreadyminified!==true) {
-          if (class_exists('JSMin') && apply_filters( 'autoptimize_js_do_minify' , true)) {
-            if (@is_callable(array("JSMin","minify"))) {
+        if ($this->alreadyminified !== true) {
+          if (class_exists('JSMin') && apply_filters('autoptimize_js_do_minify', true)) {
+            if (@is_callable(array("JSMin", "minify"))) {
                 $tmp_jscode = trim(JSMin::minify($this->jscode));
                 if (!empty($tmp_jscode)) {
                     $this->jscode = $tmp_jscode;
                     unset($tmp_jscode);
                 }
                 $this->jscode = $this->inject_minified($this->jscode);
-                $this->jscode = apply_filters( 'autoptimize_js_after_minify', $this->jscode );
+                $this->jscode = apply_filters('autoptimize_js_after_minify', $this->jscode);
                 return true;
             } else {
                 $this->jscode = $this->inject_minified($this->jscode);
@@ -265,11 +265,11 @@ 
     }
     
     //Caches the JS in uncompressed, deflated and gzipped form.
-    public function cache()    {
-        $cache = new autoptimizeCache($this->md5hash,'js');
-        if(!$cache->check()) {
+    public function cache() {
+        $cache = new autoptimizeCache($this->md5hash, 'js');
+        if (!$cache->check()) {
             //Cache our code
-            $cache->cache($this->jscode,'text/javascript');
+            $cache->cache($this->jscode, 'text/javascript');
         }
         $this->url = AUTOPTIMIZE_CACHE_URL.$cache->getname();
         $this->url = $this->url_replace_cdn($this->url);
@@ -278,32 +278,32 @@ 
     // Returns the content
     public function getcontent() {
         // Restore the full content
-        if(!empty($this->restofcontent)) {
+        if (!empty($this->restofcontent)) {
             $this->content .= $this->restofcontent;
             $this->restofcontent = '';
         }
         
         // Add the scripts taking forcehead/ deferred (default) into account
-        if($this->forcehead == true) {
-            $replaceTag=array("</head>","before");
-            $defer="";
+        if ($this->forcehead == true) {
+            $replaceTag = array("</head>", "before");
+            $defer = "";
         } else {
-            $replaceTag=array("</body>","before");
-            $defer="defer ";
+            $replaceTag = array("</body>", "before");
+            $defer = "defer ";
         }
         
-        $defer = apply_filters( 'autoptimize_filter_js_defer', $defer );
+        $defer = apply_filters('autoptimize_filter_js_defer', $defer);
         $bodyreplacementpayload = '<script type="text/javascript" '.$defer.'src="'.$this->url.'"></script>';
-        $bodyreplacementpayload = apply_filters('autoptimize_filter_js_bodyreplacementpayload',$bodyreplacementpayload);
+        $bodyreplacementpayload = apply_filters('autoptimize_filter_js_bodyreplacementpayload', $bodyreplacementpayload);
 
-        $bodyreplacement = implode('',$this->move['first']);
+        $bodyreplacement = implode('', $this->move['first']);
         $bodyreplacement .= $bodyreplacementpayload;
-        $bodyreplacement .= implode('',$this->move['last']);
+        $bodyreplacement .= implode('', $this->move['last']);
 
-        $replaceTag = apply_filters( 'autoptimize_filter_js_replacetag', $replaceTag );
+        $replaceTag = apply_filters('autoptimize_filter_js_replacetag', $replaceTag);
 
-        if (strlen($this->jscode)>0) {
-            $this->inject_in_html($bodyreplacement,$replaceTag);
+        if (strlen($this->jscode) > 0) {
+            $this->inject_in_html($bodyreplacement, $replaceTag);
         }
         
         // restore comments
@@ -323,15 +323,15 @@ 
     private function ismergeable($tag) {
         if (!empty($this->whitelist)) {
             foreach ($this->whitelist as $match) {
-                if(strpos($tag,$match)!==false) {
+                if (strpos($tag, $match) !== false) {
                     return true;
                 }
             }
             // no match with whitelist
             return false;
         } else {
-            foreach($this->domove as $match) {
-                if(strpos($tag,$match)!==false)    {
+            foreach ($this->domove as $match) {
+                if (strpos($tag, $match) !== false) {
                     // Matched something
                     return false;
                 }
@@ -341,8 +341,8 @@ 
                 return false;
             }
             
-            foreach($this->dontmove as $match) {
-                if(strpos($tag,$match)!==false)    {
+            foreach ($this->dontmove as $match) {
+                if (strpos($tag, $match) !== false) {
                     //Matched something
                     return false;
                 }
@@ -355,12 +355,12 @@ 
     
     // Checks againstt the blacklist
     private function ismovable($tag) {
-        if ($this->include_inline !== true || apply_filters('autoptimize_filter_js_unmovable',true)) {
+        if ($this->include_inline !== true || apply_filters('autoptimize_filter_js_unmovable', true)) {
             return false;
         }
         
-        foreach($this->domove as $match) {
-            if(strpos($tag,$match)!==false)    {
+        foreach ($this->domove as $match) {
+            if (strpos($tag, $match) !== false) {
                 // Matched something
                 return true;
             }
@@ -370,8 +370,8 @@ 
             return true;
         }
         
-        foreach($this->dontmove as $match) {
-            if(strpos($tag,$match)!==false) {
+        foreach ($this->dontmove as $match) {
+            if (strpos($tag, $match) !== false) {
                 // Matched something
                 return false;
             }
@@ -382,8 +382,8 @@ 
     }
     
     private function movetolast($tag) {
-        foreach($this->domovelast as $match) {
-            if(strpos($tag,$match)!==false)    {
+        foreach ($this->domovelast as $match) {
+            if (strpos($tag, $match) !== false) {
                 // Matched, return true
                 return true;
             }
@@ -409,10 +409,10 @@ 
      * original function by https://github.com/zytzagoo/ on his AO fork, thanks Tomas!
      */
     public function should_aggregate($tag) {
-        preg_match('#<(script[^>]*)>#i',$tag,$scripttag);
-        if ( strpos($scripttag[1], 'type')===false ) {
+        preg_match('#<(script[^>]*)>#i', $tag, $scripttag);
+        if (strpos($scripttag[1], 'type') === false) {
             return true;
-        } else if ( preg_match('/type\s*=\s*["\']?(?:text|application)\/(?:javascript|ecmascript)["\']?/i', $scripttag[1]) ) {
+        } else if (preg_match('/type\s*=\s*["\']?(?:text|application)\/(?:javascript|ecmascript)["\']?/i', $scripttag[1])) {
             return true;
         } else {
             return false;