Update changelog

CHANGELOG.md

@@ -0,0 +1,8 @@
+# Changelog
+
+## [x.x.x] xxxx-xx-xx
+- Coordinates set to 0, 0 will cause isGeocoded to return false
+
+## [1.0.0] 2024-08-05
+- First major version release
+

README.md

@@ -1 +1,6 @@
-Readme
+# PCM Geocode Bundle
+
+Provides an interface/trait combo to add latitude and longitude fields to an entity.
+
+Also included is a `Geocoder` service which accepts a postcode and attempts to return a `GeoCoordinates` object which contains the latitude and longitude of the postcode.
+

src/Interface/Entity/GeocodeInterface.php

@@ -4,6 +4,9 @@ declare(strict_types=1);
 
 namespace Pcm\GeocodeBundle\Interface\Entity;
 
+/**
+ * Defines latitude and longitude getters and setters
+ */
 interface GeocodeInterface
 {
     public function getLatitude(): ?float;
@@ -14,5 +17,9 @@ interface GeocodeInterface
 
     public function setLongitude(float $lon): self;
 
+    /**
+     * @return bool True if both latitude and longitude is set
+     */
     public function isGeocoded(): bool;
 }
+

src/Service/Geocoder.php

@@ -10,6 +10,9 @@ use Pcm\GeocodeBundle\Model\GeoCoordinates;
 use Symfony\Contracts\HttpClient\HttpClientInterface;
 use Symfony\Contracts\HttpClient\ResponseInterface;
 
+/**
+ * Find the geo-coordinates of a postcode
+ */
 final class Geocoder
 {
     private const string API_URL = "https://nominatim.openstreetmap.org/search";
@@ -17,12 +20,16 @@ final class Geocoder
     public function __construct(private HttpClientInterface $client) {}
 
     /**
-     * Convert a postcode into latitude and longitude. Returns null if conversion failed.
+     * Find and return the geo-coordinates of a postcode
      *
      * @param string $postcode
-     * @return null|GeoCoordinates
+     *
+     * @return GeoCoordinates
+     *
+     * @throws NoResultsFoundException when no results were found for the provided postcode
+     * @throws ApiErrorException when the API response contains an error
      */
-    public function geocodePostcode(string $postcode): ?GeoCoordinates
+    public function geocodePostcode(string $postcode): GeoCoordinates
     {
         $client = $this->createClient();
         $response = $this->makeApiRequest($client, $postcode);

src/Trait/Entity/GeocodeTrait.php

@@ -43,14 +43,9 @@ trait GeocodeTrait
         return $this;
     }
 
-    /**
-     * Returns true if both latitude and longitude have been set
-     *
-     * @return bool
-     */
     public function isGeocoded(): bool
     {
-        return null !== $this->getLatitude() && null !== $this->getLongitude();
+        return null !== $this->getLatitude() && null !== $this->getLongitude() && (0.0 !== $this->getLatitude() && 0.0 !== $this->getLongitude());
     }
 }
 

tests/Trait/GeocodeTraitTest.php

@@ -75,11 +75,18 @@ final class GeocodeTraitTest extends TestCase
         $this->assertTrue($this->obj->isGeocoded());
     }
 
-    public function testIsGeocodeReturnsTrueIfLatAndLonAreBothZero(): void
+    public function testIsGeocodedReturnsFalseIfLatAndLonAreBothZero(): void
     {
         $this->obj->setLatitude(0.000);
         $this->obj->setLongitude(0.000);
-        $this->assertTrue($this->obj->isGeocoded());
+        $this->assertFalse($this->obj->isGeocoded());
+    }
+
+    public function testIsGeocodedReturnsFalseIfLatAndLonAreBothZeroInts(): void
+    {
+        $this->obj->setLatitude(0);
+        $this->obj->setLongitude(0);
+        $this->assertFalse($this->obj->isGeocoded());
     }
 
     private function getTraitObject(): GeocodeInterface