Ronnie has proposed merging lp:~ronnie.vd.c/loco-directory/lpupdate-admin into 
lp:loco-directory.

Requested reviews:
  loco-directory-dev (loco-directory-dev)
Related bugs:
  #728766 Crash when running lpupdate
  https://bugs.launchpad.net/bugs/728766

For more details, see:
https://code.launchpad.net/~ronnie.vd.c/loco-directory/lpupdate-admin/+merge/52190

Fixed the lpupdate-admin bug
-- 
https://code.launchpad.net/~ronnie.vd.c/loco-directory/lpupdate-admin/+merge/52190
Your team loco-directory-dev is requested to review the proposed merge of 
lp:~ronnie.vd.c/loco-directory/lpupdate-admin into lp:loco-directory.
=== modified file 'loco_directory/teams/management/commands/lpupdate.py'
--- loco_directory/teams/management/commands/lpupdate.py	2011-02-19 00:55:59 +0000
+++ loco_directory/teams/management/commands/lpupdate.py	2011-03-04 12:47:34 +0000
@@ -34,10 +34,13 @@
         except UserProfile.DoesNotExist:
             pass
     for admin_name in lp_admin_set.difference(ld_admin_set):
-        admin = create_profile(admin_name)
-        admin.realname = lp_admin_names[admin_name]
-        admin.tz = launchpad.get_user_timezone(admin_name, lp)
-        admin.save()
+        try: 
+            admin = UserProfile.objects.get(user__username=admin_name)
+        except UserProfile.DoesNotExist:
+            admin = create_profile(admin_name)
+            admin.realname = lp_admin_names[admin_name]
+            admin.tz = launchpad.get_user_timezone(admin_name, lp)
+            admin.save()
         ld_team.admin_profiles.add(admin)
         
 def get_team_owner(lp_team):

_______________________________________________
Mailing list: https://launchpad.net/~loco-directory-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~loco-directory-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to