[openstreetmap/openstreetmap-website] "Placeholder Relation not found" when sending osmChange data to server (#1465)

Matt Amos notifications at github.com
Fri Feb 24 15:42:58 UTC 2017


I think a patch that would allow relations to appear in any order would be welcome, although in this instance it could have been fixed on the client since it's an ordering issue rather than circular references.

Not handling circular relation references is, to some degree, a feature rather than a bug. Circular references aren't explicitly disallowed, but it would be strongly recommended not to create them or rely upon them.

Firstly, they create a problem when trying to delete either one, as it relies on the other and so has to be "disentangled" before deletion. This is not handled automatically in the upload process, and is something the client has to handle. Most probably don't.

Secondly, recursive or circular relations might not be handled well in other software that consumes OSM data. Or possibly it might be ignored altogether. Either way, it seems unlikely that whatever was intended by the circular relation would be accurately conveyed.

In summary: Circular relation dependencies, while possible, should be avoided.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/1465#issuecomment-282323187
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20170224/938ae1f3/attachment.html>


More information about the rails-dev mailing list