Make sure email replies to diary comments come from the right user

Add tab icon for safari/touchbar

Closes #1382

Clone objects before saving in case of a retry

If a deadlock occurs then the transaction will be retried so we
need to make sure that the object will still be dirty so that it
will be saved again during the retry but that the version won't
be incremented a second time.

Localisation updates from https://translatewiki.net.

Increase password stretching to 10000 interations

This is in line with current reccomendations from various sources.

Use only_path instead of setting an explicit host

Use image_url instead of image_path with the host option

Fixes #1379

Add missing xmlns:dc for notes rss

Use the assets pipeline for images in the changeset feeds.

This brings it into line with the other feeds.

Use a full url for the image in the diary rss feed.

This is what was intended by ee2a941d3eb7279aed5829ddbafe1cc9f9631459
which fixed the same problem in a different place.

Add missing xmlns declaration for diary_entries

This is already present in the other feeds.

Localisation updates from https://translatewiki.net.

Update to leaflet 1.0.2

Merge remote-tracking branch 'openstreetmap/pull/1375'

Fix misspelling (lon -> lng)

Fixes https://github.com/openstreetmap/iD/issues/3588
Closes #1374

Update to iD v2.0.1

Localisation updates from https://translatewiki.net.

Fix misspelling, @lng -> @lon

Update to iD v2.0.0

Removed duplicate entry for turn straight

Fixed typo trun vs turn

Use image_optim_rails instead of image_optim

The rails hooks have been extracted into a separate gem so
this hasn't actually been working since the last image_optim
update happened.

Localisation updates from https://translatewiki.net.

Stub nominatim for reverse lookups in note tests

Stub nominatim for reverse lookups in diary tests

Rename stub_signup_requests to stub_hostip_requests

Convert more 'setup do' to 'def setup' for consistency

Merge remote-tracking branch 'openstreetmap/pull/1350'

Update google logo

Fixes #1364

Disable zoom buttons when zoom limit is reached

Fixes #1365

Use 'def setup' instead of 'setup do', for consistency

Extract the hostip.info stubbing into a common method.

There doesn't appear to be any before(:suite) equivalent for Minitest
so this is the least-surprising way to do this.

Expire old oauth nonces

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Reject oauth nonces over a day old

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Fix rubocop warnings

Update bundle

Merge remote-tracking branch 'openstreetmap/pull/1352'

Merge remote-tracking branch 'openstreetmap/pull/1351'

Replace changeset_tags fixtures with a factory.

Remove the changesets_subscribers fixture.

Since there is no ChangesetsSubscribers model, we can't use a factory.
Instead we create the associations explicitly when needed.

Refactor the gravatar email changes to use webmock stubbing

The main reason for doing this is to make the tests easier to read,
rather than having to look up both the gravatar fixture, and then
correlate that with the users fixture. Putting the expected response
code in the tests is much more explicit.

Use webmock to power the with_http_stubs helper

This involves a small amount of changing the fixtures since we're
using the regexp and not the full url matching powers of webmock.

Stub out requests to gravatar.com during tests.

Stub out requests to hostip.info during tests

These are detecting the correct locale for showing terms during signup

Add webmock to intercept real http connections during tests

Intercepting real http connections also allows running the test
suite without an internet connection.

Merge remote-tracking branch 'openstreetmap/pull/1349'

Merge remote-tracking branch 'openstreetmap/pull/1348'

Avoid double-escaping diary entry titles

The XML builder takes care of the escaping, and adding h() lead to
double-escaped titles in the RSS feed.

Use dc:creator for note RSS feeds

The author tag is reserved for email addresses

Use dc:creator for diary entry RSS

The author tag is reserved for email addresses, and we already use
dc:creator in other feeds.

Use a full url for the image in the diary rss feed.

Picked up by a validator, see
https://validator.w3.org/feed/docs/error/InvalidFullLink.html

Make the image title match the feed title in diary rss

This is recommended at https://validator.w3.org/feed/docs/warning/ImageTitleDoesntMatch.html

Require user_roles fixtures

Update mailmap

Merge remote-tracking branch 'openstreetmap/pull/1344'

Ensure IE/Edge compatibility for routing drag&drop

Use the DataTransfer interface properly, using the `text` format, as
IE/Edge does not support much more. We can JSON-serialize anything
there, even though right now, only a single string would be enough.

Also, IE does not support .setDragImage, so check for support prior
to calling.

Fixes #1343

Reset database before running tests

Merge remote-tracking branch 'openstreetmap/pull/1339'

Merge remote-tracking branch 'openstreetmap/pull/1338'

Merge remote-tracking branch 'openstreetmap/pull/1337'

Merge remote-tracking branch 'openstreetmap/pull/1336'

Merge remote-tracking branch 'openstreetmap/pull/1335'

Convert the languages fixtures to a factory

For some tests, the existance of an 'en' language is assumed by
the database structure (not null default 'en'::character varying) so
this can be created in a setup block.

Replace messages fixtures with a factory.

Suppress XML parsing errors.

This technique was already used in app/models/changeset.rb

This suppresses the error messages when parsing invalid XML, but
the exceptions are still raised, as tested in test_from_xml_double_lat
in test/models/node_test.rb

Replace user_preferences fixture with a factory.

Replace hardcoded user id with a reference to the relevant user fixture

This makes it easier to figure out what the 1 refers to.

Convert ACL fixture to a factory, and add some tests

The fixture was unused, so I took the opportunity to put in a couple
of basic model tests.

Keep layer labels on top of the tiles in the layer switcher

Fixes #1334

Update bundle

Fix rubocop warnings

Replace quova with support for local Maxmind GeoIP lookups

Merge remote-tracking branch 'openstreetmap/pull/1332'

Load user and language fixtures for diary model tests

Update schema

Merge remote-tracking branch 'openstreetmap/pull/1331'

Merge remote-tracking branch 'openstreetmap/pull/1330'

Remove unused countries fixture.

Use model relations when creating objects from factories

Rather than passing around record ids explicitly, we can use the
model relations and pass around the models. This makes reading the
tests slightly simpler to read.

Replace user_blocks fixture with a factory.

Traits are used here to make the tests easier to read, and the
different traits can be combined where necessary.

Replace WordPress logo with correct version

See https://wordpress.org/about/logos/ for official source.

Closes #1329

Localisation updates from https://translatewiki.net.

Patch classes on the locate control

This patches the map on the user page in the same way as the main map.

Closes #1328

Add cursor style on close icon

Without this the close icon for the routing panel doesn't change the
pointer as it isn't a link and isn't in a close-wrap container.

Closes #1327

Localisation updates from https://translatewiki.net.

Fix rubocop warning

Merge remote-tracking branch 'openstreetmap/pull/1309'

Merge remote-tracking branch 'openstreetmap/pull/1322'

merge upstream/master and resolve conflicts from https://github.com/openstreetmap/openstreetmap-website/pull/1318

Replace changeset_comments fixtures with a factory.

There's little point in testing ChangesetComment.count so I've removed that
test.

Remove reference to standard layer in "no key" message

Fixes #1319

Remove unnecessary controller references from the redirect hashes.

Rails redirect_to uses the current controller by default, so there
is no need to re-iterate this in the code when the redirect targets
the current controller.

The short-form is already used elsewhere, this just tidies up those
which were still using the long-form.

Update structure.sql

adding back trailing newlines

adding back trailing newlines

last minor fixes