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

Update bundle

Localisation updates from https://translatewiki.net.

Cache OpenID provider names on a per-locale basis

Fixes #2591

Localise formatting of point count for traces

Fixes #2592

Update bundle

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

Fix rubocop-minitest warnings

Sort cops

Fix erblint

Merge pull request #2547 from bezdna/mobile-layout

Trigger small screen layout based on the width of the screen, rather than the width of the menus.

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

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

Add rubocop-minitest

Localisation updates from https://translatewiki.net.

Add tile.openstreetmap.org to security policy

Update leaflet.osm plugin

Update leaflet.osm plugin

Remove unsued bootstrap components from Vendorfile

Remove arabic strings from en-GB locale

Add blob to frame-src in CSP for iD

Fixes #2582
Closes #2583

Localisation updates from https://translatewiki.net.

Update bundle

Ensure that urls are only valid if the entire string is a url

This replaces our homegrown regexps (that didn't quite work) with
ruby built-in regexps, and uses the termination anchor to ensure
that the entire string, not just the first part, is validated.

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

Remove custom table styling

This removes the custom table styling code, and move to using bootstrap table styling for almost all remaining tables.

Update leaflet plugins

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

Fix typo in comment

Closes #2576

Update leaflet to 1.6.0

Fix new rubocop warnings

Update bundle

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Set QT_QPA_PLATFORM=offscreen for Ubuntu etc

Fixes #2569

Localisation updates from https://translatewiki.net.

Make linkify return an HTML safe result for unsafe inputs

Fixes #2567

Fix some untranslateable strings

Fixes #2565

Pass I18n argument correctly

Fix lego translation with locate control

Fixes #2564

Update javascript bundle

Update to rails 6.0.2.2

Localisation updates from https://translatewiki.net.

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

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

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

Copy trace fixture files, rather than symlinking

Since we are using tmpdirs in order to enable parallel testing, we
can go the full hog and drop the fakefs too and just copy all the
files directly into the tmpdir. If a test makes changes (e.g.
changing the icon file during an import) the copy in the tmpdir is
thrown away at the end of the test anyway.

Remove unused directories

Enable parallel tests

Use unique test directories for each trace test

This allows tests to be run in parallel, since it avoids deleting
some of the symlinks that other tests expect to be there.

Use the settings for the destination of the gpx fixture symlinks

This lets us change the settings values and have the fixtures appear
in the right place.

Remove unnecessary h() calls on model attributes

These were necessary in the rails 2.x era, but not now.

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

Use bootstrap table striping to replace hand-cycled table stripes

This cuts down on the complexity of having to use the `cycle` function
and makes the templates easier to read. CSS-based striping has been
around for many years.

The CSS is in order to keep our custom colour for striped tables.

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

Enable remaining bootstrap components

Fixes #2511

Localisation updates from https://translatewiki.net.

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

Moved CSS rule

Fix border-radius for search query input field

Update javascript bundle

Closes #2557

Localisation updates from https://translatewiki.net.

Update bundle

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Update bundle

Localisation updates from https://translatewiki.net.

Enforce restrictions on issue visibility properly

Fix eslint warnings

Fix highlight of changesets with no bounding box

A changeset with no bounding box has no map layer so trying to
set it's style throws an exception which means we don't get to
the code to set the style on the list entry.

Fixes #2541