Remove various implicit globals in Angular app

This patch does not remove globals used by timelines model: removing
these accidental globals would require addressing a number of
circular dependencies, e.g. between `Timeline` and `Project` objects.

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Fix Karma tests reliant on implicit globals

See a3a07f43.

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Fix missing PathHelper dependency in WPController

Signed-off-by: Alex Coles <alex@alexbcoles.com>
Test WP's spent hours property with read permission
Respect time entry view permission

Conflicts:

	app/models/work_package.rb
Fix cuke
Test that version custom field is correctly displayed
Display custom fields of type version
Merge pull request #2182 from opf/hotfix/17283_value_for_version_custom_field_not_displayed_in_split_view

17283 value for version custom field not displayed in split view
Merge pull request #2190 from opf/hotfix/4_17304_view_time_entries_permission_not_respected_on_wp_spent_hours

17304 view time entries permission not respected on wp spent hours
Merge pull request #2183 from opf/hotfix/remove-js-globals
Add missing dependency to Query
Add spent time WP property to documentation
Test API V3's WP response contains spent time
Add WP spent time property
Handle spent time correctly in overview
[REMOVE THIS AFTER DEBUGGING]

On Travis specs fail that run successfuly on my local machine.