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.

create nodes from factories for the api changes test.

Update amf_controller tests to use a factory-generated user for credentials.

Update the user list pagination test to avoid counting users from fixtures.

Refactor the user list test to use factory-built users and explicit assertions.

Use factories to generate the relations

Use the user factory to generate login details.

Refactor the assert_changeset helper to take an array of changesets, instead of ids.

Only use changesets from factories for the test_query changeset controller tests.

Create objects via factories, rather than implicity relying on fixtures.

Rework the changeset feed tests.

These now include explicit tests for the changesets expected, as
well as refactoring so that they don't rely on fixtures.

Create nodes explicitly rather than relying on fixtures.

Add ways into the database explicitly, rather than relying on preloaded fixtures.

Create a redaction object before testing the index page lists it.

Use a user factory in order to log in.

Refactor changeset tags model test to remove dependency on fixtures.

Update rubocop class length limit.

refactor remaining amf controller tests to use factories.

Refactor some controller tests to use factories.

Refactor remaining relation model tests to use factories.

Refactor some controller tests to use factories.

Refactor various controller tests to use factories.

Refactor remaining way model tests to use factories.

Convert more changeset controller tests to use factories.

Refactor more changeset controller tests to use factories.

Convert more changeset controller tests to use factories.

Refactor user_changeset_comments integration test to use factories.

Refactor a few tests to use changeset factories.

Update wiki_pages.yml

Localisation updates from https://translatewiki.net.

Add string for GraphHopper car profile

Include car profile for GraphHopper routing

Closes #1547

Refactor more changeset_controller tests to use factories.

Refactor some changesets controller tests to use factories.

Refactor the changeset query tests to use factories.

Unfortunately some references to fixtures have to remain until they
are finally removed.

Convert more relation controller tests to use factories.

Convert remaining node_controller tests to use factories.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Play a bit of whack-an-seo-mole

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

Ensure invisible_way is actually deleted

Update more relation_controller tests to use factories.

Refactor some relation_controller tests to use factories.