MichaelJ2324 / sugarcrm-restv10-sdk

src/EntryPoint/Abstracts/GET/AbstractGetEntryPoint.php

Spacing +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);

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

Spacing +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);

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/DELETE/AbstractDeleteEntryPoint.php

Spacing +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);

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

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
     protected $default_message = 'Unknown SDK Exception occurred.';
 
-    public function __construct($message = '') {
+    public function __construct($message = ''){
         if (empty($message)){
             $message = $this->default_message;
         }

Braces +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
     protected $default_message = 'Unknown SDK Exception occurred.';
 
-    public function __construct($message = '') {
+    public function __construct($message = ''){
         if (empty($message)){
             $message = $this->default_message;
         }

src/Client/Abstracts/AbstractClient.php

Spacing +24 added lines, -24 removed lines

@@ -88,7 +88,7 @@ 
      */
     protected $entryPoints = array();
 
-    public function __construct($server = '',array $credentials = array()){
+    public function __construct($server = '', array $credentials = array()){
         $server = (empty($server)?$this->server:$server);
         $this->setServer($server);
         $credentials = (empty($credentials)?$this->credentials:$credentials);
@@ -100,7 +100,7 @@ 
      * @inheritdoc
      * @param string $server
      */
-    public function setServer($server) {
+    public function setServer($server){
         $this->server = $server;
         $this->apiURL = Helpers::configureAPIURL($this->server);
         return $this;
@@ -109,7 +109,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getAPIUrl() {
+    public function getAPIUrl(){
         return $this->apiURL;
     }
 
@@ -119,9 +119,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);
             }
         }
@@ -154,7 +154,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getServer() {
+    public function getServer(){
         return $this->server;
     }
 
@@ -162,7 +162,7 @@ 
      * @inheritdoc
      */
     public function authenticated(){
-        return time() < $this->expiration;
+        return time()<$this->expiration;
     }
 
     /**
@@ -184,10 +184,10 @@ 
      */
     public function registerEntryPoint($funcName, $className){
         $implements = class_implements($className);
-        if (is_array($implements) && in_array('SugarAPI\SDK\EntryPoint\Interfaces\EPInterface',$implements)){
+        if (is_array($implements) && in_array('SugarAPI\SDK\EntryPoint\Interfaces\EPInterface', $implements)){
             $this->entryPoints[$funcName] = $className;
         }else{
-            throw new EntryPointException($className,'Class must extend SugarAPI\SDK\EntryPoint\Interfaces\EPInterface');
+            throw new EntryPointException($className, 'Class must extend SugarAPI\SDK\EntryPoint\Interfaces\EPInterface');
         }
         return $this;
     }
@@ -209,7 +209,7 @@ 
             }
             return $EntryPoint;
         }else{
-            throw new EntryPointException($name,'Unregistered EntryPoint');
+            throw new EntryPointException($name, 'Unregistered EntryPoint');
         }
     }
 
@@ -217,16 +217,16 @@ 
      * @inheritdoc
      * @throws AuthenticationException - When Login request fails
      */
-    public function login() {
-        if (!empty($this->credentials)) {
+    public function login(){
+        if (!empty($this->credentials)){
             $response = $this->oauth2Token()->execute($this->credentials)->getResponse();
-            if ($response->getStatus() == '200') {
+            if ($response->getStatus()=='200'){
                 $this->setToken($response->getBody(FALSE));
                 static::storeToken($this->token, $this->credentials['client_id']);
                 return TRUE;
-            } else {
+            }else{
                 $error = $response->getBody();
-                throw new AuthenticationException("Login Response [" . $error['error'] . "] " . $error['error_message']);
+                throw new AuthenticationException("Login Response [".$error['error']."] ".$error['error_message']);
             }
         }
         return FALSE;
@@ -237,22 +237,22 @@ 
      * @throws AuthenticationException - When Refresh Request fails
      */
     public function refreshToken(){
-        if (isset($this->credentials['client_id'])&&
-            isset($this->credentials['client_secret'])&&
-            isset($this->token)) {
+        if (isset($this->credentials['client_id']) &&
+            isset($this->credentials['client_secret']) &&
+            isset($this->token)){
             $refreshOptions = array(
                 'client_id' => $this->credentials['client_id'],
                 'client_secret' => $this->credentials['client_secret'],
                 'refresh_token' => $this->token->refresh_token
             );
             $response = $this->oauth2Refresh()->execute($refreshOptions)->getResponse();
-            if ($response->getStatus() == '200') {
+            if ($response->getStatus()=='200'){
                 $this->setToken($response->getBody(FALSE));
                 static::storeToken($this->token, $this->credentials['client_id']);
                 return TRUE;
-            } else {
+            }else{
                 $error = $response->getBody();
-                throw new AuthenticationException("Refresh Response [" . $error['error'] . "] " . $error['error_message']);
+                throw new AuthenticationException("Refresh Response [".$error['error']."] ".$error['error_message']);
             }
         }
         return FALSE;
@@ -281,7 +281,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;
     }
@@ -289,14 +289,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;
     }

Braces +18 added lines, -18 removed lines

@@ -100,7 +100,7 @@ 
      * @inheritdoc
      * @param string $server
      */
-    public function setServer($server) {
+    public function setServer($server){
         $this->server = $server;
         $this->apiURL = Helpers::configureAPIURL($this->server);
         return $this;
@@ -109,7 +109,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getAPIUrl() {
+    public function getAPIUrl(){
         return $this->apiURL;
     }
 
@@ -119,9 +119,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);
             }
         }
@@ -154,7 +154,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getServer() {
+    public function getServer(){
         return $this->server;
     }
 
@@ -186,7 +186,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;
@@ -208,7 +208,7 @@ 
                 $EntryPoint->setAuth($this->token->access_token);
             }
             return $EntryPoint;
-        }else{
+        } else{
             throw new EntryPointException($name,'Unregistered EntryPoint');
         }
     }
@@ -217,14 +217,14 @@ 
      * @inheritdoc
      * @throws AuthenticationException - When Login request fails
      */
-    public function login() {
-        if (!empty($this->credentials)) {
+    public function login(){
+        if (!empty($this->credentials)){
             $response = $this->oauth2Token()->execute($this->credentials)->getResponse();
-            if ($response->getStatus() == '200') {
+            if ($response->getStatus() == '200'){
                 $this->setToken($response->getBody(FALSE));
                 static::storeToken($this->token, $this->credentials['client_id']);
                 return TRUE;
-            } else {
+            } else{
                 $error = $response->getBody();
                 throw new AuthenticationException("Login Response [" . $error['error'] . "] " . $error['error_message']);
             }
@@ -239,18 +239,18 @@ 
     public function refreshToken(){
         if (isset($this->credentials['client_id'])&&
             isset($this->credentials['client_secret'])&&
-            isset($this->token)) {
+            isset($this->token)){
             $refreshOptions = array(
                 'client_id' => $this->credentials['client_id'],
                 'client_secret' => $this->credentials['client_secret'],
                 'refresh_token' => $this->token->refresh_token
             );
             $response = $this->oauth2Refresh()->execute($refreshOptions)->getResponse();
-            if ($response->getStatus() == '200') {
+            if ($response->getStatus() == '200'){
                 $this->setToken($response->getBody(FALSE));
                 static::storeToken($this->token, $this->credentials['client_id']);
                 return TRUE;
-            } else {
+            } else{
                 $error = $response->getBody();
                 throw new AuthenticationException("Refresh Response [" . $error['error'] . "] " . $error['error_message']);
             }
@@ -269,7 +269,7 @@ 
                 unset($this->token);
                 static::removeStoredToken($this->credentials['client_id']);
                 return TRUE;
-            }else{
+            } else{
                 $error = $response->getBody();
                 throw new AuthenticationException("Logout Response [".$error['error']."] ".$error['message']);
             }
@@ -281,7 +281,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;
     }
@@ -289,14 +289,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/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;
         }
     }