Skip to content

Commit

Permalink
Fix: postApplication in ApplicationService doesn't work with Guzzle7
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
Yoav Matchulsky committed Dec 1, 2020
1 parent c322c56 commit d5574bb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/Clients/GuzzleClient.php
Original file line number Diff line number Diff line change
Expand Up @@ -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(
Expand Down

0 comments on commit d5574bb

Please sign in to comment.