Skip to content

Commit a389731

Browse files
committed
fix: update last share update time when changing user status
Signed-off-by: Robin Appelman <robin@icewind.nl>
1 parent 22bd34a commit a389731

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

‎lib/private/Sharing/SharingManager.php‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,10 @@ public function updateShareUserStatus(ShareAccessContext $accessContext, Share $
231231
throw new ShareInvalidException('Cannot set user status for the owner of the share.', $this->l10n->t('Cannot set user status for the owner of the share.'));
232232
}
233233

234-
// We don't update the share last updated value, because the user status is not part of the share itself.
234+
// We update the share last updated value, even though the user status is not part of the share itself,
235+
// so we can use the last updated value to coordinate syncing between unified and legacy shares.
236+
$time = $this->getTime();
237+
$this->backend->setLastUpdated([$share->id], $time);
235238

236239
$this->backend->updateShareUserStatus($share->id, $currentUser->getUID(), $userStatus);
237240

0 commit comments

Comments
 (0)