MichaelJ2324 / sugarcrm-restv10-sdk

src/Request/Abstracts/AbstractRequest.php

Braces +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
     /**
      * Always make sure to destroy Curl Resource
      */
-    public function __destruct() {
+    public function __destruct(){
         if ($this->status!==self::STATUS_CLOSED){
             curl_close($this->CurlRequest);
         }

src/EntryPoint/Abstracts/AbstractEntryPoint.php

Braces +18 added lines, -18 removed lines

@@ -119,9 +119,9 @@ 
     public function __construct($baseUrl,array $options = array()){
         $this->baseUrl = $baseUrl;
 
-        if (!empty($options)) {
+        if (!empty($options)){
             $this->setOptions($options);
-        }elseif(!$this->requiresOptions()){
+        } elseif(!$this->requiresOptions()){
             $this->configureURL();
         }
     }
@@ -147,7 +147,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setAuth($accessToken) {
+    public function setAuth($accessToken){
         $this->accessToken = $accessToken;
         return $this;
     }
@@ -155,7 +155,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setUrl($url) {
+    public function setUrl($url){
         $this->Url = $url;
         return $this;
     }
@@ -163,7 +163,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setRequest(RequestInterface $Request) {
+    public function setRequest(RequestInterface $Request){
         $this->Request = $Request;
         return $this;
     }
@@ -171,7 +171,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setResponse(ResponseInterface $Response) {
+    public function setResponse(ResponseInterface $Response){
         $this->Response = $Response;
         return $this;
     }
@@ -221,10 +221,10 @@ 
     public function execute($data = NULL){
         $data =  ($data === NULL?$this->Data:$data);
         $this->configureData($data);
-        if ($this->verifyOptions() && is_object($this->Request)) {
+        if ($this->verifyOptions() && is_object($this->Request)){
             $this->configureRequest();
             $this->Request->send();
-        }else{
+        } else{
             throw new InvalidRequestException(get_called_class(),"Request property not configured");
         }
         return $this;
@@ -233,7 +233,7 @@ 
     /**
      * @inheritdoc
      */
-    public function authRequired() {
+    public function authRequired(){
         return $this->_AUTH_REQUIRED;
     }
 
@@ -243,10 +243,10 @@ 
      * @throws RequiredDataException
      */
     protected function configureRequest(){
-        if ($this->verifyUrl()) {
+        if ($this->verifyUrl()){
             $this->Request->setURL($this->Url);
         }
-        if ($this->verifyData() && !empty($this->Data)) {
+        if ($this->verifyData() && !empty($this->Data)){
             $this->Request->setBody($this->Data);
         }
         $this->configureAuth();
@@ -256,7 +256,7 @@ 
      * Configures the authentication header on the Request object
      */
     protected function configureAuth(){
-        if ($this->authRequired()) {
+        if ($this->authRequired()){
             $this->Request->addHeader('OAuth-Token', $this->accessToken);
         }
     }
@@ -293,12 +293,12 @@ 
      */
     protected function configureURL(){
         $url = $this->_URL;
-        if ($this->requiresOptions()) {
+        if ($this->requiresOptions()){
             $urlParts = explode("/", $this->_URL);
             $o = 0;
-            foreach ($urlParts as $key => $part) {
-                if (strpos($part, "$") !== FALSE) {
-                    if (isset($this->Options[$o])) {
+            foreach ($urlParts as $key => $part){
+                if (strpos($part, "$") !== FALSE){
+                    if (isset($this->Options[$o])){
                         $urlParts[$key] = $this->Options[$o];
                         $o++;
                     }
@@ -344,7 +344,7 @@ 
      * @throws RequiredDataException
      */
     protected function verifyData(){
-        if (isset($this->_DATA_TYPE)||!empty($this->_DATA_TYPE)) {
+        if (isset($this->_DATA_TYPE)||!empty($this->_DATA_TYPE)){
             $this->verifyDataType();
         }
         if (!empty($this->_REQUIRED_DATA)){
@@ -359,7 +359,7 @@ 
      * @throws RequiredDataException
      */
     protected function verifyDataType(){
-        if (gettype($this->Data) !== $this->_DATA_TYPE) {
+        if (gettype($this->Data) !== $this->_DATA_TYPE){
             throw new RequiredDataException(get_called_class(),"Valid DataType is {$this->_DATA_TYPE}");
         }
         return true;

tests/Helpers/HelpersTest.php

Braces +6 added lines, -12 removed lines

@@ -12,21 +12,17 @@ 
  */
 class HelpersTest extends \PHPUnit_Framework_TestCase {
 
-    public static function setUpBeforeClass()
-    {
+    public static function setUpBeforeClass(){
     }
 
-    public static function tearDownAfterClass()
-    {
+    public static function tearDownAfterClass(){
     }
 
-    public function setUp()
-    {
+    public function setUp(){
         parent::setUp();
     }
 
-    public function tearDown()
-    {
+    public function tearDown(){
         parent::tearDown();
     }
 
@@ -35,8 +31,7 @@ 
      * @covers ::configureAPIURL
      * @group default
      */
-    public function testConfigureAPIUrl()
-    {
+    public function testConfigureAPIUrl(){
         $this->assertEquals('http://localhost/rest/v10/',Helpers::configureAPIURL('localhost'));
         $this->assertEquals('https://localhost/rest/v10/',Helpers::configureAPIURL('https://localhost'));
         $this->assertEquals('http://localhost/rest/v10/',Helpers::configureAPIURL('localhost/rest/v10'));
@@ -53,8 +48,7 @@ 
      * @covers ::getSDKVersion
      * @group default
      */
-    public function testGetSDKVersion()
-    {
+    public function testGetSDKVersion(){
         $this->assertEquals('1.0',Helpers::getSDKVersion());
     }
 

tests/Clients/SugarAPITest.php

Braces +5 added lines, -10 removed lines

@@ -13,12 +13,10 @@ 
 class SugarAPITest extends \PHPUnit_Framework_TestCase {
 
 
-    public static function setUpBeforeClass()
-    {
+    public static function setUpBeforeClass(){
     }
 
-    public static function tearDownAfterClass()
-    {
+    public static function tearDownAfterClass(){
     }
 
     protected $SugarAPI;
@@ -31,14 +29,12 @@ 
     );
 
 
-    public function setUp()
-    {
+    public function setUp(){
         $this->SugarAPI = new SugarAPI();
         parent::setUp();
     }
 
-    public function tearDown()
-    {
+    public function tearDown(){
         unset($this->SugarAPI);
         parent::tearDown();
     }
@@ -48,8 +44,7 @@ 
      * @covers ::setCredentials
      * @group sugarapi
      */
-    public function testSetCredentials()
-    {
+    public function testSetCredentials(){
         $this->SugarAPI->setCredentials(array('username' => 'admin'));
         $this->assertEquals(array(
             'username' => 'admin',

tests/EntryPoint/AbstractEntryPointTest.php

Braces +4 added lines, -8 removed lines

@@ -18,12 +18,10 @@ 
  */
 class AbstractEntryPointTest extends \PHPUnit_Framework_TestCase {
 
-    public static function setUpBeforeClass()
-    {
+    public static function setUpBeforeClass(){
     }
 
-    public static function tearDownAfterClass()
-    {
+    public static function tearDownAfterClass(){
     }
 
     protected $Stub;
@@ -33,13 +31,11 @@ 
         'foo' => 'bar'
     );
 
-    public function setUp()
-    {
+    public function setUp(){
         parent::setUp();
     }
 
-    public function tearDown()
-    {
+    public function tearDown(){
         parent::tearDown();
     }