Skip to content

Commit 32d4f84

Browse files
authored
Merge pull request #53782 from nextcloud/backport/53276/stable31
[stable31] fix(files_versions): Log failure to compute node path
2 parents 2716713 + 5e25698 commit 32d4f84

File tree

1 file changed

+19
-0
lines changed

1 file changed

+19
-0
lines changed

apps/files_versions/lib/Listener/FileEventsListener.php

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
use OC\Files\Filesystem;
1313
use OC\Files\Mount\MoveableMount;
1414
use OC\Files\Node\NonExistingFile;
15+
use OC\Files\Node\NonExistingFolder;
1516
use OC\Files\View;
1617
use OCA\Files_Versions\Storage;
1718
use OCA\Files_Versions\Versions\INeedSyncVersionBackend;
@@ -401,6 +402,24 @@ private function getPathForNode(Node $node): ?string {
401402
}
402403
}
403404

405+
if (!($node instanceof NonExistingFile) && !($node instanceof NonExistingFolder)) {
406+
$this->logger->debug('Failed to compute path for node', [
407+
'node' => [
408+
'path' => $node->getPath(),
409+
'owner' => $owner,
410+
'fileid' => $node->getId(),
411+
'size' => $node->getSize(),
412+
'mtime' => $node->getMTime(),
413+
]
414+
]);
415+
} else {
416+
$this->logger->debug('Failed to compute path for node', [
417+
'node' => [
418+
'path' => $node->getPath(),
419+
'owner' => $owner,
420+
]
421+
]);
422+
}
404423
return null;
405424
}
406425
}

0 commit comments

Comments
 (0)