Re: [openstreetmap/openstreetmap-website] [WIP] XML generation: nodes, ways, relations, map controller → view (#2223)
notifications at github.com
Wed Jun 26 19:40:52 UTC 2019
> I think there's need for .empty? guards when using .each (in app/views/api/map/_node.xml.builder)
I didn't quite get this comment. Currently the code reads as:
if node.tags.empty? <<<< check is already here
xml.node(attrs) do |nd|
node.tags.each do |k, v|
nd.tag(:k => k, :v => v)
So this should already be covered. I didn't spot any other usage of "each" in this partial.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the rails-dev