Opened 7 years ago
Closed 6 years ago
#17370 closed enhancement (fixed)
[PATCH] add deprecated tags from Osmose checks
| Reported by: | abalosc1 | Owned by: | Klumbumbus |
|---|---|---|---|
| Priority: | normal | Milestone: | 19.08 |
| Component: | Core validator | Version: | |
| Keywords: | sotmfr2019 osmose | Cc: |
Description
Attachments (2)
Change History (20)
comment:1 by , 7 years ago
| Summary: | add deprecated tags from Osmose checks → [PATCH] add deprecated tags from Osmose checks |
|---|
comment:2 by , 7 years ago
| Component: | External rule → Core validator |
|---|---|
| Type: | task → enhancement |
comment:3 by , 7 years ago
| Milestone: | → 19.02 |
|---|
comment:4 by , 7 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → needinfo |
comment:5 by , 7 years ago
I'm not in favor to add checks, which are not really needed (for performance reasons). E.g.
- man_made=mdf is used 5 times
- amenity=dry_cleaning is used 3 times
and they were added only about once per year in the last years (see taghistory).
Also *[wood] produces double warnings with already existing wood=* warnings.
The changes with tr() messed up the patch file too.
comment:6 by , 7 years ago
| Milestone: | 19.02 → 19.03 |
|---|
by , 7 years ago
| Attachment: | deprecatedmapcss.2.patch added |
|---|
removed wood, mdf, and dry cleaning from added osmose checks
comment:7 by , 7 years ago
| Milestone: | 19.03 → 19.04 |
|---|
comment:8 by , 7 years ago
| Milestone: | 19.04 → 19.05 |
|---|
comment:9 by , 7 years ago
There are still checks for very uncommon tags. The first one, aerialway=canopy, is used only 31 times. The second, amenity=abandoned is used only 1 time. I didn't check the others.
Please check them and only keep those who are frequently misused. For example anything with more than 500 uses.
comment:10 by , 7 years ago
| Milestone: | 19.05 |
|---|
comment:12 by , 7 years ago
| Resolution: | → needinfo |
|---|---|
| Status: | needinfo → closed |
comment:13 by , 6 years ago
| Milestone: | → 19.07 |
|---|---|
| Resolution: | needinfo |
| Status: | closed → reopened |
comment:14 by , 6 years ago
| Owner: | changed from to |
|---|---|
| Status: | reopened → new |
comment:15 by , 6 years ago
| Keywords: | sotmfr2019 osmose added |
|---|
comment:17 by , 6 years ago
- aerialway=canopy - usage too low
- amenity=abandoned - usage too low
- amenity=citymap_post - 102 out of 114 in the city of Warszawa
- amenity=creche - usage too low
- amenity=ev_charging - usage too low
- amenity=charging - usage too low
- amenity=fire_hydrant - warning already existing
- amenity=garages - to be added
- amenity=garage - to be added
- amenity=nursery - unclear if by replacement with only amenity=kindergarten informatin get lost. Usage of additional tag nursery=yes is pretty low.
- amenity=preschool - unclear if by replacement with only amenity=kindergarten informatin get lost. Usage of additional tag preschool=yes is pretty low.
- amenity=real_estate - usage too low
- office=real_estate - to be added
- shop=real_estate - usage too low
- amenity=sauna - warning already existing
- amenity=winery - to be added, also shop=winery
- amenity=youth_centre - to be added
- amenity=youth_center - usage too low
- bridge=arch - usage too low
- bridge=beam - usage too low
- bridge=humpback - usage too low
- bridge=suspension - warning already existing
- bridge=lift - usage too low
- bridge=swing - warning already existing
- bridge=pontoon - to be added
- building:type - to be added
- escalator - to be added
- fenced - to be added
- highway=abandoned - to be added
- highway=minor - usage too low
- historic_name - to be added
- historic=museum - can be valid if the museum itself is a historic feature, see https://wiki.openstreetmap.org/wiki/Historic
- landuse=field - to be added
- leisure=beach - to be added
- leisure=club - to be added
- leisure=video_arcade - to be added
- man_made=cooling - it seems there is no clear consensus, see man made=cooling tower wiki page history
- man_made=hot_water_tank - usage too low
- man_made=jetty - to be added
- man_made=village_pump - to be added
- man_made=water_tank - to be added
- natural=moor - to be added
- noexit=no - to be added
- office=administrative - has an own ticket: #15614
- power=underground_cable - usage too low
- railway=preserved - while I personally would support the depracation it is controversial
- shop=dive - to be added
- shop=furnace - to be added
- shop=moneylender - usage too low
- sport=paragliding - to be added
- tourism=bed_and_breakfast - to be added



Can you please make sure messages are correctly translated by using this syntax?
throwWarning: tr("{0} is deprecated", "{0.tag}");