WordPress.org

Make WordPress Core

Opened 5 years ago

Last modified 16 months ago

#8994 new feature request

Incorporate MediaRSS Plugin into core

Reported by: technosailor Owned by: technosailor
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Feeds Keywords:
Focuses: Cc:

Description

Per conversation on the hackers list, this ticket is a working area for incorporation of the MediaRSS plugin (http://wordpress.org/extend/plugins/mrss) into core for WP 2.8.

Attachments (2)

8994_mrss.diff (6.3 KB) - added by hailin 5 years ago.
mrss feed
8994-2.diff (4.5 KB) - added by technosailor 4 years ago.
Use post_thumbnail features

Download all attachments as: .zip

Change History (13)

comment:1 technosailor5 years ago

  • Type changed from defect (bug) to task (blessed)

comment:2 ryan5 years ago

Is anyone in particular working on this?

comment:3 hailin5 years ago

I am interested in this, unless Andy wants to take over (since he wrote the mrss plugin), I will work on this.

comment:4 ryan5 years ago

  • Component changed from General to Feeds

hailin5 years ago

mrss feed

comment:5 hailin5 years ago

The implementation uses media_rss filter to add media rss feeds to rss2 feed.
It improves the current mrss plugin and uses filter instead of adding action to rss2_item.

In case the installation already has the current media rss plugin installed,
it needs to comment out add_action('rss2_ns', 'mrss_ns'); in mrss_init() in the plugin because that tries to add xmlns:media="http://search.yahoo.com/mrss/" to the name space, and will confuse feed reader if it's added twice.

comment:6 ryan5 years ago

  • Milestone changed from 2.8 to Future Release

That preg_match_all on post_content will likely bust backtrack limits.

comment:7 technosailor4 years ago

We could refactor this patch for 3.1 to use the_post_image() for mrss instead of preg_match_all. Thoughts?

technosailor4 years ago

Use post_thumbnail features

comment:8 technosailor4 years ago

Refreshed patch to use post thumbnail features. However, not tested. Would prefer to have blessing before going too much deeper.

comment:9 nacin4 years ago

  • Type changed from task (blessed) to feature request

comment:11 prionkor16 months ago

I was wondering if we could focus on this..

Note: See TracTickets for help on using tickets.