Help with creating new filters [Added TED support code patch]

The problem with both those sites is that there is no ID in the URL that we can use to construct the codec. I don't really see how we can support these I'm afraid.

Try to find what really needs to be used for the embed codes, and try to create an URL pattern that your users can craft by hand, using information found on site. See the Slideshare codec for example.

I really would like those two supported too. In addition MIT World..... IF you find out let me know. THanks!

I think I can do it (I need now), but I don't know why the module don't catch my codec.

It this:

function MIMODULE_codec_info() {
  $codecs = array();
  // You can offer multiple video formats in one module.
  $codecs['TED'] = array(
    // Will be used some day in user information.
    'name' => t('TED'),

    // Special instructions for end users. Optional.
    'instructions' => t('Click in Embed and copy the link -Link to this talk- and paste here.'),

    // The callback that will output the right embed code.
    'callback' => 'MIMODULE_TED',

    // HTML5 callback, for returning something that's device agnostic.
    // @SEE video_filter_youtube_html5.
    // 'html5_callback' => 'MODULE_service_html5',

    // Regexp can be an array. $video['codec']['delta'] will be set to the key.
    'regexp' => '/ted\.com\/talks\/lang\/([a-zA-Z]+)\/([a-zA-Z0-9\-_]+)\.html',
    'sample_url' => 'http://www.ted.com/talks/lang/es/tim_berners_lee_on_the_next_web.html',

    // Ratio for resizing within user-given width and height (ratio = width / height)
    'ratio' => 425 / 355,
  );
  return $codecs;
}

Drupal return me the next:

<!-- VIDEO FILTER - INVALID CODEC IN: [video:http://www.ted.com/talks/lang/es/tim_berners_lee_on_the_next_web.html width:300 height:300] -->

Now it's run perfectly:

function customizations_mail($key,&$message,$params){
    switch ($key) {
        case 'mandatory_course_mail':
            $message['headers']['Content-Type'] = 'text/html; charset=UTF-8; format=flowed';
            $message['subject'] = $params['subject'];
            $message['body'][] = $params['body'];
            break;
    }
}


function customizations_codec_info() {
  $codecs = array();

  // You can offer multiple video formats in one module.
  $codecs['ted'] = array(
    
    // Will be used some day in user information.
    'name' => t('TED'),
    'instructions' => t('Click in Embed and copy the link -Link to this talk- and paste here.'),
    'callback' => 'customizations_TED',
    'html5_callback' => 'customizations_TED',
    'regexp' => '/ted\.com\/talks\/lang\/([a-zA-Z]+)\/([a-zA-Z0-9\-_]+)\.html/',
    'sample_url' => 'http://www.ted.com/talks/lang/es/tim_berners_lee_on_the_next_web.html',

    // Ratio for resizing within user-given width and height (ratio = width / height)
    'ratio' => 4 / 3,
  );

  return $codecs;
}

function customizations_TED($video) {
  $video['source'] = 'http://embed.ted.com/talks/'.$video['codec']['matches'][2].'.html';
  return video_filter_iframe($video);
}

I'm going to create a new patch for the module for this new provider. :) Only need time.

I added support for TED. Test it please.

I change the issue type because now have a code solution directly.

TED have changed his url (now it hasn't .html at the end of it). I change the code and I made a patch.

Hi folks,

I've created two modules which allow you to embed videos from these two providers with video_filter module:

https://www.drupal.org/project/video_filter_democracynow
https://www.drupal.org/project/video_filter_ted

Hope that helps someone.

Ok, But I think that is better create an integration with it :(

Hi estoyausente,

If the module maintainer(s) wish to integrate these with the parent module, they are welcome to. Seems to take a long time for this to happen, so in the meantime, my modules are useful/usable (and they solve the two issues posted as part of this issue, rather than just the TED issue).

One of the reasons that I didn't add them as patches was that I was using them as part of my project application review for getting full project creation rights.

Yes, @markconroy my comment isn't for your work (ty for do it). But it's a little bit sad that we have to create a new module for this "stupid" thing.

Regards!

Well like I say, the code is there now, and I have no problem with video_filter maintainers adding it to video_filter. Just means if they don't have time to do it, it's still usuable in the short term.

Thanks.

This has been added to D8 version of the module.

Committed, after tidying up the code a little.

Hi Damien,

Any chance of adding in my codec for Democracy Now (referenced in the original issue by the OP here)? The code can be found here:

https://www.drupal.org/project/video_filter_democracynow

Thanks.

I've moved the DemocracyNow codec into a new issue: #2682069: Add DemocracyNow support

Thanks Damien.