MichaelJ2324 / sugarcrm-restv10-sdk

src/Request/Abstracts/AbstractRequest.php

Doc Comments +2 added lines

@@ -125,6 +125,7 @@ 
 
     /**
      * @inheritdoc
+     * @param string $name
      */
     public function addHeader($name, $value){
         $token = $name.": ".$value;
@@ -158,6 +159,7 @@ 
 
     /**
      * @inheritdoc
+     * @param string $body
      */
     public function setBody($body){
         $this->body = $body;

Indentation +2 added lines, -2 removed lines

@@ -226,10 +226,10 @@
      */
     protected function configureType(){
         switch ($this->type) {
-               case 'POST':
+                case 'POST':
                     $this->setOption(CURLOPT_POST, TRUE);
                     break;
-               case 'DELETE':
+                case 'DELETE':
                case 'PUT':
                     $this->setOption(CURLOPT_CUSTOMREQUEST, $this->type);
                     break;

Spacing +4 added lines, -4 removed lines

@@ -101,7 +101,7 @@ 
     /**
      * Always make sure to destroy Curl Resource
      */
-    public function __destruct() {
+    public function __destruct(){
         if ($this->status!==self::STATUS_CLOSED){
             curl_close($this->CurlRequest);
         }
@@ -140,7 +140,7 @@ 
             foreach ($array as $key => $value){
                 if (is_numeric($key)){
                     $this->headers[] = $value;
-                }else {
+                }else{
                     $this->addHeader($key, $value);
                 }
             }
@@ -191,7 +191,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getOptions() {
+    public function getOptions(){
         return $this->options;
     }
 
@@ -225,7 +225,7 @@ 
      * Configure the Curl Options based on Request Type
      */
     protected function configureType(){
-        switch ($this->type) {
+        switch ($this->type){
                case 'POST':
                     $this->setOption(CURLOPT_POST, TRUE);
                     break;

Braces +4 added lines, -4 removed lines

@@ -101,7 +101,7 @@ 
     /**
      * Always make sure to destroy Curl Resource
      */
-    public function __destruct() {
+    public function __destruct(){
         if ($this->status!==self::STATUS_CLOSED){
             curl_close($this->CurlRequest);
         }
@@ -140,7 +140,7 @@ 
             foreach ($array as $key => $value){
                 if (is_numeric($key)){
                     $this->headers[] = $value;
-                }else {
+                } else{
                     $this->addHeader($key, $value);
                 }
             }
@@ -191,7 +191,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getOptions() {
+    public function getOptions(){
         return $this->options;
     }
 
@@ -225,7 +225,7 @@ 
      * Configure the Curl Options based on Request Type
      */
     protected function configureType(){
-        switch ($this->type) {
+        switch ($this->type){
                case 'POST':
                     $this->setOption(CURLOPT_POST, TRUE);
                     break;

src/Response/Abstracts/AbstractResponse.php

Spacing +4 added lines, -4 removed lines

@@ -45,14 +45,14 @@ 
      */
     protected $info;
 
-    public function __construct($curlRequest,$curlResponse = NULL){
+    public function __construct($curlRequest, $curlResponse = NULL){
         $this->CurlRequest = $curlRequest;
         if ($curlResponse!==NULL){
             $this->setCurlResponse($curlResponse);
         }
     }
 
-    public function setCurlResponse($curlResponse) {
+    public function setCurlResponse($curlResponse){
         $this->extractInfo();
         if (!$this->error){
             $this->extractResponse($curlResponse);
@@ -66,9 +66,9 @@ 
     protected function extractInfo(){
         $this->info = curl_getinfo($this->CurlRequest);
         $this->status = $this->info['http_code'];
-        if (curl_errno($this->CurlRequest)!== CURLE_OK){
+        if (curl_errno($this->CurlRequest)!==CURLE_OK){
             $this->error = curl_error($this->CurlRequest);
-        }else {
+        }else{
             $this->error = FALSE;
         }
     }

Braces +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
         }
     }
 
-    public function setCurlResponse($curlResponse) {
+    public function setCurlResponse($curlResponse){
         $this->extractInfo();
         if (!$this->error){
             $this->extractResponse($curlResponse);
@@ -68,7 +68,7 @@ 
         $this->status = $this->info['http_code'];
         if (curl_errno($this->CurlRequest)!== CURLE_OK){
             $this->error = curl_error($this->CurlRequest);
-        }else {
+        } else{
             $this->error = FALSE;
         }
     }

src/EntryPoint/Abstracts/AbstractEntryPoint.php

Spacing +27 added lines, -27 removed lines

@@ -96,10 +96,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();
@@ -126,7 +126,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setAuth($accessToken) {
+    public function setAuth($accessToken){
         $this->accessToken = $accessToken;
         return $this;
     }
@@ -134,7 +134,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setUrl($url) {
+    public function setUrl($url){
         $this->Url = $url;
         return $this;
     }
@@ -142,7 +142,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setRequest(RequestInterface $Request) {
+    public function setRequest(RequestInterface $Request){
         $this->Request = $Request;
         return $this;
     }
@@ -150,7 +150,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setResponse(ResponseInterface $Response) {
+    public function setResponse(ResponseInterface $Response){
         $this->Response = $Response;
         return $this;
     }
@@ -198,14 +198,14 @@ 
      * @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();
             $this->configureResponse();
         }else{
-            throw new InvalidRequestException(get_called_class(),"Request property not configured");
+            throw new InvalidRequestException(get_called_class(), "Request property not configured");
         }
         return $this;
     }
@@ -213,7 +213,7 @@ 
     /**
      * @inheritdoc
      */
-    public function authRequired() {
+    public function authRequired(){
         return $this->_AUTH_REQUIRED;
     }
 
@@ -223,10 +223,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();
@@ -247,7 +247,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);
         }
     }
@@ -257,7 +257,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);
@@ -269,7 +269,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;
             }
@@ -284,9 +284,9 @@ 
      */
     protected function configureURL(){
         $url = $this->_URL;
-        if ($this->requiresOptions()) {
-            foreach($this->Options as $key => $option){
-                $url = preg_replace('/(\$.*?[^\/]*)/',$option,$url,1);
+        if ($this->requiresOptions()){
+            foreach ($this->Options as $key => $option){
+                $url = preg_replace('/(\$.*?[^\/]*)/', $option, $url, 1);
             }
         }
         $url = $this->baseUrl.$url;
@@ -299,9 +299,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;
     }
@@ -312,7 +312,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)){
@@ -327,8 +327,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;
     }
@@ -340,13 +340,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;
     }
@@ -356,7 +356,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 +14 added lines, -14 removed lines

@@ -99,7 +99,7 @@ 
     public function __construct($baseUrl,array $options = array()){
         $this->baseUrl = $baseUrl;
 
-        if (!empty($options)) {
+        if (!empty($options)){
             $this->setOptions($options);
         }
         $this->configureURL();
@@ -126,7 +126,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setAuth($accessToken) {
+    public function setAuth($accessToken){
         $this->accessToken = $accessToken;
         return $this;
     }
@@ -134,7 +134,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setUrl($url) {
+    public function setUrl($url){
         $this->Url = $url;
         return $this;
     }
@@ -142,7 +142,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setRequest(RequestInterface $Request) {
+    public function setRequest(RequestInterface $Request){
         $this->Request = $Request;
         return $this;
     }
@@ -150,7 +150,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setResponse(ResponseInterface $Response) {
+    public function setResponse(ResponseInterface $Response){
         $this->Response = $Response;
         return $this;
     }
@@ -200,11 +200,11 @@ 
     public function execute($data = NULL){
         $data =  ($data === NULL?$this->Data:$data);
         $this->configureData($data);
-        if (is_object($this->Request)) {
+        if (is_object($this->Request)){
             $this->configureRequest();
             $this->Request->send();
             $this->configureResponse();
-        }else{
+        } else{
             throw new InvalidRequestException(get_called_class(),"Request property not configured");
         }
         return $this;
@@ -213,7 +213,7 @@ 
     /**
      * @inheritdoc
      */
-    public function authRequired() {
+    public function authRequired(){
         return $this->_AUTH_REQUIRED;
     }
 
@@ -223,10 +223,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();
@@ -247,7 +247,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);
         }
     }
@@ -284,7 +284,7 @@ 
      */
     protected function configureURL(){
         $url = $this->_URL;
-        if ($this->requiresOptions()) {
+        if ($this->requiresOptions()){
             foreach($this->Options as $key => $option){
                 $url = preg_replace('/(\$.*?[^\/]*)/',$option,$url,1);
             }
@@ -312,7 +312,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)){
@@ -327,7 +327,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;