Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Disable message bar feedback from geometryvalidator
  • Loading branch information
m-kuhn committed Oct 15, 2018
1 parent 91cd1ed commit ed27442
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 22 deletions.
19 changes: 1 addition & 18 deletions src/gui/qgsmaptoolcapture.cpp
Expand Up @@ -81,16 +81,6 @@ void QgsMapToolCapture::deactivate()
QgsMapToolAdvancedDigitizing::deactivate();
}

void QgsMapToolCapture::validationFinished()
{
emit messageDiscarded();
QString msgFinished = tr( "Validation finished" );
if ( !mValidationWarnings.isEmpty() )
{
emit messageEmitted( mValidationWarnings.join( QStringLiteral( "\n" ) ).append( "\n" ).append( msgFinished ), Qgis::Warning );
}
}

void QgsMapToolCapture::currentLayerChanged( QgsMapLayer *layer )
{
if ( !mCaptureModeFromLayer )
Expand Down Expand Up @@ -688,7 +678,6 @@ void QgsMapToolCapture::validateGeometry()
mValidator = nullptr;
}

mValidationWarnings.clear();
mGeomErrors.clear();
while ( !mGeomErrorMarkers.isEmpty() )
{
Expand Down Expand Up @@ -726,20 +715,17 @@ void QgsMapToolCapture::validateGeometry()
method = QgsGeometry::ValidatorGeos;
mValidator = new QgsGeometryValidator( geom, nullptr, method );
connect( mValidator, &QgsGeometryValidator::errorFound, this, &QgsMapToolCapture::addError );
connect( mValidator, &QThread::finished, this, &QgsMapToolCapture::validationFinished );
mValidator->start();
QgsDebugMsgLevel( "Validation started", 4 );
}

void QgsMapToolCapture::addError( QgsGeometry::Error e )
void QgsMapToolCapture::addError( const QgsGeometry::Error &e )
{
mGeomErrors << e;
QgsVectorLayer *vlayer = qobject_cast<QgsVectorLayer *>( mCanvas->currentLayer() );
if ( !vlayer )
return;

mValidationWarnings << e.what();

if ( e.hasWhere() )
{
QgsVertexMarker *vm = new QgsVertexMarker( mCanvas );
Expand All @@ -751,9 +737,6 @@ void QgsMapToolCapture::addError( QgsGeometry::Error e )
vm->setZValue( vm->zValue() + 1 );
mGeomErrorMarkers << vm;
}

emit messageDiscarded();
emit messageEmitted( mValidationWarnings.join( QStringLiteral( "\n" ) ), Qgis::Warning );
}

int QgsMapToolCapture::size()
Expand Down
5 changes: 1 addition & 4 deletions src/gui/qgsmaptoolcapture.h
Expand Up @@ -107,11 +107,9 @@ class GUI_EXPORT QgsMapToolCapture : public QgsMapToolAdvancedDigitizing
void clean() override;

private slots:
void validationFinished();
void addError( QgsGeometry::Error );
void addError( const QgsGeometry::Error &error );
void currentLayerChanged( QgsMapLayer *layer );


protected:

/**
Expand Down Expand Up @@ -264,7 +262,6 @@ class GUI_EXPORT QgsMapToolCapture : public QgsMapToolAdvancedDigitizing
QList<QgsPointLocator::Match> mSnappingMatches;

void validateGeometry();
QStringList mValidationWarnings;
QgsGeometryValidator *mValidator = nullptr;
QList< QgsGeometry::Error > mGeomErrors;
QList< QgsVertexMarker * > mGeomErrorMarkers;
Expand Down

0 comments on commit ed27442

Please sign in to comment.