MichaelJ2324 / sugarcrm-restv10-sdk

src/Request/Abstracts/AbstractRequest.php

Doc Comments +1 added lines

@@ -133,6 +133,7 @@
 
     /**
      * @inheritdoc
+     * @param string $body
      */
     public function setBody($body) {
         $this->body = $body;

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
     /**
      * Always make sure to destroy Curl Resource
      */
-    public function __destruct() {
+    public function __destruct(){
         if ($this->status!==self::STATUS_CLOSED){
             curl_close($this->CurlRequest);
         }

src/Exception/SDKException.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
 class EntryPointException extends SDKException {
 
-    public function __construct($message) {
+    public function __construct($message){
         parent::__construct($message);
     }
 

src/Client/Interfaces/ClientInterface.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
      * @param $EntryPoint - EPInterface Class Name
      * @return $this
      */
-    public function registerEntryPoint($function,$EntryPoint);
+    public function registerEntryPoint($function, $EntryPoint);
 
     /**
      * Login to the configured SugarCRM server
@@ -57,7 +57,7 @@ 
      * @param $client_id
      * @return boolean
      */
-    public static function storeToken($token,$client_id);
+    public static function storeToken($token, $client_id);
 
     /**
      * Get an SDK Clients authentication Token from Storage

src/Helpers/Helpers.php

Spacing +4 added lines, -4 removed lines

@@ -15,10 +15,10 @@ 
      * @return string
      */
     public static function configureAPIURL($instance){
-        if (strpos($instance,"http")===FALSE){
+        if (strpos($instance, "http")===FALSE){
             $instance = "http://".$instance;
         }
-        if (strpos($instance,"rest/v10")!==FALSE){
+        if (strpos($instance, "rest/v10")!==FALSE){
             $instance = str_replace("rest/v10", "", $instance);
         }
         return rtrim($instance, "/").self::API_URL;
@@ -39,8 +39,8 @@ 
     public static function getSDKEntryPointRegistry(){
         $entryPoints = array();
         require __DIR__.DIRECTORY_SEPARATOR.'registry.php';
-        foreach ($entryPoints as $funcName => $className) {
-            $className = "SugarAPI\\SDK\\EntryPoint\\" . $className;
+        foreach ($entryPoints as $funcName => $className){
+            $className = "SugarAPI\\SDK\\EntryPoint\\".$className;
             $entryPoints[$funcName] = $className;
         }
         return $entryPoints;

src/Client/Abstracts/AbstractClient.php

Doc Comments +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
     /**
      * @param $funcName
      * @param $className
-     * @return bool
+     * @return AbstractClient
      * @throws EntryPointException
      */
     public function registerEntryPoint($funcName, $className){

Spacing +23 added lines, -23 removed lines

@@ -63,7 +63,7 @@ 
      */
     protected $entryPoints = array();
 
-    public function __construct($server = '',array $credentials = array()){
+    public function __construct($server = '', array $credentials = array()){
         $server = (empty($server)?$this->server:$server);
         $this->setServer($server);
         $credentials = (empty($credentials)?$this->credentials:$credentials);
@@ -75,7 +75,7 @@ 
      * @inheritdoc
      * @param string $server
      */
-    public function setServer($server) {
+    public function setServer($server){
         $this->server = $server;
         $this->apiURL = Helpers::configureAPIURL($this->server);
         return $this;
@@ -84,7 +84,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getAPIUrl() {
+    public function getAPIUrl(){
         return $this->apiURL;
     }
 
@@ -94,9 +94,9 @@ 
      */
     public function setCredentials(array $credentials){
         $this->credentials = $credentials;
-        if (isset($this->credentials['client_id'])) {
+        if (isset($this->credentials['client_id'])){
             $token = static::getStoredToken($this->credentials['client_id']);
-            if (!empty($token)) {
+            if (!empty($token)){
                 $this->setToken($token);
             }
         }
@@ -129,7 +129,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getServer() {
+    public function getServer(){
         return $this->server;
     }
 
@@ -137,7 +137,7 @@ 
      * @inheritdoc
      */
     public function authenticated(){
-        return time() < $this->expiration;
+        return time()<$this->expiration;
     }
 
     /**
@@ -159,10 +159,10 @@ 
      */
     public function registerEntryPoint($funcName, $className){
         $implements = class_implements($className);
-        if (is_array($implements) && in_array('SugarAPI\SDK\EntryPoint\Interfaces\EPInterface',$implements)){
+        if (is_array($implements) && in_array('SugarAPI\SDK\EntryPoint\Interfaces\EPInterface', $implements)){
             $this->entryPoints[$funcName] = $className;
         }else{
-            throw new EntryPointException($className,'Class must extend SugarAPI\SDK\EntryPoint\Interfaces\EPInterface');
+            throw new EntryPointException($className, 'Class must extend SugarAPI\SDK\EntryPoint\Interfaces\EPInterface');
         }
         return $this;
     }
@@ -180,13 +180,13 @@ 
             $EntryPoint = new $Class($this->apiURL, $params);
 
             if ($EntryPoint->authRequired()){
-                if (isset($this->token) && $this->authenticated()) {
+                if (isset($this->token) && $this->authenticated()){
                     $EntryPoint->setAuth($this->getToken()->access_token);
                 }
             }
             return $EntryPoint;
         }else{
-            throw new EntryPointException($name,'Unregistered EntryPoint');
+            throw new EntryPointException($name, 'Unregistered EntryPoint');
         }
     }
 
@@ -194,13 +194,13 @@ 
      * @inheritdoc
      * @throws AuthenticationException - When Login request fails
      */
-    public function login() {
+    public function login(){
         $EP = new OAuth2Token($this->apiURL);
         $response = $EP->execute($this->credentials)->getResponse();
         if ($response->getStatus()=='200'){
             $this->setToken($response->getBody(FALSE));
-            static::storeToken($this->token,$this->credentials['client_id']);
-        } else {
+            static::storeToken($this->token, $this->credentials['client_id']);
+        }else{
             $error = $response->getBody();
             throw new AuthenticationException("Login Response [".$error['error']."] ".$error['error_message']);
         }
@@ -212,9 +212,9 @@ 
      * @throws AuthenticationException - When Refresh Request fails
      */
     public function refreshToken(){
-        if (isset($this->credentials['client_id'])&&
-            isset($this->credentials['client_secret'])&&
-            isset($this->token)) {
+        if (isset($this->credentials['client_id']) &&
+            isset($this->credentials['client_secret']) &&
+            isset($this->token)){
             $refreshOptions = array(
                 'client_id' => $this->credentials['client_id'],
                 'client_secret' => $this->credentials['client_secret'],
@@ -222,13 +222,13 @@ 
             );
             $EP = new RefreshToken($this->apiURL);
             $response = $EP->execute($refreshOptions)->getResponse();
-            if ($response->getStatus() == '200') {
+            if ($response->getStatus()=='200'){
                 $this->setToken($response->getBody(FALSE));
                 static::storeToken($this->token, $this->credentials['client_id']);
                 return TRUE;
-            } else {
+            }else{
                 $error = $response->getBody();
-                throw new AuthenticationException("Refresh Response [" . $error['error'] . "] " . $error['error_message']);
+                throw new AuthenticationException("Refresh Response [".$error['error']."] ".$error['error_message']);
             }
         }
         return FALSE;
@@ -258,7 +258,7 @@ 
      * @inheritdoc
      * @param \stdClass $token
      */
-    public static function storeToken($token, $client_id) {
+    public static function storeToken($token, $client_id){
         static::$_STORED_TOKENS[$client_id] = $token;
         return TRUE;
     }
@@ -266,14 +266,14 @@ 
     /**
      * @inheritdoc
      */
-    public static function getStoredToken($client_id) {
+    public static function getStoredToken($client_id){
         return (isset(static::$_STORED_TOKENS[$client_id])?static::$_STORED_TOKENS[$client_id]:NULL);
     }
 
     /**
      * @inheritdoc
      */
-    public static function removeStoredToken($client_id) {
+    public static function removeStoredToken($client_id){
         unset(static::$_STORED_TOKENS[$client_id]);
         return TRUE;
     }

src/EntryPoint/Abstracts/AbstractEntryPoint.php

Spacing +29 added lines, -29 removed lines

@@ -116,10 +116,10 @@ 
     protected $accessToken;
 
 
-    public function __construct($baseUrl,array $options = array()){
+    public function __construct($baseUrl, array $options = array()){
         $this->baseUrl = $baseUrl;
 
-        if (!empty($options)) {
+        if (!empty($options)){
             $this->setOptions($options);
         }
         $this->configureURL();
@@ -146,7 +146,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setAuth($accessToken) {
+    public function setAuth($accessToken){
         $this->accessToken = $accessToken;
         return $this;
     }
@@ -154,7 +154,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setUrl($url) {
+    public function setUrl($url){
         $this->Url = $url;
         return $this;
     }
@@ -162,7 +162,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setRequest(RequestInterface $Request) {
+    public function setRequest(RequestInterface $Request){
         $this->Request = $Request;
         return $this;
     }
@@ -170,7 +170,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setResponse(ResponseInterface $Response) {
+    public function setResponse(ResponseInterface $Response){
         $this->Response = $Response;
         return $this;
     }
@@ -218,13 +218,13 @@ 
      * @throws InvalidURLException
      */
     public function execute($data = NULL){
-        $data =  ($data === NULL?$this->Data:$data);
+        $data = ($data===NULL?$this->Data:$data);
         $this->configureData($data);
-        if (is_object($this->Request)) {
+        if (is_object($this->Request)){
             $this->configureRequest();
             $this->Request->send();
         }else{
-            throw new InvalidRequestException(get_called_class(),"Request property not configured");
+            throw new InvalidRequestException(get_called_class(), "Request property not configured");
         }
         return $this;
     }
@@ -232,7 +232,7 @@ 
     /**
      * @inheritdoc
      */
-    public function authRequired() {
+    public function authRequired(){
         return $this->_AUTH_REQUIRED;
     }
 
@@ -242,10 +242,10 @@ 
      * @throws RequiredDataException
      */
     protected function configureRequest(){
-        if ($this->verifyUrl()) {
+        if ($this->verifyUrl()){
             $this->Request->setURL($this->Url);
         }
-        if ($this->verifyData() && !empty($this->Data)) {
+        if ($this->verifyData() && !empty($this->Data)){
             $this->Request->setBody($this->Data);
         }
         $this->configureAuth();
@@ -255,7 +255,7 @@ 
      * Configures the authentication header on the Request object
      */
     protected function configureAuth(){
-        if ($this->authRequired()) {
+        if ($this->authRequired()){
             $this->Request->addHeader('OAuth-Token', $this->accessToken);
         }
     }
@@ -265,7 +265,7 @@ 
      * @var $data
      */
     protected function configureData($data){
-        if (!empty($this->_REQUIRED_DATA)&&is_array($data)){
+        if (!empty($this->_REQUIRED_DATA) && is_array($data)){
             $data = $this->configureDefaultData($data);
         }
         $this->setData($data);
@@ -277,7 +277,7 @@ 
      * @return array
      */
     protected function configureDefaultData(array $data){
-        foreach($this->_REQUIRED_DATA as $property => $value){
+        foreach ($this->_REQUIRED_DATA as $property => $value){
             if (!isset($data[$property]) && $value!==NULL){
                 $data[$property] = $value;
             }
@@ -292,18 +292,18 @@ 
      */
     protected function configureURL(){
         $url = $this->_URL;
-        if ($this->requiresOptions()) {
+        if ($this->requiresOptions()){
             $urlParts = explode("/", $this->_URL);
             $o = 0;
-            foreach ($urlParts as $key => $part) {
-                if (strpos($part, "$") !== FALSE) {
-                    if (isset($this->Options[$o])) {
+            foreach ($urlParts as $key => $part){
+                if (strpos($part, "$")!==FALSE){
+                    if (isset($this->Options[$o])){
                         $urlParts[$key] = $this->Options[$o];
                         $o++;
                     }
                 }
             }
-            $url = implode($urlParts,"/");
+            $url = implode($urlParts, "/");
         }
         $url = $this->baseUrl.$url;
         $this->setUrl($url);
@@ -315,9 +315,9 @@ 
      * @throws InvalidURLException
      */
     protected function verifyUrl(){
-        $UrlArray = explode("?",$this->Url);
-        if (strpos($UrlArray[0],"$") !== FALSE){
-            throw new InvalidURLException(get_called_class(),"Configured URL is ".$this->Url);
+        $UrlArray = explode("?", $this->Url);
+        if (strpos($UrlArray[0], "$")!==FALSE){
+            throw new InvalidURLException(get_called_class(), "Configured URL is ".$this->Url);
         }
         return true;
     }
@@ -328,7 +328,7 @@ 
      * @throws RequiredDataException
      */
     protected function verifyData(){
-        if (isset($this->_DATA_TYPE)||!empty($this->_DATA_TYPE)) {
+        if (isset($this->_DATA_TYPE) || !empty($this->_DATA_TYPE)){
             $this->verifyDataType();
         }
         if (!empty($this->_REQUIRED_DATA)){
@@ -343,8 +343,8 @@ 
      * @throws RequiredDataException
      */
     protected function verifyDataType(){
-        if (gettype($this->Data) !== $this->_DATA_TYPE) {
-            throw new RequiredDataException(get_called_class(),"Valid DataType is {$this->_DATA_TYPE}");
+        if (gettype($this->Data)!==$this->_DATA_TYPE){
+            throw new RequiredDataException(get_called_class(), "Valid DataType is {$this->_DATA_TYPE}");
         }
         return true;
     }
@@ -356,13 +356,13 @@ 
      */
     protected function verifyRequiredData(){
         $errors = array();
-        foreach($this->_REQUIRED_DATA as $property => $defaultValue){
+        foreach ($this->_REQUIRED_DATA as $property => $defaultValue){
             if (!isset($this->Data[$property])){
                 $errors[] = $property;
             }
         }
         if (count($errors)>0){
-            throw new RequiredDataException(get_called_class(),"Missing data for ".implode(",",$errors));
+            throw new RequiredDataException(get_called_class(), "Missing data for ".implode(",", $errors));
         }
         return true;
     }
@@ -372,7 +372,7 @@ 
      * @return bool
      */
     protected function requiresOptions(){
-        return strpos($this->_URL,"$") === FALSE?FALSE:TRUE;
+        return strpos($this->_URL, "$")===FALSE?FALSE:TRUE;
     }
 
 }
\ No newline at end of file

src/EntryPoint/POST/ModuleRecordFileField.php

Spacing +4 added lines, -4 removed lines

@@ -32,19 +32,19 @@
      * @throws RequiredOptionsException
      */
     protected function configureData($data){
-        if (is_string($data)) {
-            if (!empty($this->Options)) {
+        if (is_string($data)){
+            if (!empty($this->Options)){
                 $fileField = end($this->Options);
                 $data = array(
                     $fileField => $data
                 );
-            } else {
+            }else{
                 throw new RequiredOptionsException(get_called_class(), "Options are required, when passing String for data.");
             }
         }
         if (is_array($data)){
             foreach ($data as $key => $value){
-                if (!array_key_exists($key,$this->_REQUIRED_DATA)){
+                if (!array_key_exists($key, $this->_REQUIRED_DATA)){
                     $data[$key] = $this->setFileFieldValue($value);
                 }
             }

src/EntryPoint/POST/Bulk.php

Spacing +7 added lines, -7 removed lines

@@ -30,23 +30,23 @@
         'method' => ''
     );
 
-    protected function configureData($data) {
-        if (!isset($data['requests'])) {
+    protected function configureData($data){
+        if (!isset($data['requests'])){
             $requestData = array(
                 'requests' => array()
             );
             $counter = 0;
-            foreach ($data as $key => $EntryPoint) {
-                if (is_object($EntryPoint)) {
+            foreach ($data as $key => $EntryPoint){
+                if (is_object($EntryPoint)){
                     $requestData['requests'][$counter] = $this->bulkRequest;
                     $requestData['requests'][$counter]['method'] = $EntryPoint->getRequest()->getType();
-                    if ($requestData['requests'][$counter]['method'] == "POST" || $requestData['requests'][$counter]['method'] == "PUT") {
+                    if ($requestData['requests'][$counter]['method']=="POST" || $requestData['requests'][$counter]['method']=="PUT"){
                         $requestData['requests'][$counter]['data'] = json_encode($EntryPoint->getData());
-                    } else {
+                    }else{
                         unset($requestData['requests'][$counter]['data']);
                     }
                     $requestData['requests'][$counter]['headers'] = $EntryPoint->getRequest()->getHeaders();
-                    $requestData['requests'][$counter]['url'] = "v10/" . str_replace($this->baseUrl, "", $EntryPoint->getUrl());
+                    $requestData['requests'][$counter]['url'] = "v10/".str_replace($this->baseUrl, "", $EntryPoint->getUrl());
 
                     $counter++;
                 }

src/Response/File.php

Spacing +6 added lines, -6 removed lines

@@ -22,7 +22,7 @@ 
     protected $destinationPath;
 
     public function __construct($curlRequest, $curlResponse = NULL, $destination = NULL){
-        parent::__construct($curlRequest,$curlResponse);
+        parent::__construct($curlRequest, $curlResponse);
         $this->setDestinationPath($destination);
     }
 
@@ -31,10 +31,10 @@ 
      * Extract Filename from Headers
      * @param mixed $curlResponse
      */
-    public function setCurlResponse($curlResponse) {
+    public function setCurlResponse($curlResponse){
         parent::setCurlResponse($curlResponse);
-        if (!$this->error) {
-            if (empty($this->fileName)) {
+        if (!$this->error){
+            if (empty($this->fileName)){
                 $this->extractFileName();
             }
             $this->writeFile();
@@ -95,8 +95,8 @@ 
                 mkdir($this->destinationPath, 0777);
             }
             $file = $this->file();
-            $fileHandle = fopen($file,'w+');
-            fwrite($fileHandle,$this->body);
+            $fileHandle = fopen($file, 'w+');
+            fwrite($fileHandle, $this->body);
             fclose($fileHandle);
             return $file;
         }else{