Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[vertex tool] rename mSelectedFeature to mLockedFeature
  • Loading branch information
3nids committed Feb 25, 2019
1 parent a887b7d commit 0a2d3ba
Show file tree
Hide file tree
Showing 2 changed files with 33 additions and 34 deletions.
65 changes: 32 additions & 33 deletions src/app/vertextool/qgsvertextool.cpp
Expand Up @@ -484,7 +484,7 @@ void QgsVertexTool::cadCanvasReleaseEvent( QgsMapMouseEvent *e )
if ( mNewVertexFromDoubleClick )
{
QgsPointLocator::Match m( *mNewVertexFromDoubleClick );
if ( mSelectedFeature && ( mSelectedFeature->featureId() != m.featureId() || mSelectedFeature->layer() != m.layer() ) )
if ( mLockedFeature && ( mLockedFeature->featureId() != m.featureId() || mLockedFeature->layer() != m.layer() ) )
return; // when a feature is bound to the vector editor, only process actions on that feature

mNewVertexFromDoubleClick.reset();
Expand All @@ -510,7 +510,6 @@ void QgsVertexTool::cadCanvasReleaseEvent( QgsMapMouseEvent *e )
QgsRectangle map_rect( pt0, pt1 );
QList<Vertex> vertices;
QList<Vertex> selectedVertices;
QList<Vertex> editorVertices;

// the logic for selecting vertices using rectangle:
// - if we have a bound (locked) feature, we only allow selection of its vertices
Expand All @@ -526,15 +525,15 @@ void QgsVertexTool::cadCanvasReleaseEvent( QgsMapMouseEvent *e )
if ( mMode == ActiveLayer && vlayer != currentVectorLayer() )
continue;

if ( mSelectedFeature && mSelectedFeature->layer() != vlayer )
if ( mLockedFeature && mLockedFeature->layer() != vlayer )
continue; // with locked feature we only allow selection of its vertices

QgsRectangle layerRect = toLayerCoordinates( vlayer, map_rect );
QgsFeature f;
QgsFeatureIterator fi = vlayer->getFeatures( QgsFeatureRequest( layerRect ).setNoAttributes() );
while ( fi.nextFeature( f ) )
{
if ( mSelectedFeature && mSelectedFeature->featureId() != f.id() )
if ( mLockedFeature && mLockedFeature->featureId() != f.id() )
continue; // with locked feature we only allow selection of its vertices

bool isFeatureSelected = vlayer->selectedFeatureIds().contains( f.id() );
Expand All @@ -554,7 +553,7 @@ void QgsVertexTool::cadCanvasReleaseEvent( QgsMapMouseEvent *e )
}

// here's where we give precedence to vertices of selected features in case there's no bound (locked) feature
if ( !mSelectedFeature && !selectedVertices.isEmpty() )
if ( !mLockedFeature && !selectedVertices.isEmpty() )
{
vertices = selectedVertices;
}
Expand Down Expand Up @@ -987,7 +986,7 @@ void QgsVertexTool::tryToSelectFeature( QgsMapMouseEvent *e )
{
// there's really nothing under the cursor or while cycling through the list of available features
// we got to the end of the list - let's deselect any feature we may have had selected
mSelectedFeature.reset();
mLockedFeature.reset();
if ( mVertexEditor )
{
mVertexEditor->updateEditor( nullptr );
Expand Down Expand Up @@ -1064,7 +1063,7 @@ void QgsVertexTool::mouseMoveNotDragging( QgsMapMouseEvent *e )

// do not use snap from mouse event, use our own with any editable layer
QgsPointLocator::Match m = snapToEditableLayer( e );
bool targetIsAllowed = ( !mSelectedFeature || ( mSelectedFeature->featureId() == m.featureId() && mSelectedFeature->layer() == m.layer() ) );
bool targetIsAllowed = ( !mLockedFeature || ( mLockedFeature->featureId() == m.featureId() && mLockedFeature->layer() == m.layer() ) );

// possibility to move a vertex
if ( m.type() == QgsPointLocator::Vertex && targetIsAllowed )
Expand Down Expand Up @@ -1160,7 +1159,7 @@ void QgsVertexTool::mouseMoveNotDragging( QgsMapMouseEvent *e )

// when we are "locked" to a feature, we don't want to highlight any other features
// so the user does not get distracted
updateFeatureBand( mSelectedFeature ? QgsPointLocator::Match() : m );
updateFeatureBand( mLockedFeature ? QgsPointLocator::Match() : m );
}

void QgsVertexTool::updateVertexBand( const QgsPointLocator::Match &m )
Expand Down Expand Up @@ -1298,8 +1297,8 @@ void QgsVertexTool::onCachedGeometryChanged( QgsFeatureId fid, const QgsGeometry
// re-run validation for the feature
validateGeometry( layer, fid );

if ( mVertexEditor && mSelectedFeature && mSelectedFeature->featureId() == fid && mSelectedFeature->layer() == layer )
mVertexEditor->updateEditor( mSelectedFeature.get() );
if ( mVertexEditor && mLockedFeature && mLockedFeature->featureId() == fid && mLockedFeature->layer() == layer )
mVertexEditor->updateEditor( mLockedFeature.get() );
}

void QgsVertexTool::onCachedGeometryDeleted( QgsFeatureId fid )
Expand All @@ -1318,32 +1317,32 @@ void QgsVertexTool::updateVertexEditor( QgsVectorLayer *layer, QgsFeatureId fid
{
if ( layer )
{
if ( mSelectedFeature && mSelectedFeature->featureId() == fid && mSelectedFeature->layer() == layer )
if ( mLockedFeature && mLockedFeature->featureId() == fid && mLockedFeature->layer() == layer )
{
// if show feature is called on a feature that's already binded to the vertex editor, toggle it off
mSelectedFeature.reset();
mLockedFeature.reset();
if ( mVertexEditor )
{
mVertexEditor->updateEditor( nullptr );
}
return;
}

mSelectedFeature.reset( new QgsSelectedFeature( fid, layer, mCanvas ) );
connect( mSelectedFeature->layer(), &QgsVectorLayer::featureDeleted, this, &QgsVertexTool::cleanEditor );
mLockedFeature.reset( new QgsSelectedFeature( fid, layer, mCanvas ) );
connect( mLockedFeature->layer(), &QgsVectorLayer::featureDeleted, this, &QgsVertexTool::cleanEditor );
for ( int i = 0; i < mSelectedVertices.length(); ++i )
{
if ( mSelectedVertices.at( i ).layer == layer && mSelectedVertices.at( i ).fid == fid )
{
mSelectedFeature->selectVertex( mSelectedVertices.at( i ).vertexId );
mLockedFeature->selectVertex( mSelectedVertices.at( i ).vertexId );
}
}
}

// make sure the vertex editor is alive and visible
showVertexEditor(); //#spellok

mVertexEditor->updateEditor( mSelectedFeature.get() );
mVertexEditor->updateEditor( mLockedFeature.get() );
}

void QgsVertexTool::showVertexEditor() //#spellok
Expand All @@ -1369,7 +1368,7 @@ void QgsVertexTool::showVertexEditor() //#spellok

void QgsVertexTool::cleanupVertexEditor()
{
mSelectedFeature.reset();
mLockedFeature.reset();
mVertexEditor.reset();
}

Expand All @@ -1394,18 +1393,18 @@ static void _safeSelectVertex( QgsSelectedFeature &selectedFeature, int vertexNr

void QgsVertexTool::deleteVertexEditorSelection()
{
if ( !mSelectedFeature )
if ( !mLockedFeature )
return;

int firstSelectedIndex = _firstSelectedVertex( *mSelectedFeature );
int firstSelectedIndex = _firstSelectedVertex( *mLockedFeature );
if ( firstSelectedIndex == -1 )
return;

// make a list of selected vertices
QList<Vertex> vertices;
QList<QgsVertexEntry *> &selFeatureVertices = mSelectedFeature->vertexMap();
QgsVectorLayer *layer = mSelectedFeature->layer();
QgsFeatureId fid = mSelectedFeature->featureId();
QList<QgsVertexEntry *> &selFeatureVertices = mLockedFeature->vertexMap();
QgsVectorLayer *layer = mLockedFeature->layer();
QgsFeatureId fid = mLockedFeature->featureId();
QgsGeometry geometry = cachedGeometry( layer, fid );
for ( QgsVertexEntry *vertex : qgis::as_const( selFeatureVertices ) )
{
Expand All @@ -1421,18 +1420,18 @@ void QgsVertexTool::deleteVertexEditorSelection()
setHighlightedVertices( vertices );
deleteVertex();

if ( !mSelectedFeature->geometry()->isNull() )
if ( !mLockedFeature->geometry()->isNull() )
{
int nextVertexToSelect = firstSelectedIndex;
if ( mSelectedFeature->geometry()->type() == QgsWkbTypes::LineGeometry )
if ( mLockedFeature->geometry()->type() == QgsWkbTypes::LineGeometry )
{
// for lines we don't wrap around vertex selection when deleting vertices from end of line
nextVertexToSelect = std::min( nextVertexToSelect, mSelectedFeature->geometry()->constGet()->nCoordinates() - 1 );
nextVertexToSelect = std::min( nextVertexToSelect, mLockedFeature->geometry()->constGet()->nCoordinates() - 1 );
}

_safeSelectVertex( *mSelectedFeature, nextVertexToSelect );
_safeSelectVertex( *mLockedFeature, nextVertexToSelect );
}
mSelectedFeature->layer()->triggerRepaint();
mLockedFeature->layer()->triggerRepaint();
}


Expand All @@ -1448,7 +1447,7 @@ void QgsVertexTool::startDragging( QgsMapMouseEvent *e )
QgsPointLocator::Match m = snapToEditableLayer( e );
if ( !m.isValid() )
return;
if ( mSelectedFeature && ( mSelectedFeature->featureId() != m.featureId() || mSelectedFeature->layer() != m.layer() ) )
if ( mLockedFeature && ( mLockedFeature->featureId() != m.featureId() || mLockedFeature->layer() != m.layer() ) )
return; // when a feature is bound to the vertex editor, only process actions for that feature

// activate advanced digitizing dock
Expand Down Expand Up @@ -1965,7 +1964,7 @@ void QgsVertexTool::moveVertex( const QgsPointXY &mapPoint, const QgsPointLocato
}

if ( mVertexEditor )
mVertexEditor->updateEditor( mSelectedFeature.get() );
mVertexEditor->updateEditor( mLockedFeature.get() );

setHighlightedVertices( mSelectedVertices ); // update positions of existing highlighted vertices
setHighlightedVerticesVisible( true ); // time to show highlighted vertices again
Expand Down Expand Up @@ -2060,7 +2059,7 @@ void QgsVertexTool::applyEditsToLayers( QgsVertexTool::VertexEdits &edits )
layer->triggerRepaint();

if ( mVertexEditor )
mVertexEditor->updateEditor( mSelectedFeature.get() );
mVertexEditor->updateEditor( mLockedFeature.get() );
}
}

Expand Down Expand Up @@ -2220,8 +2219,8 @@ void QgsVertexTool::deleteVertex()
}
}

if ( mVertexEditor && mSelectedFeature )
mVertexEditor->updateEditor( mSelectedFeature.get() );
if ( mVertexEditor && mLockedFeature )
mVertexEditor->updateEditor( mLockedFeature.get() );
}

void QgsVertexTool::setHighlightedVertices( const QList<Vertex> &listVertices, HighlightMode mode )
Expand Down Expand Up @@ -2676,7 +2675,7 @@ void QgsVertexTool::stopRangeVertexSelection()

void QgsVertexTool::cleanEditor( QgsFeatureId id )
{
if ( mSelectedFeature && mSelectedFeature->featureId() == id )
if ( mLockedFeature && mLockedFeature->featureId() == id )
{
cleanupVertexEditor();
};
Expand Down
2 changes: 1 addition & 1 deletion src/app/vertextool/qgsvertextool.h
Expand Up @@ -436,7 +436,7 @@ class APP_EXPORT QgsVertexTool : public QgsMapToolAdvancedDigitizing
// support for vertex editor

//! Selected feature for the vertex editor
std::unique_ptr<QgsSelectedFeature> mSelectedFeature;
std::unique_ptr<QgsSelectedFeature> mLockedFeature;
//! Dock widget which allows editing vertices
std::unique_ptr<QgsVertexEditor> mVertexEditor;

Expand Down

0 comments on commit 0a2d3ba

Please sign in to comment.