Merge branch 'master' into develop

Author: Israphel

CHANGES.txt

@@ -1,3 +1,9 @@
+7.1.6 (Feb 14, 2023)
+ - Fixed logging on array conversion.
+
+7.1.5 (Oct 28, 2022)
+ - Updated phpdocs for `ClientInterface`.
+
 7.1.4 (Sep 6, 2022)
  - Updated dependencies to allow `psr/log` 2 and 3.
  - Removed `phpdocumentor/phpdocumentor` dependency.

README.md

@@ -1,6 +1,8 @@
 # Split SDK for PHP
 
-[![Build Status](https://api.travis-ci.com/splitio/php-client.svg?branch=master)](https://api.travis-ci.com/splitio/php-client)
+[![build status](https://github.com/splitio/php-client/actions/workflows/ci.yml/badge.svg)](https://github.com/splitio/php-client/actions)
+[![Latest stable](https://img.shields.io/packagist/v/splitsoftware/split-sdk-php)](https://packagist.org/packages/splitsoftware/split-sdk-php)
+[![Documentation](https://img.shields.io/badge/php_client-documentation-informational)](https://help.split.io/hc/en-us/articles/360020350372-PHP-SDK)
 
 This SDK is designed to work with Split, the platform for controlled rollouts, serving features to your users via the Split feature flag to manage your complete customer experience.
 

src/SplitIO/Component/Log/Logger.php

@@ -60,6 +60,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/Sdk/Client.php

@@ -19,6 +19,7 @@ class Client implements ClientInterface
 
     private $evaluator = null;
     private $impressionListener = null;
+    private $queueMetadata = null;
 
     /**
      * Flag to get Impression's labels feature enabled

src/SplitIO/Sdk/ClientInterface.php

@@ -29,7 +29,7 @@ interface ClientInterface
      *
      * <p>
      * This method does not throw any exceptions.
-     * It also never  returns null.
+     * It also never returns null.
      *
      * @param $key
      * @param $featureName
@@ -74,7 +74,7 @@ public function getTreatment($key, $featureName, array $attributes = null);
      * @param $key
      * @param $featureName
      * @param $attributes
-     * @return string
+     * @return array
      */
     public function getTreatmentWithConfig($key, $featureName, array $attributes = null);
 

src/SplitIO/Sdk/LocalhostClient.php

@@ -191,18 +191,18 @@ public function getTreatmentWithConfig($key, $featureName, array $attributes = n
      */
     public function getTreatments($key, $featureNames, array $attributes = null)
     {
+        $result = array();
+
         $splitNames = InputValidator::validateFeatureNames($featureNames, "getTreatments");
         if (is_null($splitNames)) {
-            return null;
+            return $result;
         }
 
         $key = InputValidator::validateKey($key, "getTreatments");
         if (is_null($key)) {
             return array_fill_keys($splitNames, TreatmentEnum::CONTROL);
         }
 
-        $result = array();
-
         foreach ($splitNames as $split) {
             $result[$split] = $this->getTreatment($key["matchingKey"], $split, $attributes);
         };
@@ -215,18 +215,18 @@ public function getTreatments($key, $featureNames, array $attributes = null)
      */
     public function getTreatmentsWithConfig($key, $featureNames, array $attributes = null)
     {
+        $result = array();
+
         $splitNames = InputValidator::validateFeatureNames($featureNames, "getTreatmentsWithConfig");
         if (is_null($splitNames)) {
-            return null;
+            return $result;
         }
 
         $key = InputValidator::validateKey($key, "getTreatmentsWithConfig");
         if (is_null($key)) {
             return array_fill_keys($splitNames, array('treatment' => TreatmentEnum::CONTROL, 'config' => null));
         }
 
-        $result = array();
-
         foreach ($splitNames as $split) {
             $result[$split] = $this->getTreatmentWithConfig($key["matchingKey"], $split, $attributes);
         };

src/SplitIO/TreatmentImpression.php

@@ -15,8 +15,8 @@ class TreatmentImpression
     public static function log($impressions, QueueMetadataMessage $metadata)
     {
         try {
-            Di::getLogger()->debug($impressions);
             if (is_null($impressions) || (is_array($impressions) && 0 == count($impressions))) {
+                Di::getLogger()->debug("no impressions were sent");
                 return null;
             }
             $impressionCache = new ImpressionCache();

src/SplitIO/Version.php

@@ -3,5 +3,5 @@
 
 class Version
 {
-    const CURRENT = '7.1.4';
+    const CURRENT = '7.1.6';
 }

tests/Suite/InputValidation/GetTreatmentValidationTest.php

@@ -366,6 +366,23 @@ public function testGetTreatmentWithConfigWithNotExistantSplitName()
         $this->assertEquals('control', $result['treatment']);
     }
 
+    public function testGetTreatmentWitConfigWithhNullFeatureName()
+    {
+        $splitSdk = $this->getFactoryClient();
+
+        $logger = $this->getMockedLogger();
+
+        $logger->expects($this->once())
+            ->method('critical')
+            ->with($this->equalTo("getTreatmentWithConfig: you passed a null split name, split name must be a non-empty"
+                . " string."));
+
+        $result = $splitSdk->getTreatmentWithConfig('some_key', null);
+        $this->assertEquals('control', $result['treatment']);
+        $this->assertEquals(null, $result['config']);
+
+    }
+
     public static function tearDownAfterClass(): void
     {
         Utils\Utils::cleanCache();

tests/Suite/InputValidation/GetTreatmentsValidationTest.php

@@ -407,6 +407,34 @@ public function testGetTreatmenstConfigWithoutExistingFeatureName()
         $this->assertEquals('control', $treatmentResult['some_feature_non_existant']['treatment']);
     }
 
+    public function testGetTreatmenstConfigWithNullFeatures()
+    {
+        $splitSdk = $this->getFactoryClient();
+
+        $logger = $this->getMockedLogger();
+
+        $logger->expects($this->once())
+            ->method('critical')
+            ->with($this->equalTo('getTreatmentsWithConfig: featureNames must be a non-empty array.'));
+
+        $treatmentResult = $splitSdk->getTreatmentsWithConfig("some_key", null);
+        $this->assertEquals(array(), $treatmentResult);
+    }
+
+    public function testGetTreatmenstConfigWithEmptyFeatures()
+    {
+        $splitSdk = $this->getFactoryClient();
+
+        $logger = $this->getMockedLogger();
+
+        $logger->expects($this->once())
+            ->method('critical')
+            ->with($this->equalTo('getTreatmentsWithConfig: featureNames must be a non-empty array.'));
+
+        $treatmentResult = $splitSdk->getTreatmentsWithConfig("some_key", array());
+        $this->assertEquals(array(), $treatmentResult);
+    }
+
     public static function tearDownAfterClass(): void
     {
         Utils\Utils::cleanCache();