Merge pull request #58

Author: willdurand

DependencyInjection/BazingaGeocoderExtension.php

@@ -114,7 +114,9 @@ public function load(array $configs, ContainerBuilder $container)
         }
 
         if (isset($config['providers']['mapquest'])) {
-            $this->addProvider('mapquest');
+            $mapquestParams = $config['providers']['mapquest'];
+            
+            $this->addProvider('mapquest', array($mapquestParams['api_key']));
         }
 
         if (isset($config['providers']['oiorest'])) {

DependencyInjection/Configuration.php

@@ -121,7 +121,13 @@ public function getConfigTreeBuilder()
                 ->arrayNode('host_ip')->end()
                 ->arrayNode('geoip')->end()
                 ->arrayNode('free_geo_ip')->end()
-                ->arrayNode('mapquest')->end()
+                ->arrayNode('mapquest')
+                    ->children()
+                        ->scalarNode('api_key')
+                            ->isRequired()->cannotBeEmpty()
+                        ->end()
+                    ->end()
+                ->end()
                 ->arrayNode('oiorest')->end()
                 ->arrayNode('geocoder_ca')->end()
                 ->arrayNode('geocoder_us')->end()

Resources/doc/index.md

@@ -244,7 +244,8 @@ bazinga_geocoder:
         host_ip:              []
         geoip:                []
         free_geo_ip:          []
-        mapquest:             []
+        mapquest:
+            api_key:              ~ # Required
         oiorest:              []
         geocoder_ca:          []
         geocoder_us:          []

Tests/DependencyInjection/Fixtures/config.yml

@@ -30,7 +30,8 @@ bazinga_geocoder:
         host_ip: ~
         # geoip: ~
         free_geo_ip: ~
-        mapquest: ~
+        mapquest:
+            api_key: 123
         oiorest: ~
         geocoder_ca: ~
         geocoder_us: ~