02-September-2018 - 6.7.0

The PMD team is pleased to announce PMD 6.7.0.

This is a minor release.

Table Of Contents

• New and noteworthy
  • Modified Rules
  • New Rules
  • Deprecated Rules
• Fixed Issues
• API Changes
• External Contributions

New and noteworthy

Modified Rules

• The Java rule OneDeclarationPerLine (java-bestpractices) has been revamped to consider not only local variable declarations, but field declarations too.

New Rules

• The new Java rule LinguisticNaming (java-codestyle) detects cases, when a method name indicates it returns a boolean (such as isSmall()) but it doesn't. Besides method names, the rule also checks field and variable names. It also checks, that getters return something but setters won't. The rule has several properties with which it can be customized.

• The new PL/SQL rule ForLoopNaming (plsql-codestyle) enforces a naming convention for "for loops". Both "cursor for loops" and "index for loops" are covered. The rule can be customized via patterns. By default, short variable names are reported.

• The new Java rule FieldNamingConventions (java-codestyle) detects field names that don't comply to a given convention. It defaults to standard Java convention of using camelCase, but can be configured with ease for e.g. constants or static fields.

• The new Apex rule OneDeclarationPerLine (apex-codestyle) enforces declaring a single field / variable per line; or per statement if the strictMode property is set. It's an Apex equivalent of the already existing Java rule of the same name.

Deprecated Rules

• The Java rules VariableNamingConventions, MIsLeadingVariableName, SuspiciousConstantFieldName, and AvoidPrefixingMethodParameters are now deprecated, and will be removed with version 7.0.0. They are replaced by the more general FieldNamingConventions, FormalParameterNamingConventions, and LocalVariableNamingConventions.

Fixed Issues

• core
  • #1191: [core] Test Framework: Sort violations by line/column
  • #1283: [core] Deprecate ReportTree
  • #1288: [core] No supported build listeners found with Gradle
  • #1300: [core] PMD stops processing file completely, if one rule in a rule chain fails
  • #1317: [ci] Coveralls hasn't built the project since June 25th
• java-bestpractices
  • #940: [java] JUnit 4 false positives for JUnit 5 tests
  • #1267: [java] MissingOverrideRule: Avoid NoClassDefFoundError with incomplete classpath
  • #1323: [java] AvoidUsingHardCodedIP ignores match pattern
  • #1327: [java] AvoidUsingHardCodedIP false positive for ":bee"
• java-codestyle
  • #1255: [java] UnnecessaryFullyQualifiedName false positive: static method on shadowed implicitly imported class
  • #1258: [java] False positive "UselessParentheses" for parentheses that contain assignment
• java-errorprone
  • #1078: [java] MissingSerialVersionUID rule does not seem to catch inherited classes
• java-performance
  • #1291: [java] InvalidSlf4jMessageFormat false positive: too many arguments with string concatenation operator
  • #1298: [java] RedundantFieldInitializer - NumberFormatException with Long
• jsp
  • #1274: [jsp] Support EL in tag attributes
  • #1276: [jsp] add support for jspf and tag extensions
• plsql
  • #681: [plsql] Parse error with Cursor For Loop

API Changes

• All classes in the package net.sourceforge.pmd.lang.dfa.report have been deprecated and will be removed with PMD 7.0.0. This includes the class net.sourceforge.pmd.lang.dfa.report.ReportTree. The reason is, that this class is very specific to Java and not suitable for other languages. It has only been used for YAHTMLRenderer, which has been rewritten to work without these classes.

• The nodes RUNSIGNEDSHIFT and RSIGNEDSHIFT are deprecated and will be removed from the AST with PMD 7.0.0. These represented the operator of ShiftExpression in two cases out of three, but they're not needed and make ShiftExpression inconsistent. The operator of a ShiftExpression is now accessible through ShiftExpression#getOperator.

External Contributions

• #109: [java] Add two linguistics rules under naming - Arda Aslan
• #1254: [ci] [GSoC] Integrating the danger and pmdtester to travis CI - BBG
• #1258: [java] Use typeof in MissingSerialVersionUID - krichter722
• #1264: [cpp] Fix NullPointerException in CPPTokenizer:99 - Rafael Cortês
• #1277: [jsp] [#1276] add support for jspf and tag extensions - Jordi Llach
• #1275: [jsp] Issue [#1274] - Support EL in tag attributes - Jordi Llach
• #1278: [ci] [GSoC] Use pmdtester 1.0.0.pre.beta3 - BBG
• #1289: [java] UselessParentheses: Fix false positive with assignments - cobratbq
• #1290: [docs] [GSoC] Create the documentation about pmdtester - BBG
• #1256: [java] [#940] Avoid JUnit 4 false positives for JUnit 5 tests - Alex Shesterov
• #1315: [apex] Add OneDeclarationPerStatement rule - Jeff Hube