updated di logger

Author: mmelograno

src/SplitIO/Component/Common/Di.php

@@ -1,29 +1,25 @@
 <?php
 namespace SplitIO\Component\Common;
 
-use Psr\Log\LoggerInterface;
-use SplitIO\Component\Cache\Pool;
+use SplitIO\Component\Log\Logger;
 
 /**
  * Class Di
  * @package SplitIO\Common
  */
 class Di
 {
-    const KEY_LOG = 'SPLIT-LOGGER';
+    private \SplitIO\Component\Log\Logger|null $logger = null;
 
-    const KEY_FACTORY_TRACKER = 'FACTORY-TRACKER';
+    private int $factoryTracker = 0;
+
+    private string|null $ipAddress = null;
 
     /**
      * @var Singleton The reference to *Singleton* instance of this class
      */
     private static $instance;
 
-    /**
-     * @var array
-     */
-    private $container = array();
-
     /**
      * Returns the *Singleton* instance of this class.
      *
@@ -66,58 +62,77 @@ public function __wakeup()
     {
     }
 
+    private function __trackFactory()
+    {
+        $this->factoryTracker += 1;
+        return $this->factoryTracker;
+    }
+
     /**
-     * @param $key
-     * @param $instance
+     * @param \SplitIO\Component\Log\Logger $logger
      */
-    private function setKey($key, $instance)
+    private function __setLogger(Logger $logger)
     {
-        $this->container[$key] = $instance;
+        if (is_null($this->logger)) {
+            $this->logger = $logger;
+        }
     }
 
     /**
-     * @param $key
-     * @return mixed
+     * @return null|\SplitIO\Component\Log\Logger
      */
-    private function getKey($key)
+    private function __getLogger()
     {
-        return (isset($this->container[$key])) ? $this->container[$key] : null;
+        return $this->logger;
     }
 
     /**
-     * Set an object instance with its key
-     * @param $key
-     * @param $instance
+     * @return null|string
      */
-    public static function set($key, $instance)
+    private function __getIPAddress()
     {
-        self::getInstance()->setKey($key, $instance);
+        return $this->ipAddress;
     }
 
     /**
-     * Given a key returns the object instance associated with this.
-     * @param $key
-     * @return mixed
+     * @return int
      */
-    public static function get($key)
+    public static function trackFactory()
     {
-        return self::getInstance()->getKey($key);
+        return self::getInstance()->__trackFactory();
     }
 
-
     /**
-     * @param LoggerInterface $logger
+     * @param \SplitIO\Component\Log\Logger $logger
      */
-    public static function setLogger($logger)
+    public static function setLogger(Logger $logger)
     {
-        self::set(self::KEY_LOG, $logger);
+        self::getInstance()->__setLogger($logger);
     }
 
     /**
-     * @return null|\Psr\Log\LoggerInterface
+     * @return null|\SplitIO\Component\Log\Logger
      */
     public static function getLogger()
     {
-        return self::get(self::KEY_LOG);
+        return self::getInstance()->__getLogger();
+    }
+
+    /**
+     * @param string $ip
+     */
+    public static function setIPAddress(string $ip)
+    {
+        if (is_null($this->ip)) {
+            $this->ip = $ip;
+        }
+    }
+
+    /**
+     * @return null|string
+     */
+    public static function getIPAddress()
+    {
+        return self::getInstance()->__getIPAddress();
     }
 }

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

@@ -1,7 +1,6 @@
 <?php
 namespace SplitIO\Grammar\Condition\Matcher;
 
-use SplitIO\Split as SplitApp;
 use SplitIO\Grammar\Condition\Matcher;
 
 class ContainsString extends AbstractMatcher

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

@@ -1,7 +1,6 @@
 <?php
 namespace SplitIO\Grammar\Condition\Matcher;
 
-use SplitIO\Split as SplitApp;
 use SplitIO\Grammar\Condition\Matcher;
 
 class EndsWith extends AbstractMatcher

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

@@ -1,7 +1,6 @@
 <?php
 namespace SplitIO\Grammar\Condition\Matcher;
 
-use SplitIO\Split as SplitApp;
 use SplitIO\Grammar\Condition\Matcher;
 
 class EqualToBoolean extends AbstractMatcher

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

