line #956,
// Only update if reply is published
if ( ! bbp_is_reply_pending( $reply_id ) ) {
As per the comment, it should check if reply is published. But this condition will be true even if the reply is marked as spam. Instead it should use if ( bbp_is_reply_published( $reply_id ) ). Same thing I have observed for topic as well. If topic is marked as spam by akismet, the last active parameters reflects the spam topic instead of last published topic.
Should I create a ticket to address this issue?
Any update on this?
I have digged again further for the issue. The issue has started during this ticket #2838 with the changeset #6923. The last reply and count should only be changed when reply is public regardless whether it is spam or in pending state.
Looks like it was fixed way back when codebase was not modularized to this extent. Here is the ticket that was closed some 13 years back with commit that fixes it.
If you observe these check doesn’t exist anymore before updating last reply id and all.
// Update if reply is published
if ( bbp_is_reply_published( $reply_id ) )
update_post_meta( $topic_id, '_bbp_last_reply_id', (int) $reply_id );
Even I am facing this issue. Ideally they should not update reply count or any other metrics if the reply or topic is form. I have found a way to tweak the codebase and do it but I want the plugin developer to provide some hook for it.
HTTP status code received for the reference page is 404 and not 200. That will be considered bad for SEO as it is giving 404 response for page which has content to display. You can check the console also too see the status code
