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 views_data_export_plugin_display_export::outputfile_path() method performs a check on either the _output_file property AND the file ID that only exists on batched requests.
So if the second check fails, this method returns $this->_output_file->uri which does not exist because the first check already stated that the _output_file property was empty.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | views_data_export-outputfile_path-method-call-uri-property-on-non-object-1778838-1.patch | 1010 bytes | B-Prod |
| #1 | filecache-recursive-calls-to-chdir-causes-errors-1778468-1.patch | 2.68 KB | B-Prod |
Comments
Comment #1
B-Prod CreditAttribution: B-Prod commentedThe patch below fixes this issue.
Comment #3
B-Prod CreditAttribution: B-Prod commentedI cloned the DEV repository then applied the patch with success, so the patch is applicable. The log details from the automatic test bot is too generic and does not help...
Comment #4
rfay@B-Prod, this is views_data_export module, and I think you're providing a patch for filecache module... That's why it doesn't apply.
Comment #5
B-Prod CreditAttribution: B-Prod commented@rfay, sorry, lost my mind sometimes...
The correct patch is attached below.
Comment #6
Steven Jones CreditAttribution: Steven Jones commented#5: views_data_export-outputfile_path-method-call-uri-property-on-non-object-1778838-1.patch queued for re-testing.
Comment #7
Steven Jones CreditAttribution: Steven Jones commentedThanks for the patch! Committed and pushed to 7.x-3.x