Mark non-action methods as protected

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Add basic tests for browse#new_note and browse#query

Allow everybody to query features

Allow everybody to create new notes

Fixes #2110

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

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

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

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

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

Use CanCanCan for oauth clients controller

Use CanCanCan for messages controller

Use a builder view for the capabilities call

This is easier to work with than building the XML document by hand
in the controller.

Use full list of osm xml root attributes in builder templates

Use CanCanCan for changesets controller

The expand_bbox method now needs require_write_api capability on tokens.

Use CanCanCan for traces controller

Use CanCanCan for swf controller

Skip authorization checks for amf controller

Add missing authorize_resource declaration to geocoder controller

Use CanCanCan for directions controller

Use CanCanCan for browse controller

Update bundle

Revert to omniauth-google-oauth2 version 0.5.3

Localisation updates from https://translatewiki.net.

Fix new rubocop warning

Update bundle

Localisation updates from https://translatewiki.net.

Use CanCanCan for export controller

Use CanCanCan for search controller

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