Skip to content

Enable pre profile update action to allow modifying user claims at profile update#8124

Open
Lashen1227 wants to merge 14 commits into
wso2:feature/pre-update-profilefrom
Lashen1227:feat/pre-profile-update-claims
Open

Enable pre profile update action to allow modifying user claims at profile update#8124
Lashen1227 wants to merge 14 commits into
wso2:feature/pre-update-profilefrom
Lashen1227:feat/pre-profile-update-claims

test: add unit test for PreUpdateProfileResponseProcessorTest and Pre…

1a17801
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded May 27, 2026 in 2m 15s

Quality Gate passed

Issues
19 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 1093 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/test/java/org/wso2/carbon/identity/user/pre/update/profile/action/execution/PreUpdateProfileResponseProcessorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Immediately return this expression instead of assigning it to the temporary variable "scimClaim". 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5qjnAbTgFurUGs3V_p&open=AZ5qjnAbTgFurUGs3V_p&pullRequest=8115

Check warning on line 228 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof String string' 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqs&open=AZ5ioOs3xRHZv9iNmJqs&pullRequest=8115

Check warning on line 639 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof String string' 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq3&open=AZ5ioOs3xRHZv9iNmJq3&pullRequest=8115

Check failure on line 251 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileRequestBuilder.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 23 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOpHxRHZv9iNmJqo&open=AZ5ioOpHxRHZv9iNmJqo&pullRequest=8115

Check warning on line 380 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Method has 9 parameters, which is greater than 7 authorized. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqq&open=AZ5ioOs3xRHZv9iNmJqq&pullRequest=8115

Check warning on line 565 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq0&open=AZ5ioOs3xRHZv9iNmJq0&pullRequest=8115

Check failure on line 147 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Operation applied." 3 times. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5kdVy8PRoj9m7Q-IqF&open=AZ5kdVy8PRoj9m7Q-IqF&pullRequest=8115

Check warning on line 70 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileRequestBuilder.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor your code to get this URI from a customizable parameter. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOpHxRHZv9iNmJqp&open=AZ5ioOpHxRHZv9iNmJqp&pullRequest=8115

Check warning on line 633 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof String string' 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq2&open=AZ5ioOs3xRHZv9iNmJq2&pullRequest=8115

Check warning on line 440 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqv&open=AZ5ioOs3xRHZv9iNmJqv&pullRequest=8115

Check warning on line 335 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused "initiatorType" local variable. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5n679f22l2yJtoC-HX&open=AZ5n679f22l2yJtoC-HX&pullRequest=8115

Check warning on line 79 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor your code to get this URI from a customizable parameter. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq5&open=AZ5ioOs3xRHZv9iNmJq5&pullRequest=8115

Check failure on line 380 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqu&open=AZ5ioOs3xRHZv9iNmJqu&pullRequest=8115

Check warning on line 696 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq4&open=AZ5ioOs3xRHZv9iNmJq4&pullRequest=8115

Check warning on line 740 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Catch Exception instead of Throwable. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5kdVy8PRoj9m7Q-IqG&open=AZ5kdVy8PRoj9m7Q-IqG&pullRequest=8115

Check failure on line 594 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 45 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq1&open=AZ5ioOs3xRHZv9iNmJq1&pullRequest=8115

Check warning on line 446 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqw&open=AZ5ioOs3xRHZv9iNmJqw&pullRequest=8115

Check failure on line 189 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 20 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqr&open=AZ5ioOs3xRHZv9iNmJqr&pullRequest=8115

Check warning on line 305 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof String string' 

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqt&open=AZ5ioOs3xRHZv9iNmJqt&pullRequest=8115
View more details on SonarQubeCloud