diff --git a/seahub/views/ajax.py b/seahub/views/ajax.py index 50dd32ce29..f6b00aa433 100644 --- a/seahub/views/ajax.py +++ b/seahub/views/ajax.py @@ -2295,11 +2295,12 @@ def set_user_folder_perm(request, repo_id): if not user_folder_perm: try: seafile_api.add_folder_user_perm(repo_id, path, perm, user) - send_perm_audit_msg('add-repo-perm', request.user.username, user, \ - repo_id, path, perm) - except SearpcError, e: - return HttpResponse(json.dumps({"error": e.msg}), status=500, - content_type=content_type) + send_perm_audit_msg('add-repo-perm', request.user.username, + user, repo_id, path, perm) + except SearpcError as e: + logger.error(e) + return HttpResponse(json.dumps({"error": _('Operation failed')}), + status=500, content_type=content_type) else: return HttpResponse(json.dumps({"error": _('Already add this folder permission')}), status=400, content_type=content_type) @@ -2308,11 +2309,12 @@ def set_user_folder_perm(request, repo_id): if user_folder_perm and user_folder_perm != perm: try: seafile_api.set_folder_user_perm(repo_id, path, perm, user) - send_perm_audit_msg('modify-repo-perm', request.user.username, user, \ - repo_id, path, perm) - except SearpcError, e: - return HttpResponse(json.dumps({"error": e.msg}), status=500, - content_type=content_type) + send_perm_audit_msg('modify-repo-perm', request.user.username, + user, repo_id, path, perm) + except SearpcError as e: + logger.error(e) + return HttpResponse(json.dumps({"error": _('Operation failed')}), + status=500, content_type=content_type) else: return HttpResponse(json.dumps({"error": _('Wrong folder permission')}), status=400, content_type=content_type) @@ -2321,14 +2323,16 @@ def set_user_folder_perm(request, repo_id): if user_folder_perm: try: seafile_api.rm_folder_user_perm(repo_id, path, user) - send_perm_audit_msg('delete-repo-perm', request.user.username, user, \ - repo_id, path, perm) - except SearpcError, e: - return HttpResponse(json.dumps({"error": e.msg}), status=500, - content_type=content_type) + send_perm_audit_msg('delete-repo-perm', request.user.username, + user, repo_id, path, perm) + except SearpcError as e: + logger.error(e) + return HttpResponse(json.dumps({"error": _('Operation failed')}), + status=500, content_type=content_type) else: return HttpResponse(json.dumps({"error": _('Please add folder permission first')}), status=400, content_type=content_type) + return HttpResponse(json.dumps({'success': True}), content_type=content_type) @@ -2364,11 +2368,12 @@ def set_group_folder_perm(request, repo_id): if not group_folder_perm: try: seafile_api.add_folder_group_perm(repo_id, path, perm, group_id) - send_perm_audit_msg('add-repo-perm', request.user.username, \ + send_perm_audit_msg('add-repo-perm', request.user.username, group_id, repo_id, path, perm) - except SearpcError, e: - return HttpResponse(json.dumps({"error": e.msg}), status=500, - content_type=content_type) + except SearpcError as e: + logger.error(e) + return HttpResponse(json.dumps({"error": _('Operation failed')}), + status=500, content_type=content_type) else: return HttpResponse(json.dumps({"error": _('Already add this folder permission')}), status=400, content_type=content_type) @@ -2377,11 +2382,12 @@ def set_group_folder_perm(request, repo_id): if group_folder_perm and group_folder_perm != perm: try: seafile_api.set_folder_group_perm(repo_id, path, perm, group_id) - send_perm_audit_msg('modify-repo-perm', request.user.username, \ + send_perm_audit_msg('modify-repo-perm', request.user.username, group_id, repo_id, path, perm) - except SearpcError, e: - return HttpResponse(json.dumps({"error": e.msg}), status=500, - content_type=content_type) + except SearpcError as e: + logger.error(e) + return HttpResponse(json.dumps({"error": _('Operation failed')}), + status=500, content_type=content_type) else: return HttpResponse(json.dumps({"error": _('Wrong folder permission')}), status=400, content_type=content_type) @@ -2390,11 +2396,12 @@ def set_group_folder_perm(request, repo_id): if group_folder_perm: try: seafile_api.rm_folder_group_perm(repo_id, path, group_id) - send_perm_audit_msg('delete-repo-perm', request.user.username, \ + send_perm_audit_msg('delete-repo-perm', request.user.username, group_id, repo_id, path, perm) - except SearpcError, e: - return HttpResponse(json.dumps({"error": e.msg}), status=500, - content_type=content_type) + except SearpcError as e: + logger.error(e) + return HttpResponse(json.dumps({"error": _('Operation failed')}), + status=500, content_type=content_type) else: return HttpResponse(json.dumps({"error": _('Please add folder permission first')}), status=400, content_type=content_type)