Aron,

when you roll out Drupal 6, could you take care that enclosure support is being picked up - I just committed ekes' patch here: http://drupal.org/node/217750#comment-750794

Thank you.

CommentFileSizeAuthor
#4 enclosure_location_support.patch1.54 KBgeodaniel

Comments

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Patch (to be ported) » Fixed
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

geodaniel’s picture

geodaniel commented
Version: 5.x-1.x-dev » 6.x-1.x-dev
Status: Closed (fixed) » Patch (to be ported)

I can't see the support for this in the 6.x module. Was it added directly into the SimplePie parser as in 5.x?

geodaniel’s picture

geodaniel commented
Title: Add enclosure support to FeedAPI Drupal 6 » Add enclosure (and location) support to FeedAPI Drupal 6
StatusFileSize
new enclosure_location_support.patch1.54 KB

The attached patch adds enclosure support to SimplePie parser for D6 (based directly off ekes' patch that Alex mentioned), and also adds location support (with the help of the mapper in #265196: Support for the Location module).

geodaniel’s picture

geodaniel commented
Status: Patch (to be ported) » Needs review
alex_b’s picture

alex_b commented
Category: bug » feature
Priority: Normal » Critical

That's a crucial feature. Not least because of the dependency of the mapper.

Arto’s picture

Arto commented

Dan's patch works for me and plays nice with #265196: Support for the Location module.

summit’s picture

summit commented

Subscribing, please port also to Drupal 5, so I can test location mapping on Drupal 5. thanks for considering!
greetings, Martijn

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented

Thanks for the patch. Committed. The missing parts are backported to 5.x as well.

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Needs review » Fixed
geodaniel’s picture

geodaniel commented

Thanks Aron.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.