Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

#4892 closed defect (bug) (fixed)

bp_activity_latest_update_content filter is present but not utilised when saving activity content in usermeta

Reported by: saurabhshukla Owned by: saurabhshukla
Priority: low Milestone: 1.8
Component: Activity Version: 1.6
Severity: minor Keywords: has-patch early commit
Cc:

Description

'bp_activity_latest_update_content' filters content. But after filtering, the original unfiltered content is added in the database. Just adding it appropriately will let developers filter it for other types of content.

Attachments (3)

bp-activity-functions.diff (721 bytes ) - added by saurabhshukla 13 years ago.
Fixes latest update filtering
4892.02.patch (1.4 KB ) - added by DJPaul 13 years ago.
bp-activity-functions.2.diff (834 bytes ) - added by saurabhshukla 13 years ago.

Download all attachments as: .zip

Change History (12)

@saurabhshukla
13 years ago

Fixes latest update filtering

#1 @saurabhshukla
13 years ago

  • Owner set to saurabhshukla
  • Status newaccepted

@DJPaul
13 years ago

#2 @DJPaul
13 years ago

  • Keywords reporter-feedback added

Good spot. All the filters for content here look a bit funky; 4892.02.patch makes it better, but it does move bp_activity_latest_update_content before bp_activity_new_update_content

#3 @DJPaul
13 years ago

  • Component CoreActivity
  • Severity majornormal

#4 @johnjamesjacoby
13 years ago

  • Keywords commit added

Patch here looks good to me. Feel free to 1.7 it if you'd like.

#5 @saurabhshukla
13 years ago

I feel, the two should be kept separate. The latest update might need different handling than the actual activity that shows up in the activity stream.

So, the array that goes into bp_activity_add can remain as it is. That is handled separately and nicely. Let this be distinct.

#6 @saurabhshukla
13 years ago

  • Keywords reporter-feedback removed

#7 @johnjamesjacoby
13 years ago

  • Keywords early added
  • Milestone Awaiting Review1.8
  • Priority normallow
  • Severity normalminor
  • Version 1.71.6

Not a regression, so moving to 1.8. This function is weird in general, so let's address it early in 1.8.

#8 @boonebgorges
13 years ago

I agree with saurabhshukla that there's no real reason to collapse these two filters into one. I did add the modified $activity_content to the bp_activity_latest_update_content filter, so that you can use the previously filtered content if you want.

#9 @boonebgorges
13 years ago

  • Resolutionfixed
  • Status acceptedclosed

In 7115:

Use filtered content when storing latest update in usermeta

Previously, the content was run through the
bp_activity_latest_update_content filter, but was then unused.

Fixes #4892

Props saurabhshukla

Note: See TracTickets for help on using tickets.