Hi, I need to use JW Player presets on Video fields (provided by the Video module - http://drupal.org/project/video). Currently JW Player only supports the core File field. I have created a patch that allows JW Player to format the Video field as well.

The patch changes the function jw_player_field_formatter_info() (line 57, jw_player.module):

/**
 * Implements hook_field_formatter_info().
 */
function jw_player_field_formatter_info() {
  $formatters = array(
    'jw_player' => array(
      'label' => t('JW player'),
      'field types' => array('file'),
      'settings' => array(
        'jwplayer_preset' => '',
      ),
    ),
  );
  return $formatters;
}

to this:

/**
 * Implements hook_field_formatter_info().
 */
function jw_player_field_formatter_info() {
  $formatters = array(
    'jw_player' => array(
      'label' => t('JW player'),
      'field types' => array(
        'file',
        'video',
      ),
      'settings' => array(
        'jwplayer_preset' => '',
      ),
    ),
  );
  return $formatters;
}

I will attach the patch shortly.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

theamoeba’s picture

Here is the patch :)

cangeceiro’s picture

Status: Active » Needs review
FileSize
1.03 KB

the above patch doesn't support transcoded videos. attached is a patch that does

monaw’s picture

Has these patches made it into the 2.x-dev release dated April 2013?

garbo’s picture

No, these patches haven't made it into the 2.x-dev release of april 23th.

  • mongolito404 committed 3dd995c on 7.x-2.x
    Issue #1807710: Video field formatter - support for transcoded videos in...
pbuyle’s picture

Version: 7.x-1.0-alpha1 » 7.x-2.x-dev
Issue summary: View changes

Commit c820aab and 3dd995c should provide support for Video field with transcoded video. Could any Video user confirm?

deanflory’s picture

Is this still an open issue or did the commits take care of the problem?

pbuyle’s picture

The commit should have taken care of this, but this has not been tested.

ron_s’s picture

Status: Needs review » Closed (fixed)

This is in 7.x-2.x. If anyone has issues with it, please post a new ticket against version 2.x. Thanks.