allow source:geometry on a node

[mkoniecz]

It is not unreasonable - or maybe in case it jarring for native speakers just add automated fix to change it into source:location?

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2014-06-09 01:34:44
Last Changed Author: akks
Revision: 7224
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2014-06-08 22:01:41 +0200 (Sun, 08 Jun 2014)
Last Changed Rev: 7224

Identification: JOSM/1.5 (7224 en) Windows 7 32-Bit
Memory Usage: 247 MB / 247 MB (75 MB allocated, but free)
Java version: 1.7.0_55, Oracle Corporation, Java HotSpot(TM) Client VM
Dataset consistency test: No problems found

Plugin: OpeningHoursEditor (30416)
Plugin: RoadSigns (30438)
Plugin: buildings_tools (30485)
Plugin: continuosDownload (28565)
Plugin: lakewalker (30416)
Plugin: measurement (30416)
Plugin: notes (v0.9.2)
Plugin: wikipedia (30449)

[Don-vip]

source:location and source:loc are not used much around the world (look at taginfo maps), source:position looks used more, so yes we could offer to replace it with this key for nodes.

[Don-vip]

Simon, can you help me?
I tried this at first:

node[source:geometry] {
  throwWarning: tr("{0} on a node", "{0.key}");
  fixAdd: "source:position={0.value}";
  fixRemove: "source:geometry";
}

but the fixAdd does nothing as the condition does not hold the value.

I then tried this:

node[source:geometry=~/.*/] {
  throwWarning: tr("{0} on a node", "{0.key}");
  fixAdd: "source:position={0.value}";
  fixRemove: "source:geometry";
}

But the value is replaced by .*.

How can we do that?

[simon04]

The problem is that {0.key} etc. is set when reading the mapcss file. Instead, fixChangeKey: "source:geometry => source:position"; should work.

[simon04]

In 7279/josm:

fix #10125 - Validator: fix source:geometry to source:position on nodes