[buddypress-trac] [BuddyPress Trac] #6177: Improve representation of blog posts in the Activity Stream

buddypress-trac noreply at wordpress.org
Sun Feb 1 14:12:16 UTC 2015


#6177: Improve representation of blog posts in the Activity Stream
-------------------------+------------------------------
 Reporter:  DJPaul       |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Activity     |     Version:
 Severity:  normal       |  Resolution:
 Keywords:               |
-------------------------+------------------------------

Comment (by DJPaul):

 I've been working on this for a couple of months, and have the following
 changes which, along with the original suggestion, need review. I've been
 building these over in Github; please review the overall branch changes,
 rather than on a per-commit basis.

 The "media-extractor" branch adds a new `BP_Media_Extractor` class which
 parses content and extracts info about media stored within.
  * https://github.com/paulgibbs/BuddyPress/compare/media-
 extractor#files_bucket
  * The change in `bp-activity-functions.php` is covered by #6159
  * The approach was inspired by a similar feature inside the Jetpack
 plugin, but the implementations are much different now (my version is more
 straightforward and just better).

 The "media-extractor-activity-stream" branch changes the Activity Stream
 to use the media extractor class to generate the richer Activity items.
  * https://github.com/paulgibbs/BuddyPress/compare/media-extractor
 ...media-extractor-activity-stream#files_bucket
  * The change in `bp-activity-filters.php` is a temporary solution;
 activity summaries that contained an audio shortcode produced too much
 HTML, and some part of this filter (and probably `balance_tags`) was
 removing the audio player HTML. I need to investigate further.

 I could go into more detail about some of the architectural decisions but
 I hope getting your fresh eyes and interpretation will help generate
 meaningful feedback, and suggestions.

 If anyone wants to contribute to this "patch", please submit pull requests
 to my Github branch (let me know if you do this because I tend to not
 notice I have incoming pull requests!). If this proposal is accepted and
 once all the code has been reviewed/iterated on, I'll then create a patch
 file, etc.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6177#comment:1>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list