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

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.

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`

Update to iD v2.3.2

Update to Leaflet.Locate 0.62.0

Mark account suspended flash message as HTML safe

Fixes #1590

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

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

Update to rails 4.2.9

Rework coordinates to avoid scientific formatting of small numbers. Fixes #1509

Refactor AMF encodevalue routine

Simplify the switch, and raise an exception if we see an unexpected
type rather than just logging and then accidentally triggering a odd
error by returning true when a string is expected.

Update bundle

Disable removeUnknownsAndDefaults instead of removeUselessStrokeAndFill

Convert XML document to a string when returning it

Otherwise the Rak::ETag module will throw an exception when it
tries to test if the response is empty.

Disable removeUselessStrokeAndFill svgo plugin
see https://github.com/openstreetmap/iD/issues/4105

No need to precompile traffico anymore, as it's been removed

Update to iD v2.2.2

Remove old traffico mapillary signs library from Vendorfile

Update bundle

Add explicit primary key declaration for changeset subscriptions

This table has a composite primary key and although we don't really
rely on that at all rails 5.0 warns about itif you don't explcitly
declare it in order to trigger the CPK extension.

Remove deprecated get_via_direct and post_via_redirect methods

Use explicit to_unsafe_h method when converting parameters to a hash

Replace deprecated ActiveModel::Errors get/set methods

Replace deprecated env method with request.env

Use distinct instead of uniq which is deprecated

Replace render :nothing with non-deprecated alternatives

Sanitise parameters used in URL generation

Replace render :nothing with head

Replace deprecated xxx_filter methods with xxx_action

Replace deprecated Mime::XXX with Mime[:xxx]

Avoid using format as a URL parameter name

This prevents rails confusing it with the builtin format
parameter derived from the URL extension.

Make delete button translatable

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

Add delete button to message view

Fixes #1504
Closes #1557

Sanitise parameters used in URL generation

Remove duplicate size specification for user image

Either the size or the width and height can be passed but passing
both doesn't make any sense and in rails 5.0 will error.

Remove all use of the :text option to render

It doesn't actually do what it says, as it sets the content type
to text/html not text/plain so is just confusing and as a result
has been deprecated in newer rails versions.

Remove conditions from delete_all

Passing conditions directory to delete_all is deprecated
in rails 5.0 so use a separate where instead.

Rescue LoadError instead of MissingSourceFile

MissingSourceFile is just an alias for LoadError in rails 4.2 and
in rails 5.0 it is no longer present.

Remove arguments from assert_nothing_raised

It has never done anything with the argument and rails 5.0 warns
that rails 5.1 will remove the argument.

Move monkey patch modules to the OpenStreetMap namespace

Having them in the OSM namespace risks blocking autoloading
of the lib/osm.rb code by defining the OSM constant.

Fix rubocop warnings

Update bundle