@@ -1,7 +1,6 @@
 <?php
 namespace SplitIO\Grammar\Condition\Matcher;
 
-use SplitIO\Split as SplitApp;
 use SplitIO\Grammar\Condition\Matcher;
 
 class Regex extends AbstractMatcher

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

@@ -1,7 +1,6 @@
 <?php
 namespace SplitIO\Grammar\Condition\Matcher;
 
-use SplitIO\Split as SplitApp;
 use SplitIO\Grammar\Condition\Matcher;
 
 class StartsWith extends AbstractMatcher

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

@@ -1,7 +1,6 @@
 <?php
 namespace SplitIO\Grammar\Condition\Matcher;
 
-use SplitIO\Split as SplitApp;
 use SplitIO\Grammar\Condition\Matcher;
 
 class Whitelist extends AbstractMatcher

src/SplitIO/Sdk.php

@@ -31,9 +31,6 @@ public static function factory($apiKey = 'localhost', array $options = array())
         //Adding API Key into args array.
         $options['apiKey'] = $apiKey;
 
-        if (self::instanceExists()) {
-            return null;
-        }
         self::registerInstance();
 
         if ($apiKey == 'localhost') {
@@ -93,31 +90,18 @@ private static function registerCache(array $options)
 
     private static function setIP($ip)
     {
-        \SplitIO\Component\Common\Di::set('ipAddress', $ip);
-    }
-
-    /**
-     * Check factory instance
-     */
-    private static function instanceExists()
-    {
-        $value = Di::get(Di::KEY_FACTORY_TRACKER);
-        /* TODO MULTIPLE ALLOW
-        if (is_null($value) || !$value) {
-            return false;
-        }
-        Di::getLogger()->critical("Factory Instantiation: creating multiple factories is not possible. "
-            . "You have already created a factory.");
-        return true;
-        */
-        return false;
+        \SplitIO\Component\Common\Di::setIPAddress('ipAddress', $ip);
     }
 
     /**
      * Register factory instance
      */
     private static function registerInstance()
     {
-        Di::set(Di::KEY_FACTORY_TRACKER, true);
+        if (Di::trackFactory() > 1) {
+            Di::getLogger()->warning("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/functions.php

@@ -1,7 +1,6 @@
 <?php
 namespace SplitIO;
 
-use SplitIO\Split as SplitApp;
 use SplitIO\Grammar\Condition\Partition\TreatmentEnum;
 
 function version()
@@ -72,7 +71,7 @@ function parseSplitsFile($fileContent)
 
 function getHostIpAddress()
 {
-    $diIpAddress = \SplitIO\Component\Common\Di::get('ipAddress');
+    $diIpAddress = \SplitIO\Component\Common\Di::getIPAddress('ipAddress');
     if (!is_null($diIpAddress) && is_string($diIpAddress) && trim($diIpAddress)) {
         return $diIpAddress;
     } elseif (isset($_SERVER['SERVER_ADDR']) && is_string($_SERVER['SERVER_ADDR'])

tests/Suite/DynamicConfigurations/EvaluatorTest.php

@@ -4,7 +4,6 @@
 use SplitIO\Test\Suite\Redis\ReflectiveTools;
 use SplitIO\Component\Cache\SplitCache;
 use SplitIO\Component\Cache\SegmentCache;
-use SplitIO\Split as SplitApp;
 use SplitIO\Grammar\Split;
 use SplitIO\Sdk\Evaluator;
 

tests/Suite/DynamicConfigurations/SplitTest.php

@@ -4,7 +4,6 @@
 use SplitIO\Component\Common\Di;
 use SplitIO\Test\Suite\Redis\ReflectiveTools;
 use SplitIO\Component\Cache\SplitCache;
-use SplitIO\Split as SplitApp;
 use SplitIO\Grammar\Split;
 
 class SplitTest extends \PHPUnit\Framework\TestCase

tests/Suite/Engine/HashTest.php

@@ -8,7 +8,6 @@
 use SplitIO\Engine\Hash\HashAlgorithmEnum;
 use SplitIO\Grammar\Split;
 use SplitIO\Test\Suite\Redis\ReflectiveTools;
-use SplitIO\Split as SplitApp;
 use SplitIO\Component\Common\Di;
 
 use SplitIO\Test\Utils;