Modify

Ticket #5257 (closed enhancement: fixed)

Opened 19 months ago

Last modified 6 months ago

[Patch] Automatic fix to Reversed water

Reported by: e2jk Owned by: team
Priority: normal Component: Plugin validator
Version: tested Keywords:
Cc:

Description

A counter-clockwise closed way with natural=water will generate the following error:

Reversed water: land not on left side

The fix for this error is to reverse the direction of the way. Currently there is no proposed automatic fix, so you need to select all those errors and click on the "Reverse the direction of all selected ways" button.

I believe this could be automatically fixed.

JOSM 3376 Validator 22204

Attachments

5257.patch Download (6.4 KB) - added by simon04 6 months ago.

Change History

comment:1 Changed 6 months ago by simon04

According to the wiki ( http://wiki.openstreetmap.org/wiki/Tag:natural=water), the orientation of natural=water does not matter. So remove testing natural=water?

comment:2 Changed 6 months ago by simon04

Btw: In WronglyOrderedWays there was is a minor bug when determining the orientation of a way, i.e., being (anti)clockwise. The problem is that the pair (last node of way, first node of way) is not taken into account.

The patch fixes this issue and reorganizes some code (e.g., provides Geometry.isClockwise(Way)).

Changed 6 months ago by simon04

comment:3 Changed 6 months ago by Casiope

  • Summary changed from Automatic fix to Reversed water to [Patch] Automatic fix to Reversed water

comment:4 Changed 6 months ago by stoecker

  • Status changed from new to closed
  • Resolution set to fixed

In [4344/josm]:

fix #5257 - patch by simon04 - fix way order validator checks

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.