erblint: avoid parentheses for method calls with no arguments

erblint: prefer double quoted strings

Exclude most views from string literal checks

This allows us to work through them slowly, while preventing errors
from creeping into directories that already pass.

erblint: Add or remove curly braces around hash parameters as required

erblint: use hash rocket syntax

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

Exclude files in the vendor directory from erb-lint rubocop checks

Fix capitalisation of div tag

Update coding style section of the contribution guide to describe our use of rubocop and erblint

Add erblint to travis

Introduce erb-lint for linting erb files

This also allows running rubocop on the ruby code within the erb
files.

Whitespace fixes to various erb files

Whitespace fixes for messages erb files

Whitespace fixes for layouts erb files

Whitespace fixes for diary_entry erb files

Whitespace fixes for browse erb files

Whitespace fixes for errors erb files

Whitespace fixes for oauth_clients erb files

Whitespace fixes for issues erb files

Whitespace fixes for users erb files

Whitespace fixes for user_blocks erb files

Whitespace fixes for notifier erb views

Whitespace fixes for site erb files

Whitespace fixes in traces erb files

Prefer String#match? over butt ugly Regexp#match?

Target ruby 2.5.1 to match Ubuntu

Rubocop fixes for ruby 2.5

Upgrade minimum version of ruby to 2.5

This prepares the way for gems that depend on newer ruby, and also
prepares for the rails 6 upgrade.

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

Prefer String#match? over butt ugly Regexp#match?

Target ruby 2.5.1 to match Ubuntu

Localisation updates from https://translatewiki.net.

Queue trace related jobs in the traces queue

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

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

Use strings consistently for database/api/gpx status

Localisation updates from https://translatewiki.net.

Move more api-only methods into api_controller

Mark all methods in api_controller and application_controller as private

They aren't designed to be used as request endpoints, so there's no
need for them to be public.

Use a lambda in order to pass parameters in before_actions

This avoid calling methods on a new instance of the controller, and
therefore allows these methods to be marked as private.

As a bonus, rubocop can now parse them and warn when they refer to
actions that don't exist in the controller.

Move the authorize method to api_controller

Simplify deny_access handling

Now that we have all api controllers inheriting from a common base,
it's easier to override the deny_access handler without having to
switch between both.

Fixes #2064

Refactor api controllers to inherit from a common ApiController

Use an enumerator for gpx.points, and process the the points in batches

Raise exception if there is an error, and import tracepoints in batches

Use activerecord-import for bulk importing tracepoint records

Non-rigourous testing shows a significant speedup, even on ssds.

Fix travis

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

Add banner against EU Copyright Directive

Set end date to 2019-03-26 because the topic is tabled on 26 March
according to the agenda.

Localisation updates from https://translatewiki.net.

Move application.yml check to config initializer

Move status into the settings object

Only the very early boot code needs to look at the value
from the environment directly.

Fix database offline mode

Sort gitignore

Add comment

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

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

Rubocop fix

Rubocop fixes

Move changeset builder views to the api namespace

Move changeset XML generation to a view

Add missing dependency

Add missing dependencies

Update to rails 5.2.2.1

Update bundle

Add Spanish attribution to copyright page

Fixes #2179

Localisation updates from https://translatewiki.net.

Remove application.yml step from travis config

Provide a helpful warning if there's an application.yml file detected

Add some settings validations

Update documentation for new approach to settings

Instruct sprockets to recompile this file based on the settings files changing

Only take the STATUS configuration from the environment

This allows us to remove the preinitializer-based configuration parsing.

Drop the old environment variable parsing

It's likely unused, and incompatible with environment variable parsing from Config

Move all settings to settings.yml

We leave the STATUS setting alone, since it's required before rails
boots. The test-specific settings now live in config/settings/test.yml

Install config gem for settings management

Rubocop fixes for ruby 2.5

Upgrade minimum version of ruby to 2.5

This prepares the way for gems that depend on newer ruby, and also
prepares for the rails 6 upgrade.

Localisation updates from https://translatewiki.net.

Prevent leading and trailing whitespace on email addresses

Strip leading and trailing whitespace for email authentication

Fixes #2173

Localisation updates from https://translatewiki.net.

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

Silence the simplecov output message

It's very rare for the information in this message to be useful or
actionable when running the tests, so it's best to silence the message.

Update OS copyright year

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

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

Move the api trace methods into a separate controller under the api namespace

Fix test file name

Move the amf and swf controllers into the api namespace

Move the notes api methods into a controller in the api namespace

Move the user api methods into a separate controller in the api namespace

Move node/way/relation/old_* controllers into the api namespace

Update to iD v2.14.3

Flip header image horizontally on RTL pages

Fixes #2158

Prevent reversal of sprite positions on welcome screen

Fixes #2156

Avoid CSP issues with OpenID login

To avoid Chrom getting upset about sending form data to sites
that our policy doesn't allow, even when it isn't, use Javascript
to jump straight to Omniauth as the direct OpenID based login
buttons were already doing.

Fixes #1909

Localisation updates from https://translatewiki.net.

Allow loading of our manifest

Move the user preferences controller into the api namespace