solved warnings in linting

src/SplitIO/Component/Cache/CacheKeyTrait.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Cache;
 
 /**

src/SplitIO/Component/Cache/EventsCache.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Cache;
 
 use SplitIO\Component\Common\Context;
@@ -7,7 +8,7 @@
 
 class EventsCache
 {
-    const KEY_EVENTS_LIST = "SPLITIO.events";
+    private const KEY_EVENTS_LIST = "SPLITIO.events";
 
     /**
      * @var \SplitIO\Component\Cache\Pool
@@ -26,7 +27,7 @@ public function addEvent(EventQueueMessage $message)
     {
         $queueJSONmessage =  json_encode($message->toArray());
 
-        Context::getLogger()->debug("Adding event item into queue: ". $queueJSONmessage);
+        Context::getLogger()->debug("Adding event item into queue: " . $queueJSONmessage);
         return ($this->cache->rightPushInList(self::KEY_EVENTS_LIST, $queueJSONmessage) > 0);
     }
 }

src/SplitIO/Component/Cache/ImpressionCache.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Cache;
 
 use SplitIO\Component\Common\Context;
@@ -7,8 +8,8 @@
 
 class ImpressionCache
 {
-    const IMPRESSIONS_QUEUE_KEY = "SPLITIO.impressions";
-    const IMPRESSION_KEY_DEFAULT_TTL = 3600;
+    public const IMPRESSIONS_QUEUE_KEY = "SPLITIO.impressions";
+    public const IMPRESSION_KEY_DEFAULT_TTL = 3600;
 
     /**
      * @var \SplitIO\Component\Cache\Pool
@@ -43,7 +44,7 @@ function ($imp) use ($metadata) {
             $impressions
         );
 
-        Context::getLogger()->debug("Adding impressions into queue: ". implode(",", $toStore));
+        Context::getLogger()->debug("Adding impressions into queue: " . implode(",", $toStore));
         $count = $this->cache->rightPushInList(self::IMPRESSIONS_QUEUE_KEY, $toStore);
         if ($count == count($impressions)) {
             $this->cache->expireKey(self::IMPRESSIONS_QUEUE_KEY, self::IMPRESSION_KEY_DEFAULT_TTL);

src/SplitIO/Component/Cache/Pool.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Cache;
 
 use SplitIO\Component\Cache\Storage\Adapter\PRedis as PRedisAdapter;

src/SplitIO/Component/Cache/SegmentCache.php

@@ -1,11 +1,12 @@
 <?php
+
 namespace SplitIO\Component\Cache;
 
 class SegmentCache implements SegmentCacheInterface
 {
-    const KEY_SEGMENT_DATA = 'SPLITIO.segment.{segmentName}';
+    public const KEY_SEGMENT_DATA = 'SPLITIO.segment.{segmentName}';
 
-    const KEY_TILL_CACHED_ITEM = 'SPLITIO.segment.{segment_name}.till';
+    public const KEY_TILL_CACHED_ITEM = 'SPLITIO.segment.{segment_name}.till';
 
     /**
      * @var \SplitIO\Component\Cache\Pool

src/SplitIO/Component/Cache/SegmentCacheInterface.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Cache;
 
 interface SegmentCacheInterface

src/SplitIO/Component/Cache/SplitCache.php

@@ -1,15 +1,16 @@
 <?php
+
 namespace SplitIO\Component\Cache;
 
 class SplitCache implements SplitCacheInterface
 {
-    const KEY_TILL_CACHED_ITEM = 'SPLITIO.splits.till';
+    public const KEY_TILL_CACHED_ITEM = 'SPLITIO.splits.till';
 
-    const KEY_SPLIT_CACHED_ITEM = 'SPLITIO.split.{splitName}';
+    public const KEY_SPLIT_CACHED_ITEM = 'SPLITIO.split.{splitName}';
 
-    const KEY_TRAFFIC_TYPE_CACHED = 'SPLITIO.trafficType.{trafficTypeName}';
+    public const KEY_TRAFFIC_TYPE_CACHED = 'SPLITIO.trafficType.{trafficTypeName}';
 
-    const KEY_FLAG_SET_CACHED = 'SPLITIO.flagSet.{set}';
+    public const KEY_FLAG_SET_CACHED = 'SPLITIO.flagSet.{set}';
 
     /**
      * @var \SplitIO\Component\Cache\Pool

src/SplitIO/Component/Cache/SplitCacheInterface.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Cache;
 
 interface SplitCacheInterface

src/SplitIO/Component/Cache/Storage/Adapter/CacheStorageAdapterInterface.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Cache\Storage\Adapter;
 
 interface CacheStorageAdapterInterface

src/SplitIO/Component/Cache/Storage/Adapter/PRedis.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Cache\Storage\Adapter;
 
 use SplitIO\Component\Cache\Storage\Exception\AdapterException;
@@ -59,9 +60,11 @@ private function validateKeyHashTag($keyHashTag)
         if (!is_string($keyHashTag)) {
             return array('valid' => false, 'msg' => 'keyHashTag must be string.');
         }
-        if ((strlen($keyHashTag) < 3) || ($keyHashTag[0] != "{") ||
+        if (
+            (strlen($keyHashTag) < 3) || ($keyHashTag[0] != "{") ||
             (substr($keyHashTag, -1) != "}") || (substr_count($keyHashTag, "{") != 1) ||
-            (substr_count($keyHashTag, "}") != 1)) {
+            (substr_count($keyHashTag, "}") != 1)
+        ) {
             return array('valid' => false, 'msg' => 'keyHashTag is not valid.');
         }
         return array('valid' => true, 'msg' => '');
@@ -205,7 +208,7 @@ private static function normalizePrefix($prefix)
             if ($prefix[strlen($prefix) - 1] == '.') {
                 return $prefix;
             } else {
-                return $prefix.'.';
+                return $prefix . '.';
             }
         } else {
             return null;

src/SplitIO/Component/Cache/Storage/Adapter/SafeRedisWrapper.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Cache\Storage\Adapter;
 
 use SplitIO\Component\Common\Context;

src/SplitIO/Component/Cache/Storage/Exception/AdapterException.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Cache\Storage\Exception;
 
 class AdapterException extends \Exception

src/SplitIO/Component/Common/Context.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Common;
 
 use SplitIO\Component\Log\Logger;
@@ -10,11 +11,11 @@
  */
 class Context
 {
-    const SAME_SDK_KEY = "Factory Instantiation: You already have %s factory/factories with this SDK Key. "
+    private const SAME_SDK_KEY = "Factory Instantiation: You already have %s factory/factories with this SDK Key. "
         . "We recommend keeping only one instance of the factory at all times (Singleton pattern) and "
         . "reusing it throughout your application.";
 
-    const MULTIPLE_INSTANCES = "Factory Instantiation: You already have an instance of the Split factory. "
+    private const MULTIPLE_INSTANCES = "Factory Instantiation: You already have an instance of the Split factory. "
         . "Make sure you definitely want this additional instance. We recommend keeping only one instance of "
         . "the factory at all times (Singleton pattern) and reusing it throughout your application.";
 

src/SplitIO/Component/Common/Enum.php

@@ -1,8 +1,10 @@
 <?php
+
 /**
  * @link    http://github.com/myclabs/php-enum
  * @license http://www.opensource.org/licenses/mit-license.php MIT (see the LICENSE file)
  */
+
 namespace SplitIO\Component\Common;
 
 /**

src/SplitIO/Component/Common/ErrorHandler.php

@@ -1,11 +1,11 @@
 <?php
+
 namespace SplitIO\Component\Common;
 
 use ErrorException;
 
 class ErrorHandler
 {
-
     /**
      * Errors stack
      * @var array

src/SplitIO/Component/Initialization/LoggerFactory.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Initialization;
 
 use Psr\Log\LogLevel;

src/SplitIO/Component/Log/Handler/Echos.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Log\Handler;
 
 /**

src/SplitIO/Component/Log/Handler/LogHandlerInterface.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Log\Handler;
 
 /**

src/SplitIO/Component/Log/Handler/Stdout.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Log\Handler;
 
 /**

src/SplitIO/Component/Log/Handler/Syslog.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Log\Handler;
 
 use Psr\Log\LogLevel;

src/SplitIO/Component/Log/Handler/VoidHandler.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Log\Handler;
 
 /**

src/SplitIO/Component/Log/LogLevelEnum.php

@@ -1,16 +1,17 @@
 <?php
+
 namespace SplitIO\Component\Log;
 
 use SplitIO\Component\Common\Enum;
 
 class LogLevelEnum extends Enum
 {
-    const EMERGENCY = 'emergency';
-    const ALERT = 'alert';
-    const CRITICAL = 'critical';
-    const ERROR = 'error';
-    const WARNING = 'warning';
-    const NOTICE = 'notice';
-    const INFO = 'info';
-    const DEBUG = 'debug';
+    public const EMERGENCY = 'emergency';
+    public const ALERT = 'alert';
+    public const CRITICAL = 'critical';
+    public const ERROR = 'error';
+    public const WARNING = 'warning';
+    public const NOTICE = 'notice';
+    public const INFO = 'info';
+    public const DEBUG = 'debug';
 }

src/SplitIO/Component/Log/Logger.php

@@ -1,8 +1,9 @@
 <?php
+
 namespace SplitIO\Component\Log;
 
 use Psr\Log\LogLevel;
-use \SplitIO\Component\Log\Handler\LogHandlerInterface;
+use SplitIO\Component\Log\Handler\LogHandlerInterface;
 use SplitIO\Component\Log\Handler\VoidHandler;
 
 /**
@@ -15,7 +16,7 @@ class Logger extends LoggerTrait
     /**
      * @var null|LogHandlerInterface
      */
-    protected $handler=null;
+    protected $handler = null;
 
     /**
      * @var null

src/SplitIO/Component/Log/LoggerTrait.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Log;
 
 use Psr\Log\LogLevel;

src/SplitIO/Component/Log/PSR3LoggerAdapter.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Component\Log;
 
 use Psr\Log\LoggerInterface;

src/SplitIO/Component/Stats/Latency.php

@@ -1,9 +1,10 @@
 <?php
+
 namespace SplitIO\Component\Stats;
 
 class Latency
 {
-    const MAX_LATENCY = 7481828;
+    public const MAX_LATENCY = 7481828;
 
     private static $buckets = array(1000, 1500, 2250, 3375, 5063, 7594, 11391, 17086, 25629, 38443,
         57665, 86498, 129746, 194620, 291929, 437894, 656841, 985261, 1477892, 2216838,

src/SplitIO/Engine.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO;
 
 use SplitIO\Grammar\Split as SplitGrammar;
@@ -8,9 +9,9 @@
 
 class Engine
 {
-    const EVALUATION_RESULT_TREATMENT = 'treatment';
+    public const EVALUATION_RESULT_TREATMENT = 'treatment';
 
-    const EVALUATION_RESULT_LABEL = 'label';
+    public const EVALUATION_RESULT_LABEL = 'label';
 
     /**
      * @param $matchingKey

src/SplitIO/Engine/Hash/HashAlgorithmEnum.php

@@ -1,10 +1,11 @@
 <?php
+
 namespace SplitIO\Engine\Hash;
 
 use SplitIO\Component\Common\Enum;
 
 class HashAlgorithmEnum extends Enum
 {
-    const LEGACY = 1;
-    const MURMUR = 2;
+    public const LEGACY = 1;
+    public const MURMUR = 2;
 }

src/SplitIO/Engine/Hash/HashInterface.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace SplitIO\Engine\Hash;
 
 interface HashInterface