Merge pull request #248 from Baachi/limit

Add limit/setMaxResults method

Author: willdurand

src/Geocoder/Geocoder.php

@@ -60,7 +60,7 @@ public function __construct(ProviderInterface $provider = null, ResultFactoryInt
         $this->provider = $provider;
 
         $this->setResultFactory($resultFactory);
-        $this->setMaxResults($maxResults);
+        $this->limit($maxResults);
     }
 
     /**
@@ -76,7 +76,7 @@ public function setResultFactory(ResultFactoryInterface $resultFactory = null)
      *
      * @return GeocoderInterface
      */
-    public function setMaxResults($maxResults)
+    public function limit($maxResults)
     {
         $this->maxResults = $maxResults;
 

src/Geocoder/Provider/AbstractProvider.php

@@ -92,11 +92,7 @@ public function setLocale($locale = null)
     }
 
     /**
-     * Sets the maximum of returned results.
-     *
-     * @param integer $maxResults
-     *
-     * @return AbstractProvider
+     * {@inheritDoc}
      */
     public function setMaxResults($maxResults)
     {

src/Geocoder/Provider/ChainProvider.php

@@ -77,6 +77,18 @@ public function getReversedData(array $coordinates)
         throw new NoResultException(sprintf('No provider could provide the coordinated %s', json_encode($coordinates)));
     }
 
+    /**
+     * {@inheritDoc}
+     */
+    public function setMaxResults($limit)
+    {
+        foreach ($this->providers as $provider) {
+            $provider->setMaxResults($limit);
+        }
+
+        return $this;
+    }
+
     /**
      * {@inheritDoc}
      */

src/Geocoder/Provider/ProviderInterface.php

@@ -51,4 +51,13 @@ public function getReversedData(array $coordinates);
      * @return string
      */
     public function getName();
+
+    /**
+     * Sets the maximum of returned results.
+     *
+     * @param integer $limit
+     *
+     * @return ProviderInterface
+     */
+    public function setMaxResults($limit);
 }

tests/Geocoder/Tests/GeocoderTest.php

@@ -202,7 +202,7 @@ public function testSetAndUseCustomMultipleResultFactory()
 
     public function testSetMaxResults()
     {
-        $this->geocoder->setMaxResults(3);
+        $this->geocoder->limit(3);
         $this->assertSame(3, $this->geocoder->getMaxResults());
     }