Fix error messages when users should not be able to do things

Add testing for moderator users and issues

Remove unnecessary token granting from the user_preferences tests

Sufficient permissions are granted by the basic authorisation, so this
isn't testing anything.

Merge branch 'master' into cancancan

Convert some model mixins to concerns

Update mailmap

Update translation keys for renaming of user to users

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Check the oauth token and then use the capabilities directly

Rework capabilities to avoid assumptions about missing tokens

The logic about missing tokens implying logged in users (and that
all logged in users have access to any method protected by a token
capability) is correct. However, I believe it is both confusing and
brittle, and leaves a security-related door ajar for future foot-gun
incidents.

Instead, apply Abilities as normal, and keep the Capabilities
involvement only for situations where a token is provided. This
reduces the cognitive burden when considering Abilities in isolation.

Rework the default denied access handler to give different responses to tokens, logged in users and other users

Update translation keys for renaming of user to users

Fixes #2031

Enable line breaks in performatted text within rich text

This stops redaction details being truncated at the edge of the
page when they have long lines of preformatted text.

Also use a div instead of a paragraph for redaction details so
that formatting is consistent with other rich text blocks.

Closes #1885
Fixes #1845

Localisation updates from https://translatewiki.net.

Calculate bounding box when deleting relations

Closes #2030
Fixes #2020

Localisation updates from https://translatewiki.net.

Avoid swapping min and max longitude in RTL mode

Closes #2027

Combine site permissions declarations

Localisation updates from https://translatewiki.net.

add australia copyright notice for PSMA Australia

Resize change bounding boxes on a zoom change

Enable automatic translation export in the test environment

Fixes #2016

Adjust minimum changeset bbox to 20 pixels

Improve variable naming

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

Tidy up notes#search

Use "user" as user id parameter for notes searches

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

Suppress changeset pagination for inactive users

Fixes #2024

Localisation updates from https://translatewiki.net.

Fix Rubocop issue

Fix some issues and improve code

Remove temporary development code

Refactor to show the Issues link based on the calculated permissions

Move issues and reports to authorization system

Fix typo in method name

Refactor site#welcome to use abilities instead of require_user

Change abilities based on upstream renamings

Merge branch 'authz' of https://github.com/rubyforgood/openstreetmap-website into rubyforgood-authz

Make paragraph breaks more visible in changeset and note comments

Fixes #2022

Fix Rubocop issue

Fix an issue where the query did not work if a display name or an id was specified

Merge branch 'master' into notes-search

Localisation updates from https://translatewiki.net.

Converted to dot notation - fixes jshint

Convert degrees min-width to pixels min-width

Rename CSS class matches for change in controller name

Remove temorary routes

Improve formatting

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

Updated min bounds algorithm

Localisation updates from https://translatewiki.net.

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

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

Rename user_controller to users_controller

Remove rubocop reference to quad_tile code

Merge branch 'master' into messages

Bump max blocklength for additional routes

Changeset boundary is enlarged if it was on a single point

Changeset border turns blue on hover

Use sshfs in preference to nfs for vagrant file sharing

Update vagrant config to use Ubuntu 18.04

Update wiki page data

Stop wiki update script trying to be "helpful"

Instead of trying to advise how to import some (but not all) of
the modules, just require them in the normal way.

Closes #1763

Fix any_relations always being false

Closes #1976

Fix Style/NumericPredicate rubocop warnings

Fix Style/SafeNavigation rubocop warnings

Fix new rubocop warnings

Update bundle

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

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

Remove Imperial from hosting credits

Fixes #1944

Localisation updates from https://translatewiki.net.

Fix incorrect reference to 12.04 in installation notes

Set the rubocop target version to 2.3

This allows rubocop to run additional cops that would not apply to
older ruby versions

Remove bogus translation

Localisation updates from https://translatewiki.net.

Remove bogus translation

Localisation updates from https://translatewiki.net.

Test that friends are shown correctly on the user profile

Fixes #1992

Set img.button size to 20x20 for directions.png

Update translation keys for renaming of user#view to user#show

Update bundle

Update bowser to 1.9.4

Update leaflet to 1.3.4

Avoid flickering welcome banner

Make the welcome banner default to being hidden and then show
it (by adding a class) if there is no cookie.

Fixes #1951

Limit map to the maximum zoom of the current base layer

Fix maximum zoom levels for various layers and ensure that the
map can't be zoomed beyond the limit of the current base layer.

Fixes #1978

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

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

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

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

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

Rename CSS class matches for changes in action names

Localisation updates from https://translatewiki.net.

Resolve 34 Rubocop Lint/AmbiguousOperator conflicts

Remove Lint/ShadowingOuterLocalVariable - already fixed

Resolve 96 Rubocop Lint/AmbiguousRegexpLiteral conflicts

Just pass the object, rather than the id, to _path methods where possible

Just pass the user object, rather than the display_name, to the user_path helper