Closed (fixed)
Project:
Brightcove Video Connect
Version:
7.x-3.2
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Jul 2011 at 16:04 UTC
Updated:
10 Oct 2012 at 20:41 UTC
Jump to comment: Most recent file
Comments
Comment #1
tamasd commentedThe integration is more or less ready.
Comment #2
e2thex commentedHope that does not mean I am doing extra work.
I started with the media_youtube and uses its structure to add bright cove as an options.
currently one can load a brightcove video by using the media_interent module upload window and typing in
brightcove:VIDEOID
also on the display configuration for the different video displays one can add a player id and key and have that player used with that display mode.
Still need to do images, and in general it is a little rough.
Comment #3
tamasd commentedTry it now, it is in git. What we already have:
- same funcionality as the youtube plugin (in the URL field it parses the brightcove embed code and the shortened url)
- additional tab for the brightcove library
- almost complete media gallery support
Comment #4
e2thex commentedOk so it looks like I total rewrote what you had already done, but they are very similar I guess that is good.
The only thing I have that you do not is, allowing a player id and key to be a parameter of the file display
I will post a patch tonight to move that over on top of your stuff.
Comment #5
e2thex commentedNot this is now against 7.x-2.x but that is not an option in version
Ok this patch adds fields to the video formatter to add a player id and key, this way different players can be used for different view modes
it also adds fields to to the image preview settings to set an image cache, but you did the preview image a little different them me so those settings are not doing anything yet.
Comment #6
e2thex commentedThe make patch had a problem
Comment #7
e2thex commentedOk so the formatter was not working, and I also add the option to change the height and width of the video
Comment #8
e2thex commentedI few edits add a check for variable values
THIS PATCH ARE FOR A DIFFERENT MODULE
PLEASE IGNORE
Comment #9
e2thex commentedComment #10
e2thex commentedPrevious patch had some extra fluff in it
Comment #11
temaruk commentedThank you for the patch, we merged it after some minor adjustments and added some more settings in 7.x-2.0-alpha4! Check it out!
Comment #12
e2thex commentedThank you using alpha4 and seems to work well
Comment #13
manuel garcia commentedI'll just use this issue instead of opnening a new one, please let me know if I should do otherwise.
Currently the player is not displayed when using a media field. I cannot see the BC formatters in the display settings for the field. This basically renders media integration broken in practice.
Comment #14
tamasd commentedYou can set the formatter in admin/config/media/file-types/manage/video/file-display.
Comment #15
damienmckennaThis works as advertised, it just needs some setup per #14.