Test Failed
Push — master ( 4902e0...4e23df )
by Mike
02:23
created
src/Client/Abstracts/AbstractClient.php 1 patch
Braces   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
     protected $entryPoints = array();
62 62
 
63 63
     public function __construct($server = '',array $credentials = array()){
64
-        if (!empty($server)) {
64
+        if (!empty($server)){
65 65
             $this->setServer($server);
66 66
         }
67 67
         if (!empty($credentials)){
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
     /**
74 74
      * @inheritdoc
75 75
      */
76
-    public function setServer($server) {
76
+    public function setServer($server){
77 77
         $this->server = $server;
78 78
         $this->apiURL = Helpers::configureAPIURL($this->server);
79 79
         return $this;
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
     /**
83 83
      * @inheritdoc
84 84
      */
85
-    public function getAPIUrl() {
85
+    public function getAPIUrl(){
86 86
         return $this->apiURL;
87 87
     }
88 88
 
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
     /**
126 126
      * @inheritdoc
127 127
      */
128
-    public function getServer() {
128
+    public function getServer(){
129 129
         return $this->server;
130 130
     }
131 131
 
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
157 157
         $implements = class_implements($className);
158 158
         if (is_array($implements) && in_array('SugarAPI\SDK\EntryPoint\Interfaces\EPInterface',$implements)){
159 159
             $this->entryPoints[$funcName] = $className;
160
-        }else{
160
+        } else{
161 161
             throw new EntryPointException($className,'Class must extend SugarAPI\SDK\EntryPoint\Interfaces\EPInterface');
162 162
         }
163 163
         return $this;
@@ -176,12 +176,12 @@  discard block
 block discarded – undo
176 176
             $EntryPoint = new $Class($this->apiURL, $params);
177 177
 
178 178
             if ($EntryPoint->authRequired()){
179
-                if (isset($this->token) && $this->authenticated()) {
179
+                if (isset($this->token) && $this->authenticated()){
180 180
                     $EntryPoint->setAuth($this->getToken()->access_token);
181 181
                 }
182 182
             }
183 183
             return $EntryPoint;
184
-        }else{
184
+        } else{
185 185
             throw new EntryPointException($name,'Unregistered EntryPoint');
186 186
         }
187 187
     }
@@ -190,13 +190,13 @@  discard block
 block discarded – undo
190 190
      * @inheritdoc
191 191
      * @throws AuthenticationException - When Login request fails
192 192
      */
193
-    public function login() {
193
+    public function login(){
194 194
         $EP = new OAuth2Token($this->apiURL);
195 195
         $response = $EP->execute($this->credentials)->getResponse();
196 196
         if ($response->getStatus()=='200'){
197 197
             $this->setToken($response->getBody(false));
198 198
             static::storeToken($this->token,$this->credentials['client_id']);
199
-        } else {
199
+        } else{
200 200
             $error = $response->getBody();
201 201
             throw new AuthenticationException("Login Response [".$error['error']."] ".$error['error_message']);
202 202
         }
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
         if ($response->getStatus()=='200'){
219 219
             $this->setToken($response->getBody(false));
220 220
             static::storeToken($this->token,$this->credentials['client_id']);
221
-        }else{
221
+        } else{
222 222
             $error = $response->getBody();
223 223
             throw new AuthenticationException("Refresh Response [".$error['error']."] ".$error['error_message']);
224 224
         }
@@ -236,7 +236,7 @@  discard block
 block discarded – undo
236 236
             if ($response->getStatus()=='200'){
237 237
                 unset($this->token);
238 238
                 static::removeStoredToken($this->credentials['client_id']);
239
-            }else{
239
+            } else{
240 240
                 $error = $response->getBody();
241 241
                 throw new AuthenticationException("Logout Response [".$error['error']."] ".$error['message']);
242 242
             }
@@ -247,7 +247,7 @@  discard block
 block discarded – undo
247 247
     /**
248 248
      * @inheritdoc
249 249
      */
250
-    public static function storeToken($token, $client_id) {
250
+    public static function storeToken($token, $client_id){
251 251
         static::$_STORED_TOKENS[$client_id] = $token;
252 252
         return true;
253 253
     }
@@ -255,14 +255,14 @@  discard block
 block discarded – undo
255 255
     /**
256 256
      * @inheritdoc
257 257
      */
258
-    public static function getStoredToken($client_id) {
258
+    public static function getStoredToken($client_id){
259 259
         return static::$_STORED_TOKENS[$client_id];
260 260
     }
261 261
 
262 262
     /**
263 263
      * @inheritdoc
264 264
      */
265
-    public static function removeStoredToken($client_id) {
265
+    public static function removeStoredToken($client_id){
266 266
         unset(static::$_STORED_TOKENS[$client_id]);
267 267
         return true;
268 268
     }
Please login to merge, or discard this patch.
src/Helpers/Helpers.php 1 patch
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
     public static function getSDKEntryPointRegistry(){
40 40
         $entryPoints = array();
41 41
         require __DIR__.DIRECTORY_SEPARATOR.'registry.php';
42
-        foreach ($entryPoints as $funcName => $className) {
42
+        foreach ($entryPoints as $funcName => $className){
43 43
             $className = "SugarAPI\\SDK\\EntryPoint\\" . $className;
44 44
             $entryPoints[$funcName] = $className;
45 45
         }
Please login to merge, or discard this patch.
src/EntryPoint/Abstracts/DELETE/AbstractDeleteEntryPoint.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -8,12 +8,12 @@
 block discarded – undo
8 8
 
9 9
 abstract class AbstractPostFileEntryPoint extends AbstractEntryPoint {
10 10
 
11
-    public function __construct($url, array $options = array()) {
11
+    public function __construct($url, array $options = array()){
12 12
         $this->setRequest(new POSTFile());
13 13
         parent::__construct($url, $options);
14 14
     }
15 15
 
16
-    public function execute($data = null) {
16
+    public function execute($data = null){
17 17
         parent::execute($data);
18 18
         $this->setResponse(new JSON($this->Request->getResponse(),$this->Request->getCurlObject()));
19 19
         return $this;
Please login to merge, or discard this patch.
src/EntryPoint/Abstracts/PUT/AbstractPutEntryPoint.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -8,12 +8,12 @@
 block discarded – undo
8 8
 
9 9
 abstract class AbstractPostFileEntryPoint extends AbstractEntryPoint {
10 10
 
11
-    public function __construct($url, array $options = array()) {
11
+    public function __construct($url, array $options = array()){
12 12
         $this->setRequest(new POSTFile());
13 13
         parent::__construct($url, $options);
14 14
     }
15 15
 
16
-    public function execute($data = null) {
16
+    public function execute($data = null){
17 17
         parent::execute($data);
18 18
         $this->setResponse(new JSON($this->Request->getResponse(),$this->Request->getCurlObject()));
19 19
         return $this;
Please login to merge, or discard this patch.
src/EntryPoint/Abstracts/GET/AbstractGetFileEntryPoint.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -8,12 +8,12 @@
 block discarded – undo
8 8
 
9 9
 abstract class AbstractPostFileEntryPoint extends AbstractEntryPoint {
10 10
 
11
-    public function __construct($url, array $options = array()) {
11
+    public function __construct($url, array $options = array()){
12 12
         $this->setRequest(new POSTFile());
13 13
         parent::__construct($url, $options);
14 14
     }
15 15
 
16
-    public function execute($data = null) {
16
+    public function execute($data = null){
17 17
         parent::execute($data);
18 18
         $this->setResponse(new JSON($this->Request->getResponse(),$this->Request->getCurlObject()));
19 19
         return $this;
Please login to merge, or discard this patch.
src/EntryPoint/Abstracts/GET/AbstractGetEntryPoint.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -8,12 +8,12 @@
 block discarded – undo
8 8
 
9 9
 abstract class AbstractPostFileEntryPoint extends AbstractEntryPoint {
10 10
 
11
-    public function __construct($url, array $options = array()) {
11
+    public function __construct($url, array $options = array()){
12 12
         $this->setRequest(new POSTFile());
13 13
         parent::__construct($url, $options);
14 14
     }
15 15
 
16
-    public function execute($data = null) {
16
+    public function execute($data = null){
17 17
         parent::execute($data);
18 18
         $this->setResponse(new JSON($this->Request->getResponse(),$this->Request->getCurlObject()));
19 19
         return $this;
Please login to merge, or discard this patch.
src/EntryPoint/Abstracts/POST/AbstractPostEntryPoint.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -8,12 +8,12 @@
 block discarded – undo
8 8
 
9 9
 abstract class AbstractPostFileEntryPoint extends AbstractEntryPoint {
10 10
 
11
-    public function __construct($url, array $options = array()) {
11
+    public function __construct($url, array $options = array()){
12 12
         $this->setRequest(new POSTFile());
13 13
         parent::__construct($url, $options);
14 14
     }
15 15
 
16
-    public function execute($data = null) {
16
+    public function execute($data = null){
17 17
         parent::execute($data);
18 18
         $this->setResponse(new JSON($this->Request->getResponse(),$this->Request->getCurlObject()));
19 19
         return $this;
Please login to merge, or discard this patch.
src/Request/Abstracts/AbstractRequest.php 1 patch
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@
 block discarded – undo
90 90
     /**
91 91
      * Always make sure to destroy Curl Resource
92 92
      */
93
-    public function __destruct() {
93
+    public function __destruct(){
94 94
         if ($this->status!==self::STATUS_CLOSED){
95 95
             curl_close($this->CurlRequest);
96 96
         }
Please login to merge, or discard this patch.
src/EntryPoint/Abstracts/AbstractEntryPoint.php 1 patch
Braces   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
     public function __construct($baseUrl,array $options = array()){
120 120
         $this->baseUrl = $baseUrl;
121 121
 
122
-        if (!empty($options)) {
122
+        if (!empty($options)){
123 123
             $this->setOptions($options);
124 124
         }
125 125
         $this->configureURL();
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
     /**
147 147
      * @inheritdoc
148 148
      */
149
-    public function setAuth($accessToken) {
149
+    public function setAuth($accessToken){
150 150
         $this->accessToken = $accessToken;
151 151
         return $this;
152 152
     }
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
     /**
155 155
      * @inheritdoc
156 156
      */
157
-    public function setUrl($url) {
157
+    public function setUrl($url){
158 158
         $this->Url = $url;
159 159
         return $this;
160 160
     }
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
     /**
163 163
      * @inheritdoc
164 164
      */
165
-    public function setRequest(RequestInterface $Request) {
165
+    public function setRequest(RequestInterface $Request){
166 166
         $this->Request = $Request;
167 167
         return $this;
168 168
     }
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
     /**
171 171
      * @inheritdoc
172 172
      */
173
-    public function setResponse(ResponseInterface $Response) {
173
+    public function setResponse(ResponseInterface $Response){
174 174
         $this->Response = $Response;
175 175
         return $this;
176 176
     }
@@ -220,10 +220,10 @@  discard block
 block discarded – undo
220 220
     public function execute($data = NULL){
221 221
         $data =  ($data === NULL?$this->Data:$data);
222 222
         $this->configureData($data);
223
-        if ($this->verifyOptions() && is_object($this->Request)) {
223
+        if ($this->verifyOptions() && is_object($this->Request)){
224 224
             $this->configureRequest();
225 225
             $this->Request->send();
226
-        }else{
226
+        } else{
227 227
             throw new InvalidRequestException(get_called_class(),"Request property not configured");
228 228
         }
229 229
         return $this;
@@ -232,7 +232,7 @@  discard block
 block discarded – undo
232 232
     /**
233 233
      * @inheritdoc
234 234
      */
235
-    public function authRequired() {
235
+    public function authRequired(){
236 236
         return $this->_AUTH_REQUIRED;
237 237
     }
238 238
 
@@ -242,10 +242,10 @@  discard block
 block discarded – undo
242 242
      * @throws RequiredDataException
243 243
      */
244 244
     protected function configureRequest(){
245
-        if ($this->verifyUrl()) {
245
+        if ($this->verifyUrl()){
246 246
             $this->Request->setURL($this->Url);
247 247
         }
248
-        if ($this->verifyData() && !empty($this->Data)) {
248
+        if ($this->verifyData() && !empty($this->Data)){
249 249
             $this->Request->setBody($this->Data);
250 250
         }
251 251
         $this->configureAuth();
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
      * Configures the authentication header on the Request object
256 256
      */
257 257
     protected function configureAuth(){
258
-        if ($this->authRequired()) {
258
+        if ($this->authRequired()){
259 259
             $this->Request->addHeader('OAuth-Token', $this->accessToken);
260 260
         }
261 261
     }
@@ -292,12 +292,12 @@  discard block
 block discarded – undo
292 292
      */
293 293
     protected function configureURL(){
294 294
         $url = $this->_URL;
295
-        if ($this->requiresOptions()) {
295
+        if ($this->requiresOptions()){
296 296
             $urlParts = explode("/", $this->_URL);
297 297
             $o = 0;
298
-            foreach ($urlParts as $key => $part) {
299
-                if (strpos($part, "$") !== FALSE) {
300
-                    if (isset($this->Options[$o])) {
298
+            foreach ($urlParts as $key => $part){
299
+                if (strpos($part, "$") !== FALSE){
300
+                    if (isset($this->Options[$o])){
301 301
                         $urlParts[$key] = $this->Options[$o];
302 302
                         $o++;
303 303
                     }
@@ -343,7 +343,7 @@  discard block
 block discarded – undo
343 343
      * @throws RequiredDataException
344 344
      */
345 345
     protected function verifyData(){
346
-        if (isset($this->_DATA_TYPE)||!empty($this->_DATA_TYPE)) {
346
+        if (isset($this->_DATA_TYPE)||!empty($this->_DATA_TYPE)){
347 347
             $this->verifyDataType();
348 348
         }
349 349
         if (!empty($this->_REQUIRED_DATA)){
@@ -358,7 +358,7 @@  discard block
 block discarded – undo
358 358
      * @throws RequiredDataException
359 359
      */
360 360
     protected function verifyDataType(){
361
-        if (gettype($this->Data) !== $this->_DATA_TYPE) {
361
+        if (gettype($this->Data) !== $this->_DATA_TYPE){
362 362
             throw new RequiredDataException(get_called_class(),"Valid DataType is {$this->_DATA_TYPE}");
363 363
         }
364 364
         return true;
Please login to merge, or discard this patch.