On /admin/config/media/file-types/manage/video/file-display/media_original the Brightcove Video display asks for a Player ID and Player Key. However, in theme_brightcove_field_embed() on line 66 of brightcove_field/theme.inc it calls a brightcover_player_load($variables['player']); using the Player ID. This fails as it's looking for the local Player Name instead.

Attached patch that renames the Player Id field on the display settings page and removes the Player Key field as it does not seem to be used in this location, as it inherits from the local player settings defined at /admin/config/media/brightcove/players

CommentFileSizeAuthor
brightcove_media.patch1.17 KBgarethsprice

Comments

muldos’s picture

muldos commented

Hi, I have the same bug, the patch helps but I think this should be fixed with an alternate solution :
Providing a select list of players known by drupal like when selecting a player for a node with a brightcove field.
I will see if I can provide a patch for this solution.
Give me your feedback about this alternative fix.

David

garethsprice’s picture

garethsprice commented

A drop-down showing available players would definitely make it easier to configure!

tamasd’s picture

tamasd commented
Status: Needs review » Fixed

Fix is pushed to 7.x-3.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.