Closed (fixed)
Project:
Feed Element Mapper
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 Sep 2008 at 20:47 UTC
Updated:
23 Oct 2008 at 15:52 UTC
Hi Alex
Thanks for the D6 version!
I got the video_cck mapper working by changing 'video_cck' to 'emvideo'. I believe that emvideo is the correct name of emfield's video module now (video_cck is deprecated.) The mapper filename should probably get changed as well.
| Comment | File | Size | Author |
|---|---|---|---|
| feedapi-mapper-emvideo.patch | 1.53 KB | jody lynn |
Comments
Comment #1
LeonidShamis commentedI have successfully applied the patch - it works. Thank you very much.
Result can be seen here: http://dev.imcrunch.com/taxonomy/term/5
Comment #2
robertdouglass commentedTrying with latest (Sept. 19th) DRUPAL-6--1 of emfield and feedapi_mapper and can't get the field to map. The mapping form comes up ok, but the field never gets populated.
Comment #3
robertdouglass commentedTake it back. Works nicely now that I allowed for the appropriate providers at q=admin/content/emfield
Comment #4
robertdouglass commentedYouTube's description doesn't map to Body.
Comment #5
thomas23@drupal.org commentedHi.
Thanks for porting to D6 and all that work! I'm new to emfield and mapper (trying out Drupaltherapy's Video on D6). When trying to map, though, only taxonomy shows up as available mappers.
Here's my question related to this issue: Does this have to do with the above reported or is that a new issue I should report separately? Or can you guys tell what I could have done wrong? Does it matter that I'm using Common syndication parser (CSP) and not Simple Pie? I read that
Cheers.
Comment #6
jody lynnthomas23,
My experience was that emfield mapping will not show up as an available mapper yet unless you apply my patch above.
Comment #7
thomas23@drupal.org commentedThanks, Lynn, for your explaination.
I patched
modules/feedapi_mapper/mappers/feedapi_mapper_video_cck.incusing Aptana Studio (~ecliplse) and now field_emvideo (emvideo module) shows up, too. And, BAM, I got kittens all over the place^ :D Meaning, it's working for me (please set status back if you need more review).Cheers.
Comment #8
alex_b commentedThanks for everybody's work here.
#4 - I take this is rather a feature request than a bug of this patch, right?
Shouldn't the mapper's file name (feedapi_mapper_video_cck.inc) be renamed to feedapi_mapper_emfield.inc now?
Comment #9
alex_b commentedIs this patch here related to this issue?
http://drupal.org/node/310883
Comment #10
thomas23@drupal.org commentedRe: #8
alex_b, that's exactly what I was thinking when applying the patch. It would be consistent.
Cheers.
Comment #11
jody lynnYes, please rename the mapper as well.
I don't think it's related to http://drupal.org/node/310883 as that looks like purely an issue in emfield HEAD.
Comment #12
alex ua commentedemfield HEAD is still being used for 5, which I know is confusing and we should address shortly. DRUPAL-6--1 is the development version.
Comment #13
toemaz commentedApplied to 1.x-dev, tested and it works.
Comment #14
infines commentedCan some one patch this for me? I'm not really sure how and I can't find the file if its already patched. This would be a great help. As, I'd like to get my site working properly. Thankyou.
EDIT:
nevermind. I just had to rename the file.
Comment #15
alex_b commentedI applied patch, renamed feedapi_mapper_video_cck.inc to feedapi_mapper_emfield.inc and committed.
Thank you everybody.
Comment #16
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.