MichaelJ2324 / sugarcrm-restv10-sdk

src/Helpers/Helpers.php

Braces +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     public static function getSDKEntryPointRegistry(){
         $entryPoints = array();
         require __DIR__.DIRECTORY_SEPARATOR.'registry.php';
-        foreach ($entryPoints as $funcName => $className) {
+        foreach ($entryPoints as $funcName => $className){
             $className = "SugarAPI\\SDK\\EntryPoint\\" . $className;
             $entryPoints[$funcName] = $className;
         }

src/Request/Abstracts/AbstractRequest.php

Braces +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
     /**
      * Always make sure to destroy Curl Resource
      */
-    public function __destruct() {
+    public function __destruct(){
         if ($this->status!==self::STATUS_CLOSED){
             curl_close($this->CurlRequest);
         }

src/EntryPoint/Abstracts/AbstractEntryPoint.php

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 (is_object($this->Request)) {
+        if (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++;
                     }
@@ -328,7 +328,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)){
@@ -343,7 +343,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/ModuleRecordFileField.php

Braces +3 added lines, -3 removed lines

@@ -32,13 +32,13 @@
      * @throws RequiredOptionsException
      */
     protected function configureData($data){
-        if (is_string($data)) {
-            if (!empty($this->Options)) {
+        if (is_string($data)){
+            if (!empty($this->Options)){
                 $fileField = end($this->Options);
                 $data = array(
                     $fileField => $data
                 );
-            } else {
+            } else{
                 throw new RequiredOptionsException(get_called_class(), "Options are required, when passing String for data.");
             }
         }

src/EntryPoint/POST/Bulk.php

Braces +6 added lines, -6 removed lines

@@ -30,19 +30,19 @@
         'method' => ''
     );
 
-    protected function configureData($data) {
-        if (!isset($data['requests'])) {
+    protected function configureData($data){
+        if (!isset($data['requests'])){
             $requestData = array(
                 'requests' => array()
             );
             $counter = 0;
-            foreach ($data as $key => $EntryPoint) {
-                if (is_object($EntryPoint)) {
+            foreach ($data as $key => $EntryPoint){
+                if (is_object($EntryPoint)){
                     $requestData['requests'][$counter] = $this->bulkRequest;
                     $requestData['requests'][$counter]['method'] = $EntryPoint->getRequest()->getType();
-                    if ($requestData['requests'][$counter]['method'] == "POST" || $requestData['requests'][$counter]['method'] == "PUT") {
+                    if ($requestData['requests'][$counter]['method'] == "POST" || $requestData['requests'][$counter]['method'] == "PUT"){
                         $requestData['requests'][$counter]['data'] = json_encode($EntryPoint->getData());
-                    } else {
+                    } else{
                         unset($requestData['requests'][$counter]['data']);
                     }
                     $requestData['requests'][$counter]['headers'] = $EntryPoint->getRequest()->getHeaders();

src/Response/File.php

Braces +5 added lines, -6 removed lines

@@ -31,10 +31,10 @@ 
      * Extract Filename from Headers
      * @param mixed $curlResponse
      */
-    public function setCurlResponse($curlResponse) {
+    public function setCurlResponse($curlResponse){
         parent::setCurlResponse($curlResponse);
-        if (!$this->error) {
-            if (empty($this->fileName)) {
+        if (!$this->error){
+            if (empty($this->fileName)){
                 $this->extractFileName();
             }
             $this->writeFile();
@@ -59,8 +59,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->setFileName(substr($header, (strpos($header, "\"")+1), -1));
             }
@@ -99,7 +98,7 @@ 
             fwrite($fileHandle,$this->body);
             fclose($fileHandle);
             return $file;
-        }else{
+        } else{
             return FALSE;
         }
     }

src/Response/Abstracts/AbstractResponse.php

Braces +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
         }
     }
 
-    public function setCurlResponse($curlResponse) {
+    public function setCurlResponse($curlResponse){
         $this->extractInfo();
         if (!$this->getError()){
             $this->extractResponse($curlResponse);

src/EntryPoint/Abstracts/PUT/AbstractPutEntryPoint.php

Braces +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 
 abstract class AbstractPutEntryPoint extends AbstractEntryPoint {
 
-    public function __construct($url, array $options = array()) {
+    public function __construct($url, array $options = array()){
         $this->setRequest(new PUT());
         $this->setResponse(new JSON($this->Request->getCurlObject()));
         parent::__construct($url, $options);

src/EntryPoint/Abstracts/GET/AbstractGetEntryPoint.php

Braces +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 
 abstract class AbstractPutEntryPoint extends AbstractEntryPoint {
 
-    public function __construct($url, array $options = array()) {
+    public function __construct($url, array $options = array()){
         $this->setRequest(new PUT());
         $this->setResponse(new JSON($this->Request->getCurlObject()));
         parent::__construct($url, $options);