nextcloud / server

apps/user_ldap/lib/Access.php

Spacing +60 added lines, -60 removed lines

@@ -230,13 +230,13 @@ 
 					return $values;
 				} else {
 					$low = $result['rangeHigh'] + 1;
-					$attrToRead = $result['attributeName'] . ';range=' . $low . '-*';
+					$attrToRead = $result['attributeName'].';range='.$low.'-*';
 					$isRangeRequest = true;
 				}
 			}
 		} while ($isRangeRequest);
 
-		$this->logger->debug('Requested attribute ' . $attr . ' not found for ' . $dn, ['app' => 'user_ldap']);
+		$this->logger->debug('Requested attribute '.$attr.' not found for '.$dn, ['app' => 'user_ldap']);
 		return false;
 	}
 
@@ -254,13 +254,13 @@ 
 		if (!$this->ldap->isResource($rr)) {
 			if ($attribute !== '') {
 				//do not throw this message on userExists check, irritates
-				$this->logger->debug('readAttribute failed for DN ' . $dn, ['app' => 'user_ldap']);
+				$this->logger->debug('readAttribute failed for DN '.$dn, ['app' => 'user_ldap']);
 			}
 			//in case an error occurs , e.g. object does not exist
 			return false;
 		}
 		if ($attribute === '' && ($filter === 'objectclass=*' || $this->invokeLDAPMethod('countEntries', $rr) === 1)) {
-			$this->logger->debug('readAttribute: ' . $dn . ' found', ['app' => 'user_ldap']);
+			$this->logger->debug('readAttribute: '.$dn.' found', ['app' => 'user_ldap']);
 			return true;
 		}
 		$er = $this->invokeLDAPMethod('firstEntry', $rr);
