PMD Results
The following document contains the results of PMD 7.17.0.
Violations By Priority
Priority 3
com/randomnoun/common/Base64.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getAlphabet': private methods cannot be overridden | 421 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getDecodabet': private methods cannot be overridden | 439 |
| EmptyCatchBlock | Avoid empty catch blocks | 695 |
| EmptyCatchBlock | Avoid empty catch blocks | 696 |
| EmptyCatchBlock | Avoid empty catch blocks | 697 |
| EmptyCatchBlock | Avoid empty catch blocks | 698 |
| EmptyCatchBlock | Avoid empty catch blocks | 937 |
| EmptyCatchBlock | Avoid empty catch blocks | 938 |
| EmptyCatchBlock | Avoid empty catch blocks | 939 |
| EmptyCatchBlock | Avoid empty catch blocks | 1298 |
| EmptyCatchBlock | Avoid empty catch blocks | 1299 |
| EmptyCatchBlock | Avoid empty catch blocks | 1300 |
| EmptyCatchBlock | Avoid empty catch blocks | 1390 |
| EmptyCatchBlock | Avoid empty catch blocks | 1391 |
| EmptyCatchBlock | Avoid empty catch blocks | 1430 |
| EmptyCatchBlock | Avoid empty catch blocks | 1462 |
| EmptyCatchBlock | Avoid empty catch blocks | 1523 |
| EmptyCatchBlock | Avoid empty catch blocks | 1576 |
| EmptyCatchBlock | Avoid empty catch blocks | 1605 |
| EmptyCatchBlock | Avoid empty catch blocks | 1633 |
com/randomnoun/common/ErrorList.java
| Rule | Violation | Line |
|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 112 |
| UselessOverridingMethod | Overriding method merely calls super | 117 |
com/randomnoun/common/ExceptionUtil.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 122–124 |
| EmptyCatchBlock | Avoid empty catch blocks | 124–126 |
| EmptyCatchBlock | Avoid empty catch blocks | 126–128 |
| EmptyCatchBlock | Avoid empty catch blocks | 128–130 |
| EmptyCatchBlock | Avoid empty catch blocks | 130–132 |
| EmptyCatchBlock | Avoid empty catch blocks | 302–304 |
| EmptyCatchBlock | Avoid empty catch blocks | 304–305 |
| EmptyCatchBlock | Avoid empty catch blocks | 416–418 |
| EmptyCatchBlock | Avoid empty catch blocks | 474–476 |
| EmptyCatchBlock | Avoid empty catch blocks | 511–513 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 578–580 |
| EmptyCatchBlock | Avoid empty catch blocks | 605–607 |
| EmptyCatchBlock | Avoid empty catch blocks | 607–609 |
| EmptyCatchBlock | Avoid empty catch blocks | 609–611 |
| EmptyCatchBlock | Avoid empty catch blocks | 611–613 |
| EmptyCatchBlock | Avoid empty catch blocks | 613–615 |
com/randomnoun/common/IniFile.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 52–70 |
| EmptyCatchBlock | Avoid empty catch blocks | 77–79 |
com/randomnoun/common/Keyboard.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–292 |
| EmptyCatchBlock | Avoid empty catch blocks | 152 |
com/randomnoun/common/MRUCache.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'RetrievalCallback': member interfaces are implicitly static | 81 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'get': the method is declared in an interface type | 89 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 259–263 |
com/randomnoun/common/PropertyParser.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 446–448 |
| EmptyCatchBlock | Avoid empty catch blocks | 513–514 |
com/randomnoun/common/Registry.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryReturn | Unnecessary return statement | 102 |
com/randomnoun/common/ResourceFinder.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'preProcess': the method is declared in an interface type | 311 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'postProcess': the method is declared in an interface type | 329 |
| EmptyControlStatement | Empty while statement | 371 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 478 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 516 |
| UnnecessaryReturn | Unnecessary return statement | 720 |
| EmptyControlStatement | Empty if statement | 726–754 |
| UnnecessaryReturn | Unnecessary return statement | 755 |
| EmptyControlStatement | Empty if statement | 818–853 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 1170 |
com/randomnoun/common/Struct.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ToStringReturnsJson': member interfaces are implicitly static | 76 |
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ToJson': member interfaces are implicitly static | 79 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'toJson': the method is declared in an interface type | 80 |
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ToJsonFormat': member interfaces are implicitly static | 87 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'toJson': the method is declared in an interface type | 88 |
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'WriteJsonFormat': member interfaces are implicitly static | 95 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'writeJsonFormat': the method is declared in an interface type | 96 |
| UnnecessarySemicolon | Unnecessary semicolon | 119 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 502 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 502 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 504 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 504 |
| PrimitiveWrapperInstantiation | Do not use `new Double(...)`, prefer `Double.valueOf(...)` | 506 |
| PrimitiveWrapperInstantiation | Do not use `new Double(...)`, prefer `Double.valueOf(...)` | 506 |
| PrimitiveWrapperInstantiation | Do not use `new Float(...)`, prefer `Float.valueOf(...)` | 508 |
| PrimitiveWrapperInstantiation | Do not use `new Float(...)`, prefer `Float.valueOf(...)` | 508 |
| PrimitiveWrapperInstantiation | Do not use `new Short(...)`, prefer `Short.valueOf(...)` | 510 |
| PrimitiveWrapperInstantiation | Do not use `new Short(...)`, prefer `Short.valueOf(...)` | 510 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 514 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 516 |
| PrimitiveWrapperInstantiation | Do not use `new Double(...)`, prefer `Double.valueOf(...)` | 518 |
| PrimitiveWrapperInstantiation | Do not use `new Float(...)`, prefer `Float.valueOf(...)` | 520 |
| PrimitiveWrapperInstantiation | Do not use `new Short(...)`, prefer `Short.valueOf(...)` | 522 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 586 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 588 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1091–1093 |
| UnusedLocalVariable | Avoid unused local variables such as 'entry'. | 1637 |
com/randomnoun/common/Text.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 131–137 |
| EmptyControlStatement | Empty if statement | 133–137 |
| EmptyControlStatement | Empty else statement | 432–434 |
| EmptyControlStatement | Empty if statement | 482–490 |
| EmptyControlStatement | Empty else statement | 574–576 |
| EmptyControlStatement | Empty if statement | 643–652 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'encodeUrl': private methods cannot be overridden | 1343 |
com/randomnoun/common/XmlUtil.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 197–199 |
com/randomnoun/common/db/SqlParser.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Consumer': member interfaces are implicitly static | 85 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'consume': the method is declared in an interface type | 86 |
| UnusedLocalVariable | Avoid unused local variables such as 'snc'. | 123 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 264–267 |
com/randomnoun/common/db/dao/MysqlDatabaseReader.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 145 |
com/randomnoun/common/db/to/TableTO.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 28 |
com/randomnoun/common/email/EmailWrapper.java
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address ${variableName} | 287 |
com/randomnoun/common/jexl/DateSpan.java
| Rule | Violation | Line |
|---|---|---|
| OverrideBothEqualsAndHashcode | Ensure you override both equals() and hashCode() | 368 |
com/randomnoun/common/jexl/eval/EvalFunction.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifiers 'public static' on field 'logger': the field is declared in an interface type | 35 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'evaluate': the method is declared in an interface type | 53 |
com/randomnoun/common/jexl/eval/Evaluator.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 102–110 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 114–116 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 120–122 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 126–129 |
| PrimitiveWrapperInstantiation | Do not use `Boolean.valueOf(true)`, prefer `Boolean.TRUE` | 686 |
| PrimitiveWrapperInstantiation | Do not use `Boolean.valueOf(false)`, prefer `Boolean.FALSE` | 688 |
com/randomnoun/common/jexl/sql/SqlFunction.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifiers 'public abstract' on method 'toSql': the method is declared in an interface type | 33 |
com/randomnoun/common/jexl/sql/SqlGenerator.java
| Rule | Violation | Line |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'ops'. | 674 |
| UnusedLocalVariable | Avoid unused local variables such as 'ops'. | 714 |
| UnusedLocalVariable | Avoid unused local variables such as 'ops'. | 759 |
| UnusedLocalVariable | Avoid unused local variables such as 'ops'. | 798 |
| PrimitiveWrapperInstantiation | Do not use `Boolean.valueOf(true)`, prefer `Boolean.TRUE` | 1075 |
| PrimitiveWrapperInstantiation | Do not use `Boolean.valueOf(false)`, prefer `Boolean.FALSE` | 1077 |
com/randomnoun/common/log4j2/LoggingOutputStream.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 43–56 |
com/randomnoun/common/security/SecurityAuthenticator.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'initialise': the method is declared in an interface type | 29 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'authenticate': the method is declared in an interface type | 41 |
com/randomnoun/common/security/SecurityLoader.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'initialise': the method is declared in an interface type | 31 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllRolePermissions': the method is declared in an interface type | 42 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadRolePermissions': the method is declared in an interface type | 48 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadUserRolePermissions': the method is declared in an interface type | 60 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadUserPermissions': the method is declared in an interface type | 66 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadUserRoles': the method is declared in an interface type | 71 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadUser': the method is declared in an interface type | 80 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllUsers': the method is declared in an interface type | 92 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllResources': the method is declared in an interface type | 104 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllActivities': the method is declared in an interface type | 118 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllRoles': the method is declared in an interface type | 130 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllPermissions': the method is declared in an interface type | 142 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllRoleDetails': the method is declared in an interface type | 153 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllUserDetails': the method is declared in an interface type | 162 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'resetSecurityContext': the method is declared in an interface type | 166 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'saveUserRolesAndPermissions': the method is declared in an interface type | 175 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'saveRolePermissions': the method is declared in an interface type | 185 |
com/randomnoun/common/security/impl/SpringSecurityLoaderImpl.java
| Rule | Violation | Line |
|---|---|---|
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 206 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 208 |
com/randomnoun/common/spring/SelectFromResultSetExtractor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getCte': the method is declared in an interface type | 13 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getSelect': the method is declared in an interface type | 15 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getFrom': the method is declared in an interface type | 17 |
com/randomnoun/common/spring/SelectFromRowMapper.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getCte': the method is declared in an interface type | 15 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getSelect': the method is declared in an interface type | 17 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getFrom': the method is declared in an interface type | 19 |
com/randomnoun/common/spring/StringRowMapper.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 60–79 |
com/randomnoun/common/spring/StructuredListResultSetExtractor.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 376–382 |
com/randomnoun/common/spring/StructuredMapCallbackHandlerResultSetExtractor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'StructuredMapCallbackHandler': member interfaces are implicitly static | 45 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'processMap': the method is declared in an interface type | 46 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 242–248 |
| EmptyControlStatement | Empty else statement | 320–322 |
com/randomnoun/common/timer/HiResTimer.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 217–220 |
com/randomnoun/common/webapp/taglib/AuthCheckTag.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 87–89 |
com/randomnoun/common/webapp/taglib/ExtendJavascriptVarTag.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 222–224 |
com/randomnoun/common/webapp/taglib/SelectTag.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 57 |
| EmptyCatchBlock | Avoid empty catch blocks | 312–314 |
| EmptyCatchBlock | Avoid empty catch blocks | 426–428 |
com/randomnoun/common/webapp/taglib/SetJavascriptVarTag.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 243–245 |
com/randomnoun/common/jexl/parser/ExpressionParser.java
| Rule | Violation | Line |
|---|---|---|
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 72 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 102 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 132 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 179 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 242 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 289 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 344 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 406 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 456 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 490 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 501 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 535 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 565 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 615 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 640 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 649 |
com/randomnoun/common/jexl/parser/ExpressionParserTokenManager.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjStopStringLiteralDfa_0': private methods cannot be overridden | 15 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjStartNfa_0': private methods cannot be overridden | 58 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 440 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 500 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 502 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjCanMove_0': private methods cannot be overridden | 544 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjCanMove_1': private methods cannot be overridden | 556 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 706 |
com/randomnoun/common/jexl/parser/JavaCharStream.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryReturn | Unnecessary return statement | 150 |
com/randomnoun/common/jexl/parser/TokenMgrError.java
| Rule | Violation | Line |
|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 117 |
com/randomnoun/common/jexl/ast/Node.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 11 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 12 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 13 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 14 |
com/randomnoun/common/jexl/ast/NodeListInterface.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'addNode': the method is declared in an interface type | 12 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'elementAt': the method is declared in an interface type | 13 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'elements': the method is declared in an interface type | 14 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'size': the method is declared in an interface type | 15 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 17 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 18 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 19 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 20 |
com/randomnoun/common/jexl/visitor/GJDepthFirst.java
| Rule | Violation | Line |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 19 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 30 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 50 |
com/randomnoun/common/jexl/visitor/GJNoArguDepthFirst.java
| Rule | Violation | Line |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 19 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 30 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 50 |
com/randomnoun/common/jexl/visitor/GJNoArguVisitor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 19 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 20 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 21 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 22 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 23 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 35 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 43 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 51 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 59 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 67 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 75 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 83 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 91 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 101 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 109 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 117 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 126 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 134 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 146 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 154 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 161 |
com/randomnoun/common/jexl/visitor/GJVisitor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 19 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 20 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 21 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 22 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 23 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 35 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 43 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 51 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 59 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 67 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 75 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 83 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 91 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 101 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 109 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 117 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 126 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 134 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 146 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 154 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 161 |
com/randomnoun/common/jexl/visitor/GJVoidDepthFirst.java
| Rule | Violation | Line |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 18 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 27 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 41 |
com/randomnoun/common/jexl/visitor/GJVoidVisitor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 19 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 20 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 21 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 22 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 23 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 35 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 43 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 51 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 59 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 67 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 75 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 83 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 91 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 101 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 109 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 117 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 126 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 134 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 146 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 154 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 161 |
com/randomnoun/common/jexl/visitor/Visitor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 19 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 20 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 21 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 22 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 23 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 35 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 43 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 51 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 59 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 67 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 75 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 83 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 91 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 101 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 109 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 117 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 126 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 134 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 146 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 154 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 161 |
Priority 4
com/randomnoun/common/Base64.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `source[srcOffset] << 24`. | 521 |
| UselessParentheses | Useless parentheses around `source[srcOffset + 1] << 24`. | 522 |
| UselessParentheses | Useless parentheses around `source[srcOffset + 2] << 24`. | 523 |
| UselessParentheses | Useless parentheses around `inBuff`. | 531 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'encode3to4' is already in scope | 573 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'NO_OPTIONS' is already in scope because it is declared in an enclosing type | 573 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'encode3to4' is already in scope | 597 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'NO_OPTIONS' is already in scope because it is declared in an enclosing type | 597 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'NO_OPTIONS' is already in scope because it is declared in an enclosing type | 869 |
| UselessParentheses | Useless parentheses around `len / 3`. | 956 |
| UselessParentheses | Duplicate parentheses around `DECODABET[source[srcOffset + 3]] & 0xFF`. | 1099 |
| UselessParentheses | Useless parentheses around `outBuff`. | 1104 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'NO_OPTIONS' is already in scope because it is declared in an enclosing type | 1131 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'ENCODE' is already in scope because it is declared in an enclosing type | 1423 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'DECODE' is already in scope because it is declared in an enclosing type | 1455 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'DECODE' is already in scope because it is declared in an enclosing type | 1507 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'ENCODE' is already in scope because it is declared in an enclosing type | 1561 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'PREFERRED_ENCODING' is already in scope because it is declared in an enclosing type | 1569 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'encodeFromFile' is already in scope | 1593 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'decodeFromFile' is already in scope | 1621 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'decode4to3' is already in scope | 1966 |
com/randomnoun/common/ExceptionUtil.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `highlight != HIGHLIGHT_NONE && isHighlighted(st...`. | 251 |
com/randomnoun/common/PropertyParser.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util.Map': 'Entry' is already in scope because it is imported in this file | 331 |
com/randomnoun/common/Struct.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Struct': 'getValue' is already in scope | 986 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Struct': 'getValue' is already in scope | 1128 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 1565 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 1568 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 1709 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 1715 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 1842 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Struct': 'toDate' is already in scope | 1845 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 1845 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 1947 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Struct': 'toDate' is already in scope | 1953 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 1953 |
com/randomnoun/common/Text.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `text == null || text.equals("")`. | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'isNumericDecimalExp' is already in scope | 327 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 337 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'parseCsv' is already in scope | 695 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 707 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 768 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 769 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 770 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 771 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 786 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 787 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 788 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 789 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 817 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 818 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 819 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'unescapePathComponent' is already in scope | 890 |
| UselessParentheses | Useless parentheses around `(iLen + 2) / 3`. | 1400 |
com/randomnoun/common/XmlUtil.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 108 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 110 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 112 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 114 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 143 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 145 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 147 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 149 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 191 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 193 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 195 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 197 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 474 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 479 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 480 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 485 |
com/randomnoun/common/db/DatabaseReader.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.lang': 'String' is already in scope because it is declared in java.lang | 101 |
com/randomnoun/common/email/EmailWrapper.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `string == null || "".equals(string)`. | 54 |
com/randomnoun/common/jexl/DateSpan.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `this.fixedTime != null`. | 173 |
| UselessParentheses | Useless parentheses around `(day < 10 ? "0" : "") + day`. | 274 |
| UselessParentheses | Useless parentheses around `(month < 10 ? "0" : "") + month`. | 274 |
com/randomnoun/common/jexl/eval/Evaluator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `b == null ? 0 : ((Long) b).longValue()`. | 71 |
| UselessParentheses | Useless parentheses around `b == null ? 0 : ((Double) b).doubleValue()`. | 81 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.text': 'ParseException' is already in scope because it is imported in this file | 107 |
com/randomnoun/common/security/Permission.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unnecessary import from the current package 'com.randomnoun.common.security.ResourceCriteria' | 9 |
| UnnecessaryImport | Unnecessary import from the current package 'com.randomnoun.common.security.User' | 10 |
com/randomnoun/common/security/SecurityAuthenticator.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unnecessary import from the current package 'com.randomnoun.common.security.User' | 10 |
com/randomnoun/common/security/SecurityLoader.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unnecessary import from the current package 'com.randomnoun.common.security.User' | 10 |
com/randomnoun/common/security/User.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'com.randomnoun.common.security.User' | 10 |
| UselessParentheses | Useless parentheses around `(this.customerId == otherUser.getCustomerId()) ...`. | 144–145 |
| UselessParentheses | Useless parentheses around `this.username.equals(otherUser.getUsername())`. | 145 |
com/randomnoun/common/security/impl/NullSecurityLoaderImpl.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'com.randomnoun.common.security.impl.NullSecurityLoaderImpl' | 15 |
com/randomnoun/common/security/impl/SpringSecurityAuthenticatorImpl.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'com.randomnoun.common.security.impl.SpringSecurityAuthenticatorImpl' | 17 |
com/randomnoun/common/security/impl/SpringSecurityLoaderImpl.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unnecessary import from the current package 'com.randomnoun.common.security.impl.ResourceCriteriaImpl' | 22 |
| UnnecessaryImport | Unused import 'com.randomnoun.common.security.impl.SpringSecurityLoaderImpl' | 23 |
com/randomnoun/common/spring/ClobRowMapper.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'com.randomnoun.common.spring.ClobRowMapper' | 22 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.sql': 'Clob' is already in scope because it is imported in this file | 133 |
com/randomnoun/common/spring/StringRowMapper.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.sql': 'Clob' is already in scope because it is imported in this file | 66 |
com/randomnoun/common/timer/Benchmark.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `benchmarkA.startTime > benchmarkB.startTime ? 1...`. | 79 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Benchmark': 'dateFormatText' is already in scope because it is declared in an enclosing type | 120 |
com/randomnoun/common/timer/HiResTimer.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'com.randomnoun.common.timer.HiResTimer' | 9 |
| UselessParentheses | Useless parentheses around `(timestamp2 - timestamp1) * 1000`. | 148 |
| UselessParentheses | Useless parentheses around `(timestamp2 - timestamp1) * 1000000`. | 159 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'isNativeTimerAvailable' is already in scope | 173 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'isHiResTimerAvailable' is already in scope | 175 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getFrequency' is already in scope | 179 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getTimestamp' is already in scope | 189 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getTimestamp' is already in scope | 193 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getElapsedMillis' is already in scope | 196 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getElapsedNanos' is already in scope | 198 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getTimestamp' is already in scope | 200 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getTimestamp' is already in scope | 202 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getElapsedMillis' is already in scope | 204 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getElapsedNanos' is already in scope | 206 |
com/randomnoun/common/webapp/taglib/AuthCheckTag.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 50 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::SKIP_BODY' is already in scope because it is inherited by an enclosing type | 60 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 76 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::EVAL_PAGE' is already in scope because it is inherited by an enclosing type | 80 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::SKIP_PAGE' is already in scope because it is inherited by an enclosing type | 94 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::SKIP_PAGE' is already in scope because it is inherited by an enclosing type | 97 |
com/randomnoun/common/webapp/taglib/ExtendJavascriptVarTag.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 166 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 205 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 209 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::SKIP_BODY' is already in scope because it is inherited by an enclosing type | 230 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 241 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::EVAL_PAGE' is already in scope because it is inherited by an enclosing type | 247 |
com/randomnoun/common/webapp/taglib/SelectTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `Struct.getValue(obj, displayColumn)`. | 375 |
com/randomnoun/common/webapp/taglib/SetJavascriptVarTag.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 200 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 226 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 230 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::SKIP_BODY' is already in scope because it is inherited by an enclosing type | 251 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 262 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::EVAL_PAGE' is already in scope because it is inherited by an enclosing type | 268 |
com/randomnoun/common/jexl/parser/ExpressionParser.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'java.util.Vector' | 6 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'FileInputStream' is already in scope because it is imported in this file | 36 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'FileNotFoundException' is already in scope because it is imported in this file | 38 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 713 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 717 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'UnsupportedEncodingException' is already in scope because it is imported in this file | 718 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 725 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 729 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'UnsupportedEncodingException' is already in scope because it is imported in this file | 730 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'Reader' is already in scope because it is imported in this file | 737 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'Reader' is already in scope because it is imported in this file | 745 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.lang': 'Error' is already in scope because it is declared in java.lang | 778 |
| UselessParentheses | Useless parentheses around `jj_ntk = (token.next = token_source.getNextToke...`. | 817 |
| UselessParentheses | Useless parentheses around `jj_ntk = jj_nt.kind`. | 819 |
com/randomnoun/common/jexl/parser/ExpressionParserTokenManager.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'java.io.*' | 3 |
| UnnecessaryImport | Unused import 'com.randomnoun.common.jexl.ast.*' | 4 |
| UnnecessaryImport | Unused import 'java.util.Vector' | 5 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'PrintStream' is already in scope because it is imported in this file | 12 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'PrintStream' is already in scope because it is imported in this file | 14 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 120 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 157 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 160 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 177 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 180 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 203 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 206 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 226 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 537 |
| UselessParentheses | Useless parentheses around `(jjbitVec2[i2] & l2) != 0L`. | 549 |
| UselessParentheses | Useless parentheses around `(jjbitVec4[i2] & l2) != 0L`. | 561 |
| UselessParentheses | Useless parentheses around `(jjbitVec5[i2] & l2) != 0L`. | 563 |
| UselessParentheses | Useless parentheses around `(jjbitVec6[i2] & l2) != 0L`. | 565 |
| UselessParentheses | Useless parentheses around `(jjbitVec7[i2] & l2) != 0L`. | 567 |
| UselessParentheses | Useless parentheses around `(jjbitVec8[i2] & l2) != 0L`. | 569 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 687 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 698 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 721 |
com/randomnoun/common/jexl/parser/JavaCharStream.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `bufsize = 2048`. | 131 |
| UselessParentheses | Useless parentheses around `column = 1`. | 222 |
| UselessParentheses | Useless parentheses around `column = 1`. | 232 |
com/randomnoun/common/jexl/parser/TokenMgrError.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `"Lexical error at line " + errorLine + ", colum...`. | 101–105 |
| UselessParentheses | Useless parentheses around `"\"" + addEscapes(String.valueOf(curChar)) + "\""`. | 104 |
com/randomnoun/common/jexl/ast/NodeToken.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'NoSuchElementException' is already in scope because it is imported in this file | 28 |
com/randomnoun/common/jexl/visitor/GJNoArguVisitor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'java.util.*' | 7 |
com/randomnoun/common/jexl/visitor/GJVisitor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'java.util.*' | 7 |
com/randomnoun/common/jexl/visitor/GJVoidVisitor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'java.util.*' | 7 |
com/randomnoun/common/jexl/visitor/Visitor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'java.util.*' | 7 |
Files
com/randomnoun/common/Base64.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getAlphabet': private methods cannot be overridden | 3 | 421 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getDecodabet': private methods cannot be overridden | 3 | 439 |
| UselessParentheses | Useless parentheses around `source[srcOffset] << 24`. | 4 | 521 |
| UselessParentheses | Useless parentheses around `source[srcOffset + 1] << 24`. | 4 | 522 |
| UselessParentheses | Useless parentheses around `source[srcOffset + 2] << 24`. | 4 | 523 |
| UselessParentheses | Useless parentheses around `inBuff`. | 4 | 531 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'encode3to4' is already in scope | 4 | 573 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'NO_OPTIONS' is already in scope because it is declared in an enclosing type | 4 | 573 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'encode3to4' is already in scope | 4 | 597 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'NO_OPTIONS' is already in scope because it is declared in an enclosing type | 4 | 597 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 695 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 696 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 697 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 698 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'NO_OPTIONS' is already in scope because it is declared in an enclosing type | 4 | 869 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 937 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 938 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 939 |
| UselessParentheses | Useless parentheses around `len / 3`. | 4 | 956 |
| UselessParentheses | Duplicate parentheses around `DECODABET[source[srcOffset + 3]] & 0xFF`. | 4 | 1099 |
| UselessParentheses | Useless parentheses around `outBuff`. | 4 | 1104 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'NO_OPTIONS' is already in scope because it is declared in an enclosing type | 4 | 1131 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1298 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1299 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1300 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1390 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1391 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'ENCODE' is already in scope because it is declared in an enclosing type | 4 | 1423 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1430 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'DECODE' is already in scope because it is declared in an enclosing type | 4 | 1455 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1462 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'DECODE' is already in scope because it is declared in an enclosing type | 4 | 1507 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1523 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'ENCODE' is already in scope because it is declared in an enclosing type | 4 | 1561 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'PREFERRED_ENCODING' is already in scope because it is declared in an enclosing type | 4 | 1569 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1576 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'encodeFromFile' is already in scope | 4 | 1593 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1605 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'decodeFromFile' is already in scope | 4 | 1621 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1633 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Base64': 'decode4to3' is already in scope | 4 | 1966 |
com/randomnoun/common/ErrorList.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 3 | 112 |
| UselessOverridingMethod | Overriding method merely calls super | 3 | 117 |
com/randomnoun/common/ExceptionUtil.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 122–124 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 124–126 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 126–128 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 128–130 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 130–132 |
| UselessParentheses | Useless parentheses around `highlight != HIGHLIGHT_NONE && isHighlighted(st...`. | 4 | 251 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 302–304 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 304–305 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 416–418 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 474–476 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 511–513 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 578–580 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 605–607 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 607–609 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 609–611 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 611–613 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 613–615 |
com/randomnoun/common/IniFile.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 52–70 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 77–79 |
com/randomnoun/common/Keyboard.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–292 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 152 |
com/randomnoun/common/MRUCache.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'RetrievalCallback': member interfaces are implicitly static | 3 | 81 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'get': the method is declared in an interface type | 3 | 89 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 259–263 |
com/randomnoun/common/PropertyParser.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util.Map': 'Entry' is already in scope because it is imported in this file | 4 | 331 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 446–448 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 513–514 |
com/randomnoun/common/Registry.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryReturn | Unnecessary return statement | 3 | 102 |
com/randomnoun/common/ResourceFinder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'preProcess': the method is declared in an interface type | 3 | 311 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'postProcess': the method is declared in an interface type | 3 | 329 |
| EmptyControlStatement | Empty while statement | 3 | 371 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 3 | 478 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 3 | 516 |
| UnnecessaryReturn | Unnecessary return statement | 3 | 720 |
| EmptyControlStatement | Empty if statement | 3 | 726–754 |
| UnnecessaryReturn | Unnecessary return statement | 3 | 755 |
| EmptyControlStatement | Empty if statement | 3 | 818–853 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 3 | 1170 |
com/randomnoun/common/Struct.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ToStringReturnsJson': member interfaces are implicitly static | 3 | 76 |
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ToJson': member interfaces are implicitly static | 3 | 79 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'toJson': the method is declared in an interface type | 3 | 80 |
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ToJsonFormat': member interfaces are implicitly static | 3 | 87 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'toJson': the method is declared in an interface type | 3 | 88 |
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'WriteJsonFormat': member interfaces are implicitly static | 3 | 95 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'writeJsonFormat': the method is declared in an interface type | 3 | 96 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 119 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 3 | 502 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 3 | 502 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 3 | 504 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 3 | 504 |
| PrimitiveWrapperInstantiation | Do not use `new Double(...)`, prefer `Double.valueOf(...)` | 3 | 506 |
| PrimitiveWrapperInstantiation | Do not use `new Double(...)`, prefer `Double.valueOf(...)` | 3 | 506 |
| PrimitiveWrapperInstantiation | Do not use `new Float(...)`, prefer `Float.valueOf(...)` | 3 | 508 |
| PrimitiveWrapperInstantiation | Do not use `new Float(...)`, prefer `Float.valueOf(...)` | 3 | 508 |
| PrimitiveWrapperInstantiation | Do not use `new Short(...)`, prefer `Short.valueOf(...)` | 3 | 510 |
| PrimitiveWrapperInstantiation | Do not use `new Short(...)`, prefer `Short.valueOf(...)` | 3 | 510 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 3 | 514 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 3 | 516 |
| PrimitiveWrapperInstantiation | Do not use `new Double(...)`, prefer `Double.valueOf(...)` | 3 | 518 |
| PrimitiveWrapperInstantiation | Do not use `new Float(...)`, prefer `Float.valueOf(...)` | 3 | 520 |
| PrimitiveWrapperInstantiation | Do not use `new Short(...)`, prefer `Short.valueOf(...)` | 3 | 522 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 3 | 586 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 3 | 588 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Struct': 'getValue' is already in scope | 4 | 986 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1091–1093 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Struct': 'getValue' is already in scope | 4 | 1128 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 1565 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 1568 |
| UnusedLocalVariable | Avoid unused local variables such as 'entry'. | 3 | 1637 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 1709 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 1715 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 1842 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Struct': 'toDate' is already in scope | 4 | 1845 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 1845 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 1947 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Struct': 'toDate' is already in scope | 4 | 1953 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 1953 |
com/randomnoun/common/Text.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `text == null || text.equals("")`. | 4 | 53 |
| EmptyControlStatement | Empty if statement | 3 | 131–137 |
| EmptyControlStatement | Empty if statement | 3 | 133–137 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'isNumericDecimalExp' is already in scope | 4 | 327 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 337 |
| EmptyControlStatement | Empty else statement | 3 | 432–434 |
| EmptyControlStatement | Empty if statement | 3 | 482–490 |
| EmptyControlStatement | Empty else statement | 3 | 574–576 |
| EmptyControlStatement | Empty if statement | 3 | 643–652 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'parseCsv' is already in scope | 4 | 695 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 707 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 768 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 769 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 770 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 771 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 786 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 787 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 788 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 789 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 817 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 818 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'replaceString' is already in scope | 4 | 819 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Text': 'unescapePathComponent' is already in scope | 4 | 890 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'encodeUrl': private methods cannot be overridden | 3 | 1343 |
| UselessParentheses | Useless parentheses around `(iLen + 2) / 3`. | 4 | 1400 |
com/randomnoun/common/XmlUtil.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 108 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 110 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 112 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 114 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 143 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 145 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 147 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 149 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 191 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 193 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 195 |
| EmptyControlStatement | Empty if statement | 3 | 197–199 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 197 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 474 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 479 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 480 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.w3c.dom': 'Node' is already in scope because it is imported in this file | 4 | 485 |
com/randomnoun/common/db/DatabaseReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.lang': 'String' is already in scope because it is declared in java.lang | 4 | 101 |
com/randomnoun/common/db/SqlParser.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Consumer': member interfaces are implicitly static | 3 | 85 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'consume': the method is declared in an interface type | 3 | 86 |
| UnusedLocalVariable | Avoid unused local variables such as 'snc'. | 3 | 123 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 264–267 |
com/randomnoun/common/db/dao/MysqlDatabaseReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 145 |
com/randomnoun/common/db/to/TableTO.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 28 |
com/randomnoun/common/email/EmailWrapper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `string == null || "".equals(string)`. | 4 | 54 |
| AvoidUsingHardCodedIP | Do not hard code the IP address ${variableName} | 3 | 287 |
com/randomnoun/common/jexl/DateSpan.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `this.fixedTime != null`. | 4 | 173 |
| UselessParentheses | Useless parentheses around `(day < 10 ? "0" : "") + day`. | 4 | 274 |
| UselessParentheses | Useless parentheses around `(month < 10 ? "0" : "") + month`. | 4 | 274 |
| OverrideBothEqualsAndHashcode | Ensure you override both equals() and hashCode() | 3 | 368 |
com/randomnoun/common/jexl/eval/EvalFunction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifiers 'public static' on field 'logger': the field is declared in an interface type | 3 | 35 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'evaluate': the method is declared in an interface type | 3 | 53 |
com/randomnoun/common/jexl/eval/Evaluator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `b == null ? 0 : ((Long) b).longValue()`. | 4 | 71 |
| UselessParentheses | Useless parentheses around `b == null ? 0 : ((Double) b).doubleValue()`. | 4 | 81 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 102–110 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.text': 'ParseException' is already in scope because it is imported in this file | 4 | 107 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 114–116 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 120–122 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 126–129 |
| PrimitiveWrapperInstantiation | Do not use `Boolean.valueOf(true)`, prefer `Boolean.TRUE` | 3 | 686 |
| PrimitiveWrapperInstantiation | Do not use `Boolean.valueOf(false)`, prefer `Boolean.FALSE` | 3 | 688 |
com/randomnoun/common/jexl/sql/SqlFunction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifiers 'public abstract' on method 'toSql': the method is declared in an interface type | 3 | 33 |
com/randomnoun/common/jexl/sql/SqlGenerator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'ops'. | 3 | 674 |
| UnusedLocalVariable | Avoid unused local variables such as 'ops'. | 3 | 714 |
| UnusedLocalVariable | Avoid unused local variables such as 'ops'. | 3 | 759 |
| UnusedLocalVariable | Avoid unused local variables such as 'ops'. | 3 | 798 |
| PrimitiveWrapperInstantiation | Do not use `Boolean.valueOf(true)`, prefer `Boolean.TRUE` | 3 | 1075 |
| PrimitiveWrapperInstantiation | Do not use `Boolean.valueOf(false)`, prefer `Boolean.FALSE` | 3 | 1077 |
com/randomnoun/common/log4j2/LoggingOutputStream.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 43–56 |
com/randomnoun/common/security/Permission.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unnecessary import from the current package 'com.randomnoun.common.security.ResourceCriteria' | 4 | 9 |
| UnnecessaryImport | Unnecessary import from the current package 'com.randomnoun.common.security.User' | 4 | 10 |
com/randomnoun/common/security/SecurityAuthenticator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unnecessary import from the current package 'com.randomnoun.common.security.User' | 4 | 10 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'initialise': the method is declared in an interface type | 3 | 29 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'authenticate': the method is declared in an interface type | 3 | 41 |
com/randomnoun/common/security/SecurityLoader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unnecessary import from the current package 'com.randomnoun.common.security.User' | 4 | 10 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'initialise': the method is declared in an interface type | 3 | 31 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllRolePermissions': the method is declared in an interface type | 3 | 42 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadRolePermissions': the method is declared in an interface type | 3 | 48 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadUserRolePermissions': the method is declared in an interface type | 3 | 60 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadUserPermissions': the method is declared in an interface type | 3 | 66 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadUserRoles': the method is declared in an interface type | 3 | 71 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadUser': the method is declared in an interface type | 3 | 80 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllUsers': the method is declared in an interface type | 3 | 92 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllResources': the method is declared in an interface type | 3 | 104 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllActivities': the method is declared in an interface type | 3 | 118 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllRoles': the method is declared in an interface type | 3 | 130 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllPermissions': the method is declared in an interface type | 3 | 142 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllRoleDetails': the method is declared in an interface type | 3 | 153 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'loadAllUserDetails': the method is declared in an interface type | 3 | 162 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'resetSecurityContext': the method is declared in an interface type | 3 | 166 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'saveUserRolesAndPermissions': the method is declared in an interface type | 3 | 175 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'saveRolePermissions': the method is declared in an interface type | 3 | 185 |
com/randomnoun/common/security/User.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'com.randomnoun.common.security.User' | 4 | 10 |
| UselessParentheses | Useless parentheses around `(this.customerId == otherUser.getCustomerId()) ...`. | 4 | 144–145 |
| UselessParentheses | Useless parentheses around `this.username.equals(otherUser.getUsername())`. | 4 | 145 |
com/randomnoun/common/security/impl/NullSecurityLoaderImpl.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'com.randomnoun.common.security.impl.NullSecurityLoaderImpl' | 4 | 15 |
com/randomnoun/common/security/impl/SpringSecurityAuthenticatorImpl.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'com.randomnoun.common.security.impl.SpringSecurityAuthenticatorImpl' | 4 | 17 |
com/randomnoun/common/security/impl/SpringSecurityLoaderImpl.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unnecessary import from the current package 'com.randomnoun.common.security.impl.ResourceCriteriaImpl' | 4 | 22 |
| UnnecessaryImport | Unused import 'com.randomnoun.common.security.impl.SpringSecurityLoaderImpl' | 4 | 23 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 3 | 206 |
| PrimitiveWrapperInstantiation | Do not use `new Long(...)`, prefer `Long.valueOf(...)` | 3 | 208 |
com/randomnoun/common/spring/ClobRowMapper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'com.randomnoun.common.spring.ClobRowMapper' | 4 | 22 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.sql': 'Clob' is already in scope because it is imported in this file | 4 | 133 |
com/randomnoun/common/spring/SelectFromResultSetExtractor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getCte': the method is declared in an interface type | 3 | 13 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getSelect': the method is declared in an interface type | 3 | 15 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getFrom': the method is declared in an interface type | 3 | 17 |
com/randomnoun/common/spring/SelectFromRowMapper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getCte': the method is declared in an interface type | 3 | 15 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getSelect': the method is declared in an interface type | 3 | 17 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getFrom': the method is declared in an interface type | 3 | 19 |
com/randomnoun/common/spring/StringRowMapper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 60–79 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.sql': 'Clob' is already in scope because it is imported in this file | 4 | 66 |
com/randomnoun/common/spring/StructuredListResultSetExtractor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 376–382 |
com/randomnoun/common/spring/StructuredMapCallbackHandlerResultSetExtractor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'StructuredMapCallbackHandler': member interfaces are implicitly static | 3 | 45 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'processMap': the method is declared in an interface type | 3 | 46 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 242–248 |
| EmptyControlStatement | Empty else statement | 3 | 320–322 |
com/randomnoun/common/timer/Benchmark.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `benchmarkA.startTime > benchmarkB.startTime ? 1...`. | 4 | 79 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Benchmark': 'dateFormatText' is already in scope because it is declared in an enclosing type | 4 | 120 |
com/randomnoun/common/timer/HiResTimer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'com.randomnoun.common.timer.HiResTimer' | 4 | 9 |
| UselessParentheses | Useless parentheses around `(timestamp2 - timestamp1) * 1000`. | 4 | 148 |
| UselessParentheses | Useless parentheses around `(timestamp2 - timestamp1) * 1000000`. | 4 | 159 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'isNativeTimerAvailable' is already in scope | 4 | 173 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'isHiResTimerAvailable' is already in scope | 4 | 175 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getFrequency' is already in scope | 4 | 179 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getTimestamp' is already in scope | 4 | 189 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getTimestamp' is already in scope | 4 | 193 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getElapsedMillis' is already in scope | 4 | 196 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getElapsedNanos' is already in scope | 4 | 198 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getTimestamp' is already in scope | 4 | 200 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getTimestamp' is already in scope | 4 | 202 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getElapsedMillis' is already in scope | 4 | 204 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'HiResTimer': 'getElapsedNanos' is already in scope | 4 | 206 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 217–220 |
com/randomnoun/common/webapp/taglib/AuthCheckTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 4 | 50 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::SKIP_BODY' is already in scope because it is inherited by an enclosing type | 4 | 60 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 4 | 76 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::EVAL_PAGE' is already in scope because it is inherited by an enclosing type | 4 | 80 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 87–89 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::SKIP_PAGE' is already in scope because it is inherited by an enclosing type | 4 | 94 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::SKIP_PAGE' is already in scope because it is inherited by an enclosing type | 4 | 97 |
com/randomnoun/common/webapp/taglib/ExtendJavascriptVarTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 4 | 166 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 205 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 209 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 222–224 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::SKIP_BODY' is already in scope because it is inherited by an enclosing type | 4 | 230 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 4 | 241 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::EVAL_PAGE' is already in scope because it is inherited by an enclosing type | 4 | 247 |
com/randomnoun/common/webapp/taglib/SelectTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 3 | 57 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 312–314 |
| UselessParentheses | Useless parentheses around `Struct.getValue(obj, displayColumn)`. | 4 | 375 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 426–428 |
com/randomnoun/common/webapp/taglib/SetJavascriptVarTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 4 | 200 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 226 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Date' is already in scope because it is imported in this file | 4 | 230 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 243–245 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::SKIP_BODY' is already in scope because it is inherited by an enclosing type | 4 | 251 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'jakarta.servlet.jsp': 'JspException' is already in scope because it is imported in this file | 4 | 262 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'BodyTag': 'Tag::EVAL_PAGE' is already in scope because it is inherited by an enclosing type | 4 | 268 |
com/randomnoun/common/jexl/parser/ExpressionParser.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'java.util.Vector' | 4 | 6 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'FileInputStream' is already in scope because it is imported in this file | 4 | 36 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'FileNotFoundException' is already in scope because it is imported in this file | 4 | 38 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 72 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 102 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 132 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 179 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 242 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 289 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 344 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 406 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 456 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 490 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 501 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 535 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 565 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 615 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 640 |
| UnconditionalIfStatement | Do not use if statements that are always true or always false | 3 | 649 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 4 | 713 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 4 | 717 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'UnsupportedEncodingException' is already in scope because it is imported in this file | 4 | 718 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 4 | 725 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 4 | 729 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'UnsupportedEncodingException' is already in scope because it is imported in this file | 4 | 730 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'Reader' is already in scope because it is imported in this file | 4 | 737 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'Reader' is already in scope because it is imported in this file | 4 | 745 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.lang': 'Error' is already in scope because it is declared in java.lang | 4 | 778 |
| UselessParentheses | Useless parentheses around `jj_ntk = (token.next = token_source.getNextToke...`. | 4 | 817 |
| UselessParentheses | Useless parentheses around `jj_ntk = jj_nt.kind`. | 4 | 819 |
com/randomnoun/common/jexl/parser/ExpressionParserTokenManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'java.io.*' | 4 | 3 |
| UnnecessaryImport | Unused import 'com.randomnoun.common.jexl.ast.*' | 4 | 4 |
| UnnecessaryImport | Unused import 'java.util.Vector' | 4 | 5 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'PrintStream' is already in scope because it is imported in this file | 4 | 12 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'PrintStream' is already in scope because it is imported in this file | 4 | 14 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjStopStringLiteralDfa_0': private methods cannot be overridden | 3 | 15 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjStartNfa_0': private methods cannot be overridden | 3 | 58 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 120 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 4 | 157 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 160 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 4 | 177 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 180 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 4 | 203 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 206 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 226 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 3 | 440 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 3 | 500 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 3 | 502 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 537 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjCanMove_0': private methods cannot be overridden | 3 | 544 |
| UselessParentheses | Useless parentheses around `(jjbitVec2[i2] & l2) != 0L`. | 4 | 549 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjCanMove_1': private methods cannot be overridden | 3 | 556 |
| UselessParentheses | Useless parentheses around `(jjbitVec4[i2] & l2) != 0L`. | 4 | 561 |
| UselessParentheses | Useless parentheses around `(jjbitVec5[i2] & l2) != 0L`. | 4 | 563 |
| UselessParentheses | Useless parentheses around `(jjbitVec6[i2] & l2) != 0L`. | 4 | 565 |
| UselessParentheses | Useless parentheses around `(jjbitVec7[i2] & l2) != 0L`. | 4 | 567 |
| UselessParentheses | Useless parentheses around `(jjbitVec8[i2] & l2) != 0L`. | 4 | 569 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 687 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 698 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 3 | 706 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 721 |
com/randomnoun/common/jexl/parser/JavaCharStream.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `bufsize = 2048`. | 4 | 131 |
| UnnecessaryReturn | Unnecessary return statement | 3 | 150 |
| UselessParentheses | Useless parentheses around `column = 1`. | 4 | 222 |
| UselessParentheses | Useless parentheses around `column = 1`. | 4 | 232 |
com/randomnoun/common/jexl/parser/TokenMgrError.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `"Lexical error at line " + errorLine + ", colum...`. | 4 | 101–105 |
| UselessParentheses | Useless parentheses around `"\"" + addEscapes(String.valueOf(curChar)) + "\""`. | 4 | 104 |
| UselessOverridingMethod | Overriding method merely calls super | 3 | 117 |
com/randomnoun/common/jexl/ast/Node.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 3 | 11 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 3 | 12 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 3 | 13 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 3 | 14 |
com/randomnoun/common/jexl/ast/NodeListInterface.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'addNode': the method is declared in an interface type | 3 | 12 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'elementAt': the method is declared in an interface type | 3 | 13 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'elements': the method is declared in an interface type | 3 | 14 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'size': the method is declared in an interface type | 3 | 15 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 3 | 17 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 3 | 18 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 3 | 19 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'accept': the method is declared in an interface type | 3 | 20 |
com/randomnoun/common/jexl/ast/NodeToken.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'NoSuchElementException' is already in scope because it is imported in this file | 4 | 28 |
com/randomnoun/common/jexl/visitor/GJDepthFirst.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 3 | 19 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 3 | 30 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 3 | 50 |
com/randomnoun/common/jexl/visitor/GJNoArguDepthFirst.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 3 | 19 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 3 | 30 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 3 | 50 |
com/randomnoun/common/jexl/visitor/GJNoArguVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'java.util.*' | 4 | 7 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 19 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 20 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 21 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 22 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 23 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 35 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 43 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 51 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 59 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 67 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 75 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 83 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 91 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 101 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 109 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 117 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 126 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 134 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 146 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 154 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 161 |
com/randomnoun/common/jexl/visitor/GJVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'java.util.*' | 4 | 7 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 19 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 20 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 21 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 22 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 23 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 35 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 43 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 51 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 59 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 67 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 75 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 83 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 91 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 101 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 109 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 117 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 126 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 134 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 146 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 154 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 161 |
com/randomnoun/common/jexl/visitor/GJVoidDepthFirst.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 3 | 18 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 3 | 27 |
| UnusedLocalVariable | Avoid unused local variables such as '_count'. | 3 | 41 |
com/randomnoun/common/jexl/visitor/GJVoidVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'java.util.*' | 4 | 7 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 19 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 20 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 21 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 22 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 23 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 35 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 43 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 51 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 59 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 67 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 75 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 83 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 91 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 101 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 109 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 117 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 126 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 134 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 146 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 154 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 161 |
com/randomnoun/common/jexl/visitor/Visitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'java.util.*' | 4 | 7 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 19 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 20 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 21 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 22 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 23 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 35 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 43 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 51 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 59 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 67 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 75 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 83 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 91 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 101 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 109 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 117 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 126 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 134 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 146 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 154 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'visit': the method is declared in an interface type | 3 | 161 |