id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
17108	Should not warn about unclosed way with natural=tree	naoliv	team	"Have an unclosed way with `natural=tree` and validate it.
We can see 2 warnings:

* `natural=tree on a way. Should be used on a node.`
* `Unclosed way - natural type tree`

The first warning is right while the second not.

I am not really sure if this is the most correct solution but it solves the problem:

{{{
#!patch
diff --git a/src/org/openstreetmap/josm/data/validation/tests/UnclosedWays.java b/src/org/openstreetmap/josm/data/validation/tests/UnclosedWays.java
index e44a5b440..cc306c4cb 100644
--- a/src/org/openstreetmap/josm/data/validation/tests/UnclosedWays.java
+++ b/src/org/openstreetmap/josm/data/validation/tests/UnclosedWays.java
@@ -135,7 +135,7 @@ public class UnclosedWays extends Test {
     private static final UnclosedWaysCheck[] checks = {
         // CHECKSTYLE.OFF: SingleSpaceSeparator
         new UnclosedWaysCheck(1101, ""natural"",   marktr(""natural type {0}""),
-                new HashSet<>(Arrays.asList(""cave"", ""coastline"", ""cliff"", ""tree_row"", ""ridge"", ""valley"", ""arete"", ""gorge"", ""gully""))),
+                new HashSet<>(Arrays.asList(""cave"", ""coastline"", ""cliff"", ""tree"", ""tree_row"", ""ridge"", ""valley"", ""arete"", ""gorge"", ""gully""))),
         new UnclosedWaysCheck(1102, ""landuse"",   marktr(""landuse type {0}"")),
         new UnclosedWaysCheck(1103, ""amenities"", marktr(""amenities type {0}"")),
         new UnclosedWaysCheck(1104, ""sport"",     marktr(""sport type {0}""),
}}}"	defect	closed	normal	18.12	Core validator		fixed