Remove non functioning changeset_cache, user_display_name_cache

Avoid unnecessary arrays

Simplify partial rendering

Avoid using @variables in partials

Move xml gen. into view for (old) nodes, ways, relations, map

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Switch travis to use xenial

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

Localisation updates from https://translatewiki.net.

Resourceful(ish) naming for api/users_controller

Resourceful route names for api/trace_controller

Localisation updates from https://translatewiki.net.

Enable a number of eslint style rules

Enable some more eslint checks

Exclude generated i18n files from eslint checks

Use double quotes consistently in javascript code

Enable some more eslint rules from iD rules

Localisation updates from https://translatewiki.net.

Fix queuing of trace imports for API uploads

Fixes #2200

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

Fix issues found by eslint

Replace jshint with eslint

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.