reads and saves coordinates in reverse order (logitude, latitude)

yawik · Aug 28, 2018 · 0b18c8e · 0b18c8e
1 parent e3646c9
commit 0b18c8e
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/src/Solr/Bridge/Util.php b/src/Solr/Bridge/Util.php
@@ -30,7 +30,7 @@ class Util
     static public function convertLocationCoordinates(Location $location)
     {
         $coordinates = $location->getCoordinates()->getCoordinates();
-        $coordinate = doubleval($coordinates[0]) . '%' . doubleval($coordinates[1]);
+        $coordinate = doubleval($coordinates[1]) . '%' . doubleval($coordinates[0]);
         $coordinate = strtr($coordinate, [
             '%'=>',',
             ','=>'.'

diff --git a/test/SolrTest/Bridge/UtilTest.php b/test/SolrTest/Bridge/UtilTest.php
@@ -51,7 +51,7 @@ public function testConvertLocationCoordinates()
 
         $coordinates->expects($this->once())
             ->method('getCoordinates')
-            ->willReturn([0.1,0.2])
+            ->willReturn([0.2,0.1])
         ;
 
         $this->assertEquals('0.1,0.2',Util::convertLocationCoordinates($location));

diff --git a/test/SolrTest/Filter/EntityToDocument/JobEntityToSolrDocumentTest.php b/test/SolrTest/Filter/EntityToDocument/JobEntityToSolrDocumentTest.php
@@ -134,7 +134,7 @@ public function testGetDocumentIds()
         $locations = new ArrayCollection([$location]);
         $job->setLocations($locations);
 
-        $this->assertSame([$id, "{$id}-{$coordinate1},{$coordinate2}"], $this->jobFilter->getDocumentIds($job));
+        $this->assertSame([$id, "{$id}-{$coordinate2},{$coordinate1}"], $this->jobFilter->getDocumentIds($job));
     }
 
     /**
@@ -189,7 +189,7 @@ public function testProcessLocation()
     {
         $locationText = 'some location';
         $coordinatesArray = [1.2,2.1];
-        $coordinatesConverted = '1.2,2.1';
+        $coordinatesConverted = '2.1,1.2';
         $city = 'some city';
         $country = 'some country';
         $postalCode = 'some postal code';

diff --git a/test/SolrTest/Filter/JobBoardPaginationQueryTest.php b/test/SolrTest/Filter/JobBoardPaginationQueryTest.php
@@ -120,7 +120,7 @@ public function testCreateQuery()
         $location->method('getCoordinates')->willReturn($coordinates);
         $coordinates->expects($this->once())
             ->method('getCoordinates')
-            ->willReturn([1.2,2.1])
+            ->willReturn([2.1,1.2])
         ;
         $publishedSince = new DateTime();