Completed
Push — master ( 15515b...fd48c6 )
by Mike
02:14
created
src/Response/Abstracts/AbstractResponse.php 1 patch
Braces   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 
5 5
 use SugarAPI\SDK\Response\Interfaces\ResponseInterface;
6 6
 
7
-abstract class AbstractResponse implements ResponseInterface{
7
+abstract class AbstractResponse implements ResponseInterface {
8 8
 
9 9
     /**
10 10
      * Full Curl Response
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 
39 39
     public function __construct($curlResponse,$curlRequest){
40 40
         $this->CurlResponse = $curlResponse;
41
-        if ($this->checkErrors($curlRequest)) {
41
+        if ($this->checkErrors($curlRequest)){
42 42
             $this->extractResponse($curlRequest);
43 43
         }
44 44
         $this->setStatus($curlRequest);
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
      * @return bool
69 69
      */
70 70
     protected function checkErrors($curlRequest){
71
-        if (curl_errno($curlRequest) !== CURLE_OK) {
71
+        if (curl_errno($curlRequest) !== CURLE_OK){
72 72
             $this->error = curl_error($curlRequest);
73 73
             return false;
74 74
         }
@@ -85,14 +85,14 @@  discard block
 block discarded – undo
85 85
     /**
86 86
      * @inheritdoc
87 87
      */
88
-    public function getBody() {
88
+    public function getBody(){
89 89
         return $this->body;
90 90
     }
91 91
 
92 92
     /**
93 93
      * @inheritdoc
94 94
      */
95
-    public function getHeaders() {
95
+    public function getHeaders(){
96 96
         return $this->headers;
97 97
     }
98 98
 
Please login to merge, or discard this patch.
src/Response/JSON.php 1 patch
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@
 block discarded – undo
17 17
     /**
18 18
      * @inheritdoc
19 19
      */
20
-    public function getBody() {
20
+    public function getBody(){
21 21
         return json_decode($this->body);
22 22
     }
23 23
 
Please login to merge, or discard this patch.
src/Response/File.php 1 patch
Braces   +3 added lines, -4 removed lines patch added patch discarded remove patch
@@ -18,10 +18,10 @@  discard block
 block discarded – undo
18 18
      */
19 19
     protected $destinationPath;
20 20
 
21
-    public function __construct($curlResponse, $curlRequest,$destination = null) {
21
+    public function __construct($curlResponse, $curlRequest,$destination = null){
22 22
         parent::__construct($curlResponse, $curlRequest);
23 23
         $this->extractFileName();
24
-        if (!empty($destination)) {
24
+        if (!empty($destination)){
25 25
             $this->setupDestination($destination);
26 26
             $this->writeFile();
27 27
         }
@@ -45,8 +45,7 @@  discard block
 block discarded – undo
45 45
      * Extract the filename from the Headers, and store it in filename property
46 46
      */
47 47
     protected function extractFileName(){
48
-        foreach (explode("\r\n",$this->headers) as $header)
49
-        {
48
+        foreach (explode("\r\n",$this->headers) as $header){
50 49
             if (strpos($header,'filename')!==FALSE){
51 50
                 $this->fileName = substr($header,(strpos($header,"\"")+1),-1);
52 51
             }
Please login to merge, or discard this patch.
src/Exception/EntryPointException.php 1 patch
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@
 block discarded – undo
4 4
 
5 5
 class EntryPointException extends SDKException {
6 6
 
7
-    public function __construct($message) {
7
+    public function __construct($message){
8 8
         parent::__construct($message);
9 9
     }
10 10
 
Please login to merge, or discard this patch.
src/Exception/SDKException.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -2,9 +2,9 @@
 block discarded – undo
2 2
 
3 3
 namespace SugarAPI\SDK\Exception;
4 4
 
5
-class SDKException extends \Exception{
5
+class SDKException extends \Exception {
6 6
 
7
-    public function __construct($message = 'Unknown SugarAPI SDK Exception Occurred.') {
7
+    public function __construct($message = 'Unknown SugarAPI SDK Exception Occurred.'){
8 8
         parent::__construct($message);
9 9
     }
10 10
 
Please login to merge, or discard this patch.
src/Exception/AuthenticationException.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -2,11 +2,11 @@
 block discarded – undo
2 2
 
3 3
 namespace SugarAPI\SDK\Exception;
4 4
 
5
-class AuthenticationException extends SDKException{
5
+class AuthenticationException extends SDKException {
6 6
 
7 7
     protected $message = 'Authentication Exception occurred when attempting to Login to SugarCRM instance. Message: %s';
8 8
 
9
-    public function __construct($message) {
9
+    public function __construct($message){
10 10
         $message = sprintf($this->message,$message);
11 11
         parent::__construct($message);
12 12
     }
Please login to merge, or discard this patch.
src/EntryPoint/Abstracts/AbstractEntryPoint.php 1 patch
Braces   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -89,9 +89,9 @@  discard block
 block discarded – undo
89 89
         $this->url = $url;
90 90
         $this->Module = $this->_MODULE;
91 91
 
92
-        if (!empty($options)) {
93
-            if (empty($this->Module)) {
94
-                if (strpos($this->_URL, '$module') !== FALSE) {
92
+        if (!empty($options)){
93
+            if (empty($this->Module)){
94
+                if (strpos($this->_URL, '$module') !== FALSE){
95 95
                     $this->module($options[0]);
96 96
                     array_shift($options);
97 97
                 }
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
      * @inheritdoc
131 131
      */
132 132
     public function execute(){
133
-        if ($this->verifyURL() && $this->validateData()) {
133
+        if ($this->verifyURL() && $this->validateData()){
134 134
             $this->configureURL();
135 135
             $this->Request->setURL($this->url);
136 136
             $this->Request->send();
@@ -144,14 +144,14 @@  discard block
 block discarded – undo
144 144
     /**
145 145
      * @inheritdoc
146 146
      */
147
-    public function authRequired() {
147
+    public function authRequired(){
148 148
         return $this->_AUTH_REQUIRED;
149 149
     }
150 150
 
151 151
     /**
152 152
      * @inheritdoc
153 153
      */
154
-    public function configureAuth($accessToken) {
154
+    public function configureAuth($accessToken){
155 155
         if ($this->authRequired()){
156 156
             $this->accessToken = $accessToken;
157 157
             $this->Request->addHeader('OAuth-Token', $accessToken);
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
     /**
163 163
      * @inheritdoc
164 164
      */
165
-    public function getModule() {
165
+    public function getModule(){
166 166
         return $this->Module;
167 167
     }
168 168
 
@@ -201,24 +201,24 @@  discard block
 block discarded – undo
201 201
      */
202 202
     protected function configureURL(){
203 203
         $url = $this->_URL;
204
-        if (strpos($this->_URL,"$")!==FALSE) {
205
-            if (count($this->Options) > 0 || !empty($this->Module)) {
204
+        if (strpos($this->_URL,"$")!==FALSE){
205
+            if (count($this->Options) > 0 || !empty($this->Module)){
206 206
                 $urlParts = explode("/", $this->_URL);
207 207
                 $o = 0;
208
-                foreach ($urlParts as $key => $part) {
209
-                    if (strpos($part, '$module') !== FALSE) {
210
-                        if (isset($this->Module)) {
208
+                foreach ($urlParts as $key => $part){
209
+                    if (strpos($part, '$module') !== FALSE){
210
+                        if (isset($this->Module)){
211 211
                             $urlParts[$key] = $this->Module;
212 212
                             continue;
213
-                        } else {
214
-                            if (isset($this->Options[$o])) {
213
+                        } else{
214
+                            if (isset($this->Options[$o])){
215 215
                                 $this->Module = $this->Options[$o];
216 216
                                 array_shift($this->Options);
217 217
                             }
218 218
                         }
219 219
                     }
220
-                    if (strpos($part, "$") !== FALSE) {
221
-                        if (isset($this->Options[$o])) {
220
+                    if (strpos($part, "$") !== FALSE){
221
+                        if (isset($this->Options[$o])){
222 222
                             $urlParts[$key] = $this->Options[$o];
223 223
                             $o++;
224 224
                         }
@@ -259,10 +259,10 @@  discard block
 block discarded – undo
259 259
         if ($urlVarCount!==$optionCount){
260 260
             if (empty($this->Module) && strpos($this->_URL,'$module')){
261 261
                 throw new EntryPointException('Module is required for EntryPoint '.get_called_class());
262
-            }else{
262
+            } else{
263 263
                 throw new EntryPointException('EntryPoint URL ('.$this->_URL.') requires more parameters than passed.');
264 264
             }
265
-        }else{
265
+        } else{
266 266
             return true;
267 267
         }
268 268
     }
@@ -275,18 +275,18 @@  discard block
 block discarded – undo
275 275
     protected function validateData(){
276 276
         if (empty($this->_REQUIRED_DATA)||count($this->_REQUIRED_DATA)==0){
277 277
             return true;
278
-        }else{
278
+        } else{
279 279
             $errors = array();
280 280
             foreach($this->_REQUIRED_DATA as $property){
281 281
                 if (isset($this->Data[$property]) || $this->Data[$property]!==null){
282 282
                     continue;
283
-                }else{
283
+                } else{
284 284
                     $errors[] = $property;
285 285
                 }
286 286
             }
287 287
             if (count($errors)>0){
288 288
                 throw new EntryPointException('EntryPoint requires specific properties in Request data. Missing the following '.implode($errors,","));
289
-            }else{
289
+            } else{
290 290
                 return true;
291 291
             }
292 292
         }
Please login to merge, or discard this patch.
src/EntryPoint/Abstracts/GET/FileEntryPoint.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
     /**
18 18
      * @inheritdoc
19 19
      */
20
-    protected function setupRequest() {
20
+    protected function setupRequest(){
21 21
         $this->Request = new GETFile();
22 22
     }
23 23
 
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
      * @inheritdoc
26 26
      * File Response Object has option File Destination parameter, which is passed using DownloadDir property
27 27
      */
28
-    protected function setupResponse() {
28
+    protected function setupResponse(){
29 29
         $this->Response = new FileResponse($this->Request->getResponse(),$this->Request->getCurlObject(),$this->downloadDir);
30 30
     }
31 31
 
Please login to merge, or discard this patch.
src/EntryPoint/Abstracts/POST/FileEntryPoint.php 1 patch
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@
 block discarded – undo
10 10
     /**
11 11
      * @inheritdoc
12 12
      */
13
-    protected function setupRequest() {
13
+    protected function setupRequest(){
14 14
         $this->Request = new POSTFile();
15 15
     }
16 16
 
Please login to merge, or discard this patch.