The Brightcove module now has a 6.x branch that adds support for multiple accounts, players, etc., as well as turning the Brightcove Clients, Videos and Players into separate entity types. This module needs to be updated to work with the new API provided by Brightcove 7.x-6.x.

CommentFileSizeAuthor
#19 interdiff-18-19.diff931 bytessleepingmonk
#19 brightcove_media_Adding-support-to-brightcove-v6-module-2578405-19-d7.patch40.21 KBsleepingmonk
#18 brightcove_media_Adding-support-to-brightcove-v6-module-2578405-18-d7.patch41.29 KBartofeclipse
#16 Adding-support-to-brightcove-v6-module.patch13.93 KBartofeclipse
#15 new_brightcove_api_compatibility-2578405-15.patch10.34 KBsleepingmonk
#14 interdiff-9-14.diff4.55 KBsleepingmonk
#14 new_brightcove_api_compatibility-2578405-14.patch0 bytessleepingmonk
#9 new_brightcove_api_compatibility-2578405-9.patch10.16 KBsleepingmonk
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

q0rban created an issue. See original summary.

fabianderijk’s picture

fabianderijk
him/his
Primary language Dutch
Location Alphen aan den Rijn
CreditAttribution: fabianderijk at Finalist commented

I've got the same issue. Is there any timeline available when this version will be ready?

q0rban’s picture

q0rban CreditAttribution: q0rban at Lullabot commented

@fabianderijk, no there is no timeline. The Brightcove API from 5.x to 6.x is drastically different, so the upgrade is non-trivial.

Jayaher’s picture

Jayaher CreditAttribution: Jayaher commented

We have already used drupal brightcove 7.x-5.x version module. Now we need to upgrade the latest brightcove 7.x-6.x module version.

How to upgrade the latest brightcove version into existing one?

Please advice.

sleepingmonk’s picture

sleepingmonk
Primary language English
Location Vancouver Island 🇨🇦
CreditAttribution: sleepingmonk commented

Hey q0rban,

The upgrade is definitely 'non-trivial'. Just wondering if any work has been started on it that's not reflected in the commit log. Or if the maintainers have the intention starting an update in the near future.

Any info appreciated. Thanks for a very useful module!

q0rban’s picture

q0rban CreditAttribution: q0rban at Lullabot commented

Hi sleepingmonk!

There are currently no plans that I am aware of to upgrade this module for compatibility with the 7.x-6.x branch of Brightcove. Are there features in 6.x that you need? The 6.x branch takes a radical departure from normal handling of file-like entities such as videos by creating new entities for videos, instead of using the File entity. This may prove to make integration with Media module nigh on impossible, but even if it is feasible, the upgrade path alone for this change is enough to send the bravest of developers running. :)

sleepingmonk’s picture

sleepingmonk
Primary language English
Location Vancouver Island 🇨🇦
CreditAttribution: sleepingmonk commented

Thanks for the insight q0rban! Looks like we'll have to work out how to migrate existing video content (as file entities) to the new entity type(s).

sleepingmonk’s picture

sleepingmonk
Primary language English
Location Vancouver Island 🇨🇦
CreditAttribution: sleepingmonk commented

Hey q0rban,

Just wanted to let you know I'm starting work to make this compatible with Brightcove 7.x-6.x.

sleepingmonk’s picture

sleepingmonk
Primary language English
Location Vancouver Island 🇨🇦
CreditAttribution: sleepingmonk commented
FileSize
new_brightcove_api_compatibility-2578405-9.patch10.16 KB

Attaching a first pass against 7.x-2.x. Probably needs to be a 7.x-3.x branch?

This works with Brightcove 7.x-6.x module.

Only works with video embed through the wysiwyg, so far.

q0rban’s picture

q0rban CreditAttribution: q0rban at Lullabot commented

Thanks for working on this, @sleepingmonk! With something as big as this, I wonder if we should copy the repo to GitHub and create a Pull Request. Thoughts?

sleepingmonk’s picture

sleepingmonk
Primary language English
Location Vancouver Island 🇨🇦
CreditAttribution: sleepingmonk commented

I'm open to that. I'm sleepingmonk on github too. Let me know if you put the repo up and I can make a PR.

Will you make a 7.x-3.x branch for compatibility with Brightcove 7.x-6.x?

q0rban’s picture

q0rban CreditAttribution: q0rban at Lullabot commented

