Merge pull request #7 from faulteh/lca2017
Only allow managers of the review sections to email the speaker
This commit is contained in:
commit
85d4272080
2 changed files with 19 additions and 6 deletions
|
@ -185,7 +185,7 @@ def review_detail(request, pk):
|
|||
if not request.user.is_superuser and request.user in speakers:
|
||||
return access_not_permitted(request)
|
||||
|
||||
admin = request.user.is_staff
|
||||
admin = request.user.has_perm("reviews.can_manage_%s" % proposal.kind.section.slug)
|
||||
|
||||
try:
|
||||
latest_vote = LatestVote.objects.get(proposal=proposal, user=request.user)
|
||||
|
@ -208,7 +208,7 @@ def review_detail(request, pk):
|
|||
return redirect(request.path)
|
||||
else:
|
||||
message_form = SpeakerCommentForm()
|
||||
elif "message_submit" in request.POST:
|
||||
elif "message_submit" in request.POST and admin:
|
||||
message_form = SpeakerCommentForm(request.POST)
|
||||
if message_form.is_valid():
|
||||
|
||||
|
@ -282,7 +282,8 @@ def review_detail(request, pk):
|
|||
"reviews": reviews,
|
||||
"review_messages": messages,
|
||||
"review_form": review_form,
|
||||
"message_form": message_form
|
||||
"message_form": message_form,
|
||||
"is_manager": admin
|
||||
})
|
||||
|
||||
|
||||
|
|
|
@ -16,15 +16,27 @@ class TeamPermissionsBackend(object):
|
|||
if user_obj.is_anonymous() or obj is not None:
|
||||
return set()
|
||||
if not hasattr(user_obj, "_team_perm_cache"):
|
||||
# Member permissions
|
||||
memberships = Team.objects.filter(
|
||||
Q(memberships__user=user_obj),
|
||||
Q(memberships__state="manager") | Q(memberships__state="member"),
|
||||
Q(memberships__user=user_obj),
|
||||
Q(memberships__state="member"),
|
||||
)
|
||||
perms = memberships.values_list(
|
||||
"permissions__content_type__app_label",
|
||||
"permissions__codename"
|
||||
).order_by()
|
||||
user_obj._team_perm_cache = set(["%s.%s" % (ct, name) for ct, name in perms])
|
||||
permissions = ["%s.%s" % (ct, name) for ct, name in perms]
|
||||
# Manager permissions
|
||||
memberships = Team.objects.filter(
|
||||
Q(memberships__user=user_obj),
|
||||
Q(memberships__state="manager"),
|
||||
)
|
||||
perms = memberships.values_list(
|
||||
"manager_permissions__content_type__app_label",
|
||||
"manager_permissions__codename"
|
||||
).order_by()
|
||||
permissions += ["%s.%s" % (ct, name) for ct, name in perms]
|
||||
user_obj._team_perm_cache = set(permissions)
|
||||
return user_obj._team_perm_cache
|
||||
|
||||
def has_perm(self, user_obj, perm, obj=None):
|
||||
|
|
Loading…
Reference in a new issue