Merge pull request #51415 from nextcloud/backport/51384/stable29

skjnldsv · web-flow · commit 241fcb0f32ee · 2025-03-12T11:14:47.000+01:00
[stable29] fix(lookup-server): Only flag new users for lookup update / delete
diff --git a/core/BackgroundJobs/LookupServerSendCheckBackgroundJob.php b/core/BackgroundJobs/LookupServerSendCheckBackgroundJob.php
@@ -45,7 +45,11 @@ public function __construct(
 	 */
 	public function run($argument): void {
 		$this->userManager->callForSeenUsers(function (IUser $user) {
-			$this->config->setUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '1');
+			// If the user data was not updated yet (check if LUS is enabled and if then update on LUS or delete on LUS)
+			// then we need to flag the user data to be checked
+			if ($this->config->getUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '') === '') {
+				$this->config->setUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '1');
+			}
 		});
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,11 @@ public function __construct(`
`45`	`45`	`*/`
`46`	`46`	`public function run($argument): void {`
`47`	`47`	`$this->userManager->callForSeenUsers(function (IUser $user) {`
`48`		`- $this->config->setUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '1');`
	`48`	`+ // If the user data was not updated yet (check if LUS is enabled and if then update on LUS or delete on LUS)`
	`49`	`+ // then we need to flag the user data to be checked`
	`50`	`+ if ($this->config->getUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '') === '') {`
	`51`	`+ $this->config->setUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '1');`
	`52`	`+ }`
`49`	`53`	`});`
`50`	`54`	`}`
`51`	`55`	`}`