[buddypress-trac] [BuddyPress Trac] #8578: "Send To" field text/recipient of new message thread cleared when user attempts to send message without a "Subject" or "Message" text

[buddypress-trac]

#8578: "Send To" field text/recipient of new message thread cleared when user
attempts to send message without a "Subject" or "Message" text
-------------------------+---------------------
 Reporter:  niftythree   |       Owner:  (none)
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  10.0.0
Component:  Messages     |     Version:  2.4.0
 Severity:  normal       |  Resolution:
 Keywords:  needs-patch  |
-------------------------+---------------------
Changes (by imath):

 * keywords:   => needs-patch
 * version:  9.1.1 => 2.4.0
 * type:  defect (bug) => enhancement
 * milestone:  Awaiting Review => 10.0.0


Comment:

 Hi thanks for your feedback. The reason is if these two post variables are
 empty we stop the sending process and redirect the user to the same
 screen.

 I agree it's a bit annoying to have to retype the conversation
 participants. So it could be improved :
 - using JavaScript to intercept the form submission and interrupt it if
 the corresponding field are missing if it's not a sitewide notice.
 - From the `bp_messages_action_create_message()` function adding query
 vars containing the usernames to the redirection URL.

 Who wants to submit a patch! I'd be happy to review it and help you have
 it committed 😉

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8578#comment:1>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac