webmarketingroi / optimizely-php

lib/WebMarketingROI/OptimizelyPHP/Service/v2/Pages.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) {
+        if ($perPage < 0) {
             throw new Exception('Invalid page size passed');
         }
         

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

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
                 case 'unit': $this->setUnit($value); break;
                 case 'variation_results': $this->setVariationResults($value); break;
                 default:
-                    throw new Exception('Unknown option: ' . $name);
+                    throw new Exception('Unknown option: '.$name);
             }
         }
     }

lib/WebMarketingROI/OptimizelyPHP/Exception.php

Braces +15 added lines, -10 removed lines

@@ -55,20 +55,25 @@
     {
         parent::__construct($message, $code);
         
-        if (isset($options['http_code']))
-            $this->setHttpCode($options['http_code']);
+        if (isset($options['http_code'])) {
+                    $this->setHttpCode($options['http_code']);
+        }
                 
-        if (isset($options['uuid']))
-            $this->setUuid($options['uuid']);
+        if (isset($options['uuid'])) {
+                    $this->setUuid($options['uuid']);
+        }
         
-        if (isset($options['rate_limit']))
-            $this->setRateLimit($options['rate_limit']);
+        if (isset($options['rate_limit'])) {
+                    $this->setRateLimit($options['rate_limit']);
+        }
         
-        if (isset($options['rate_limit_remaining']))
-            $this->setRateLimitRemaining($options['rate_limit_remaining']);
+        if (isset($options['rate_limit_remaining'])) {
+                    $this->setRateLimitRemaining($options['rate_limit_remaining']);
+        }
         
-        if (isset($options['rate_limit_reset']))
-            $this->setRateLimitReset($options['rate_limit_reset']);
+        if (isset($options['rate_limit_reset'])) {
+                    $this->setRateLimitReset($options['rate_limit_reset']);
+        }
     }
     
     /**

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()
     {

examples/Projects/CreateProject.php

Spacing +3 added lines, -3 removed lines

@@ -17,10 +17,10 @@ 
 use WebMarketingROI\OptimizelyPHP\Resource\v2\Project;
 
 // Init class autloading.
-include dirname(__FILE__) . '/../../vendor/autoload.php';
+include dirname(__FILE__).'/../../vendor/autoload.php';
 
 // Include Utils.php - a file containing helper functions
-include dirname(__FILE__) . '/../Utils.php';
+include dirname(__FILE__).'/../Utils.php';
 
 // Get OAuth 2.0 credentials from auth_credentials.json and access_token.json files.
 $credentials = load_credentials_from_file();
@@ -117,6 +117,6 @@ 
 
 // Save access token for later use
 $accessToken = $optimizelyClient->getAccessToken();
-file_put_contents(dirname(__FILE__) . '/../access_token.json', json_encode($accessToken));
+file_put_contents(dirname(__FILE__).'/../access_token.json', json_encode($accessToken));
 
 echo "Done!\n";
\ No newline at end of file

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/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/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");
         }