Merge pull request #59415 from hamidout/fix/user-ldap-last-login-config-key

susnux · web-flow · commit 92e4c882fc04 · 2026-04-03T15:00:44.000+02:00
fix(user_ldap): fix lastLogin reading wrong appid and configkey
diff --git a/apps/user_ldap/lib/User/OfflineUser.php b/apps/user_ldap/lib/User/OfflineUser.php
@@ -164,7 +164,7 @@ protected function fetchDetails(): void {
 		$this->foundDeleted = $this->userConfig->getValueInt($this->ocName, 'user_ldap', 'foundDeleted');
 		$this->extStorageHome = $this->userConfig->getValueString($this->ocName, 'user_ldap', 'extStorageHome');
 		$this->email = $this->userConfig->getValueString($this->ocName, 'user_ldap', 'email');
-		$this->lastLogin = $this->userConfig->getValueInt($this->ocName, 'user_ldap', 'email');
+		$this->lastLogin = $this->userConfig->getValueInt($this->ocName, 'login', 'lastLogin');
 	}
 
 	/**

Original file line number	Diff line number	Diff line change
`@@ -164,7 +164,7 @@ protected function fetchDetails(): void {`
`164`	`164`	`$this->foundDeleted = $this->userConfig->getValueInt($this->ocName, 'user_ldap', 'foundDeleted');`
`165`	`165`	`$this->extStorageHome = $this->userConfig->getValueString($this->ocName, 'user_ldap', 'extStorageHome');`
`166`	`166`	`$this->email = $this->userConfig->getValueString($this->ocName, 'user_ldap', 'email');`
`167`		`- $this->lastLogin = $this->userConfig->getValueInt($this->ocName, 'user_ldap', 'email');`
	`167`	`+ $this->lastLogin = $this->userConfig->getValueInt($this->ocName, 'login', 'lastLogin');`
`168`	`168`	`}`
`169`	`169`
`170`	`170`	`/**`