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.The "RSS element" formatter doesn't work as expected when using stream wrappers to handle files (e.g. when using the Storage API module without any local file storage).
In views_rss_core_field_formatter_view(), the use of drupal_realpath() means that the formatter tries to generate a derivative on every request, which results in a database duplicate key error.
As far as I can tell, it should be possible to get rid of the call to drupal_realpath() since the functions using $real_path all support stream wrappers.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | views_rss-improved-stream-wrapper-support-1882274-1.patch | 954 bytes | wulff |
Comments
Comment #1
wulff CreditAttribution: wulff commentedI have attached a patch (based on 7.x-2.0-rc3, since that's the version we're currently using). It might apply to HEAD without too much fuss.
Comment #4
maciej.zgadzaj CreditAttribution: maciej.zgadzaj commentedCommitted and pushed to 7.x-2.x and 6.x-2.x.