Skip to content

Commit

Permalink
refactor: Apply getter at field level
Browse files Browse the repository at this point in the history
  • Loading branch information
abhvsn committed Apr 29, 2024
1 parent ec3d10f commit bc9836e
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -596,7 +596,7 @@ private Mono<Boolean> checkPermissionsForForking(
.findIdsAndPoliciesByApplicationIdIn(List.of(application.getId()))
.map(idPoliciesOnly -> {
NewPage newPage = new NewPage();
newPage.setId(idPoliciesOnly.getId());
newPage.setId(idPoliciesOnly.id());
newPage.setPolicies(idPoliciesOnly.getPolicies());
return newPage;
})
Expand All @@ -606,7 +606,7 @@ private Mono<Boolean> checkPermissionsForForking(
.findIdsAndPoliciesByApplicationIdIn(List.of(application.getId()))
.map(idPoliciesOnly -> {
NewAction newAction = new NewAction();
newAction.setId(idPoliciesOnly.getId());
newAction.setId(idPoliciesOnly.id());
newAction.setPolicies(idPoliciesOnly.getPolicies());
return newAction;
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,4 @@

import java.util.Set;

@Getter
public record IdPoliciesOnly(@NonNull String id, Set<Policy> policies) {}
public record IdPoliciesOnly(@NonNull String id, @Getter Set<Policy> policies) {}
Original file line number Diff line number Diff line change
Expand Up @@ -1470,7 +1470,7 @@ private Mono<Boolean> validateAllObjectsForPermissions(
.findIdsAndPoliciesByApplicationIdIn(List.of(application.getId()))
.map(idPoliciesOnly -> {
NewPage newPage = new NewPage();
newPage.setId(idPoliciesOnly.getId());
newPage.setId(idPoliciesOnly.id());
newPage.setPolicies(idPoliciesOnly.getPolicies());
return newPage;
})
Expand All @@ -1479,7 +1479,7 @@ private Mono<Boolean> validateAllObjectsForPermissions(
.findIdsAndPoliciesByApplicationIdIn(List.of(application.getId()))
.map(idPoliciesOnly -> {
NewAction newAction = new NewAction();
newAction.setId(idPoliciesOnly.getId());
newAction.setId(idPoliciesOnly.id());
newAction.setPolicies(idPoliciesOnly.getPolicies());
return newAction;
})
Expand All @@ -1488,7 +1488,7 @@ private Mono<Boolean> validateAllObjectsForPermissions(
.findIdsAndPoliciesByApplicationIdIn(List.of(application.getId()))
.map(idPoliciesOnly -> {
ActionCollection actionCollection = new ActionCollection();
actionCollection.setId(idPoliciesOnly.getId());
actionCollection.setId(idPoliciesOnly.id());
actionCollection.setPolicies(idPoliciesOnly.getPolicies());
return actionCollection;
})
Expand Down Expand Up @@ -1538,7 +1538,7 @@ private Mono<Boolean> validateDatasourcesForCreatePermission(Mono<Application> a
.findIdsAndPoliciesByIdIn(datasourceIds)
.flatMap(idPolicy -> {
Datasource datasource = new Datasource();
datasource.setId(idPolicy.getId());
datasource.setId(idPolicy.id());
datasource.setPolicies(idPolicy.getPolicies());
return datasourceRepository.setUserPermissionsInObject(datasource);
}));
Expand Down

0 comments on commit bc9836e

Please sign in to comment.