TwentyTwoDigital\CashierFastspring\Fastspring\Fastspring

Complexity

Total Complexity

4

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__callStatic()	0	10	4

<?php
/**
 * This class helps to reach the Fastspring class with Laravel config and static
 * methods.
 *
 * @author    Bilal Gultekin <bilal@gultekin.me>
 * @version   0.1:
 * @copyright 2019 22 Digital
 * @license   MIT
 * @see       https://docs.fastspring.com/integrating-with-fastspring/fastspring-api
 */

namespace TwentyTwoDigital\CashierFastspring\Fastspring;

/**
 * This class describes the Fastspring implementation.
 */
class Fastspring
{
    /**
     * Instance of Fastspring class.
     *
     * @var array
     */
    public static $instance;

    /**
     * Static method.
     *
     * It is not useful to construct this Fastspring class everytime. This helps
     * to construct this class with the current config. if there is not any
     * constructed instance then construct and save it to self::$instance
     *
     * @param string $method     The method
     * @param array  $parameters The parameters for username and password
     *
     * @return self
     */
    public static function __callStatic($method, $parameters)
    {
        if (!self::$instance) {

The expression self::instance of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            $username = (getenv('FASTSPRING_USERNAME') ?: config('services.fastspring.username'));
            $password = (getenv('FASTSPRING_PASSWORD') ?: config('services.fastspring.password'));

            self::$instance = new ApiClient($username, $password);

It seems like new TwentyTwoDigital\CashierFastspring\Fastspring\ApiClient($username, $password) of type TwentyTwoDigital\CashierFastspring\Fastspring\ApiClient is incompatible with the declared type array of property $instance.

        }

        return call_user_func_array([self::$instance, $method], $parameters);
    }
}