What is the issue:
Undefined property: stdClass::$success
The method is assuming that the success property will always be passed as part of the content body. Pipedrive does not pass this key on unsuccessful response codes (404 in this use case) so this will cause a status 500 from the dependency.
Where is the cause of the issue
File
vendor/devio/pipedrive/src/Http/Response.php
Codeblock
Line 53
/**
* Check if the request was successful.
*
* @return bool
*/
public function isSuccess()
{
if (! $this->getContent()) {
return false;
}
return $this->getContent()->success;
}Resolution
You can either use isset in the conditional statement or the return.
As there may be at some point a success property passed for status codes, it's probably best to use it in the if statement for this use case.
/**
* Check if the request was successful.
*
* @return bool
*/
public function isSuccess()
{
if (! $this->getContent() || ! isset($this->getContent()->success)) {
return false;
}
return $this->getContent()->success;
}
What is the issue:
The method is assuming that the
successproperty will always be passed as part of the content body. Pipedrive does not pass this key on unsuccessful response codes (404in this use case) so this will cause a status500from the dependency.Where is the cause of the issue
File
Codeblock
Resolution
You can either use
issetin the conditional statement or the return.As there may be at some point a
successproperty passed for status codes, it's probably best to use it in the if statement for this use case.