webmarketingroi / optimizely-php

lib/WebMarketingROI/OptimizelyPHP/Exception.php

Doc Comments +1 added lines, -3 removed lines

@@ -82,7 +82,6 @@ 
             
     /**
      * Set HTTP response code.
-     * @param integer $code
      */
     public function setHttpCode($httpCode)
     {
@@ -100,7 +99,6 @@ 
             
     /**
      * Set message
-     * @param string $message
      */
     public function setUuid($uuid)
     {
@@ -145,7 +143,7 @@ 
     
     /**
      * Return the exact time that your fresh new rate limit kicks in.
-     * @return integer|null
+     * @return string
      */
     public function getRateLimitReset()
     {

lib/WebMarketingROI/OptimizelyPHP/Result.php

Doc Comments +2 added lines, -5 removed lines

@@ -70,8 +70,6 @@ 
     /**
      * Constructor.
      * @param array $decodedJsonData
-     * @param string $rawHttpResponseData
-     * @param array[string] $rawHttpResponseHeaders
      */
     public function __construct($decodedJsonData, $httpCode)
     {
@@ -90,7 +88,6 @@ 
             
     /**
      * Set HTTP response code.
-     * @param integer $code
      */
     public function setHttpCode($httpCode)
     {
@@ -108,7 +105,7 @@ 
     
     /**
      * Set payload (parsed and wrapped response data).
-     * @param mixed $payload
+     * @param integer $payload
      */
     public function setPayload($payload)
     {
@@ -225,7 +222,7 @@ 
     
     /**
      * Return the exact time that your fresh new rate limit kicks in.
-     * @return integer|null
+     * @return string
      */
     public function getRateLimitReset()
     {

tests/OptimizelyPHPTest/Service/v2/ProjectsTest.php

Indentation +36 added lines, -36 removed lines

@@ -176,16 +176,16 @@ 
                             "platform" => "web",
                             "status" => "active",
                             "web_snippet" => array(
-                              "enable_force_variation" => false,
-                              "exclude_disabled_experiments" => false,
-                              "exclude_names" => true,
-                              "include_jquery" => true,
-                              "ip_anonymization" => false,
-                              "ip_filter" => "^206\\.23\\.100\\.([5-9][0-9]|1([0-4][0-9]|50))$",
-                              "library" => "jquery-1.11.3-trim",
-                              "project_javascript" => "alert(\"Active Experiment\")",
-                              "code_revision" => 0,
-                              "js_file_size" => 63495
+                                "enable_force_variation" => false,
+                                "exclude_disabled_experiments" => false,
+                                "exclude_names" => true,
+                                "include_jquery" => true,
+                                "ip_anonymization" => false,
+                                "ip_filter" => "^206\\.23\\.100\\.([5-9][0-9]|1([0-4][0-9]|50))$",
+                                "library" => "jquery-1.11.3-trim",
+                                "project_javascript" => "alert(\"Active Experiment\")",
+                                "code_revision" => 0,
+                                "js_file_size" => 63495
                             ),
                             "created" => "2016-10-17T07:04:59.991Z",
                             "id" => 1000,
@@ -206,14 +206,14 @@ 
             "platform" => "web",
             "status" => "active",
             "web_snippet" => array(
-              "enable_force_variation" => false,
-              "exclude_disabled_experiments" => false,
-              "exclude_names" => true,
-              "include_jquery" => true,
-              "ip_anonymization" => false,
-              "ip_filter" => "^206\\.23\\.100\\.([5-9][0-9]|1([0-4][0-9]|50))$",
-              "library" => "jquery-1.11.3-trim",
-              "project_javascript" => "alert(\"Active Experiment\")"
+                "enable_force_variation" => false,
+                "exclude_disabled_experiments" => false,
+                "exclude_names" => true,
+                "include_jquery" => true,
+                "ip_anonymization" => false,
+                "ip_filter" => "^206\\.23\\.100\\.([5-9][0-9]|1([0-4][0-9]|50))$",
+                "library" => "jquery-1.11.3-trim",
+                "project_javascript" => "alert(\"Active Experiment\")"
             )
         ));
         
@@ -257,16 +257,16 @@ 
                             "platform" => "web",
                             "status" => "active",
                             "web_snippet" => array(
-                              "enable_force_variation" => false,
-                              "exclude_disabled_experiments" => false,
-                              "exclude_names" => true,
-                              "include_jquery" => true,
-                              "ip_anonymization" => false,
-                              "ip_filter" => "^206\\.23\\.100\\.([5-9][0-9]|1([0-4][0-9]|50))$",
-                              "library" => "jquery-1.11.3-trim",
-                              "project_javascript" => "alert(\"Active Experiment\")",
-                              "code_revision" => 0,
-                              "js_file_size" => 63495
+                                "enable_force_variation" => false,
+                                "exclude_disabled_experiments" => false,
+                                "exclude_names" => true,
+                                "include_jquery" => true,
+                                "ip_anonymization" => false,
+                                "ip_filter" => "^206\\.23\\.100\\.([5-9][0-9]|1([0-4][0-9]|50))$",
+                                "library" => "jquery-1.11.3-trim",
+                                "project_javascript" => "alert(\"Active Experiment\")",
+                                "code_revision" => 0,
+                                "js_file_size" => 63495
                             ),
                             "created" => "2016-10-17T07:04:59.999Z",
                             "id" => 1000,
@@ -286,14 +286,14 @@ 
             "name" => "Test Project",
             "status" => "active",
             "web_snippet" => array(
-              "enable_force_variation" => false,
-              "exclude_disabled_experiments" => false,
-              "exclude_names" => true,
-              "include_jquery" => true,
-              "ip_anonymization" => false,
-              "ip_filter" => "^206\\.23\\.100\\.([5-9][0-9]|1([0-4][0-9]|50))$",
-              "library" => "jquery-1.11.3-trim",
-              "project_javascript" => "alert(\"Active Experiment\")"
+                "enable_force_variation" => false,
+                "exclude_disabled_experiments" => false,
+                "exclude_names" => true,
+                "include_jquery" => true,
+                "ip_anonymization" => false,
+                "ip_filter" => "^206\\.23\\.100\\.([5-9][0-9]|1([0-4][0-9]|50))$",
+                "library" => "jquery-1.11.3-trim",
+                "project_javascript" => "alert(\"Active Experiment\")"
             )
         ));
         

Spacing +16 added lines, -16 removed lines

@@ -51,9 +51,9 @@ 
         $result = $projectsService->listAll();
         $projects = $result->getPayload();
         
-        $this->assertTrue(count($projects)==1);
+        $this->assertTrue(count($projects) == 1);
         $this->assertTrue($projects[0] instanceOf Project);
-        $this->assertTrue($projects[0]->getName()=='Some Optimizely Project');        
+        $this->assertTrue($projects[0]->getName() == 'Some Optimizely Project');        
     }
     
     /**
@@ -126,7 +126,7 @@ 
         $project = $result->getPayload();
         
         $this->assertTrue($project instanceOf Project);
-        $this->assertTrue($project->getName()=='Some Optimizely Project');        
+        $this->assertTrue($project->getName() == 'Some Optimizely Project');        
     }
     
     /**
@@ -221,8 +221,8 @@ 
         $createdProject = $result->getPayload();
         
         $this->assertTrue($createdProject instanceOf Project);
-        $this->assertTrue($createdProject->getName()=='Test Project');        
-        $this->assertTrue($createdProject->getAccountId()==12345);        
+        $this->assertTrue($createdProject->getName() == 'Test Project');        
+        $this->assertTrue($createdProject->getAccountId() == 12345);        
     }
     
     /**
@@ -301,8 +301,8 @@ 
         $updatedProject = $result->getPayload();
         
         $this->assertTrue($updatedProject instanceOf Project);
-        $this->assertTrue($updatedProject->getName()=='Test Project');        
-        $this->assertTrue($updatedProject->getAccountId()==12345);        
+        $this->assertTrue($updatedProject->getName() == 'Test Project');        
+        $this->assertTrue($updatedProject->getAccountId() == 12345);        
     }
     
     /**
@@ -362,7 +362,7 @@ 
         $credentials = $this->loadCredentialsFromFile();
         
         $optimizelyClient = new OptimizelyApiClient($credentials, 'v2');
-        $this->assertTrue($optimizelyClient!=null);
+        $this->assertTrue($optimizelyClient != null);
         
         $curDate = date('Y-m-d H:i:s');
         /*// Create new project        
@@ -408,17 +408,17 @@ 
                 $projects = $result->getPayload();
 
                 foreach ($projects as $project) {
-                    if ($project->getName()=="Test Project $curDate") {
+                    if ($project->getName() == "Test Project $curDate") {
                         $projectId = $project->getId();
                         $found = true;
                         break;
                     }
                 }
 
-                if ($result->getNextPage()==null)
+                if ($result->getNextPage() == null)
                     break;
 
-                $page ++;
+                $page++;
             }
         }
         catch (Exception $e) {
@@ -437,13 +437,13 @@ 
         try {
             $result = $optimizelyClient->projects()->get(12345678);        
         } 
-        catch(Exception $e) {
+        catch (Exception $e) {
             $this->assertEquals(Exception::CODE_API_ERROR, $e->getCode());
             $this->assertEquals(400, $e->getHttpCode());
-            $this->assertTrue(strlen($e->getUuid())!=0);
-            $this->assertTrue($e->getRateLimit()>0);
-            $this->assertTrue($e->getRateLimitRemaining()>0);
-            $this->assertTrue($e->getRateLimitReset()>0);
+            $this->assertTrue(strlen($e->getUuid()) != 0);
+            $this->assertTrue($e->getRateLimit() > 0);
+            $this->assertTrue($e->getRateLimitRemaining() > 0);
+            $this->assertTrue($e->getRateLimitReset() > 0);
         }
         
         // Retrieve existing project by ID        

Braces +8 added lines, -8 removed lines

@@ -356,8 +356,9 @@ 
     
     public function testIntegration()
     {
-        if (!getenv('OPTIMIZELY_PHP_TEST_INTEGRATION')) 
-            $this->markTestSkipped('OPTIMIZELY_PHP_TEST_INTEGRATION env var is not set');
+        if (!getenv('OPTIMIZELY_PHP_TEST_INTEGRATION')) {
+                    $this->markTestSkipped('OPTIMIZELY_PHP_TEST_INTEGRATION env var is not set');
+        }
         
         $credentials = $this->loadCredentialsFromFile();
         
@@ -415,13 +416,13 @@ 
                     }
                 }
 
-                if ($result->getNextPage()==null)
-                    break;
+                if ($result->getNextPage()==null) {
+                                    break;
+                }
 
                 $page ++;
             }
-        }
-        catch (Exception $e) {
+        } catch (Exception $e) {
             print_r($optimizelyClient->getDiagnosticsInfo());
             // Handle error.
             $code = $e->getCode();
@@ -436,8 +437,7 @@ 
         // Try a non-existing project by ID - assume failure
         try {
             $result = $optimizelyClient->projects()->get(12345678);        
-        } 
-        catch(Exception $e) {
+        } catch(Exception $e) {
             $this->assertEquals(Exception::CODE_API_ERROR, $e->getCode());
             $this->assertEquals(400, $e->getHttpCode());
             $this->assertTrue(strlen($e->getUuid())!=0);

tests/OptimizelyPHPTest/OptimizelyApiClientTest.php

Braces +1 added lines, -2 removed lines

@@ -163,8 +163,7 @@
         // refresh token, since the provided access token is invalid.
         try {
             $result = $client->sendApiRequest('/projects');        
-        }
-        catch (Exception $e) {
+        } catch (Exception $e) {
             $this->assertEquals(Exception::CODE_API_ERROR, $e->getCode());
             $this->assertEquals(400, $e->getHttpCode());            
         }

tests/OptimizelyPHPTest/ResultTest.php

Indentation +8 added lines, -8 removed lines

@@ -16,14 +16,14 @@
             "platform" => "web",
             "status" => "active",
             "web_snippet" => array(
-              "enable_force_variation" => false,
-              "exclude_disabled_experiments" => false,
-              "exclude_names" => true,
-              "include_jquery" => true,
-              "ip_anonymization" => false,
-              "ip_filter" => "^206\\.23\\.100\\.([5-9][0-9]|1([0-4][0-9]|50))$",
-              "library" => "jquery-1.11.3-trim",
-              "project_javascript" => "alert(\"Active Experiment\")"
+                "enable_force_variation" => false,
+                "exclude_disabled_experiments" => false,
+                "exclude_names" => true,
+                "include_jquery" => true,
+                "ip_anonymization" => false,
+                "ip_filter" => "^206\\.23\\.100\\.([5-9][0-9]|1([0-4][0-9]|50))$",
+                "library" => "jquery-1.11.3-trim",
+                "project_javascript" => "alert(\"Active Experiment\")"
             ));
         
         $result = new Result($data, 200);

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,41 +199,41 @@ 
         
         // 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);            
         
         // 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);
@@ -252,7 +252,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);
         }        
