• Also, ImportDeclaration is not a TypeNode anymore, for the reasons described in the comment I'm deleting on the class
  • The two issues are dependent. Name is a TypeNode so making ImportDeclaration not a TypeNode strongly suggests the removal of the Name. It was clutter anyway to have a nested node just to carry a string.
• Also, this removes getPackage on ImportDeclaration. For the same reasons ImportDeclaration shouldn't be a TypeNode, getPackage is hard to define (it was defined as the Package of the type of ImportDeclaration). Besides, it was not used even once in our codebase and I think it's utterly useless. If you have the Class then you have the Package. This should be deprecated on master
• Fixing compilation required editing a few rules. They use the ClassTypeResolver of the CompilationUnit, which is only a temporary crutch. Those two rules (UnnecessaryQualifiedName and UnusedImport) will anyway probably be completely rewritten to make use of the new symbol table, instead of doing their thing in isolation.

See https://github.com/pmd/pmd/wiki/Java_clean_changes#import-and-package-declarations