Generalize PurityChecker APIs

tsantalis · Dec 15, 2024 · a4d21b3 · a4d21b3
1 parent af0be77
commit a4d21b3
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/src/main/java/org/refactoringminer/api/PurityChecker.java b/src/main/java/org/refactoringminer/api/PurityChecker.java
@@ -2532,8 +2532,8 @@ private static PurityCheckResult detectMoveMethodPurity(MoveOperationRefactoring
     }
 
     private static void checkForParameterizeVariableOnTop(Refactoring refactoring, List<Refactoring> refactorings, List<AbstractCodeFragment> nonMappedLeavesT1) {
-        UMLOperation originalOperation = null;
-        UMLOperation movedOperation = null;
+        VariableDeclarationContainer originalOperation = null;
+        VariableDeclarationContainer movedOperation = null;
 
         if (refactoring.getRefactoringType().equals(RefactoringType.MOVE_OPERATION)  || refactoring.getRefactoringType().equals(RefactoringType.MOVE_AND_RENAME_OPERATION)) {
             originalOperation = ((MoveOperationRefactoring) (refactoring)).getOriginalOperation();
@@ -2567,8 +2567,8 @@ private static void checkForParameterizeVariableOnTop(Refactoring refactoring, L
     }
 
     private static void checkForLocalizeParameterOnTop(Refactoring refactoring, List<Refactoring> refactorings, List<AbstractCodeFragment> nonMappedLeavesT2) {
-        UMLOperation originalOperation = null;
-        UMLOperation movedOperation = null;
+        VariableDeclarationContainer originalOperation = null;
+        VariableDeclarationContainer movedOperation = null;
 
         if (refactoring.getRefactoringType().equals(RefactoringType.MOVE_OPERATION)  || refactoring.getRefactoringType().equals(RefactoringType.MOVE_AND_RENAME_OPERATION)) {
             originalOperation = ((MoveOperationRefactoring) (refactoring)).getOriginalOperation();
@@ -2696,8 +2696,8 @@ private static void omitMoveMethodRelatedReplacements(Refactoring refactoring, L
             int foundInBefore = before.indexOf(after);
             List<String> classBefore = new ArrayList<>();
             List<String> classAfter = new ArrayList<>();
-            UMLOperation originalOperation = null;
-            UMLOperation refactoredOperation = null;
+            VariableDeclarationContainer originalOperation = null;
+            VariableDeclarationContainer refactoredOperation = null;
 
             if (refactoring.getRefactoringType().equals(RefactoringType.MOVE_OPERATION) || refactoring.getRefactoringType().equals(RefactoringType.MOVE_AND_RENAME_OPERATION)) {
                 String classBeforeString = ((MoveOperationRefactoring) (refactoring)).getOriginalOperation().getNonQualifiedClassName();
@@ -2810,7 +2810,7 @@ private static void omitMoveMethodRelatedReplacements(Refactoring refactoring, L
         replacementsToCheck.removeAll(replacementsToRemove);
     }
 
-    private static boolean relaxSearch(UMLOperation movedOperation, List<Refactoring> refactorings, String instanceOrVariable, String method, UMLModelDiff modelDiff) {
+    private static boolean relaxSearch(VariableDeclarationContainer movedOperation, List<Refactoring> refactorings, String instanceOrVariable, String method, UMLModelDiff modelDiff) {
         for (Refactoring refactoring : refactorings) {
             if (isMethodRelatedRefactoring(refactoring)) {
                 if (refactoring instanceof AddMethodAnnotationRefactoring) {
@@ -2842,7 +2842,7 @@ private static boolean isMethodRelatedRefactoring(Refactoring refactoring) {
         return false;
     }
 
-    private static boolean searchInVariableDeclarations(UMLOperation operation, String instanceOrVariable, List<String> className, UMLModelDiff umlModelDiff) {
+    private static boolean searchInVariableDeclarations(VariableDeclarationContainer operation, String instanceOrVariable, List<String> className, UMLModelDiff umlModelDiff) {
         // This iteration also covers the parameters list
         for (VariableDeclaration variableDeclaration : operation.getAllVariableDeclarations()) {
             if (variableDeclaration.getVariableName().equals(instanceOrVariable)) {
@@ -2913,7 +2913,7 @@ private static boolean specificCheckForPullUpMethodOnTopFoundInBefore(Refactorin
             return true;
         }
 
-        UMLOperation originalOperation = null;
+        VariableDeclarationContainer originalOperation = null;
 
         if (refactoring instanceof MoveOperationRefactoring) {
             originalOperation = ((MoveOperationRefactoring) refactoring).getOriginalOperation();