From d5574bbc7698ac610411db64643e42cafac96ed7 Mon Sep 17 00:00:00 2001 From: Yoav Matchulsky Date: Tue, 1 Dec 2020 14:49:39 +0200 Subject: [PATCH] Fix: postApplication in ApplicationService doesn't work with Guzzle7 After upgrading to GuzzleHttp 7, it stopped treating null urls as empty and instead throws an InvalidArgumentException. This commit changes $url parameter's default value to an empty string instead of null. --- src/Clients/GuzzleClient.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Clients/GuzzleClient.php b/src/Clients/GuzzleClient.php index 5bb8f46..264a7fe 100644 --- a/src/Clients/GuzzleClient.php +++ b/src/Clients/GuzzleClient.php @@ -66,7 +66,7 @@ public function get($url="") * @return string * @throws GreenhouseAPIResponseException for non-200 responses */ - public function post(Array $postVars, Array $headers, $url=null) + public function post(Array $postVars, Array $headers, $url='') { try { $this->guzzleResponse = $this->_client->request(