Opened 6 years ago

Last modified 4 years ago

#17809 closed enhancement

convert FIXME=* to fixme=* — at Version 1

Reported by: mkoniecz Owned by: team
Priority: normal Milestone:
Component: Core validator Version:
Keywords: template_report fixme Cc:

Description (last modified by mkoniecz)

What steps will reproduce the problem?

  1. tag node with FIXME=geometry and building=hangar
  2. run validator

What is the expected result?

Validator offer to change FIXME=geometry to fixme=geometry to help eleiminate FIXME duplicating fixme tag.

What happens instead?

Validator is not proposing tag changes.

Please provide any additional information below. Attach a screenshot if possible.

I can implement this, after confirmation that it would not be rejected.

See equivalent iD issue at https://github.com/openstreetmap/iD/issues/6477

@matkoniecz Thanks for the links! I'm glad this was already discussed since it saves us time. I read through the mailing list discussion and it convinced me that we should definitely deprecate FIXME in iD. Most people agreed that FIXME is dated and wrong. Another developer even mentioned how FIXME caused them hours of extra work. The disagreements concerned the necessity and effectiveness of the edit. Responding to the primary arguments I saw against your bot edit:

FIXME should purposely cause validator tools to flag it by being all capitalized.

A validator tool doesn't need special capitalization to flag something. In fact, having two tags makes it more likely that a validator will miss a fixme.

Fixmes aren't important enough to create new entity versions over.

Users would manually be updating features in iD, so I think creating new versions is warranted.

A single bot edit won't prevent future uses of FIXME. Editors should handle this.

This is the part we can address!

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2019-06-12 22:04:53 +0200 (Wed, 12 Jun 2019)
Build-Date:2019-06-13 01:30:52
Revision:15169
Relative:URL: ^/trunk

Identification: JOSM/1.5 (15169 en) Linux Ubuntu 16.04.6 LTS
Memory Usage: 423 MB / 869 MB (185 MB allocated, but free)
Java version: 1.8.0_201-b09, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Screen: :0.0 1920x1080
Maximum Screen Size: 1920x1080
Dataset consistency test: No problems found

Plugins:
+ OpeningHoursEditor (34977)
+ PicLayer (35030)
+ buildings_tools (34982)
+ continuosDownload (82)
+ ejml (34908)
+ geotools (34908)
+ imagery_offset_db (34908)
+ jts (34908)
+ log4j (34908)
+ measurement (34977)
+ reverter (34999)
+ todo (30306)

Validator rules:
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/deprecated.mapcss
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/unnecessary.mapcss
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/combinations.mapcss

Last errors/warnings:
- W: No configuration settings found.  Using hardcoded default values for all pools.
- W: java.net.SocketTimeoutException: connect timed out
- E: java.net.SocketTimeoutException: connect timed out
- E: org.openstreetmap.josm.io.OsmTransferException: Could not connect to the OSM server. Please check your internet connection.. Cause: java.net.SocketTimeoutException: connect timed out
- E: IO Exception - org.openstreetmap.josm.gui.widgets.HtmlPanel[,0,0,0x0,invalid,layout=java.awt.BorderLayout,alignmentX=0.0,alignmentY=0.0,border=,flags=9,maximumSize=,minimumSize=,preferredSize=]
- W: java.net.SocketTimeoutException: connect timed out
- W: Already here java.net.SocketTimeoutException: connect timed out
- E: java.net.SocketTimeoutException: connect timed out
- E: org.openstreetmap.josm.io.OsmTransferException: Could not connect to the OSM server. Please check your internet connection.. Cause: java.net.SocketTimeoutException: connect timed out
- E: IO Exception - org.openstreetmap.josm.gui.widgets.HtmlPanel[,0,0,0x0,invalid,layout=java.awt.BorderLayout,alignmentX=0.0,alignmentY=0.0,border=,flags=9,maximumSize=,minimumSize=,preferredSize=]

Change History (1)

comment:1 by mkoniecz, 6 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.