https://issues.qgis.org/https://issues.qgis.org/favicon.ico2015-10-24T23:45:01ZQGIS Issue TrackingQGIS Application - Bug report #13661: Node tool should not allow to delete nodes and leave just 1 in lines or two in polygonshttps://issues.qgis.org/issues/13661?journal_id=648462015-10-24T23:45:01ZSaber Razmjooeisaber.razmjooei@lutraconsulting.co.uk
<ul><li><strong>Category</strong> changed from <i>Vectors</i> to <i>Digitising</i></li></ul> QGIS Application - Bug report #13661: Node tool should not allow to delete nodes and leave just 1 in lines or two in polygonshttps://issues.qgis.org/issues/13661?journal_id=651992015-11-07T09:14:22ZGiovanni Manghigiovanni.manghi@gmail.com
<ul><li><strong>Subject</strong> changed from <i>Shapefile Editing</i> to <i>Node tool should not allow to delete nodes and leave just 1 in lines or two in polygons</i></li><li><strong>Operating System</strong> deleted (<del><i>Windows 7 </i></del>)</li><li><strong>OS version</strong> deleted (<del><i>64 bit</i></del>)</li></ul><p>by the way, with polygons is possible to save one with just two nodes (after deleting the others with the node tool), creating polygon with area = 0.</p> QGIS Application - Bug report #13661: Node tool should not allow to delete nodes and leave just 1 in lines or two in polygonshttps://issues.qgis.org/issues/13661?journal_id=653152015-11-08T02:19:08ZJürgen Fischerjef@norbit.de
<ul></ul><p>Duplicate of <a class="issue tracker-1 status-5 priority-10 priority- closed" href="https://issues.qgis.org/issues/13099" title="[regression] nodetool cannot remove parts and rings or full geometries by removing all vertices (Closed)">#13099</a>?</p> QGIS Application - Bug report #13661: Node tool should not allow to delete nodes and leave just 1 in lines or two in polygonshttps://issues.qgis.org/issues/13661?journal_id=653412015-11-08T11:30:47ZNyall Dawson
<ul></ul><p>I think the correct behaviour should still be to allow <3 nodes in polygons and 1 nodes for lines as a temporary step (see <a class="issue tracker-1 status-5 priority-10 priority- closed" href="https://issues.qgis.org/issues/13099" title="[regression] nodetool cannot remove parts and rings or full geometries by removing all vertices (Closed)">#13099</a> for reasoning), but then catch these invalid geometries when node tool operation is finalised for the feature.</p> QGIS Application - Bug report #13661: Node tool should not allow to delete nodes and leave just 1 in lines or two in polygonshttps://issues.qgis.org/issues/13661?journal_id=653992015-11-10T01:53:58ZGiovanni Manghigiovanni.manghi@gmail.com
<ul></ul><p>Nyall Dawson wrote:</p>
<blockquote>
<p>I think the correct behaviour should still be to allow <3 nodes in polygons and 1 nodes for lines as a temporary step (see <a class="issue tracker-1 status-5 priority-10 priority- closed" href="https://issues.qgis.org/issues/13099" title="[regression] nodetool cannot remove parts and rings or full geometries by removing all vertices (Closed)">#13099</a> for reasoning), but then catch these invalid geometries when node tool operation is finalised for the feature.</p>
</blockquote>
<p>Hi Nyall, I see your point, but anyway as it is right now in master is a bit confusing. Take the polygon example: the user deletes all but two nodes, as soon as the focus of the node tool switches to another polygon, the two noded one disappear... but after saving edits it re-appear (as a "line"). Cheers!</p> QGIS Application - Bug report #13661: Node tool should not allow to delete nodes and leave just 1 in lines or two in polygonshttps://issues.qgis.org/issues/13661?journal_id=654002015-11-10T02:09:04ZJürgen Fischerjef@norbit.de
<ul></ul><p>Giovanni Manghi wrote:</p>
<blockquote>
<p>Hi Nyall, I see your point, but anyway as it is right now in master is a bit confusing. Take the polygon example: the user deletes all but two nodes, as soon as the focus of the node tool switches to another polygon, the two noded one disappear... but after saving edits it re-appear (as a "line"). Cheers!</p>
</blockquote>
<p>Right, earlier removing the last point that would make a ring or line invalid (in terms of number of points) in the node tool would also remove the other remaining points.</p> QGIS Application - Bug report #13661: Node tool should not allow to delete nodes and leave just 1 in lines or two in polygonshttps://issues.qgis.org/issues/13661?journal_id=654022015-11-10T02:13:23ZNyall Dawson
<ul></ul><p>Jürgen Fischer wrote:</p>
<blockquote>
<p>Giovanni Manghi wrote:</p>
<blockquote>
<p>Hi Nyall, I see your point, but anyway as it is right now in master is a bit confusing. Take the polygon example: the user deletes all but two nodes, as soon as the focus of the node tool switches to another polygon, the two noded one disappear... but after saving edits it re-appear (as a "line"). Cheers!</p>
</blockquote>
<p>Right, earlier removing the last point that would make a ring or line invalid (in terms of number of points) in the node tool would also remove the other remaining points.</p>
</blockquote>
<p>Ah ok - that makes a lot more sense.</p> QGIS Application - Bug report #13661: Node tool should not allow to delete nodes and leave just 1 in lines or two in polygonshttps://issues.qgis.org/issues/13661?journal_id=654032015-11-10T02:24:02ZSaber Razmjooeisaber.razmjooei@lutraconsulting.co.uk
<ul></ul><p>We also had a similiar discussion here <a class="issue tracker-1 status-5 priority-10 priority- closed" href="https://issues.qgis.org/issues/13674" title="Crash when deleting features with node tool (2.12) (Closed)">#13674</a>.</p> QGIS Application - Bug report #13661: Node tool should not allow to delete nodes and leave just 1 in lines or two in polygonshttps://issues.qgis.org/issues/13661?journal_id=669042016-01-03T22:53:15ZMarco Hugentoblermarco.hugentobler@sourcepole.ch
<ul></ul><p><a class="external" href="https://github.com/qgis/QGIS/pull/2632">https://github.com/qgis/QGIS/pull/2632</a> provides a fix.</p> QGIS Application - Bug report #13661: Node tool should not allow to delete nodes and leave just 1 in lines or two in polygonshttps://issues.qgis.org/issues/13661?journal_id=669162016-01-04T04:03:13ZGiovanni Manghigiovanni.manghi@gmail.com
<ul><li><strong>Resolution</strong> set to <i>fixed/implemented</i></li><li><strong>Status</strong> changed from <i>Open</i> to <i>Closed</i></li></ul>