Merge pull request #1086 from nextcloud/fix/false-setup-status

Ashim-Stha · web-flow · commit 671806e9d861 · 2026-07-01T10:43:36.000+05:45
fix: false `integration setup` status even after complete oauth setup
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -31,6 +31,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 - Fix: Do not create OAuth client while updating OIDC setup [#1075](https://github.com/nextcloud/integration_openproject/pull/1075)
 - Fix: Handle invalid boolean auth-method value [#1074](https://github.com/nextcloud/integration_openproject/pull/1074)
 - Fix: Update OpenProject icon [#1078](https://github.com/nextcloud/integration_openproject/pull/1078)
+- Fix: False integration setup status even after complete oauth setup [#1086](https://github.com/nextcloud/integration_openproject/pull/1086)
 
 ## 2.11.3 - 2026-06-18
 
diff --git a/lib/Controller/ConfigController.php b/lib/Controller/ConfigController.php
@@ -656,6 +656,7 @@ public function setUpIntegration(?array $values): DataResponse {
 
 			if ($values['authorization_method'] === Application::AUTH_METHOD_OAUTH) {
 				$response = \array_merge($response, $this->recreateOauthClientInformation());
+				$response['status'] = OpenProjectAPIService::isAdminConfigOk($this->config);
 			}
 
 			if ($setup['oPOAuthTokenRevokeStatus']) {
@@ -711,6 +712,7 @@ public function updateIntegration(?array $values): DataResponse {
 				$response = \array_merge($response, $this->oauthService->getClientInfo($oauthDbId));
 			} else {
 				$response = \array_merge($response, $this->recreateOauthClientInformation());
+				$response['status'] = OpenProjectAPIService::isAdminConfigOk($this->config);
 			}
 
 			if ($setup['oPOAuthTokenRevokeStatus']) {

Original file line number	Diff line number	Diff line change
`@@ -656,6 +656,7 @@ public function setUpIntegration(?array $values): DataResponse {`
`656`	`656`
`657`	`657`	`if ($values['authorization_method'] === Application::AUTH_METHOD_OAUTH) {`
`658`	`658`	`$response = \array_merge($response, $this->recreateOauthClientInformation());`
	`659`	`+ $response['status'] = OpenProjectAPIService::isAdminConfigOk($this->config);`
`659`	`660`	`}`
`660`	`661`
`661`	`662`	`if ($setup['oPOAuthTokenRevokeStatus']) {`
`@@ -711,6 +712,7 @@ public function updateIntegration(?array $values): DataResponse {`
`711`	`712`	`$response = \array_merge($response, $this->oauthService->getClientInfo($oauthDbId));`
`712`	`713`	`} else {`
`713`	`714`	`$response = \array_merge($response, $this->recreateOauthClientInformation());`
	`715`	`+ $response['status'] = OpenProjectAPIService::isAdminConfigOk($this->config);`
`714`	`716`	`}`
`715`	`717`
`716`	`718`	`if ($setup['oPOAuthTokenRevokeStatus']) {`