Protect message reply functions with 'create' permission.
This commit is contained in:
parent
46923d40da
commit
ad9cd8be8e
|
@ -43,8 +43,10 @@
|
||||||
<%def name="message_tools()">
|
<%def name="message_tools()">
|
||||||
% if recipient:
|
% if recipient:
|
||||||
<div class="message-tools">
|
<div class="message-tools">
|
||||||
|
% if request.has_perm('messages.create'):
|
||||||
${h.link_to("Reply", url('messages.reply', uuid=instance.uuid), class_='button')}
|
${h.link_to("Reply", url('messages.reply', uuid=instance.uuid), class_='button')}
|
||||||
${h.link_to("Reply to All", url('messages.reply_all', uuid=instance.uuid), class_='button')}
|
${h.link_to("Reply to All", url('messages.reply_all', uuid=instance.uuid), class_='button')}
|
||||||
|
% endif
|
||||||
% if recipient.status == rattail.enum.MESSAGE_STATUS_INBOX:
|
% if recipient.status == rattail.enum.MESSAGE_STATUS_INBOX:
|
||||||
${h.link_to("Move to Archive", url('messages.move', uuid=instance.uuid) + '?dest=archive', class_='button')}
|
${h.link_to("Move to Archive", url('messages.move', uuid=instance.uuid) + '?dest=archive', class_='button')}
|
||||||
% else:
|
% else:
|
||||||
|
|
|
@ -333,11 +333,13 @@ class MessagesView(MasterView):
|
||||||
|
|
||||||
# reply
|
# reply
|
||||||
config.add_route('messages.reply', '/messages/{uuid}/reply')
|
config.add_route('messages.reply', '/messages/{uuid}/reply')
|
||||||
config.add_view(cls, attr='reply', route_name='messages.reply')
|
config.add_view(cls, attr='reply', route_name='messages.reply',
|
||||||
|
permission='messages.create')
|
||||||
|
|
||||||
# reply-all
|
# reply-all
|
||||||
config.add_route('messages.reply_all', '/messages/{uuid}/reply-all')
|
config.add_route('messages.reply_all', '/messages/{uuid}/reply-all')
|
||||||
config.add_view(cls, attr='reply_all', route_name='messages.reply_all')
|
config.add_view(cls, attr='reply_all', route_name='messages.reply_all',
|
||||||
|
permission='messages.create')
|
||||||
|
|
||||||
# move (single)
|
# move (single)
|
||||||
config.add_route('messages.move', '/messages/{uuid}/move')
|
config.add_route('messages.move', '/messages/{uuid}/move')
|
||||||
|
|
Loading…
Reference in a new issue