Merge pull request #784 from JulesPrimo/fix-nominatim-bounds

Nyholm · web-flow · commit 88ff22ee2a08 · 2017-10-06T07:59:36.000-04:00
Fixing order of coordinates passed to AddressBuilder::setBound
diff --git a/src/Provider/Nominatim/Nominatim.php b/src/Provider/Nominatim/Nominatim.php
@@ -156,7 +156,7 @@ private function xmlResultToArray(\DOMElement $resultNode, \DOMElement $addressN
         if ($boundsAttr) {
             $bounds = [];
             list($bounds['south'], $bounds['north'], $bounds['west'], $bounds['east']) = explode(',', $boundsAttr);
-            $builder->setBounds($bounds['south'], $bounds['north'], $bounds['west'], $bounds['east']);
+            $builder->setBounds($bounds['south'], $bounds['west'], $bounds['north'], $bounds['east']);
         }
 
         return $builder->build();

Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ private function xmlResultToArray(\DOMElement $resultNode, \DOMElement $addressN`
`156`	`156`	`if ($boundsAttr) {`
`157`	`157`	`$bounds = [];`
`158`	`158`	`list($bounds['south'], $bounds['north'], $bounds['west'], $bounds['east']) = explode(',', $boundsAttr);`
`159`		`- $builder->setBounds($bounds['south'], $bounds['north'], $bounds['west'], $bounds['east']);`
	`159`	`+ $builder->setBounds($bounds['south'], $bounds['west'], $bounds['north'], $bounds['east']);`
`160`	`160`	`}`
`161`	`161`
`162`	`162`	`return $builder->build();`