uDIG

CLONE -Can't delete hole from polygon with Remove Vertex tool

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: UDIG 1.4.1
  • Component/s: tools and editing
  • Labels:
    None
  • Environment:
    Win 2K, Java 5

Description

I'm trying to remove a hole from a polygon vertex-by-vertex using the Remove Vertex tool. After I remove all the vertices of the hole I get the following stack trace:

java.lang.IllegalArgumentException: holes must not contain null elements
at com.vividsolutions.jts.geom.Polygon.<init>(Polygon.java:127)
at com.vividsolutions.jts.geom.GeometryFactory.createPolygon(GeometryFactory.java:383)
at net.refractions.udig.tools.edit.support.GeometryCreationUtil.createPolygon(GeometryCreationUtil.java:227)
at net.refractions.udig.tools.edit.support.GeometryCreationUtil.createGeom(GeometryCreationUtil.java:170)
at net.refractions.udig.tools.edit.support.GeometryCreationUtil.createAllGeoms(GeometryCreationUtil.java:99)
at net.refractions.udig.tools.edit.behaviour.WriteChangesBehaviour.getCommand(WriteChangesBehaviour.java:152)
at net.refractions.udig.tools.edit.MutualExclusiveBehavior.getCommand(MutualExclusiveBehavior.java:66)
at net.refractions.udig.tools.edit.BehaviourCommand.execute(BehaviourCommand.java:58)
at net.refractions.udig.project.command.CommandManager$Executor.execute(CommandManager.java:394)
at net.refractions.udig.project.command.CommandManager$Executor.run(CommandManager.java:325)
at net.refractions.udig.project.command.CommandManager$Executor.run(CommandManager.java:311)
at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Kaarina Kruus-Blomgren added a comment -
I can reproduce this with ugid 1.2-M9.
Show
Kaarina Kruus-Blomgren added a comment - I can reproduce this with ugid 1.2-M9.
Hide
Permalink
Jody Garnett added a comment -
Interesting; do we need a special remove hole tool? I would like to ask Jesse what the intended workflow is before making any change.


It looks like the workflow being tried here is vertices of the hole being removed one by one until the hole is removed?

This is troubled because a hole should have at minimum 3 vertices to forma valid linear ring....
Show
Jody Garnett added a comment - Interesting; do we need a special remove hole tool? I would like to ask Jesse what the intended workflow is before making any change. It looks like the workflow being tried here is vertices of the hole being removed one by one until the hole is removed? This is troubled because a hole should have at minimum 3 vertices to forma valid linear ring....
Jody Garnett made changes -
Field Original Value New Value
Fix Version/s UDIG 1.2.0 [ 16264 ]
Fix Version/s UDIG 1.1.x [ 15093 ]
Jody Garnett made changes -
Fix Version/s UDIG 1.2.1 [ 16440 ]
Fix Version/s UDIG 1.2.0 [ 16264 ]
Jody Garnett made changes -
Fix Version/s UDIG 1.2.x [ 15072 ]
Fix Version/s UDIG 1.2.1 [ 16440 ]
Jody Garnett made changes -
Fix Version/s UDIG 1.2.3 [ 17485 ]
Fix Version/s UDIG 1.2.2 [ 15072 ]
Jody Garnett made changes -
Fix Version/s uDig 1.3.0 [ 17860 ]
Fix Version/s UDIG 1.2.3 [ 17485 ]
Jody Garnett made changes -
Fix Version/s UDIG 1.2.3 [ 17485 ]
Fix Version/s uDig 1.3.0 [ 17860 ]
Jody Garnett made changes -
Fix Version/s UDIG 1.3.1 [ 18149 ]
Fix Version/s UDIG 1.3.0 [ 17485 ]
Jody Garnett made changes -
Fix Version/s UDIG 1.3.2 [ 18235 ]
Fix Version/s UDIG 1.3.1 [ 18149 ]
Jody Garnett made changes -
Fix Version/s UDIG 1.3.3 [ 18773 ]
Fix Version/s UDIG 1.3.2 [ 18235 ]
Jody Garnett made changes -
Fix Version/s UDIG 1.4.1 [ 19165 ]
Fix Version/s UDIG 1.4.0 [ 18773 ]

People

  • Assignee:
    Silvia Franceschi
    Reporter:
    Kaarina Kruus-Blomgren
Vote (0)
Watch (0)

Dates

  • Created:
    Updated: