rubocop and erblint fixes

Refactor diary entry form

This uses the built-in helpers for naming form elements, rather than
having to override with specific i18n labels. It also changes the label
to 'Update', and capitalizes the title on the edit page.

Refactor some diary entry routes to use resources

Refactor diary entries to use create and update methods

This brings it slightly more into line with resourceful routing.

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

Update annotations to show 'using' clause on indexes

Update annotations to correctly show bigint types

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

Localisation updates from https://translatewiki.net.

Update to iD v2.15.1

Localisation updates from https://translatewiki.net.

Update to iD v2.15.0

Localisation updates from https://translatewiki.net.

Update database schema

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

Remove left over string from first draft

Exclude per-user diary pages from indexing

Add some extra indexes for ACL lookups

Block indexing of diaries

Localisation updates from https://translatewiki.net.

Reduce es lint moaning

Update user_creation_test for new parameter

Try to make erblint happy

Add checkbox for contributor terms and improve text

Remove unnecessary test for 0

Fix tests for change in users_controller

Rubocop fixes and increase per method complexity limit from 23 to 25

Style fixes

Fix tests for change in users_controller, add test for missing read_tou

Require that tou_read checkbox is set

Use jquery

Rename string to avoid translation confusion

Move link to ToU to attribution footer

Move onclick handling of tou checkbox to ujs

Add links to the ToU and include them in signup

- this adds links to the ToU in the Welcome box and on the About page
- adds a timestamp column to the users table to record acceptance of the ToU
- require ToU to be accepted before signup can proceed
- rearrange and change text on signup page

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Update bundle

Update polyfills

Update leaflet to 1.5.1

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Remove version constraint on postgres driver

Add geolink class to login/logout referer links

Fixes #2216

Localisation updates from https://translatewiki.net.

Teach travis to validate the database schema

Update database schema

Use rails 4.2 defaults for older migrations

Avoid error when there is no layer with a keyid

Fixes #2215

Return correct base layer ID

Add rubocop-performance

Fix new rubocop warnings

Disable failing cop pending upstream fix

Update bundle

Revert to tracking new versions of rinku

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Switch travis to use xenial

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

Localisation updates from https://translatewiki.net.

Resourceful(ish) naming for api/users_controller

Resourceful route names for api/trace_controller

Localisation updates from https://translatewiki.net.

Enable a number of eslint style rules

Enable some more eslint checks

Exclude generated i18n files from eslint checks

Use double quotes consistently in javascript code

Enable some more eslint rules from iD rules

Localisation updates from https://translatewiki.net.

Fix queuing of trace imports for API uploads

Fixes #2200

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

Fix issues found by eslint

Replace jshint with eslint

Localisation updates from https://translatewiki.net.

Use erb-lint to prohibit inline scripting

Format closed_at correctly in changeset API responses

Fixes #2195

Localisation updates from https://translatewiki.net.

Fix dependencies

Update to rails 5.2.3

Expect CORS preflight responses to not have a content type

https://github.com/cyu/rack-cors/pull/180

Update bundle

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

erblint: use consistent spacing in HTML tags

erblint: remove leading blank lines

erblint: avoid dynamic find_by methods

erblint: add rel=noopener to links that open in a new tab

erblint: use conditional assignment

erblint: use word arrays

erblint: use parentheses for nested function calls

erblint: use symbols as procs

erblint: avoid negated if conditions

erblint: prefer symbolic logical operators over and/or/not

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

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

Disentangle the api abilities from the web abilities

This will allow us to rename api actions without causing permissions headaches. The choice of
abilities files is made by inheriting from either api_controller or application_controller.

Also rename capabilities to api_capabilites, for consistency.

Set image dimensions in order to avoid a reflow after the page is drawn by the browser

Refactor notes#mine to simplify code

erblint: fix remaining single quoted strings