Merge pull request #3214 from nextcloud/fix/s3_export

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

Author: susnux

lib/Service/SubmissionService.php

@@ -380,9 +380,19 @@ function (array $carry, Answer $answer) use ($questionPerQuestionId, $gridRowsPe
 	 * @param list<non-empty-list<array{columns?: list<mixed|string>, label?: string, url?: string}|mixed|null|string>> $data
 	 */
 	private function exportData(array $header, array $data, string $fileFormat, ?File $file = null): string {
-		if ($file && $file->getContent()) {
+		$content = null;
+		if ($file) {
+			try {
+				$content = $file->getContent();
+			} catch (\Exception $e) {
+				$this->logger->warning('Failed to read existing linked file content: {msg}', ['msg' => $e->getMessage()]);
+			}
+		}
+
+		// Ignore whitespace-only files (e.g. S3 single-space placeholder).
+		if ($content !== null && trim($content) !== '') {
 			$existentFile = $this->tempManager->getTemporaryFile($fileFormat);
-			file_put_contents($existentFile, $file->getContent());
+			file_put_contents($existentFile, $content);
 			$spreadsheet = IOFactory::load($existentFile);
 		} else {
 			$spreadsheet = new Spreadsheet();