Reviewed & tested by the community
Project:
Feed Element Mapper
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
20 Nov 2008 at 17:48 UTC
Updated:
18 Aug 2009 at 08:52 UTC
Jump to comment: Most recent file
Comments
Comment #1
alex_b commentedIs this still planned? Fixing typo for better searchability.
Comment #2
alex ua commentedSorry, kind of fell off of the radar with the file sprint and work on the iPhone app. It is definitely planned, and I'll have one of our devs work on it on Monday.
Comment #3
alex ua commentedOK- I guess this was a lot simpler than I expected. Attached is an emaudio mapper. HOWEVER, while this does "work" I think that it is a great example of the shortcomings of the current way that emaudio works (which may have to do more with the ways that the providers handle podcasts). ATM emaudio posts entire podcasts within each node, instead of an individual audio file, meaning that if you post two episodes from the same show you basically get the same media, with only a different starting point to mark the difference.
Anyway, this seems to work for now. I'm going to open an emfield issue to think through how to rework emaudio.
Comment #4
alex ua commentedComment #5
alex_b commentedNice work. This is looking good to me - users of emfield audio: can I get a review from somebody?
Comment #6
alex ua commentedI'll ping moonray and jaochim as they seem to be two of the only users of this (mostly experimental) sub-module.
Comment #7
sjf commentedwarning: Missing argument 3 for emaudio_feedapi_mapper() in ***/sites/all/modules/feedapi_mapper/mappers/feedapi_mapper_emaudio.inc on line 14.I tried replacing $field_name with $field_name = '' (cribbed from the emvideo mapper) and this cleared the error but emaudio was still not available as one of the mapping elements.
Comment #8
webslinger commentedSame problem as #7, unfortunately.
Comment #9
alex_b commentedComment #10
lyricnz commentedThe format of the mapper function has changed. Here's a new version, which is literally just feedapi_mapper_emvideo.inc with "audio" substituted for "video" wherever it occurs. Works for me.
Comment #11
sjf commentedWould you believe I tried that before my initial post but couldn't get it to work. Your file works perfectly - many thanks.
Comment #12
tandersonmd commentedUsing this I was able to get the emaudio option to show up in mapper. However, the url is not saved in the cck field on any of the created nodes. I checked with emvideo and this works perfectly. Any help would be appreciated.
Comment #13
alex_b commentedGreat. Need to review. We should write a test for this mapper.
Comment #14
sjf commentedThe mapper no longer works with 6.x-2.x-dev. I've tried taking the latest feedapi_mapper_emvideo.inc that ships with 2.x and substituting video for audio but all I got was a white screen. Given that I couldn't manage this simple task before, perhaps someone else could try.
Comment #15
alex_b commented#15: there is a slight api change. compare feedapi_mapper.api.php between 1.x and 2.x. Expect one more change before 2.x goes beta #541814: Breaks when enabling multiple processors
Comment #16
sjf commentedI've taken the latest feedapi_mapper_emvideo.inc that ships with 2.x, substituted video for audio, and this now works again. Many thanks.