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.When using the S3 File System module in your site along with Brightcove Connect, the brightcove_remote_image() will always return FALSE because the $fullpath variable is set by running the path schema through drupal_real_path() and this is not the path for images when using S3FS. This causes the file_prepare_directory() call to fail and therefore return FALSE. Patch coming shortly.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | brightcove-s3fs-breaks-remote-image-2792431-10.patch | 1.08 KB | eugenesia |
| #10 | interdiff-9-10.txt | 310 bytes | eugenesia |
Comments
Comment #2
r0nn1ef CreditAttribution: r0nn1ef as a volunteer commentedComment #3
r0nn1ef CreditAttribution: r0nn1ef as a volunteer commentedComment #4
jan.mashat CreditAttribution: jan.mashat at Pronovix for BrightCove commentedComment #5
jan.mashat CreditAttribution: jan.mashat at Pronovix for BrightCove commentedComment #6
r0nn1ef CreditAttribution: r0nn1ef as a volunteer commentedThis is still and issue with the 7.x-6.5 release. Why has my patch not been reviewed?
Comment #7
robdubparker CreditAttribution: robdubparker commented+1 for the patch here
Comment #8
eugenesia CreditAttribution: eugenesia as a volunteer commentedRe-rolled patch for latest dev version 7.x-6.x-dev. Fixed error message.
Tested to make sure it works for filesystem and S3FS scenarios.
Comment #9
eugenesia CreditAttribution: eugenesia as a volunteer commentedFixed incorrect patch file name - changed from 2792341 (typo) to 2792431 (correct issue number).
Comment #10
eugenesia CreditAttribution: eugenesia as a volunteer commentedFixed bug where thumbnails were saved using random names using
drupal_tempnam()e.g. fileABC123.