In template_preprocess_videojs_formatter_videojs(), the $vars['items']['thumbnail'] variable that is derived from content-type-level $poster_image is referencing the field's filepath string directly. This must be incorrect, as the rest of the code expects this variable to be an array that represents the file object. Consequently, this causes multiple errors:

- and
get rendered in the template, which causes IE to puke.
- PHP notices under strict mode, alerting that a string is wrongfully being used as an array in multiple areas.
- $vars['poster'] derived from the 'thumbnail' string always fails and returns an empty string in the template for the video element's poster attribute.

CommentFileSizeAuthor
#1 poster-1459254-1.patch885 byteshelior
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

helior’s picture

Status: Active » Needs review
FileSize
885 bytes

This patch references assigns the entire file object (array) in $vars['items']['thumbnail'], instead of the filepath directly.

Jorrit’s picture

Status: Needs review » Fixed

Fixed, thanks.

Status: Fixed » Closed (fixed)

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