Merge pull request #446 from qrazi/patch-2

oscarotero · web-flow · commit 2c0e112f7332 · 2021-05-30T13:21:47.000+02:00
Fixed #441: force type to string when overriding adapter url
diff --git a/src/Providers/OEmbed/Youtube.php b/src/Providers/OEmbed/Youtube.php
@@ -21,7 +21,7 @@ public static function create(Adapter $adapter)
             // The Adapter URL should be forced to starting URL as well
             // to prevent consumers of the packages having to deal
             // with a URL to a consent page (#441)
-            $adapter->url = $response->getStartingUrl();
+            $adapter->url = (string) $response->getStartingUrl();
             return new static($response, $response->getStartingUrl());
         }
 

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ public static function create(Adapter $adapter)`
`21`	`21`	`// The Adapter URL should be forced to starting URL as well`
`22`	`22`	`// to prevent consumers of the packages having to deal`
`23`	`23`	`// with a URL to a consent page (#441)`
`24`		`- $adapter->url = $response->getStartingUrl();`
	`24`	`+ $adapter->url = (string) $response->getStartingUrl();`
`25`	`25`	`return new static($response, $response->getStartingUrl());`
`26`	`26`	`}`
`27`	`27`