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>
Merge pull request #2183 from opf/hotfix/remove-js-globals
Merge branch 'release/4.0' into feature/merge_release_4_into_dev

Conflicts:
	app/assets/javascripts/angular/helpers/path-helper.js
	app/assets/javascripts/angular/models/filter.js
	app/assets/javascripts/angular/services/timezone-service.js
	app/assets/javascripts/angular/timelines/directives/timeline-container-directive.js
	app/assets/javascripts/angular/ui_components/focus-helper.js
	app/assets/javascripts/angular/work_packages/controllers/work-packages-controller.js
Merge branch 'feature/merge_release_4_into_dev' into dev