MichaelJ2324 / sugarcrm-restv10-sdk

src/Response/JSON.php

Spacing +2 added lines, -2 removed lines

@@ -17,8 +17,8 @@
     /**
      * @inheritdoc
      */
-    public function getBody($asArray = true) {
-        return json_decode($this->body,$asArray);
+    public function getBody($asArray = true){
+        return json_decode($this->body, $asArray);
     }
 
 }
\ No newline at end of file

Braces +1 added lines, -1 removed lines

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

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

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

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);
     }
 

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/Authentication/AuthenticationException.php

Spacing +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
 
     protected $message = 'Authentication Exception [%s] occurred in SDK Client. Message: %s';
 
-    public function __construct($message) {
-        parent::__construct(sprintf($this->message,get_called_class(),$message));
+    public function __construct($message){
+        parent::__construct(sprintf($this->message, get_called_class(), $message));
     }
 
 }
\ No newline at end of file

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/EntryPoint/EntryPointException.php

Spacing +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
 
     protected $message = 'EntryPoint Exception [%s] occurred on EntryPoint %s: %s';
 
-    public function __construct($EntryPoint,$data) {
-        parent::__construct(sprintf($this->message,get_called_class(),$EntryPoint,$data));
+    public function __construct($EntryPoint, $data){
+        parent::__construct(sprintf($this->message, get_called_class(), $EntryPoint, $data));
     }
 
 }
\ No newline at end of file

Braces +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 
     protected $message = 'EntryPoint Exception [%s] occurred on EntryPoint %s: %s';
 
