Java 7 refactorings can only be applied to Java 7+ projects

Author: JnRouvignac

plugin/src/main/java/org/autorefactor/refactoring/rules/TryWithResourceRefactoring.java

@@ -27,6 +27,7 @@
 
 import java.util.List;
 
+import org.autorefactor.preferences.Preferences;
 import org.autorefactor.refactoring.ASTBuilder;
 import org.autorefactor.refactoring.Refactorings;
 import org.eclipse.jdt.core.dom.Assignment;
@@ -47,14 +48,21 @@ public class TryWithResourceRefactoring extends AbstractRefactoringRule {
 
     @Override
     public String getDescription() {
-        return "";
+        return ""
+            + "Changes code to make use of Java 7 try-with-resources feature. "
+            + "In particular, it removes now useless finally clauses.";
     }
 
     @Override
     public String getName() {
         return "Use try-with-resource";
     }
 
+    @Override
+    public boolean isEnabled(Preferences preferences) {
+        return ctx.getJavaProjectOptions().getJavaSERelease().getMinorVersion() >= 7;
+    }
+
     @Override
     public boolean visit(TryStatement node) {
         final List<Statement> tryStmts = asList(node.getBody());

plugin/src/main/java/org/autorefactor/refactoring/rules/UseDiamondOperatorRefactoring.java

@@ -29,6 +29,7 @@
 import java.util.Collections;
 import java.util.List;
 
+import org.autorefactor.preferences.Preferences;
 import org.autorefactor.refactoring.Release;
 import org.eclipse.jdt.core.dom.ASTNode;
 import org.eclipse.jdt.core.dom.Assignment;
@@ -47,7 +48,6 @@
 import static org.eclipse.jdt.core.dom.ASTNode.*;
 
 /** See {@link #getDescription()} method. */
-@SuppressWarnings("javadoc")
 public class UseDiamondOperatorRefactoring extends AbstractRefactoringRule {
 
     @Override
@@ -60,16 +60,19 @@ public String getName() {
         return "Diamond operator";
     }
 
+    @Override
+    public boolean isEnabled(Preferences preferences) {
+        return ctx.getJavaProjectOptions().getJavaSERelease().isCompatibleWith(Release.javaSE("1.7.0"));
+    }
+
     @Override
     public boolean visit(ClassInstanceCreation node) {
-        if (this.ctx.getJavaProjectOptions().getJavaSERelease().isCompatibleWith(Release.javaSE("1.7.0"))) {
-            final Type type = node.getType();
-            if (type.isParameterizedType()
-                    && node.getAnonymousClassDeclaration() == null
-                    && parentAllowsDiamondOperator(node)
-                    && canUseDiamondOperator(node, type)) {
-                return removeAllTypeArguments((ParameterizedType) type);
-            }
+        final Type type = node.getType();
+        if (type.isParameterizedType()
+                && node.getAnonymousClassDeclaration() == null
+                && parentAllowsDiamondOperator(node)
+                && canUseDiamondOperator(node, type)) {
+            return removeAllTypeArguments((ParameterizedType) type);
         }
         return VISIT_SUBTREE;
     }

plugin/src/main/java/org/autorefactor/refactoring/rules/UseMultiCatchRefactoring.java

@@ -36,6 +36,7 @@
 import java.util.Map.Entry;
 import java.util.Set;
 
+import org.autorefactor.preferences.Preferences;
 import org.autorefactor.refactoring.ASTBuilder;
 import org.autorefactor.refactoring.ASTHelper;
 import org.autorefactor.refactoring.Refactorings;
@@ -61,7 +62,6 @@
 import static org.eclipse.jdt.core.dom.ASTNode.*;
 
 /** See {@link #getDescription()} method. */
-@SuppressWarnings("javadoc")
 public class UseMultiCatchRefactoring extends AbstractRefactoringRule {
 
     @Override
@@ -74,6 +74,11 @@ public String getName() {
         return "Multi-catch";
     }
 
+    @Override
+    public boolean isEnabled(Preferences preferences) {
+        return ctx.getJavaProjectOptions().getJavaSERelease().getMinorVersion() >= 7;
+    }
+
     private static enum MergeDirection {
         NONE, UP, DOWN;
     }