Missed return statement for account deauthorize method. Or wrong hinting in phpDoc

[fre5h]

Hello. Check this method https://github.com/stripe/stripe-php/blob/master/lib/Account.php#L116
I also copy its body here too

    /**
     * @param array|null $clientId
     * @param array|string|null $opts
     *
     * @return StripeObject Object containing the response from the API.
     */
    public function deauthorize($clientId = null, $opts = null)
    {
        $params = [
            'client_id' => $clientId,
            'stripe_user_id' => $this->id,
        ];
        OAuth::deauthorize($params, $opts);
    }

There is a @return docBlock which says that StripeObject should be returned. But return statement is missed.
This can be fixed in two ways.

First. Remove docBlock, so the method won't return anything

    /**
     * @param array|null $clientId
     * @param array|string|null $opts
     */
    public function deauthorize($clientId = null, $opts = null)
    {
        $params = [
            'client_id' => $clientId,
            'stripe_user_id' => $this->id,
        ];
        OAuth::deauthorize($params, $opts);
    }

Second. Add missed return statement. I think it is better solution

    /**
     * @param array|null $clientId
     * @param array|string|null $opts
     *
     * @return StripeObject Object containing the response from the API.
     */
    public function deauthorize($clientId = null, $opts = null)
    {
        $params = [
            'client_id' => $clientId,
            'stripe_user_id' => $this->id,
        ];
        
        return OAuth::deauthorize($params, $opts);
    }

I'm using stripe/stripe-php": "^6.8" same issue in master branch too.

[ob-stripe]

Hi @fre5h, thanks for the report! I've implemented the second solution in #486.

Note that in practice, there shouldn't be any need to check the returned object (IIRC it only contains the ID of the account anyway). You would simply wrap the call in a try/catch block to ensure that it succeeded.

[ob-stripe]

Released as 6.8.2.