gracefully handle deleteFromSelf when share is already gone
- handling race conditions Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
This commit is contained in:
parent
2614d99656
commit
85225cce0a
|
@ -470,7 +470,8 @@ class DefaultShareProvider implements IShareProvider {
|
|||
}
|
||||
|
||||
if (!$group->inGroup($user)) {
|
||||
throw new ProviderException('Recipient not in receiving group');
|
||||
// nothing left to do
|
||||
return;
|
||||
}
|
||||
|
||||
// Try to fetch user specific share
|
||||
|
|
|
@ -1575,9 +1575,6 @@ class DefaultShareProviderTest extends \Test\TestCase {
|
|||
|
||||
|
||||
public function testDeleteFromSelfGroupUserNotInGroup() {
|
||||
$this->expectException(\OC\Share20\Exception\ProviderException::class);
|
||||
$this->expectExceptionMessage('Recipient not in receiving group');
|
||||
|
||||
$qb = $this->dbConn->getQueryBuilder();
|
||||
$stmt = $qb->insert('share')
|
||||
->values([
|
||||
|
|
Loading…
Reference in New Issue