Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
oweno will be creating a mapper for emaudio some time over the next week or so (right after he finishes the port of emaudio from 5.x to 6.x).
Comment | File | Size | Author |
---|---|---|---|
#10 | feedapi_mapper_emaudio.inc_.txt | 2.2 KB | lyricnz |
#3 | feedapi_mapper_emaudio.inc_.txt | 2.02 KB | Alex UA |
Comments
Comment #1
alex_b CreditAttribution: alex_b commentedIs this still planned? Fixing typo for better searchability.
Comment #2
Alex UA CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: Alex UA commentedComment #5
alex_b CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: webslinger commentedSame problem as #7, unfortunately.
Comment #9
alex_b CreditAttribution: alex_b commentedComment #10
lyricnz CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: alex_b commentedGreat. Need to review. We should write a test for this mapper.
Comment #14
sjf CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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.