Violation | Line |
---|
The class 'UnPackThemeMojo' has a Cyclomatic Complexity of 13 (Highest = 28). | 53 - 231 |
The class 'UnPackThemeMojo' has a Cyclomatic Complexity of 13 (Highest = 28). | 53 - 231 |
Parameter 'jarFile' is not assigned and could be declared final | 86 |
The method 'getThemeDescriptor' has a Cyclomatic Complexity of 9. | 86 - 121 |
Local variable 'files' could be declared final | 90 |
Local variable 'nextEntry' could be declared final | 92 |
Local variable 'name' could be declared final | 96 |
Position literals first in String comparisons | 97 |
Position literals first in String comparisons | 97 |
Local variable 'stringWriter' could be declared final | 100 |
Avoid instantiating new objects inside loops | 100 |
Avoid empty catch blocks | 115 - 117 |
Avoid empty catch blocks | 115 - 117 |
The method 'execute' has a Cyclomatic Complexity of 28. | 123 - 217 |
The method execute() has an NPath complexity of 4575 | 123 - 217 |
The method 'execute' has a Cyclomatic Complexity of 28. | 123 - 217 |
Local variable 'artifacts' could be declared final | 125 |
Local variable 'artifact' could be declared final | 131 |
Local variable 'file' could be declared final | 136 |
Avoid instantiating new objects inside loops | 136 |
Local variable 'descriptor' could be declared final | 138 |
Local variable 'name' could be declared final | 141 |
Local variable 'tempLocation' could be declared final | 142 |
Avoid instantiating new objects inside loops | 142 |
Avoid if (x != y) ..; else ..; | 144 - 149 |
Deeply nested if..then statements are hard to read | 147 - 149 |
Local variable 'xpp3Dom' could be declared final | 156 |
Avoid instantiating new objects inside loops | 156 |
Local variable 'themeDefinitions' could be declared final | 157 |
Deeply nested if..then statements are hard to read | 158 - 177 |
Local variable 'versionedDom' could be declared final | 160 |
Local variable 'versioned' could be declared final | 162 |
Local variable 'resourcePathDom' could be declared final | 164 |
Local variable 'properties' could be declared final | 166 |
Avoid instantiating new objects inside loops | 166 |
Local variable 'metaInf' could be declared final | 167 |
Avoid instantiating new objects inside loops | 168 |
Deeply nested if..then statements are hard to read | 183 - 187 |
Local variable 'fileNames' could be declared final | 188 |
Local variable 'fileNamesLength' could be declared final | 189 |
Local variable 'fileName' could be declared final | 190 |
Local variable 'fromFile' could be declared final | 191 |
Avoid instantiating new objects inside loops | 191 |
Deeply nested if..then statements are hard to read | 193 - 196 |
Deeply nested if..then statements are hard to read | 197 - 199 |
Local variable 'toFile' could be declared final | 200 |
Avoid instantiating new objects inside loops | 200 |
Parameter 'location' is not assigned and could be declared final | 219 |
Parameter 'file' is not assigned and could be declared final | 219 |
Local variable 'archiveExt' could be declared final | 221 |
Local variable 'unArchiver' could be declared final | 223 |