Merge pull request #51416 from nextcloud/backport/51384/stable30

skjnldsv · web-flow · commit 4860967c8ac4 · 2025-03-12T11:14:32.000+01:00
[stable30] fix(lookup-server): Only flag new users for lookup update / delete
diff --git a/core/BackgroundJobs/LookupServerSendCheckBackgroundJob.php b/core/BackgroundJobs/LookupServerSendCheckBackgroundJob.php
@@ -28,7 +28,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
`@@ -28,7 +28,11 @@ public function __construct(`
`28`	`28`	`*/`
`29`	`29`	`public function run($argument): void {`
`30`	`30`	`$this->userManager->callForSeenUsers(function (IUser $user) {`
`31`		`- $this->config->setUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '1');`
	`31`	`+ // If the user data was not updated yet (check if LUS is enabled and if then update on LUS or delete on LUS)`
	`32`	`+ // then we need to flag the user data to be checked`
	`33`	`+ if ($this->config->getUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '') === '') {`
	`34`	`+ $this->config->setUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '1');`
	`35`	`+ }`
`32`	`36`	`});`
`33`	`37`	`}`
`34`	`38`	`}`