From 8f54e25b0da9081748b3996bd3c990e7621df251 Mon Sep 17 00:00:00 2001
From: Ricky Zhou <ricky@fedoraproject.org>
Date: Wed, 12 Mar 2008 12:18:17 -0400
Subject: [PATCH] Looks like we're not using person.approve() right now.

---
 fas/fas/model.py | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/fas/fas/model.py b/fas/fas/model.py
index 6b312ca..b3ecb85 100644
--- a/fas/fas/model.py
+++ b/fas/fas/model.py
@@ -136,16 +136,6 @@ class People(SABase):
             role.member = cls
             role.group = group
 
-    def approve(cls, group, requester):
-        '''
-        Approve a person in a group  - requester for logging purposes
-        '''
-        if group not in cls.unapproved_memberships:
-            raise fas.ApproveError, _('user is not an unapproved member')
-        else:
-            role = PersonRoles.query.filter_by(member=cls, group=group).one()
-            role.role_status = 'approved'
-
     def upgrade(cls, group, requester):
         '''
         Upgrade a user in a group - requester for logging purposes