Skip to content

Commit dcd6228

Browse files
committed
Fix sonar issue
Signed-off-by: Ayoub LABIDI <[email protected]>
1 parent 523b7dc commit dcd6228

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

src/main/java/org/gridsuite/study/server/service/RootNetworkNodeInfoService.java

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -96,15 +96,15 @@ public RootNetworkNodeInfoService(RootNetworkNodeInfoRepository rootNetworkNodeI
9696
public void createRootNetworkLinks(@NonNull UUID studyUuid, @NonNull RootNetworkEntity rootNetworkEntity) {
9797
// For each network modification node (nodeInfoEntity) create a link with the root network
9898
networkModificationNodeInfoRepository.findAllByNodeStudyId(studyUuid).forEach(networkModificationNodeEntity -> {
99-
RootNetworkNodeInfoEntity newRootNetworkNodeInfoEntity = createDefaultEntity(networkModificationNodeEntity.getId());
99+
RootNetworkNodeInfoEntity newRootNetworkNodeInfoEntity = createDefaultEntity();
100100
addLink(networkModificationNodeEntity, rootNetworkEntity, newRootNetworkNodeInfoEntity);
101101
});
102102
}
103103

104104
public void createNodeLinks(@NonNull StudyEntity studyEntity, @NonNull NetworkModificationNodeInfoEntity modificationNodeInfoEntity) {
105105
// For each root network create a link with the node
106106
studyEntity.getRootNetworks().forEach(rootNetworkEntity -> {
107-
RootNetworkNodeInfoEntity newRootNetworkNodeInfoEntity = createDefaultEntity(modificationNodeInfoEntity.getId());
107+
RootNetworkNodeInfoEntity newRootNetworkNodeInfoEntity = createDefaultEntity();
108108
addLink(modificationNodeInfoEntity, rootNetworkEntity, newRootNetworkNodeInfoEntity);
109109
});
110110
}
@@ -115,18 +115,17 @@ public void duplicateNodeLinks(List<RootNetworkNodeInfoEntity> sourceNodeLinks,
115115
// when duplicating a rootNetworkNodeInfoEntity, we need to keep modificationsToExclude
116116
// use correspondence map to use duplicate modification uuids
117117
RootNetworkNodeInfoEntity newRootNetworkNodeInfoEntity = createDefaultEntity(
118-
destinationNodeInfoEntity.getId(),
119118
nodeLink.getModificationsUuidsToExclude().stream().map(originToDuplicateModificationUuidMap::get).collect(Collectors.toSet())
120119
);
121120
addLink(destinationNodeInfoEntity, originToDuplicateRootNetworkMap.get(nodeLink.getRootNetwork()), newRootNetworkNodeInfoEntity);
122121
});
123122
}
124123

125-
private static RootNetworkNodeInfoEntity createDefaultEntity(UUID nodeUuid) {
126-
return createDefaultEntity(nodeUuid, new HashSet<>());
124+
private static RootNetworkNodeInfoEntity createDefaultEntity() {
125+
return createDefaultEntity(new HashSet<>());
127126
}
128127

129-
private static RootNetworkNodeInfoEntity createDefaultEntity(UUID nodeUuid, Set<UUID> modificationsToExclude) {
128+
private static RootNetworkNodeInfoEntity createDefaultEntity(Set<UUID> modificationsToExclude) {
130129
return RootNetworkNodeInfoEntity.builder()
131130
.nodeBuildStatus(NodeBuildStatusEmbeddable.from(BuildStatus.NOT_BUILT))
132131
.variantId(UUID.randomUUID().toString())

0 commit comments

Comments
 (0)