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.
```php
/**
* 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
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 status500
from the dependency.Where is the cause of the issue
File
Codeblock