GNSS enhancements bundle

CMakeLists.txt

@@ -104,7 +104,7 @@ if (IOS)
   set(QGIS_QUICK_DATA_PATH_DEFAULT "INPUT")
 elseif (ANDROID)
   # Android sets this up automatically at runtime
-  set(QGIS_QUICK_DATA_PATH_DEFAULT "")
+  set(QGIS_QUICK_DATA_PATH_DEFAULT "INPUT")
 elseif (WIN)
   set(QGIS_QUICK_DATA_PATH_DEFAULT "INPUT")
 elseif (LNX OR MACOS)
@@ -389,6 +389,9 @@ file(
   PATTERN "*.cmake" EXCLUDE
   PATTERN "vcpkg*" EXCLUDE
 )
+file(COPY ${CMAKE_CURRENT_SOURCE_DIR}/app/assets/us_nga_egm96_15.tif
+     DESTINATION ${ASSETS_DIR_PATH}/app/android/assets/qgis-data/proj
+)
 
 message(STATUS "QGIS_QUICK_DATA_PATH set to ${QGIS_QUICK_DATA_PATH}")
 # ########################################################################################

app/inpututils.cpp

@@ -238,36 +238,36 @@ QString InputUtils::formatDateTimeDiff( const QDateTime &tMin, const QDateTime &
     else if ( secsDiff < 60 * 60 )
     {
       int period = secsDiff / 60 ;
-      return ( period > 1 ) ? tr( "%1 minutes ago" ).arg( period ) : tr( "%1 minute ago" ).arg( period );
+      return tr( "%n minute(s) ago", "", period );
     }
     else if ( secsDiff < 60 * 60 * 24 )
     {
       int period = secsDiff / ( 60 * 60 );
-      return ( period > 1 ) ? tr( "%1 hours ago" ).arg( period ) : tr( "%1 hour ago" ).arg( period );
+      return tr( "%n hour(s) ago", "", period );
     }
     else
     {
-      return ( daysDiff > 1 ) ? tr( "%1 days ago" ).arg( daysDiff ) : tr( "%1 day ago" ).arg( daysDiff );
+      return tr( "%n day(s) ago", "", daysDiff );
     }
   }
   else if ( daysDiff < 7 )
   {
-    return ( daysDiff > 1 ) ? tr( "%1 days ago" ).arg( daysDiff ) : tr( "%1 day ago" ).arg( daysDiff );
+    return tr( "%n day(s) ago", "", daysDiff );
   }
   else if ( daysDiff < 31 )
   {
     int period = daysDiff / 7;
-    return ( period > 1 ) ? tr( "%1 weeks ago" ).arg( period ) : tr( "%1 week ago" ).arg( period );
+    return tr( "%n week(s) ago", "", period );
   }
   else if ( daysDiff < 365 )
   {
     int period = daysDiff / 31;
-    return ( period > 1 ) ? tr( "%1 months ago" ).arg( period ) : tr( "%1 month ago" ).arg( period );
+    return tr( "%n month(s) ago", "", period );
   }
   else
   {
     int period = daysDiff / 365;
-    return ( period > 1 ) ? tr( "%1 years ago" ).arg( period ) : tr( "%1 year ago" ).arg( period );
+    return tr( "%n year(s) ago", "", period );
   }
 
   return INVALID_DATETIME_STR;
