@@ -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 | |
@@ -31,7 +31,7 @@ discard block |
||
| 31 | 31 | * @param $EntryPoint - EPInterface Class Name |
| 32 | 32 | * @return $this |
| 33 | 33 | */ |
| 34 | - public function registerEntryPoint($function,$EntryPoint); |
|
| 34 | + public function registerEntryPoint($function, $EntryPoint); |
|
| 35 | 35 | |
| 36 | 36 | /** |
| 37 | 37 | * Login to the configured SugarCRM server |
@@ -57,7 +57,7 @@ discard block |
||
| 57 | 57 | * @param $client_id |
| 58 | 58 | * @return boolean |
| 59 | 59 | */ |
| 60 | - public static function storeToken($token,$client_id); |
|
| 60 | + public static function storeToken($token, $client_id); |
|
| 61 | 61 | |
| 62 | 62 | /** |
| 63 | 63 | * Get an SDK Clients authentication Token from Storage |
@@ -15,10 +15,10 @@ discard block |
||
| 15 | 15 | * @return string |
| 16 | 16 | */ |
| 17 | 17 | public static function configureAPIURL($instance){ |
| 18 | - if (strpos($instance,"http")===FALSE){ |
|
| 18 | + if (strpos($instance, "http")===FALSE){ |
|
| 19 | 19 | $instance = "http://".$instance; |
| 20 | 20 | } |
| 21 | - if (strpos($instance,"rest/v10")!==FALSE){ |
|
| 21 | + if (strpos($instance, "rest/v10")!==FALSE){ |
|
| 22 | 22 | $instance = str_replace("rest/v10", "", $instance); |
| 23 | 23 | } |
| 24 | 24 | return rtrim($instance, "/").self::API_URL; |
@@ -39,8 +39,8 @@ discard block |
||
| 39 | 39 | public static function getSDKEntryPointRegistry(){ |
| 40 | 40 | $entryPoints = array(); |
| 41 | 41 | require __DIR__.DIRECTORY_SEPARATOR.'registry.php'; |
| 42 | - foreach ($entryPoints as $funcName => $className) { |
|
| 43 | - $className = "SugarAPI\\SDK\\EntryPoint\\" . $className; |
|
| 42 | + foreach ($entryPoints as $funcName => $className){ |
|
| 43 | + $className = "SugarAPI\\SDK\\EntryPoint\\".$className; |
|
| 44 | 44 | $entryPoints[$funcName] = $className; |
| 45 | 45 | } |
| 46 | 46 | return $entryPoints; |
@@ -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 | } |
@@ -116,10 +116,10 @@ discard block |
||
| 116 | 116 | protected $accessToken; |
| 117 | 117 | |
| 118 | 118 | |
| 119 | - public function __construct($baseUrl,array $options = array()){ |
|
| 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 | } |
@@ -218,13 +218,13 @@ discard block |
||
| 218 | 218 | * @throws InvalidURLException |
| 219 | 219 | */ |
| 220 | 220 | public function execute($data = NULL){ |
| 221 | - $data = ($data === NULL?$this->Data:$data); |
|
| 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 | 226 | }else{ |
| 227 | - throw new InvalidRequestException(get_called_class(),"Request property not configured"); |
|
| 227 | + throw new InvalidRequestException(get_called_class(), "Request property not configured"); |
|
| 228 | 228 | } |
| 229 | 229 | return $this; |
| 230 | 230 | } |
@@ -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 | } |
@@ -265,7 +265,7 @@ discard block |
||
| 265 | 265 | * @var $data |
| 266 | 266 | */ |
| 267 | 267 | protected function configureData($data){ |
| 268 | - if (!empty($this->_REQUIRED_DATA)&&is_array($data)){ |
|
| 268 | + if (!empty($this->_REQUIRED_DATA) && is_array($data)){ |
|
| 269 | 269 | $data = $this->configureDefaultData($data); |
| 270 | 270 | } |
| 271 | 271 | $this->setData($data); |
@@ -277,7 +277,7 @@ discard block |
||
| 277 | 277 | * @return array |
| 278 | 278 | */ |
| 279 | 279 | protected function configureDefaultData(array $data){ |
| 280 | - foreach($this->_REQUIRED_DATA as $property => $value){ |
|
| 280 | + foreach ($this->_REQUIRED_DATA as $property => $value){ |
|
| 281 | 281 | if (!isset($data[$property]) && $value!==NULL){ |
| 282 | 282 | $data[$property] = $value; |
| 283 | 283 | } |
@@ -292,18 +292,18 @@ 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 | } |
| 304 | 304 | } |
| 305 | 305 | } |
| 306 | - $url = implode($urlParts,"/"); |
|
| 306 | + $url = implode($urlParts, "/"); |
|
| 307 | 307 | } |
| 308 | 308 | $url = $this->baseUrl.$url; |
| 309 | 309 | $this->setUrl($url); |
@@ -315,9 +315,9 @@ discard block |
||
| 315 | 315 | * @throws InvalidURLException |
| 316 | 316 | */ |
| 317 | 317 | protected function verifyUrl(){ |
| 318 | - $UrlArray = explode("?",$this->Url); |
|
| 319 | - if (strpos($UrlArray[0],"$") !== FALSE){ |
|
| 320 | - throw new InvalidURLException(get_called_class(),"Configured URL is ".$this->Url); |
|
| 318 | + $UrlArray = explode("?", $this->Url); |
|
| 319 | + if (strpos($UrlArray[0], "$")!==FALSE){ |
|
| 320 | + throw new InvalidURLException(get_called_class(), "Configured URL is ".$this->Url); |
|
| 321 | 321 | } |
| 322 | 322 | return true; |
| 323 | 323 | } |
@@ -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,8 +343,8 @@ discard block |
||
| 343 | 343 | * @throws RequiredDataException |
| 344 | 344 | */ |
| 345 | 345 | protected function verifyDataType(){ |
| 346 | - if (gettype($this->Data) !== $this->_DATA_TYPE) { |
|
| 347 | - throw new RequiredDataException(get_called_class(),"Valid DataType is {$this->_DATA_TYPE}"); |
|
| 346 | + if (gettype($this->Data)!==$this->_DATA_TYPE){ |
|
| 347 | + throw new RequiredDataException(get_called_class(), "Valid DataType is {$this->_DATA_TYPE}"); |
|
| 348 | 348 | } |
| 349 | 349 | return true; |
| 350 | 350 | } |
@@ -356,13 +356,13 @@ discard block |
||
| 356 | 356 | */ |
| 357 | 357 | protected function verifyRequiredData(){ |
| 358 | 358 | $errors = array(); |
| 359 | - foreach($this->_REQUIRED_DATA as $property => $defaultValue){ |
|
| 359 | + foreach ($this->_REQUIRED_DATA as $property => $defaultValue){ |
|
| 360 | 360 | if (!isset($this->Data[$property])){ |
| 361 | 361 | $errors[] = $property; |
| 362 | 362 | } |
| 363 | 363 | } |
| 364 | 364 | if (count($errors)>0){ |
| 365 | - throw new RequiredDataException(get_called_class(),"Missing data for ".implode(",",$errors)); |
|
| 365 | + throw new RequiredDataException(get_called_class(), "Missing data for ".implode(",", $errors)); |
|
| 366 | 366 | } |
| 367 | 367 | return true; |
| 368 | 368 | } |
@@ -372,7 +372,7 @@ discard block |
||
| 372 | 372 | * @return bool |
| 373 | 373 | */ |
| 374 | 374 | protected function requiresOptions(){ |
| 375 | - return strpos($this->_URL,"$") === FALSE?FALSE:TRUE; |
|
| 375 | + return strpos($this->_URL, "$")===FALSE?FALSE:TRUE; |
|
| 376 | 376 | } |
| 377 | 377 | |
| 378 | 378 | } |
| 379 | 379 | \ No newline at end of file |
@@ -32,19 +32,19 @@ |
||
| 32 | 32 | * @throws RequiredOptionsException |
| 33 | 33 | */ |
| 34 | 34 | protected function configureData($data){ |
| 35 | - if (is_string($data)) { |
|
| 36 | - if (!empty($this->Options)) { |
|
| 35 | + if (is_string($data)){ |
|
| 36 | + if (!empty($this->Options)){ |
|
| 37 | 37 | $fileField = end($this->Options); |
| 38 | 38 | $data = array( |
| 39 | 39 | $fileField => $data |
| 40 | 40 | ); |
| 41 | - } else { |
|
| 41 | + }else{ |
|
| 42 | 42 | throw new RequiredOptionsException(get_called_class(), "Options are required, when passing String for data."); |
| 43 | 43 | } |
| 44 | 44 | } |
| 45 | 45 | if (is_array($data)){ |
| 46 | 46 | foreach ($data as $key => $value){ |
| 47 | - if (!array_key_exists($key,$this->_REQUIRED_DATA)){ |
|
| 47 | + if (!array_key_exists($key, $this->_REQUIRED_DATA)){ |
|
| 48 | 48 | $data[$key] = $this->setFileFieldValue($value); |
| 49 | 49 | } |
| 50 | 50 | } |
@@ -30,23 +30,23 @@ |
||
| 30 | 30 | 'method' => '' |
| 31 | 31 | ); |
| 32 | 32 | |
| 33 | - protected function configureData($data) { |
|
| 34 | - if (!isset($data['requests'])) { |
|
| 33 | + protected function configureData($data){ |
|
| 34 | + if (!isset($data['requests'])){ |
|
| 35 | 35 | $requestData = array( |
| 36 | 36 | 'requests' => array() |
| 37 | 37 | ); |
| 38 | 38 | $counter = 0; |
| 39 | - foreach ($data as $key => $EntryPoint) { |
|
| 40 | - if (is_object($EntryPoint)) { |
|
| 39 | + foreach ($data as $key => $EntryPoint){ |
|
| 40 | + if (is_object($EntryPoint)){ |
|
| 41 | 41 | $requestData['requests'][$counter] = $this->bulkRequest; |
| 42 | 42 | $requestData['requests'][$counter]['method'] = $EntryPoint->getRequest()->getType(); |
| 43 | - if ($requestData['requests'][$counter]['method'] == "POST" || $requestData['requests'][$counter]['method'] == "PUT") { |
|
| 43 | + if ($requestData['requests'][$counter]['method']=="POST" || $requestData['requests'][$counter]['method']=="PUT"){ |
|
| 44 | 44 | $requestData['requests'][$counter]['data'] = json_encode($EntryPoint->getData()); |
| 45 | - } else { |
|
| 45 | + }else{ |
|
| 46 | 46 | unset($requestData['requests'][$counter]['data']); |
| 47 | 47 | } |
| 48 | 48 | $requestData['requests'][$counter]['headers'] = $EntryPoint->getRequest()->getHeaders(); |
| 49 | - $requestData['requests'][$counter]['url'] = "v10/" . str_replace($this->baseUrl, "", $EntryPoint->getUrl()); |
|
| 49 | + $requestData['requests'][$counter]['url'] = "v10/".str_replace($this->baseUrl, "", $EntryPoint->getUrl()); |
|
| 50 | 50 | |
| 51 | 51 | $counter++; |
| 52 | 52 | } |
@@ -22,7 +22,7 @@ discard block |
||
| 22 | 22 | protected $destinationPath; |
| 23 | 23 | |
| 24 | 24 | public function __construct($curlRequest, $curlResponse = NULL, $destination = NULL){ |
| 25 | - parent::__construct($curlRequest,$curlResponse); |
|
| 25 | + parent::__construct($curlRequest, $curlResponse); |
|
| 26 | 26 | $this->setDestinationPath($destination); |
| 27 | 27 | } |
| 28 | 28 | |
@@ -31,10 +31,10 @@ discard block |
||
| 31 | 31 | * Extract Filename from Headers |
| 32 | 32 | * @param mixed $curlResponse |
| 33 | 33 | */ |
| 34 | - public function setCurlResponse($curlResponse) { |
|
| 34 | + public function setCurlResponse($curlResponse){ |
|
| 35 | 35 | parent::setCurlResponse($curlResponse); |
| 36 | - if (!$this->error) { |
|
| 37 | - if (empty($this->fileName)) { |
|
| 36 | + if (!$this->error){ |
|
| 37 | + if (empty($this->fileName)){ |
|
| 38 | 38 | $this->extractFileName(); |
| 39 | 39 | } |
| 40 | 40 | $this->writeFile(); |
@@ -95,8 +95,8 @@ discard block |
||
| 95 | 95 | mkdir($this->destinationPath, 0777); |
| 96 | 96 | } |
| 97 | 97 | $file = $this->file(); |
| 98 | - $fileHandle = fopen($file,'w+'); |
|
| 99 | - fwrite($fileHandle,$this->body); |
|
| 98 | + $fileHandle = fopen($file, 'w+'); |
|
| 99 | + fwrite($fileHandle, $this->body); |
|
| 100 | 100 | fclose($fileHandle); |
| 101 | 101 | return $file; |
| 102 | 102 | }else{ |
@@ -45,14 +45,14 @@ discard block |
||
| 45 | 45 | */ |
| 46 | 46 | protected $info; |
| 47 | 47 | |
| 48 | - public function __construct($curlRequest,$curlResponse = NULL){ |
|
| 48 | + public function __construct($curlRequest, $curlResponse = NULL){ |
|
| 49 | 49 | $this->CurlRequest = $curlRequest; |
| 50 | 50 | if ($curlResponse!==NULL){ |
| 51 | 51 | $this->setCurlResponse($curlResponse); |
| 52 | 52 | } |
| 53 | 53 | } |
| 54 | 54 | |
| 55 | - public function setCurlResponse($curlResponse) { |
|
| 55 | + public function setCurlResponse($curlResponse){ |
|
| 56 | 56 | $this->extractInfo(); |
| 57 | 57 | if (!$this->getError()){ |
| 58 | 58 | $this->extractResponse($curlResponse); |
@@ -66,7 +66,7 @@ discard block |
||
| 66 | 66 | protected function extractInfo(){ |
| 67 | 67 | $this->info = curl_getinfo($this->CurlRequest); |
| 68 | 68 | $this->status = $this->info['http_code']; |
| 69 | - if (curl_errno($this->CurlRequest)!== CURLE_OK){ |
|
| 69 | + if (curl_errno($this->CurlRequest)!==CURLE_OK){ |
|
| 70 | 70 | $this->error = curl_error($this->CurlRequest); |
| 71 | 71 | } |
| 72 | 72 | $this->error = FALSE; |