[openstreetmap/openstreetmap-website] move changeset xml generation to view (#2164)

[Frederik Ramm]

This pull request gets rid of the to_xml method in the changeset model, and replaces it with XML builder code in the view which is the canonical way of generating output. This paves the way to parametrising XML output later. Some changes to the tests were necessary to make up for the lost to_xml functionality in the model. The plan is to get rid of other, similar to_xml methods one by one.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/2164

-- Commit Summary --

  * move changeset xml generation to view

-- File Changes --

    M app/controllers/api/changeset_comments_controller.rb (9)
    M app/controllers/api/changesets_controller.rb (38)
    M app/models/changeset.rb (61)
    A app/views/changesets/_changeset.builder (43)
    A app/views/changesets/changeset.builder (7)
    A app/views/changesets/changesets.builder (9)
    M test/controllers/api/changesets_controller_test.rb (26)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/2164.patch
https://github.com/openstreetmap/openstreetmap-website/pull/2164.diff

-- 
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/2164
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20190228/45974486/attachment.html>