• This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn more.

Fixed Warnings - conversation sending (lack of error handling)

Xon

Well-known member
#1
When creating a warning, the option to send a conversation doesn't check if the title is set. If it isn't set (but the message and checkbox are), the warning is issued and the conversation silently fails to be created.

Code:
			$conversationInput = $this->_input->filter(array(
				'conversation_title' => XenForo_Input::STRING,
				'conversation_message' => XenForo_Input::STRING,
				'conversation_locked' => XenForo_Input::UINT,
				'open_invite' => XenForo_Input::UINT,
			));

			if ($conversationInput['conversation_title'] && $conversationInput['conversation_message'])
			{
...
			}
 

Chris D

XenForo developer
Staff member
#2
Thanks, this is sorted now.

We give a specific error in the event that we have a title and no message or have a message and no title.