fix(export): Handle exceptions when reading file content and ignore whitespace-only files

[Chartman123]

This fixes #3175 by adding a check if the file really has content.

Signed-off-by: Christian Hartmann chris-hartmann@gmx.de

[Chartman123]

@sheyaln could you please try this fix? I don't have access to an instance that uses S3 storage

[codecov]

Codecov Report

❌ Patch coverage is 57.14286% with 3 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
lib/Service/SubmissionService.php	57.14%	3 Missing ⚠️

📢 Thoughts on this report? Let us know!

[Chartman123]

/backport to stable5.2

[sheyaln]

@sheyaln could you please try this fix? I don't have access to an instance that uses S3 storage

Apologies, how do you recommend I test this fix?

[Chartman123]

You can go to the Forms app directory on your server and manually patch the lib/service/SubmissionService.php file with the changes from this PR. Make a backup of the original file. You can also apply the patch file https://github.com/nextcloud/forms/pull/3214.patch

[susnux]

Nice, just to small comments

[Chartman123]

Nice, just to small comments

@susnux thanks for the review, comments are resolved now :)