Changeset 6385 in josm


Ignore:
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
Files:
4 edited
1 moved

Legend:

Unmodified
Added
Removed
  • trunk/data/defaultpresets.xml

    r6358 r6385  
    52445244            <combo key="opening_hours" text="Operation times" values="Mo-Fr 22:00-05:00" />                                                                                       
    52455245        </item>
    5246         <item name="Measurement Station" icon="presets/measurement_station.png" type="node,closedway">
    5247             <link href="http://wiki.openstreetmap.org/wiki/Proposed_features/measurement_station" />
    5248             <label text="Edit Measurement Station" />
     5246        <item name="Monitoring Station" icon="presets/monitoring_station.png" type="node,closedway">
     5247            <link href="http://wiki.openstreetmap.org/wiki/Tag:man_made=monitoring_station"
     5248                  pt_BR.href="http://wiki.openstreetmap.org/wiki/Pt-br:Tag:man_made=monitoring_station"
     5249                  ru.href="http://wiki.openstreetmap.org/wiki/RU:Tag:man_made=monitoring_station" />
     5250            <label text="Edit Monitoring Station" />
    52495251            <space />           
    5250             <key key="man_made" value="measurement_station" />
     5252            <key key="man_made" value="monitoring_station" />
    52515253            <text key="name" text="Name" />
    52525254            <text key="operator" text="Operator" />             
    5253             <combo key="measurement" text="Measurement" values="particulate_matter,seismic,water_level,weather" />
     5255            <label text="Monitoring:" />
     5256            <checkgroup columns="3">
     5257                <check key="monitoring:weather"          text="Weather" />
     5258                <check key="monitoring:water_level"      text="Water level" />
     5259                <check key="monitoring:tide_gauge"       text="Tide Level" />
     5260                <check key="monitoring:traffic"          text="Traffic" />
     5261                <check key="monitoring:air_quality"      text="Air Quality" />
     5262                <check key="monitoring:noise"            text="Noise" />
     5263                <check key="monitoring:gps"              text="GPS Signals" />
     5264                <check key="monitoring:glonass"          text="GLONASS Signals" />
     5265                <check key="monitoring:galileo"          text="Galileo Signals" />
     5266                <check key="monitoring:radiation"        text="Radiation" />
     5267                <check key="monitoring:seismic_activity" text="Seismic Activity" />
     5268            </checkgroup>
    52545269            <combo key="recording" text="Recording" values="manually,automated,remote,no" /> 
    52555270            <combo key="display" text="Display" values="analog,digital,no" />
  • trunk/src/org/openstreetmap/josm/data/validation/tests/DeprecatedTags.java

    r6346 r6385  
    9494                testAndRemove("amenity", "emergency_phone").
    9595                add("emergency", "phone"));
    96         // http://wiki.openstreetmap.org/wiki/Tag:sport=gaelic_football
    97         // fix #8132
     96        // fix #8132 - http://wiki.openstreetmap.org/wiki/Tag:sport=gaelic_football
    9897        checks.add(new DeprecationCheck(2117).
    9998                testAndRemove("sport", "gaelic_football").
    10099                add("sport", "gaelic_games"));
    101         // http://wiki.openstreetmap.org/wiki/Tag:power=station
    102         // see #8847 / #8961
     100        // see #8847 / #8961 - http://wiki.openstreetmap.org/wiki/Tag:power=station
    103101        checks.add(new DeprecationCheck(2118).
    104102                test("power", "station").
     
    114112                testAndRemove("generator:method", "pumping").
    115113                add("generator:method", "water-pumped-storage"));
    116         // http://wiki.openstreetmap.org/wiki/Key:fence_type
    117         // see #8962
     114        // see #8962 - http://wiki.openstreetmap.org/wiki/Key:fence_type
    118115        checks.add(new DeprecationCheck(2122).
    119116                test("fence_type", "chain").
    120117                alternative("barrier", "chain").
    121118                alternative("fence_type", "chain_link"));
    122         // http://wiki.openstreetmap.org/wiki/Key:entrance
    123         // see #9000
     119        // see #9000 - http://wiki.openstreetmap.org/wiki/Key:entrance
    124120        checks.add(new DeprecationCheck(2123).
    125121                test("building", "entrance").
    126122                alternative("entrance"));
    127         // Useless tag proposed in internal preset for years
    128         // see #9213
     123        // see #9213 - Useless tag proposed in internal preset for years
    129124        checks.add(new DeprecationCheck(2124).
    130125                testAndRemove("board_type", "board"));
     126        // see #8434 - http://wiki.openstreetmap.org/wiki/Proposed_features/monitoring_station
     127        checks.add(new DeprecationCheck(2125).
     128                testAndRemove("man_made", "measurement_station").
     129                add("man_made", "monitoring_station"));
     130        checks.add(new DeprecationCheck(2126).
     131                testAndRemove("measurement", "water_level").
     132                add("monitoring:water_level", "yes"));
     133        checks.add(new DeprecationCheck(2127).
     134                testAndRemove("measurement", "weather").
     135                add("monitoring:weather", "yes"));
     136        checks.add(new DeprecationCheck(2128).
     137                testAndRemove("measurement", "seismic").
     138                add("monitoring:seismic_activity", "yes"));
     139        checks.add(new DeprecationCheck(2129).
     140                test("monitoring:river_level").
     141                alternative("monitoring:water_level"));
    131142    }
    132143
  • trunk/src/org/openstreetmap/josm/data/validation/tests/TagChecker.java

    r6354 r6385  
    5757import org.openstreetmap.josm.gui.tagging.TaggingPreset;
    5858import org.openstreetmap.josm.gui.tagging.TaggingPresetItem;
     59import org.openstreetmap.josm.gui.tagging.TaggingPresetItems.Check;
     60import org.openstreetmap.josm.gui.tagging.TaggingPresetItems.CheckGroup;
    5961import org.openstreetmap.josm.gui.tagging.TaggingPresetItems.KeyedItem;
    6062import org.openstreetmap.josm.io.MirroredInputStream;
     
    6870 * @author frsantos
    6971 */
    70 public class TagChecker extends Test
    71 {
     72public class TagChecker extends Test {
     73   
    7274    /** The default data files */
    7375    public static final String DATA_FILE = "resource://data/tagchecker.cfg";
     
    307309                for (TaggingPresetItem i : p.data) {
    308310                    if (i instanceof KeyedItem) {
    309                         KeyedItem ky = (KeyedItem) i;
    310                         if (ky.key != null && ky.getValues() != null) {
    311                             try {
    312                                 presetsValueData.putAll(ky.key, ky.getValues());
    313                             } catch (NullPointerException e) {
    314                                 Main.error(p+": Unable to initialize "+ky);
    315                             }
     311                        addPresetValue(p, (KeyedItem) i);
     312                    } else if (i instanceof CheckGroup) {
     313                        for (Check c : ((CheckGroup) i).checks) {
     314                            addPresetValue(p, c);
    316315                        }
    317316                    }
    318317                }
     318            }
     319        }
     320    }
     321
     322    private static void addPresetValue(TaggingPreset p, KeyedItem ky) {
     323        if (ky.key != null && ky.getValues() != null) {
     324            try {
     325                presetsValueData.putAll(ky.key, ky.getValues());
     326            } catch (NullPointerException e) {
     327                Main.error(p+": Unable to initialize "+ky);
    319328            }
    320329        }
  • trunk/styles/standard/elemstyles.xml

    r6359 r6385  
    14161416
    14171417        <rule>
    1418                 <condition k="man_made" v="measurement_station"/>
     1418                <condition k="man_made" v="monitoring_station"/>
    14191419                <icon annotate="true" src="misc/landmark/measurement_station.png"/>
    14201420                <area colour="manmade#d8d8d8"/>
Note: See TracChangeset for help on using the changeset viewer.