MorphGlobal/fog — pull request #51 by relaxdiego (2012-05-03 05:44:43+00:00)
Title: User role corrections in Tenant model

* Renaming the method from add_user to add_user_role clarifies the intent of the method which is to grant the user with a certain role in the tenant. Also, it is clear that we are adding another role to the user on top of any existing roles that she already has in the tenant.

* The opposite of add_user_role is remove_user_role which is provided as the second commit to this pull request. The naming also makes it clear that we are revoking a specific role previously granted to the user in the project. Not naming it "remove_user" removes any ambiguity on the intent of the method.