webmarketingroi / optimizely-php

tests/OptimizelyPHPTest/Service/v2/AudiencesTest.php

Spacing +5 added lines, -5 removed lines

@@ -49,9 +49,9 @@ 
         $result = $audiencesService->listAll(1000);
         $audiences = $result->getPayload();
         
-        $this->assertTrue(count($audiences)==1);
+        $this->assertTrue(count($audiences) == 1);
         $this->assertTrue($audiences[0] instanceOf Audience);
-        $this->assertTrue($audiences[0]->getName()=='Spanish speaking San Franciscans');        
+        $this->assertTrue($audiences[0]->getName() == 'Spanish speaking San Franciscans');        
     }
     
     /**
@@ -122,7 +122,7 @@ 
         $audience = $result->getPayload();
         
         $this->assertTrue($audience instanceOf Audience);
-        $this->assertTrue($audience->getName()=='Spanish speaking San Franciscans');        
+        $this->assertTrue($audience->getName() == 'Spanish speaking San Franciscans');        
     }
     
     public function testCreate()
@@ -182,7 +182,7 @@ 
         $createdAudience = $result->getPayload();
         
         $this->assertTrue($createdAudience instanceOf Audience);
-        $this->assertTrue($createdAudience->getName()=='Spanish speaking San Franciscans');                
+        $this->assertTrue($createdAudience->getName() == 'Spanish speaking San Franciscans');                
     }
     
     public function testUpdate()
@@ -242,7 +242,7 @@ 
         $createdAudience = $result->getPayload();
         
         $this->assertTrue($createdAudience instanceOf Audience);
-        $this->assertTrue($createdAudience->getName()=='Spanish speaking San Franciscans');                
+        $this->assertTrue($createdAudience->getName() == 'Spanish speaking San Franciscans');                
     }
 }
 

lib/WebMarketingROI/OptimizelyPHP/OptimizelyApiClient.php

Spacing +40 added lines, -40 removed lines

@@ -50,13 +50,13 @@ 
      * @param array $authCredentials Auth credentials.
      * @param string $apiVersion Optional. Currently supported 'v2' only.     
      */
