fix: app disable listener now deletes sources for all users

- introduce a call to /deleteSourcesByProviderForAllUsers

Signed-off-by: Anupam Kumar <[email protected]>

lib/Listener/AppDisableListener.php

@@ -28,7 +28,6 @@ public function __construct(
 		private ProviderConfigService $configService,
 		private LangRopeService $service,
 		private LoggerInterface $logger,
-		private ?string $userId,
 	) {
 	}
 
@@ -37,11 +36,6 @@ public function handle(Event $event): void {
 			return;
 		}
 
-		if ($this->userId === null) {
-			$this->logger->warning('No user id provided for app disable');
-			return;
-		}
-
 		foreach ($this->configService->getProviders() as $key => $values) {
 			/** @var string[] */
 			$identifierValues = explode('__', $key, 2);
@@ -58,7 +52,7 @@ public function handle(Event $event): void {
 			}
 
 			$this->configService->removeProvider($appId, $providerId);
-			$this->service->deleteSourcesByProvider($this->userId, $key);
+			$this->service->deleteSourcesByProviderForAllUsers($providerId);
 		}
 	}
 }

lib/Service/LangRopeService.php

@@ -131,6 +131,18 @@ public function deleteSourcesByProvider(string $userId, string $providerKey): vo
 		$this->requestToExApp('/deleteSourcesByProvider', 'POST', $params);
 	}
 
+	/**
+	 * @param string $providerKey
+	 * @return void
+	 */
+	public function deleteSourcesByProviderForAllUsers(string $providerKey): void {
+		$params = [
+			'providerKey' => $providerKey,
+		];
+
+		$this->requestToExApp('/deleteSourcesByProviderForAllUsers', 'POST', $params);
+	}
+
 	/**
 	 * @param string $userId
 	 * @param string[] $sourceNames