MichaelJ2324 / sugarcrm-restv10-sdk

src/EntryPoint/Abstracts/AbstractEntryPoint.php

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

@@ -44,7 +44,7 @@ 
      * Whether or not Authentication is Required
      * @var bool
      */
-    protected $_AUTH_REQUIRED = true;
+    protected $_AUTH_REQUIRED = TRUE;
 
     /**
      * The URL for the EntryPoint
@@ -320,7 +320,7 @@ 
         if (strpos($UrlArray[0],"$") !== FALSE){
             throw new InvalidURLException(get_called_class(),"Configured URL is ".$this->Url);
         }
-        return true;
+        return TRUE;
     }
 
     /**
@@ -335,7 +335,7 @@ 
         if ($urlVarCount>$optionCount){
             throw new RequiredOptionsException(get_called_class(),"URL requires $urlVarCount options, only $optionCount provided.");
         }
-        return true;
+        return TRUE;
     }
 
     /**
@@ -350,7 +350,7 @@ 
         if (!empty($this->_REQUIRED_DATA)){
             $this->verifyRequiredData();
         }
-        return true;
+        return TRUE;
     }
 
     /**
@@ -362,7 +362,7 @@ 
         if (gettype($this->Data) !== $this->_DATA_TYPE) {
             throw new RequiredDataException(get_called_class(),"Valid DataType is {$this->_DATA_TYPE}");
         }
-        return true;
+        return TRUE;
     }
 
     /**
@@ -380,7 +380,7 @@ 
         if (count($errors)>0){
             throw new RequiredDataException(get_called_class(),"Missing data for ".implode(",",$errors));
         }
-        return true;
+        return TRUE;
     }
 
     /**

Spacing +31 added lines, -31 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 ($this->verifyOptions() && is_object($this->Request)) {
+        if ($this->verifyOptions() && 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,11 +328,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, only $optionCount provided.");
+            throw new RequiredOptionsException(get_called_class(), "URL requires $urlVarCount options, only $optionCount provided.");
         }
         return true;
     }
@@ -343,7 +343,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)){
@@ -358,8 +358,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;
     }
@@ -371,13 +371,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;
     }
@@ -387,7 +387,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

Braces +17 added lines, -17 removed lines

@@ -119,7 +119,7 @@ 
     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;
     }
@@ -220,10 +220,10 @@ 
     public function execute($data = NULL){
         $data =  ($data === NULL?$this->Data:$data);
         $this->configureData($data);
-        if ($this->verifyOptions() && is_object($this->Request)) {
+        if ($this->verifyOptions() && is_object($this->Request)){
             $this->configureRequest();
             $this->Request->send();
-        }else{
+        } else{
             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);
         }
     }
@@ -292,12 +292,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++;
                     }
@@ -343,7 +343,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)){
@@ -358,7 +358,7 @@ 
      * @throws RequiredDataException
      */
     protected function verifyDataType(){
-        if (gettype($this->Data) !== $this->_DATA_TYPE) {
+        if (gettype($this->Data) !== $this->_DATA_TYPE){
             throw new RequiredDataException(get_called_class(),"Valid DataType is {$this->_DATA_TYPE}");
         }
         return true;

src/EntryPoint/POST/OAuth2Token.php

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

@@ -6,15 +6,15 @@
 
 class OAuth2Token extends AbstractPostEntryPoint {
 
-    protected $_AUTH_REQUIRED = false;
+    protected $_AUTH_REQUIRED = FALSE;
     protected $_URL = 'oauth2/token';
     protected $_REQUIRED_DATA = array(
-        'username' => null,
-        'password' => null,
+        'username' => NULL,
+        'password' => NULL,
         'grant_type' => 'password',
-        'client_id' => null,
-        'client_secret' => null,
-        'platform' => null
+        'client_id' => NULL,
+        'client_secret' => NULL,
+        'platform' => NULL
     );
     protected $_DATA_TYPE = 'array';
 

src/Response/JSON.php

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/Response/File.php

Braces +1 added lines, -2 removed lines

@@ -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 +1 added lines, -1 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)){

src/EntryPoint/Abstracts/GET/AbstractGetFileEntryPoint.php

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;

src/EntryPoint/Abstracts/DELETE/AbstractDeleteEntryPoint.php

Spacing +3 added lines, -3 removed lines

@@ -9,14 +9,14 @@
 
 abstract class AbstractDeleteEntryPoint extends AbstractEntryPoint {
 
-    public function __construct($url, array $options = array()) {
+    public function __construct($url, array $options = array()){
         $this->setRequest(new DELETE());
         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

@@ -9,12 +9,12 @@
 
 abstract class AbstractDeleteEntryPoint extends AbstractEntryPoint {
 
-    public function __construct($url, array $options = array()) {
+    public function __construct($url, array $options = array()){
         $this->setRequest(new DELETE());
         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

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;