@@ -313,8 +313,8 @@ 
 	public function extractRangeData($result, $attribute) {
 		$keys = array_keys($result);
 		foreach ($keys as $key) {
-			if ($key !== $attribute && strpos((string)$key, $attribute) === 0) {
-				$queryData = explode(';', (string)$key);
+			if ($key !== $attribute && strpos((string) $key, $attribute) === 0) {
+				$queryData = explode(';', (string) $key);
 				if (strpos($queryData[1], 'range=') === 0) {
 					$high = substr($queryData[1], 1 + strpos($queryData[1], '-'));
 					$data = [
@@ -340,7 +340,7 @@ 
 	 * @throws \Exception
 	 */
 	public function setPassword($userDN, $password) {
-		if ((int)$this->connection->turnOnPasswordChange !== 1) {
+		if ((int) $this->connection->turnOnPasswordChange !== 1) {
 			throw new \Exception('LDAP password changes are disabled.');
 		}
 		$cr = $this->connection->getConnectionResource();
@@ -354,7 +354,7 @@ 
 			return @$this->invokeLDAPMethod('exopPasswd', $userDN, '', $password) ||
 				@$this->invokeLDAPMethod('modReplace', $userDN, $password);
 		} catch (ConstraintViolationException $e) {
-			throw new HintException('Password change rejected.', \OC::$server->getL10N('user_ldap')->t('Password change rejected. Hint: ') . $e->getMessage(), (int)$e->getCode());
+			throw new HintException('Password change rejected.', \OC::$server->getL10N('user_ldap')->t('Password change rejected. Hint: ').$e->getMessage(), (int) $e->getCode());
 		}
 	}
 
@@ -494,7 +494,7 @@ 
 	 */
 	public function dn2ocname($fdn, $ldapName = null, $isUser = true, &$newlyMapped = null, array $record = null) {
 		static $intermediates = [];
-		if (isset($intermediates[($isUser ? 'user-' : 'group-') . $fdn])) {
+		if (isset($intermediates[($isUser ? 'user-' : 'group-').$fdn])) {
 			return false; // is a known intermediate
 		}
 
@@ -525,26 +525,26 @@ 
 			}
 		} else {
 			//If the UUID can't be detected something is foul.
-			$this->logger->debug('Cannot determine UUID for ' . $fdn . '. Skipping.', ['app' => 'user_ldap']);
+			$this->logger->debug('Cannot determine UUID for '.$fdn.'. Skipping.', ['app' => 'user_ldap']);
 			return false;
 		}
 
 		if (is_null($ldapName)) {
 			$ldapName = $this->readAttribute($fdn, $nameAttribute, $filter);
 			if (!isset($ldapName[0]) || empty($ldapName[0])) {
-				$this->logger->debug('No or empty name for ' . $fdn . ' with filter ' . $filter . '.', ['app' => 'user_ldap']);
-				$intermediates[($isUser ? 'user-' : 'group-') . $fdn] = true;
+				$this->logger->debug('No or empty name for '.$fdn.' with filter '.$filter.'.', ['app' => 'user_ldap']);
+				$intermediates[($isUser ? 'user-' : 'group-').$fdn] = true;
 				return false;
 			}
 			$ldapName = $ldapName[0];
 		}
 
 		if ($isUser) {
-			$usernameAttribute = (string)$this->connection->ldapExpertUsernameAttr;
+			$usernameAttribute = (string) $this->connection->ldapExpertUsernameAttr;
 			if ($usernameAttribute !== '') {
 				$username = $this->readAttribute($fdn, $usernameAttribute);
 				if (!isset($username[0]) || empty($username[0])) {
-					$this->logger->debug('No or empty username (' . $usernameAttribute . ') for ' . $fdn . '.', ['app' => 'user_ldap']);
+					$this->logger->debug('No or empty username ('.$usernameAttribute.') for '.$fdn.'.', ['app' => 'user_ldap']);
 					return false;
 				}
 				$username = $username[0];
@@ -554,7 +554,7 @@ 
 			try {
 				$intName = $this->sanitizeUsername($username);
 			} catch (\InvalidArgumentException $e) {
-				$this->logger->warning('Error sanitizing username: ' . $e->getMessage(), [
+				$this->logger->warning('Error sanitizing username: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 				// we don't attempt to set a username here. We can go for
@@ -604,7 +604,7 @@ 
 		}
 
 		//if everything else did not help..
-		$this->logger->info('Could not create unique name for ' . $fdn . '.', ['app' => 'user_ldap']);
+		$this->logger->info('Could not create unique name for '.$fdn.'.', ['app' => 'user_ldap']);
 		return false;
 	}
 
@@ -710,7 +710,7 @@ 
 	 * @param string|false $home the home directory path
 	 */
 	public function cacheUserHome(string $ocName, $home): void {
-		$cacheKey = 'getHome' . $ocName;
+		$cacheKey = 'getHome'.$ocName;
 		$this->connection->writeToCache($cacheKey, $home);
 	}
 
@@ -718,14 +718,14 @@ 
 	 * caches a user as existing
 	 */
 	public function cacheUserExists(string $ocName): void {
-		$this->connection->writeToCache('userExists' . $ocName, true);
+		$this->connection->writeToCache('userExists'.$ocName, true);
 	}
 
 	/**
 	 * caches a group as existing
 	 */
 	public function cacheGroupExists(string $gid): void {
-		$this->connection->writeToCache('groupExists' . $gid, true);
+		$this->connection->writeToCache('groupExists'.$gid, true);
 	}
 
 	/**
@@ -743,11 +743,11 @@ 
 		}
 		$displayName = $user->composeAndStoreDisplayName($displayName, $displayName2);
 		$cacheKeyTrunk = 'getDisplayName';
-		$this->connection->writeToCache($cacheKeyTrunk . $ocName, $displayName);
+		$this->connection->writeToCache($cacheKeyTrunk.$ocName, $displayName);
 	}
 
 	public function cacheGroupDisplayName(string $ncName, string $displayName): void {
-		$cacheKey = 'group_getDisplayName' . $ncName;
+		$cacheKey = 'group_getDisplayName'.$ncName;
 		$this->connection->writeToCache($cacheKey, $displayName);
 	}
 
@@ -765,7 +765,7 @@ 
 		//while loop is just a precaution. If a name is not generated within
 		//20 attempts, something else is very wrong. Avoids infinite loop.
 		while ($attempts < 20) {
-			$altName = $name . '_' . rand(1000, 9999);
+			$altName = $name.'_'.rand(1000, 9999);
 			if (!$this->ncUserManager->userExists($altName)) {
 				return $altName;
 			}
@@ -794,9 +794,9 @@ 
 		} else {
 			natsort($usedNames);
 			$lastName = array_pop($usedNames);
-			$lastNo = (int)substr($lastName, strrpos($lastName, '_') + 1);
+			$lastNo = (int) substr($lastName, strrpos($lastName, '_') + 1);
 		}
-		$altName = $name . '_' . (string)($lastNo + 1);
+		$altName = $name.'_'.(string) ($lastNo + 1);
 		unset($usedNames);
 
 		$attempts = 1;
@@ -807,7 +807,7 @@ 
 			if (!\OC::$server->getGroupManager()->groupExists($altName)) {
 				return $altName;
 			}
-			$altName = $name . '_' . ($lastNo + $attempts);
+			$altName = $name.'_'.($lastNo + $attempts);
 			$attempts++;
 		}
 		return false;
@@ -870,12 +870,12 @@ 
 		if (!$forceApplyAttributes) {
 			$isBackgroundJobModeAjax = $this->config
 					->getAppValue('core', 'backgroundjobs_mode', 'ajax') === 'ajax';
-			$listOfDNs = array_reduce($ldapRecords, function ($listOfDNs, $entry) {
+			$listOfDNs = array_reduce($ldapRecords, function($listOfDNs, $entry) {
 				$listOfDNs[] = $entry['dn'][0];
 				return $listOfDNs;
 			}, []);
 			$idsByDn = $this->getUserMapper()->getListOfIdsByDn($listOfDNs);
-			$recordsToUpdate = array_filter($ldapRecords, function ($record) use ($isBackgroundJobModeAjax, $idsByDn) {
+			$recordsToUpdate = array_filter($ldapRecords, function($record) use ($isBackgroundJobModeAjax, $idsByDn) {
 				$newlyMapped = false;
 				$uid = $idsByDn[$record['dn'][0]] ?? null;
 				if ($uid === null) {
@@ -899,7 +899,7 @@ 
 	 * @throws \Exception
 	 */
 	public function batchApplyUserAttributes(array $ldapRecords): void {
-		$displayNameAttribute = strtolower((string)$this->connection->ldapUserDisplayName);
+		$displayNameAttribute = strtolower((string) $this->connection->ldapUserDisplayName);
 		foreach ($ldapRecords as $userRecord) {
 			if (!isset($userRecord[$displayNameAttribute])) {
 				// displayName is obligatory
@@ -926,20 +926,20 @@ 
 	 * @return array[]
 	 */
 	public function fetchListOfGroups(string $filter, array $attr, int $limit = null, int $offset = null): array {
-		$cacheKey = 'fetchListOfGroups_' . $filter . '_' . implode('-', $attr) . '_' . (string)$limit . '_' . (string)$offset;
+		$cacheKey = 'fetchListOfGroups_'.$filter.'_'.implode('-', $attr).'_'.(string) $limit.'_'.(string) $offset;
 		$listOfGroups = $this->connection->getFromCache($cacheKey);
 		if (!is_null($listOfGroups)) {
 			return $listOfGroups;
 		}
 		$groupRecords = $this->searchGroups($filter, $attr, $limit, $offset);
 
-		$listOfDNs = array_reduce($groupRecords, function ($listOfDNs, $entry) {
+		$listOfDNs = array_reduce($groupRecords, function($listOfDNs, $entry) {
 			$listOfDNs[] = $entry['dn'][0];
 			return $listOfDNs;
 		}, []);
 		$idsByDn = $this->getGroupMapper()->getListOfIdsByDn($listOfDNs);
 
-		array_walk($groupRecords, function (array $record) use ($idsByDn) {
+		array_walk($groupRecords, function(array $record) use ($idsByDn) {
 			$newlyMapped = false;
 			$gid = $idsByDn[$record['dn'][0]] ?? null;
 			if ($gid === null) {
@@ -958,7 +958,7 @@ 
 		if ($manyAttributes) {
 			return $list;
 		} else {
-			$list = array_reduce($list, function ($carry, $item) {
+			$list = array_reduce($list, function($carry, $item) {
 				$attribute = array_keys($item)[0];
 				$carry[] = $item[$attribute][0];
 				return $carry;
@@ -987,7 +987,7 @@ 
 		$result = false;
 		foreach ($this->connection->ldapBaseUsers as $base) {
 			$count = $this->count($filter, [$base], $attr, $limit ?? 0, $offset ?? 0);
-			$result = is_int($count) ? (int)$result + $count : $result;
+			$result = is_int($count) ? (int) $result + $count : $result;
 		}
 		return $result;
 	}
@@ -1018,7 +1018,7 @@ 
 		$result = false;
 		foreach ($this->connection->ldapBaseGroups as $base) {
 			$count = $this->count($filter, [$base], $attr, $limit ?? 0, $offset ?? 0);
-			$result = is_int($count) ? (int)$result + $count : $result;
+			$result = is_int($count) ? (int) $result + $count : $result;
 		}
 		return $result;
 	}
@@ -1033,7 +1033,7 @@ 
 		$result = false;
 		foreach ($this->connection->ldapBase as $base) {
 			$count = $this->count('objectclass=*', [$base], ['dn'], $limit ?? 0, $offset ?? 0);
-			$result = is_int($count) ? (int)$result + $count : $result;
+			$result = is_int($count) ? (int) $result + $count : $result;
 		}
 		return $result;
 	}
@@ -1060,7 +1060,7 @@ 
 			return null;
 		}
 		array_unshift($arguments, $this->connection->getConnectionResource());
-		$doMethod = function () use ($command, &$arguments) {
+		$doMethod = function() use ($command, &$arguments) {
 			return call_user_func_array([$this->ldap, $command], $arguments);
 		};
 		try {
@@ -1116,7 +1116,7 @@ 
 
 		//check whether paged search should be attempted
 		try {
-			[$pagedSearchOK, $pageSize, $cookie] = $this->initPagedSearch($filter, $base, $attr, (int)$pageSize, (int)$offset);
+			[$pagedSearchOK, $pageSize, $cookie] = $this->initPagedSearch($filter, $base, $attr, (int) $pageSize, (int) $offset);
 		} catch (NoMoreResults $e) {
 			// beyond last results page
 			return false;
@@ -1125,7 +1125,7 @@ 
 		$sr = $this->invokeLDAPMethod('search', $base, $filter, $attr, 0, 0, $pageSize, $cookie);
 		$error = $this->ldap->errno($this->connection->getConnectionResource());
 		if (!$this->ldap->isResource($sr) || $error !== 0) {
-			$this->logger->error('Attempt for Paging?  ' . print_r($pagedSearchOK, true), ['app' => 'user_ldap']);
+			$this->logger->error('Attempt for Paging?  '.print_r($pagedSearchOK, true), ['app' => 'user_ldap']);
 			return false;
 		}
 
@@ -1169,7 +1169,7 @@ 
 				$this->pagedSearchedSuccessful = true;
 			}
 		} else {
-			if ((int)$this->connection->ldapPagingSize !== 0) {
+			if ((int) $this->connection->ldapPagingSize !== 0) {
 				$this->logger->debug(
 					'Paged search was not available',
 					['app' => 'user_ldap']
@@ -1211,7 +1211,7 @@ 
 			'filter' => $filter
 		]);
 
-		$limitPerPage = (int)$this->connection->ldapPagingSize;
+		$limitPerPage = (int) $this->connection->ldapPagingSize;
 		if ($limit < $limitPerPage && $limit > 0) {
 			$limitPerPage = $limit;
 		}
@@ -1253,7 +1253,7 @@ 
 	 * @throws ServerNotAvailableException
 	 */
 	private function countEntriesInSearchResults($sr): int {
-		return (int)$this->invokeLDAPMethod('countEntries', $sr);
+		return (int) $this->invokeLDAPMethod('countEntries', $sr);
 	}
 
 	/**
@@ -1269,7 +1269,7 @@ 
 		?int $offset = null,
 		bool $skipHandling = false
 	): array {
-		$limitPerPage = (int)$this->connection->ldapPagingSize;
+		$limitPerPage = (int) $this->connection->ldapPagingSize;
 		if (!is_null($limit) && $limit < $limitPerPage && $limit > 0) {
 			$limitPerPage = $limit;
 		}
@@ -1422,7 +1422,7 @@ 
 		}
 		$search = ['*', '\\', '(', ')'];
 		$replace = ['\\*', '\\\\', '\\(', '\\)'];
-		return $asterisk . str_replace($search, $replace, $input);
+		return $asterisk.str_replace($search, $replace, $input);
 	}
 
 	/**
@@ -1454,10 +1454,10 @@ 
 	 * @return string the combined filter
 	 */
 	private function combineFilter(array $filters, string $operator): string {
-		$combinedFilter = '(' . $operator;
+		$combinedFilter = '('.$operator;
 		foreach ($filters as $filter) {
 			if ($filter !== '' && $filter[0] !== '(') {
-				$filter = '(' . $filter . ')';
+				$filter = '('.$filter.')';
 			}
 			$combinedFilter .= $filter;
 		}
@@ -1510,7 +1510,7 @@ 
 			//every word needs to appear at least once
 			$wordMatchOneAttrFilters = [];
 			foreach ($searchAttributes as $attr) {
-				$wordMatchOneAttrFilters[] = $attr . '=' . $word;
+				$wordMatchOneAttrFilters[] = $attr.'='.$word;
 			}
 			$wordFilters[] = $this->combineFilterWithOr($wordMatchOneAttrFilters);
 		}
@@ -1544,17 +1544,17 @@ 
 				return '';
 			}
 			// wildcards don't work with some attributes
-			$filter[] = $fallbackAttribute . '=' . $originalSearch;
-			$filter[] = $fallbackAttribute . '=' . $search;
+			$filter[] = $fallbackAttribute.'='.$originalSearch;
+			$filter[] = $fallbackAttribute.'='.$search;
 		} else {
 			foreach ($searchAttributes as $attribute) {
 				// wildcards don't work with some attributes
-				$filter[] = $attribute . '=' . $originalSearch;
-				$filter[] = $attribute . '=' . $search;
+				$filter[] = $attribute.'='.$originalSearch;
+				$filter[] = $attribute.'='.$search;
 			}
 		}
 		if (count($filter) === 1) {
-			return '(' . $filter[0] . ')';
+			return '('.$filter[0].')';
 		}
 		return $this->combineFilterWithOr($filter);
 	}
@@ -1573,7 +1573,7 @@ 
 		if ($term === '') {
 			$result = '*';
 		} elseif ($allowEnum !== 'no') {
-			$result = $term . '*';
+			$result = $term.'*';
 		}
 		return $result;
 	}
@@ -1584,7 +1584,7 @@ 
 	public function getFilterForUserCount(): string {
 		$filter = $this->combineFilterWithAnd([
 			$this->connection->ldapUserFilter,
-			$this->connection->ldapUserDisplayName . '=*'
+			$this->connection->ldapUserDisplayName.'=*'
 		]);
 
 		return $filter;
@@ -1650,7 +1650,7 @@ 
 			$uuid = $this->formatGuid2ForFilterUser($uuid);
 		}
 
-		$filter = $uuidAttr . '=' . $uuid;
+		$filter = $uuidAttr.'='.$uuid;
 		$result = $this->searchUsers($filter, ['dn'], 2);
 		if (isset($result[0]['dn']) && count($result) === 1) {
 			// we put the count into account to make sure that this is
@@ -1777,8 +1777,8 @@ 
 		for ($k = 1; $k <= 2; ++$k) {
 			$hex_guid_to_guid_str .= substr($hex_guid, 16 - 2 * $k, 2);
 		}
-		$hex_guid_to_guid_str .= '-' . substr($hex_guid, 16, 4);
-		$hex_guid_to_guid_str .= '-' . substr($hex_guid, 20);
+		$hex_guid_to_guid_str .= '-'.substr($hex_guid, 16, 4);
+		$hex_guid_to_guid_str .= '-'.substr($hex_guid, 20);
 
 		return strtoupper($hex_guid_to_guid_str);
 	}
@@ -1806,7 +1806,7 @@ 
 			 * user. Instead we write a log message.
 			 */
 			$this->logger->info(
-				'Passed string does not resemble a valid GUID. Known UUID ' .
+				'Passed string does not resemble a valid GUID. Known UUID '.
 				'({uuid}) probably does not match UUID configuration.',
 				['app' => 'user_ldap', 'uuid' => $guid]
 			);
@@ -1819,7 +1819,7 @@ 
 		}
 		for ($i = 0; $i < 5; $i++) {
 			$pairs = str_split($blocks[$i], 2);
-			$blocks[$i] = '\\' . implode('\\', $pairs);
+			$blocks[$i] = '\\'.implode('\\', $pairs);
 		}
 		return implode('', $blocks);
 	}
@@ -1833,7 +1833,7 @@ 
 	 */
 	public function getSID($dn) {
 		$domainDN = $this->getDomainDNFromDN($dn);
-		$cacheKey = 'getSID-' . $domainDN;
+		$cacheKey = 'getSID-'.$domainDN;
 		$sid = $this->connection->getFromCache($cacheKey);
 		if (!is_null($sid)) {
 			return $sid;
@@ -2019,7 +2019,7 @@ 
 			$this->abandonPagedSearch();
 			// in case someone set it to 0 … use 500, otherwise no results will
 			// be returned.
-			$pageSize = (int)$this->connection->ldapPagingSize > 0 ? (int)$this->connection->ldapPagingSize : 500;
+			$pageSize = (int) $this->connection->ldapPagingSize > 0 ? (int) $this->connection->ldapPagingSize : 500;
 			return [true, $pageSize, $this->lastCookie];
 		}