Surge\LaravelSalesforce\Salesforce

Complexity

Total Complexity

24

Size/Duplication

Total Lines	162
Duplicated Lines	13.58 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	2
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
B	__call()	0	26	6
A	callCreateOnObject()	11	11	2
A	callDeleteOnObject()	11	11	2
A	callGetOnObject()	0	11	2
A	callUpdateOnObject()	0	11	2
B	callExistsOnObject()	0	17	5
A	runQuery()	0	10	3
A	getReport()	0	4	1

<?php

namespace Surge\LaravelSalesforce;

use GuzzleHttp\ClientInterface;
use Surge\LaravelSalesforce\Objects\BaseObject;

class Salesforce
{
    /**
     * @var string
     */
    protected $objName;

    public $baseUrl;

    public $instanceUrl;

    public $client;

    /**
     * Salesforce constructor.
     *
     * @param ClientInterface $client
     * @param string          $url
     * @param string          $instanceUrl
     */
    public function __construct(ClientInterface $client, string $url, string $instanceUrl)
    {
        $this->client = $client;
        $this->baseUrl = $url;
        $this->instanceUrl = $instanceUrl;
    }

    /**
     * @param  string $method
     * @param  array  $args
     * @return bool|mixed|string
     */
    public function __call($method, $args)
    {
        if (starts_with($method, 'create')) {
            return $this->callCreateOnObject($method, $args);
        }

        if (starts_with($method, 'update')) {
            return $this->callUpdateOnObject($method, $args);
        }

        if (starts_with($method, 'delete')) {
            return $this->callDeleteOnObject($method, $args);
        }

        if (starts_with($method, 'get')) {
            return $this->callGetOnObject($method, $args);
        }

        if (starts_with($method, 'exists')) {
            return $this->callExistsOnObject($method, $args);
        }

        $class = new BaseObject($this);

        return call_user_func_array([$class, $method], $args);
    }

    /**
     * Create object dynamically
     *
     * @param $method
     * @param $args
     * @return bool
     */
    private function callCreateOnObject($method, $args)

This method seems to be duplicated in your project.

    {
        $type = substr($method, 6);
        $class = '\\Surge\\LaravelSalesforce\\Objects\\' . $type;

        if (class_exists($class)) {
            return (new $class($this))->create($args[0]);
        }

        return (new BaseObject($this, $type))->create($args[0]);
    }

    private function callUpdateOnObject($method, $args)
    {
        $type = substr($method, 6);
        $class = '\\Surge\\LaravelSalesforce\\Objects\\' . $type;

        if (class_exists($class)) {
            return (new $class($this))->update($args[0], $args[1]);
        }

        return (new BaseObject($this, $type))->update($type, $args[0]);
    }

    private function callDeleteOnObject($method, $args)

This method seems to be duplicated in your project.

    {
        $type = substr($method, 6);
        $class = '\\Surge\\LaravelSalesforce\\Objects\\' . $type;

        if (class_exists($class)) {
            return (new $class($this))->delete($args[0]);
        }

        return (new BaseObject($this, $type))->delete($args[0]);
    }

    private function callGetOnObject($method, $args)
    {
        $type = substr($method, 3);
        $class = '\\Surge\\LaravelSalesforce\\Objects\\' . $type;

        if (class_exists($class)) {
            return (new $class($this))->get($args[0]);
        }

        return (new BaseObject($this, $type))->get($args[0]);
    }

    private function callExistsOnObject($method, $args)
    {
        $type = substr($method, 6);
        $class = '\\Surge\\LaravelSalesforce\\Objects\\' . $type;

        if (class_exists($class)) {
            if (isset($args[1]) && isset($args[2])) { //second and third params are optional
                return (new $class($this))->exists($args[0], $args[1], $args[2]);
            } elseif (isset($args[1])) {
                return (new $class($this))->exists($args[0], $args[1]);
            } else {
                return (new $class($this))->exists($args[0]);
            }
        }

        return (new BaseObject($this, $type))->get($args[0]);
    }

    /**
     * Run query
     *
     * @param $query
     * @return mixed
     */
    public function runQuery($query)
    {
        $response = $this->query($query);

The method query does not exist on object<Surge\LaravelSalesforce\Salesforce>? Since you implemented __call, maybe consider adding a @method annotation.

        if ($response && $response->totalSize > 0) {
            return $response->records;
        }

        return false;
    }

    /**
     * Run report.
     *
     * @param string $id
     * @param bool   $includeDetails
     * @return mixed
     *
     */
    public function getReport(string $id, bool $includeDetails = true)
    {
        return $this->report($id, $includeDetails);

The method report() does not exist on Surge\LaravelSalesforce\Salesforce. Did you maybe mean getReport()?

    }
}