Fixes #4723: Adds an explicit restart of tomcat to the reset script
to ensure that tomcat is restarted before db:seed is run.
fixes 4725 - fixes deletion of puppet environments with no modules

Without this change, puppet environment would be deleted if for some reason the modules
did not exist on the puppet master in /etc/puppet/environments/

Also new content view versions without puppet modules would not have their environment
changed on the file system to reflect the modules actually in the the repo (none)
Fixes #4751: Environment and Content View were being omitted from the
allowed parameters for systems and preventing updating.
fixes #4767 - fixing content view search
Refs #4779 - Adding system group items to key API
fixes #4765: Content Views: Filters: sort errata by id
Fixes #4784: Replaces development setup documentation with references to
katello-devel-installer and katello-deploy.
Fixes #4501: rename katello JS i18n/mark translated bastion strings.

There was a javascript error on all katello pages when mark_translation was
turned on.  This was because katello was overwriting foreman's i18n global
var with it's own global var.  Renamed katello's global var to fix this.

Also added Xs around bastion strings marked for translation.
Merge pull request #3868 from daviddavis/ak-sysgroup

Refs #4779 - Adding system group items to key API
Fixes #4689 - Handle default content views in API/CLI
Merge pull request #3851 from jlsherrill/env_delete

fixes #4725 - fixes deletion of puppet environments with no modules
fixes #4780 - fixing link to system errata details

I believe this was caused with the upgrade to angular which
broke the way we were using isolate scope
fixes #4706 - adding back the ability to download a debug certificate

As part of an earlier commit to eliminate dead code, this
feature was accidentally removed.  Since is still needed
on the org edit page, it is being reintroduced.

A couple of notes:
- rather than bring back the old ui controller, the action
  is being added to the v2 api controller
- in order to reduce chance of the overrides being removed
  in the future, they are being placed in app/overrides/foreman
Merge pull request #3863 from bbuckingham/issue-4765

fixes #4765: Content Views: Filters: sort errata by id
fixes #4773 - Content Views: fix issue which did not allow for multiple errata ids per filter

The erratum filter rule 'types' is a serialized array.  The scope
that was placed on the model was incorrectly looking for 'nil';
however, with a serialized type, we really need to fetch the records
to look at the value of 'types', as the value for an existing
rule will not be 'nil', even when types is not specified.
Merge pull request #3858 from bbuckingham/issue-4706

fixes #4706 - adding back the ability to download a debug certificate
Fixes #4793 - Fixed typo in activation key show
Merge pull request #3870 from ehelms/fixes-4784

Fixes #4784: Replaces development setup documentation with references to
Merge pull request #3840 from ehelms/fixes-4723

Fixes #4723: Adds an explicit restart of tomcat to the reset script
Merge pull request #3864 from bbuckingham/issue-4773

fixes #4773 - Content Views: fix issue which did not allow for multiple errata ids per filter