Unify link to user/group in application helper
link_to_principal method combines the functionality of the other two. They don't have much use right now. I propose to unify all 3 methods in one.
An attached patch removes
link_to_group methods and adds two aliases to
link_to_principal (user and group respectively).
link_to_group was used to redirect admin user to edit group page which is obsolete since all users have access to show group page and there is an edit link for admin.
Patch also adds a new test to confirm edit link presence for admins.
#3 Updated by Holger Just about 1 month ago
This changes the behavior in that the aliases
link_to_group now links to the group's show page, rather than its edit page. As far as I'm aware, the only place where we call
link_to_group today is in
app/views/users/show.html.erb template. Here, this change is probably okay.
(We might want to improve the
groups#show page a bit however, e.g. by adding links to the member users there. This is mostly unrelated to this here issue though.)