From a39c995083c6cec88f79558699d4d20bd66f30f2 Mon Sep 17 00:00:00 2001 From: Arthur Schiwon Date: Wed, 11 Apr 2018 15:33:36 +0200 Subject: [PATCH] FIX: emit assignedUserId only for users Signed-off-by: Arthur Schiwon --- apps/user_ldap/lib/Access.php | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/apps/user_ldap/lib/Access.php b/apps/user_ldap/lib/Access.php index 91089e779a0..a03b4a4cb9c 100644 --- a/apps/user_ldap/lib/Access.php +++ b/apps/user_ldap/lib/Access.php @@ -609,23 +609,23 @@ class Access extends LDAPUtility implements IUserTools { //NOTE: mind, disabling cache affects only this instance! Using it // outside of core user management will still cache the user as non-existing. $originalTTL = $this->connection->ldapCacheTTL; - $this->connection->setConfiguration(array('ldapCacheTTL' => 0)); + $this->connection->setConfiguration(['ldapCacheTTL' => 0]); if(($isUser && $intName !== '' && !$this->ncUserManager->userExists($intName)) || (!$isUser && !\OC::$server->getGroupManager()->groupExists($intName))) { if($mapper->map($fdn, $intName, $uuid)) { - $this->connection->setConfiguration(array('ldapCacheTTL' => $originalTTL)); - if($this->ncUserManager instanceof PublicEmitter) { + $this->connection->setConfiguration(['ldapCacheTTL' => $originalTTL]); + if($this->ncUserManager instanceof PublicEmitter && $isUser) { $this->ncUserManager->emit('\OC\User', 'assignedUserId', [$intName]); } $newlyMapped = true; return $intName; } } - $this->connection->setConfiguration(array('ldapCacheTTL' => $originalTTL)); + $this->connection->setConfiguration(['ldapCacheTTL' => $originalTTL]); $altName = $this->createAltInternalOwnCloudName($intName, $isUser); if(is_string($altName) && $mapper->map($fdn, $altName, $uuid)) { - if($this->ncUserManager instanceof PublicEmitter) { + if($this->ncUserManager instanceof PublicEmitter && $isUser) { $this->ncUserManager->emit('\OC\User', 'assignedUserId', [$intName]); } $newlyMapped = true; -- 2.39.5