lpupdate.py should check if the team_owner is a person (not a team)

Bug #714718 reported by Ronnie
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
LoCo Team Portal
Fix Released
High
Ronnie

Bug Description

The script /teams/management/commands/lpupdate.py checks for a changed team_owner on launchpad, but it does not check if the owner is again a team.

If the owner is a team, then the team_owner should be the owner of the owner_team (loco_team.team_owner.team_owner)
All the members of the owner_teams should become team admins in LD

the "is_admin_or_owner" check in launchpad.py will than be unneccairy because it checks for "team.admin_profiles.filter(user__username=username):". This will increase the page speed for some pages in some special cases. And will also remove inconsistency, because this is the only function that do calls to launchpadlib on a page request. The other calls to launchpadlib are all scheduled tasks

Related branches

Ronnie (ronnie.vd.c)
description: updated
Michael Hall (mhall119)
Changed in loco-directory:
status: New → Confirmed
importance: Undecided → High
Ronnie (ronnie.vd.c)
Changed in loco-directory:
assignee: nobody → Ronnie (ronnie.vd.c)
Ronnie (ronnie.vd.c)
Changed in loco-directory:
status: Confirmed → In Progress
Ronnie (ronnie.vd.c)
Changed in loco-directory:
status: In Progress → Fix Committed
milestone: none → 0.3.1
Ronnie (ronnie.vd.c)
Changed in loco-directory:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.