SugarAPI\SDK\SugarAPI

Complexity

Total Complexity

30

Size/Duplication

Total Lines	120
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Importance

Changes	4
Bugs	2	Features	0

11 Methods

Rating	Name	Duplication	Size	Complexity
B	loadDefaults()	0	12	5
A	__construct()	0	10	3
A	setAuthOptions()	0	7	3
A	registerSDKEntryPoints()	0	7	2
A	registerEntryPoint()	0	6	2
A	__call()	0	17	4
A	login()	0	12	4
A	setInstance()	0	13	4
A	getURL()	0	3	1
A	getToken()	0	3	1
A	getAuthOptions()	0	3	1

<?php

namespace SugarAPI\SDK;

use SugarAPI\SDK\Exception\InitializationFailure;
use SugarAPI\SDK\Exception\InvalidEntryPoint;
use SugarAPI\SDK\Exception\AuthenticationException;
use SugarAPI\SDK\Exception\SDKException;

class SugarAPI {

    const API_URL = '/rest/v10/';

    protected static $_DEFAULTS = array();

    protected $instance;
    protected $url;

    protected $authOptions = array(
        'grant_type' => 'password',
        'username' => '',
        'password' => '',
        'client_id' => '',
        'client_secret' => '',
        'platform' => ''
    );
    protected $authToken;

    private $entryPoints = array();

    public function __construct($instance='',array $authOptions = array()){
        $this->loadDefaults();
        if (!empty($instance)){
            $this->setInstance($instance);
        }
        if (!empty($authOptions)){
            $this->setAuthOptions($authOptions);
        }
        $this->registerSDKEntryPoints();
    }

    protected function loadDefaults(){
        include __DIR__ .DIRECTORY_SEPARATOR.'defaults.php';
        if (isset($defaults)) {

The variable $defaults seems to never exist, and therefore isset should always return false. Did you maybe rename this variable?

            static::$_DEFAULTS = $defaults;
            if (isset($defaults['instance'])){
                $this->setInstance($defaults['instance']);
            }
            if (isset($defaults['auth']) && is_array($defaults['auth'])){
                $this->setAuthOptions($defaults['auth']);
            }
        }
    }

    public function setAuthOptions(array $options){
        foreach($this->authOptions as $key => $value){
            if (isset($options[$key])){
                $this->authOptions[$key] = $options[$key];
            }
        }
    }

    protected function registerSDKEntryPoints(){
        require __DIR__ .DIRECTORY_SEPARATOR.'EntryPoint' .DIRECTORY_SEPARATOR.'registry.php';
        foreach($entryPoints as $funcName => $className){

The variable $entryPoints does not exist. Did you forget to declare it?

            $className = "SugarAPI\\SDK\\EntryPoint\\".$className;
            $this->registerEntryPoint($funcName,$className);
        }
    }

    public function registerEntryPoint($funcName,$className){
        if (isset($this->entryPoints[$funcName])){
            throw new SDKException('SDK method already defined. Method '.$funcName.' references Class '.$className);
        }
        $this->entryPoints[$funcName] = $className;
    }

    public function __call($name,$params){
        if (array_key_exists($name,$this->entryPoints)){
            $Class = $this->entryPoints[$name];
            $EntryPoint = new $Class($this->url,$params);

            if ($EntryPoint->authRequired()){
                if (isset($this->authToken)) {
                    $EntryPoint->getRequest()->addHeader('OAuth-Token', $this->authToken->access_token);
                }else{
                    throw new AuthenticationException('no_auth');
                }
            }
            return $EntryPoint;
        }else{
            throw new SDKException('Method '.$name.', is not a registered method of the SugarAPI SDK');
        }
    }
    public function login(){
        if (empty($this->authOptions['username']) || empty($this->authOptions['password'])){
            throw new AuthenticationException("Username or Password was not provided.");
        }
        $EP = $this->accessToken();

The method accessToken does not exist on object<SugarAPI\SDK\SugarAPI>? Since you implemented __call, maybe consider adding a @method annotation.

        $response = $EP->data($this->authOptions)->execute()->getResponse();
        if ($response->getStatus()=='200'){
            $this->authToken = $response->getBody();
        }else{
            throw new AuthenticationException($response->getBody());
        }
    }
    public function setInstance($instance){
        if (strpos("https",$instance)!==FALSE){
            $this->secure = TRUE;

The property secure does not exist. Did you maybe forget to declare it?

        }
        if (strpos("http",$instance)===FALSE){
            $instance = "http://".$instance;
        }
        if (strpos("rest/v10",$instance)!==FALSE){
            $instance = str_replace("rest/v10","",$instance);
        }
        $this->instance = $instance;
        $this->url = rtrim($this->instance,"/").self::API_URL;
    }
    public function getURL(){
        return $this->url;
    }
    public function getToken(){
        return $this->authToken;
    }
    public function getAuthOptions(){
        return $this->authOptions;
    }
}