Changeset 6385 in josm for trunk/src

Timestamp:

2013-11-15T01:43:48+01:00 (11 years ago)

Author:

Don-vip

Message:

fix #8434 - Full support of man_made=monitoring_station instead of man_made=measurement_station:

• rename icon
• update presets
• update map style
• deprecate old tags
• fix a bug in tagchecker where keys defined inside checkgroups were not known from validator

Location:

trunk/src/org/openstreetmap/josm/data/validation/tests

Files:

• DeprecatedTags.java (modified) (2 diffs)
• TagChecker.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/data/validation/tests/DeprecatedTags.java

@@ -94 +94 @@
                 testAndRemove("amenity", "emergency_phone").
                 add("emergency", "phone"));
-        // http://wiki.openstreetmap.org/wiki/Tag:sport=gaelic_football
-        // fix #8132
+        // fix #8132 - http://wiki.openstreetmap.org/wiki/Tag:sport=gaelic_football
         checks.add(new DeprecationCheck(2117).
                 testAndRemove("sport", "gaelic_football").
                 add("sport", "gaelic_games"));
-        // http://wiki.openstreetmap.org/wiki/Tag:power=station
-        // see #8847 / #8961
+        // see #8847 / #8961 - http://wiki.openstreetmap.org/wiki/Tag:power=station
         checks.add(new DeprecationCheck(2118).
                 test("power", "station").
@@ -114 +112 @@
                 testAndRemove("generator:method", "pumping").
                 add("generator:method", "water-pumped-storage"));
-        // http://wiki.openstreetmap.org/wiki/Key:fence_type
-        // see #8962
+        // see #8962 - http://wiki.openstreetmap.org/wiki/Key:fence_type
         checks.add(new DeprecationCheck(2122).
                 test("fence_type", "chain").
                 alternative("barrier", "chain").
                 alternative("fence_type", "chain_link"));
-        // http://wiki.openstreetmap.org/wiki/Key:entrance
-        // see #9000
+        // see #9000 - http://wiki.openstreetmap.org/wiki/Key:entrance
         checks.add(new DeprecationCheck(2123).
                 test("building", "entrance").
                 alternative("entrance"));
-        // Useless tag proposed in internal preset for years
-        // see #9213
+        // see #9213 - Useless tag proposed in internal preset for years
         checks.add(new DeprecationCheck(2124).
                 testAndRemove("board_type", "board"));
+        // see #8434 - http://wiki.openstreetmap.org/wiki/Proposed_features/monitoring_station
+        checks.add(new DeprecationCheck(2125).
+                testAndRemove("man_made", "measurement_station").
+                add("man_made", "monitoring_station"));
+        checks.add(new DeprecationCheck(2126).
+                testAndRemove("measurement", "water_level").
+                add("monitoring:water_level", "yes"));
+        checks.add(new DeprecationCheck(2127).
+                testAndRemove("measurement", "weather").
+                add("monitoring:weather", "yes"));
+        checks.add(new DeprecationCheck(2128).
+                testAndRemove("measurement", "seismic").
+                add("monitoring:seismic_activity", "yes"));
+        checks.add(new DeprecationCheck(2129).
+                test("monitoring:river_level").
+                alternative("monitoring:water_level"));
     }
 

trunk/src/org/openstreetmap/josm/data/validation/tests/TagChecker.java

@@ -57 +57 @@
 import org.openstreetmap.josm.gui.tagging.TaggingPreset;
 import org.openstreetmap.josm.gui.tagging.TaggingPresetItem;
+import org.openstreetmap.josm.gui.tagging.TaggingPresetItems.Check;
+import org.openstreetmap.josm.gui.tagging.TaggingPresetItems.CheckGroup;
 import org.openstreetmap.josm.gui.tagging.TaggingPresetItems.KeyedItem;
 import org.openstreetmap.josm.io.MirroredInputStream;
@@ -68 +70 @@
  * @author frsantos
  */
-public class TagChecker extends Test
-{
+public class TagChecker extends Test {
+    
     /** The default data files */
     public static final String DATA_FILE = "resource://data/tagchecker.cfg";
@@ -307 +309 @@
                 for (TaggingPresetItem i : p.data) {
                     if (i instanceof KeyedItem) {
-                        KeyedItem ky = (KeyedItem) i;
-                        if (ky.key != null && ky.getValues() != null) {
-                            try {
-                                presetsValueData.putAll(ky.key, ky.getValues());
-                            } catch (NullPointerException e) {
-                                Main.error(p+": Unable to initialize "+ky);
-                            }
+                        addPresetValue(p, (KeyedItem) i);
+                    } else if (i instanceof CheckGroup) {
+                        for (Check c : ((CheckGroup) i).checks) {
+                            addPresetValue(p, c);
                         }
                     }
                 }
+            }
+        }
+    }
+
+    private static void addPresetValue(TaggingPreset p, KeyedItem ky) {
+        if (ky.key != null && ky.getValues() != null) {
+            try {
+                presetsValueData.putAll(ky.key, ky.getValues());
+            } catch (NullPointerException e) {
+                Main.error(p+": Unable to initialize "+ky);
             }
         }