Changed getProvider() method to no longer be deprecated, and instead return the currently set provider, or if none set, the first configured provider.

mikebronner · mikebronner · commit 2cd8813f3922 · 2020-06-21T09:01:41.000-07:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,14 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## [4.3.4] - 2020-06-21
+### Fixed
+- non-caching declaration to only apply to current query.
+
+### Changed
+- `getProvider()` method to no longer be deprecated, and instead return the
+    currently set provider, or if none set, the first configured provider.
+
 ## [4.3.3] - 2020-06-20
 ### Added
 - functionality to not cache requests by using `doNotCache()`.
diff --git a/src/ProviderAndDumperAggregator.php b/src/ProviderAndDumperAggregator.php
@@ -130,12 +130,14 @@ public function limit(int $limit) : self
         return $this;
     }
 
-    /**
-     * @deprecated Use `getProviders()` instead.
-     */
     public function getProvider()
     {
-        return $this->getProviders()->first();
+        $reflectedClass = new ReflectionClass(ProviderAggregator::class);
+        $reflectedProperty = $reflectedClass->getProperty('provider');
+        $reflectedProperty->setAccessible(true);
+
+        return $reflectedProperty->getValue($this->aggregator)
+            ?? $this->getProviders()->first();
     }
 
     public function getProviders() : Collection
@@ -190,6 +192,8 @@ public function using(string $name) : self
     protected function cacheRequest(string $cacheKey, array $queryElements, string $queryType)
     {
         if (! $this->isCaching) {
+            $this->isCaching = true;
+
             return collect($this->aggregator->{$queryType}(...$queryElements));
         }
 
diff --git a/tests/Feature/Providers/GeocoderServiceTest.php b/tests/Feature/Providers/GeocoderServiceTest.php
@@ -326,4 +326,13 @@ public function testCachingCanBeDisabled()
             $results->first()->getFormattedAddress()
         );
     }
+
+    public function testGetProviderReturnsCurrentProvider()
+    {
+        $provider = app("geocoder")
+            ->using("google_maps")
+            ->getProvider();
+
+        $this->assertEquals("google_maps", $provider->getName());
+    }
 }
