"Messages have been posted since you loaded this page": Please also indicate if a thread has meanwhile been closed

Steffen

Well-known member
It would be nice if XF\Pub\Controller\Thread::actionDraft did not only check for new replies but for the thread's discussion_open state, too. This would prevent users from typing lenthy replies only to find out that the thread has meanwhile been closed by a moderator.

Simply adding a second message to the thread_save_draft template is not enough, the JavaScript function XF.QuickReply.onDraft needs to changed as well: It currently only inserts new messages if data.hasNew is true. Furthermore, if a .js-newMessagesIndicator element already exists then it skips new "messages" sent by the actionDraft method.

(Additionally, the next feature request would of course be to also notify the user when the thread is re-opened again. ;))
 
Upvote 5
Top Bottom