haiwen/seahub
1
0
Fork 0
mirror of https://github.com/haiwen/seahub.git synced 2025-09-02 15:38:15 +00:00
Code Issues Releases Wiki Activity

update groups api

Browse Source
This commit is contained in:
lian
2015-12-08 18:43:32 +08:00
parent 038a6122b4
commit c31f201d5b
2 changed files with 17 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
seahub/api2/endpoints/groups.py
View File

@@ -225,20 +225,23 @@ class Groups(APIView):
error_msg = _('Email %s is not valid.') % email
return api_error(status.HTTP_400_BAD_REQUEST, error_msg)
if email != username:
try:
if not seaserv.is_group_user(group_id, email):
seaserv.ccnet_threaded_rpc.group_add_member(group_id, username, email)
seaserv.ccnet_threaded_rpc.set_group_creator(group_id, email)
seaserv.ccnet_threaded_rpc.group_set_admin(group_id, email)
except SearpcError as e:
logger.error(e)
error_msg = _(u'Internal Server Error')
return api_error(status.HTTP_500_INTERNAL_SERVER_ERROR, error_msg)
else:
error_msg = _('You can not transfer a group to yourself')
if email == group.creator_name:
error_msg = _('%s is already group owner') % email
return api_error(status.HTTP_400_BAD_REQUEST, error_msg)
try:
if not seaserv.is_group_user(group_id, email):
seaserv.ccnet_threaded_rpc.group_add_member(group_id, username, email)
if not seaserv.check_group_staff(group_id, email):
seaserv.ccnet_threaded_rpc.group_set_admin(group_id, email)
seaserv.ccnet_threaded_rpc.set_group_creator(group_id, email)
except SearpcError as e:
logger.error(e)
error_msg = _(u'Internal Server Error')
return api_error(status.HTTP_500_INTERNAL_SERVER_ERROR, error_msg)
else:
error_msg = _(u'Operation can only be rename or transfer.')
return api_error(status.HTTP_400_BAD_REQUEST, error_msg)

2
tests/api/endpoints/test_groups.py
View File

@@ -126,7 +126,7 @@ class GroupsTest(BaseTestCase):
json_resp = json.loads(resp.content)
assert json_resp['creator'] == new_creator
def test_can_not_transfer_group_to_myself(self):
def test_can_not_transfer_group_to_group_owner(self):
new_creator = self.user.email
url = reverse('api-v2.1-group', args=[self.group_id])
data = 'operation=transfer&email=%s' % new_creator