Merge remote-tracking branch 'gravitystorm/terms_capabilities'

Require terms agreement for abilities and capabilities related to api write methods

Replace custom panning with new panInside leaflet method

Update leaflet.locate.js

Update leaflet to 1.4.0

Localisation updates from https://translatewiki.net.

Allow inline styling on pages that display the map

Both leaflet itself and at least one of our plugins use inline
styling to style markers so we need to allow it.

Fixes #2093

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

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

Update to iD v2.12.2

Localisation updates from https://translatewiki.net.

Allow abilities that require no login for token based access

Fixes #2085

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

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

Refactor users_controller to use CanCanCan for authorisation

Use only token capabilities when a token is provided

The Authenticate#allow? method (from oauth-plugin) sets current_user as a side
effect of checking the token. But this allows a valid token to access
all actions that are available to that user, beyond the capabilities for
that token.

Localisation updates from https://translatewiki.net.

Only check IP addresses for anonymous note comments

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

Update to iD v2.12.1

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

Update to rails 5.2.2

Update to iD v2.12.0

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

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

Use CanCanCan for user_roles auth

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

Split browse_helper.rb into two modules due to rubocop ModuleLength

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

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

Remove user_roles integration test since it is not meaningful

This test has not been meaningful for a long while, since both check_success and check_fail contain exactly the same code.

Additionally, the test doesn't cover any integrations (beyond logging in), and so it is only covering the same ground as the controller test.

Remove the unused require_moderator filter

Use of this filter has been refactored to use CanCanCan

Use CanCanCan for notes authorization

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

Use CanCanCan for changeset comments

This introduces different deny_access handlers for web and api requests, since we want to avoid sending redirects as API responses. See #2064 for discussion.

Update to rails 5.2.1.1

Show tel: links for multiple phone numbers separated by ;

Closes #2069

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Add links to Welcome Mat on /welcome and /help

Closes #2056

Update Potlatch 2 to 2.5-59-gdd728d5e build

Allow connect_src to match all sites in Potlatch

It seems that Safari matches connections made from a flash application
against connect_src while Firefox uses object_src instead.

Fixes #2067

Localisation updates from https://translatewiki.net.

Fix issues with renaming of diary entry controller

Take security policy URLs from the configuration file

Localisation updates from https://translatewiki.net.

Fix tests for rails 5.2.1 compatibility

Rails 5.2.1 has changed how the request body is handled
internally for a test which means we can no longer cheat
by stashing it in the request environment and must instead
pass it properly to the request method.

Update to rails 5.2.1

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

Remove custom deny_access handlers

Since these pages are not accessed by normal users, except for url fiddling, it's fine to respond with a generic access denied.

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

Add a changeset to exercise that part of the contact rendering

Ensure that the blocked template rendering works

Check that a request that requires authentication is redirected when the user hasn't seen the terms

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

Pluralize changesets controller

Skip CSRF verification for changeset comment actions

Fixes #2057

Localisation updates from https://translatewiki.net.

Update Potlatch 2 to 2.5-57-gaa163622 build

Update Potlatch 2 to 2.5-56-g550aab49 build

Attempt to send pretty 403 errors to web browsers

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

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

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

Localisation updates from https://translatewiki.net.

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

Remove unnecessary include from redaction model test

Remove unnecessary require statements from tests

Pluralize diary entries controller

Pluralize old_ controllers

Pluralize nodes, ways and relations controllers

Use CanCanCan for redaction authorizations

Migrate UserBlocksController to use CanCanCan

Reduce the max class length after splitting the changeset comments controller tests out

Use relative translations for changeset comments

Rename hide_comment and unhide_comment to destroy and restore

This preserves the API endpoints and HTTP methods, which could be changed in the next API version

Rename comments_feed to index

Rename comment to create

Split changeset comment handling into a changeset_comments controller

Fix rubocop warning

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

Update leaflet.locate.js

Avoid ordering points from public and private traces

Closes #2046

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

Use character validate to exclude URL characters for trace tags

Improve character validator error messages

Merge character validators

Merge leading and trailing whitespace validators

Improve consistency of text validations

Added tests for validators

Changed User model to not allow nil display_name (w/ tests)

Converted invalid_chars validator to use locale

Fix rubocop errors

Create invalid_char validators and apply to models

Added more non-ascii chars to validation (matching list of chars in other models)

Fix rubucop errors

Make invalid char list more explicit (between ascii and non ascii chars)

Added trailing/leading whitespace errors to locale