Opened 5 years ago
Closed 4 years ago
#3356 closed defect (bug) (duplicate)
_bbp_last_active_id not updating on new reply
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | high | |
| Severity: | normal | Version: | trunk |
| Component: | Component - Replies | Keywords: | has-patch |
| Cc: |
Description
I've found that _bbp_last_active_id is not being updated on new reply.
I think this is the wrong behaviour because on the repair tool it picks the last reply
I've been checking includes/replies/functions.php function: bbp_update_reply_walker
And it only updates the freshness for 1 ancestor (the post, not the forum or the category, within the forum)
The problem is solved in the patch, just a conditional that is wrongly tested (checking if reply is pending, instead of checking that is not pending)
This is the first time I contribute with SVN so I'm not sure if this is the right way
Attachments (1)
Change History (5)
#3
@
4 years ago
@mike80222 it's a little bit slow the bbpress review process, isn't it?
@johnjamesjacoby did not even take into consideration my patch, solving it in #3376 which is 20 posts further than this (and 79 away from #3297 where I also stated the patch)
I'm not sure which order reviewers follow to move forward in the bug solving process, but its surely time wasted for JJJ having to debug and solve this issue.
#4
@
4 years ago
- Milestone Awaiting Review deleted
- Resolution set to duplicate
- Status changed from new to closed
Duplicate of #3376.
Sorry, @SirLouen.
In this instance, I simply didn’t notice that issue was a duplicate of this one. You are correct that the first issue is the one that should have taken precedence. The way it went down was I saw the firehose notification in Slack for the latest one and happened to have a moment to work on it then. I didn’t think to check for dupes.
Thank you for the patch, and I apologize sincerely for missing it. 😞
Going to officially close this one as a dupe of that one. I know it’s backwards, but it’s the way it went down.
Thank you again, and sorry again. 🙏
patch #3356