Fix some issues with the ldap data extractor

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
nextcloud · Mar 29, 2022 · 66fdc61 · 66fdc61
1 parent 32ddc1f
commit 66fdc61
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/apps/user_ldap/lib/DataCollector/LdapDataCollector.php b/apps/user_ldap/lib/DataCollector/LdapDataCollector.php
@@ -33,7 +33,7 @@ public function startLdapRequest(string $query, array $args): void {
 			'start' => microtime(true),
 			'query' => $query,
 			'args' => $args,
-			'end' => -1
+			'end' => microtime(true),
 		];
 	}
 

diff --git a/apps/user_ldap/lib/LDAP.php b/apps/user_ldap/lib/LDAP.php
@@ -320,7 +320,7 @@ private function preFunctionCall(string $functionName, array $args): void {
 		$this->curArgs = $args;
 
 		if ($this->dataCollector !== null) {
-			$args = array_reduce($this->curArgs, static function (array $carry, $item): array {
+			$args = array_reduce($this->curArgs, function (array $carry, $item): array {
 				$carry[] = !$this->isResource($item) ? $item : '(resource)';
 				return $carry;
 			}, []);
@@ -329,7 +329,7 @@ private function preFunctionCall(string $functionName, array $args): void {
 		}
 
 		if ($this->logFile !== '' && is_writable(dirname($this->logFile)) && (!file_exists($this->logFile) || is_writable($this->logFile))) {
-			$args = array_reduce($this->curArgs, static function (array $carry, $item): array {
+			$args = array_reduce($this->curArgs, function (array $carry, $item): array {
 				$carry[] = !$this->isResource($item) ? $item : '(resource)';
 				return $carry;
 			}, []);