Skip to content

GitLab

Commit 040c5411 authored by Robin Appelman's avatar Robin Appelman
Browse files

Remove a fed share from the local table before trying to notify the remote server

parent 1e1903e4
Hide whitespace changes
Inline Side-by-side
apps/federatedfilesharing/lib/FederatedShareProvider.php
View file @ 040c5411
...@@ -473,6 +473,8 @@ class FederatedShareProvider implements IShareProvider { ...@@ -473,6 +473,8 @@ class FederatedShareProvider implements IShareProvider {
$isOwner = false; $isOwner = false;
$this->removeShareFromTable($share);
// if the local user is the owner we can send the unShare request directly... // if the local user is the owner we can send the unShare request directly...
if ($this->userManager->userExists($share->getShareOwner())) { if ($this->userManager->userExists($share->getShareOwner())) {
$this->notifications->sendRemoteUnShare($remote, $share->getId(), $share->getToken()); $this->notifications->sendRemoteUnShare($remote, $share->getId(), $share->getToken());
...@@ -494,8 +496,6 @@ class FederatedShareProvider implements IShareProvider { ...@@ -494,8 +496,6 @@ class FederatedShareProvider implements IShareProvider {
} }
$this->notifications->sendRevokeShare($remote, $remoteId, $share->getToken()); $this->notifications->sendRevokeShare($remote, $remoteId, $share->getToken());
} }
$this->removeShareFromTable($share);
} }
/** /**
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment