Geometry validator: do not report result summary as error

m-kuhn · m-kuhn · commit 968e08c6c9e8 · 2019-02-20T18:14:41.000+01:00
diff --git a/python/core/auto_generated/qgsgeometryvalidator.sip.in b/python/core/auto_generated/qgsgeometryvalidator.sip.in
@@ -28,11 +28,29 @@ Constructor for QgsGeometryValidator.
 
     static void validateGeometry( const QgsGeometry &geometry, QVector<QgsGeometry::Error> &errors /Out/, QgsGeometry::ValidationMethod method = QgsGeometry::ValidatorQgisInternal );
 %Docstring
-Validate geometry and produce a list of geometry errors
+Validate geometry and produce a list of geometry errors.
+This method blocks the thread until the validation is finished.
 %End
 
   signals:
-    void errorFound( const QgsGeometry::Error & );
+
+    void errorFound( const QgsGeometry::Error &error );
+%Docstring
+Sent when an error has been found during the validation process.
+
+The ``error`` contains details about the error.
+%End
+
+    void validationFinished( const QString &summary );
+%Docstring
+Sent when the validation is finished.
+
+The result is in a human readable ``summary``, mentioning
+if the validation has been aborted, successfully been validated
+or how many errors have been found.
+
+.. versionadded:: 3.6
+%End
 
   public slots:
     void addError( const QgsGeometry::Error & );
diff --git a/src/core/qgsgeometryvalidator.cpp b/src/core/qgsgeometryvalidator.cpp
@@ -349,18 +349,16 @@ void QgsGeometryValidator::run()
 
       if ( mStop )
       {
-        emit errorFound( QgsGeometry::Error( QObject::tr( "Geometry validation was aborted." ) ) );
+        emit validationFinished( QObject::tr( "Geometry validation was aborted." ) );
       }
       else if ( mErrorCount > 0 )
       {
-        emit errorFound( QgsGeometry::Error( QObject::tr( "Geometry has %1 errors." ).arg( mErrorCount ) ) );
+        emit validationFinished( QObject::tr( "Geometry has %1 errors." ).arg( mErrorCount ) );
       }
-#if 0
       else
       {
-        emit errorFound( QgsGeometry::Error( QObject::tr( "Geometry is valid." ) ) );
+        emit validationFinished( QObject::tr( "Geometry is valid." ) );
       }
-#endif
       break;
     }
   }
diff --git a/src/core/qgsgeometryvalidator.h b/src/core/qgsgeometryvalidator.h
@@ -40,11 +40,31 @@ class CORE_EXPORT QgsGeometryValidator : public QThread
     void run() override;
     void stop();
 
-    //! Validate geometry and produce a list of geometry errors
+    /**
+     * Validate geometry and produce a list of geometry errors.
+     * This method blocks the thread until the validation is finished.
+     */
     static void validateGeometry( const QgsGeometry &geometry, QVector<QgsGeometry::Error> &errors SIP_OUT, QgsGeometry::ValidationMethod method = QgsGeometry::ValidatorQgisInternal );
 
   signals:
-    void errorFound( const QgsGeometry::Error & );
+
+    /**
+     * Sent when an error has been found during the validation process.
+     *
+     * The \a error contains details about the error.
+     */
+    void errorFound( const QgsGeometry::Error &error );
+
+    /**
+     * Sent when the validation is finished.
+     *
+     * The result is in a human readable \a summary, mentioning
+     * if the validation has been aborted, successfully been validated
+     * or how many errors have been found.
+     *
+     * \since QGIS 3.6
+     */
+    void validationFinished( const QString &summary );
 
   public slots:
     void addError( const QgsGeometry::Error & );

Original file line number	Diff line number	Diff line change
`@@ -349,18 +349,16 @@ void QgsGeometryValidator::run()`
`349`	`349`
`350`	`350`	`if ( mStop )`
`351`	`351`	`{`
`352`		`- emit errorFound( QgsGeometry::Error( QObject::tr( "Geometry validation was aborted." ) ) );`
	`352`	`+ emit validationFinished( QObject::tr( "Geometry validation was aborted." ) );`
`353`	`353`	`}`
`354`	`354`	`else if ( mErrorCount > 0 )`
`355`	`355`	`{`
`356`		`- emit errorFound( QgsGeometry::Error( QObject::tr( "Geometry has %1 errors." ).arg( mErrorCount ) ) );`
	`356`	`+ emit validationFinished( QObject::tr( "Geometry has %1 errors." ).arg( mErrorCount ) );`
`357`	`357`	`}`
`358`		`-#if 0`
`359`	`358`	`else`
`360`	`359`	`{`
`361`		`- emit errorFound( QgsGeometry::Error( QObject::tr( "Geometry is valid." ) ) );`
	`360`	`+ emit validationFinished( QObject::tr( "Geometry is valid." ) );`
`362`	`361`	`}`
`363`		`-#endif`
`364`	`362`	`break;`
`365`	`363`	`}`
`366`	`364`	`}`