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

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

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

Move the search controller to the api namespace

Move the api methods from changeset_comments_controller into the api namespaced controller

Move the api methods from changesets_controller into the api namespaced controller

Move map method to its own controller

Move the changes api to its own controller

Move the permissions call out of api_controller

Rename api controller test files

Move the trackpoints call into its own controller (and rename to tracepoints)

Move the capabilities call out of api_controller

Improve detection of browsers needing ES6 shims

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

Update to iD v2.14.2

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

Update to iD v2.14.1

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

Avoid bundler 2.x for now

Fixes #2149

Update to iD v2.14.0

Set language and text direction properly on the copyright page

Also add style overrides for explicit LTR and RTL text that work
independently of the main page styling.

Fixes #2148

Localisation updates from https://translatewiki.net.

Revert to rinku 2.0.4

Revert to bootsnap 1.3.2

Localisation updates from https://translatewiki.net.

Update bundle

Update tests for changes in FactoryBot 5.x

Update bundle

Always look to the global scope for the GPX module

Localisation updates from https://translatewiki.net.

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

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

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

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

Output both the local simplecov html and the coveralls report

Fixes #2066

Merge pull request #2126 from MaZderMind/patch-1

Advise to run i18n:js:export before assets:precompile

Fix typo

Remove require_terms_agreed configuration option

This has been set to true for 6 years in production. Refs #2097

As per other user settings, we set the terms as seen by default for tests,
and we can override that when necessary for specific tests.

Remove the require_terms_seen configuration option

This option has been set to 'true' for over six years in production.

Refs #2097

Copy i18n strings from validates_email_format_of plugin

This allows our translators to work in languages not yet supported
by the upstream plugin. Fixes #2128

Remove unnecessary requires from app

Remove unnecessary requires from helpers

Move utf8 validator into app/validators

Remove unnecessary requires from tests

Localisation updates from https://translatewiki.net.