Rubocop fixes

Skip gif animation optimization in case both frames are identical
This works around an issue in libgd2 library which would otherwise
cause segfaults due to zero sized images

Fix rubocop warnings and rename a variable

Remove unnessary init

Fix incorrect start index

Gif animation prototype

Localisation updates from https://translatewiki.net.

Use erb-lint to prohibit inline scripting

Format closed_at correctly in changeset API responses

Fixes #2195

Localisation updates from https://translatewiki.net.

Fix dependencies

Update to rails 5.2.3

Expect CORS preflight responses to not have a content type

https://github.com/cyu/rack-cors/pull/180

Update bundle

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

erblint: use consistent spacing in HTML tags

erblint: remove leading blank lines

erblint: avoid dynamic find_by methods

erblint: add rel=noopener to links that open in a new tab

erblint: use conditional assignment

erblint: use word arrays

erblint: use parentheses for nested function calls

erblint: use symbols as procs

erblint: avoid negated if conditions

erblint: prefer symbolic logical operators over and/or/not

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'

Use trace jobs by default

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