ci: Enforce Static Code Analyzer (#155)

Author: oakbani

.travis.yml

@@ -6,29 +6,34 @@ php:
   - '7.1'
   - '7.2'
   - '7.3'
-before_install:
-- composer install --dev
+install: "composer install"
 addons:
   srcclr: true
 script:
 - mkdir -p build/logs
 - ./vendor/bin/phpunit --coverage-clover build/logs/clover.xml
-after_script:
-- vendor/bin/php-coveralls -v
+after_script: "vendor/bin/php-coveralls -v"
 
-# Integration tests need to run first to reset the PR build status to pending
+# Linting and integration tests need to run first to reset the PR build status to pending
 stages:
+  - 'Linting'
   - 'Integration tests'
   - 'Test'
 
 jobs:
   include:
+    - stage: 'Linting'
+      language: php
+      php: '7.0'
+      install: 'composer require "squizlabs/php_codesniffer=*"'
+      script: 'composer lint'
+      after_script: skip
+      after_success: travis_terminate 0
     - stage: 'Integration tests'
       merge_mode: replace
       env: SDK=php
       cache: false
       language: python
-      before_install: skip
       install:
         - "pip install awscli"
       before_script:

CONTRIBUTING.md

@@ -17,6 +17,11 @@ We welcome contributions and feedback! All contributors must sign our [Contribut
 
 * **All code must have test coverage.** We use PHPUnit. Changes in functionality should have accompanying unit tests. Bug fixes should have accompanying regression tests.
 * Tests are located in `tests` with one file per class.
+* Lint your code with PHP CodeSniffer before submitting.
+
+## Style
+We enforce [PSR-2](https://www.php-fig.org/psr/psr-2/) rules with some minor [deviations](phpcs.xml). Run linter by executing `composer lint` and autocorrect lint errors by executing `composer beautify`.
+
 
 ## License
 

composer.json

@@ -9,6 +9,11 @@
       "email": "developers@optimizely.com"
     }
   ],
+  "scripts": {
+        "test": "phpunit",
+        "lint": "phpcs",
+        "beautify": "phpcbf"
+  },
   "require": {
     "php": ">=5.5",
     "justinrainbow/json-schema": "^1.6 || ^2.0 || ^4.0 || ^5.0",
@@ -19,7 +24,8 @@
   },
   "require-dev": {
     "phpunit/phpunit": "^4.8|^5.0",
-    "php-coveralls/php-coveralls": "v2.0.0"
+    "php-coveralls/php-coveralls": "v2.0.0",
+    "squizlabs/php_codesniffer": "3.*"
   },
   "autoload": {
     "psr-4": {

phpcs.xml

@@ -0,0 +1,23 @@
+<?xml version="1.0"?>
+<ruleset name="OptimizelyPSR2">
+  <!-- description for custom ruleset -->
+  <description>A custom coding standard for Optimizely PHP-SDK based on PSR2</description>
+
+  <!-- If no files or directories are specified on commandline, these files will be sniffed -->
+  <file>./src</file>
+  <file>./tests</file>
+
+  <!-- Exclude patterns for files to be excluded from sniffing -->
+  <!-- <exclude-pattern>./tests/EventTests/*</exclude-pattern> -->
+
+  <!-- Embed command line arguments in config file. -->
+  <arg name="tab-width" value="4"/>
+
+  <!-- To exclude any rule sniff, get sniff name by running phpcs with -s switch -->
+  <rule ref="PSR2">
+    <exclude name="Generic.Files.LineLength.TooLong"/>
+    <exclude name="PSR2.Classes.PropertyDeclaration.Underscore"/>
+    <exclude name="PSR1.Classes.ClassDeclaration.MultipleClasses"/>
+    <exclude name="PSR1.Files.SideEffects.FoundWithSymbols"/>
+  </rule >
+</ruleset>

src/Optimizely/Bucketer.php

@@ -1,6 +1,6 @@
 <?php
 /**
- * Copyright 2016-2018, Optimizely
+ * Copyright 2016-2019, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -90,9 +90,9 @@ protected function generateBucketValue($bucketingKey)
 
         /* murmurhash3_int returns both positive and negative integers for PHP x86 versions
         it returns negative integers when it tries to create 2^32 integers while PHP doesn't support
-        unsigned integers and can store integers only upto 2^31. 
+        unsigned integers and can store integers only upto 2^31.
         Observing generated hashcodes and their corresponding bucket values after normalization
-        indicates that a negative bucket number on x86 is exactly 10,000 less than it's 
+        indicates that a negative bucket number on x86 is exactly 10,000 less than it's
         corresponding bucket number on x64. Hence we can safely add 10,000 to a negative number to
         make it consistent across both of the PHP variants. */
 

src/Optimizely/DecisionService/DecisionService.php

@@ -1,6 +1,6 @@
 <?php
 /**
- * Copyright 2017-2018, Optimizely Inc and Contributors
+ * Copyright 2017-2019, Optimizely Inc and Contributors
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -93,8 +93,8 @@ protected function getBucketingId($userId, $userAttributes)
     {
         $bucketingIdKey = ControlAttributes::BUCKETING_ID;
 
-        if (isset($userAttributes[$bucketingIdKey])){
-            if (is_string($userAttributes[$bucketingIdKey])){
+        if (isset($userAttributes[$bucketingIdKey])) {
+            if (is_string($userAttributes[$bucketingIdKey])) {
                 return $userAttributes[$bucketingIdKey];
             }
             $this->_logger->log(Logger::WARNING, 'Bucketing ID attribute is not a string. Defaulted to user ID.');
@@ -308,7 +308,7 @@ public function getVariationForFeatureRollout(FeatureFlag $featureFlag, $userId,
             break;
         }
         // Evaluate Everyone Else Rule / Last Rule now
-        $experiment = $rolloutRules[sizeof($rolloutRules)-1];        
+        $experiment = $rolloutRules[sizeof($rolloutRules)-1];
 
         // Evaluate if user meets the audience condition of Everyone Else Rule / Last Rule now
         if (!Validator::isUserInExperiment($this->_projectConfig, $experiment, $userAttributes)) {

src/Optimizely/Entity/Experiment.php

@@ -1,6 +1,6 @@
 <?php
 /**
- * Copyright 2016, 2018, Optimizely
+ * Copyright 2016, 2018-2019, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -78,7 +78,7 @@ class Experiment
 
     /**
      * @var array/string Single audience ID the experiment is attached to or
-     *                   hierarchical conditions array of audience IDs related by AND/OR/NOT operators. 
+     *                   hierarchical conditions array of audience IDs related by AND/OR/NOT operators.
      */
     private $_audienceConditions;
 

src/Optimizely/Entity/FeatureVariable.php

@@ -1,6 +1,6 @@
 <?php
 /**
- * Copyright 2017, Optimizely
+ * Copyright 2017, 2019, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -61,7 +61,7 @@ class FeatureVariable
     private $_defaultValue;
 
 
-    public function __construct($id =null, $key = null, $type = null, $defaultValue = null)
+    public function __construct($id = null, $key = null, $type = null, $defaultValue = null)
     {
         $this->_id = $id;
         $this->_key = $key;

src/Optimizely/Entity/Variation.php

@@ -1,6 +1,6 @@
 <?php
 /**
- * Copyright 2016-2018, Optimizely
+ * Copyright 2016-2019, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -115,7 +115,7 @@ public function getVariables()
     }
 
     /**
-     * @param  string Variable ID
+     * @param string Variable ID
      *
      * @return VariableUsage Variable usage instance corresponding to given variable ID
      */

src/Optimizely/Event/Builder/EventBuilder.php

@@ -100,7 +100,7 @@ private function getCommonParams($config, $userId, $attributes)
             ENRICH_DECISIONS => true
         ];
 
-        if(!is_null($attributes)) {
+        if (!is_null($attributes)) {
             foreach ($attributes as $attributeKey => $attributeValue) {
                 // Omit attributes that are not supported by the log endpoint.
                 if (Validator::isAttributeValid($attributeKey, $attributeValue)) {
@@ -199,7 +199,7 @@ private function getConversionParams($eventEntity, $eventTags)
                 $eventDict[EventTagUtils::NUMERIC_EVENT_METRIC_NAME] = $eventValue;
             }
 
-            if(count($eventTags) > 0) {
+            if (count($eventTags) > 0) {
                 $eventDict['tags'] = $eventTags;
             }
         }