merged with latest in master

Author: mmelograno

.github/workflows/ci.yml

@@ -19,10 +19,10 @@ jobs:
           - 6379:6379
     strategy:
       matrix:
-        version: ['7.4', '8.0', '8.2']
+        version: ['7.4', '8.0', '8.1', '8.2']
     steps:
       - name: Checkout code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
 
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
@@ -48,7 +48,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
         with:
           fetch-depth: 0
 

.github/workflows/update-license-year.yml

@@ -13,7 +13,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
         with:
           fetch-depth: 0
 

CHANGES.txt

@@ -1,7 +1,10 @@
-8.0.0 (Jan XX, 2023)
+8.0.0 (Feb XX, 2023)
  - BREAKING CHANGE: Removed support from versions older than PHP 7.4.
  - BREAKING CHANGE: Added support for Multiple Factory Instantiation.
 
+7.1.6 (Feb 14, 2023)
+ - Fixed logging on array conversion.
+
 7.1.5 (Oct 28, 2022)
  - Updated phpdocs for `ClientInterface`.
 

src/SplitIO/Component/Cache/SplitCache.php

@@ -68,7 +68,7 @@ public function getSplit($splitName)
      */
     public function getSplits($splitNames)
     {
-        $cacheItems = $this->cache->fetchMany(array_map('self::getCacheKeyForSplit', $splitNames));
+        $cacheItems = $this->cache->fetchMany(array_map(self::class . '::getCacheKeyForSplit', $splitNames));
         $toReturn = array();
         foreach ($cacheItems as $key => $value) {
             $toReturn[self::getSplitNameFromCacheKey($key)] = $value;

src/SplitIO/Component/Log/Logger.php

@@ -59,6 +59,9 @@ public function __construct(LogHandlerInterface $handler = null, $level = LogLev
     public function log($level, $message, array $context = array())
     {
         if ($this->logLevels[$level] <= $this->logLevel) {
+            if (is_array($message)) {
+                $message = json_encode($message);
+            }
             $this->handler->write($level, $message);
         }
     }

src/SplitIO/Grammar/Condition/Matcher/ContainsString.php

@@ -7,6 +7,8 @@ class ContainsString extends AbstractMatcher
 {
     protected $ContainsStringMatcherData = null;
 
+    private $containsStringMatcherData;
+
     public function __construct($data, $negate = false, $attribute = null)
     {
         parent::__construct(Matcher::CONTAINS_STRING, $negate, $attribute);

src/SplitIO/Sdk/Client.php

@@ -16,6 +16,7 @@ class Client implements ClientInterface
 
     private $evaluator = null;
     private $impressionListener = null;
+    private $queueMetadata = null;
 
     /**
      * Flag to get Impression's labels feature enabled
@@ -39,13 +40,13 @@ class Client implements ClientInterface
     private $impressionCache;
 
     /**
-     * @var \SplitIO\Component\Cache\EventCache
+     * @var \SplitIO\Component\Cache\EventsCache
      */
     private $eventCache;
 
     /**
-     * @param array $options
      * @param array $storages
+     * @param array $options
      */
     public function __construct($storages, $options = array())
     {

src/SplitIO/Sdk/Events/EventDTO.php

@@ -10,6 +10,7 @@ class EventDTO
     private $eventTypeId;
     private $value;
     private $timestamp;
+    private $properties;
 
     public function __construct($key, $trafficTypeName, $eventTypeId, $value, $properties)
     {

src/SplitIO/Sdk/Factory/SplitFactory.php

@@ -37,6 +37,7 @@ class SplitFactory implements SplitFactoryInterface
 
     /**
      * @param string $apiKey
+     * @param Pool $cache
      * @param array $options
      */
     public function __construct($apiKey, Pool $cache, array $options = array())

src/SplitIO/Version.php

@@ -3,5 +3,5 @@
 
 class Version
 {
-    const CURRENT = '8.0.0-rc1';
+    const CURRENT = '8.0.0-rc2';
 }

tests/Suite/Sdk/SdkReadOnlyTest.php

@@ -83,7 +83,7 @@ public function testException()
             ->getMock();
 
         // Discard (ignore) first logging statement
-        $logger->expects($this->exactly(2))
+        $logger->expects($this->exactly(1))
             ->method('debug');
 
         $logger->expects($this->exactly(2))