Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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']) ?>" />
Comment | File | Size | Author |
---|---|---|---|
videojs-wrong-variable-call.2011-10-21.patch | 873 bytes | Chris-Ju | |
Comments
Comment #1
steinmb CreditAttribution: steinmb commentedHi. Was debugging an other issue but felt I need to comment on this. Not sure that this patch is correct either:
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
Comment #2
Jorrit CreditAttribution: Jorrit commentedFixed in 6.x-1.x, please test this in tomorrows -dev release.