Skip to content

Commit

Permalink
Merge pull request #5748 from nyalldawson/offset_tool
Browse files Browse the repository at this point in the history
Fix offset curve tool offsets curves in opposite direction to mouse
  • Loading branch information
nyalldawson authored Nov 28, 2017
2 parents 721551f + 5c66526 commit 2156e89
Show file tree
Hide file tree
Showing 2 changed files with 54 additions and 55 deletions.
98 changes: 48 additions & 50 deletions src/app/qgsmaptooloffsetcurve.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -25,18 +25,13 @@
#include "qgssnappingconfig.h"
#include "qgssettings.h"
#include "qgisapp.h"
#include "qgsgeos.h"

#include <QGraphicsProxyWidget>
#include <QMouseEvent>
#include "qgisapp.h"

QgsMapToolOffsetCurve::QgsMapToolOffsetCurve( QgsMapCanvas *canvas )
: QgsMapToolEdit( canvas )
, mModifiedFeature( -1 )
, mGeometryModified( false )
, mForceCopy( false )
, mMultiPartGeometry( false )
{
}

Expand All @@ -47,6 +42,19 @@ QgsMapToolOffsetCurve::~QgsMapToolOffsetCurve()
delete mSnapVertexMarker;
}

void QgsMapToolOffsetCurve::keyPressEvent( QKeyEvent *e )
{
if ( e && e->key() == Qt::Key_Escape && !e->isAutoRepeat() )
{
deleteRubberBandAndGeometry();
deleteDistanceWidget();
}
else
{
QgsMapToolEdit::keyPressEvent( e );
}
}


void QgsMapToolOffsetCurve::canvasReleaseEvent( QgsMapMouseEvent *e )
{
Expand All @@ -72,14 +80,10 @@ void QgsMapToolOffsetCurve::canvasReleaseEvent( QgsMapMouseEvent *e )

if ( mOriginalGeometry.isNull() )
{
// first click, get feature to modify
deleteRubberBandAndGeometry();
mGeometryModified = false;
mForceCopy = false;

if ( e->button() == Qt::RightButton )
{
return;
}
mCtrlWasHeldOnFeatureSelection = false;

QgsSnappingUtils *snapping = mCanvas->snappingUtils();

Expand All @@ -106,7 +110,7 @@ void QgsMapToolOffsetCurve::canvasReleaseEvent( QgsMapMouseEvent *e )
QgsFeature fet;
if ( match.layer()->getFeatures( QgsFeatureRequest( match.featureId() ) ).nextFeature( fet ) )
{
mForceCopy = ( e->modifiers() & Qt::ControlModifier ); //no geometry modification if ctrl is pressed
mCtrlWasHeldOnFeatureSelection = ( e->modifiers() & Qt::ControlModifier ); //no geometry modification if ctrl is pressed
mOriginalGeometry = createOriginGeometry( match.layer(), match, fet );
mRubberBand = createRubberBand();
if ( mRubberBand )
Expand All @@ -122,13 +126,15 @@ void QgsMapToolOffsetCurve::canvasReleaseEvent( QgsMapMouseEvent *e )
{
emit messageEmitted( tr( "Could not find a nearby feature in any vector layer." ) );
}
return;
}

applyOffset();
else
{
// second click - apply changes
applyOffset( e->modifiers() & Qt::ControlModifier );
}
}

void QgsMapToolOffsetCurve::applyOffset()
void QgsMapToolOffsetCurve::applyOffset( bool forceCopy )
{
QgsVectorLayer *layer = currentVectorLayer();
if ( !layer )
Expand All @@ -155,7 +161,7 @@ void QgsMapToolOffsetCurve::applyOffset()
layer->beginEditCommand( tr( "Offset curve" ) );

bool editOk;
if ( mSourceLayerId == layer->id() && !mForceCopy )
if ( mSourceLayerId == layer->id() && !mCtrlWasHeldOnFeatureSelection && !forceCopy )
{
editOk = layer->changeGeometry( mModifiedFeature, mModifiedGeometry );
}
Expand Down Expand Up @@ -188,7 +194,7 @@ void QgsMapToolOffsetCurve::applyOffset()
deleteDistanceWidget();
delete mSnapVertexMarker;
mSnapVertexMarker = nullptr;
mForceCopy = false;
mCtrlWasHeldOnFeatureSelection = false;
layer->triggerRepaint();
}

Expand Down Expand Up @@ -248,13 +254,13 @@ void QgsMapToolOffsetCurve::canvasMoveEvent( QgsMapMouseEvent *e )
if ( mDistanceWidget )
{
// this will also set the rubber band
mDistanceWidget->setValue( leftOf < 0 ? offset : -offset );
mDistanceWidget->setValue( leftOf < 0 ? -offset : offset );
mDistanceWidget->setFocus( Qt::TabFocusReason );
}
else
{
//create offset geometry using geos
setOffsetForRubberBand( leftOf < 0 ? offset : -offset );
setOffsetForRubberBand( leftOf < 0 ? -offset : offset );
}
}

