be careful with mixed return values even if it may look unsuspicious…

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
This commit is contained in:
Arthur Schiwon 2018-11-27 17:09:55 +01:00 committed by Backportbot
parent e3fd241aba
commit c6b540a738
2 changed files with 4 additions and 2 deletions

View File

@ -261,7 +261,7 @@ class SyncService {
/** /**
* @param IUser $user * @param IUser $user
*/ */
public function updateUser($user) { public function updateUser(IUser $user) {
$systemAddressBook = $this->getLocalSystemAddressBook(); $systemAddressBook = $this->getLocalSystemAddressBook();
$addressBookId = $systemAddressBook['id']; $addressBookId = $systemAddressBook['id'];
$converter = new Converter($this->accountManager); $converter = new Converter($this->accountManager);

View File

@ -101,7 +101,9 @@ class HookManager {
public function postCreateUser($params) { public function postCreateUser($params) {
$user = $this->userManager->get($params['uid']); $user = $this->userManager->get($params['uid']);
$this->syncService->updateUser($user); if ($user instanceof IUser) {
$this->syncService->updateUser($user);
}
} }
public function preDeleteUser($params) { public function preDeleteUser($params) {