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

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

Disentangle the api abilities from the web abilities

This will allow us to rename api actions without causing permissions headaches. The choice of
abilities files is made by inheriting from either api_controller or application_controller.

Also rename capabilities to api_capabilites, for consistency.

Set image dimensions in order to avoid a reflow after the page is drawn by the browser

Refactor notes#mine to simplify code

erblint: fix remaining single quoted strings

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

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

Use the trace jobs for creating and destroy traces via the API

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

Check validity token in changesets controller

It makes no difference today, but prevents problems in future if
methods accepting post requests are added to this controller in future.

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