futtta / autoptimize

classes/autoptimizeBase.php

Doc Comments +20 added lines

@@ -139,6 +139,10 @@ 
     }
 
     // hide everything between noptimize-comment tags
+
+    /**
+     * @param string $noptimize_in
+     */
     protected function hide_noptimize($noptimize_in) {
         if ( preg_match( '/<!--\s?noptimize\s?-->/', $noptimize_in ) ) { 
             $noptimize_out = preg_replace_callback(
@@ -267,6 +271,9 @@ 
         return $url;
     }
 
+    /**
+     * @param string $payload
+     */
     protected function inject_in_html($payload,$replaceTag) {
         if (strpos($this->content,$replaceTag[0])!== false) {
             if ($replaceTag[1]==="after") {
@@ -286,6 +293,9 @@ 
         }
     }
     
+    /**
+     * @param string $tag
+     */
     protected function isremovable($tag, $removables) {
         foreach ($removables as $match) {
             if (strpos($tag,$match)!==false) {
@@ -296,6 +306,10 @@ 
     }
     
     // inject already minified code in optimized JS/CSS
+
+    /**
+     * @return string
+     */
     protected function inject_minified($in) {
         if ( strpos( $in, '%%INJECTLATER%%' ) !== false ) {
             $out = preg_replace_callback(
@@ -340,6 +354,9 @@ 
         return $out;
     }
     
+    /**
+     * @param string $pathIn
+     */
     protected function minify_single($pathIn) {
 		// determine JS or CSS and set var (also mimetype), return false if neither
 		if ( $this->str_ends_in($pathIn,".js") === true ) {
@@ -403,6 +420,9 @@ 
 		return $_CachedMinifiedUrl;
 	}
 	
+	/**
+	 * @param string $needle
+	 */
 	protected function str_ends_in($haystack,$needle) {
 		$needleLength = strlen($needle);
 		$haystackLength = strlen($haystack);

Indentation +57 added lines, -57 removed lines

@@ -341,43 +341,43 @@ 
     }
     
     protected function minify_single($pathIn) {
-		// determine JS or CSS and set var (also mimetype), return false if neither
-		if ( $this->str_ends_in($pathIn,".js") === true ) {
-			$codeType="js";
-			$codeMime="text/javascript";
-		} else if ( $this->str_ends_in($pathIn,".css") === true ) {
-			$codeType="css";
-			$codeMime="text/css";			
-		} else {
-			return false;
-		}
+        // determine JS or CSS and set var (also mimetype), return false if neither
+        if ( $this->str_ends_in($pathIn,".js") === true ) {
+            $codeType="js";
+            $codeMime="text/javascript";
+        } else if ( $this->str_ends_in($pathIn,".css") === true ) {
+            $codeType="css";
+            $codeMime="text/css";			
+        } else {
+            return false;
+        }
 		
-		// if min.js or min.css return false
-		if (( $this->str_ends_in($pathIn,"-min.".$codeType) === true ) || ( $this->str_ends_in($pathIn,".min.".$codeType) === true ) || ( $this->str_ends_in($pathIn,"js/jquery/jquery.js") === true ) ) {
-			return false;
-		}
+        // if min.js or min.css return false
+        if (( $this->str_ends_in($pathIn,"-min.".$codeType) === true ) || ( $this->str_ends_in($pathIn,".min.".$codeType) === true ) || ( $this->str_ends_in($pathIn,"js/jquery/jquery.js") === true ) ) {
+            return false;
+        }
 		
-		// read file, return false if empty
-		$_toMinify = file_get_contents($pathIn);
-		if ( empty($_toMinify) ) return false;
+        // read file, return false if empty
+        $_toMinify = file_get_contents($pathIn);
+        if ( empty($_toMinify) ) return false;
 		
-		// check cache
-		$_md5hash = "single_".md5($_toMinify);
-		$_cache = new autoptimizeCache($_md5hash,$codeType);
-		if ($_cache->check() ) {
-			$_CachedMinifiedUrl = AUTOPTIMIZE_CACHE_URL.$_cache->getname();
-		} else {
-			// if not in cache first minify
-			$_Minified = $_toMinify;
-			if ($codeType === "js") {
-				if (class_exists('JSMin') && apply_filters( 'autoptimize_js_do_minify' , true)) {
-					if (@is_callable(array("JSMin","minify"))) {
-						$tmp_code = trim(JSMin::minify($_toMinify));
-					}
-				}
-			} else if ($codeType === "css") {
+        // check cache
+        $_md5hash = "single_".md5($_toMinify);
+        $_cache = new autoptimizeCache($_md5hash,$codeType);
+        if ($_cache->check() ) {
+            $_CachedMinifiedUrl = AUTOPTIMIZE_CACHE_URL.$_cache->getname();
+        } else {
+            // if not in cache first minify
+            $_Minified = $_toMinify;
+            if ($codeType === "js") {
+                if (class_exists('JSMin') && apply_filters( 'autoptimize_js_do_minify' , true)) {
+                    if (@is_callable(array("JSMin","minify"))) {
+                        $tmp_code = trim(JSMin::minify($_toMinify));
+                    }
+                }
+            } else if ($codeType === "css") {
                 if (class_exists('Minify_CSS_Compressor')) {
-					$tmp_code = trim(Minify_CSS_Compressor::process($_toMinify));
+                    $tmp_code = trim(Minify_CSS_Compressor::process($_toMinify));
                 } else if(class_exists('CSSmin')) {
                     $cssmin = new CSSmin();
                     if (method_exists($cssmin,"run")) {
@@ -386,31 +386,31 @@ 
                         $tmp_code = trim(CssMin::minify($_toMinify));
                     }
                 }
-			}
-			if (!empty($tmp_code)) {
-				$_Minified = $tmp_code;
-				unset($tmp_code);
-			}
-			// and then cache
-			$_cache->cache($_Minified,$codeMime);
-			$_CachedMinifiedUrl = AUTOPTIMIZE_CACHE_URL.$_cache->getname();
-		}
-		unset($_cache);
+            }
+            if (!empty($tmp_code)) {
+                $_Minified = $tmp_code;
+                unset($tmp_code);
+            }
+            // and then cache
+            $_cache->cache($_Minified,$codeMime);
+            $_CachedMinifiedUrl = AUTOPTIMIZE_CACHE_URL.$_cache->getname();
+        }
+        unset($_cache);
 	
-		// if CDN, then CDN
-		$_CachedMinfiedUrl = $this->url_replace_cdn($_CachedMinifiedUrl);									
+        // if CDN, then CDN
+        $_CachedMinfiedUrl = $this->url_replace_cdn($_CachedMinifiedUrl);									
 
-		return $_CachedMinifiedUrl;
-	}
+        return $_CachedMinifiedUrl;
+    }
 	
-	protected function str_ends_in($haystack,$needle) {
-		$needleLength = strlen($needle);
-		$haystackLength = strlen($haystack);
-		$lastPos=strrpos($haystack,$needle);
-		if ($lastPos === $haystackLength - $needleLength) {
-			return true;
-		} else {
-			return false;
-		}
-	}
+    protected function str_ends_in($haystack,$needle) {
+        $needleLength = strlen($needle);
+        $haystackLength = strlen($haystack);
+        $lastPos=strrpos($haystack,$needle);
+        if ($lastPos === $haystackLength - $needleLength) {
+            return true;
+        } else {
+            return false;
+        }
+    }
 }

classes/autoptimizeStyles.php

Doc Comments +3 added lines

@@ -656,6 +656,9 @@
         return $code;

     }

     

+    /**
+     * @param string $tag
+     */
     private function ismovable($tag) {

 		if (apply_filters('autoptimize_filter_css_dontaggregate',false)) {

 			return false;

Indentation +22 added lines, -22 removed lines

@@ -162,29 +162,29 @@ 
                     // Remove the original style tag

                     $this->content = str_replace($tag,'',$this->content);

                 } else {

-					// excluded CSS, minify if getpath 

-					if (preg_match('#<link.*href=("|\')(.*)("|\')#Usmi',$tag,$source)) {

-						$url = current(explode('?',$source[2],2));

+                    // excluded CSS, minify if getpath 

+                    if (preg_match('#<link.*href=("|\')(.*)("|\')#Usmi',$tag,$source)) {

+                        $url = current(explode('?',$source[2],2));

                         $path = $this->getpath($url);

  					

-						if ($path && apply_filters('autoptimize_filter_css_minify_excluded',false)) {

-							$_CachedMinifiedUrl = $this->minify_single($path);

-

-							if (!empty($_CachedMinifiedUrl)) {

-								// replace orig URL with URL to cache

-								$newTag = str_replace($url, $_CachedMinifiedUrl, $tag);

-							} else {

-								$newTag = $tag;

-							}

+                        if ($path && apply_filters('autoptimize_filter_css_minify_excluded',false)) {

+                            $_CachedMinifiedUrl = $this->minify_single($path);

+

+                            if (!empty($_CachedMinifiedUrl)) {

+                                // replace orig URL with URL to cache

+                                $newTag = str_replace($url, $_CachedMinifiedUrl, $tag);

+                            } else {

+                                $newTag = $tag;

+                            }

 							

-							// remove querystring from URL

-							$_querystr = next(explode('?',$source[2],2));

-							$newTag = str_replace("?".$_querystr,"",$newTag);

-							// and replace

-							$this->content = str_replace($tag,$newTag,$this->content);

-						}

-					}					

-				}

+                            // remove querystring from URL

+                            $_querystr = next(explode('?',$source[2],2));

+                            $newTag = str_replace("?".$_querystr,"",$newTag);

+                            // and replace

+                            $this->content = str_replace($tag,$newTag,$this->content);

+                        }

+                    }					

+                }

             }

             return true;

         }

@@ -657,8 +657,8 @@ 
     }

     

     private function ismovable($tag) {

-		if (apply_filters('autoptimize_filter_css_dontaggregate',false)) {

-			return false;

+        if (apply_filters('autoptimize_filter_css_dontaggregate',false)) {

+            return false;

         } else if (!empty($this->whitelist)) {

             foreach ($this->whitelist as $match) {

                 if(strpos($tag,$match)!==false) {

classes/autoptimizeScripts.php

Indentation +19 added lines, -19 removed lines

@@ -128,26 +128,26 @@ 
                             // should we add flags?
                             foreach ($excludeJS as $exclTag => $exclFlags) {
                                 if ( strpos($origTag,$exclTag)!==false && in_array($exclFlags,array("async","defer")) ) {
-                                   $newTag = str_replace('<script ','<script '.$exclFlags.' ',$newTag);
+                                    $newTag = str_replace('<script ','<script '.$exclFlags.' ',$newTag);
                                 }
                             }
                         }
                         
-   						// should we minify the non-aggregated script?
-						if ($path && apply_filters('autoptimize_filter_js_minify_excluded',false)) {
-							$_CachedMinifiedUrl = $this->minify_single($path);
+                            // should we minify the non-aggregated script?
+                        if ($path && apply_filters('autoptimize_filter_js_minify_excluded',false)) {
+                            $_CachedMinifiedUrl = $this->minify_single($path);
 
-							// replace orig URL with minified URL from cache if so
-							if (!empty($_CachedMinifiedUrl)) {
-								$newTag = str_replace($url, $_CachedMinifiedUrl, $newTag);
-							}
+                            // replace orig URL with minified URL from cache if so
+                            if (!empty($_CachedMinifiedUrl)) {
+                                $newTag = str_replace($url, $_CachedMinifiedUrl, $newTag);
+                            }
 							
-							// remove querystring from URL in newTag
-							$_querystr = next(explode('?',$source[2],2));
-							$newTag = str_replace("?".$_querystr,"",$newTag);
-						}
+                            // remove querystring from URL in newTag
+                            $_querystr = next(explode('?',$source[2],2));
+                            $newTag = str_replace("?".$_querystr,"",$newTag);
+                        }
 
-						// should we move the non-aggregated script?
+                        // should we move the non-aggregated script?
                         if( $this->ismovable($newTag) ) {
                             // can be moved, flags and all
                             if( $this->movetolast($newTag) )    {
@@ -260,7 +260,7 @@ 
         
         //$this->jscode has all the uncompressed code now.
         if ($this->alreadyminified!==true) {
-          if (class_exists('JSMin') && apply_filters( 'autoptimize_js_do_minify' , 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)) {
@@ -274,10 +274,10 @@ 
                 $this->jscode = $this->inject_minified($this->jscode);
                 return false;
             }
-          } else {
-              $this->jscode = $this->inject_minified($this->jscode);
+            } else {
+                $this->jscode = $this->inject_minified($this->jscode);
             return false;
-          }
+            }
         }
         return true;
     }
@@ -339,8 +339,8 @@ 
     
     // Checks against the white- and blacklists
     private function ismergeable($tag) {
-		if (apply_filters('autoptimize_filter_js_dontaggregate',false)) {
-			return false;
+        if (apply_filters('autoptimize_filter_js_dontaggregate',false)) {
+            return false;
         } else if (!empty($this->whitelist)) {
             foreach ($this->whitelist as $match) {
                 if(strpos($tag,$match)!==false) {