Avoid removing leaf, if the refactoring is already detected

tsantalis · Sep 27, 2024 · 50c3705 · 50c3705
1 parent 29e2156
commit 50c3705
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/src/main/java/gr/uom/java/xmi/decomposition/UMLOperationBodyMapper.java b/src/main/java/gr/uom/java/xmi/decomposition/UMLOperationBodyMapper.java
@@ -908,8 +908,10 @@ else if(mapping instanceof CompositeStatementObjectMapping) {
 										LeafMapping leafMapping = new LeafMapping(subExpression, initializer, operation1, operation2);
 										ref.addSubExpressionMapping(leafMapping);
 									}
-									refactorings.add(ref);
-									leavesToBeRemovedT2.add(statement);
+									if(!refactorings.contains(ref)) {
+										refactorings.add(ref);
+										leavesToBeRemovedT2.add(statement);
+									}
 								}
 							}
 						}
@@ -956,8 +958,10 @@ else if(mapping instanceof CompositeStatementObjectMapping) {
 										LeafMapping leafMapping = new LeafMapping(initializer, subExpression, operation1, operation2);
 										ref.addSubExpressionMapping(leafMapping);
 									}
-									refactorings.add(ref);
-									leavesToBeRemovedT1.add(statement);
+									if(!refactorings.contains(ref)) {
+										refactorings.add(ref);
+										leavesToBeRemovedT1.add(statement);
+									}
 								}
 							}
 						}