Revert "initial implementation for excluding diffs with only comment changes"

This reverts commit f237cff.

Author: pouryafard75

src/main/java/gui/webdiff/dir/DirectoryDiffView.java

@@ -206,9 +206,9 @@ else if(isMovedCode(nodeInfo)) {
                             }
                         }
                     }
-                    ASTDiff astDiff = comparator.getASTDiff(nodeInfo.getId());
-                    boolean empty = astDiff.isEmpty();
-                    if(!empty && astDiff.changesOtherThanCommentUpdates()) {
+                    boolean empty = comparator.getASTDiff(nodeInfo.getId()).isEmpty();
+                    if(!empty) {
+                        ASTDiff astDiff = comparator.getASTDiff(nodeInfo.getId());
                         String datatype;
                         String datapath;
                         if (astDiff.getSrcPath().equals(astDiff.getDstPath())) {

src/main/java/gui/webdiff/viewers/spv/AbstractSinglePageView.java

@@ -4,8 +4,6 @@
 import gui.webdiff.viewers.monaco.MonacoCore;
 import gui.webdiff.dir.DirComparator;
 import gui.webdiff.dir.DirectoryDiffView;
-
-import org.refactoringminer.astDiff.models.ASTDiff;
 import org.refactoringminer.astDiff.models.DiffMetaInfo;
 import org.rendersnake.DocType;
 import org.rendersnake.HtmlCanvas;
@@ -44,10 +42,9 @@ public void renderOn(HtmlCanvas html) {
             // Generate panels for /monaco-0 to /monaco-n
             for (int i = 0; i < n; i++) {
                 Pair<String, String> fileContentsPair = comparator.getFileContentsPair(i);
-                ASTDiff astDiff = comparator.getASTDiff(i);
-				boolean empty = astDiff.isEmpty();
-                if(!empty && astDiff.changesOtherThanCommentUpdates()) {
-                    MonacoCore core = new MonacoCore(comparator, astDiff, i, comparator.isMoveDiff(i), fileContentsPair.first, fileContentsPair.second, comparator.getRefactorings(), metaInfo.getComments());
+                boolean empty = comparator.getASTDiff(i).isEmpty();
+                if(!empty) {
+                    MonacoCore core = new MonacoCore(comparator, comparator.getASTDiff(i), i, comparator.isMoveDiff(i), fileContentsPair.first, fileContentsPair.second, comparator.getRefactorings(), metaInfo.getComments());
                     core.setShowFilenames(false);
                     html.div(class_("card"))
                             .div(class_("card-header").id("heading-" + i).style("padding-right: 0;"))

src/main/java/org/refactoringminer/astDiff/models/ASTDiff.java

@@ -2,13 +2,11 @@
 
 import com.github.gumtreediff.actions.*;
 import com.github.gumtreediff.actions.model.Action;
-import com.github.gumtreediff.actions.model.Update;
 import com.github.gumtreediff.matchers.MappingStore;
 import com.github.gumtreediff.tree.Tree;
 import com.github.gumtreediff.tree.TreeContext;
 import org.refactoringminer.astDiff.actions.classifier.ExtendedOnlyRootsClassifier;
 import org.refactoringminer.astDiff.actions.editscript.ExtendedEditScriptGenerator;
-import org.refactoringminer.astDiff.actions.model.MultiMove;
 
 import java.util.Map;
 import java.util.Objects;
@@ -125,24 +123,4 @@ public boolean equals(Object obj) {
 		ASTDiff other = (ASTDiff) obj;
 		return Objects.equals(dstPath, other.dstPath) && Objects.equals(srcPath, other.srcPath);
 	}
-
-	public boolean changesOtherThanCommentUpdates() {
-        for(com.github.gumtreediff.actions.model.Action action : this.editScript.asList()) {
-        	boolean validMultiMove = false;
-        	if(action instanceof MultiMove) {
-        		validMultiMove = action.getNode().getType().name.equals("TextElement") ||
-        				action.getNode().getType().name.equals("LineComment");
-        	}
-        	boolean validUpdate = false;
-        	if(action instanceof Update) {
-        		validUpdate = action.getNode().getType().name.equals("TextElement") ||
-        				action.getNode().getType().name.equals("LineComment") ||
-        				action.getNode().getType().name.equals("BlockComment");
-        	}
-        	if(!validUpdate && !validMultiMove) {
-        		return true;
-        	}
-        }
-        return false;
-	}
 }