Fix functionality and usages of SpecialUserrights::userCanChangeRights
SpecialUserrights::userCanChangeRights is a funky method that is
called by things outside of SpecialUserrights to see if they should
display a link to it. It had some bugs and missing documentation.
* SpecialUserrights::userCanChangeRights relied on $this->isself to
see if the acting user and target user are the same, but it gets set
in execute() which never runs when used like this.
* It wasn't clear whether the $user parameter represented the acting
user or the target user, resulting in incorrect usage in
SkinTemplate.
The net effect of these bugs is that skin sidebar would display
"Change user groups" instead of "View user groups" for all users when
the current user was only allowed to change some of their own groups.
Change-Id: Ie47b9c7463b373fe17006567239aa09e824b015d