Drop redundant monkey patches

Add functions tests for the oauth_clients controller

Remove unused controllers

Remove stub test for non-existent controller

Don't linkify comments in the changeset list

It's confusing to make links in comments active in the changeset list
as the rest of the comment is an active link to the changeset.

We do want to make them active in the changeset view however, so add
linkification to the comment there.

Fixes https://trac.openstreetmap.org/ticket/5061

Don't show lat/lon for a deleted node, in case it is redacted

Fixes #642

Fix untranslatable string

Test XHR rendering of pages which can load in the sidebar

Do basic testing of all site controller methods

Revert "Remove unused file"

This reverts commit db41978f098f6044bf007b051525ff9d1c9149c9.

Add link to per-user RSS feed on diary view page

Improve display of alternative download sources

Makes the list of alternative sources displayed all the time
rather than just when the bounding box is too large, and adds
the Overpass API to the list of sources.

Fixes #606
Fixed #635

Remove unused file

Put all feed discovery tags in the auto_discovery_link_tag section

Diary and trace feeds are RSS not Atom

Fixes #638

Make title of "More" menu translatable

Fix interpolation variable errors

Add some more unit tests for the Trace model

Localisation updates from https://translatewiki.net.

Reapply leaflet.locate patch

Update leaflet.locate

Update to Leaflet 0.7.1

Obey the location hash for edit URLs with objects

For a link like http://osm.org/edit?node=n#map=19/37.77683/-122.41459,
select the feature but also respect the map location.

This is the fix for iD. It appears P2 requires internal code
modifications.

Refs #620

Pass details of any selected object to remote editors

Fix translation interpolations

Localisation updates from https://translatewiki.net.

Run unload method before pushing the new state

If running unload causes any hash changes, those should
not end up on the new URL.

Fixes #622

Ensure object parameter is removed from edit link when closing sidebar

Refs #620

Restore marker for non-browse search results

Fixes #615

Remove object parameter from share URL

Fixes #616
Fixes #624

Extend API to allow changesets to be queried by ids

Make changesets queryable by specifying a list of ids, for example:

    /api/0.6/changesets?changesets=1,2,3

This condition may be combined with others in the normal way.

Report an error if a bogus limit value is passed to a notes API call

Test get_nodes_undelete methods of the OldWay model

Make remote edit work when /edit is loaded

Change detection of the edit request to use the URL as the class
will not be set on the body now, and force the router to think we
actually loaded the normal view.

Use the map layout when rendering index for a remote edit

Always use controller computed zoom when starting Potlatch

The value of mapParams.zoom may have come from the cookie and be
unrelated to the actual view the user came from with this click, so
just use the default zoom set by the controller.

Fixes #621.

The value of mapParams.object is always a node/way/relation now

Simplify a number of object model methods

Add tests for member methods on relation models

Add tests for node methods on way models

Update i18n to fix bogus deprecation warning

Add tests for tag methods on object models

Fix OldRelation.tags

Position on notes correctly when a link to a comment is loaded

Links to note comments in the RSS feed have a #id on the end to
position on the right comment, so don't let those hash strings
stop the map from panning to the note.

Fixes #603.

Unconditionally unbind moveend event

Can't rely on checking location.pathname -- inside unload,
it already contains the new path. Fortunately, checking is
unnecessary.

Fixes TypeError: Cannot call method 'split' of undefined.

More refactoring of common code in object models

Tidy up code and extract generation of tag elements to the
common code in the ObjectMetadata module.

Refactor generation of object metadata in API calls

Factor out duplicate from the object models into common code.

Use a table for tags

Fixes #595

Update to rails 4.0.2

Update history feed URL with bbox parameter

https://lists.openstreetmap.org/pipermail/dev/2013-December/027599.html

Remove map data size warning (#592)

Restore support for #layers=X with no map parameter

https://trac.openstreetmap.org/ticket/5056

Replace some legacy routes with redirects

Remove debugging code

Fix UCL link

Throw an exception instead of logging to the console

Log failures to match a route for the initial path

Increase z-index for maximised content

Makes sure maximised content sits on top of the header.

Remove unused code for handling search results in Potlatch

Improve zoom level selection when invoking Potlatch on an object

When invoking Potlatch on a node/way/relation object we should
normally have valid location information in the map parameters, so
use any zoom which is there if possible, otherwise fall back to an
object type specific default value.

Fixes #605.

Display the size warning when the data layer is disabled

Auto-uncheck the data and notes layers at low zoom

Reenable them when returning to high zoom in the
same session.

Fixes #592

Use jQuery Simulate to simulate click events

Merge remote-tracking branch 'jfirebaugh/562'

Improve click behavior on changeset entries

On Chrome, this lets clicks with modifier keys anywhere
on the changeset entry work as expected. On other browsers,
modifier keys still only work on the link itself.

Align "Way" in relation member list

Fixes #590

Allow word-breaks in long keys/values

Fixes #561

Generate valid URLs for search results with no object attached

Restore correct link behavior with modifier keys

Fixes #562

Fix application error on non-feature search results

Fixes #560

Merge remote-tracking branch 'jfirebaugh/legacy-params'

Document need for a Javascript runtime

Don't show "Load More" if we found less than 20 changesets

Support legacy bbox param on /history as well

Fixes #586

Restore legacy bbox/min/max params support

https://trac.openstreetmap.org/ticket/5050
https://trac.openstreetmap.org/ticket/5051

Remove unused strings

Restore click handler for changeset entries

The default click handler only handle the, not the rest of the
entry, so retore the handler but add an e.preventDefault() call
to stop the second handler running.

Drop click handler for changeset entries in the history sidebar

The default click handler in index.js handles them fine, and as this
one was not calling preventDefault both were actually firing in some
browsers, causing the changeset to be loaded twice and two history
entries to be created.

Removed bogus translation

Localisation updates from https://translatewiki.net.

Prevent wrapping of dates in message lists

Fix javascript eror redirecting to new notes

Fix URL updating in Potlatch 1

Potlatch 1 has hard coded calls to updatelinks, so rename the
updatelinks function to updateLinks and add an updatelinks function
using the old style argument list for Potlatch 1 to call.

Fix stripping of trailing slash

Strip any trailing slash from the initial location

Add 'x' to close welcome box; remember preference

Drop monkey patch now we're using option style redirects

Option style redirects escape path parameters correctly, so we
can drop out monkey patch.

Force the X-Page-Title value to be treated as UTF-8

This is basically a black magic hack to get javascript to treat
a string as UTF-8 and decode it to the correct form.

http://monsur.hossa.in/2012/07/20/utf-8-in-javascript.html

Preserve the bounding box when redirecting history URLs

Using the options style for the redirect seems to cause parameters
to be preserved, which the path style of redirect does not.

Rename changed translation to avoid test failures

Localisation updates from https://translatewiki.net.

Don't offer iD on IE11

Remove spurious < character as reported in #508

Load gravatar images over https on https pages

Add margin space below lists on non-map content pages

Test for the map argument correctly

Remove monkey patch which has been merged upstream in rails 4

Monkey patch escaping in redirect routes

The correct method of escaping depends on whether the parameter
is being substituted in the path or the query, but all our ones
are substitued in the path so use URI.escape instead of the
standard Rack::Utils.escape which does query escaping.

https://github.com/rails/rails/issues/13110

Don't throw an exception if the hash doesn't contain a map argument

Redirect /?query= to /search?query=