Closed (fixed)
Project:
Embedded Media Field
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
19 Jul 2009 at 01:05 UTC
Updated:
17 Feb 2010 at 18:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
aaron commentedGood idea. Should be easy enough; I'll try to get it next week.
Comment #2
robloachThis is probably very backwards, but it's a start. It exposes a Display Setting called "Link to Video", which returns the URL of the video. Would be nice to have a "Link to Video" which is an actual link though.
Comment #3
mvcThe patch in #2 worked perfectly for me. I also needed an equivalent feature for embedded images, so I followed this example and wrote a similar patch.
I agree that ideally this should return an actual link, but for now this is all I need (for a CSV feed display). Posting in the hopes that it's useful to someone.
Comment #4
mvcFor reference, here is the patch to the node import module which handles embedded media fields: #565424: Support for embedded media fields
Comment #5
jeffschulerThanks Rob Loach and mvc.
I actually think you've gone about it the right way, so I'm changing to CNR.
Having the plain URL allows us to use it when creating links around subsequent Views fields.
Having the already-created <a> tag, on the other hand, would prevent us from using other fields as the link text.
Comment #6
jeffschulerAnd now I see that
theme_emvideo_formatter_video_linkandtheme_emvideo_formatter_video_urlare already in there...!Comment #7
mvc@jeffschuler the emimage patch in #3 isn't committed yet.
Comment #8
aaron commented@mvc & @Rob Loach: thanks! got that patch committed... sorry it slipped off my radar :P
Comment #9
aaron commentedto the 2.x-dev branch anyway...