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.

CommentFileSizeAuthor
feedapi-mapper-emvideo.patch1.53 KBjody lynn

Comments

LeonidShamis’s picture

I have successfully applied the patch - it works. Thank you very much.
Result can be seen here: http://dev.imcrunch.com/taxonomy/term/5

robertdouglass’s picture

Trying 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.

robertdouglass’s picture

Take it back. Works nicely now that I allowed for the appropriate providers at q=admin/content/emfield

robertdouglass’s picture

YouTube's description doesn't map to Body.

thomas23@drupal.org’s picture

Hi.

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

Common syndication parser does not parse enough into the object to be able to get many custom elements.

Cheers.

jody lynn’s picture

thomas23,
My experience was that emfield mapping will not show up as an available mapper yet unless you apply my patch above.

thomas23@drupal.org’s picture

Status: Needs review » Reviewed & tested by the community

Thanks, Lynn, for your explaination.

I patched modules/feedapi_mapper/mappers/feedapi_mapper_video_cck.inc using 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.

alex_b’s picture

Thanks 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?

alex_b’s picture

Is this patch here related to this issue?
http://drupal.org/node/310883

thomas23@drupal.org’s picture

Re: #8
alex_b, that's exactly what I was thinking when applying the patch. It would be consistent.

Cheers.

jody lynn’s picture

Yes, 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.

alex ua’s picture

emfield HEAD is still being used for 5, which I know is confusing and we should address shortly. DRUPAL-6--1 is the development version.

toemaz’s picture

Applied to 1.x-dev, tested and it works.

infines’s picture

Can 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.

alex_b’s picture

Version: 6.x-1.0-alpha1 » 6.x-1.x-dev
Status: Reviewed & tested by the community » Fixed

I applied patch, renamed feedapi_mapper_video_cck.inc to feedapi_mapper_emfield.inc and committed.

Thank you everybody.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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