Use bootstrap_form as the formbuilder for GPX upload form

This also uses the recommended bs-custom-file-input javascript plugin
for the file input control.

Add bootstrap_form formbuilder gem

Remove the red colour override, and use the bootstrap default

Full red clashes badly when side-by-side with our blues

Refactor away from global form control styling

This enables forms to be converted one-by-one to bootstrap. Without this
refactoring, these global styles interfere with form control styling from
bootstrap.

The rearrangement of the search and directions styling is to solve CSS selector specitivity issues.

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

Rework user#set_status and #delete to avoid GET requests

This renames the delete action to destroy, and starts using resourceful
routing for user actions.

Localisation updates from https://translatewiki.net.

Rename `many` keys to `other` for the cs locale.

Fixes #2645

Update bundle

More improvements to locale selection for the data browser

Use the browser language preferences for non-logged in users.

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

Improve selection of name to display in data browser

If the user is logged in then consider all their preferred
languages rather than just the UI language, and use the full
locale matching algorithm instead of just checking parents.

Fixes #2636

Improve locale selection algorithm

Don't include locales which only have rails translations in
the candidates, and ensure that user specified options take
priority over less specific variants of earlier choices.

Localisation updates from https://translatewiki.net.

Fix new rubocop warnings

Update bundle

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

Rename friend to friendship in notifier

Followup to #2269

Move make_friend and remove_friend to friendships controller

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

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

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

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

Move friendship related tests to separate file

Convert lib files to model concerns

Remove some unused scripts

Merge pull request #2624 from francois2metz/diary-entry

Add link to my diary in the diary index

Use rails 6.0 framework defaults

Keep the old autoloader and utf-8 forms as they were, for now.

Remove the framework defaults initializer for 5.2

We're already using these defaults, as per 6a25ee9ec962bc10997a4c62a6246873130d41a1
so we don't need to keep this file

Remove ineffective query cache disable initializer

This doesn't have any effect, since there is no middleware by that name.

Localisation updates from https://translatewiki.net.

Update bundle

Localisation updates from https://translatewiki.net.

Update bundle

Update to rails 6.0.3.1

Localisation updates from https://translatewiki.net.

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

Remove unused test helper

Refactor user controller tests

Because we don't have direct access to the session and cookies, some
of the tests need more setup e.g. by moving through the signup process
to get the session tokens into the right place for the confirmation
steps to work.

Refactor api traces_controller tests

Don't try and resize images that aren't resizable

Attach resized images to notification emails

Fixes #2625

Fix new rubocop warnings

Update bundle

Add link to my diary in the diary index.

Refactor the link to create a new entry diary.

Localisation updates from https://translatewiki.net.

Refactor users_controller_test and user_preferences_controller_test

Fix issue with iD embedded document sometimes showing scrollbars

Localisation updates from https://translatewiki.net.

Update to rails 6.0.3

Refactor notes_controller_test

Add test for browsing a note closed by a deleted user

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

More controller test refactoring

Look at all note comments to find the close event

Fixes #2612

Refactor permissions_controller_test

Refactor versions_controller_test

Refactor tracepoints_controller_test

Remove unused function

Refactor map_controller_test

Switch oauth integration tests to use new shared helper

Refactor changeset comments API controller tests

Add helper methods for making OAuth signed requests

Refactor the AMF controller API tests

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Update bundle

Localisation updates from https://translatewiki.net.

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

Refactor changesets_controller_test

Introduces some route names which need to be unique from the non-api versions

Refactor more controller tests

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

Refactor traces controller tests

Move document-level styling for iD to its stylesheet

Fix issue where iD would not take the full height of its iframe

Refactor more controller tests

Localisation updates from https://translatewiki.net.

Refactor more controller tests

Refactor more controller tests

Update to iD v2.17.3

Fix deprecation warning

Update bundle

Update vagrant provision script for switch to selenium

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

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

Refactor some more controller tests

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

Remove tempfile patch for trace data

Effectively reverts c0d2ad40c30e5a0837b6012d7b9067d69ce41dd0

This patch is no longer required, since we only use send_data in
combination with Tempfile.read and that all works fine.

Switch to using selenium for system tests

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

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

Use Open3.capture2 instead of backticks, to avoid command line injection risks

In this situation, trace_name can be trivially checked as legitimate, but this
removes any lingering risks from interpolating into a command line instead of
passing parameters explicitly.

Refs #2229

Redirect to the error page for non-existent issues

Send meta information with error pages

Refactor site controller tests to inherit from ActionDispatch::IntegrationTest

This is the modern way of writing controller tests, since it uses the
rails middleware (among other things) and leads to more realistic tests.

Refs #2563

Remove last traces of konacha

Localisation updates from https://translatewiki.net.

Fix new rubocop warnings

Enable new cops by default