Requested changes

Signed-off-by: Ayoub LABIDI <[email protected]>

Author: ayolab

network-area-diagram/src/main/java/com/powsybl/nad/svg/iidm/TopologicalStyleProvider.java

@@ -22,24 +22,24 @@
  */
 public class TopologicalStyleProvider extends AbstractVoltageStyleProvider {
 
-    private final Map<String, String> violationStyles;
+    private Map<String, String> limitViolationStyles = Map.of();
 
     public TopologicalStyleProvider(Network network) {
-        this(network, Collections.emptyMap());
+        super(network);
     }
 
     public TopologicalStyleProvider(Network network, BaseVoltagesConfig baseVoltageStyle) {
-        this(network, baseVoltageStyle, Collections.emptyMap());
+        super(network, baseVoltageStyle);
     }
 
-    public TopologicalStyleProvider(Network network, Map<String, String> violationStyles) {
+    public TopologicalStyleProvider(Network network, Map<String, String> limitViolationStyles) {
         super(network);
-        this.violationStyles = violationStyles != null ? violationStyles : Collections.emptyMap();
+        this.limitViolationStyles = limitViolationStyles;
     }
 
-    public TopologicalStyleProvider(Network network, BaseVoltagesConfig baseVoltageStyle, Map<String, String> violationStyles) {
+    public TopologicalStyleProvider(Network network, BaseVoltagesConfig baseVoltageStyle, Map<String, String> limitViolationStyles) {
         super(network, baseVoltageStyle);
-        this.violationStyles = violationStyles != null ? violationStyles : Collections.emptyMap();
+        this.limitViolationStyles = limitViolationStyles;
     }
 
     @Override
@@ -50,8 +50,8 @@ public List<String> getCssFilenames() {
     @Override
     public List<String> getBranchEdgeStyleClasses(BranchEdge branchEdge) {
         // Check custom violations first
-        if (!violationStyles.isEmpty()) {
-            String customStyle = violationStyles.get(branchEdge.getEquipmentId());
+        if (!limitViolationStyles.isEmpty()) {
+            String customStyle = limitViolationStyles.get(branchEdge.getEquipmentId());
             if (customStyle != null && !customStyle.isBlank()) {
                 return List.of(customStyle);
             }

single-line-diagram/single-line-diagram-core/src/main/java/com/powsybl/sld/svg/styles/iidm/LimitHighlightStyleProvider.java

@@ -22,21 +22,21 @@
  */
 public class LimitHighlightStyleProvider extends EmptyStyleProvider {
     private final Network network;
-    private final Map<String, String> violationStyles;
+    private Map<String, String> limitViolationStyles = Map.of();
 
     public LimitHighlightStyleProvider(Network network) {
         this(network, Collections.emptyMap());
     }
 
-    public LimitHighlightStyleProvider(Network network, Map<String, String> violationStyles) {
+    public LimitHighlightStyleProvider(Network network, Map<String, String> limitViolationStyles) {
         this.network = network;
-        this.violationStyles = violationStyles != null ? violationStyles : Collections.emptyMap();
+        this.limitViolationStyles = limitViolationStyles;
     }
 
     @Override
     public List<String> getEdgeStyles(Graph graph, Edge edge) {
         // Check custom violations first
-        if (!violationStyles.isEmpty()) {
+        if (!limitViolationStyles.isEmpty()) {
             Optional<String> customStyle = getCustomViolationStyle(edge);
             if (customStyle.isPresent()) {
                 return List.of(customStyle.get());
@@ -50,7 +50,7 @@ public List<String> getEdgeStyles(Graph graph, Edge edge) {
     private Optional<String> getCustomViolationStyle(Edge edge) {
         for (Node node : edge.getNodes()) {
             if (node instanceof FeederNode feederNode) {
-                String style = violationStyles.get(feederNode.getEquipmentId());
+                String style = limitViolationStyles.get(feederNode.getEquipmentId());
                 if (style != null && !style.isBlank()) {
                     return Optional.of(style);
                 }