-    public function __construct($authCredentials, $apiVersion='v2')
+    public function __construct($authCredentials, $apiVersion = 'v2')
     {
         if (!is_array($authCredentials)) {
             throw new Exception('Auth credentials must be an array');            
         }
         
-        if ($apiVersion!='v2') {
+        if ($apiVersion != 'v2') {
             throw new Exception('Invalid API version passed');
         }
         
@@ -85,7 +85,7 @@ 
      */
     public function setApiVersion($apiVersion)
     {
-        if ($apiVersion!='v2') {
+        if ($apiVersion != 'v2') {
             throw new Exception('Invalid API version passed');
         }
         
@@ -107,7 +107,7 @@ 
      */
     public function setAuthCredentials($authCredentials)
     {
-        if (!is_array($authCredentials) || count($authCredentials)==0) {
+        if (!is_array($authCredentials) || count($authCredentials) == 0) {
             throw new Exception('Auth credentials must be an non-empty array');            
         }
         
@@ -166,16 +166,16 @@ 
      * @return array Optimizely response in form of array.
      * @throws Exception
      */
-    public function sendApiRequest($url, $queryParams = array(), $method='GET', 
+    public function sendApiRequest($url, $queryParams = array(), $method = 'GET', 
             $postData = array())
     {
         // If access token has expired, try to get another one with refresh token.
-        if ($this->isAccessTokenExpired() && $this->getRefreshToken()!=null) {
+        if ($this->isAccessTokenExpired() && $this->getRefreshToken() != null) {
             $this->getAccessTokenByRefreshToken();
         }
         
         // Produce absolute URL
-        $url = 'https://api.optimizely.com/' . $this->apiVersion . $url;
+        $url = 'https://api.optimizely.com/'.$this->apiVersion.$url;
         
         $result = $this->sendHttpRequest($url, $queryParams, $method, $postData);
         
@@ -191,7 +191,7 @@ 
      * @return array Optimizely response in form of array.
      * @throws Exception
      */
-    private function sendHttpRequest($url, $queryParams = array(), $method='GET', 
+    private function sendHttpRequest($url, $queryParams = array(), $method = 'GET', 
             $postData = array())
     {
         // Reset diagnostics info.
@@ -199,36 +199,36 @@ 
         
         // Check if CURL is initialized (it should have been initialized in 
         // constructor).
-        if ($this->curlHandle==false) {
+        if ($this->curlHandle == false) {
             throw new Exception('CURL is not initialized', 
                     Exception::CODE_CURL_ERROR);
         }
         
-        if ($method!='GET' && $method!='POST' && $method!='PUT' && 
-            $method!='PATCH' && $method!='DELETE') {
-            throw new Exception('Invalid HTTP method passed: ' . $method);
+        if ($method != 'GET' && $method != 'POST' && $method != 'PUT' && 
+            $method != 'PATCH' && $method != 'DELETE') {
+            throw new Exception('Invalid HTTP method passed: '.$method);
         }
         
         if (!isset($this->authCredentials['access_token'])) {
-            throw new Exception('OAuth access token is not set. You should pass ' . 
+            throw new Exception('OAuth access token is not set. You should pass '. 
                     'it to the class constructor when initializing the Optimizely client.');
         }
                 
         // Append query parameters to URL.
-        if (count($queryParams)!=0) {            
+        if (count($queryParams) != 0) {            
             $query = http_build_query($queryParams);
-            $url .= '?' . $query;
+            $url .= '?'.$query;
         }
         
         $headers = array(
-            "Authorization: Bearer " . $this->authCredentials['access_token'],
+            "Authorization: Bearer ".$this->authCredentials['access_token'],
             "Content-Type: application/json"
             );
         $content = '';
-        if (count($postData)!=0) {
+        if (count($postData) != 0) {
             $content = json_encode($postData);            
         }
-        $headers[] = "Content-length:" . strlen($content);            
+        $headers[] = "Content-length:".strlen($content);            
         
         // Reset CURL state.
         if (!function_exists('curl_reset')) {
@@ -241,7 +241,7 @@ 
         // Set HTTP options.
         curl_setopt($this->curlHandle, CURLOPT_URL, $url);
         curl_setopt($this->curlHandle, CURLOPT_CUSTOMREQUEST, $method);        
-        if (count($postData)!=0) {
+        if (count($postData) != 0) {
             curl_setopt($this->curlHandle, CURLOPT_POSTFIELDS, $content);            
         }
         curl_setopt($this->curlHandle, CURLOPT_RETURNTRANSFER, true);
@@ -260,7 +260,7 @@ 
         if ($result === false) {
             $code = curl_errno($this->curlHandle);
             $error = curl_error($this->curlHandle);
-            throw new Exception("Failed to send HTTP request $method '$url', " . 
+            throw new Exception("Failed to send HTTP request $method '$url', ". 
                     "the error code was $code, error message was: '$error'", 
                     Exception::CODE_CURL_ERROR, $code, $error);
         }        
@@ -274,12 +274,12 @@ 
         $headers = explode("\n", $headers);
         $parsedHeaders = array();
         foreach ($headers as $i=>$header) {
-            if ($i==0)
+            if ($i == 0)
                 continue; // Skip first line (http code).
             $pos = strpos($header, ':');
-            if ($pos!=false) {
+            if ($pos != false) {
                 $headerName = trim(strtolower(substr($header, 0, $pos)));
-                $headerValue = trim(substr($header, $pos+1));
+                $headerValue = trim(substr($header, $pos + 1));
                 $parsedHeaders[$headerName] = $headerValue;
             }
         }
@@ -311,20 +311,20 @@ 
         
         // JSON-decode payload.
         $decodedPayload = json_decode($body, true);
-        if ($decodedPayload===false) {
-            throw new Exception('Could not JSON-decode the Optimizely API response. Request was ' . 
-                    $method . ' "' . $url . '". The response was: "' . $body . '"',
+        if ($decodedPayload === false) {
+            throw new Exception('Could not JSON-decode the Optimizely API response. Request was '. 
+                    $method.' "'.$url.'". The response was: "'.$body.'"',
                     Exception::CODE_API_ERROR, array('http_code'=>$httpCode));
         }
         
         // Check HTTP response code.
-        if ($httpCode<200 || $httpCode>299) {
+        if ($httpCode < 200 || $httpCode > 299) {
             
             if (!isset($decodedPayload['message']) || 
                 !isset($decodedPayload['code']) ||
                 !isset($decodedPayload['uuid'])) {
-                throw new Exception('Optimizely API responded with error code ' . $httpCode . 
-                    '. Request was ' . $method . ' "' . $url . '". Response was "' . $body . '"',
+                throw new Exception('Optimizely API responded with error code '.$httpCode. 
+                    '. Request was '.$method.' "'.$url.'". Response was "'.$body.'"',
                     Exception::CODE_API_ERROR, array(
                         'http_code' => $httpCode,
                         'rate_limit' => $rateLimit,
@@ -356,7 +356,7 @@ 
             $matched = preg_match_all('/<(.+)>;\s+rel=(\w+)(,|\z)/U', 
                     $parsedHeaders['link'], $matches, PREG_SET_ORDER);
             if (!$matched) {
-                throw new Exception('Error parsing LINK header: ' . 
+                throw new Exception('Error parsing LINK header: '. 
                         $parsedHeaders['link'], Exception::CODE_API_ERROR, $httpCode);
             }
             
@@ -366,22 +366,22 @@ 
                 $rel = $match[2];
                 
                 $matched = preg_match('/page=(\d+)/U', $url, $pageMatches);
-                if (!$matched || count($pageMatches)!=2) {
-                    throw new Exception('Error extracting page argument while parsing LINK header: ' . 
+                if (!$matched || count($pageMatches) != 2) {
+                    throw new Exception('Error extracting page argument while parsing LINK header: '. 
                             $parsedHeaders['link'], Exception::CODE_API_ERROR, 
                             array('http_code'=>$httpCode));
                 }
                 
                 $pageNumber = $pageMatches[1];
                 
-                if ($rel=='prev') {
+                if ($rel == 'prev') {
                     $result->setPrevPage($pageNumber);
-                } else if ($rel=='next') {
+                } else if ($rel == 'next') {
                     $result->setNextPage($pageNumber);
-                } else if ($rel=='last') {
+                } else if ($rel == 'last') {
                     $result->setLastPage($pageNumber);
                 } else {
-                    throw new Exception('Unexpected rel argument while parsing LINK header: ' . 
+                    throw new Exception('Unexpected rel argument while parsing LINK header: '. 
                             $parsedHeaders['link'], Exception::CODE_API_ERROR, 
                             array('http_code'=>$httpCode));
                 }
@@ -399,7 +399,7 @@ 
      */
     private function isAccessTokenExpired() 
     {
-        if(!isset($this->authCredentials['access_token'])) {
+        if (!isset($this->authCredentials['access_token'])) {
             return true; // We do not have access token.
         }
         
@@ -440,15 +440,15 @@ 
         $clientSecret = $this->authCredentials['client_secret'];
         $refreshToken = $this->authCredentials['refresh_token'];
         
-        $url = "https://app.optimizely.com/oauth2/token?refresh_token=$refreshToken" . 
+        $url = "https://app.optimizely.com/oauth2/token?refresh_token=$refreshToken". 
                 "&client_id=$clientId&client_secret=$clientSecret&grant_type=refresh_token";
         
         $response = $this->sendHttpRequest($url, array(), 'POST');
         $decodedJsonData = $response->getDecodedJsonData();
         
         if (!isset($decodedJsonData['access_token'])) {
-            throw new Exception('Not found access token in response. Request URL was "' . 
-                    $url. '". Response was "' . print_r(json_encode($decodedJsonData), true). '"',
+            throw new Exception('Not found access token in response. Request URL was "'. 
+                    $url.'". Response was "'.print_r(json_encode($decodedJsonData), true).'"',
                     Exception::CODE_API_ERROR, $response->getHttpCode());
         }
         

lib/WebMarketingROI/OptimizelyPHP/Service/v2/Projects.php

Spacing +6 added lines, -6 removed lines

@@ -35,13 +35,13 @@ 
      * @return Result
      * @throws Exception
      */
-    public function listAll($page=1, $perPage=25)
+    public function listAll($page = 1, $perPage = 25)
     {
-        if ($page<0) {
+        if ($page < 0) {
             throw new Exception('Invalid page number passed');
         }
         
-        if ($perPage<0 || $perPage>100) {
+        if ($perPage < 0 || $perPage > 100) {
             throw new Exception('Invalid page size passed');
         }
         
@@ -69,7 +69,7 @@ 
      */
     public function get($projectId)
     {
-        $projectId = (string)$projectId;
+        $projectId = (string) $projectId;
         
         if (!preg_match('/^\d+$/', $projectId)) {
             throw new Exception("A positive integer project ID expected, while got $projectId");
@@ -114,13 +114,13 @@ 
      */
     public function update($projectId, $project) 
     {
-        $projectId = (string)$projectId;
+        $projectId = (string) $projectId;
         
         if (!preg_match('/\d+/', $projectId)) {
             throw new Exception("A positive integer project ID expected, while got $projectId");
         }
         
-        if ($projectId<0) {
+        if ($projectId < 0) {
             throw new Exception("A positive project ID expected");
         }
         

lib/WebMarketingROI/OptimizelyPHP/Resource/v2/Event.php

Braces +3 added lines, -2 removed lines

@@ -91,8 +91,9 @@
         // Remove options with empty values
         $cleanedOptions = array();
         foreach ($options as $name=>$value) {
-            if ($value!==null)
-                $cleanedOptions[$name] = $value;
+            if ($value!==null) {
+                            $cleanedOptions[$name] = $value;
+            }
         }
         
         return $cleanedOptions;

lib/WebMarketingROI/OptimizelyPHP/Resource/v2/VariantResults.php

Braces +3 added lines, -2 removed lines

@@ -91,8 +91,9 @@
         // Remove options with empty values
         $cleanedOptions = array();
         foreach ($options as $name=>$value) {
-            if ($value!==null)
-                $cleanedOptions[$name] = $value;
+            if ($value!==null) {
+                            $cleanedOptions[$name] = $value;
+            }
         }
         
         return $cleanedOptions;

lib/WebMarketingROI/OptimizelyPHP/Resource/v2/VariationReach.php

Braces +3 added lines, -2 removed lines

@@ -91,8 +91,9 @@
         // Remove options with empty values
         $cleanedOptions = array();
         foreach ($options as $name=>$value) {
-            if ($value!==null)
-                $cleanedOptions[$name] = $value;
+            if ($value!==null) {
+                            $cleanedOptions[$name] = $value;
+            }
         }
         
         return $cleanedOptions;

lib/WebMarketingROI/OptimizelyPHP/Resource/v2/Project.php

Spacing +3 added lines, -3 removed lines

@@ -122,7 +122,7 @@ 
                 case 'last_modified': $this->setLastModified($value); break;
                 case 'socket_token': $this->setSocketToken($value); break;
                 default:
-                    throw new Exception('Unknown option: ' . $name);
+                    throw new Exception('Unknown option: '.$name);
             }
         }
     }
@@ -140,7 +140,7 @@ 
             'platform' => $this->getPlatform(),
             'sdks' => $this->getSdks(),
             'status' => $this->getStatus(),
-            'web_snippet' => $this->getWebSnippet()?$this->getWebSnippet()->toArray():null,
+            'web_snippet' => $this->getWebSnippet() ? $this->getWebSnippet()->toArray() : null,
             'created' => $this->getCreated(),
             'id' => $this->getId(),
             'is_classic' => $this->getIsClassic(),
@@ -151,7 +151,7 @@ 
         // Remove options with empty values
         $cleanedOptions = array();
         foreach ($options as $name=>$value) {
-            if ($value!==null)
+            if ($value !== null)
                 $cleanedOptions[$name] = $value;
         }
         

Braces +3 added lines, -2 removed lines

@@ -91,8 +91,9 @@
         // Remove options with empty values
         $cleanedOptions = array();
         foreach ($options as $name=>$value) {
-            if ($value!==null)
-                $cleanedOptions[$name] = $value;
+            if ($value!==null) {
+                            $cleanedOptions[$name] = $value;
+            }
         }
         
         return $cleanedOptions;

lib/WebMarketingROI/OptimizelyPHP/Resource/v2/ExperimentResults.php

Braces +3 added lines, -2 removed lines

@@ -91,8 +91,9 @@
         // Remove options with empty values
         $cleanedOptions = array();
         foreach ($options as $name=>$value) {
-            if ($value!==null)
-                $cleanedOptions[$name] = $value;
+            if ($value!==null) {
+                            $cleanedOptions[$name] = $value;
+            }
         }
         
         return $cleanedOptions;

lib/WebMarketingROI/OptimizelyPHP/Resource/v2/Experiment.php

Braces +3 added lines, -2 removed lines

@@ -91,8 +91,9 @@
         // Remove options with empty values
         $cleanedOptions = array();
         foreach ($options as $name=>$value) {
-            if ($value!==null)
-                $cleanedOptions[$name] = $value;
+            if ($value!==null) {
+                            $cleanedOptions[$name] = $value;
+            }
         }
         
         return $cleanedOptions;