Fix regression when adding feature without geom #46687

qgis · Mar 17, 2022 · ff87afd · ff87afd
1 parent 180ae3b
commit ff87afd
Showing 1 changed file with 25 additions and 21 deletions.
diff --git a/src/gui/qgsmaptooldigitizefeature.cpp b/src/gui/qgsmaptooldigitizefeature.cpp
@@ -73,30 +73,34 @@ void QgsMapToolDigitizeFeature::layerGeometryCaptured( const QgsGeometry &geomet
   if ( !vlayer )
     return;
 
-  const QgsWkbTypes::Type layerWKBType = vlayer->wkbType();
-
-  QgsGeometry layerGeometry;
+  QgsFeature f( vlayer->fields(), 0 );
 
-  if ( mCheckGeometryType )
+  if ( vlayer->isSpatial() )
   {
-    double defaultZ = QgsSettingsRegistryCore::settingsDigitizingDefaultZValue.value();
-    double defaultM = QgsSettingsRegistryCore::settingsDigitizingDefaultMValue.value();
-    QVector<QgsGeometry> layerGeometries = geometry.coerceToType( layerWKBType, defaultZ, defaultM );
-    if ( layerGeometries.count() > 0 )
-      layerGeometry = layerGeometries.at( 0 );
-
-    if ( layerGeometry.wkbType() != layerWKBType && layerGeometry.wkbType() != QgsWkbTypes::linearType( layerWKBType ) )
-    {
-      emit messageEmitted( tr( "The digitized geometry type (%1) does not correspond to the layer geometry type (%2)." ).arg( QgsWkbTypes::displayString( layerGeometry.wkbType() ) ).arg( QgsWkbTypes::displayString( layerWKBType ) ), Qgis::MessageLevel::Warning );
-      return;
-    }
+      const QgsWkbTypes::Type layerWKBType = vlayer->wkbType();
+
+      QgsGeometry layerGeometry;
+
+      if ( mCheckGeometryType )
+      {
+        double defaultZ = QgsSettingsRegistryCore::settingsDigitizingDefaultZValue.value();
+        double defaultM = QgsSettingsRegistryCore::settingsDigitizingDefaultMValue.value();
+        QVector<QgsGeometry> layerGeometries = geometry.coerceToType( layerWKBType, defaultZ, defaultM );
+        if ( layerGeometries.count() > 0 )
+          layerGeometry = layerGeometries.at( 0 );
+
+        if ( layerGeometry.wkbType() != layerWKBType && layerGeometry.wkbType() != QgsWkbTypes::linearType( layerWKBType ) )
+        {
+          emit messageEmitted( tr( "The digitized geometry type (%1) does not correspond to the layer geometry type (%2)." ).arg( QgsWkbTypes::displayString( layerGeometry.wkbType() ), QgsWkbTypes::displayString( layerWKBType ) ), Qgis::MessageLevel::Warning );
+          return;
+        }
+      }
+      else
+      {
+        layerGeometry = geometry;
+      }
+      f.setGeometry( layerGeometry );
   }
-  else
-  {
-    layerGeometry = geometry;
-  }
-  QgsFeature f( vlayer->fields(), 0 );
-  f.setGeometry( layerGeometry );
   f.setValid( true );
   emit digitizingCompleted( f );
   featureDigitized( f );