#7146 closed defect (bug) (fixed)
Activity permalink access doesn't work with legacy activity query
Reported by: | r-a-y | Owned by: | r-a-y |
---|---|---|---|
Milestone: | 2.6.1 | Priority: | normal |
Severity: | normal | Version: | 2.6.0 |
Component: | Activity | Keywords: | has-patch |
Cc: |
Description (last modified by )
In #7113, we fixed a bug where anybody's activity item could be displayed under a member profile.
However, due to the strict activity user ID check for the fix, this broke activity permalink access on some installs:
https://buddypress.trac.wordpress.org/browser/tags/2.6.0/src/bp-activity/bp-activity-screens.php?marks=253#L252
$activity->user_id
is a string when the older 'bp_use_legacy_activity_query'
filter is set to true via a plugin.
#6977 marks ongoing work to cast all our numeric strings as integers. r10853 missed an integer casting instance in the older, legacy activity query, which caused the problem.
Attached patch fixes this up.
A user on the forums confirms the patch works:
https://buddypress.org/support/topic/post-update-unable-to-see-the-buddypress-activity/#post-255374
Going to commit.