Correct topology rubber band bug for first and last vertexes

leyan · leyan · commit 2689f99703d3 · 2013-10-26T08:38:46.000+08:00
diff --git a/src/app/nodetool/qgsmaptoolnodetool.cpp b/src/app/nodetool/qgsmaptoolnodetool.cpp
@@ -311,7 +311,7 @@ void QgsMapToolNodeTool::canvasMoveEvent( QMouseEvent * e )
       offset = posMapCoord - mPosMapCoordBackup;
       for ( int i = 0; i < mTopologyRubberBand.size(); i++ )
       {
-        for ( int pointIndex = 0; pointIndex < mTopologyRubberBand[i]->numberOfVertices() - 1; pointIndex++ )
+        for ( int pointIndex = 0; pointIndex < mTopologyRubberBand[i]->numberOfVertices(); pointIndex++ )
         {
           if ( mTopologyRubberBandVertexes[i]->contains( pointIndex ) )
           {
@@ -321,10 +321,6 @@ void QgsMapToolNodeTool::canvasMoveEvent( QMouseEvent * e )
               break;
             }
             mTopologyRubberBand[i]->movePoint( pointIndex, *point + offset );
-            if ( pointIndex == 0 )
-            {
-              mTopologyRubberBand[i]->movePoint( pointIndex, *point + offset );
-            }
           }
         }
       }

Original file line number	Diff line number	Diff line change
`@@ -311,7 +311,7 @@ void QgsMapToolNodeTool::canvasMoveEvent( QMouseEvent * e )`
`311`	`311`	`offset = posMapCoord - mPosMapCoordBackup;`
`312`	`312`	`for ( int i = 0; i < mTopologyRubberBand.size(); i++ )`
`313`	`313`	`{`
`314`		`- for ( int pointIndex = 0; pointIndex < mTopologyRubberBand[i]->numberOfVertices() - 1; pointIndex++ )`
	`314`	`+ for ( int pointIndex = 0; pointIndex < mTopologyRubberBand[i]->numberOfVertices(); pointIndex++ )`
`315`	`315`	`{`
`316`	`316`	`if ( mTopologyRubberBandVertexes[i]->contains( pointIndex ) )`
`317`	`317`	`{`
`@@ -321,10 +321,6 @@ void QgsMapToolNodeTool::canvasMoveEvent( QMouseEvent * e )`
`321`	`321`	`break;`
`322`	`322`	`}`
`323`	`323`	`mTopologyRubberBand[i]->movePoint( pointIndex, *point + offset );`
`324`		`- if ( pointIndex == 0 )`
`325`		`- {`
`326`		`- mTopologyRubberBand[i]->movePoint( pointIndex, *point + offset );`
`327`		`- }`
`328`	`324`	`}`
`329`	`325`	`}`
`330`	`326`	`}`