MichaelJ2324 / sugarcrm-restv10-sdk

src/Request/Abstracts/AbstractRequest.php

Doc Comments +1 added lines

@@ -133,6 +133,7 @@
 
     /**
      * @inheritdoc
+     * @param string $body
      */
     public function setBody($body) {
         $this->body = $body;

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

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/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/EntryPoint/Abstracts/PUT/AbstractPutEntryPoint.php

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/GET/AbstractGetEntryPoint.php

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/AbstractPostEntryPoint.php

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/Client/Abstracts/AbstractClient.php

Doc Comments +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
     /**
      * @param $funcName
      * @param $className
-     * @return bool
+     * @return AbstractClient
      * @throws EntryPointException
      */
     public function registerEntryPoint($funcName, $className){

Braces +16 added lines, -16 removed lines

@@ -65,7 +65,7 @@ 
 
     public function __construct($server = '',array $credentials = array()){
         $server = (empty($server)?$this->server:$server);
-        if (!empty($server)) {
+        if (!empty($server)){
             $this->setServer($server);
         }
         $credentials = (empty($credentials)?$this->credentials:$credentials);
@@ -79,7 +79,7 @@ 
      * @inheritdoc
      * @param string $server
      */
-    public function setServer($server) {
+    public function setServer($server){
         $this->server = $server;
         $this->apiURL = Helpers::configureAPIURL($this->server);
         return $this;
@@ -88,7 +88,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getAPIUrl() {
+    public function getAPIUrl(){
         return $this->apiURL;
     }
 
@@ -98,9 +98,9 @@ 
      */
     public function setCredentials(array $credentials){
         $this->credentials = $credentials;
-        if (isset($this->credentials['client_id'])) {
+        if (isset($this->credentials['client_id'])){
             $token = static::getStoredToken($this->credentials['client_id']);
-            if (!empty($token)) {
+            if (!empty($token)){
                 $this->setToken($token);
             }
         }
@@ -133,7 +133,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getServer() {
+    public function getServer(){
         return $this->server;
     }
 
@@ -165,7 +165,7 @@ 
         $implements = class_implements($className);
         if (is_array($implements) && in_array('SugarAPI\SDK\EntryPoint\Interfaces\EPInterface',$implements)){
             $this->entryPoints[$funcName] = $className;
-        }else{
+        } else{
             throw new EntryPointException($className,'Class must extend SugarAPI\SDK\EntryPoint\Interfaces\EPInterface');
         }
         return $this;
@@ -184,12 +184,12 @@ 
             $EntryPoint = new $Class($this->apiURL, $params);
 
             if ($EntryPoint->authRequired()){
-                if (isset($this->token) && $this->authenticated()) {
+                if (isset($this->token) && $this->authenticated()){
                     $EntryPoint->setAuth($this->getToken()->access_token);
                 }
             }
             return $EntryPoint;
-        }else{
+        } else{
             throw new EntryPointException($name,'Unregistered EntryPoint');
         }
     }
@@ -198,13 +198,13 @@ 
      * @inheritdoc
      * @throws AuthenticationException - When Login request fails
      */
-    public function login() {
+    public function login(){
         $EP = new OAuth2Token($this->apiURL);
         $response = $EP->execute($this->credentials)->getResponse();
         if ($response->getStatus()=='200'){
             $this->setToken($response->getBody(FALSE));
             static::storeToken($this->token,$this->credentials['client_id']);
-        } else {
+        } else{
             $error = $response->getBody();
             throw new AuthenticationException("Login Response [".$error['error']."] ".$error['error_message']);
         }
@@ -226,7 +226,7 @@ 
         if ($response->getStatus()=='200'){
             $this->setToken($response->getBody(FALSE));
             static::storeToken($this->token,$this->credentials['client_id']);
-        }else{
+        } else{
             $error = $response->getBody();
             throw new AuthenticationException("Refresh Response [".$error['error']."] ".$error['error_message']);
         }
@@ -244,7 +244,7 @@ 
             if ($response->getStatus()=='200'){
                 unset($this->token);
                 static::removeStoredToken($this->credentials['client_id']);
-            }else{
+            } else{
                 $error = $response->getBody();
                 throw new AuthenticationException("Logout Response [".$error['error']."] ".$error['message']);
             }
@@ -256,7 +256,7 @@ 
      * @inheritdoc
      * @param \stdClass $token
      */
-    public static function storeToken($token, $client_id) {
+    public static function storeToken($token, $client_id){
         static::$_STORED_TOKENS[$client_id] = $token;
         return TRUE;
     }
@@ -264,14 +264,14 @@ 
     /**
      * @inheritdoc
      */
-    public static function getStoredToken($client_id) {
+    public static function getStoredToken($client_id){
         return (isset(static::$_STORED_TOKENS[$client_id])?static::$_STORED_TOKENS[$client_id]:NULL);
     }
 
     /**
      * @inheritdoc
      */
-    public static function removeStoredToken($client_id) {
+    public static function removeStoredToken($client_id){
         unset(static::$_STORED_TOKENS[$client_id]);
         return TRUE;
     }

src/EntryPoint/Abstracts/DELETE/AbstractDeleteEntryPoint.php

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

Braces +2 added lines, -3 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);
             }
@@ -72,7 +71,7 @@ 
             fwrite($fileHandle,$this->body);
             fclose($fileHandle);
             return $file;
-        }else{
+        } else{
             return FALSE;
         }
     }