fix: Avoid background fetch if not configured

Signed-off-by: Julius Härtl <[email protected]>
nextcloud · Jun 24, 2024 · a9c36fd · a9c36fd
1 parent 612379f
commit a9c36fd
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/lib/Backgroundjobs/ObtainCapabilities.php b/lib/Backgroundjobs/ObtainCapabilities.php
@@ -6,6 +6,7 @@
 
 namespace OCA\Richdocuments\Backgroundjobs;
 
+use OCA\Richdocuments\AppConfig;
 use OCA\Richdocuments\Service\CapabilitiesService;
 use OCA\Richdocuments\Service\DiscoveryService;
 use OCP\AppFramework\Utility\ITimeFactory;
@@ -18,13 +19,18 @@ public function __construct(
 		private LoggerInterface $logger,
 		private CapabilitiesService $capabilitiesService,
 		private DiscoveryService $discoveryService,
+		private AppConfig $appConfig,
 	) {
 		parent::__construct($time);
 
 		$this->setInterval(60 * 60);
 	}
 
 	protected function run($argument) {
+		if (!$this->appConfig->getCollaboraUrlInternal()) {
+			return;
+		}
+
 		try {
 			$this->capabilitiesService->fetch();
 		} catch (\Exception $e) {

diff --git a/lib/Service/ConnectivityService.php b/lib/Service/ConnectivityService.php
@@ -38,7 +38,7 @@ public function testDiscovery(OutputInterface $output): void {
 
 	public function testCapabilities(OutputInterface $output): void {
 		$this->capabilitiesService->resetCache();
-		$this->capabilitiesService->fetch(true);
+		$this->capabilitiesService->fetch();
 		$output->writeln('<info>✓ Fetched /hosting/capabilities endpoint</info>');
 
 		if ($this->capabilitiesService->getCapabilities() === []) {