@@ -266,12 +266,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;
             }
         }
@@ -303,21 +303,21 @@ 
         
         // 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']) ||
                 $httpCode != !isset($decodedPayload['code'])) {
-                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,
@@ -349,7 +349,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);
             }
             
@@ -359,22 +359,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));
                 }
@@ -392,7 +392,7 @@ 
      */
     private function isAccessTokenExpired() 
     {
-        if(!isset($this->authCredentials['access_token'])) {
+        if (!isset($this->authCredentials['access_token'])) {
             return true; // We do not have access token.
         }
         
@@ -433,15 +433,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/Campaigns.php

Spacing +5 added lines, -5 removed lines

@@ -37,21 +37,21 @@ 
      * @return Result
      * @throws Exception
      */
-    public function listAll($projectId, $page=1, $perPage=25)
+    public function listAll($projectId, $page = 1, $perPage = 25)
     {
         if (!is_int($projectId)) {
             throw new Exception("Integer project ID expected, while got '$projectId'");
         }
         
-        if ($projectId<0) {
+        if ($projectId < 0) {
             throw new Exception("Expected positive integer project ID");
         }
         
-        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');
         }
         
@@ -107,7 +107,7 @@ 
                     Exception::CODE_INVALID_ARG);
         }
         
-        if ($campaignId<0) {
+        if ($campaignId < 0) {
             throw new Exception("Expected positive integer campaign ID",
                     Exception::CODE_INVALID_ARG);
         }

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

Spacing +3 added lines, -3 removed lines

@@ -36,13 +36,13 @@
      * @return Result
      * @throws Exception
      */
-    public function listAll($projectId, $page=1, $perPage=25)
+    public function listAll($projectId, $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');
         }
         

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

Spacing +6 added lines, -6 removed lines

@@ -39,17 +39,17 @@ 
      * @return Result
      * @throws Exception
      */
-    public function listAll($projectId, $campaignId=null, $includeClassic=false, $page=1, $perPage=25)
+    public function listAll($projectId, $campaignId = null, $includeClassic = false, $page = 1, $perPage = 25)
     {
-        if ($projectId==null && $campaignId==null) {
+        if ($projectId == null && $campaignId == null) {
             throw new Exception('Project ID or Campaign ID must be non-null');
         }
         
-        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');
         }
         
@@ -85,7 +85,7 @@ 
                     Exception::CODE_INVALID_ARG);
         }
         
-        if ($experimentId<0) {
+        if ($experimentId < 0) {
             throw new Exception("A positive experiment ID expected");
         }
         
@@ -175,7 +175,7 @@ 
             throw new Exception("Expected argument of type Experiment");
         }
         
-        if ($experimentId<0) {
+        if ($experimentId < 0) {
             throw new Exception("Expected positive experiment ID argument");
         }