Tagged: bug, friendship
Hi all,
it seems I just had to find a bug in the plugin extended friendship requests.
The upgrade from v1.2.0 to v1.2.1 added some support for buddyboss / buddypress.
So instead of one hook, you now have two hooks:
add_action( 'bp_friend_requests_item', array( $this, 'show_message_legacy' ) ); add_action( 'bp_directory_members_item', array( $this, 'show_message_nouveau' ) );
You have two functions, show_message_legacy and show_message_nouveau, basically doing the same. Within those functions, there is a check, whether the hook is for BB or BP. This check seems to be buggy / reversed, so I think the ultimate function to show the message never runs.
We use buddyboss, and I traced it, for us it’s the hook bp_friend_requests_item which is ran – but the check checks for Buddypress…
And what’s more:
The check is not needed at all, because you simply run the internal show_message() function anyhow.
So why not remove show_message_legacy() and show_message_nouveau() completely from the code and simply hook into:add_action( 'bp_friend_requests_item', array( $this, 'show_message' ) ); add_action( 'bp_directory_members_item', array( $this, 'show_message' ) );
?
THANKS SO MUCH!!
BTW: Maybe it worked back in July.
I assume, that maybe buddyboss implemented bp_directory_members_item earlier, which was the base for you. But at least in the newest version, bp_friend_requests_item is used. bp_directory_members_item does certainly not run.But at least, with my suggestion, of using the same callback for both hooks, it works as expected!!
Hi SX,
Thank you for reporting.Our team will test and get back to you with details after 3rd January.
Regards
BrajeshHi SX,
I am very sorry we did not reply earlier. We had a miscommunication on the assignment on this task.Your assumption of bug is incorrect.
I believe you are referring to this commit.Please check their purpose. There is no inversion of logic. We can not guarantee that it works with current version of BuddyBoss as BuddyBoss keeps changing things.
Please correct me where is the “VERY NASTY BUGS” that you reported?
Thank you
Brajesh
You must be logged in to reply to this topic.