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;

src/SugarAPI.php

Doc Comments +3 added lines

@@ -68,6 +68,9 @@
         }
     }
 
+    /**
+     * @param string $className
+     */
     public function registerEntryPoint($funcName,$className){
         if (isset($this->entryPoints[$funcName])){
             throw new SDKException('SDK method already defined. Method '.$funcName.' references Class '.$className);

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
 
 namespace SugarAPI\SDK;
 
-use SugarAPI\SDK\Exception\InitializationFailure;
-use SugarAPI\SDK\Exception\InvalidEntryPoint;
 use SugarAPI\SDK\Exception\AuthenticationException;
 use SugarAPI\SDK\Exception\SDKException;
 

Spacing +35 added lines, -35 removed lines

@@ -28,102 +28,102 @@
 
     private $entryPoints = array();
 
-    public function __construct($instance='',array $authOptions = array()){
+    public function __construct($instance = '', array $authOptions = array()) {
         $this->loadDefaults();
-        if (!empty($instance)){
+        if (!empty($instance)) {
             $this->setInstance($instance);
         }
-        if (!empty($authOptions)){
+        if (!empty($authOptions)) {
             $this->setAuthOptions($authOptions);
         }
         $this->registerSDKEntryPoints();
     }
 
-    protected function loadDefaults(){
-        include __DIR__ .DIRECTORY_SEPARATOR.'defaults.php';
+    protected function loadDefaults() {
+        include __DIR__.DIRECTORY_SEPARATOR.'defaults.php';
         if (isset($defaults)) {
             static::$_DEFAULTS = $defaults;
-            if (isset($defaults['instance'])){
+            if (isset($defaults['instance'])) {
                 $this->setInstance($defaults['instance']);
             }
-            if (isset($defaults['auth']) && is_array($defaults['auth'])){
+            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])){
+    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){
+    protected function registerSDKEntryPoints() {
+        require __DIR__.DIRECTORY_SEPARATOR.'EntryPoint'.DIRECTORY_SEPARATOR.'registry.php';
+        foreach ($entryPoints as $funcName => $className) {
             $className = "SugarAPI\\SDK\\EntryPoint\\".$className;
-            $this->registerEntryPoint($funcName,$className);
+            $this->registerEntryPoint($funcName, $className);
         }
     }
 
-    public function registerEntryPoint($funcName,$className){
-        if (isset($this->entryPoints[$funcName])){
+    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)){
+    public function __call($name, $params) {
+        if (array_key_exists($name, $this->entryPoints)) {
             $Class = $this->entryPoints[$name];
-            $EntryPoint = new $Class($this->url,$params);
+            $EntryPoint = new $Class($this->url, $params);
 
-            if ($EntryPoint->authRequired()){
+            if ($EntryPoint->authRequired()) {
                 if (isset($this->authToken)) {
                     $EntryPoint->getRequest()->addHeader('OAuth-Token', $this->authToken->access_token);
-                }else{
+                }else {
                     throw new AuthenticationException('no_auth');
                 }
             }
             return $EntryPoint;
-        }else{
+        }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'])){
+    public function login() {
+        if (empty($this->authOptions['username']) || empty($this->authOptions['password'])) {
             throw new AuthenticationException("Username or Password was not provided.");
         }
         $EP = $this->accessToken();
         $response = $EP->data($this->authOptions)->execute()->getResponse();
-        if ($response->getStatus()=='200'){
+        if ($response->getStatus()=='200') {
             $this->authToken = $response->getBody();
-        }else{
+        }else {
             throw new AuthenticationException($response->getBody());
         }
     }
-    public function setInstance($instance){
-        if (strpos("https",$instance)!==FALSE){
+    public function setInstance($instance) {
+        if (strpos("https", $instance)!==FALSE) {
             $this->secure = TRUE;
         }
-        if (strpos("http",$instance)===FALSE){
+        if (strpos("http", $instance)===FALSE) {
             $instance = "http://".$instance;
         }
-        if (strpos("rest/v10",$instance)!==FALSE){
-            $instance = str_replace("rest/v10","",$instance);
+        if (strpos("rest/v10", $instance)!==FALSE) {
+            $instance = str_replace("rest/v10", "", $instance);
         }
         $this->instance = $instance;
-        $this->url = rtrim($this->instance,"/").self::API_URL;
+        $this->url = rtrim($this->instance, "/").self::API_URL;
     }
-    public function getURL(){
+    public function getURL() {
         return $this->url;
     }
-    public function getToken(){
+    public function getToken() {
         return $this->authToken;
     }
-    public function getAuthOptions(){
+    public function getAuthOptions() {
         return $this->authOptions;
     }
 }
\ No newline at end of file

Braces +3 added lines, -3 removed lines

@@ -83,12 +83,12 @@ 
             if ($EntryPoint->authRequired()){
                 if (isset($this->authToken)) {
                     $EntryPoint->getRequest()->addHeader('OAuth-Token', $this->authToken->access_token);
-                }else{
+                } else{
                     throw new AuthenticationException('no_auth');
                 }
             }
             return $EntryPoint;
-        }else{
+        } else{
             throw new SDKException('Method '.$name.', is not a registered method of the SugarAPI SDK');
         }
     }
@@ -100,7 +100,7 @@ 
         $response = $EP->data($this->authOptions)->execute()->getResponse();
         if ($response->getStatus()=='200'){
             $this->authToken = $response->getBody();
-        }else{
+        } else{
             throw new AuthenticationException($response->getBody());
         }
     }

src/EntryPoint/GET/RecordFileField.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
 use SugarAPI\SDK\EntryPoint\Abstracts\GET\FileEntryPoint as GETFileEntryPoint;
 
-class RecordFileField extends GETFileEntryPoint{
+class RecordFileField extends GETFileEntryPoint {
 
     protected $_URL = '$module/$record/file/$field';
 

src/EntryPoint/Abstracts/AbstractEntryPoint.php

Spacing +36 added lines, -36 removed lines

@@ -22,13 +22,13 @@ 
     protected $Request;
     protected $Response;
 
-    public function __construct($url,$options = array()){
+    public function __construct($url, $options = array()) {
         $this->url = $url;
         $this->Module = $this->_MODULE;
 
         if (!empty($options)) {
             if (empty($this->Module)) {
-                if (strpos($this->_URL, '$module') !== FALSE) {
+                if (strpos($this->_URL, '$module')!==FALSE) {
                     $this->module($options[0]);
                     array_shift($options);
                 }
@@ -41,7 +41,7 @@ 
     /**
      * @inheritdoc
      */
-    public function module($module){
+    public function module($module) {
         $this->Module = $module;
         return $this;
     }
@@ -49,7 +49,7 @@ 
     /**
      * @inheritdoc
      */
-    public function options(array $options){
+    public function options(array $options) {
         $this->Options = $options;
         return $this;
     }
@@ -57,7 +57,7 @@ 
     /**
      * @inheritdoc
      */
-    public function data(array $data){
+    public function data(array $data) {
         $this->Data = $data;
         $this->Request->setBody($this->Data);
         return $this;
@@ -66,7 +66,7 @@ 
     /**
      * @inheritdoc
      */
-    public function execute(){
+    public function execute() {
         if ($this->verifyURL() && $this->validateData()) {
             $this->configureURL();
             $this->Request->setURL($this->url);
@@ -96,28 +96,28 @@ 
     /**
      *
      */
-    public function getData(){
+    public function getData() {
         return $this->Data;
     }
 
     /**
      * @inheritdoc
      */
-    public function getURL(){
+    public function getURL() {
         return $this->url;
     }
 
     /**
      * @inheritdoc
      */
-    public function getResponse(){
+    public function getResponse() {
         return $this->Response;
     }
 
     /**
      * @inheritdoc
      */
-    public function getRequest(){
+    public function getRequest() {
         return $this->Request;
     }
 
@@ -126,32 +126,32 @@ 
      * - Replaces $module with $this->Module
      * - Replcaes all other variables starting with $, with options in the order they were given
      */
-    protected function configureURL(){
+    protected function configureURL() {
         $url = $this->_URL;
-        if (strpos($this->_URL,"$")!==FALSE) {
-            if (count($this->Options) > 0 || !empty($this->Module)) {
+        if (strpos($this->_URL, "$")!==FALSE) {
+            if (count($this->Options)>0 || !empty($this->Module)) {
                 $urlParts = explode("/", $this->_URL);
                 $o = 0;
                 foreach ($urlParts as $key => $part) {
-                    if (strpos($part, '$module') !== FALSE) {
+                    if (strpos($part, '$module')!==FALSE) {
                         if (isset($this->Module)) {
                             $urlParts[$key] = $this->Module;
                             continue;
-                        } else {
+                        }else {
                             if (isset($this->Options[$o])) {
                                 $this->Module = $this->Options[$o];
                                 array_shift($this->Options);
                             }
                         }
                     }
-                    if (strpos($part, "$") !== FALSE) {
+                    if (strpos($part, "$")!==FALSE) {
                         if (isset($this->Options[$o])) {
                             $urlParts[$key] = $this->Options[$o];
                             $o++;
                         }
                     }
                 }
-                $url = implode($urlParts,"/");
+                $url = implode($urlParts, "/");
             }
         }
         $this->url = $this->url.$url;
@@ -160,15 +160,15 @@ 
     /**
      * Setup the Request Object property, setup on initial Construct of EntryPoint
      */
-    protected function setupRequest(){
+    protected function setupRequest() {
         $this->Request = new POST();
     }
 
     /**
      * Setup the Response Object Property, not called until after Request Execution
      */
-    protected function setupResponse(){
-        $this->Response = new JSONResponse($this->Request->getResponse(),$this->Request->getCurlObject());
+    protected function setupResponse() {
+        $this->Response = new JSONResponse($this->Request->getResponse(), $this->Request->getCurlObject());
     }
 
     /**
@@ -176,20 +176,20 @@ 
      * @return bool
      * @throws EntryPointExecutionFailure
      */
-    protected function verifyURL(){
-        $urlVarCount = substr_count($this->_URL,"$");
+    protected function verifyURL() {
+        $urlVarCount = substr_count($this->_URL, "$");
         $optionCount = 0;
-        if (!empty($this->Module)){
+        if (!empty($this->Module)) {
             $optionCount++;
         }
         $optionCount += count($this->Options);
-        if ($urlVarCount!==$optionCount){
-            if (empty($this->Module) && strpos($this->_URL,'$module')){
+        if ($urlVarCount!==$optionCount) {
+            if (empty($this->Module) && strpos($this->_URL, '$module')) {
                 throw new EntryPointException('Module is required for EntryPoint '.get_called_class());
-            }else{
+            }else {
                 throw new EntryPointException('EntryPoint URL ('.$this->_URL.') requires more parameters than passed.');
             }
-        }else{
+        }else {
             return true;
         }
     }
@@ -197,21 +197,21 @@ 
     /**
      * @return bool
      */
-    protected function validateData(){
-        if (empty($this->_REQUIRED_DATA)||count($this->_REQUIRED_DATA)==0){
+    protected function validateData() {
+        if (empty($this->_REQUIRED_DATA) || count($this->_REQUIRED_DATA)==0) {
             return true;
-        }else{
+        }else {
             $errors = array();
-            foreach($this->_REQUIRED_DATA as $property){
-                if (isset($this->Data[$property]) || $this->Data[$property]!==null){
+            foreach ($this->_REQUIRED_DATA as $property) {
+                if (isset($this->Data[$property]) || $this->Data[$property]!==null) {
                     continue;
-                }else{
+                }else {
                     $errors[] = $property;
                 }
             }
-            if (count($errors)>0){
-                throw new EntryPointException('EntryPoint requires specific properties in Request data. Missing the following '.implode($errors,","));
-            }else{
+            if (count($errors)>0) {
+                throw new EntryPointException('EntryPoint requires specific properties in Request data. Missing the following '.implode($errors, ","));
+            }else {
                 return true;
             }
         }

src/EntryPoint/Abstracts/GET/FileEntryPoint.php

Spacing +2 added lines, -2 removed lines

@@ -14,10 +14,10 @@
         $this->Request = new GETFile();
     }
     protected function setupResponse() {
-        $this->Response = new FileResponse($this->Request->getResponse(),$this->Request->getCurlObject(),$this->downloadDir);
+        $this->Response = new FileResponse($this->Request->getResponse(), $this->Request->getCurlObject(), $this->downloadDir);
     }
 
-    public function downloadTo($path){
+    public function downloadTo($path) {
         $this->downloadDir = $path;
         return $this;
     }

src/EntryPoint/POST/CreateRecord.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
 use SugarAPI\SDK\EntryPoint\Abstracts\POST\JSONEntryPoint as POSTEntryPoint;
 
-class CreateRecord extends POSTEntryPoint{
+class CreateRecord extends POSTEntryPoint {
 
     protected $_URL = '$module';
 

src/Response/JSON.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     /**
      * Get JSON Response
      */
-    public function json(){
+    public function json() {
         return $this->body;
     }
 

src/Response/Abstracts/AbstractResponse.php

Spacing +10 added lines, -10 removed lines

@@ -4,7 +4,7 @@ 
 
 use SugarAPI\SDK\Response\Interfaces\ResponseInterface;
 
-abstract class AbstractResponse implements ResponseInterface{
+abstract class AbstractResponse implements ResponseInterface {
 
     protected $CurlResponse;
     protected $headers;
@@ -12,7 +12,7 @@ 
     protected $status;
     protected $error;
 
-    public function __construct($curlResponse,$curlRequest){
+    public function __construct($curlResponse, $curlRequest) {
         $this->CurlResponse = $curlResponse;
         if ($this->checkErrors($curlRequest)) {
             $this->extractResponse($curlRequest);
@@ -20,18 +20,18 @@ 
         }
     }
 
-    protected function setStatus($curlRequest){
-        $this->status = curl_getinfo($curlRequest,CURLINFO_HTTP_CODE);
+    protected function setStatus($curlRequest) {
+        $this->status = curl_getinfo($curlRequest, CURLINFO_HTTP_CODE);
     }
 
-    protected function extractResponse($curlRequest){
-        $header_size = curl_getinfo($curlRequest,CURLINFO_HEADER_SIZE);
+    protected function extractResponse($curlRequest) {
+        $header_size = curl_getinfo($curlRequest, CURLINFO_HEADER_SIZE);
         $this->headers = substr($this->CurlResponse, 0, $header_size);
         $this->body = substr($this->CurlResponse, $header_size);
     }
 
-    protected function checkErrors($curlRequest){
-        if (curl_errno($curlRequest) !== CURLE_OK) {
+    protected function checkErrors($curlRequest) {
+        if (curl_errno($curlRequest)!==CURLE_OK) {
             $this->error = curl_error($curlRequest);
             return false;
         }
@@ -41,7 +41,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getStatus(){
+    public function getStatus() {
         return $this->status;
     }
 
@@ -62,7 +62,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getError(){
+    public function getError() {
         return $this->error;
     }
 

src/Response/File.php

Spacing +15 added lines, -15 removed lines

@@ -17,7 +17,7 @@ 
      */
     protected $destinationPath;
 
-    public function __construct($curlResponse, $curlRequest,$destination = null) {
+    public function __construct($curlResponse, $curlRequest, $destination = null) {
         parent::__construct($curlResponse, $curlRequest);
         $this->extractFileName();
         if (!empty($destination)) {
@@ -26,37 +26,37 @@ 
         }
     }
 
-    protected function setupDestiantion($destination = null){
-        if (empty($destination)){
+    protected function setupDestiantion($destination = null) {
+        if (empty($destination)) {
             $destination = sys_get_temp_dir().'/SugarAPI';
-            if (!file_exists($destination)){
-                mkdir($destination,0777);
+            if (!file_exists($destination)) {
+                mkdir($destination, 0777);
             }
         }
         $this->destinationPath = $destination;
     }
 
-    protected function extractFileName(){
-        foreach (explode("\r\n",$this->headers) as $header)
+    protected function extractFileName() {
+        foreach (explode("\r\n", $this->headers) as $header)
         {
-            if (strpos($header,'filename')!==FALSE){
-                $this->fileName = substr($header,(strpos($header,"\"")+1),-1);
+            if (strpos($header, 'filename')!==FALSE) {
+                $this->fileName = substr($header, (strpos($header, "\"") + 1), -1);
             }
         }
     }
 
-    public function getFileName(){
+    public function getFileName() {
         return $this->fileName;
     }
 
-    protected function writeFile(){
-        $fileHandle = fopen($this->file(),'w+');
-        fwrite($fileHandle,$this->body);
+    protected function writeFile() {
+        $fileHandle = fopen($this->file(), 'w+');
+        fwrite($fileHandle, $this->body);
         fclose($fileHandle);
     }
 
-    public function file(){
-        return rtrim($this->destinationPath,DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR.$this->fileName;
+    public function file() {
+        return rtrim($this->destinationPath, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR.$this->fileName;
     }
 
 }
\ No newline at end of file