Report timestamp: May 10, 2010 10:17:32 PM
Package | Total Files | Files with Violations | Priority 1 | Priority 2 | Priority 3 |
---|---|---|---|---|---|
All Packages | 103 | 36 | 0 | 104 | 14 |
scripts | 50 | 19 | 0 | 76 | 10 |
src/cli/griffon/test | 1 | 1 | 0 | 3 | 0 |
src/cli/org/codehaus/griffon | 1 | 0 | 0 | 0 | 0 |
src/cli/org/codehaus/griffon/commons | 1 | 1 | 0 | 3 | 0 |
src/cli/org/codehaus/griffon/commons/cfg | 1 | 1 | 0 | 3 | 0 |
src/cli/org/codehaus/griffon/plugins | 2 | 2 | 0 | 2 | 1 |
src/cli/org/codehaus/griffon/plugins/logging | 1 | 1 | 0 | 0 | 1 |
src/cli/org/codehaus/griffon/support | 1 | 1 | 0 | 1 | 0 |
src/cli/org/codehaus/griffon/test | 2 | 0 | 0 | 0 | 0 |
src/cli/org/codehaus/griffon/util | 1 | 0 | 0 | 0 | 0 |
src/griffon/griffon-app/conf | 3 | 0 | 0 | 0 | 0 |
src/griffon/griffon-app/lifecycle | 5 | 0 | 0 | 0 | 0 |
src/rt/griffon/app | 3 | 0 | 0 | 0 | 0 |
src/rt/griffon/applet | 2 | 0 | 0 | 0 | 0 |
src/rt/griffon/application | 2 | 1 | 0 | 1 | 0 |
src/rt/griffon/builder | 1 | 0 | 0 | 0 | 0 |
src/rt/griffon/core | 7 | 1 | 0 | 1 | 0 |
src/rt/griffon/util | 8 | 2 | 0 | 4 | 1 |
test/cli/org/codehaus/griffon/cli | 6 | 5 | 0 | 10 | 0 |
test/rt/griffon/builder | 1 | 0 | 0 | 0 | 0 |
test/rt/griffon/gui | 1 | 0 | 0 | 0 | 0 |
test/rt/griffon/gui/test | 1 | 0 | 0 | 0 | 0 |
test/rt/griffon/util | 1 | 0 | 0 | 0 | 0 |
test/rt/groovy/util | 1 | 1 | 0 | 0 | 1 |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedVariable | 2 | 35 | [SRC]def (pkg, name) = extractArtifactName(argsMap["params"][0]) |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [67.9] | |
NestedBlockDepth | 2 | 84 | [SRC]try { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 97 | [SRC]catch(Throwable t) { [MSG]The nested block depth is 6 |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [253.5] | |
CyclomaticComplexity | 2 | [MSG]The cyclomatic complexity for method [run] is [33] | |
UnusedImport | 3 | 24 | [SRC]import org.codehaus.griffon.util.BuildSettings |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyCatchBlock | 2 | 273 | [SRC]catch(SVNException e) { |
EmptyCatchBlock | 2 | 277 | [SRC]catch(SVNException e) { |
EmptyCatchBlock | 2 | 281 | [SRC]catch(SVNException e) { |
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [167.8] | |
CyclomaticComplexity | 2 | [MSG]The cyclomatic complexity for method [run] is [28] | |
NestedBlockDepth | 2 | 85 | [SRC]if (!repos.info("griffon-$pluginName", -1)) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 88 | [SRC]else { [MSG]The nested block depth is 6 |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [84.4] | |
UnusedImport | 3 | 23 | [SRC]import static griffon.util.GriffonApplicationUtils.isLinux |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedImport | 3 | 23 | [SRC]import org.codehaus.griffon.commons.ConfigurationHolder |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedImport | 3 | 24 | [SRC]import org.codehaus.griffon.commons.ConfigurationHolder |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [61.7] | |
CyclomaticComplexity | 2 | [MSG]The cyclomatic complexity for method [run] is [23] |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyIfStatement | 2 | 44 | [SRC]if (!force) { |
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [153.1] | |
CyclomaticComplexity | 2 | [MSG]The cyclomatic complexity for method [run] is [38] | |
NestedBlockDepth | 2 | 113 | [SRC]if (!Boolean.valueOf(System.getProperty(GriffonContext.E..DEFAULT))) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 126 | [SRC]if (indent != '') { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 160 | [SRC]if (portions.keySet().contains('view')) { [MSG]The nested block depth is 6 |
UnusedVariable | 2 | 230 | [SRC]def fileText = it.getFile().getText() |
UnnecessaryTernaryExpression | 3 | 36 | [SRC]boolean force = args?.indexOf('-force') > -1 ? true : false |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [85.6] |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedImport | 3 | 17 | [SRC]import org.codehaus.griffon.util.GriffonNameUtils |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyCatchBlock | 2 | 68 | [SRC]catch (MissingPropertyException e) { |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [92.5] | |
UnusedImport | 3 | 26 | [SRC]import org.springframework.core.io.FileSystemResource |
UnusedImport | 3 | 27 | [SRC]import org.codehaus.griffon.util.GriffonNameUtils |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [423.4] | |
CyclomaticComplexity | 2 | [MSG]The cyclomatic complexity for method [run] is [88] | |
NestedBlockDepth | 2 | 187 | [SRC]if(entry.suffix) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 189 | [SRC]} else { [MSG]The nested block depth is 6 |
UnusedVariable | 2 | 308 | [SRC]long originalLastMod = targetFile.lastModified() |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [65.0] |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [172.0] | |
CyclomaticComplexity | 2 | [MSG]The cyclomatic complexity for method [run] is [39] | |
NestedBlockDepth | 2 | 236 | [SRC]plugin.'release'.each { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 252 | [SRC]if( release.'title'.text() ) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 254 | [SRC]} else { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 258 | [SRC]if( release.'author'.text() ) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 262 | [SRC]if( release.'authorEmail'.text() ) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 266 | [SRC]if( release.'documentation'.text() ) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 270 | [SRC]if( release.'description'.text() ) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 274 | [SRC]if( release.'toolkits'.text() ) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 277 | [SRC]} else { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 281 | [SRC]if( release.'platforms'.text() ) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 284 | [SRC]} else { [MSG]The nested block depth is 6 |
UnusedVariable | 2 | 314 | [SRC]def displayFullPluginInfo = { pluginName -> |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [149.1] | |
CyclomaticComplexity | 2 | [MSG]The cyclomatic complexity for method [run] is [28] | |
UnusedImport | 3 | 20 | [SRC]import org.codehaus.griffon.plugins.GriffonPluginUtils |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [139.6] |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyCatchBlock | 2 | 827 | [SRC]} catch (Exception e) { |
EmptyCatchBlock | 2 | 898 | [SRC]} catch (Exception e) { |
EmptyCatchBlock | 2 | 913 | [SRC]} catch (Exception e) { |
EmptyCatchBlock | 2 | 920 | [SRC]} catch (Exception e) { |
AbcComplexity | 2 | [MSG]The ABC score for method [run] is [430.2] | |
CyclomaticComplexity | 2 | [MSG]The cyclomatic complexity for method [run] is [111] | |
NestedBlockDepth | 2 | 214 | [SRC]if(griffonContext == null) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 233 | [SRC]for(p in pluginManager.failedLoadPlugins) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 360 | [SRC]line.eachMatch(/<li><a href="griffon-(.+?)">/) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 554 | [SRC]def release = plugin.'release'.find {rel -> rel.'@versio..ginRelease } [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 555 | [SRC]if (release) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 557 | [SRC]} else { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 691 | [SRC]metadata.propertyNames().grep{ it.startsWith('plugins.') }.each { p -> [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 691 | [SRC]metadata.propertyNames().grep{ it.startsWith('plugins.') }.each { p -> [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 699 | [SRC]if(supportedToolkitsByPlugin.intersect(unsupportedToolkits)) { [MSG]The nested block depth is 7 |
NestedBlockDepth | 2 | 703 | [SRC]if(pluginsToUninstall) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 709 | [SRC]pluginsToUninstall.each { n, v -> uninstallPluginsMessag..n}-${v}\n" } [MSG]The nested block depth is 7 |
NestedBlockDepth | 2 | 712 | [SRC]Do you wish to proceed?\n""", { [MSG]The nested block depth is 7 |
NestedBlockDepth | 2 | 713 | [SRC]pluginsToUninstall.each { n, v -> [MSG]The nested block depth is 8 |
NestedBlockDepth | 2 | 718 | [SRC]},{ [MSG]The nested block depth is 7 |
NestedBlockDepth | 2 | 744 | [SRC]if (!GriffonPluginUtils.isValidVersion(dependency.@versi..pVersion)) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 746 | [SRC]} else { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 773 | [SRC]def providedScripts = resolveResources("file:${pluginsHo..tsWith('_')} [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 774 | [SRC]if (providedScripts) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 777 | [SRC]providedScripts.file.each {file -> [MSG]The nested block depth is 7 |
NestedBlockDepth | 2 | 784 | [SRC]if (pluginEvents.exists()) { [MSG]The nested block depth is 6 |
UnusedVariable | 2 | 205 | [SRC]def application |
DuplicateImport | 3 | 38 | [SRC]import java.util.zip.ZipEntry |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyCatchBlock | 2 | 147 | [SRC]catch (InterruptedException ex) { |
UnusedPrivateField | 2 | 38 | [SRC]private final Condition waiting = lock.newCondition() |
UnusedVariable | 2 | 137 | [SRC]final monitor = "monitor" |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyCatchBlock | 2 | 570 | [SRC]catch (NoSuchFieldException e) |
EmptyCatchBlock | 2 | 636 | [SRC]catch (Exception e) |
UnusedVariable | 2 | 451 | [SRC]for (Iterator j = words.iterator(); j.hasNext();) { |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
NestedBlockDepth | 2 | 110 | [SRC]if(classLoader) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 116 | [SRC]if(resource.filename.endsWith('.groovy')) { [MSG]The nested block depth is 6 |
NestedBlockDepth | 2 | 120 | [SRC]else if(resource.filename.endsWith('.properties')) { [MSG]The nested block depth is 6 |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EqualsAndHashCode | 2 | 551 | [SRC]class VersionComparator implements Comparator{ |
UnusedVariable | 2 | 294 | [SRC]def userHome = System.getProperty("user.home") |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedImport | 3 | 20 | [SRC]import groovy.util.slurpersupport.GPathResult |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedImport | 3 | 29 | [SRC]import org.apache.log4j.net.SMTPAppender |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyCatchBlock | 2 | 54 | [SRC]} catch( MissingPropertyException mpe ) { |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyCatchBlock | 2 | 78 | [SRC]} catch (Throwable ignored) { |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyCatchBlock | 2 | 86 | [SRC]} catch( ignored ) { |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyCatchBlock | 2 | 202 | [SRC]} catch (MissingPropertyException ignored) { |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
EmptyCatchBlock | 2 | 359 | [SRC]} catch (Throwable ignored) { |
EmptyCatchBlock | 2 | 372 | [SRC]} catch (Throwable ignored) { |
AbcComplexity | 2 | [MSG]The ABC score for method [buildMVCGroup] is [72.2] | |
MethodSize | 3 | 200 | [MSG]Method "buildMVCGroup" is 108 lines |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
JUnitSetUpCallsSuper | 2 | 43 | [SRC]void setUp() { |
JUnitTearDownCallsSuper | 2 | 52 | [SRC]void tearDown() { |
AbcComplexity | 2 | [MSG]The ABC score for method [gantRun] is [73.8] |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
JUnitPublicNonTestMethod | 2 | 14 | [SRC]void tryIntegrationTest(String className) { |
JUnitPublicNonTestMethod | 2 | 18 | [SRC]void tryIntegrationTest(String scriptArg, String className) { |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
JUnitPublicNonTestMethod | 2 | 30 | [SRC]void tryMvc(String name) { |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
JUnitPublicNonTestMethod | 2 | 13 | [SRC]void tryScript(String className) { |
JUnitPublicNonTestMethod | 2 | 17 | [SRC]void tryScript(String scriptArg, String className) { |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
JUnitPublicNonTestMethod | 2 | 14 | [SRC]void tryUnitTest(String className) { |
JUnitPublicNonTestMethod | 2 | 18 | [SRC]void tryUnitTest(String scriptArg, String className) { |
Rule Name | Priority | Line # | Source Line / Message |
---|---|---|---|
UnusedImport | 3 | 4 | [SRC]import java.lang.reflect.InvocationTargetException |