diff --git a/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/bestpractices/LooseCouplingRule.java b/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/bestpractices/LooseCouplingRule.java index 38eb211aa9..953eb486ea 100644 --- a/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/bestpractices/LooseCouplingRule.java +++ b/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/bestpractices/LooseCouplingRule.java @@ -29,6 +29,10 @@ public class LooseCouplingRule extends AbstractJavaRule { // "java.util.TreeMap", "java.util.Vector" // }); + public LooseCouplingRule() { + addRuleChainVisit(ASTClassOrInterfaceType.class); + } + @Override public Object visit(ASTClassOrInterfaceType node, Object data) { if (methodHasOverride(node)) { diff --git a/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/bestpractices/xml/LooseCoupling.xml b/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/bestpractices/xml/LooseCoupling.xml index 2434052cc6..5e2a190e8c 100644 --- a/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/bestpractices/xml/LooseCoupling.xml +++ b/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/bestpractices/xml/LooseCoupling.xml @@ -7,6 +7,7 @@ returning a HashSet, bad 1 + 3 + + + + Verify with nested (local) classes + 3 + 6,11,15 +