Refactor the from_xml methods to act on existing trace objects.

Setting the new tags with the = operator takes care of removing the
old ones, and is the same approach as taken by the tagstring= method.

Fixes #1600

Add failing test demonstrating tag duplication when updating traces via the API.

Fix rubocop warning

Switch to using openstreetmap-deadlock_retry

Closes #1602

Merge remote-tracking branch 'upstream/pull/1604'

Add maintainers to README

It's nice to be explicit, so that new contributers know who is who.

While github shows "owner" and "member" labels these are for the
openstreetmap organisation as a whole, and can be misleading.

Make sure the account form is POSTed

Fixes #1601

Merge pull request #1588 from Georeactor/master

Update gem, fix icons on RTL/Arabic pages

Merge remote-tracking branch 'upstream/pull/1603'

Remove unused user save template.

This template isn't used, since all the controller methods redirect
to other views.

Fix user settings form

Fixes #1601

Update to rails 5.0.5

Update bundle

Update rack-cors and fix tests for changes in behaviour

Merge pull request #1597 from tomhughes/current-user

Complete @user to current_user transition

Replace @user with @current_user

This ensures that that we will find any more hidden references
to @user that might be hanging around...

Change user forms to use current user instead of @user

Convert @user to current_user in helpers

Merge remote-tracking branch 'upstream/pull/1595'

Convert @user to current_user

Use a current_user helper for accessing the logged in user in all views.

Convert remaining controller code to use current_user

The `self.current_user` is important when assigning to the current user,
to avoid creating a local variable called `current_user`

use no-r2 comment to block RTL gem from changing sprites

Update to iD v2.3.2

Update to Leaflet.Locate 0.62.0

Mark account suspended flash message as HTML safe

Fixes #1590

Merge remote-tracking branch 'upstream/pull/1587'

new png file for banner with square 350x350

Delete sotmlatam-2016.jpg

Delete sotmasia-2016.jpg

Delete donate-2016.jpg

image with proper width 350px

remove old entries in banner.yml

add sotm2017 in banner.yml

add png file with background

Add geoip_database key to example configuration

Fixes #1586

Merge remote-tracking branch 'upstream/pull/1583'

Use current_user to represent the currently logged in user.

This is already used by the oauth plugin, and is a general rails convention.

Standardise on basic_authorization without the parentheses

Update to iD v2.3.1

Replace timecop with the rails builtin travel_to method

Add iD walkthrough autostart parameter

Update to iD v2.3.0

Merge remote-tracking branch 'upstream/pull/1575'

Add some tests for the user helper

Hide contributor terms details, unless the user hasn't agreed

It's no longer important to show when users agreed to the contributor terms.
For users that haven't agreed, we still show their status on their profile.

Logic was also re-written to avoid lots of `if not` things, and tests added.

Don't disable search submission buttons on form submission

Fix more parameter sanitisation issues and add tests

Disable forgery protection for notes API methods

Fixes #1571

Sanitize parameters for various paged views

Merge remote-tracking branch 'upstream/pull/1561'

Update to leaflet 1.1.0

Monkey patch oauth-plugin to avoid using deprecated methods

Fix some tests for changes in rails 5

Update get/post etc to remove use of deprecated syntax

Update to rails 5.0.4

Update to rails 4.2.9

Rework coordinates to avoid scientific formatting of small numbers. Fixes #1509

Refactor AMF encodevalue routine

Simplify the switch, and raise an exception if we see an unexpected
type rather than just logging and then accidentally triggering a odd
error by returning true when a string is expected.

Update bundle

Disable removeUnknownsAndDefaults instead of removeUselessStrokeAndFill

Convert XML document to a string when returning it

Otherwise the Rak::ETag module will throw an exception when it
tries to test if the response is empty.

Disable removeUselessStrokeAndFill svgo plugin
see https://github.com/openstreetmap/iD/issues/4105

No need to precompile traffico anymore, as it's been removed

Update to iD v2.2.2

Remove old traffico mapillary signs library from Vendorfile

Update bundle

Add explicit primary key declaration for changeset subscriptions

This table has a composite primary key and although we don't really
rely on that at all rails 5.0 warns about itif you don't explcitly
declare it in order to trigger the CPK extension.

Remove deprecated get_via_direct and post_via_redirect methods

Use explicit to_unsafe_h method when converting parameters to a hash

Replace deprecated ActiveModel::Errors get/set methods

Replace deprecated env method with request.env

Use distinct instead of uniq which is deprecated

Replace render :nothing with non-deprecated alternatives

Sanitise parameters used in URL generation

Replace render :nothing with head

Replace deprecated xxx_filter methods with xxx_action

Replace deprecated Mime::XXX with Mime[:xxx]

Avoid using format as a URL parameter name

This prevents rails confusing it with the builtin format
parameter derived from the URL extension.

Make delete button translatable

Merge remote-tracking branch 'openstreetmap/pull/1553'

Add delete button to message view

Fixes #1504
Closes #1557

Sanitise parameters used in URL generation

Remove duplicate size specification for user image

Either the size or the width and height can be passed but passing
both doesn't make any sense and in rails 5.0 will error.

Remove all use of the :text option to render

It doesn't actually do what it says, as it sets the content type
to text/html not text/plain so is just confusing and as a result
has been deprecated in newer rails versions.

Remove conditions from delete_all

Passing conditions directory to delete_all is deprecated
in rails 5.0 so use a separate where instead.

Rescue LoadError instead of MissingSourceFile

MissingSourceFile is just an alias for LoadError in rails 4.2 and
in rails 5.0 it is no longer present.

Remove arguments from assert_nothing_raised

It has never done anything with the argument and rails 5.0 warns
that rails 5.1 will remove the argument.

Move monkey patch modules to the OpenStreetMap namespace

Having them in the OSM namespace risks blocking autoloading
of the lib/osm.rb code by defining the OSM constant.

Fix rubocop warnings

Update bundle

Fix up monkey patching of ActiveRecord for migrations

Replace alias_method_chain with Module#prepend

Drop monkey patches that are no longer required

Drop hack used for rails 4 upgrade

Ensure closed changesets appear in the feed, and update some misleading comments.

Remove test fixtures.

Remove remaining references to fixtures.

Remove the api_fixtures helper

This is no longer required, as the tests no longer use fixtures.

Rubocop fixes.