diff --git a/pmd-swift/src/main/java/net/sourceforge/pmd/lang/swift/rule/bestpractices/ProhibitedInterfaceBuilderRule.java b/pmd-swift/src/main/java/net/sourceforge/pmd/lang/swift/rule/bestpractices/ProhibitedInterfaceBuilderRule.java index 41cc4226d2..494c92e0a9 100644 --- a/pmd-swift/src/main/java/net/sourceforge/pmd/lang/swift/rule/bestpractices/ProhibitedInterfaceBuilderRule.java +++ b/pmd-swift/src/main/java/net/sourceforge/pmd/lang/swift/rule/bestpractices/ProhibitedInterfaceBuilderRule.java @@ -12,7 +12,7 @@ import net.sourceforge.pmd.lang.swift.antlr4.SwiftParser; import net.sourceforge.pmd.lang.swift.antlr4.SwiftParser.FunctionHeadContext; import net.sourceforge.pmd.lang.swift.antlr4.SwiftParser.VariableDeclarationHeadContext; -public class ProhibitedInterfaceBuilderRule extends AbstractSwiftRule { +public class ProhibitedInterfaceBuilderRule extends AbstractSwiftRule { private static final String IBACTION = "@IBAction"; private static final String IBOUTLET = "@IBOutlet"; @@ -24,24 +24,24 @@ public class ProhibitedInterfaceBuilderRule extends AbstractSwiftRule { } @Override - public Boolean visitFunctionHead(FunctionHeadContext ctx) { + public Void visitFunctionHead(FunctionHeadContext ctx) { if (ctx == null || ctx.attributes() == null) { - return false; + return null; } return visitDeclarationHead(ctx, ctx.attributes().attribute(), IBACTION); } @Override - public Boolean visitVariableDeclarationHead(final VariableDeclarationHeadContext ctx) { + public Void visitVariableDeclarationHead(final VariableDeclarationHeadContext ctx) { if (ctx == null || ctx.attributes() == null) { - return false; + return null; } return visitDeclarationHead(ctx, ctx.attributes().attribute(), IBOUTLET); } - private boolean visitDeclarationHead(final Node node, final List attributes, + private Void visitDeclarationHead(final Node node, final List attributes, final String match) { final boolean violate = attributes.stream().anyMatch(atr -> match.equals(atr.getText())); @@ -49,6 +49,6 @@ public class ProhibitedInterfaceBuilderRule extends AbstractSwiftRule { addViolation(data, node); } - return violate; + return null; } }