Use optional defaults in constructor

Author: ivanvermeyen

src/Localizer.php

@@ -4,6 +4,7 @@
 
 use CodeZero\Localizer\Detectors\Detector;
 use Illuminate\Support\Facades\App;
+use Illuminate\Support\Facades\Config;
 
 class Localizer
 {
@@ -35,11 +36,11 @@ class Localizer
      * @param \Illuminate\Support\Collection|array $detectors
      * @param \Illuminate\Support\Collection|array $stores
      */
-    public function __construct($locales, $detectors, $stores = [])
+    public function __construct($locales = null, $detectors = null, $stores = null)
     {
-        $this->locales = $locales;
-        $this->detectors = $detectors;
-        $this->stores = $stores;
+        $this->locales = $locales ?: Config::get('localizer.supported-locales');
+        $this->detectors = $detectors ?: Config::get('localizer.detectors');
+        $this->stores = $stores ?: Config::get('localizer.stores');
     }
 
     /**

src/LocalizerServiceProvider.php

@@ -31,7 +31,6 @@ public function boot()
     public function register()
     {
         $this->mergeConfig();
-        $this->registerLocalizer();
     }
 
     /**
@@ -56,20 +55,4 @@ protected function mergeConfig()
     {
         $this->mergeConfigFrom(__DIR__."/../config/{$this->name}.php", $this->name);
     }
-
-    /**
-     * Register Localizer.
-     *
-     * @return void
-     */
-    protected function registerLocalizer()
-    {
-        $this->app->bind(Localizer::class, function ($app) {
-            $locales = $app['config']->get("{$this->name}.supported-locales");
-            $detectors = $app['config']->get("{$this->name}.detectors");
-            $stores = $app['config']->get("{$this->name}.stores");
-
-            return new Localizer($locales, $detectors, $stores);
-        });
-    }
 }