Changing a team owner assumes that it's the previous owner doing it

[William Grant]

From TeamReassignmentView._afterOwnerChange():

        if newOwner not in team.inactivemembers:
            team.addMember(
                newOwner, reviewer=oldOwner,
                status=TeamMembershipStatus.ADMIN, force_team_add=True)
        if oldOwner not in team.inactivemembers:
            team.addMember(
                oldOwner, reviewer=oldOwner,
                status=TeamMembershipStatus.ADMIN, force_team_add=True)

But a Launchpad admin, or a member of the old owning team, can also perform the action. The reviewer should be the current user, not the old owner.

[Robert Collins (lifeless)]

This may be a dupe; I remember whinging about it a couple months back.

[Launchpad QA Bot (lpqabot)]

Fixed in stable r16637 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/16637>.

[Curtis Hovey (sinzui)]

\o/

--
Curtis Hovey
Canonical Cloud Engineering
http://launchpad.net/~sinzui