@@ -476,7 +476,7 @@ QString InputUtils::geometryLengthAsString( const QgsGeometry &geometry )
 {
   QgsDistanceArea distanceArea;
   distanceArea.setEllipsoid( QStringLiteral( "WGS84" ) );
-  distanceArea.setSourceCrs( QgsCoordinateReferenceSystem::fromEpsgId( 4326 ), QgsCoordinateTransformContext() );
+  distanceArea.setSourceCrs( PositionKit::positionCrs2D(), QgsProject::instance()->transformContext() );
 
   qreal length = distanceArea.measureLength( geometry );
 
@@ -879,32 +879,66 @@ QgsPoint InputUtils::transformPoint( const QgsCoordinateReferenceSystem &srcCrs,
   // QGIS would convert them to a valid (0, 0) points
   if ( srcPoint.isEmpty() )
   {
-    return QgsPoint();
+    return {};
   }
 
   try
   {
-    QgsCoordinateTransform ct( srcCrs, destCrs, context );
+    const QgsCoordinateTransform ct( srcCrs, destCrs, context );
     if ( ct.isValid() )
     {
       if ( !ct.isShortCircuited() )
       {
-        const QgsPointXY transformed = ct.transform( srcPoint.x(), srcPoint.y() );
-        const QgsPoint pt( transformed.x(), transformed.y(), srcPoint.z(), srcPoint.m() );
+        const QgsVector3D transformed = ct.transform( QgsVector3D( srcPoint.x(), srcPoint.y(), srcPoint.z() ) );
+        const QgsPoint pt( transformed.x(), transformed.y(), transformed.z(), srcPoint.m() );
         return pt;
       }
-      else
+
+      return srcPoint;
+    }
+  }
+  catch ( QgsCsException &cse )
+  {
+    Q_UNUSED( cse )
+  }
+
+  return {};
+}
+
+QgsPoint InputUtils::transformPoint( const QgsCoordinateReferenceSystem &srcCrs,
+                                     const QgsCoordinateReferenceSystem &destCrs,
+                                     const QgsCoordinateTransformContext &context,
+                                     const QgsPoint &srcPoint, bool &fallbackOperationOccurred )
+{
+  // we do not want to transform empty points,
+  // QGIS would convert them to a valid (0, 0) points
+  if ( srcPoint.isEmpty() )
+  {
+    return {};
+  }
+
+  try
+  {
+    const QgsCoordinateTransform ct( srcCrs, destCrs, context );
+    if ( ct.isValid() )
+    {
+      if ( !ct.isShortCircuited() )
       {
-        return srcPoint;
+        const QgsVector3D transformed = ct.transform( QgsVector3D( srcPoint.x(), srcPoint.y(), srcPoint.z() ) );
+        fallbackOperationOccurred = ct.fallbackOperationOccurred();
+        const QgsPoint pt( transformed.x(), transformed.y(), transformed.z(), srcPoint.m() );
+        return pt;
       }
+
+      return srcPoint;
     }
   }
   catch ( QgsCsException &cse )
   {
     Q_UNUSED( cse )
   }
 
-  return QgsPoint();
+  return {};
 }
 
 QPointF InputUtils::transformPointToScreenCoordinates( const QgsCoordinateReferenceSystem &srcCrs, InputMapSettings *mapSettings, const QgsPoint &srcPoint )
@@ -953,12 +987,11 @@ QgsPoint InputUtils::mapPointToGps( QPointF mapPosition, InputMapSettings *mapSe
     return QgsPoint();
 
   QgsPoint positionMapCrs = mapSettings->screenToCoordinate( mapPosition );
-  QgsCoordinateReferenceSystem crsGPS = coordinateReferenceSystemFromEpsgId( 4326 );
 
   const QgsPointXY transformedXY = transformPoint(
                                      mapSettings->destinationCrs(),
-                                     crsGPS,
-                                     QgsCoordinateTransformContext(),
+                                     PositionKit::positionCrs2D(),
+                                     mapSettings->transformContext(),
                                      positionMapCrs
                                    );
 
@@ -1715,7 +1748,7 @@ qreal InputUtils::distanceBetweenGpsAndFeature( QgsPoint gpsPosition, const Feat
 
   // Transform gps position to map CRS
   QgsPointXY transformedPosition = transformPoint(
-                                     coordinateReferenceSystemFromEpsgId( 4326 ),
+                                     PositionKit::positionCrs3D(),
                                      mapSettings->destinationCrs(),
                                      mapSettings->transformContext(),
                                      gpsPosition
@@ -1763,7 +1796,7 @@ qreal InputUtils::angleBetweenGpsAndFeature( QgsPoint gpsPoint, const FeatureLay
 
   // Transform gps position to map CRS
   QgsPointXY transformedPosition = transformPoint(
-                                     coordinateReferenceSystemFromEpsgId( 4326 ),
+                                     PositionKit::positionCrs3D(),
                                      mapSettings->destinationCrs(),
                                      mapSettings->transformContext(),
                                      gpsPoint
@@ -2108,11 +2141,11 @@ QString InputUtils::invalidGeometryWarning( QgsVectorLayer *layer )
 
   if ( QgsWkbTypes::isMultiType( layer->wkbType() ) )
   {
-    return tr( "You need to add at least %1 point(s) to every part." ).arg( nPoints );
+    return tr( "You need to add at least %n point(s) to every part.", "", nPoints );
   }
   else
   {
-    return tr( "You need to add at least %1 point(s)." ).arg( nPoints );
+    return tr( "You need to add at least %n point(s).", "", nPoints );
   }
 }
 

app/inpututils.h

@@ -305,6 +305,14 @@ class InputUtils: public QObject
         const QgsCoordinateTransformContext &context,
         const QgsPoint &srcPoint );
 
+    /**
+     * Overload of transformPoint function, which also notifies the caller if PROJ fallback operation occurred
+     */
+    static QgsPoint transformPoint( const QgsCoordinateReferenceSystem &srcCrs,
+                                    const QgsCoordinateReferenceSystem &destCrs,
+                                    const QgsCoordinateTransformContext &context,
+                                    const QgsPoint &srcPoint, bool &fallbackOperationOccurred );
+
     /**
       * Transforms point between CRS and screen pixels
       * Return empty QgsPoint if the transformation could not be applied or srcPoint is empty

app/main.cpp

@@ -544,13 +544,13 @@ int main( int argc, char *argv[] )
   LayerDetailLegendImageProvider *layerDetailLegendImageProvider( new LayerDetailLegendImageProvider );
 
   // build position kit, save active provider to QSettings and load previously active provider
-  PositionKit pk;
-  QObject::connect( &pk, &PositionKit::positionProviderChanged, as, [as]( AbstractPositionProvider * provider )
+  PositionKit *pk = engine.singletonInstance<PositionKit *>( "MMInput", "PositionKit" );
+  QObject::connect( pk, &PositionKit::positionProviderChanged, as, [as]( AbstractPositionProvider * provider )
   {
     as->setActivePositionProviderId( provider ? provider->id() : QLatin1String() );
   } );
-  pk.setPositionProvider( pk.constructActiveProvider( as ) );
-  pk.setAppSettings( as );
+  pk->setPositionProvider( PositionKit::constructActiveProvider( as ) );
+  pk->setAppSettings( as );
 
   // Lambda context object can be used in all lambda functions defined here,
   // it secures lambdas, so that they are destroyed when this object is destroyed to avoid crashes.
@@ -648,7 +648,7 @@ int main( int argc, char *argv[] )
     notificationModel.addError( message );
   } );
   // Direct connections
-  QObject::connect( &app, &QGuiApplication::applicationStateChanged, &pk, &PositionKit::appStateChanged );
+  QObject::connect( &app, &QGuiApplication::applicationStateChanged, pk, &PositionKit::appStateChanged );
   QObject::connect( &pw, &ProjectWizard::projectCreated, &localProjectsManager, &LocalProjectsManager::addLocalProject );
   QObject::connect( &activeProject, &ActiveProject::projectReloaded, vm.get(), &VariablesManager::merginProjectChanged );
   QObject::connect( &activeProject, &ActiveProject::projectWillBeReloaded, &inputProjUtils, &InputProjUtils::resetHandlers );
@@ -679,7 +679,7 @@ int main( int argc, char *argv[] )
   if ( tests.testingRequested() )
   {
     tests.initTestDeclarative();
-    tests.init( ma.get(), &iu, vm.get(), &pk, as );
+    tests.init( ma.get(), &iu, vm.get(), pk, as );
     return tests.runTest();
   }
 #endif
@@ -722,7 +722,6 @@ int main( int argc, char *argv[] )
   engine.rootContext()->setContextProperty( "__projectWizard", &pw );
   engine.rootContext()->setContextProperty( "__localProjectsManager", &localProjectsManager );
   engine.rootContext()->setContextProperty( "__variablesManager", vm.get() );
-  engine.rootContext()->setContextProperty( "__positionKit", &pk );
 
   // add image provider to pass QIcons/QImages from C++ to QML
   engine.rootContext()->setContextProperty( "__layerTreeModelPixmapProvider", layerTreeModelPixmapProvider );

app/map/inputcoordinatetransformer.cpp

@@ -13,12 +13,14 @@
  */
 
 #include "inputcoordinatetransformer.h"
+
+#include "positionkit.h"
 #include "qgslogger.h"
 
 InputCoordinateTransformer::InputCoordinateTransformer( QObject *parent )
   : QObject( parent )
 {
-  mCoordinateTransform.setSourceCrs( QgsCoordinateReferenceSystem::fromEpsgId( 4326 ) );
+  mCoordinateTransform.setSourceCrs( PositionKit::positionCrs3D() );
 }
 
 QgsPoint InputCoordinateTransformer::projectedPosition() const

app/maptools/recordingmaptool.cpp

@@ -59,7 +59,7 @@ void RecordingMapTool::addPoint( const QgsPoint &point )
     pointToAdd = mPositionKit->positionCoordinate();
 
     QgsPoint transformed = InputUtils::transformPoint(
-                             PositionKit::positionCRS(),
+                             PositionKit::positionCrs3D(),
                              mActiveLayer->crs(),
                              mActiveLayer->transformContext(),
                              pointToAdd
@@ -594,7 +594,7 @@ void RecordingMapTool::onPositionChanged()
       QgsPoint position = mPositionKit->positionCoordinate();
 
       QgsPointXY transformed = InputUtils::transformPoint(
-                                 PositionKit::positionCRS(),
+                                 PositionKit::positionCrs3D(),
                                  mActiveLayer->sourceCrs(),
                                  mActiveLayer->transformContext(),
                                  position

app/position/geoposition.cpp

@@ -15,6 +15,7 @@ GeoPosition::GeoPosition() : QgsGpsInformation()
   latitude = std::numeric_limits<double>::quiet_NaN();
   longitude = std::numeric_limits<double>::quiet_NaN();
   elevation = std::numeric_limits<double>::quiet_NaN();
+  elevation_diff = std::numeric_limits<double>::quiet_NaN();
   direction = -1;
   speed = -1;
   pdop = -1;
@@ -48,6 +49,11 @@ GeoPosition GeoPosition::fromQgsGpsInformation( const QgsGpsInformation &other )
     out.elevation = other.elevation;
   }
 
+  if ( !qgsDoubleNear( other.elevation_diff, 0 ) )
+  {
+    out.elevation_diff = other.elevation_diff;
+  }
+
   if ( !std::isnan( other.direction ) )
   {
     out.direction = other.direction;
@@ -88,7 +94,6 @@ GeoPosition GeoPosition::fromQgsGpsInformation( const QgsGpsInformation &other )
     out.vdop = other.vdop;
   }
 
-  out.elevation_diff = other.elevation_diff;
   out.satellitesVisible = other.satellitesInView.count();
   out.satellitesInView = other.satellitesInView;
   out.satellitesUsed = other.satellitesUsed;

app/position/geoposition.h

@@ -29,6 +29,8 @@ class GeoPosition : public QgsGpsInformation
 
     QString fixStatusString;
 
+    bool isMock = false;
+
     // copies all data from QgsGpsInformation other and updates satellitesVisible
     static GeoPosition fromQgsGpsInformation( const QgsGpsInformation &other );
 

app/position/mapposition.cpp

@@ -105,7 +105,7 @@ void MapPosition::recalculateMapPosition()
     {
       QgsPointXY srcPoint = QgsPointXY( geoposition.x(), geoposition.y() );
       QgsPointXY mapPositionXY = InputUtils::transformPointXY(
-                                   mPositionKit->positionCRS(),
+                                   mPositionKit->positionCrs3D(),
                                    mMapSettings->destinationCrs(),
                                    mMapSettings->transformContext(),
                                    srcPoint