Merge pull request #456 from Nyholm/patch-1

Bugfix, Sometimes $data is null at this point.

Author: willdurand

src/Geocoder/Provider/Yandex.php

@@ -100,7 +100,9 @@ private function executeQuery($query)
         $content = (string) $this->getAdapter()->get($query)->getBody();
         $json    = (array) json_decode($content, true);
 
-        if (empty($json) || '0' === $json['response']['GeoObjectCollection']['metaDataProperty']['GeocoderResponseMetaData']['found']) {
+        if (empty($json) || isset($json['error']) ||
+            (isset($json['response']) &&  '0' === $json['response']['GeoObjectCollection']['metaDataProperty']['GeocoderResponseMetaData']['found'])
+        ) {
             throw new NoResult(sprintf('Could not execute query "%s".', $query));
         }
 

tests/Geocoder/Tests/Provider/YandexTest.php

@@ -42,7 +42,7 @@ public function testGeocodeWithLocalhostIPv6()
      */
     public function testGeocodeWithNull()
     {
-        $provider = new Yandex($this->getMockAdapter());
+        $provider = new Yandex($this->getMockAdapterReturns('{"error":{"status":"400","message":"missing geocode parameter"}}'));
         $provider->geocode(null);
     }
 
@@ -52,7 +52,7 @@ public function testGeocodeWithNull()
      */
     public function testGeocodeWithEmpty()
     {
-        $provider = new Yandex($this->getMockAdapter());
+        $provider = new Yandex($this->getMockAdapterReturns('{"error":{"status":"400","message":"missing geocode parameter"}}'));
         $provider->geocode('');
     }