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

Replace trace-related fixtures with factories.

The mocking and stubbing in the controller tests is
unfortunate, but the models interact directly with the filesystem
using the trace id so that's hard to control any other way.

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

rubocop clean up

break out author subscriptions to its own migration

Add a note_with_comments factory

Makr author/author_ip tests more consistent

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

clean up comments

fix rubocop warnings

fix rubocop warnings

css in right place; fix structure.sql

tests passing

more idiomatic models for diary entry subscriptions

Remove the note and note_comments fixtures

You may need to run "DELETE FROM notes" and "DELETE FROM note_comments"
on your local test database.

Update notes_controller_test to use note and note_comment factories

I've ported across the naming for each note from the old fixture names.
This helps in the mega-tests where multiple notes are tested at the
same time.

The number or results have changed in some tests, since there's little
point in creating e.g. 5 note comments when 2 is enough to test the
functionality.

Update site_controller_test to use note and note_comment factories.

Update browse_controller_test to use note and note_comment factories.

Fix bugs->notes in comment

Update note model tests to use factories.

The NoteTest.test_visible? has changed slightly, so that the second
test of .visible? is using a closed note, which is a more useful test
than the original.

Add factories for notes and note_comments

Localisation updates from https://translatewiki.net.

Replace friends fixture with a factory

not api endpoints, moved to button, fixed notifier message, fixed tests

WIP diary comment subscriptions

Update leaflet to 1.0.1

Patch classes on the locate control

This replaces the patching of the vendored source that we had been
doing before and which we always forget to apply after updating it.

Fixes #1306

Remove z-index override from leaflet controls on small screens

Fixes #1305

Patch leaflet-locationfilter for leaflet 1.x support

https://github.com/kajic/leaflet-locationfilter/issues/22
https://github.com/kajic/leaflet-locationfilter/pull/23

Update leaflet to 1.0.0

Localisation updates from https://translatewiki.net.

Extend donation drive banner end date to 2016-oct-31

Localisation updates from https://translatewiki.net.

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

Fix rubocop warnings

Centre loading icon

Add links to new approved OSMF privacy policy and some rewording.

look for extra place tags in Nominatim search results

Administrative boundaries at municipality level are generally
better described by the place node (as town, village etc.).
Nominatim exports the place type in extratags if it was able
to merge place nodes with admin boundaries. Use this preferably
to create the descriptive term.

Remove SOTM 2016 banner, add Donation Drive 2016 Banner

Localisation updates from https://translatewiki.net.

Add thunderforest API key to embedded maps

Fixes #1293

Fix some rubocop warnings

Update to rails 4.2.7.1

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

Fix new rubocop warnings

Rework R2 support for changes in sprockets

Update bundle

Add additional list items to ensure they are not selected

The other list tests already have assetions showing the different
lengths of lists.

Localisation updates from https://translatewiki.net.

Prioritize next upcoming event in banner rotation
(closes #1284)

Remove unneccessary fixture requirement.

Remove diary_entry and diary_comment fixtures, and refactor tests.

Note that you might need to empty the tables in your local test
database, since the removed fixtures may still linger there.