@mmd-osm commented on this pull request.


> @@ -2,7 +2,7 @@ class OldRelationMembersController < OldElementsController
   def show
     @type = "relation"
     @current_feature = Relation.find(params[:id])
-    @feature = OldRelation.preload(:old_members => :member).find([params[:id], 
params[:version]])
+    @feature = OldRelation.preload(:old_members => { :member => :old_tags 
}).find([params[:id], params[:version]])

Prefetching isn't working here as well. In my branch, I used `element_tags` 
instead of `old_tags`. Members of a relation point to the current 
nodes/ways/relations, even for OldRelations.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6338#pullrequestreview-3141429602
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/6338/review/3141429...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to