Power line checks won't report existing non-power nodes

[francois.lacombe]

Hello

I found out that current power line checks won't report a power line passing by an exiting node of a highway, for instance.

Here the sketch:
I draw a power line and instead of inadvertently connecting it to a highway with a new node, I inadvertently click on an existing node of this highway.
The highway's node will remain unchanged, no new tag, so it won't be reported in "node without power" check nor "Node connects a power line or cable with an object".

Those checks are currently done in the Java class PowerLines.java. I didn't studied it in detail yet but I bet it would be possible to move part of it to mapcss like

way[power=line] > node[!power],
  throwWarning: tr("{0} node without power");
}

If correct, this would allow to strip a bit of specific code and make validation more efficient.

Regarding connections between power lines and highways or waterways, I'm not sure we can achieve them in pure mapcss. Let me know.

Best regards

[GerdP]

I see. You don't get a warning on upload, only when you do a full check. So I think this is a bug in class PowerLines, possibly a regression of r17111.

[francois.lacombe]

Thank you, I indeed get the warning with a full check.

I didn't noticed we already moved from mapcss to java code, so I'm not sure the opposite move is welcome.
By the way r17111 shows what was done before about intersections with building with pure mapcss.

[GerdP]

Seems the problem is a regression of #23397. If you set preference validator.partial.removeIrrelevant to false you also get the messages on upload. I'll dig deeper into this...