950943 - Fixing tags method for content views

The tags method called id instead of ids which returned the tag_id instead of a
hash with display_name which in turn caused an exception in the CLI code.
Adding a CONTRIBUTING document

Adding a CONTRIBUTING document to take advantages of this:
https://github.com/blog/1184-contributing-guidelines
 #961122: fixes "Missing template users/destroy" error.

 Since user.destroy may fail (if any of the callbacks fail, for example), we need to handle this case as well.
 Fixed similar issues in environment and repositories controllers.
Handle logout and session expiration better

Logout page was removed. Also when user logouts he ends up on Signo page
where he can login again. He is then redirected back to katello.
Fixes login issues with JS

When user entered wrong credentials a login screen still tried to fetch
allowed organizations. Also an error notification was not displayed.
965899 - do not include component view label in composite view path

Prior to this change, when a composite view is promoted
(e.g. Library to dev), the content path on pulp file
system would include both name of the composite view
as well as the name of the component view.  The component
view name is not necessary; therefore, it is being removed
from the path.

E.g.
Before:
/var/lib/pulp/published/https/repos/ACME_Corporation/dev/comp_view/zoo_view/custom/zoo_product/zoo_repo

After:
/var/lib/pulp/published/https/repos/ACME_Corporation/dev/comp_view/custom/zoo_product/zoo_repo
966396: Add missing ids in relation tables

Relation tables having its models has to have primary key `id`.
Without it ActiveRecord is not functioning properly. It was causing
nasty errors with malformed SQL statements e.g.
`"content_view_version_environments".""`.

There were two models without ids: RolesUser,
ContentViewVersionEnvironment. Unique index for table
`content_view_version_environments` on columns `content_view_version_id`
and `environment_id` was added.

Org and environment deletion were not working. Missing ids were breaking
deletion of relations to these models.

This was probably causing more problems than the two mentioned.

https://bugzilla.redhat.com/show_bug.cgi?id=966396
Merge pull request #2349 from pitr-ch/bug/missing-ids

Add missing ids in relation tables
Merge pull request #2324 from witlessbird/961122

 #961122: fixes "Missing template users/destroy" error.
Merge pull request #2310 from ares/bug/2502_session_expirations

Handle logout and session expiration better
965688 - properly looking through class deps to find indexed classes
Merge pull request #2357 from jlsherrill/reindex

965688 - properly looking through class deps to find indexed classes
965899 - added test for clone_repo_path for component view

Added a test to validate that the repo path that is used
when cloning a component view does not include that view's
label.  Instead, when a client interfaces with the repo
the full path will include the composite view.
Merge pull request #2361 from bbuckingham/965899

965899 - do not include component view label in composite view path
965735 - Default System Custom Info : No error displayed on trying
to enter a duplicate keyname
Merge pull request #2331 from daviddavis/temp_1368894675

950943 - Fixing tags method for content views
Merge pull request #2348 from ares/bug/login_unauthenticated_requests

Fixes login issues with JS
require ruby193scl-wrapper-ruby

but rather using path, because the name of package can be changed in RHSCLv2
And yes, we actually do not use scl_rake in spec
965790 - Publish user admin flag
Merge pull request #2364 from komidore64/bug_965735

965735 - Default System Custom Info : No error displayed on trying to enter a duplicate keyname