Upgrade the cod_session_video Feature to D7

Question: are we sticking with upgrading emfield here or do we want to switch to video_embed_field (drupal.org/project/video_embed_field)?

Looks like sticking with emfield would be the way to go, since it allows for other media types besides video. D7 emfield just has one embed type vs. 8 types in D6 version, details apparently handled by Media module required by D7 emfield.

After seeing this post I started a thread about Using the media module to create one content type called "session resource" that would allow multiple types of file formats.

#1603718: Replace Video and Slides with Media Module for Session Resources - Videos, Powerpoints (slideshare) & Photos

OK, I'm a bit of a noob with the D7 Fields API and don't know enough to figure out what I'm not doing right. I've modified the field definition in cod_session_video.features.content.inc using cod_session.features.field.inc as a model, but not having any luck yet getting my new field to show up in existing fields when I go to add it in admin/structure/types/manage/session/fields. Here's the patch, in hope that someone can show me the error of my ways.

Step 1, replace 'content' with 'field' in the filename and the default_fields hook.

New patch, per comment #5.

This patch gets rid of some comments and debug statements.

This is now adding a video field to the Session content type.

Still have some work to do getting the Embedded Media Field to work (see http://drupal.org/node/1414606).

Media and remote stream wrapper are stable and allows u to use multiple formats.

The difficulty seems to be with the Embedded Media Field, not the Media modules.

Update: they are now playing well together in cod_session_video. Working on a few final config settings so everything will just work out of the box.

New question:

Per the Embedded Media Field page 1.x version of emfield was packaged with the emvideo module, which contained support for numerous providers. In 2.x, Media provider modules need to be installed separately. It looks like YouTube and Vimeo are the only providers included in emvideo that have corresponding Media provider modules.

Should I include YouTube and Vimeo as part of this upgrade, or should we document this requirement and leave it up to COD users to install the modules they want to use?

PS--I've currently got cod_session_video set up to use the Media module widget instead of the emfield widget. Where the emfield widget expects embed code, the Media widget lets users uploaded video, embed 3rd-party video or choose from videos in the Media library, in accordance with comment #3.

I've gone with the Embedded Media Field option over Media and Video Embed Field because that's the one I've had the best luck getting to work.

Some notes:
1. Embedded Media Field requires provider modules. Of the types supported in the 6.x version of EMF, only YouTube and Vimeo have provider modules for the 7.x version at this time.
2. In order for the video to display, the provider options (e.g., YouTube Video) have to be checked on admin/structure/file-types/manage/video/file-display. No luck yet getting this to happen in the .inc file.
3. Patch still includes commented-out code for Media and Video Embed Field, just for reference.

Not claiming to be completely done, but some feedback would be good. So, changing status on this one.

Thanks,
SB

patch does not apply clean:

cod_session_video-upgrade-feature-1150348-11.patch:53: trailing whitespace.
        ),        
warning: 1 line adds whitespace errors.

Re-rolled and attached

Feature is also asking for content_permissions and emvideo, what module(s) are those contained in?

status update

I created a patch to add the video functionality with the Media Module. #1603718: Replace Video and Slides with Media Module for Session Resources - Videos, Powerpoints (slideshare) & Photos

These patches were not providing the functionality from d6 cod.

This feature should enable an embedded media field and field permissions.

I rolled a patch for this against current dev.

Added vimeo and youtube sources as dependencies
switched input widget from media button to a field
added help text
set default display as large

Rolled a new patch, does not require #17