upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-06-11 09:42:09 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 40

Only trigger postDelete hooks when the user was deleted...

Browse source 
Signed-off-by: Joas Schilling <coding@schilljs.com>
This commit is contained in:
Joas Schilling 2016-10-11 10:55:50 +02:00
parent 203c5f745e
commit 1d9738006a
No known key found for this signature in database
GPG key ID: E166FD8976B3BAC8
2 changed files with 8 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
lib/private/user/user.php
View file

@ -216,10 +216,10 @@ class User implements IUser {
$notification = \OC::$server->getNotificationManager()->createNotification();
$notification->setUser($this->uid);
\OC::$server->getNotificationManager()->markProcessed($notification);
}
if ($this->emitter) {
$this->emitter->emit('\OC\User', 'postDelete', array($this));
if ($this->emitter) {
$this->emitter->emit('\OC\User', 'postDelete', array($this));
}
}
return !($result === false);
}

9
tests/lib/user/user.php
View file

@ -437,16 +437,17 @@ class User extends \Test\TestCase {
public function dataDeleteHooks() {
return [
[true],
[false],
[true, 2],
[false, 1],
];
}
/**
* @dataProvider dataDeleteHooks
* @param bool $result
* @param int $expectedHooks
*/
public function testDeleteHooks($result) {
public function testDeleteHooks($result, $expectedHooks) {
$hooksCalled = 0;
$test = $this;
@ -523,7 +524,7 @@ class User extends \Test\TestCase {
$this->restoreService('CommentsManager');
$this->restoreService('NotificationManager');
$this->assertEquals(2, $hooksCalled);
$this->assertEquals($expectedHooks, $hooksCalled);
}
public function testGetCloudId() {