Update Profiler to accept Config or array

Author: glensc

src/Profiler.php

@@ -22,7 +22,7 @@ final class Profiler
     /**
      * Profiler configuration.
      *
-     * @var array
+     * @var Config
      */
     private $config;
 
@@ -54,11 +54,15 @@ final class Profiler
     /**
      * Profiler constructor.
      *
-     * @param array $config
+     * @param array|Config $config
      */
-    public function __construct(array $config)
+    public function __construct($config)
     {
-        $this->config = array_replace($this->getDefaultConfig(), $config);
+        if ($config instanceof Config) {
+            $this->config = $config;
+        } else {
+            $this->config = new Config($config);
+        }
     }
 
     /**

src/ProfilerFactory.php

@@ -21,7 +21,7 @@ final class ProfilerFactory
      *
      * @return ProfilerInterface|null
      */
-    public static function create(array $config)
+    public static function create(Config $config)
     {
         $adapters = array(
             Profiler::PROFILER_TIDEWAYS_XHPROF => function () {

src/ProfilingData.php

@@ -14,7 +14,7 @@ final class ProfilingData
     /** @var callable|null */
     private $replaceUrl;
 
-    public function __construct(array $config = array())
+    public function __construct(Config $config)
     {
         $this->excludeEnv = isset($config['profiler.exclude-env']) ? (array)$config['profiler.exclude-env'] : array();
         $this->simpleUrl = isset($config['profiler.simple_url']) ? $config['profiler.simple_url'] : null;

src/SaverFactory.php

@@ -14,10 +14,10 @@ final class SaverFactory
 {
     /**
      * @param string $saveHandler
-     * @param array $config
+     * @param Config $config
      * @return SaverInterface|null
      */
-    public static function create($saveHandler, array $config = array())
+    public static function create($saveHandler, Config $config)
     {
         switch ($saveHandler) {
             case Profiler::SAVER_FILE:
@@ -64,7 +64,7 @@ public static function create($saveHandler, array $config = array())
                     throw new ProfilerException("For {$saveHandler} you need to install xhgui-collector package: composer require perftools/xhgui-collector");
                 }
                 $config = self::migrateConfig($config, $saveHandler);
-                $legacySaver = Xhgui_Saver::factory($config);
+                $legacySaver = Xhgui_Saver::factory($config->toArray());
                 $saver = self::getAdapter($legacySaver);
                 break;
         }
@@ -79,11 +79,11 @@ public static function create($saveHandler, array $config = array())
     /**
      * Prepare config for Xhgui_Saver specific to $saveHandler
      *
-     * @param array $config
+     * @param Config $config
      * @param string $saveHandler
-     * @return array
+     * @return Config
      */
-    private static function migrateConfig(array $config, $saveHandler)
+    private static function migrateConfig(Config $config, $saveHandler)
     {
         switch ($saveHandler) {
             case Profiler::SAVER_MONGODB:

tests/TestCase.php

@@ -2,6 +2,7 @@
 
 namespace Xhgui\Profiler\Test;
 
+use Xhgui\Profiler\Config;
 use Xhgui\Profiler\Profilers\ProfilerInterface;
 use Xhgui\Profiler\Saver\SaverInterface;
 use Xhgui\Profiler\SaverFactory;
@@ -30,7 +31,7 @@ protected function runProfiler($flags = array(), $options = array())
 
     protected function createSaver($saveHandler, array $config = array())
     {
-        $saver = SaverFactory::create($saveHandler, $config);
+        $saver = SaverFactory::create($saveHandler, new Config($config));
         $this->assertNotNull($saver);
 
         return $saver;