Expand All @@ -269,7 +275,7 @@ QgsGeometry QgsMapToolOffsetCurve::createOriginGeometry( QgsVectorLayer *vl, con
//assign feature part by vertex number (snap to vertex) or by before vertex number (snap to segment)
int partVertexNr = match.vertexIndex();

if ( vl == currentVectorLayer() && !mForceCopy )
if ( vl == currentVectorLayer() && !mCtrlWasHeldOnFeatureSelection )
{
//don't consider selected geometries, only the snap result
return convertToSingleLine( snappedFeature.geometry(), partVertexNr, mMultiPartGeometry );
Expand Down Expand Up @@ -333,15 +339,14 @@ void QgsMapToolOffsetCurve::createDistanceWidget()
mDistanceWidget->setFocus( Qt::TabFocusReason );

connect( mDistanceWidget, static_cast < void ( QDoubleSpinBox::* )( double ) > ( &QDoubleSpinBox::valueChanged ), this, &QgsMapToolOffsetCurve::placeOffsetCurveToValue );
connect( mDistanceWidget, &QAbstractSpinBox::editingFinished, this, &QgsMapToolOffsetCurve::applyOffset );
connect( mDistanceWidget, &QAbstractSpinBox::editingFinished, this, [ = ] { applyOffset(); } );
}

void QgsMapToolOffsetCurve::deleteDistanceWidget()
{
if ( mDistanceWidget )
{
disconnect( mDistanceWidget, static_cast < void ( QDoubleSpinBox::* )( double ) > ( &QDoubleSpinBox::valueChanged ), this, &QgsMapToolOffsetCurve::placeOffsetCurveToValue );
disconnect( mDistanceWidget, &QAbstractSpinBox::editingFinished, this, &QgsMapToolOffsetCurve::applyOffset );
mDistanceWidget->releaseKeyboard();
mDistanceWidget->deleteLater();
}
Expand All @@ -368,34 +373,27 @@ void QgsMapToolOffsetCurve::setOffsetForRubberBand( double offset )
return;
}

QgsGeometry geomCopy( mOriginalGeometry );
geos::unique_ptr geosGeom( geomCopy.exportToGeos() );
if ( geosGeom )
{
QgsSettings s;
int joinStyle = s.value( QStringLiteral( "/qgis/digitizing/offset_join_style" ), 0 ).toInt();
int quadSegments = s.value( QStringLiteral( "/qgis/digitizing/offset_quad_seg" ), 8 ).toInt();
double miterLimit = s.value( QStringLiteral( "/qgis/digitizing/offset_miter_limit" ), 5.0 ).toDouble();

geos::unique_ptr offsetGeom( GEOSOffsetCurve_r( QgsGeometry::getGEOSHandler(), geosGeom.get(), offset, quadSegments, joinStyle, miterLimit ) );
if ( !offsetGeom )
{
deleteRubberBandAndGeometry();
deleteDistanceWidget();
delete mSnapVertexMarker;
mSnapVertexMarker = nullptr;
mForceCopy = false;
mGeometryModified = false;
deleteDistanceWidget();
emit messageEmitted( tr( "Creating offset geometry failed" ), QgsMessageBar::CRITICAL );
return;
}
QgsSettings s;
QgsGeometry::JoinStyle joinStyle = static_cast< QgsGeometry::JoinStyle >( s.value( QStringLiteral( "/qgis/digitizing/offset_join_style" ), 0 ).toInt() );
int quadSegments = s.value( QStringLiteral( "/qgis/digitizing/offset_quad_seg" ), 8 ).toInt();
double miterLimit = s.value( QStringLiteral( "/qgis/digitizing/offset_miter_limit" ), 5.0 ).toDouble();

if ( offsetGeom )
{
mModifiedGeometry.fromGeos( offsetGeom.release() );
mRubberBand->setToGeometry( mModifiedGeometry, sourceLayer );
}
QgsGeometry offsetGeom = mOriginalGeometry.offsetCurve( offset, quadSegments, joinStyle, miterLimit );
if ( !offsetGeom )
{
deleteRubberBandAndGeometry();
deleteDistanceWidget();
delete mSnapVertexMarker;
mSnapVertexMarker = nullptr;
mCtrlWasHeldOnFeatureSelection = false;
mGeometryModified = false;
deleteDistanceWidget();
emit messageEmitted( tr( "Creating offset geometry failed: %1" ).arg( offsetGeom.lastError() ), QgsMessageBar::CRITICAL );
}
else
{
mModifiedGeometry = offsetGeom;
mRubberBand->setToGeometry( mModifiedGeometry, sourceLayer );
}
}

Expand Down
11 changes: 6 additions & 5 deletions src/app/qgsmaptooloffsetcurve.h
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ class APP_EXPORT QgsMapToolOffsetCurve: public QgsMapToolEdit
QgsMapToolOffsetCurve( QgsMapCanvas *canvas );
~QgsMapToolOffsetCurve();

void keyPressEvent( QKeyEvent *e ) override;
void canvasReleaseEvent( QgsMapMouseEvent *e ) override;
void canvasMoveEvent( QgsMapMouseEvent *e ) override;

Expand All @@ -39,7 +40,7 @@ class APP_EXPORT QgsMapToolOffsetCurve: public QgsMapToolEdit
void placeOffsetCurveToValue();

//! Apply the offset either from the spin box or from the mouse event
void applyOffset();
void applyOffset( bool forceCopy = false );

private:
//! Rubberband that shows the position of the offset curve
Expand All @@ -49,18 +50,18 @@ class APP_EXPORT QgsMapToolOffsetCurve: public QgsMapToolEdit
//! Geometry after manipulation
QgsGeometry mModifiedGeometry;
//! ID of manipulated feature
QgsFeatureId mModifiedFeature;
QgsFeatureId mModifiedFeature = -1;
//! Layer ID of source layer
QString mSourceLayerId;
//! Internal flag to distinguish move from click
bool mGeometryModified;
bool mGeometryModified = false;
//! Shows current distance value and allows numerical editing
QgsDoubleSpinBox *mDistanceWidget = nullptr;
//! Marker to show the cursor was snapped to another location
QgsVertexMarker *mSnapVertexMarker = nullptr;
//! Forces geometry copy (no modification of geometry in current layer)
bool mForceCopy;
bool mMultiPartGeometry;
bool mCtrlWasHeldOnFeatureSelection = false;
bool mMultiPartGeometry = false;


void deleteRubberBandAndGeometry();
Expand Down

0 comments on commit 2156e89

Please sign in to comment.