[geocoder] Fix inverted lat lon in nominatim's viewport

qgis · Dec 17, 2020 · e488a57 · e488a57
1 parent c56d2d8
commit e488a57
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/core/geocoding/qgsnominatimgeocoder.cpp b/src/core/geocoding/qgsnominatimgeocoder.cpp
@@ -217,10 +217,10 @@ QgsGeocoderResult QgsNominatimGeocoder::jsonToResult( const QVariantMap &json )
   {
     QVariantList boundingBox = json.value( QStringLiteral( "boundingbox" ) ).toList();
     if ( boundingBox.size() == 4 )
-      res.setViewport( QgsRectangle( boundingBox.at( 0 ).toDouble(),
-                                     boundingBox.at( 2 ).toDouble(),
-                                     boundingBox.at( 1 ).toDouble(),
-                                     boundingBox.at( 3 ).toDouble() ) );
+      res.setViewport( QgsRectangle( boundingBox.at( 2 ).toDouble(),
+                                     boundingBox.at( 0 ).toDouble(),
+                                     boundingBox.at( 3 ).toDouble(),
+                                     boundingBox.at( 1 ).toDouble() ) );
   }
 
   res.setAdditionalAttributes( attributes );

diff --git a/tests/src/python/test_qgsnominatimgeocoder.py b/tests/src/python/test_qgsnominatimgeocoder.py
@@ -130,7 +130,7 @@ def test_json_to_result(self):
                                                       'place_id': '157298780', 'postcode': 'G1C 4X5',
                                                       'road': 'Rue Anick', 'osm_type': 'way',
                                                       'type': 'school', 'class': 'amenity'})
-        self.assertEqual(res.viewport(), QgsRectangle(46.887, -71.199, 46.889, -71.201))
+        self.assertEqual(res.viewport(), QgsRectangle(-71.201, 46.887, -71.199, 46.889))
         self.assertEqual(res.group(), 'Quebec')