[buddypress-trac] [BuddyPress Trac] #6177: Improve representation of blog posts in the Activity Stream
buddypress-trac
noreply at wordpress.org
Wed Feb 4 10:40:28 UTC 2015
#6177: Improve representation of blog posts in the Activity Stream
----------------------------------+------------------
Reporter: DJPaul | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 2.3
Component: Component - Activity | Version:
Severity: normal | Resolution:
Keywords: 2nd-opinion |
----------------------------------+------------------
Comment (by DJPaul):
Thanks for taking a quick look, Boone. I really appreciate it.
I had done some testing using xhprof to performance test the unit tests I
wrote. That revealed some useful areas of improvements, such as
[https://github.com/paulgibbs/BuddyPress/commit/e3eab90dcd127a42eb5bc42eef5f6a802caf7b4c
avoiding regex calls where possible]. In terms of how
`bp_activity_create_summary` is currently, it's only used for Posts on the
transition_post_status hook, which means it isn't being run at real-time.
Please would someone remind me what the change was to activity objects, I
think in one of the last couple of versions? One of the fields were
hardcoded (so they couldn't be translated? or something) and then we
changed it so they were dynamically generated, which I assume is what
Boone is getting at here.
I will look into using bp_activity_create_summary on the activity objects
with type = `activity_update`. I can't think of any other types of
activity object that have a content item set that may be worth doing this
one (maybe forums, but that's bbPress)
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6177#comment:5>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list