[openstreetmap/openstreetmap-website] Refactor trace api_update to avoid duplicating tags (#1614)

[Andy Allan]

> Assigning to tags is enough to trigger deletion of the old tags I take it?

Yes. From http://api.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html#method-i-has_many :

"collection=objects :  Replaces the collections content by deleting and adding objects as appropriate."

> I certainly think that if we go this route we should change the method name to update_from_xml or something so that we don't have different models using the same name in different ways.

OK, I can do that.

-- 
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/pull/1614#issuecomment-323314557
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20170818/5287656e/attachment.html>