BP Messages - Causes PHP Error - Wrong call of a function

[Jon Ziemlich]

FIX: Remove line 104 at bp-messages/bp-messages-actions.php: $feedback = $send->get_error_message();

Hey there,

bp-messages/bp-messages-actions.php on line 104:

$feedback = $send->get_error_message();

The Variable $send is defined on line 87 $send = messages_new_message( ... .

The function messages_new_message is defined in bp-messages/bp-messages-functions.php

This function returns in no case an object. Only an int or bool.
So line 104 at bp-messages/bp-messages-actions.php is wrong and causes errors!

Thank you.

Best wishes
Jon Ziemlich

[DJPaul]

Thank you for the report, @Jon Ziemlich.

How exactly are you causing this to fail? How did you discover the problem?

[Jon Ziemlich]

Sorry i saw now, that in some cases WP_Error Objects are returned. I will test it and send you as i know more.

[boonebgorges]

See [10286].

It does appear that there's one situation where a failed message post may result in false being returned: when $message->send() fails for a miscellaneous reason (such as a database error) near the end of messages_new_message(). @r-a-y we should probably do an error_type check here, right? Can we write a test that triggers this kind of failure?

[r-a-y]

Ugh... my bad!

Attached patch should fix the issue.

Jon - Can you test the patch and see if that fixes the issue for you?

[r-a-y]

Received a report that the patch works.

Going to commit.

[r-a-y]

In 10585:

Messages: Return generic WP Error in messages_new_message() when necessary.

In #6535, we added an 'error_type' parameter in messages_new_message()
that supports returning a WP Error object if error type is set to
'wp_error'. However, we did not return a WP error further down in the
messages_new_message() function, which can result in a potential fatal
error to occur.

This commit rectifies this and includes a unit test.

Anti-props r-a-y.

See #6894 (trunk).

[r-a-y]

In 10586:

Messages: Return generic WP Error in messages_new_message() when necessary.

In #6535, we added an 'error_type' parameter in messages_new_message()
that supports returning a WP Error object if error type is set to
'wp_error'. However, we did not return a WP error further down in the
messages_new_message() function, which can result in a potential fatal
error to occur.

This commit rectifies this and includes a unit test.

Anti-props r-a-y.

Fixes #6894 (2.4-branch).