@@ -4,7 +4,7 @@ |
||
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 |
@@ -39,7 +39,7 @@ |
||
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 | } |
@@ -8,12 +8,12 @@ |
||
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; |
@@ -8,12 +8,12 @@ |
||
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; |
@@ -8,12 +8,12 @@ |
||
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; |
@@ -90,7 +90,7 @@ |
||
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 | } |
@@ -9,12 +9,12 @@ |
||
9 | 9 | |
10 | 10 | abstract class AbstractDeleteEntryPoint extends AbstractEntryPoint { |
11 | 11 | |
12 | - public function __construct($url, array $options = array()) { |
|
12 | + public function __construct($url, array $options = array()){ |
|
13 | 13 | $this->setRequest(new DELETE()); |
14 | 14 | parent::__construct($url, $options); |
15 | 15 | } |
16 | 16 | |
17 | - public function execute($data = NULL) { |
|
17 | + public function execute($data = NULL){ |
|
18 | 18 | parent::execute($data); |
19 | 19 | $this->setResponse(new JSON($this->Request->getResponse(),$this->Request->getCurlObject())); |
20 | 20 | return $this; |
@@ -45,8 +45,7 @@ discard block |
||
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 | } |
@@ -72,7 +71,7 @@ discard block |
||
72 | 71 | fwrite($fileHandle,$this->body); |
73 | 72 | fclose($fileHandle); |
74 | 73 | return $file; |
75 | - }else{ |
|
74 | + } else{ |
|
76 | 75 | return FALSE; |
77 | 76 | } |
78 | 77 | } |
@@ -119,7 +119,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
220 | 220 | public function execute($data = NULL){ |
221 | 221 | $data = ($data === NULL?$this->Data:$data); |
222 | 222 | $this->configureData($data); |
223 | - if (is_object($this->Request)) { |
|
223 | + if (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 |
||
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 |
||
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 |
||
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 |
||
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 | } |
@@ -328,7 +328,7 @@ discard block |
||
328 | 328 | * @throws RequiredDataException |
329 | 329 | */ |
330 | 330 | protected function verifyData(){ |
331 | - if (isset($this->_DATA_TYPE)||!empty($this->_DATA_TYPE)) { |
|
331 | + if (isset($this->_DATA_TYPE)||!empty($this->_DATA_TYPE)){ |
|
332 | 332 | $this->verifyDataType(); |
333 | 333 | } |
334 | 334 | if (!empty($this->_REQUIRED_DATA)){ |
@@ -343,7 +343,7 @@ discard block |
||
343 | 343 | * @throws RequiredDataException |
344 | 344 | */ |
345 | 345 | protected function verifyDataType(){ |
346 | - if (gettype($this->Data) !== $this->_DATA_TYPE) { |
|
346 | + if (gettype($this->Data) !== $this->_DATA_TYPE){ |
|
347 | 347 | throw new RequiredDataException(get_called_class(),"Valid DataType is {$this->_DATA_TYPE}"); |
348 | 348 | } |
349 | 349 | return true; |