Update Potlatch 2 to 2.5 build

Localisation updates from https://translatewiki.net.

Add explicit tests for lat/lon methods on node models

Use changeset factory for association in node factory

Set canonical port based on protocol

Remove various osm.org email addresses from example configuration

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

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

Make the publisher URL configurable, with no default

Add server_protocol configuration option

Use node and old_node factories for redaction model tests.

Use canonical-rails gem to generate canonical URLs

Create a node factory, and use it for most node model tests

Set server_url in the test environment

Comment out server_url in the default configuration

Add a canonical URL to the head

Use old_node factory for OldNode model tests.

The valid/invalid tests were simplified, since there is little to be
gained from inserting (invalid) nodes into the database and comparing
the node with another copy of itself.

Rework the changeset helper tests to use factories.

Use changeset factory for factory associations

One test needed updating since it assumed the comments were being
created on a particular changeset.

Create a changeset factory, and use it for model tests.

Remove the api_fixtures call from changeset model tests, since they do not need the fixtures.

Localisation updates from https://translatewiki.net.

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

Remove the link between the redaction owner and the user doing the redaction in tests.

Localisation updates from https://translatewiki.net.

Use redactions factory for old_way controller tests

Adjust rubocop threshold

Use redactions factory for old_node controller tests.

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

Use redaction factory for old_relation controller tests.

Fix typos in assertion messages.

Use redactions factory in redaction controller test.

Create a redactions factory and use for model tests.

Refactor the oauth integration test to avoid using user fixtures.

Use user factory for user_terms_seen integration test.

Use user factories for some changeset_controller tests.

Others are tightly interwoven with the fixtures and will be tackled seperately.

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

Use a pending user for testing password resets.

Use users factory for user_controller tests.

Throw an exception if adding a note comment fails

Make search use HTTPS automatically

Currently users end up having a HTTP search function even if they
install it to their browsers from the HTTPS site. This is a unexpected
reduction in privacy.

Since search requests are private data they should be sent via HTTPS.

Closes #936

Build pending users via factory

Return maximum size of bounding box for note retrieval

Localisation updates from https://translatewiki.net.

Improve styling of flash messages on the front page

Show offline/readonly messages as normal flash messages

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

Add some whitespace

Use user factories for user_login integration tests.

The display_name logins were reworked to use downcase since the
factories supply users with an already-title-cased display_name.

The tests were also renamed to align better with the purpose of the
test rather than the name of the fixture used.

Use user factories for page_locale integration tests.

Use user factories for user_block integration tests.

Use user factories for user_roles integration tests.

Use user factory for user_creation test.

Use user factory for application_helper tests.

Use user factory for note_helper tests.

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

Improve trace#api_data tests

The api_data action doesn't actually take the username as an
arguement and should be modelled on api_read rather than the
actions that handle web views.

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

Remove redundant tests for deleted traces, and rearrange comments and factory calls for consistency.

Use user factory for user_roles_helper test

Create a super_user factory.

Create a fresh user when requesting non-existent traces.

Use user factory for message model tests.

Take the user from the node or changeset, rather than referring to the fixture.

Use user factory for note model test.

Use user factory for oauth_token model test.

Use user factory for user_preference model tests.

Use user factory for trace model tests.

Use user factory for trace_controller tests

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Do more preloading in browse controller methods

Fixes #1476

Fix accidental change in comments

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

Detect a timeout encapsulated in ActionView::Template::Error

Fixes #1476

Use user factory for swf_controller tests.

Use user factory for message_controller tests.

Use user factory for user_roles_controller tests.

Fix hiding of sidebar loading indicator

Fixes #1475

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Remove inline styles

Use users factory for user_preferences_controller tests.

Use users factory for user_blocks_controller tests.

Use users factory for redactions_controller tests.

Use local api to export map.osm

Closes #1282

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

Update HSTS to publish a max-age=0 to disable it

Silence remaining jshint warnings

Don't try and modify policy if we don't have one

Update Potlatch 2 to 2.4-46-gea38865 build

Fix some jshint warnings

Improve the content security policy

Remove inline javascript from page header

Remove inline javascript from id views

Correct name of zoom argument for Potlatch 2

Fixes #1471

Remove inline javascript from potlatch2 view

Remove inline javascript from potlatch view

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

Remove inline javascript from user views