Fix clang build error

By default, Clang builds C code in GNU C11 mode, so it uses
standard C99 semantics for the inline keyword:

http://clang.llvm.org/compatibility.html#inline

Closes #1399

avatar inline attachment; text alignment

avatar inline attachment

reverting erroneous deletion in previous commit

intial commit, for 'en' locale only, for changeset_comment_notification only, doesn't load images properly yet

Add directions for straight on and slight left/right at roundabouts

Add directions for left/right turn at a roundabout

Localisation updates from https://translatewiki.net.

Update to iD v2.0.2

Localisation updates from https://translatewiki.net.

Use meta.wikimedia.org for wikimedia authentication

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

Update OWG policy links.

Add support for Wikimedia authentication

Closes #1146

Localisation updates from https://translatewiki.net.

Load api fixtures in tag model tests

We still need them so we have the object to attach tags to.

Get rid of hardwired counts in assertions

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

Fix misleading wording

We can't provide a free unlimited API for any third-party, not just developers. The change should further make translations less bumpy.

INSTALL.md grammar

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Don't collapse spaces in tag keys and values

Fixes #1386

Work around upcoming minitest insanity

Minitest 6 will not allow assert_equal to compare for equality
with nil and minitest 5 has already started warning about it.

That's fine if you're comparing with a nil constant, but if you're
comparing with an expression that is sometimes nil and sometimes
not nil it's an absolute pain in the rear end.

Fix rubocop warnings

Update bundle

Simplify monkey patch

The log method has grown new arguments which we weren't passing
on, but fortunately the exception translation is now delegated to
another method so we can patch that instead.

Remove redundant creation of an exception object

Make API and web roll back any open transactions on timeout

By default the exception thrown by Timeout::timeout is caught
using Kernel::catch so that it cannot be stopped by intermediate
exception handlers. The problem with that is that it stops any
database transactions that were in progress being rolled back
because they never see the exception.

Fortunately passing a class to Timeout::timeout changes it's
behaviour so that the exception is thrown and caught in the normal
way, allowing the database transactions to rollback.

Localisation updates from https://translatewiki.net.

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

Add tags to features before propogating them.

Fix spelling of propagate.

Be explicit in which traces are expected.

I think this is an improvement compared to running db queries to
fetch the expected results.

Localisation updates from https://translatewiki.net.

Create object for list tests, and assert the responses have the
required number of results.

Previously, the tests would pass regardless of whether anything was
in the database.

Rename test trace fixtures to use letters instead of numbers.

This prevents them from being deleted by mistake, if trace.delete is
called on a factory-generated trace with a coincidental id.

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.

Move instance creation to the top of the test methods.

Localisation updates from https://translatewiki.net.

Fix rubocop warnings

Update bundle

Replace fixtures with factory for relation_tags

Replace fixtures with factory for way_tags

Replace fixtures with factory for node_tags

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

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

Replace fixtures with a factory for old_relation_tags

Convert fixtures to factory for old_way_tags

Replace fixtures with a factory for old_node_tags

Also includes a slight simplification of the old_node_tags tests.

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.

Use stub_any_instance in order to stub find results.

This is a much cleaner approach than before.

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

Rubocop fix.

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