Debug commit, should be reverted

qgis · Jan 20, 2021 · ff6e625 · ff6e625
1 parent 703555a
commit ff6e625
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/gui/qgsabstractrelationeditorwidget.cpp b/src/gui/qgsabstractrelationeditorwidget.cpp
@@ -263,7 +263,9 @@ void QgsAbstractRelationEditorWidget::addFeature( const QgsGeometry &geometry )
 
     QgsMessageLog::logMessage( QStringLiteral( "ADDED FEATURES1 %1 %2" ).arg( mRelation.referencingLayer()->name() ).arg( mRelation.referencingLayer()->featureCount() ) );
 
-    Q_ASSERT( vlTools->addFeature( mRelation.referencingLayer(), keyAttrs, geometry ) );
+    bool success = vlTools->addFeature( mRelation.referencingLayer(), keyAttrs, geometry );
+    Q_ASSERT( success );
+    QgsMessageLog::logMessage( QStringLiteral( "SUCESS??? %1" ).arg( success ) );
 
     QgsMessageLog::logMessage( QStringLiteral( "ADDED FEATURES2 %1 %2" ).arg( mRelation.referencingLayer()->name() ).arg( mRelation.referencingLayer()->featureCount() ) );
     mRelation.referencingLayer()->dataProvider()->reloadData();

diff --git a/tests/src/python/test_qgsrelationeditwidget.py b/tests/src/python/test_qgsrelationeditwidget.py
@@ -330,6 +330,9 @@ def addFeature(self, layer, defaultValues, defaultGeometry):
         table_view = widget.findChild(QTableView)
         self.assertEqual(table_view.model().rowCount(), 1)
 
+        vl_leaks.raiseError.connect(TestQgsRelationEditWidget.error_raised)
+        vl_leaks.featureAdded.connect(TestQgsRelationEditWidget.feature_added)
+
         btn = widget.findChild(QToolButton, 'mAddFeatureGeometryButton')
         self.assertTrue(btn.isVisible())
         self.assertTrue(btn.isEnabled())
@@ -354,6 +357,14 @@ def addFeature(self, layer, defaultValues, defaultGeometry):
 
         vl_leaks.rollBack()
 
+    @classmethod
+    def error_raised(cls, msg):
+        print("LAYER ERROR RAISED: ", len(msg), msg)
+
+    @classmethod
+    def feature_added(cls, fid):
+        print("LAYER FEATURE ADDDED: ", fid)
+
     def startTransaction(self):
         """
         Start a new transaction and set all layers into transaction mode.