Completed
Push — master ( 15515b...fd48c6 )
by Mike
02:14
created
src/Request/POSTFile.php 1 patch
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@
 block discarded – undo
16 16
      * Overrides POST setBody, so that Body is not json encoded
17 17
      * @inheritdoc
18 18
      */
19
-    public function setBody(array $body) {
19
+    public function setBody(array $body){
20 20
         $this->body = $body;
21 21
         $this->setOption(CURLOPT_POSTFIELDS,$this->body);
22 22
         return $this;
Please login to merge, or discard this patch.
src/Request/POST.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 
5 5
 use SugarAPI\SDK\Request\Abstracts\AbstractRequest;
6 6
 
7
-class POST extends AbstractRequest{
7
+class POST extends AbstractRequest {
8 8
 
9 9
     /**
10 10
      * @inheritdoc
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
      * JSON Encode Body
33 33
      * @inheritdoc
34 34
      */
35
-    public function setBody($body) {
35
+    public function setBody($body){
36 36
         return parent::setBody(json_encode($body));
37 37
     }
38 38
 }
39 39
\ No newline at end of file
Please login to merge, or discard this patch.
src/Request/Abstracts/AbstractRequest.php 1 patch
Braces   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 
5 5
 use SugarAPI\SDK\Request\Interfaces\RequestInterface;
6 6
 
7
-abstract class AbstractRequest implements RequestInterface{
7
+abstract class AbstractRequest implements RequestInterface {
8 8
 
9 9
     const STATUS_INIT = 'initialized';
10 10
     const STATUS_SENT = 'sent';
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
     /**
90 90
      * @inheritdoc
91 91
      */
92
-    public function setURL($url) {
92
+    public function setURL($url){
93 93
         $this->url = $url;
94 94
         $this->setOption(CURLOPT_URL,$this->url);
95 95
         return $this;
@@ -98,14 +98,14 @@  discard block
 block discarded – undo
98 98
     /**
99 99
      * @inheritdoc
100 100
      */
101
-    public function getURL() {
101
+    public function getURL(){
102 102
         return $this->url;
103 103
     }
104 104
 
105 105
     /**
106 106
      * @inheritdoc
107 107
      */
108
-    public function addHeader($name, $value) {
108
+    public function addHeader($name, $value){
109 109
         $token = $name.": ".$value;
110 110
         $this->headers[] = $token;
111 111
         return $this;
@@ -114,9 +114,9 @@  discard block
 block discarded – undo
114 114
     /**
115 115
      * @inheritdoc
116 116
      */
117
-    public function setHeaders(array $array = array()) {
118
-        if (count($array)>0) {
119
-            foreach ($array as $key => $value) {
117
+    public function setHeaders(array $array = array()){
118
+        if (count($array)>0){
119
+            foreach ($array as $key => $value){
120 120
                 $this->addHeader($key, $value);
121 121
             }
122 122
         }
@@ -127,14 +127,14 @@  discard block
 block discarded – undo
127 127
     /**
128 128
      * @inheritdoc
129 129
      */
130
-    public function getHeaders() {
130
+    public function getHeaders(){
131 131
         return $this->headers;
132 132
     }
133 133
 
134 134
     /**
135 135
      * @inheritdoc
136 136
      */
137
-    public function setBody(array $body) {
137
+    public function setBody(array $body){
138 138
         $this->body = $body;
139 139
         $this->setOption(CURLOPT_POSTFIELDS, $this->body);
140 140
         return $this;
@@ -143,28 +143,28 @@  discard block
 block discarded – undo
143 143
     /**
144 144
      * @inheritdoc
145 145
      */
146
-    public function getBody() {
146
+    public function getBody(){
147 147
         return $this->body;
148 148
     }
149 149
 
150 150
     /**
151 151
      * @inheritdoc
152 152
      */
153
-    public function getCurlObject() {
153
+    public function getCurlObject(){
154 154
         return $this->CurlRequest;
155 155
     }
156 156
 
157 157
     /**
158 158
      * @inheritdoc
159 159
      */
160
-    public function setOption($option, $value) {
160
+    public function setOption($option, $value){
161 161
         curl_setopt($this->CurlRequest, $option, $value);
162 162
     }
163 163
 
164 164
     /**
165 165
      * @inheritdoc
166 166
      */
167
-    public function send() {
167
+    public function send(){
168 168
         $this->setHeaders();
169 169
         $this->CurlResponse = curl_exec($this->CurlRequest);
170 170
         $this->status = self::STATUS_SENT;
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
     /**
200 200
      * @inheritdoc
201 201
      */
202
-    public function start() {
202
+    public function start(){
203 203
         $this->CurlRequest = curl_init();
204 204
         $this->status = self::STATUS_INIT;
205 205
         return $this;
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
     /**
219 219
      * @inheritdoc
220 220
      */
221
-    public function getCurlStatus() {
221
+    public function getCurlStatus(){
222 222
         return $this->status;
223 223
     }
224 224
 }
225 225
\ No newline at end of file
Please login to merge, or discard this patch.
src/Request/PUT.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 
5 5
 use SugarAPI\SDK\Request\Abstracts\AbstractRequest;
6 6
 
7
-class PUT extends AbstractRequest{
7
+class PUT extends AbstractRequest {
8 8
 
9 9
     /**
10 10
      * @inheritdoc
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
      * JSON Encode Body
33 33
      * @inheritdoc
34 34
      */
35
-    public function setBody(array $body) {
35
+    public function setBody(array $body){
36 36
         return parent::setBody(json_encode($body));
37 37
     }
38 38
 
Please login to merge, or discard this patch.
src/Request/DELETE.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 
5 5
 use SugarAPI\SDK\Request\Abstracts\AbstractRequest;
6 6
 
7
-class DELETE extends AbstractRequest{
7
+class DELETE extends AbstractRequest {
8 8
 
9 9
     /**
10 10
      * @inheritdoc
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
      * JSON Encode Body
33 33
      * @inheritdoc
34 34
      */
35
-    public function setBody(array $body) {
35
+    public function setBody(array $body){
36 36
         return parent::setBody(json_encode($body));
37 37
     }
38 38
 
Please login to merge, or discard this patch.
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/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/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.