From b4e2d8f8adf100654a45bd7ab1d8fa3d482b7e9a Mon Sep 17 00:00:00 2001 From: Robin Appelman Date: Thu, 6 Jun 2019 16:09:27 +0200 Subject: handle storage exceptions when trying to set mtime not all storage backends can handle setting the mtime and they might not always handle that error correctly. Signed-off-by: Robin Appelman --- lib/private/Files/View.php | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'lib/private/Files/View.php') diff --git a/lib/private/Files/View.php b/lib/private/Files/View.php index fef6153fb9a..2516b782ee7 100644 --- a/lib/private/Files/View.php +++ b/lib/private/Files/View.php @@ -566,7 +566,12 @@ class View { $hooks[] = 'create'; $hooks[] = 'write'; } - $result = $this->basicOperation('touch', $path, $hooks, $mtime); + try { + $result = $this->basicOperation('touch', $path, $hooks, $mtime); + } catch (\Exception $e) { + $this->logger->logException($e, ['level' => ILogger::INFO, 'message' => 'Error while setting modified time']); + $result = false; + } if (!$result) { // If create file fails because of permissions on external storage like SMB folders, // check file exists and return false if not. -- cgit v1.2.3