The way the checks are currently done, one cannot swap the login and alt_login field in one go.
This is useful, since the normal login field is the `canonical' author name.

The appended patch makes more precise requests to allow for alt_login to be the login for the same uid and vice-versa.

It relies on alt_login vs. login comparison to make certain we can't set both to the same value.

i don't think I missed anything, but still it needs review...

CommentFileSizeAuthor
z.patch1.31 KBespie

Comments

hunmonk’s picture

Status: Needs review » Fixed

patch tested out well. code style looks good. committed to 5.x and 6.x branches.

thanks!

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.