Line 13 :

<source src="<?php print($item['filepath']) ?>" type="<?php print($item['videotype']) ?>" />

Replaced by :

<source src="<?php print($filepath) ?>" type="<?php print($item['videotype']) ?>" />
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

steinmb’s picture

Status: Active » Needs work

Hi. Was debugging an other issue but felt I need to comment on this. Not sure that this patch is correct either:

<?php $filepath = file_create_url($item['filepath']); ?>
<source src="<?php print($item['filepath']) ?>" type="<?php print($item['videotype']) ?>" />

In the first line it create the $filepath variable, but If I look at the content at my local sandbox setup is the content of this field:
http://hardanger:8888/sites/hardanger.libriotech.no/files/vedlegg/screen_recording.webmvp8.webm

Reading up on http://api.drupal.org/api/drupal/includes--file.inc/function/file_create... and from what understand, it does not support URI (D7 does though), it needs the path. Trying to add an URI will give us in this example:
http://hardanger:8888/sites/hardanger.libriotech.no/files/http://hardanger:8888/sites/hardanger.libriotech.no/files/vedlegg/screen_recording.webmvp8.webm

Jorrit’s picture

Status: Needs work » Fixed

Fixed in 6.x-1.x, please test this in tomorrows -dev release.

Status: Fixed » Closed (fixed)

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