-    public function __construct($EntryPoint,$data) {
+    public function __construct($EntryPoint,$data){
         parent::__construct(sprintf($this->message,get_called_class(),$EntryPoint,$data));
     }
 

src/EntryPoint/Interfaces/EPInterface.php

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

@@ -58,7 +58,7 @@
      * @param null $data
      * @return mixed
      */
-    public function execute($data = null);
+    public function execute($data = NULL);
 
     /**
      * Get the module that is set on the EntryPoint

src/EntryPoint/Abstracts/AbstractEntryPoint.php

Spacing +29 added lines, -29 removed lines

@@ -94,12 +94,12 @@ 
     protected $accessToken;
 
 
-    public function __construct($url,$options = array()){
+    public function __construct($url, $options = array()){
         $this->baseUrl = $url;
 
-        if (!empty($options)) {
+        if (!empty($options)){
             $this->setOptions($options);
-        }elseif(!$this->requiresOptions()){
+        }elseif (!$this->requiresOptions()){
             $this->configureURL();
         }
     }
@@ -121,7 +121,7 @@ 
      */
     public function setData($data){
         $this->Data = $data;
-        if ($this->verifyData()) {
+        if ($this->verifyData()){
             $this->Request->setBody($data);
         }
         return $this;
@@ -130,7 +130,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setAuth($accessToken) {
+    public function setAuth($accessToken){
         if ($this->authRequired()){
             $this->accessToken = $accessToken;
             $this->Request->addHeader('OAuth-Token', $accessToken);
@@ -142,9 +142,9 @@ 
      * @inheritdoc
      * @throws InvalidURLException - When passed in URL contains $variables
      */
-    public function setUrl($url) {
+    public function setUrl($url){
         $this->Url = $url;
-        if ($this->verifyUrl()) {
+        if ($this->verifyUrl()){
             $this->Request->setURL($this->Url);
         }
         return $this;
@@ -153,7 +153,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setRequest(RequestInterface $Request) {
+    public function setRequest(RequestInterface $Request){
         $this->Request = $Request;
         return $this;
     }
@@ -161,7 +161,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setResponse(ResponseInterface $Response) {
+    public function setResponse(ResponseInterface $Response){
         $this->Response = $Response;
         return $this;
     }
@@ -169,7 +169,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getModule() {
+    public function getModule(){
         return $this->Module;
     }
 
@@ -215,7 +215,7 @@ 
     /**
      * @inheritdoc
      */
-    public function authRequired() {
+    public function authRequired(){
         return $this->_AUTH_REQUIRED;
     }
 
@@ -223,8 +223,8 @@ 
      * Override function for configuring Default Values on some EntryPoints to allow for short hand
      */
     protected function configureData($data){
-        if (!empty($this->_REQUIRED_DATA)&&is_array($data)){
-            foreach($this->_REQUIRED_DATA as $property => $value){
+        if (!empty($this->_REQUIRED_DATA) && is_array($data)){
+            foreach ($this->_REQUIRED_DATA as $property => $value){
                 if (!isset($data[$property])){
                     $data[$property] = $value;
                 }
@@ -240,18 +240,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);
@@ -263,9 +263,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;
     }
@@ -276,11 +276,11 @@ 
      * @throws RequiredOptionsException
      */
     protected function verifyOptions(){
-        $urlVarCount = substr_count($this->_URL,"$");
+        $urlVarCount = substr_count($this->_URL, "$");
         $optionCount = 0;
         $optionCount += count($this->Options);
         if ($urlVarCount!==$optionCount){
-            throw new RequiredOptionsException(get_called_class(),"URL requires $urlVarCount options.");
+            throw new RequiredOptionsException(get_called_class(), "URL requires $urlVarCount options.");
         }
         return true;
     }
@@ -291,20 +291,20 @@ 
      * @throws RequiredDataException
      */
     protected function verifyData(){
-        if (isset($this->_DATA_TYPE)||!empty($this->_DATA_TYPE)) {
-            if (gettype($this->Data) !== $this->_DATA_TYPE) {
-                throw new RequiredDataException(get_called_class(),"Valid DataType is {$this->_DATA_TYPE}");
+        if (isset($this->_DATA_TYPE) || !empty($this->_DATA_TYPE)){
+            if (gettype($this->Data)!==$this->_DATA_TYPE){
+                throw new RequiredDataException(get_called_class(), "Valid DataType is {$this->_DATA_TYPE}");
             }
         }
         if (!empty($this->_REQUIRED_DATA)){
             $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 $errors");
+                throw new RequiredDataException(get_called_class(), "Missing data for $errors");
             }
         }
         return true;
@@ -315,7 +315,7 @@ 
      * @return bool
      */
     protected function requiresOptions(){
-        return strpos($this->_URL,"$") !== FALSE?TRUE:FALSE;
+        return strpos($this->_URL, "$")!==FALSE?TRUE:FALSE;
     }
 
 }
\ No newline at end of file

Braces +16 added lines, -16 removed lines

@@ -97,9 +97,9 @@ 
     public function __construct($url,$options = array()){
         $this->baseUrl = $url;
 
-        if (!empty($options)) {
+        if (!empty($options)){
             $this->setOptions($options);
-        }elseif(!$this->requiresOptions()){
+        } elseif(!$this->requiresOptions()){
             $this->configureURL();
         }
     }
@@ -121,7 +121,7 @@ 
      */
     public function setData($data){
         $this->Data = $data;
-        if ($this->verifyData()) {
+        if ($this->verifyData()){
             $this->Request->setBody($data);
         }
         return $this;
@@ -130,7 +130,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setAuth($accessToken) {
+    public function setAuth($accessToken){
         if ($this->authRequired()){
             $this->accessToken = $accessToken;
             $this->Request->addHeader('OAuth-Token', $accessToken);
@@ -142,9 +142,9 @@ 
      * @inheritdoc
      * @throws InvalidURLException - When passed in URL contains $variables
      */
-    public function setUrl($url) {
+    public function setUrl($url){
         $this->Url = $url;
-        if ($this->verifyUrl()) {
+        if ($this->verifyUrl()){
             $this->Request->setURL($this->Url);
         }
         return $this;
@@ -153,7 +153,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setRequest(RequestInterface $Request) {
+    public function setRequest(RequestInterface $Request){
         $this->Request = $Request;
         return $this;
     }
@@ -161,7 +161,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setResponse(ResponseInterface $Response) {
+    public function setResponse(ResponseInterface $Response){
         $this->Response = $Response;
         return $this;
     }
@@ -169,7 +169,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getModule() {
+    public function getModule(){
         return $this->Module;
     }
 
@@ -215,7 +215,7 @@ 
     /**
      * @inheritdoc
      */
-    public function authRequired() {
+    public function authRequired(){
         return $this->_AUTH_REQUIRED;
     }
 
@@ -240,12 +240,12 @@ 
      */
     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++;
                     }
@@ -291,8 +291,8 @@ 
      * @throws RequiredDataException
      */
     protected function verifyData(){
-        if (isset($this->_DATA_TYPE)||!empty($this->_DATA_TYPE)) {
-            if (gettype($this->Data) !== $this->_DATA_TYPE) {
+        if (isset($this->_DATA_TYPE)||!empty($this->_DATA_TYPE)){
+            if (gettype($this->Data) !== $this->_DATA_TYPE){
                 throw new RequiredDataException(get_called_class(),"Valid DataType is {$this->_DATA_TYPE}");
             }
         }

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

@@ -16,7 +16,7 @@ 
      * Whether or not Authentication is Required
      * @var bool
      */
-    protected $_AUTH_REQUIRED = true;
+    protected $_AUTH_REQUIRED = TRUE;
 
     /**
      * The URL for the EntryPoint
@@ -204,8 +204,8 @@ 
     /**
      * @inheritdoc
      */
-    public function execute($data = null){
-        if ($data!==null){
+    public function execute($data = NULL){
+        if ($data!==NULL){
             $this->configureData($data);
         }
         $this->Request->send();
@@ -267,7 +267,7 @@ 
         if (strpos($UrlArray[0],"$") !== FALSE){
             throw new InvalidURLException(get_called_class(),"Configured URL is ".$this->Url);
         }
-        return true;
+        return TRUE;
     }
 
     /**
@@ -282,7 +282,7 @@ 
         if ($urlVarCount!==$optionCount){
             throw new RequiredOptionsException(get_called_class(),"URL requires $urlVarCount options.");
         }
-        return true;
+        return TRUE;
     }
 
     /**
@@ -307,7 +307,7 @@ 
                 throw new RequiredDataException(get_called_class(),"Missing data for $errors");
             }
         }
-        return true;
+        return TRUE;
     }
 
     /**

src/EntryPoint/Abstracts/GET/AbstractGetFileEntryPoint.php

Spacing +3 added lines, -3 removed lines

@@ -14,14 +14,14 @@
      */
     protected $downloadDir = null;
 
-    public function __construct($url, array $options = array()) {
+    public function __construct($url, array $options = array()){
         $this->setRequest(new GETFile());
         parent::__construct($url, $options);
     }
 
-    public function execute($data = null) {
+    public function execute($data = null){
         parent::execute($data);
-        $this->setResponse(new FileResponse($this->Request->getResponse(),$this->Request->getCurlObject(),$this->downloadDir));
+        $this->setResponse(new FileResponse($this->Request->getResponse(), $this->Request->getCurlObject(), $this->downloadDir));
         return $this;
     }
 

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

@@ -12,14 +12,14 @@
      * The directory in which to download the File
      * @var string
      */
-    protected $downloadDir = null;
+    protected $downloadDir = NULL;
 
     public function __construct($url, array $options = array()) {
         $this->setRequest(new GETFile());
         parent::__construct($url, $options);
     }
 
-    public function execute($data = null) {
+    public function execute($data = NULL) {
         parent::execute($data);
         $this->setResponse(new FileResponse($this->Request->getResponse(),$this->Request->getCurlObject(),$this->downloadDir));
         return $this;

Braces +2 added lines, -2 removed lines

@@ -8,12 +8,12 @@
 
 abstract class AbstractPostFileEntryPoint extends AbstractEntryPoint {
 
-    public function __construct($url, array $options = array()) {
+    public function __construct($url, array $options = array()){
         $this->setRequest(new POSTFile());
         parent::__construct($url, $options);
     }
 
-    public function execute($data = null) {
+    public function execute($data = null){
         parent::execute($data);
         $this->setResponse(new JSON($this->Request->getResponse(),$this->Request->getCurlObject()));
         return $this;

src/EntryPoint/Abstracts/POST/AbstractPostFileEntryPoint.php

Spacing +3 added lines, -3 removed lines

@@ -8,14 +8,14 @@
 
 abstract class AbstractPostFileEntryPoint extends AbstractEntryPoint {
 
-    public function __construct($url, array $options = array()) {
+    public function __construct($url, array $options = array()){
         $this->setRequest(new POSTFile());
         parent::__construct($url, $options);
     }
 
-    public function execute($data = null) {
+    public function execute($data = null){
         parent::execute($data);
-        $this->setResponse(new JSON($this->Request->getResponse(),$this->Request->getCurlObject()));
+        $this->setResponse(new JSON($this->Request->getResponse(), $this->Request->getCurlObject()));
         return $this;
     }
 

Braces +2 added lines, -2 removed lines

@@ -8,12 +8,12 @@
 
 abstract class AbstractPostFileEntryPoint extends AbstractEntryPoint {
 
-    public function __construct($url, array $options = array()) {
+    public function __construct($url, array $options = array()){
         $this->setRequest(new POSTFile());
         parent::__construct($url, $options);
     }
 
-    public function execute($data = null) {
+    public function execute($data = null){
         parent::execute($data);
         $this->setResponse(new JSON($this->Request->getResponse(),$this->Request->getCurlObject()));
         return $this;

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

@@ -13,7 +13,7 @@
         parent::__construct($url, $options);
     }
 
-    public function execute($data = null) {
+    public function execute($data = NULL) {
         parent::execute($data);
         $this->setResponse(new JSON($this->Request->getResponse(),$this->Request->getCurlObject()));
         return $this;

src/EntryPoint/POST/ModuleRecordFileField.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
                 $data[$fileField] = $this->setFileFieldValue($data[$fileField]);
             }else{
                 foreach ($this->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);
                     }
                 }

Braces +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
     protected function configureData($data){
         if (!empty($this->Options)){
             $fileField = end($this->Options);
-        }else{
+        } else{
             throw new RequiredOptionsException(get_called_class());
         }
         if (is_string($data) && isset($fileField)){
@@ -45,7 +45,7 @@ 
         if (is_array($data)){
             if (isset($fileField)){
                 $data[$fileField] = $this->setFileFieldValue($data[$fileField]);
-            }else{
+            } else{
                 foreach ($this->Data as $key => $value){
                     if (!array_key_exists($key,$this->_REQUIRED_DATA)){
                         $data[$key] = $this->setFileFieldValue($value);

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

@@ -22,7 +22,7 @@
      */
     protected $_REQUIRED_DATA = array(
         'format' => 'sugar-html-json',
-        'delete_if_fails' => false
+        'delete_if_fails' => FALSE
     );
 
     /**