Merge pull request #147 from Codeinwp/fix/lazyload_query_urls

fix: lazyload query urls part of #145

composer.json

@@ -30,7 +30,7 @@
     "format": "phpcbf --standard=phpcs.xml --report-summary --report-source",
     "phpcs": "phpcs --standard=phpcs.xml --extensions=php -s",
     "phpmd": "phpmd inc text phpmd.xml",
-    "test": "phpunit --confirguration=phpunit.xml",
+    "test": "phpunit --configuration=phpunit.xml",
     "lint": "composer run-script phpcs"
   },
   "require": {

inc/lazyload_replacer.php

@@ -200,6 +200,8 @@ public function can_lazyload_for( $url, $tag = '' ) {
 		if ( false === Optml_Filters::should_do_image( $url, self::$filters[ Optml_Settings::FILTER_TYPE_LAZYLOAD ][ Optml_Settings::FILTER_FILENAME ] ) ) {
 			return false;
 		}
+		$url = strtok( $url, '?' );
+
 		$type = wp_check_filetype(
 			basename( $url ),
 			Optml_Config::$extensions

tests/test-lazyload.php

@@ -255,4 +255,12 @@ public function test_lazyload_json_data_disabled() {
 		$this->assertEquals( 2, substr_count( $replaced_content2, '/http:\/\/example.org' ) );
 	}
 
+	public function test_should_replace_query_string_url() {
+		$content = '<img src="https://example.org/photos/814499/pexels-photo-814499.jpeg?auto=compress&cs=tinysrgb&dpr=1&w=500" alt="">';
+		$replaced_content = Optml_Manager::instance()->replace_content($content);
+		$this->assertContains('i.optimole.com', $replaced_content);
+		$this->assertContains('data-opt-src', $replaced_content);
+		$this->assertContains('example.org', $replaced_content);
+	}
+
 }