fix: integration setup status is false even though the oauth setup is complete

Ashim-Stha · web-flow · commit 43adb174584b · 2026-07-01T11:44:00.000+05:45
Signed-off-by: Ashim Shrestha &lt;ashimshrestha2384@gmail.com&gt;
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 setting up with OIDC auth method [#1073](https://github.com/nextcloud/integration_openproject/pull/1073)
 - Fix: Do not create OAuth client while updating OIDC setup [#1077](https://github.com/nextcloud/integration_openproject/pull/1077)
 - Fix: Handle invalid boolean auth-method value [#1079](https://github.com/nextcloud/integration_openproject/pull/1079)
+- Fix: False integration setup status even after complete oauth setup [#1088](https://github.com/nextcloud/integration_openproject/pull/1088)
 
 ## 3.1.0 - 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']) {`