Pass the models, not the ids, when dealing with friendships

Rename user.friend_users to user.friends

This is possible now since we've renamed Friend to Friendship

Rename Friend model to Friendship

This better describes what the model represents, which is the relationship
between the two users.

Add translations for distance_in_time_ago

This allows translators to accurately translate times when they
need to modify the distance_in_time beyond just adding 'ago'.

Refs #2255

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

Update bundle

Pass through `photo_overlay` parameter
(re: https://github.com/openstreetmap/iD/issues/6492)

Update to iD v2.15.2

Localisation updates from https://translatewiki.net.

Allow indexing of diary entries again

Allow indexing of diary entries again

Fix importing of GPX traces with a leading byte order marker

Fixes #2258

Localisation updates from https://translatewiki.net.

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

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

Fix new eslint warnings

Replace eslint-rails-ee with direct calls to eslint

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

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

Use built-in form submit translation keys for traces

Use built-in form submit translation keys for oauth client applications

Use i18n for submit button for issue comments

Normally when an en-GB.yml translation is missing, rails falls back to en.yml. But when
using the submit helpers, if the en-GB translation is missing, rails knows how to create
a fallback like 'Create {model_name}' without touching the en.yml file. This string might
then be different from what the test expects, e.g. 'Add Comment'.

So it's important to set the language headers, to avoid phantomjs from picking up your desktop
preferences in this specific case.

Use built-in form sumbit translation keys for diary comments

Also use a diary comment object as the basis for the form

Use built-in form sumbit translation keys for messages

Use built-in form sumbit translation keys for redactions

Use built-in form sumbit translation keys for user blocks

Add a note about avoiding commits to other locale files

Round up the frame size for trace animations

Include first frame twice in trace animations

Test import of various trace formats

Fixes #2254

Fix rubocop warning

Install libarchive-dev for travis

Parse each GPX file in an archive separately

Fixes #2253

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

Update to released version 0.4.0 of gd2-ffij

Localisation updates from https://translatewiki.net.

Fix erblint warnings

Make hidden diary entries and comments more visible

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

Localisation updates from https://translatewiki.net.

Show all diary comments to administrators

Allow admins to unhide diary comments, if they wish

Show deleted diary entries to administrators, if the user isn't also deleted

This will allow administrators to review diary entry deletions from non-spam users.

Allow administrators to unhide diary entries, if they wish

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

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

Ignore the block length of the routes

Since the routes are more of a dsl than code, we've been happy to
repeatedly bump up this number. But then that hides other code which
has over-long block lengths hiding behind.

So instead let's exclude the routes from this metric.

Remove dead link to sitemap

Fixes #1113

Fix rubocop warning

Update rubocop todo

Add rubocop-rails

Update bundle

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

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

Update bundle

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

Use strong_migrations to help developers avoid problems in production database schema changes

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

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

rubocop and erblint fixes

Allow a configurable delay to keep diary entries from being published in feeds

This is to provide another tool to help spam fighting.

The configuration value is set to zero hours for new deploys in
order to be 'least surprising'. You can change the setting without
interfering with your test suite.

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

Remove SWF gpx trackpoints feature for Potlatch1

Exclude per-user diary pages from indexing

Add some extra indexes for ACL lookups

Block indexing of diaries

Localisation updates from https://translatewiki.net.

Allow moderators to hide diary entries and comments

Allow user_blocks to be created for longer periods

This avoids admins from having to manually change end periods in the database

Adjust to final animated GIF interface

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