@sleepingmonk:

https://github.com/q0rban/brightcove_media

Let me know if that works for you! :)

sleepingmonk’s picture

sleepingmonk
Primary language English
Location Vancouver Island 🇨🇦
CreditAttribution: sleepingmonk commented

Thanks @q0rban I'll get a PR up there soon!

sleepingmonk’s picture

sleepingmonk
Primary language English
Location Vancouver Island 🇨🇦
CreditAttribution: sleepingmonk commented
FileSize
new_brightcove_api_compatibility-2578405-14.patch0 bytes
interdiff-9-14.diff4.55 KB

A few fixes from the previous patch.

sleepingmonk’s picture

sleepingmonk
Primary language English
Location Vancouver Island 🇨🇦
CreditAttribution: sleepingmonk commented
FileSize
new_brightcove_api_compatibility-2578405-15.patch10.34 KB

For some reason the patch in #14 lost data...

artofeclipse’s picture

artofeclipse CreditAttribution: artofeclipse commented
FileSize
Adding-support-to-brightcove-v6-module.patch13.93 KB

Hello,

I have done some more extra work on @sleepingmonk patch, this patch apply to 7.x-3.x* and 7.x-2.x* but I think we should add new dev release.

Attached is my changes to the latest by @sleepingmonk.

Thanks.

q0rban’s picture

q0rban CreditAttribution: q0rban at Lullabot commented

Thanks @artofeclipse! Can we please work on GitHub due to the size of this work? https://github.com/q0rban/brightcove_media

I think @sleepingmonk's work is up to date here? https://github.com/q0rban/brightcove_media/pull/1

artofeclipse’s picture

artofeclipse CreditAttribution: artofeclipse commented
FileSize
brightcove_media_Adding-support-to-brightcove-v6-module-2578405-18-d7.patch41.29 KB

Adding latest patch version to make it possible to include in build script.

sleepingmonk’s picture

sleepingmonk
Primary language English
Location Vancouver Island 🇨🇦
CreditAttribution: sleepingmonk at Phase2 commented
FileSize
brightcove_media_Adding-support-to-brightcove-v6-module-2578405-19-d7.patch40.21 KB
interdiff-18-19.diff931 bytes

Re-roll for latest dev.

Add a check for empty $video in brightcove_media_file_formatter_image_view() which causes a PHP Fatal error if a video is not loaded (i.e. removed from brightcove).

vijaycs85’s picture

vijaycs85
Location London, UK
CreditAttribution: vijaycs85 as a volunteer and at TATA Consultancy Services for Pfizer, Inc. commented

More update at https://github.com/q0rban/brightcove_media/pull/2

tamasd’s picture

tamasd CreditAttribution: tamasd at Pronovix commented
Status: Active » Fixed

Thanks for all the work, I have implemented the new integration as a part of the brightcove module. It will be released as the part of the 7.x-6.5 release soon.

vijaycs85’s picture

vijaycs85
Location London, UK
CreditAttribution: vijaycs85 as a volunteer and at TATA Consultancy Services for Pfizer, Inc. commented

awesome! thanks @tamasd.

q0rban’s picture

q0rban CreditAttribution: q0rban at Lullabot commented
Status: Fixed » Active

@tamasd, so are you saying that Brightcove Media module will not be needed for Brightcove module 7.x-6.5? If so—awesome! I'm curious, though, how will you be handling the upgrade path for this module to that version (since the Entity API content model is completely different)? I think that needs to be answered in this module before this can be "fixed" here.

tamasd’s picture

tamasd CreditAttribution: tamasd at Pronovix commented
Category: Task » Support request

The brightcove media module has been moved to the brightcove project, it is still a separate submodule.

I didn't do anything with the entity content model. This is a straight port, if you use the brightcove field and the media integration at the same time, then you will store references to a given video in two ways. There is no easy way to migrate the brightcove field to media.

The upgrade path from brightcove_media-1.x -> brightcove_media-2.x -> brightcove_media inside the brightcove module should work, it is being tested at the moment.

jan.mashat’s picture

jan.mashat CreditAttribution: jan.mashat at Pronovix for BrightCove commented
Status: Active » Closed (duplicate)

I would like to consolidate any discussions around the new media support in 7.x-6.x-dev, so please share your test results and any issues here: https://www.drupal.org/node/2875335