Create an IssueCommentsController for managing IssueComments

Fix parameter spacing

Rework the 'issues not found' notice slightly.

Previously it would go into a redirect loop if there were no issues
in the database.

The link is to the list of issues, not of reports, and there's no need for it to be bold.

Ensure that issue comments have associated issues and users.

Add missing structure.sql changes from 948fd8c8faf1016d

Remove created_at / updated_at column definitions since these duplicate t.timestamps

Add banner for State of the Map Asia 2017

Closes #1630

Add padding between the last diary post and the pagination links

Closes #1634

Update secure_headers configuration for upstream changes

Update travis configuration to work on trusty based builders

Switch to postgres 9.5 and link postgres functions to a directory
that is accessible to the postgres server.

Update rails to 5.0.6

Ensure report details are not blank.

Refactor creating a new report to use a ReportsController

It's really a report that the user is creating, the issue is created indirectly. Similar
refactoring will be required for issue comments.

Use reports_count so that counter_cache works automatically.

Remove the border around the radio boxes

It looks strange.

Check that the flag is not show when not logged in

Add a basic feature test for reporting diary entries.

Add minitest-rails-capybara

This allows us to write feature tests using capybara. These can be
easily changed to system tests when we upgrade to rails 5.1

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

Rename xml processing methods with an update_ prefix

They behave differently from the other from_xml methods on other models.

Enable retina tiles for Thunderforest layers

Fixes #1623

Update to iD v2.4.1

Add iD hashtags parameter

Update to iD v2.4.0

Refactor the from_xml methods to act on existing trace objects.

Setting the new tags with the = operator takes care of removing the
old ones, and is the same approach as taken by the tagstring= method.

Fixes #1600

Add failing test demonstrating tag duplication when updating traces via the API.

Make sure we show errors correctly when resetting a password

Discard empty ways from query result geometries

Fixes #1608

Update to leaflet 1.2.0

Fix rubocop warning

Switch to using openstreetmap-deadlock_retry

Closes #1602

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

Add maintainers to README

It's nice to be explicit, so that new contributers know who is who.

While github shows "owner" and "member" labels these are for the
openstreetmap organisation as a whole, and can be misleading.

Make sure the account form is POSTed

Fixes #1601

Merge pull request #1588 from Georeactor/master

Update gem, fix icons on RTL/Arabic pages

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

Avoid passing the current_user.id when commenting

Doing so doesn't appear to be necessary, and it opens up a spoofing
hole if it's actually used.

Remove unused user save template.

This template isn't used, since all the controller methods redirect
to other views.

Use current_user instead of @user.

Merge branch 'master' into moderation

Fix user settings form

Fixes #1601

Update to rails 5.0.5

Update bundle

Update rack-cors and fix tests for changes in behaviour

Merge pull request #1597 from tomhughes/current-user

Complete @user to current_user transition

Replace @user with @current_user

This ensures that that we will find any more hidden references
to @user that might be hanging around...

Change user forms to use current user instead of @user

Convert @user to current_user in helpers

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

Convert @user to current_user

Use a current_user helper for accessing the logged in user in all views.

Convert remaining controller code to use current_user

The `self.current_user` is important when assigning to the current user,
to avoid creating a local variable called `current_user`

use no-r2 comment to block RTL gem from changing sprites

Update to iD v2.3.2

Update to Leaflet.Locate 0.62.0

Mark account suspended flash message as HTML safe

Fixes #1590

Clarify image dimensions

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

new png file for banner with square 350x350

Delete sotmlatam-2016.jpg

Delete sotmasia-2016.jpg

Delete donate-2016.jpg

image with proper width 350px

remove old entries in banner.yml

add sotm2017 in banner.yml

add png file with background

Add geoip_database key to example configuration

Fixes #1586

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

Use current_user to represent the currently logged in user.

This is already used by the oauth plugin, and is a general rails convention.

Standardise on basic_authorization without the parentheses

Use rails 5 version of redirect_back.

Remove indirection.

Set the reported_user in a callback

This avoids passing around the reported_user via forms. There was no
validation anywhere that the reported_user corresponded to the object
being reported. This approach removes those worries too.

Rubocop autofixes.

Rename association to reported_user, for clarity

Get tests passing. Use factories instead of fixtures.

Merge branch 'master' into moderation

Update to iD v2.3.1

Add FAQ and information about creating a banner image

Replace timecop with the rails builtin travel_to method

Add iD walkthrough autostart parameter

Update to iD v2.3.0

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

Add some tests for the user helper

Revert unrelated changes from db/structure.sql

Whitespace fixes.

Remove unused files

Revert changes to Vagrantfile

Hide contributor terms details, unless the user hasn't agreed

It's no longer important to show when users agreed to the contributor terms.
For users that haven't agreed, we still show their status on their profile.

Logic was also re-written to avoid lots of `if not` things, and tests added.

Don't disable search submission buttons on form submission

Fix more parameter sanitisation issues and add tests

Disable forgery protection for notes API methods

Fixes #1571

Sanitize parameters for various paged views

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

Update to leaflet 1.1.0

Monkey patch oauth-plugin to avoid using deprecated methods

Fix some tests for changes in rails 5

Update get/post etc to remove use of deprecated syntax

Update to rails 5.0.4