From e311465d28b13a6670ca961bf2bb849155818c42 Mon Sep 17 00:00:00 2001
From: Andreas Bircher <bircher@gmx.ch>
Date: Mon, 27 Nov 2017 17:15:19 -0500
Subject: [PATCH] minor fixes

---
 src/QtLocationPlugin/QGCMapUrlEngine.cpp | 8 ++++----
 src/Terrain.cc                           | 6 ++++--
 src/TerrainTile.cc                       | 8 ++++----
 3 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/src/QtLocationPlugin/QGCMapUrlEngine.cpp b/src/QtLocationPlugin/QGCMapUrlEngine.cpp
index b61079153..fe2a8a2eb 100644
--- a/src/QtLocationPlugin/QGCMapUrlEngine.cpp
+++ b/src/QtLocationPlugin/QGCMapUrlEngine.cpp
@@ -404,10 +404,10 @@ UrlFactory::_getURL(MapType type, int x, int y, int zoom, QNetworkAccessManager*
     break;
     case AirmapElevation:
     {
-        return QString("https://api.airmap.com/elevation/stage/srtm1/ele/carpet?points=%1,%2,%3,%4").arg(static_cast<double>(x)*QGCMapEngine::srtm1TileSize - 180.0).arg(
-                                                                                                         static_cast<double>(y)*QGCMapEngine::srtm1TileSize - 90.0).arg(
-                                                                                                         static_cast<double>(x + 1)*QGCMapEngine::srtm1TileSize - 180.0).arg(
-                                                                                                         static_cast<double>(y + 1)*QGCMapEngine::srtm1TileSize - 90.0);
+        return QString("https://api.airmap.com/elevation/stage/srtm1/ele/carpet?points=%1,%2,%3,%4").arg(static_cast<double>(y)*QGCMapEngine::srtm1TileSize - 90.0).arg(
+                                                                                                         static_cast<double>(x)*QGCMapEngine::srtm1TileSize - 180.0).arg(
+                                                                                                         static_cast<double>(y + 1)*QGCMapEngine::srtm1TileSize - 90.0).arg(
+                                                                                                         static_cast<double>(x + 1)*QGCMapEngine::srtm1TileSize - 180.0);
     }
     break;
 
diff --git a/src/Terrain.cc b/src/Terrain.cc
index c0fb92cef..bf5fced0e 100644
--- a/src/Terrain.cc
+++ b/src/Terrain.cc
@@ -210,12 +210,14 @@ void ElevationProvider::_fetchedTile()
     TerrainTile* terrainTile = new TerrainTile(responseJson);
     if (terrainTile->isValid()) {
         _tilesMutex.lock();
-        if (!_tiles.contains(_getTileHash(terrainTile->centerCoordinate()))) {
-            _tiles.insert(_getTileHash(terrainTile->centerCoordinate()), *terrainTile);
+        if (!_tiles.contains(hash)) {
+            _tiles.insert(hash), *terrainTile);
         } else {
             delete terrainTile;
         }
         _tilesMutex.unlock();
+    } else {
+        qCDebug(TerrainLog) << "Received invalid tile";
     }
     reply->deleteLater();
 
diff --git a/src/TerrainTile.cc b/src/TerrainTile.cc
index d52ccd236..38a1d277e 100644
--- a/src/TerrainTile.cc
+++ b/src/TerrainTile.cc
@@ -85,10 +85,10 @@ TerrainTile::TerrainTile(QJsonDocument document)
         qCDebug(TerrainTileLog) << "Incomplete bounding location";
         return;
     }
-    _southWest.setLatitude(swArray[1].toDouble());
-    _southWest.setLongitude(swArray[0].toDouble());
-    _northEast.setLatitude(neArray[1].toDouble());
-    _northEast.setLongitude(neArray[0].toDouble());
+    _southWest.setLatitude(swArray[0].toDouble());
+    _southWest.setLongitude(swArray[1].toDouble());
+    _northEast.setLatitude(neArray[0].toDouble());
+    _northEast.setLongitude(neArray[1].toDouble());
 
     // Stats
     const QJsonObject& statsObject = dataObject[_jsonStatsKey].toObject();
-- 
2.22.0