Just like #1823078: Create video/vimeo mimetype and #1812976: Register video/youtube mimetype and provide an upgrade path this module needs to register a video/bliptv mimetype and associate it with video files. This is required to work with the new media module.

It should look something like this:

/**
 * Implements hook_file_mimetype_mapping_alter().
 */
function media_bliptv_file_mimetype_mapping_alter(&$mapping) {
  $mapping['mimetypes'][] = 'video/bliptv';
}

/*
 * Implements hook_file_default_types_alter().
 *
 * Adds the video/bliptv fake mimetype to video files.
 */
function media_bliptv_file_default_types_alter(&$types) {
  $types['video']->mimetypes[] = 'video/bliptv';
}

But it should also probably be automatically enabled when blip.tv is checked off as an allowed stream type. (right now you have to put video/bliptv in the mimetype textarea)

Comments

Rob_Feature’s picture

Priority: Major » Critical
Status: Active » Needs review
StatusFileSize
new758 bytes

Here's a starter patch, please review and tweak as needed....

Rob C’s picture

Status: Needs review » Reviewed & tested by the community

I created the exact same patch before i started looking :/ Anyway :) Patch works fine, implemented and running again on my media test site.

oystercrackher’s picture

The patch in #1 did not work for me; it gave the following errors:

Warning: media_bliptv.module has type 100644, expected 100755
error: while searching for:
}
/**
*Implements hook_ctools_plugin_api().
*/
function media_bliptv_ctools_plugin_api($owner, $api) {
error: patch failed: media_bliptv.module:108
error: media_bliptv.module: patch does not apply

Thanks

Rob_Feature’s picture

Status: Reviewed & tested by the community » Needs work

probably needs to be updated to work with the latest version of ctools?