Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
ThreadLocalNotStaticFinal | 2 | 22 | [SRC]private final ThreadLocal<Boolean> configuring = new Thr..l<Boolean>() [MSG]The ThreadLocal field configuring is not static |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 54 | [SRC]if (!getMetaClass().respondsTo(this, name, args.size())) { [MSG]getMetaClass() can probably be rewritten as metaClass |
UnnecessaryDefInMethodDeclaration | 3 | 51 | [SRC]public def methodMissing(String name, args) { [MSG]The def keyword is unneeded when a method is marked public |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 80 | [SRC]if (!getMetaClass().respondsTo(this, name, args.size())) { [MSG]getMetaClass() can probably be rewritten as metaClass |
UnnecessaryDefInMethodDeclaration | 3 | 77 | [SRC]public def methodMissing(String name, args) { [MSG]The def keyword is unneeded when a method is marked public |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
ExplicitCallToOrMethod | 2 | 31 | [SRC]node.or { [MSG]Explicit call to node.or(<not implemented yet for class: org.codehaus.groovy.ast.expr.ClosureExpression>) method can be rewritten as node | (<not implemented yet for class: org.codehaus.groovy.ast.expr.ClosureExpression>) |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryOverridingMethod | 3 | 27 | [SRC]void setProperty(String property, value) { [MSG]The method setProperty contains no logic and can be safely deleted |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 75 | [SRC]antProject.setUserProperty(MagicNames.ANT_FILE, file.get..olutePath()) [MSG]getAbsolutePath() can probably be rewritten as absolutePath |
UnnecessaryOverridingMethod | 3 | 36 | [SRC]def Object invokeMethod(String methodName, Object args) { [MSG]The method invokeMethod contains no logic and can be safely deleted |
UnnecessaryDefInMethodDeclaration | 3 | 36 | [SRC]def Object invokeMethod(String methodName, Object args) { [MSG]The def keyword is unneeded when a method returns the Object type |
UnnecessaryDefInMethodDeclaration | 3 | 40 | [SRC]def propertyMissing(String property, Object newValue) { [MSG]The def keyword is unneeded when a method returns the Object type |
UnnecessaryDefInMethodDeclaration | 3 | 44 | [SRC]private def doSetProperty(String property, newValue) { [MSG]The def keyword is unneeded when a method is marked private |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryObjectReferences | 2 | 137 | [SRC]builder.nodeCompleted(parent, node) [MSG]The code could be more concise by using a with() or identity() block |
UnnecessaryObjectReferences | 2 | 141 | [SRC]builder.postNodeCompletion(parent, node) [MSG]The code could be more concise by using a with() or identity() block |
UnnecessaryGetter | 2 | 91 | [SRC]antBuilder.project.removeBuildListener(antBuilder.projec..teners()[0]) [MSG]getBuildListeners() can probably be rewritten as buildListeners |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedPrivateField | 2 | 37 | [SRC]private static Logger logger = LoggerFactory.getLogger(A..ovyCompiler) [MSG]The field logger is not used within AntGroovyCompiler.groovy |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyMethod | 2 | 45 | [SRC]void setDependencyCacheDir(File dir) { [MSG]The method setDependencyCacheDir is both empty and not marked with @Override |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedPrivateField | 2 | 26 | [SRC]private static Logger logger = LoggerFactory.getLogger(A..alaCompiler) [MSG]The field logger is not used within AntScalaCompiler.groovy |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
NestedBlockDepth | 2 | 39 | [SRC]td(class: testPackage.statusClass) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 63 | [SRC]tr { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 64 | [SRC]td(class: testClass.statusClass) { [MSG]The nested block depth is 7 |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
NestedBlockDepth | 2 | 100 | [SRC]tr { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 101 | [SRC]td { [MSG]The nested block depth is 7 |
NestedBlockDepth | 2 | 102 | [SRC]div(class: 'summaryGroup') { [MSG]The nested block depth is 8 |
NestedBlockDepth | 2 | 103 | [SRC]table { [MSG]The nested block depth is 9 |
NestedBlockDepth | 2 | 104 | [SRC]tr { [MSG]The nested block depth is 10 |
NestedBlockDepth | 2 | 105 | [SRC]td { [MSG]The nested block depth is 11 |
NestedBlockDepth | 2 | 106 | [SRC]div(id: 'tests', class: 'infoBox') { [MSG]The nested block depth is 12 |
NestedBlockDepth | 2 | 111 | [SRC]td { [MSG]The nested block depth is 11 |
NestedBlockDepth | 2 | 112 | [SRC]div(id: 'failures', class: 'infoBox') { [MSG]The nested block depth is 12 |
NestedBlockDepth | 2 | 117 | [SRC]td { [MSG]The nested block depth is 11 |
NestedBlockDepth | 2 | 118 | [SRC]div(id: 'duration', class: 'infoBox') { [MSG]The nested block depth is 12 |
NestedBlockDepth | 2 | 127 | [SRC]td { [MSG]The nested block depth is 7 |
NestedBlockDepth | 2 | 128 | [SRC]div(id: 'successRate', class: "infoBox ${results.statusClass}") { [MSG]The nested block depth is 8 |
UnnecessaryGetter | 2 | 141 | [SRC]mkp.yield(" at ${DateFormat.getDateTimeInstance().format..w Date())}") [MSG]getDateTimeInstance() can probably be rewritten as dateTimeInstance |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedPrivateMethodParameter | 2 | 104 | [SRC]private void configureDistZipTask(Project project, Appli.. distSpec) { [MSG]Method parameter [applicationPluginConvention] is never referenced |
UnnecessaryGetter | 2 | 100 | [SRC]project.getAnt().chmod(file: "${installTask.destinationD..rm: 'ugo+x') [MSG]getAnt() can probably be rewritten as ant |
UnnecessaryDefInMethodDeclaration | 3 | 55 | [SRC]private def CopySpec createDistSpec(Project project) { [MSG]The def keyword is unneeded when a method is marked private |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 51 | [SRC]JavaExec runTask = project.getTasks().withType(JavaExec...find{task -> [MSG]getTasks() can probably be rewritten as tasks |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 114 | [SRC]project.tasks.add(taskName).dependsOn(configuration.getB..figuration)) [MSG]getBuildArtifacts() can probably be rewritten as buildArtifacts |
UnnecessaryGetter | 2 | 152 | [SRC]Upload upload = project.getTasks().add(name, Upload.class) [MSG]getTasks() can probably be rewritten as tasks |
UnnecessaryGetter | 2 | 155 | [SRC]upload.descriptorDestination = new File(project.getBuild.., "ivy.xml") [MSG]getBuildDir() can probably be rewritten as buildDir |
UnnecessaryGetter | 2 | 162 | [SRC]ConfigurationContainer configurations = project.getConfigurations(); [MSG]getConfigurations() can probably be rewritten as configurations |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedPrivateMethod | 2 | 117 | [SRC]private File getReportsDir() { [MSG]The method getReportsDir is not used within JavaPluginConvention.groovy |
UnnecessaryGetter | 2 | 167 | [SRC]return ConfigureUtil.configure(closure, new DefaultManif..eResolver)); [MSG]getProject() can probably be rewritten as project |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedPrivateField | 2 | 31 | [SRC]private static final Logger logger = LoggerFactory.getLo..(NotifySend) [MSG]The field logger is not used within NotifySend.groovy |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 37 | [SRC]with(new PrintWriter(sock.getOutputStream(), true)) { out -> [MSG]getOutputStream() can probably be rewritten as outputStream |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 32 | [SRC]project.getPlugins().apply(JavaBasePlugin.class); [MSG]getPlugins() can probably be rewritten as plugins |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedPrivateMethodParameter | 2 | 63 | [SRC]private void configureCompileDefaults(final Project proj..avaPlugin) { [MSG]Method parameter [javaPlugin] is never referenced |
UnnecessaryGetter | 2 | 70 | [SRC]project.getTasks().withType(ScalaDoc.class) {ScalaDoc scalaDoc -> [MSG]getTasks() can probably be rewritten as tasks |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 37 | [SRC]project.getTasks().withType(ScalaDoc.class) {ScalaDoc scalaDoc -> [MSG]getTasks() can probably be rewritten as tasks |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 71 | [SRC]if (!getApplicationName()) { [MSG]getApplicationName() can probably be rewritten as applicationName |
UnnecessaryGetter | 2 | 74 | [SRC]return "${GUtil.toConstant(getApplicationName())}_OPTS" [MSG]getApplicationName() can probably be rewritten as applicationName |
UnnecessaryGetter | 2 | 82 | [SRC]if (!getApplicationName()) { [MSG]getApplicationName() can probably be rewritten as applicationName |
UnnecessaryGetter | 2 | 85 | [SRC]return "${GUtil.toConstant(getApplicationName())}_EXIT_CONSOLE" [MSG]getApplicationName() can probably be rewritten as applicationName |
UnnecessaryGetter | 2 | 90 | [SRC]getOutputDir().mkdirs(); [MSG]getOutputDir() can probably be rewritten as outputDir |
UnnecessaryGetter | 2 | 105 | [SRC]applicationName: getApplicationName(), [MSG]getApplicationName() can probably be rewritten as applicationName |
UnnecessaryGetter | 2 | 106 | [SRC]optsEnvironmentVar: getOptsEnvironmentVar(), [MSG]getOptsEnvironmentVar() can probably be rewritten as optsEnvironmentVar |
UnnecessaryGetter | 2 | 107 | [SRC]mainClassName: getMainClassName(), [MSG]getMainClassName() can probably be rewritten as mainClassName |
UnnecessaryGetter | 2 | 110 | [SRC]applicationName: getApplicationName(), [MSG]getApplicationName() can probably be rewritten as applicationName |
UnnecessaryGetter | 2 | 111 | [SRC]optsEnvironmentVar: getOptsEnvironmentVar(), [MSG]getOptsEnvironmentVar() can probably be rewritten as optsEnvironmentVar |
UnnecessaryGetter | 2 | 112 | [SRC]exitEnvironmentVar: getExitEnvironmentVar(), [MSG]getExitEnvironmentVar() can probably be rewritten as exitEnvironmentVar |
UnnecessaryGetter | 2 | 113 | [SRC]mainClassName: getMainClassName(), [MSG]getMainClassName() can probably be rewritten as mainClassName |
UnnecessaryGetter | 2 | 122 | [SRC]def windowsScript = new File(getOutputDir(), "${project.name}.bat") [MSG]getOutputDir() can probably be rewritten as outputDir |
UnnecessaryGetter | 2 | 133 | [SRC]def unixScript = new File(getOutputDir(), project.name); [MSG]getOutputDir() can probably be rewritten as outputDir |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 46 | [SRC]Manifest manifest = getManifest() ?: new DefaultManifest(null) [MSG]getManifest() can probably be rewritten as manifest |
UnnecessaryGetter | 2 | 46 | [SRC]Manifest manifest = getManifest() ?: new DefaultManifest(null) [MSG]getManifest() can probably be rewritten as manifest |
UnnecessaryGetter | 2 | 85 | [SRC]if (getManifest() == null) { [MSG]getManifest() can probably be rewritten as manifest |
UnnecessaryGetter | 2 | 88 | [SRC]ConfigureUtil.configure(configureClosure, getManifest()); [MSG]getManifest() can probably be rewritten as manifest |
UnnecessaryGetter | 2 | 106 | [SRC]return ConfigureUtil.configure(configureClosure, getMetaInf()) [MSG]getMetaInf() can probably be rewritten as metaInf |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 46 | [SRC]def classpath = getClasspath() [MSG]getClasspath() can probably be rewritten as classpath |
UnnecessaryGetter | 2 | 52 | [SRC]def classpath = getClasspath() [MSG]getClasspath() can probably be rewritten as classpath |
UnnecessaryGetter | 2 | 58 | [SRC]getWebXml() [MSG]getWebXml() can probably be rewritten as webXml |
UnnecessaryGetter | 2 | 80 | [SRC]return ConfigureUtil.configure(configureClosure, getWebInf()) [MSG]getWebInf() can probably be rewritten as webInf |
UnnecessaryGetter | 2 | 110 | [SRC]FileCollection oldClasspath = getClasspath() [MSG]getClasspath() can probably be rewritten as classpath |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 48 | [SRC]((field.getModifiers() & Modifier.STATIC) == 0) && [MSG]getModifiers() can probably be rewritten as modifiers |
UnnecessaryGetter | 2 | 49 | [SRC](field.getName() != "metaClass") && [MSG]getName() can probably be rewritten as name |
UnnecessaryDefInMethodDeclaration | 3 | 53 | [SRC]private def antProperty(String fieldName) { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 61 | [SRC]private def antValue(String fieldName, def value) { [MSG]The def keyword is unneeded when a method is marked private |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedPrivateField | 2 | 24 | [SRC]private static Logger logger = LoggerFactory.getLogger(AntScalaDoc) [MSG]The field logger is not used within AntScalaDoc.groovy |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
ExplicitHashSetInstantiation | 2 | 53 | [SRC]Set<String> includeGroups = new HashSet<String>() [MSG]HashSet objects are better instantiated using the form "[] as HashSet" |
ExplicitHashSetInstantiation | 2 | 58 | [SRC]Set<String> excludeGroups = new HashSet<String>() [MSG]HashSet objects are better instantiated using the form "[] as HashSet" |
SerializableClassMustDefineSerialVersionUID | 2 | 27 | [SRC]public class TestNGOptions extends TestFrameworkOptions ..rializable { [MSG]The class org.gradle.api.tasks.testing.testng.TestNGOptions implements Serializable but does not define a serialVersionUID |
UnnecessaryGetter | 2 | 199 | [SRC]return suiteXmlBuilder.getMetaClass()."${name}" [MSG]getMetaClass() can probably be rewritten as metaClass |
UnnecessaryGetter | 2 | 207 | [SRC]return suiteXmlBuilder.getMetaClass().invokeMethod(suite..name, args); [MSG]getMetaClass() can probably be rewritten as metaClass |
UnnecessaryDefInMethodDeclaration | 3 | 197 | [SRC]public def propertyMissing(String name) { [MSG]The def keyword is unneeded when a method is marked public |
UnnecessaryDefInMethodDeclaration | 3 | 205 | [SRC]public def methodMissing(String name, args) { [MSG]The def keyword is unneeded when a method is marked public |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
ExplicitCallToAndMethod | 2 | 205 | [SRC]node.and { [MSG]Explicit call to node.and(<not implemented yet for class: org.codehaus.groovy.ast.expr.ClosureExpression>) method can be rewritten as node & (<not implemented yet for class: org.codehaus.groovy.ast.expr.ClosureExpression>) |
ExplicitCallToAndMethod | 2 | 238 | [SRC]node.and { [MSG]Explicit call to node.and(<not implemented yet for class: org.codehaus.groovy.ast.expr.ClosureExpression>) method can be rewritten as node & (<not implemented yet for class: org.codehaus.groovy.ast.expr.ClosureExpression>) |
ExplicitHashSetInstantiation | 2 | 45 | [SRC]private static final Set<String> GLOBAL_EXCLUDES = new H..et<String>() [MSG]HashSet objects are better instantiated using the form "[] as HashSet" |
UnnecessaryGetter | 2 | 234 | [SRC]return new AndSpec<FileTreeElement>([super.getAsSpec(), ..] as Spec[]) [MSG]getAsSpec() can probably be rewritten as asSpec |
UnnecessaryGetter | 2 | 234 | [SRC]return new AndSpec<FileTreeElement>([super.getAsSpec(), ..] as Spec[]) [MSG]getAsSpec() can probably be rewritten as asSpec |
UnnecessaryDefInMethodDeclaration | 3 | 57 | [SRC]static def setGlobalExcludes(Collection<String> excludes) { [MSG]The def keyword is unneeded when a method is marked static |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryDefInMethodDeclaration | 3 | 37 | [SRC]def Object getScriptTarget() { [MSG]The def keyword is unneeded when a method returns the Object type |
UnnecessaryDefInMethodDeclaration | 3 | 67 | [SRC]def methodMissing(String name, Object params) { [MSG]The def keyword is unneeded when a method returns the Object type |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryObjectReferences | 2 | 104 | [SRC]fileOperations.relativePath(path) [MSG]The code could be more concise by using a with() or identity() block |
UnnecessaryObjectReferences | 2 | 108 | [SRC]fileOperations.fileTree(baseDir) [MSG]The code could be more concise by using a with() or identity() block |
UnnecessaryObjectReferences | 2 | 112 | [SRC]fileOperations.fileTree(args) [MSG]The code could be more concise by using a with() or identity() block |
UnnecessaryObjectReferences | 2 | 116 | [SRC]fileOperations.fileTree(closure) [MSG]The code could be more concise by using a with() or identity() block |
UnnecessaryObjectReferences | 2 | 120 | [SRC]fileOperations.zipTree(zipPath) [MSG]The code could be more concise by using a with() or identity() block |
UnnecessaryObjectReferences | 2 | 124 | [SRC]fileOperations.tarTree(tarPath) [MSG]The code could be more concise by using a with() or identity() block |
UnnecessaryObjectReferences | 2 | 128 | [SRC]fileOperations.copy(closure) [MSG]The code could be more concise by using a with() or identity() block |
UnnecessaryObjectReferences | 2 | 132 | [SRC]fileOperations.copySpec(closure) [MSG]The code could be more concise by using a with() or identity() block |
UnnecessaryGetter | 2 | 80 | [SRC]ConfigureUtil.configure(configureClosure, getBuildscript()) [MSG]getBuildscript() can probably be rewritten as buildscript |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 23 | [SRC]getBuildscript() [MSG]getBuildscript() can probably be rewritten as buildscript |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 44 | [SRC]jdt.sourceCompatibility = getSourceCompatibility() [MSG]getSourceCompatibility() can probably be rewritten as sourceCompatibility |
UnnecessaryGetter | 2 | 45 | [SRC]jdt.targetCompatibility = getTargetCompatibility() [MSG]getTargetCompatibility() can probably be rewritten as targetCompatibility |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
NestedBlockDepth | 2 | 121 | [SRC]whenConfigured { Classpath classpath -> [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 122 | [SRC]for (entry in classpath.entries) { [MSG]The nested block depth is 7 |
NestedBlockDepth | 2 | 123 | [SRC]if (entry instanceof Library) { [MSG]The nested block depth is 8 |
NestedBlockDepth | 2 | 168 | [SRC]getMainSourceDirs(otherProject).collect { new WbResource..ePath(it)) } [MSG]The nested block depth is 6 |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 62 | [SRC]def allAttributes = attributes.findAll { it.value } + [k.. path: path] [MSG]getKind() can probably be rewritten as kind |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
SerializableClassMustDefineSerialVersionUID | 2 | 21 | [SRC]class BuildCommand implements Serializable { [MSG]The class org.gradle.plugins.eclipse.model.BuildCommand implements Serializable but does not define a serialVersionUID |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 41 | [SRC]node.appendNode('classpathentry', [kind: getKind(), path: path]) [MSG]getKind() can probably be rewritten as kind |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryDefInMethodDeclaration | 3 | 77 | [SRC]private def readReferencedProjects() { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 83 | [SRC]private def readNatures() { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 87 | [SRC]private def readBuildCommands() { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 97 | [SRC]private def readLinks() { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 133 | [SRC]private def addReferencedProjectsToXml() { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 140 | [SRC]private def addNaturesToXml() { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 147 | [SRC]private def addBuildSpecToXml() { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 161 | [SRC]private def addLinksToXml() { [MSG]The def keyword is unneeded when a method is marked private |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryCollectCall | 2 | 45 | [SRC]def sourceDirs = sourceDirSets.collect { it.srcDirs }.flatten() [MSG]The call to collect could probably be rewritten as a spread expression: sourceDirSets*.srcDirs |
UnnecessaryCollectCall | 2 | 75 | [SRC]return getDependencies(eclipseClasspath.plusConfiguratio..ependency -> [MSG]The call to collect could probably be rewritten as a spread expression: this.getDependencies(eclipseClasspath.plusConfigurations, eclipseClasspath.minusConfigurations, <not implemented yet for class: org.codehaus.groovy.ast.expr.ClosureExpression>)*.dependencyProject |
UnnecessaryGetter | 2 | 33 | [SRC]entries.addAll(getEntriesFromContainers(eclipseClasspath..ntainers())) [MSG]getContainers() can probably be rewritten as containers |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryCollectCall | 2 | 92 | [SRC]dependencies.collect { it.resolve() }.flatten() [MSG]The call to collect could probably be rewritten as a spread expression: dependencies*.resolve() |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 121 | [SRC]module.configure(getContentPath(), getSourcePaths(), get..ludePaths(), [MSG]getContentPath() can probably be rewritten as contentPath |
UnnecessaryGetter | 2 | 121 | [SRC]module.configure(getContentPath(), getSourcePaths(), get..ludePaths(), [MSG]getSourcePaths() can probably be rewritten as sourcePaths |
UnnecessaryGetter | 2 | 121 | [SRC]module.configure(getContentPath(), getSourcePaths(), get..ludePaths(), [MSG]getTestSourcePaths() can probably be rewritten as testSourcePaths |
UnnecessaryGetter | 2 | 121 | [SRC]module.configure(getContentPath(), getSourcePaths(), get..ludePaths(), [MSG]getExcludePaths() can probably be rewritten as excludePaths |
UnnecessaryGetter | 2 | 122 | [SRC]inheritOutputDirs, getOutputPath(), getTestOutputPath(),..javaVersion) [MSG]getOutputPath() can probably be rewritten as outputPath |
UnnecessaryGetter | 2 | 122 | [SRC]inheritOutputDirs, getOutputPath(), getTestOutputPath(),..javaVersion) [MSG]getTestOutputPath() can probably be rewritten as testOutputPath |
UnnecessaryGetter | 2 | 122 | [SRC]inheritOutputDirs, getOutputPath(), getTestOutputPath(),..javaVersion) [MSG]getDependencies() can probably be rewritten as dependencies |
UnnecessaryGetter | 2 | 126 | [SRC]getPath(getModuleDir()) [MSG]getModuleDir() can probably be rewritten as moduleDir |
UnnecessaryGetter | 2 | 130 | [SRC]getOutputDir() ? getPath(getOutputDir()) : null [MSG]getOutputDir() can probably be rewritten as outputDir |
UnnecessaryGetter | 2 | 130 | [SRC]getOutputDir() ? getPath(getOutputDir()) : null [MSG]getOutputDir() can probably be rewritten as outputDir |
UnnecessaryGetter | 2 | 134 | [SRC]getTestOutputDir() ? getPath(getTestOutputDir()) : null [MSG]getTestOutputDir() can probably be rewritten as testOutputDir |
UnnecessaryGetter | 2 | 134 | [SRC]getTestOutputDir() ? getPath(getTestOutputDir()) : null [MSG]getTestOutputDir() can probably be rewritten as testOutputDir |
UnnecessaryGetter | 2 | 138 | [SRC]getSourceDirs().findAll { it.exists() }.collect { getPath(it) } [MSG]getSourceDirs() can probably be rewritten as sourceDirs |
UnnecessaryGetter | 2 | 142 | [SRC]getTestSourceDirs().findAll { it.exists() }.collect { getPath(it) } [MSG]getTestSourceDirs() can probably be rewritten as testSourceDirs |
UnnecessaryGetter | 2 | 146 | [SRC]getExcludeDirs().collect { getPath(it) } [MSG]getExcludeDirs() can probably be rewritten as excludeDirs |
UnnecessaryGetter | 2 | 208 | [SRC]firstLevelDependencies.addAll(configuration.getAllDepend..All(filter)) [MSG]getAllDependencies() can probably be rewritten as allDependencies |
UnnecessaryGetter | 2 | 211 | [SRC]configuration.getAllDependencies().findAll(filter).each { minusDep -> [MSG]getAllDependencies() can probably be rewritten as allDependencies |
UnnecessaryGetter | 2 | 280 | [SRC]factory.addPathVariable('MODULE_DIR', getOutputFile().parentFile) [MSG]getOutputFile() can probably be rewritten as outputFile |
UnnecessaryDefInMethodDeclaration | 3 | 196 | [SRC]private def getSelfResolvingFiles(Collection dependencie..ing scope) { [MSG]The def keyword is unneeded when a method is marked private |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryDefInMethodDeclaration | 3 | 43 | [SRC]private def configureIdeaWorkspace(Project project) { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 52 | [SRC]private def configureIdeaModule(Project project) { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 63 | [SRC]private def configureIdeaProject(Project project) { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 75 | [SRC]private def configureForJavaPlugin(Project project) { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 82 | [SRC]private def configureIdeaProjectForJava(Project project) { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 90 | [SRC]private def configureIdeaModuleForJava(Project project) { [MSG]The def keyword is unneeded when a method is marked private |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 49 | [SRC]return new Project(xmlTransformer, getPathFactory()) [MSG]getPathFactory() can probably be rewritten as pathFactory |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 43 | [SRC]parentNode.appendNode('orderEntry', [type: 'module', 'mo..dExported()) [MSG]getAttributeMapForScopeAndExported() can probably be rewritten as attributeMapForScopeAndExported |
UnnecessaryGetter | 2 | 77 | [SRC]result = 31 * result + getScopeHash(); [MSG]getScopeHash() can probably be rewritten as scopeHash |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryGetter | 2 | 61 | [SRC]Node libraryNode = parentNode.appendNode('orderEntry', [..e('library') [MSG]getAttributeMapForScopeAndExported() can probably be rewritten as attributeMapForScopeAndExported |
UnnecessaryGetter | 2 | 118 | [SRC]result = 31 * result + getScopeHash() [MSG]getScopeHash() can probably be rewritten as scopeHash |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnnecessaryDefInMethodDeclaration | 3 | 97 | [SRC]private def findProjectRootManager() { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 101 | [SRC]private def findWildcardResourcePatterns() { [MSG]The def keyword is unneeded when a method is marked private |
UnnecessaryDefInMethodDeclaration | 3 | 105 | [SRC]private def findModules() { [MSG]The def keyword is unneeded when a method is marked private |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
SimpleDateFormatMissingLocale | 2 | 25 | [SRC]private static final SimpleDateFormat DATE_FORMAT = new .. HH:mm:ss"); [MSG]Created an instance of SimpleDateFormat without specifying a Locale |