Skip to content
Snippets Groups Projects

Issue #3241213: Enhance previous S3 bucket support patch with file database URI update.

Open Issue #3241213: Enhance previous S3 bucket support patch with file database URI update.
Open George Kaffezas requested to merge issue/views_data_export-3241213:8.x-1.x into 8.x-1.x

Working on a site that had #2 comment's patch applied, we faced a problem with the automatic download after the batch finished. More specifically:

  • The files were properly saved in the private directory, however the respective database entry in file_managed table was not updated and pointed to the temporary directory.
  • Thus $headers in finishBatch() were empty because the function failed to find the file via its URI and consequently it was not downloaded.

Based on that patch, I added some code in order to update the respective database entry with the new URI (that points to private directory), along with a couple of log messages to have a better overview of the process.

Merge request reports

Members who can merge are allowed to add commits.
Approval is optional
Code Quality is loading
Test summary results are being parsed
Ready to merge by members who can write to the target branch.
  • 1 commit will be added to 8.x-1.x.
  • Source branch will not be deleted.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading