MichaelJ2324 / sugarcrm-restv10-sdk

examples/GetFile.php

Braces +2 added lines, -2 removed lines

@@ -14,12 +14,12 @@
             echo "File downloaded to ".$response->file();
         }
 
-    }else{
+    } else{
         echo "Failed to retrieve Note<br>";
         echo "Response: ".$response->getStatus()."<br>";
         print_r($response->getBody());
     }
 
-}catch(\SugarAPI\SDK\Exception\SDKException $ex){
+} catch(\SugarAPI\SDK\Exception\SDKException $ex){
     print $ex->getMessage();
 }
\ No newline at end of file

Spacing +4 added lines, -4 removed lines

@@ -1,13 +1,13 @@ 
 <?php
 
-require_once __DIR__ . '/../vendor/autoload.php';
+require_once __DIR__.'/../vendor/autoload.php';
 
 $recordID = 'ac096703-67fd-8dd2-980a-57097932f07a';
 
 try{
-    $SugarAPI = new \SugarAPI\SDK\SugarAPI('instances.this/Ent/7700/',array('username' => 'admin','password'=>'asdf'));
+    $SugarAPI = new \SugarAPI\SDK\SugarAPI('instances.this/Ent/7700/', array('username' => 'admin', 'password'=>'asdf'));
     $SugarAPI->login();
-    $EP = $SugarAPI->getAttachment('Notes',$recordID,'filename')->downloadTo(__DIR__)->execute();
+    $EP = $SugarAPI->getAttachment('Notes', $recordID, 'filename')->downloadTo(__DIR__)->execute();
     $response = $EP->getResponse();
     if ($response->getStatus()=='200'){
         if (file_exists($response->file())){
@@ -20,6 +20,6 @@ 
         print_r($response->getBody());
     }
 
-}catch(\SugarAPI\SDK\Exception\SDKException $ex){
+}catch (\SugarAPI\SDK\Exception\SDKException $ex){
     print $ex->getMessage();
 }
\ No newline at end of file

src/Request/POST.php

Spacing +2 added lines, -2 removed lines

@@ -4,7 +4,7 @@ 
 
 use SugarAPI\SDK\Request\Abstracts\AbstractRequest;
 
-class POST extends AbstractRequest{
+class POST extends AbstractRequest {
 
     /**
      * @inheritdoc
@@ -32,7 +32,7 @@ 
      * JSON Encode Body
      * @inheritdoc
      */
-    public function setBody($body) {
+    public function setBody($body){
         return parent::setBody(json_encode($body));
     }
 }
\ No newline at end of file

Braces +2 added lines, -2 removed lines

@@ -4,7 +4,7 @@ 
 
 use SugarAPI\SDK\Request\Abstracts\AbstractRequest;
 
-class POST extends AbstractRequest{
+class POST extends AbstractRequest {
 
     /**
      * @inheritdoc
@@ -32,7 +32,7 @@ 
      * JSON Encode Body
      * @inheritdoc
      */
-    public function setBody($body) {
+    public function setBody($body){
         return parent::setBody(json_encode($body));
     }
 }
\ No newline at end of file

src/Response/Abstracts/AbstractResponse.php

Spacing +8 added lines, -8 removed lines

@@ -4,7 +4,7 @@ 
 
 use SugarAPI\SDK\Response\Interfaces\ResponseInterface;
 
-abstract class AbstractResponse implements ResponseInterface{
+abstract class AbstractResponse implements ResponseInterface {
 
     /**
      * Full Curl Response
@@ -36,9 +36,9 @@ 
      */
     protected $error;
 
-    public function __construct($curlResponse,$curlRequest){
+    public function __construct($curlResponse, $curlRequest){
         $this->CurlResponse = $curlResponse;
-        if ($this->checkErrors($curlRequest)) {
+        if ($this->checkErrors($curlRequest)){
             $this->extractResponse($curlRequest);
         }
         $this->setStatus($curlRequest);
@@ -49,7 +49,7 @@ 
      * @param $curlRequest - Curl resource
      */
     protected function setStatus($curlRequest){
-        $this->status = curl_getinfo($curlRequest,CURLINFO_HTTP_CODE);
+        $this->status = curl_getinfo($curlRequest, CURLINFO_HTTP_CODE);
     }
 
     /**
@@ -57,7 +57,7 @@ 
      * @param $curlRequest
      */
     protected function extractResponse($curlRequest){
-        $header_size = curl_getinfo($curlRequest,CURLINFO_HEADER_SIZE);
+        $header_size = curl_getinfo($curlRequest, CURLINFO_HEADER_SIZE);
         $this->headers = substr($this->CurlResponse, 0, $header_size);
         $this->body = substr($this->CurlResponse, $header_size);
     }
@@ -68,7 +68,7 @@ 
      * @return bool
      */
     protected function checkErrors($curlRequest){
-        if (curl_errno($curlRequest) !== CURLE_OK) {
+        if (curl_errno($curlRequest)!==CURLE_OK){
             $this->error = curl_error($curlRequest);
             return false;
         }
@@ -85,14 +85,14 @@ 
     /**
      * @inheritdoc
      */
-    public function getBody() {
+    public function getBody(){
         return $this->body;
     }
 
     /**
      * @inheritdoc
      */
-    public function getHeaders() {
+    public function getHeaders(){
         return $this->headers;
     }
 

Braces +5 added lines, -5 removed lines

@@ -4,7 +4,7 @@ 
 
 use SugarAPI\SDK\Response\Interfaces\ResponseInterface;
 
-abstract class AbstractResponse implements ResponseInterface{
+abstract class AbstractResponse implements ResponseInterface {
 
     /**
      * Full Curl Response
@@ -38,7 +38,7 @@ 
 
     public function __construct($curlResponse,$curlRequest){
         $this->CurlResponse = $curlResponse;
-        if ($this->checkErrors($curlRequest)) {
+        if ($this->checkErrors($curlRequest)){
             $this->extractResponse($curlRequest);
         }
         $this->setStatus($curlRequest);
@@ -68,7 +68,7 @@ 
      * @return bool
      */
     protected function checkErrors($curlRequest){
-        if (curl_errno($curlRequest) !== CURLE_OK) {
+        if (curl_errno($curlRequest) !== CURLE_OK){
             $this->error = curl_error($curlRequest);
             return false;
         }
@@ -85,14 +85,14 @@ 
     /**
      * @inheritdoc
      */
-    public function getBody() {
+    public function getBody(){
         return $this->body;
     }
 
     /**
      * @inheritdoc
      */
-    public function getHeaders() {
+    public function getHeaders(){
         return $this->headers;
     }
 

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -70,9 +70,9 @@
     protected function checkErrors($curlRequest){
         if (curl_errno($curlRequest) !== CURLE_OK) {
             $this->error = curl_error($curlRequest);
-            return false;
+            return FALSE;
         }
-        return true;
+        return TRUE;
     }
 
     /**

src/Response/JSON.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     /**
      * @inheritdoc
      */
-    public function getBody() {
+    public function getBody(){
         return json_decode($this->body);
     }
 

Braces +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     /**
      * @inheritdoc
      */
-    public function getBody() {
+    public function getBody(){
         return json_decode($this->body);
     }
 

src/Response/File.php

Spacing +9 added lines, -9 removed lines

@@ -18,10 +18,10 @@ 
      */
     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)) {
+        if (!empty($destination)){
             $this->setupDestination($destination);
             $this->writeFile();
         }
@@ -35,7 +35,7 @@ 
         if (empty($destination)){
             $destination = sys_get_temp_dir().'/SugarAPI';
             if (!file_exists($destination)){
-                mkdir($destination,0777);
+                mkdir($destination, 0777);
             }
         }
         $this->destinationPath = $destination;
@@ -45,10 +45,10 @@ 
      * Extract the filename from the Headers, and store it in filename property
      */
     protected function extractFileName(){
-        foreach (explode("\r\n",$this->headers) as $header)
+        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);
             }
         }
     }
@@ -65,8 +65,8 @@ 
      * Write the downloaded file
      */
     protected function writeFile(){
-        $fileHandle = fopen($this->file(),'w+');
-        fwrite($fileHandle,$this->body);
+        $fileHandle = fopen($this->file(), 'w+');
+        fwrite($fileHandle, $this->body);
         fclose($fileHandle);
     }
 
@@ -75,7 +75,7 @@ 
      * @return string
      */
     public function file(){
-        return rtrim($this->destinationPath,DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR.$this->fileName;
+        return rtrim($this->destinationPath, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR.$this->fileName;
     }
 
 }
\ No newline at end of file

Braces +3 added lines, -4 removed lines

@@ -18,10 +18,10 @@ 
      */
     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)) {
+        if (!empty($destination)){
             $this->setupDestination($destination);
             $this->writeFile();
         }
@@ -45,8 +45,7 @@ 
      * Extract the filename from the Headers, and store it in filename property
      */
     protected function extractFileName(){
-        foreach (explode("\r\n",$this->headers) as $header)
-        {
+        foreach (explode("\r\n",$this->headers) as $header){
             if (strpos($header,'filename')!==FALSE){
                 $this->fileName = substr($header,(strpos($header,"\"")+1),-1);
             }

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -18,7 +18,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)) {
@@ -31,7 +31,7 @@ 
      * Configure the Destination path to store the File response
      * @param null $destination
      */
-    protected function setupDestination($destination = null){
+    protected function setupDestination($destination = NULL){
         if (empty($destination)){
             $destination = sys_get_temp_dir().'/SugarAPI';
             if (!file_exists($destination)){

src/Exception/EntryPointException.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);
     }
 

Braces +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/Exception/SDKException.php

Spacing +2 added lines, -2 removed lines

@@ -2,9 +2,9 @@
 
 namespace SugarAPI\SDK\Exception;
 
-class SDKException extends \Exception{
+class SDKException extends \Exception {
 
-    public function __construct($message = 'Unknown SugarAPI SDK Exception Occurred.') {
+    public function __construct($message = 'Unknown SugarAPI SDK Exception Occurred.'){
         parent::__construct($message);
     }
 

Braces +2 added lines, -2 removed lines

@@ -2,9 +2,9 @@
 
 namespace SugarAPI\SDK\Exception;
 
-class SDKException extends \Exception{
+class SDKException extends \Exception {
 
-    public function __construct($message = 'Unknown SugarAPI SDK Exception Occurred.') {
+    public function __construct($message = 'Unknown SugarAPI SDK Exception Occurred.'){
         parent::__construct($message);
     }
 

src/Exception/AuthenticationException.php

Spacing +3 added lines, -3 removed lines

@@ -2,12 +2,12 @@
 
 namespace SugarAPI\SDK\Exception;
 
-class AuthenticationException extends SDKException{
+class AuthenticationException extends SDKException {
 
     protected $message = 'Authentication Exception occurred when attempting to Login to SugarCRM instance. Message: %s';
 
-    public function __construct($message) {
-        $message = sprintf($this->message,$message);
+    public function __construct($message){
+        $message = sprintf($this->message, $message);
         parent::__construct($message);
     }
 

Braces +2 added lines, -2 removed lines

@@ -2,11 +2,11 @@
 
 namespace SugarAPI\SDK\Exception;
 
-class AuthenticationException extends SDKException{
+class AuthenticationException extends SDKException {
 
     protected $message = 'Authentication Exception occurred when attempting to Login to SugarCRM instance. Message: %s';
 
-    public function __construct($message) {
+    public function __construct($message){
         $message = sprintf($this->message,$message);
         parent::__construct($message);
     }

src/EntryPoint/Abstracts/AbstractEntryPoint.php

Spacing +24 added lines, -24 removed lines

@@ -85,13 +85,13 @@ 
      */
     protected $accessToken;
 
-    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 (!empty($options)){
+            if (empty($this->Module)){
+                if (strpos($this->_URL, '$module')!==FALSE){
                     $this->module($options[0]);
                     array_shift($options);
                 }
@@ -130,7 +130,7 @@ 
      * @inheritdoc
      */
     public function execute(){
-        if ($this->verifyURL() && $this->validateData()) {
+        if ($this->verifyURL() && $this->validateData()){
             $this->configureURL();
             $this->Request->setURL($this->url);
             $this->Request->send();
@@ -144,14 +144,14 @@ 
     /**
      * @inheritdoc
      */
-    public function authRequired() {
+    public function authRequired(){
         return $this->_AUTH_REQUIRED;
     }
 
     /**
      * @inheritdoc
      */
-    public function configureAuth($accessToken) {
+    public function configureAuth($accessToken){
         if ($this->authRequired()){
             $this->accessToken = $accessToken;
             $this->Request->addHeader('OAuth-Token', $accessToken);
@@ -162,7 +162,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getModule() {
+    public function getModule(){
         return $this->Module;
     }
 
@@ -201,30 +201,30 @@ 
      */
     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 (isset($this->Module)) {
+                foreach ($urlParts as $key => $part){
+                    if (strpos($part, '$module')!==FALSE){
+                        if (isset($this->Module)){
                             $urlParts[$key] = $this->Module;
                             continue;
-                        } else {
-                            if (isset($this->Options[$o])) {
+                        }else{
+                            if (isset($this->Options[$o])){
                                 $this->Module = $this->Options[$o];
                                 array_shift($this->Options);
                             }
                         }
                     }
-                    if (strpos($part, "$") !== FALSE) {
-                        if (isset($this->Options[$o])) {
+                    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;
@@ -241,7 +241,7 @@ 
      * Setup the Response Object Property, not called until after Request Execution
      */
     protected function setupResponse(){
-        $this->Response = new JSONResponse($this->Request->getResponse(),$this->Request->getCurlObject());
+        $this->Response = new JSONResponse($this->Request->getResponse(), $this->Request->getCurlObject());
     }
 
     /**
@@ -250,14 +250,14 @@ 
      * @throws EntryPointException
      */
     protected function verifyURL(){
-        $urlVarCount = substr_count($this->_URL,"$");
+        $urlVarCount = substr_count($this->_URL, "$");
         $optionCount = 0;
         if (!empty($this->Module)){
             $optionCount++;
         }
         $optionCount += count($this->Options);
         if ($urlVarCount!==$optionCount){
-            if (empty($this->Module) && strpos($this->_URL,'$module')){
+            if (empty($this->Module) && strpos($this->_URL, '$module')){
                 throw new EntryPointException('Module is required for EntryPoint '.get_called_class());
             }else{
                 throw new EntryPointException('EntryPoint URL ('.$this->_URL.') requires more parameters than passed.');
@@ -273,11 +273,11 @@ 
      * @throws EntryPointException
      */
     protected function validateData(){
-        if (empty($this->_REQUIRED_DATA)||count($this->_REQUIRED_DATA)==0){
+        if (empty($this->_REQUIRED_DATA) || count($this->_REQUIRED_DATA)==0){
             return true;
         }else{
             $errors = array();
-            foreach($this->_REQUIRED_DATA as $property){
+            foreach ($this->_REQUIRED_DATA as $property){
                 if (isset($this->Data[$property]) || $this->Data[$property]!==null){
                     continue;
                 }else{
@@ -285,7 +285,7 @@ 
                 }
             }
             if (count($errors)>0){
-                throw new EntryPointException('EntryPoint requires specific properties in Request data. Missing the following '.implode($errors,","));
+                throw new EntryPointException('EntryPoint requires specific properties in Request data. Missing the following '.implode($errors, ","));
             }else{
                 return true;
             }

Braces +21 added lines, -21 removed lines

@@ -89,9 +89,9 @@ 
         $this->url = $url;
         $this->Module = $this->_MODULE;
 
-        if (!empty($options)) {
-            if (empty($this->Module)) {
-                if (strpos($this->_URL, '$module') !== FALSE) {
+        if (!empty($options)){
+            if (empty($this->Module)){
+                if (strpos($this->_URL, '$module') !== FALSE){
                     $this->module($options[0]);
                     array_shift($options);
                 }
@@ -130,7 +130,7 @@ 
      * @inheritdoc
      */
     public function execute(){
-        if ($this->verifyURL() && $this->validateData()) {
+        if ($this->verifyURL() && $this->validateData()){
             $this->configureURL();
             $this->Request->setURL($this->url);
             $this->Request->send();
@@ -144,14 +144,14 @@ 
     /**
      * @inheritdoc
      */
-    public function authRequired() {
+    public function authRequired(){
         return $this->_AUTH_REQUIRED;
     }
 
     /**
      * @inheritdoc
      */
-    public function configureAuth($accessToken) {
+    public function configureAuth($accessToken){
         if ($this->authRequired()){
             $this->accessToken = $accessToken;
             $this->Request->addHeader('OAuth-Token', $accessToken);
@@ -162,7 +162,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getModule() {
+    public function getModule(){
         return $this->Module;
     }
 
@@ -201,24 +201,24 @@ 
      */
     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 (isset($this->Module)) {
+                foreach ($urlParts as $key => $part){
+                    if (strpos($part, '$module') !== FALSE){
+                        if (isset($this->Module)){
                             $urlParts[$key] = $this->Module;
                             continue;
-                        } else {
-                            if (isset($this->Options[$o])) {
+                        } else{
+                            if (isset($this->Options[$o])){
                                 $this->Module = $this->Options[$o];
                                 array_shift($this->Options);
                             }
                         }
                     }
-                    if (strpos($part, "$") !== FALSE) {
-                        if (isset($this->Options[$o])) {
+                    if (strpos($part, "$") !== FALSE){
+                        if (isset($this->Options[$o])){
                             $urlParts[$key] = $this->Options[$o];
                             $o++;
                         }
@@ -259,10 +259,10 @@ 
         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;
         }
     }
@@ -275,18 +275,18 @@ 
     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){
                     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{
+            } else{
                 return true;
             }
         }

Upper-Lower-Casing +5 added lines, -5 removed lines

@@ -14,7 +14,7 @@ 
      * Whether or not Authentication is Required
      * @var bool
      */
-    protected $_AUTH_REQUIRED = true;
+    protected $_AUTH_REQUIRED = TRUE;
 
     /**
      * The default Module for the EntryPoint
@@ -263,7 +263,7 @@ 
                 throw new EntryPointException('EntryPoint URL ('.$this->_URL.') requires more parameters than passed.');
             }
         }else{
-            return true;
+            return TRUE;
         }
     }
 
@@ -274,11 +274,11 @@ 
      */
     protected function validateData(){
         if (empty($this->_REQUIRED_DATA)||count($this->_REQUIRED_DATA)==0){
-            return true;
+            return TRUE;
         }else{
             $errors = array();
             foreach($this->_REQUIRED_DATA as $property){
-                if (isset($this->Data[$property]) || $this->Data[$property]!==null){
+                if (isset($this->Data[$property]) || $this->Data[$property]!==NULL){
                     continue;
                 }else{
                     $errors[] = $property;
@@ -287,7 +287,7 @@ 
             if (count($errors)>0){
                 throw new EntryPointException('EntryPoint requires specific properties in Request data. Missing the following '.implode($errors,","));
             }else{
-                return true;
+                return TRUE;
             }
         }
     }