I'm getting a lot of PHP errors in my logs from the Brightcove Integration module:

Notice: Trying to get property of non-object in brightcove_field_preprocess_brightcove_field_embed() (line 85 of ../sites/all/modules/brightcove/brightcove_field/theme.inc).

Notice: Undefined index: 1 in brightcove_player_load() (line 763 of ../sites/all/modules/brightcove/brightcove.module).

Comments

mattking5000’s picture

Seeing this as well. Noticed that update.php fails for part of this module and requires a manual DB update to both field_data_field_brightcove and field_revision_field_brightcove

k.dani’s picture

Status: Active » Postponed (maintainer needs more info)

I can't reproduce these errors. Could you please give me guidence to be able to reproduce the problem? Or can you check it with the latest dev version?

k.dani’s picture

hi mattking5000,

I'm not sure what problem is what you mentioned, but maybe the following is related: #2277923: Update path fail
If it is another problem, please create a separeted ticket for it, and give me a deatailed description.

Thanks!

mstef’s picture

Same issue:

Notice: Undefined index: in brightcove_player_load() (line 763 of /var/www/docroot/sites/all/modules/contrib/brightcove/brightcove.module).
Notice: Trying to get property of non-object in brightcove_field_preprocess_brightcove_field_embed() (line 85 of /var/www/docroot/sites/all/modules/contrib/brightcove/brightcove_field/theme.inc).

Have a brightcove field with a video added to it. Trying to render the video on node view.

mstef’s picture

Problem is if there is no default video player set in the admin-settings or the field settings.

jiaqi.yin’s picture

We have a custom content type which contains a field of type brightcove video. We came across the same error message since upgrading this module from 7.x-3.4 to 7.x-5.1.

The cause in our case is from the function:

function brightcove_field_get_value($instance, $stored) {
  if ($stored) {
    return $stored;
  }
  if (!empty($instance['settings']['brightcove_player'])) {
    return $instance['settings']['brightcove_player'];
  }
  // If no player is set to the content and the BC video field,
  // try to get the global default player.
  elseif ($pid = variable_get('brightcove_player_default')) {
    return $pid;
  }
  return NULL;
}

The value of $instance['settings']['brightcove_player'] was '1', the player pid from brightcove_player.pid; after upgrading, the expected value is the player name from brightcove_player.name but it stays pid.

We fixed that by just going to the content field edit page and select one player from the Brightcove Player list and Save settings.

I noticed that in brightcove_update_7007(), there is function to update the value of $instance['settings']['brightcove_player'] by replace pid with name, and I ran the update successfully. But no idea why it's not change in this case. I would dig into this hook update if time permits.

jan.mashat’s picture

Status: Postponed (maintainer needs more info) » Closed (won't fix)
Issue tags: +unsupported

This version of the module is no longer supported. Only the latest 2 major versions are supported, which are currently 8.x-1.x and 7.x-6.x. Therefore I'm tagging this issue as 'unsupported' and closing it (won't fix). If this issue is business critical for your organization, please contact your Brightcove account manager for further support.