diff options
| author | Arthur Schiwon <blizzz@arthur-schiwon.de> | 2018-11-27 17:09:55 +0100 |
|---|---|---|
| committer | Arthur Schiwon <blizzz@arthur-schiwon.de> | 2018-11-27 17:16:58 +0100 |
| commit | ea283c0248135e6a9eac6ce1119f21e1ade0387c (patch) | |
| tree | 156449c2edba230ce82c6cf08397ce6ff57b6e46 /apps | |
| parent | c32cc4a194532f531786c14b99edaee7fc872105 (diff) | |
| download | nextcloud-server-ea283c0248135e6a9eac6ce1119f21e1ade0387c.tar.gz nextcloud-server-ea283c0248135e6a9eac6ce1119f21e1ade0387c.zip | |
be careful with mixed return values even if it may look unsuspicious…
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/dav/lib/CardDAV/SyncService.php | 2 | ||||
| -rw-r--r-- | apps/dav/lib/HookManager.php | 4 |
2 files changed, 4 insertions, 2 deletions
diff --git a/apps/dav/lib/CardDAV/SyncService.php b/apps/dav/lib/CardDAV/SyncService.php index 5bd92015ad7..6f6fa0ba379 100644 --- a/apps/dav/lib/CardDAV/SyncService.php +++ b/apps/dav/lib/CardDAV/SyncService.php @@ -261,7 +261,7 @@ class SyncService { /** * @param IUser $user */ - public function updateUser($user) { + public function updateUser(IUser $user) { $systemAddressBook = $this->getLocalSystemAddressBook(); $addressBookId = $systemAddressBook['id']; $converter = new Converter($this->accountManager); diff --git a/apps/dav/lib/HookManager.php b/apps/dav/lib/HookManager.php index b1bd039c65e..27b6525be47 100644 --- a/apps/dav/lib/HookManager.php +++ b/apps/dav/lib/HookManager.php @@ -101,7 +101,9 @@ class HookManager { public function postCreateUser($params) { $user = $this->userManager->get($params['uid']); - $this->syncService->updateUser($user); + if ($user instanceof IUser) { + $this->syncService->updateUser($user); + } } public function preDeleteUser($params) { |