Le mercredi 04 juin 2008 à 17:08 +0200, Morgan Collett a écrit : > The main code to review is at: > http://dev.laptop.org/git?p=users/morgan/sugar;a=shortlog;h=6298 (3 > most recent patches).
As bundle_id is passed to both constructor, you could move it to BaseInvite.__init__ Didn't read code carefully but InviteButton and InvitePalette still contain lot of: if shared: ... else: # private Maybe it would be worth to abstract these 2 classes too if possible? G. _______________________________________________ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar