use singleton instead of share

Author: toin0u

src/Toin0u/Geocoder/GeocoderServiceProvider.php

@@ -11,7 +11,6 @@
 
 namespace Toin0u\Geocoder;
 
-use Config;
 use Geocoder\Geocoder;
 use Illuminate\Support\ServiceProvider;
 
@@ -48,18 +47,16 @@ public function register()
     {
         $app = $this->app;
 
-        $this->app['geocoder.adapter'] = $this->app->share(function() {
-            $adapter = Config::get('geocoder-laravel::adapter');
-            $class   = 'Geocoder\HttpAdapter\\' . $adapter;
+        $this->app->singleton('geocoder.adapter', function($app) {
+            $adapter = $app['config']->get('geocoder-laravel::adapter');
 
-            return new $class;
+            return new $adapter;
         });
 
-        $this->app['geocoder.provider'] = $this->app->share(function($app) {
-            $provider = Config::get('geocoder-laravel::provider');
-            $class    = '\Geocoder\Provider\\' . $provider;
+        $this->app->singleton('geocoder.provider', function($app) {
+            $provider = $app['config']->get('geocoder-laravel::provider');
 
-            return new $class($app['geocoder.adapter']);
+            return new $provider($app['geocoder.adapter']);
         });
 
         $this->app['geocoder'] = $this->app->share(function($app) {
@@ -77,6 +74,6 @@ public function register()
      */
     public function provides()
     {
-        return array('geocoder');
+        return array('geocoder', 'geocoder.adapter', 'geocoder.provider');
     }
 }

src/config/config.php

@@ -10,6 +10,6 @@
  */
 
 return array(
-    'provider' => 'FreeGeoIpProvider',
-    'adapter'  => 'CurlHttpAdapter'
+    'provider' => 'Geocoder\Provider\FreeGeoIpProvider',
+    'adapter'  => 'Geocoder\HttpAdapter\CurlHttpAdapter'
 );

tests/Geocoder/Tests/GeocoderServiceProviderTest.php

@@ -11,17 +11,15 @@
 
 namespace Geocoder\Tests;
 
-use Config;
-
 /**
  * @author Antoine Corcy <contact@sbin.dk>
  */
 class GeocoderServiceProviderTest extends TestCase
 {
     public function testConfig()
     {
-        $this->assertSame('FreeGeoIpProvider', Config::get('geocoder-laravel::provider'));
-        $this->assertSame('CurlHttpAdapter', Config::get('geocoder-laravel::adapter'));
+        $this->assertSame('Geocoder\Provider\FreeGeoIpProvider', $this->app['config']->get('geocoder-laravel::provider'));
+        $this->assertSame('Geocoder\HttpAdapter\CurlHttpAdapter', $this->app['config']->get('geocoder-laravel::adapter'));
     }
 
     public function testLoadedProviders()