Hi, I am using YouTube parser to import my videos from youtube and am trying to use your filed so I can automatically embed them but something weird is happening. The Video Embed field doesn't show as an option in the parsers Target field drop down. I've tried updating the DB and clearing the cache several times, along with a lot of other refresh attemps to no avail. Thanks for you hard work!

CommentFileSizeAuthor
#6 video_embed_field-add-feeds-mapper-functionality-1306324-6.patch2.09 KBptrl
#3 video_embed_field-add-feeds-mapper-functionality-1306324-3.patch2.04 KBpatrickfgoddard

Comments

jec006’s picture

jec006 commented

Hi, I'll look into this - its probably because the parser can only use certain types of fields. You may be able to just use feeds standalone.

I don't have a quick fix and this is gonna take a bit to set up and figure out. Hopefully I can get to it next weekend

patrickfgoddard’s picture

patrickfgoddard commented

Would like to second this request. And can help out any way I can. If there hasn't been any work on this yet, I can start digging into how the feeds mapping works.

patrickfgoddard’s picture

patrickfgoddard commented
StatusFileSize
new video_embed_field-add-feeds-mapper-functionality-1306324-3.patch2.04 KB

Here is my first pass at adding mapping functionality. This is my first time submitting a git-created patch and I'm just figuring out the process, so there is a good chance I got it wrong. There is also a good chance the functionality is incomplete, but hopefully this gets the ball rolling.

Feel free to provide feedback and I can improve.

The mapper only deals with the video_url part of the field_youtube_video array, as that seems like the only part that made sense to map. Should I add description, too? Guess it couldn't hurt, but would most people map any description content to the body field of the node?

In any case, here it is...

patrickfgoddard’s picture

patrickfgoddard commented
Status: Active » Needs review

Forgot to change status.

TimelessDomain’s picture

TimelessDomain commented
Status: Needs review » Needs work

It works! Thanks. At least it does for normal rss feed parsing. The youtube parser itself is having trouble telling which rss feeds users should use #1095362: Documentation how to use standard feeds?

I did change
'description' => t('The @label field of the node.', array('@label' => instance['label'])),
to
'description' => t('The @label field of the node.'),

since there was an error that was stopping it from working. i did not try the description, only the video url. which works great.

ptrl’s picture

ptrl commented
Version: 7.x-1.0-alpha4 » 7.x-2.x-dev
Component: Miscellaneous » Code
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new video_embed_field-add-feeds-mapper-functionality-1306324-6.patch2.09 KB

This patch works awesome with Feeds imports! I had some minor issues applying it with some whitespace so I have clean up the patch file, no new code. This really should be included in Video Embed Field.

jec006’s picture

jec006 commented
Status: Reviewed & tested by the community » Fixed

Awesome - thank you guys.

Committed here: http://drupal.org/commitlog/commit/25834/2cdfb6d2960479ebd550b5a658b822e...

Status: Fixed » Closed (fixed)

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

kappaluppa’s picture

kappaluppa commented
Category: feature » bug
Status: Closed (fixed) » Active

I am using
video embed field 7.x-2.x-dev
feeds 7.x-2.0-alpha8

The video embed field is still does not show up as an option. I've created additional 'test fields' to see if they show up. Other types of fields, i.e., text, show up, but not the video embed fields.

Are there any other requirements?

Thanks!
K

plopesc’s picture

plopesc
he/him
Primary language Spanish
commented
Issue summary: View changes
Status: Active » Closed (fixed)

It should be fixed now once 7.x-2.0-beta6 has been released

donbon’s picture

donbon commented

Should the front project page make mention of the fact that the module works with feeds?