Use REQ_topic in update_message_backend().

Note that we now strip the topic in the converter.
We also strip it in the view handler, which is now
redundant, but I left it in as defensive code.
This commit is contained in:
Steve Howell 2018-11-09 17:45:12 +00:00 committed by Tim Abbott
parent 3de1289d3e
commit c7515a853c
1 changed files with 1 additions and 3 deletions

View File

@ -1344,12 +1344,10 @@ def get_message_edit_history(request: HttpRequest, user_profile: UserProfile,
@has_request_variables @has_request_variables
def update_message_backend(request: HttpRequest, user_profile: UserMessage, def update_message_backend(request: HttpRequest, user_profile: UserMessage,
message_id: int=REQ(converter=to_non_negative_int), message_id: int=REQ(converter=to_non_negative_int),
subject: Optional[str]=REQ(default=None), topic_name: Optional[str]=REQ_topic(),
propagate_mode: Optional[str]=REQ(default="change_one"), propagate_mode: Optional[str]=REQ(default="change_one"),
content: Optional[str]=REQ(default=None)) -> HttpResponse: content: Optional[str]=REQ(default=None)) -> HttpResponse:
topic_name = subject
if not user_profile.realm.allow_message_editing: if not user_profile.realm.allow_message_editing:
return json_error(_("Your organization has turned off message editing")) return json_error(_("Your organization has turned off message editing"))