Protocol-less URL support

I’ll look into that, a solution for that might just still make it into 1.8.2. Now that I come to think of it; maybe I should switch to protocol-less URL’s. What do you think?

Hi Ross;
Not going to change output for now, but regarding aggregating protocol-less JS/CSS-links, could you try this?

1. Open up /wp-content/plugins/autoptimize/classes/autoptimizeBase.php
2. On line 26 change:

//Converts an URL to a full path
	protected function getpath($url) {
		if ((strpos($url,'//')===false) && 	(strpos($url,parse_url(AUTOPTIMIZE_WP_SITE_URL,PHP_URL_HOST))===false))		{
			$url = AUTOPTIMIZE_WP_SITE_URL.$url;
		}
        $path = str_replace(AUTOPTIMIZE_WP_ROOT_URL,'',$url);
	    if(preg_match('#^((https?|ftp):)?//#i',$path)) {
            /** External script/css (adsense, etc) */
       		return false;
       	}
        $path = str_replace('//','/',WP_ROOT_DIR.$path);
        return $path;
	}

into:

//Converts an URL to a full path
	protected function getpath($url) {
		if (strpos($url,'//')===0) {
			$url = "http:".$url;
		} else if ((strpos($url,'//')===false) && (strpos($url,parse_url(AUTOPTIMIZE_WP_SITE_URL,PHP_URL_HOST))===false)) {
			$url = AUTOPTIMIZE_WP_SITE_URL.$url;
		}
        $path = str_replace(AUTOPTIMIZE_WP_ROOT_URL,'',$url);
	    if(preg_match('#^((https?|ftp):)?//#i',$path)) {
            /** External script/css (adsense, etc) */
       		return false;
       	}
        $path = str_replace('//','/',WP_ROOT_DIR.$path);
        return $path;
	}

3. save file
4. empty Autoptimize Cache (and whatever page caching plugin you use).

Based on (limited) local tests this should do the trick. If this indeed does work, it’ll be included in 1.8.2.

frank

great, thanks for the feedback. enjoy your weekend!

frank