From: Tom Hughes <tom@compton.nu>
Date: Sun, 3 Feb 2008 16:31:50 +0000 (+0000)
Subject: Explain why we don't include the nodes in the way load.
X-Git-Tag: live~7936
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/276f165f3faca4913b63126de271530d2992a0ba

Explain why we don't include the nodes in the way load.
---

diff --git a/app/controllers/amf_controller.rb b/app/controllers/amf_controller.rb
index 9592cf941..ac0809219 100644
--- a/app/controllers/amf_controller.rb
+++ b/app/controllers/amf_controller.rb
@@ -167,7 +167,11 @@ class AmfController < ApplicationController
 
     RAILS_DEFAULT_LOGGER.info("  Message: getway, id=#{wayid}")
 
+    # Ideally we would do ":include => :nodes" here but if we do that
+    # then rails only seems to return the first copy of a node when a
+    # way includes a node more than once
     way = Way.find(wayid)
+
     long_array = []
     lat_array = []
     points = []