Skip to content

Commit 8299c23

Browse files
CarlSchwanbackportbot[bot]
authored andcommitted
fix: Don't add empty and broken ExternalShareScanJob on upgrade
This is added manually in apps/files_sharing/lib/Controller/ExternalSharesController.php when needed and with arguments. Signed-off-by: Carl Schwan <carl@carlschwan.eu>
1 parent d7bee33 commit 8299c23

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

‎apps/files_sharing/appinfo/info.xml‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ Turning the feature off removes shared files and folders on the server for all s
3535
<job>OCA\Files_Sharing\ExpireSharesJob</job>
3636
<job>OCA\Files_Sharing\SharesReminderJob</job>
3737
<job>OCA\Files_Sharing\BackgroundJob\FederatedSharesDiscoverJob</job>
38-
<job>OCA\Files_Sharing\BackgroundJob\ExternalShareScanJob</job>
3938
</background-jobs>
4039

4140
<repair-steps>

‎apps/files_sharing/lib/BackgroundJob/ExternalShareScanJob.php‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,10 @@ protected function run($argument): void {
3535
}
3636

3737
[$userId, $path] = $argument;
38+
if ($userId === null || $userId === '') {
39+
return;
40+
}
41+
3842
try {
3943
$this->rootFolder
4044
->getUserFolder($userId)

0 commit comments

Comments
 (0)