The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'topicVersion' doesn't match '_[a-z][a-zA-Z0-9]*' | 10 |
| FieldNamingConventions | The field name 'parentPageName' doesn't match '_[a-z][a-zA-Z0-9]*' | 11 |
| FieldNamingConventions | The field name 'topicId' doesn't match '_[a-z][a-zA-Z0-9]*' | 12 |
| FieldNamingConventions | The field name 'topicTitle' doesn't match '_[a-z][a-zA-Z0-9]*' | 13 |
| FieldNamingConventions | The field name 'topicContent' doesn't match '_[a-z][a-zA-Z0-9]*' | 15 |
| FieldNamingConventions | The field name 'language' doesn't match '_[a-z][a-zA-Z0-9]*' | 17 |
| FieldNamingConventions | The field name 'wikiHtmlContent' doesn't match '_[a-z][a-zA-Z0-9]*' | 19 |
| FieldNamingConventions | The field name 'wikiPageUrl' doesn't match '_[a-z][a-zA-Z0-9]*' | 21 |
| FieldNamingConventions | The field name 'editComment' doesn't match '_[a-z][a-zA-Z0-9]*' | 22 |
| FieldNamingConventions | The field name 'viewRole' doesn't match '_[a-z][a-zA-Z0-9]*' | 24 |
| FieldNamingConventions | The field name 'editRole' doesn't match '_[a-z][a-zA-Z0-9]*' | 26 |
| FieldNamingConventions | The field name 'topicPageName' doesn't match '_[a-z][a-zA-Z0-9]*' | 28 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'LuteceHtmlParser' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 42–168 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'SpecialChar' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 3–44 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'WikiCreoleToMarkdown' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 45–206 |
| Rule | Violation | Line |
|---|---|---|
| ReturnEmptyArrayRatherThanNull | Return an empty array rather than null. | 108–111 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 39–88 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'WikiDynamicInputs' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 58–145 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Utils' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 32–410 |
| FieldNamingConventions | The static field name 'entities' doesn't match '_[a-z][a-zA-Z0-9]*' | 34 |
| FieldNamingConventions | The constant name 'translitTable' doesn't match '[A-Z][A-Z_0-9]*' | 35 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'wikiLength' doesn't match '_[a-z][a-zA-Z0-9]*' | 63 |
| FieldNamingConventions | The field name 'wikiChars' doesn't match '_[a-z][a-zA-Z0-9]*' | 64 |
| FieldNamingConventions | The field name 'sb' doesn't match '_[a-z][a-zA-Z0-9]*' | 65 |
| FieldNamingConventions | The field name 'toc' doesn't match '_[a-z][a-zA-Z0-9]*' | 66 |
| FieldNamingConventions | The field name 'tocLevel' doesn't match '_[a-z][a-zA-Z0-9]*' | 67 |
| FieldNamingConventions | The field name 'tocAnchorIds' doesn't match '_[a-z][a-zA-Z0-9]*' | 68 |
| FieldNamingConventions | The field name 'wikiText' doesn't match '_[a-z][a-zA-Z0-9]*' | 69 |
| FieldNamingConventions | The field name 'pos' doesn't match '_[a-z][a-zA-Z0-9]*' | 70 |
| FieldNamingConventions | The field name 'listLevel' doesn't match '_[a-z][a-zA-Z0-9]*' | 71 |
| FieldNamingConventions | The field name 'listLevels' doesn't match '_[a-z][a-zA-Z0-9]*' | 72 |
| FieldNamingConventions | The field name 'blockquoteBR' doesn't match '_[a-z][a-zA-Z0-9]*' | 73 |
| FieldNamingConventions | The field name 'inTable' doesn't match '_[a-z][a-zA-Z0-9]*' | 74 |
| FieldNamingConventions | The field name 'mediawikiTableLevel' doesn't match '_[a-z][a-zA-Z0-9]*' | 75 |
| FieldNamingConventions | The field name 'HEADING_LEVEL_SHIFT' doesn't match '_[a-z][a-zA-Z0-9]*' | 76 |
| FieldNamingConventions | The field name 'HEADING_ID_PREFIX' doesn't match '_[a-z][a-zA-Z0-9]*' | 77 |
| ConstructorCallsOverridableMethod | Overridable method 'parse' called during object construction | 90 |
| ReturnEmptyArrayRatherThanNull | Return an empty array rather than null. | 647–691 |
| FieldNamingConventions | The field name 'position' doesn't match '_[a-z][a-zA-Z0-9]*' | 1243 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'strQuery' | 81 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'topic' | 60 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'htmlFromEditor' | 47 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 26 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 48 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'topic' | 997 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'text' | 64 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'wikiText' | 115 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 613 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 78 |
| Rule | Violation | Line |
|---|---|---|
| MethodReturnsInternalArray | Returning '_byValue' may expose an internal array. | 99 |
| ArrayIsStoredDirectly | The user-supplied array 'value' is stored directly. | 108 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 88–104 |
| Rule | Violation | Line |
|---|---|---|
| LinguisticNaming | Linguistics Antipattern - The setter 'setTopicWithDaoUtil' should not return any type except void linguistically | 199–208 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_mapWikiContent' could be made final; it is only initialized in the declaration or constructor. | 55 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 55 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 45–323 |
| CommentRequired | Public method and constructor comments are required | 68–79 |
| LinguisticNaming | Linguistics Antipattern - The setter 'setTopicVersionWithDaoUtil' should not return any type except void linguistically | 310–320 |
| Rule | Violation | Line |
|---|---|---|
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the removeByTopic method if you want a default access modifier | 90–93 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 90–93 |
| Rule | Violation | Line |
|---|---|---|
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 194 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 247–252 |
| Rule | Violation | Line |
|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 113–119 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 121 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 7–92 |
| DataClass | The class 'ContentDeserializer' is suspected to be a Data Class (WOC=7.692%, NOPA=12, NOAM=12, WMC=14) | 7–92 |
| CommentRequired | Public method and constructor comments are required | 82–91 |
| Rule | Violation | Line |
|---|---|---|
| AvoidDuplicateLiterals | The String literal "diff" appears 5 times in this file; the first occurrence is on line 99 | 99 |
| AvoidDuplicateLiterals | The String literal "diffreport" appears 4 times in this file; the first occurrence is on line 117 | 117 |
| AvoidDuplicateLiterals | The String literal "css" appears 4 times in this file; the first occurrence is on line 154 | 154 |
| AvoidDuplicateLiterals | The String literal "link" appears 4 times in this file; the first occurrence is on line 162 | 162 |
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'PLUGIN_NAME'. | 50 |
| Rule | Violation | Line |
|---|---|---|
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 49–107 |
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 104–107 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 46–169 |
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as '_plugin'. | 60 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 53–56 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 101 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 108–115 |
| AvoidCatchingNPE | Avoid catching NullPointerException; consider removing the cause of the NPE. | 117 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 117 |
| Rule | Violation | Line |
|---|---|---|
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 56 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–168 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 43–168 |
| CommentRequired | Public method and constructor comments are required | 47–95 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 59 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 74–94 |
| AvoidDuplicateLiterals | The String literal "div" appears 4 times in this file; the first occurrence is on line 79 | 79 |
| CommentRequired | Public method and constructor comments are required | 97–166 |
| CyclomaticComplexity | The method 'createTableOfContent(Element, String, String)' has a cyclomatic complexity of 10. | 97–166 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 120 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 148 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 148 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_strPageName' could be made final; it is only initialized in the declaration or constructor. | 67 |
| ImmutableField | Private field '_strPageUrl' could be made final; it is only initialized in the declaration or constructor. | 68 |
| ImmutableField | Private field '_strLanguage' could be made final; it is only initialized in the declaration or constructor. | 69 |
| CyclomaticComplexity | The method 'appendImage(String)' has a cyclomatic complexity of 12. | 129–196 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 134 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 135 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 136 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 137 |
| MissingBreakInSwitch | A switch statement does not contain a break | 149–161 |
| SwitchStmtsShouldHaveDefault | Switch statements should have a default label | 149–161 |
| AvoidDuplicateLiterals | The String literal "\" " appears 6 times in this file; the first occurrence is on line 164 | 164 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 166 |
| CyclomaticComplexity | The method 'appendLink(String)' has a cyclomatic complexity of 12. | 202–259 |
| InefficientEmptyStringCheck | String.trim().length() == 0 / String.trim().isEmpty() is an inefficient way to validate a blank String. | 247 |
| Rule | Violation | Line |
|---|---|---|
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3–44 |
| CommentRequired | Header comments are required | 3–44 |
| CommentRequired | Public method and constructor comments are required | 26–41 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 45–206 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 46–206 |
| CommentRequired | Public method and constructor comments are required | 48–70 |
| CommentRequired | Public method and constructor comments are required | 72–204 |
| CyclomaticComplexity | The method 'wikiCreoleToMd(String, String, String, String)' has a cyclomatic complexity of 13. | 72–204 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 72 |
| NcssCount | The method 'wikiCreoleToMd(String, String, String, String)' has a NCSS line count of 88. | 72–204 |
| ExcessiveMethodLength | Avoid really long methods. | 72–204 |
| AvoidDuplicateLiterals | The String literal "$$span" appears 8 times in this file; the first occurrence is on line 85 | 85 |
| AvoidDuplicateLiterals | The String literal "class" appears 5 times in this file; the first occurrence is on line 97 | 97 |
| ShortVariable | Avoid variables with short names like p | 111 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 117–175 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'i' | 124 |
| StringToString | Avoid calling toString() on String objects; this is unnecessary. | 132 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 133 |
| ShortVariable | Avoid variables with short names like p | 134 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'i' | 137 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 142 |
| ShortVariable | Avoid variables with short names like p | 143 |
| StringToString | Avoid calling toString() on String objects; this is unnecessary. | 160 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 161 |
| ShortVariable | Avoid variables with short names like p | 162 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 170 |
| ShortVariable | Avoid variables with short names like p | 171 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'line' | 192 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'line' | 193 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 196 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 200 |
| Rule | Violation | Line |
|---|---|---|
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 109 |
| Rule | Violation | Line |
|---|---|---|
| UncommentedEmptyMethodBody | Document empty method body | 83–84 |
| UncommentedEmptyMethodBody | Document empty method body | 309–310 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 45–84 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–1232 |
| GodClass | Possible God Class (WMC=52, ATFD=52, TCC=4.735%) | 95–1232 |
| ExcessiveClassLength | Avoid really long classes. | 95–1232 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 95–1232 |
| SingularField | Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. | 179 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 193 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 215 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 248 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 249 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 432 |
| UnusedLocalVariable | Avoid unused local variables such as 'nVersion'. | 432 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 450 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 463–472 |
| ShortVariable | Avoid variables with short names like sb | 510 |
| AssignmentInOperand | Avoid assignments in operands | 513 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 535 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'page' | 645 |
| CyclomaticComplexity | The method 'doUploadImage(HttpServletRequest)' has a cyclomatic complexity of 11. | 658–708 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 670 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 687–695 |
| EmptyStatementNotInLoop | An empty statement (semicolon) not part of a loop | 808 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 892–902 |
| CyclomaticComplexity | The method 'getPageExtendedPath(Topic, HttpServletRequest)' has a cyclomatic complexity of 11. | 997–1047 |
| CollapsibleIfStatements | These nested if statements could be combined | 1025–1029 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1097–1105 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1161–1172 |
| Rule | Violation | Line |
|---|---|---|
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 59–145 |
| CommentRequired | Public method and constructor comments are required | 62–144 |
| ShortVariable | Avoid variables with short names like sb | 64 |
| AssignmentInOperand | Avoid assignments in operands | 67 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 131 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 136 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 32–410 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 33–410 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 33–410 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'entities' | 34 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 34 |
| CommentRequired | Public method and constructor comments are required | 37–50 |
| ShortVariable | Avoid variables with short names like c | 37 |
| ShortVariable | Avoid variables with short names like c | 52 |
| CommentRequired | Public method and constructor comments are required | 52–55 |
| ShortVariable | Avoid variables with short names like sb | 77 |
| ShortVariable | Avoid variables with short names like c | 83 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 85 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'p' | 90 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 98 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 107 |
| CommentRequired | Public method and constructor comments are required | 127–172 |
| ShortVariable | Avoid variables with short names like s | 127 |
| ShortVariable | Avoid variables with short names like sb | 134 |
| ShortVariable | Avoid variables with short names like ch | 139 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 141 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 146 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 151 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 156 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 161 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 174 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 174–201 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 178 |
| CommentRequired | Public method and constructor comments are required | 203–268 |
| CyclomaticComplexity | The method 'unescapeHTML(String)' has a cyclomatic complexity of 11. | 203–268 |
| ShortVariable | Avoid variables with short names like sb | 216 |
| ShortVariable | Avoid variables with short names like c | 221 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 223 |
| ShortVariable | Avoid variables with short names like ce | 225 |
| ShortVariable | Avoid variables with short names like i1 | 226 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 230 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 232 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'i' | 254 |
| CommentRequired | Public method and constructor comments are required | 270–280 |
| ShortVariable | Avoid variables with short names like s | 270 |
| AvoidCatchingThrowable | A catch statement should never catch throwable since it includes errors. | 276 |
| CommentRequired | Public method and constructor comments are required | 282–292 |
| ShortVariable | Avoid variables with short names like s | 282 |
| AvoidCatchingThrowable | A catch statement should never catch throwable since it includes errors. | 288 |
| CommentRequired | Public method and constructor comments are required | 294–311 |
| ShortVariable | Avoid variables with short names like to | 294 |
| ShortVariable | Avoid variables with short names like i1 | 298 |
| ShortVariable | Avoid variables with short names like i2 | 299 |
| AssignmentInOperand | Avoid assignments in operands | 301 |
| ShortVariable | Avoid variables with short names like s | 313 |
| CommentRequired | Public method and constructor comments are required | 313–336 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 321 |
| AssignmentInOperand | Avoid assignments in operands | 325 |
| ShortVariable | Avoid variables with short names like a | 333 |
| CyclomaticComplexity | The method 'translit(String)' has a cyclomatic complexity of 17. | 346–389 |
| ShortVariable | Avoid variables with short names like s | 346 |
| ShortVariable | Avoid variables with short names like sb | 353 |
| ShortVariable | Avoid variables with short names like ch | 359 |
| ShortVariable | Avoid variables with short names like c | 365 |
| CommentRequired | Public method and constructor comments are required | 391–394 |
| ShortVariable | Avoid variables with short names like s | 391 |
| CommentRequired | Public method and constructor comments are required | 396–399 |
| ShortVariable | Avoid variables with short names like s | 396 |
| ShortVariable | Avoid variables with short names like s | 401 |
| CommentRequired | Public method and constructor comments are required | 401–404 |
| CommentRequired | Public method and constructor comments are required | 406–409 |
| ShortVariable | Avoid variables with short names like s | 406 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveClassLength | Avoid really long classes. | 40–1266 |
| CyclomaticComplexity | The class 'WikiParser' has a total cyclomatic complexity of 284 (highest 112). | 40–1266 |
| TooManyFields | Too many fields | 40–1266 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 41–1266 |
| AvoidDuplicateLiterals | The String literal "}}}" appears 4 times in this file; the first occurrence is on line 44 | 44 |
| AvoidStringBufferField | StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). | 65 |
| ShortVariable | Avoid variables with short names like sb | 65 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 150 characters appended. | 65 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 46 characters appended. | 66 |
| AvoidStringBufferField | StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). | 66 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'tocLevel' | 67 |
| LooseCoupling | Avoid using implementation types like 'HashSet'; use the interface instead | 68 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 68 |
| ImmutableField | Private field 'tocAnchorIds' could be made final; it is only initialized in the declaration or constructor. | 68 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'pos' | 70 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'blockquoteBR' | 73 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'inTable' | 74 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'mediawikiTableLevel' | 75 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'HEADING_ID_PREFIX' | 77 |
| CommentRequired | Protected method constructor comments are required | 82–86 |
| CommentRequired | Protected method constructor comments are required | 88–91 |
| CommentRequired | Protected method constructor comments are required | 93–96 |
| CommentRequired | Protected method constructor comments are required | 99–102 |
| CommentRequired | Protected method constructor comments are required | 105–108 |
| CommentRequired | Public method and constructor comments are required | 110–113 |
| CommentRequired | Protected method constructor comments are required | 115–133 |
| ControlStatementBraces | This statement should have braces | 124–125 |
| EmptyWhileStmt | Avoid empty while statements | 125 |
| ControlStatementBraces | This statement should have braces | 129–130 |
| AssignmentInOperand | Avoid assignments in operands | 129 |
| MissingOverride | The method 'toString()' is missing an @Override annotation. | 135–138 |
| CommentRequired | Public method and constructor comments are required | 135–138 |
| CyclomaticComplexity | The method 'parseBlock()' has a cyclomatic complexity of 81. | 155–457 |
| NPathComplexity | The method 'parseBlock()' has an NPath complexity of 6386688 | 155–457 |
| ExcessiveMethodLength | Avoid really long methods. | 155–457 |
| NcssCount | The method 'parseBlock()' has a NCSS line count of 146. | 155–457 |
| ControlStatementBraces | This statement should have braces | 157–158 |
| ShortVariable | Avoid variables with short names like c | 165 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 167 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 175 |
| ShortVariable | Avoid variables with short names like pp | 180 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 187 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 193–197 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 193 |
| ControlStatementBraces | This statement should have braces | 199–200 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 207–210 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 222 |
| ShortVariable | Avoid variables with short names like lc | 242 |
| ControlStatementBraces | This statement should have braces | 245–246 |
| ShortVariable | Avoid variables with short names like cc | 269 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 290 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 306 |
| ShortVariable | Avoid variables with short names like hc | 309 |
| ControlStatementBraces | This statement should have braces | 312–313 |
| ShortVariable | Avoid variables with short names like p | 320 |
| ControlStatementBraces | This statement should have braces | 322–323 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 341 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 353 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 358 |
| ShortVariable | Avoid variables with short names like pp | 377 |
| ControlStatementBraces | This statement should have braces | 379–380 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 384 |
| ShortVariable | Avoid variables with short names like p | 395 |
| ControlStatementBraces | This statement should have braces | 397–398 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 409 |
| CollapsibleIfStatements | These nested if statements could be combined | 412–449 |
| ShortVariable | Avoid variables with short names like nc | 414 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 436 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 441 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 452 |
| ControlStatementBraces | This statement should have braces | 480–481 |
| NPathComplexity | The method 'parseTableRow(int)' has an NPath complexity of 5762 | 531–606 |
| CyclomaticComplexity | The method 'parseTableRow(int)' has a cyclomatic complexity of 18. | 531–606 |
| ControlStatementBraces | This statement should have braces | 546–547 |
| ShortVariable | Avoid variables with short names like th | 552 |
| ControlStatementBraces | This statement should have braces | 555–556 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 567 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 587 |
| ControlStatementBraces | This statement should have braces | 615–616 |
| NPathComplexity | The method 'checkURI(int, int, int)' has an NPath complexity of 290 | 647–691 |
| ShortVariable | Avoid variables with short names like p | 647 |
| CyclomaticComplexity | The method 'checkURI(int, int, int)' has a cyclomatic complexity of 15. | 647–691 |
| ShortVariable | Avoid variables with short names like pb | 652 |
| ControlStatementBraces | This statement should have braces | 654–655 |
| ShortVariable | Avoid variables with short names like pe | 657 |
| ControlStatementBraces | This statement should have braces | 659–660 |
| ControlStatementBraces | This statement should have braces | 666–667 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'e' | 701 |
| NPathComplexity | The method 'parseItemThrow(int, String, ContextType)' has an NPath complexity of 18711842 | 705–1103 |
| ExcessiveMethodLength | Avoid really long methods. | 705–1103 |
| CyclomaticComplexity | The method 'parseItemThrow(int, String, ContextType)' has a cyclomatic complexity of 112. | 705–1103 |
| NcssCount | The method 'parseItemThrow(int, String, ContextType)' has a NCSS line count of 185. | 705–1103 |
| ShortVariable | Avoid variables with short names like tb | 707 |
| ShortVariable | Avoid variables with short names like p | 710 |
| CollapsibleIfStatements | These nested if statements could be combined | 724–729 |
| ShortVariable | Avoid variables with short names like c | 732 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 736 |
| ControlStatementBraces | This statement should have braces | 750–751 |
| ShortVariable | Avoid variables with short names like pp | 764 |
| ControlStatementBraces | This statement should have braces | 766–767 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 799 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 804 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 809 |
| CollapsibleIfStatements | These nested if statements could be combined | 812–823 |
| ShortVariable | Avoid variables with short names like pp | 814 |
| ControlStatementBraces | This statement should have braces | 816–817 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 832 |
| CollapsibleIfStatements | These nested if statements could be combined | 834–838 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 843 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 859 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 864 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 910 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 929 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 942 |
| CollapsibleIfStatements | These nested if statements could be combined | 946–960 |
| AssignmentInOperand | Avoid assignments in operands | 964 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 971 |
| ShortVariable | Avoid variables with short names like pb | 978 |
| ShortVariable | Avoid variables with short names like pe | 979 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1013 |
| CollapsibleIfStatements | These nested if statements could be combined | 1020–1064 |
| ShortVariable | Avoid variables with short names like nc | 1022 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1048 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1053 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1079 |
| CollapsibleIfStatements | These nested if statements could be combined | 1082–1091 |
| CommentRequired | Protected method constructor comments are required | 1105–1117 |
| CommentRequired | Protected method constructor comments are required | 1119–1144 |
| EmptyCatchBlock | Avoid empty catch blocks | 1128–1130 |
| CommentRequired | Protected method constructor comments are required | 1146–1170 |
| EmptyCatchBlock | Avoid empty catch blocks | 1155–1157 |
| CommentRequired | Protected method constructor comments are required | 1172–1175 |
| CommentRequired | Protected method constructor comments are required | 1177–1192 |
| ShortVariable | Avoid variables with short names like i | 1179 |
| ShortVariable | Avoid variables with short names like id | 1180 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1180 |
| CommentRequired | Protected method constructor comments are required | 1194–1216 |
| CommentRequired | Protected method constructor comments are required | 1218–1233 |
| AssignmentInOperand | Avoid assignments in operands | 1229 |
| CommentRequired | Protected method constructor comments are required | 1235–1238 |
| CommentRequired | Header comments are required | 1240–1250 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the position field if you want a default access modifier | 1243 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 1243 |
| CommentRequired | Public method and constructor comments are required | 1245–1249 |
| CommentRequired | Header comments are required | 1252–1260 |
| CommentRequired | Public method and constructor comments are required | 1256–1259 |
| CommentRequired | Enum comments are required | 1262–1265 |
| UnnecessaryModifier | Unnecessary modifier 'static' on enum 'ContextType': nested enums are implicitly static | 1262–1265 |
| Rule | Violation | Line |
|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 81–84 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 78 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MethodReturnsInternalArray | Returning '_byValue' may expose an internal array. | 3 | 99 |
| ArrayIsStoredDirectly | The user-supplied array 'value' is stored directly. | 3 | 108 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 88–104 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| LinguisticNaming | Linguistics Antipattern - The setter 'setTopicWithDaoUtil' should not return any type except void linguistically | 3 | 199–208 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_mapWikiContent' could be made final; it is only initialized in the declaration or constructor. | 3 | 55 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 55 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 45–323 |
| CommentRequired | Public method and constructor comments are required | 3 | 68–79 |
| LinguisticNaming | Linguistics Antipattern - The setter 'setTopicVersionWithDaoUtil' should not return any type except void linguistically | 3 | 310–320 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the removeByTopic method if you want a default access modifier | 3 | 90–93 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 90–93 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 194 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 247–252 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'strQuery' | 2 | 81 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 113–119 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 121 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 7–92 |
| DataClass | The class 'ContentDeserializer' is suspected to be a Data Class (WOC=7.692%, NOPA=12, NOAM=12, WMC=14) | 3 | 7–92 |
| FieldNamingConventions | The field name 'topicVersion' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 10 |
| FieldNamingConventions | The field name 'parentPageName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 11 |
| FieldNamingConventions | The field name 'topicId' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 12 |
| FieldNamingConventions | The field name 'topicTitle' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 13 |
| FieldNamingConventions | The field name 'topicContent' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 15 |
| FieldNamingConventions | The field name 'language' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 17 |
| FieldNamingConventions | The field name 'wikiHtmlContent' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 19 |
| FieldNamingConventions | The field name 'wikiPageUrl' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 21 |
| FieldNamingConventions | The field name 'editComment' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 22 |
| FieldNamingConventions | The field name 'viewRole' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 24 |
| FieldNamingConventions | The field name 'editRole' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 26 |
| FieldNamingConventions | The field name 'topicPageName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 28 |
| CommentRequired | Public method and constructor comments are required | 3 | 82–91 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidDuplicateLiterals | The String literal "diff" appears 5 times in this file; the first occurrence is on line 99 | 3 | 99 |
| AvoidDuplicateLiterals | The String literal "diffreport" appears 4 times in this file; the first occurrence is on line 117 | 3 | 117 |
| AvoidDuplicateLiterals | The String literal "css" appears 4 times in this file; the first occurrence is on line 154 | 3 | 154 |
| AvoidDuplicateLiterals | The String literal "link" appears 4 times in this file; the first occurrence is on line 162 | 3 | 162 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'PLUGIN_NAME'. | 3 | 50 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 49–107 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'topic' | 2 | 60 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 104–107 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 46–169 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 81–84 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as '_plugin'. | 3 | 60 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 53–56 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 101 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 108–115 |
| AvoidCatchingNPE | Avoid catching NullPointerException; consider removing the cause of the NPE. | 3 | 117 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 117 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 56 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'LuteceHtmlParser' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 42–168 |
| CommentRequired | Header comments are required | 3 | 42–168 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 43–168 |
| CommentRequired | Public method and constructor comments are required | 3 | 47–95 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'htmlFromEditor' | 2 | 47 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 59 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 74–94 |
| AvoidDuplicateLiterals | The String literal "div" appears 4 times in this file; the first occurrence is on line 79 | 3 | 79 |
| CommentRequired | Public method and constructor comments are required | 3 | 97–166 |
| CyclomaticComplexity | The method 'createTableOfContent(Element, String, String)' has a cyclomatic complexity of 10. | 3 | 97–166 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 120 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 148 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 148 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_strPageName' could be made final; it is only initialized in the declaration or constructor. | 3 | 67 |
| ImmutableField | Private field '_strPageUrl' could be made final; it is only initialized in the declaration or constructor. | 3 | 68 |
| ImmutableField | Private field '_strLanguage' could be made final; it is only initialized in the declaration or constructor. | 3 | 69 |
| CyclomaticComplexity | The method 'appendImage(String)' has a cyclomatic complexity of 12. | 3 | 129–196 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 134 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 135 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 136 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 137 |
| MissingBreakInSwitch | A switch statement does not contain a break | 3 | 149–161 |
| SwitchStmtsShouldHaveDefault | Switch statements should have a default label | 3 | 149–161 |
| AvoidDuplicateLiterals | The String literal "\" " appears 6 times in this file; the first occurrence is on line 164 | 3 | 164 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 166 |
| CyclomaticComplexity | The method 'appendLink(String)' has a cyclomatic complexity of 12. | 3 | 202–259 |
| InefficientEmptyStringCheck | String.trim().length() == 0 / String.trim().isEmpty() is an inefficient way to validate a blank String. | 3 | 247 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 3–44 |
| ClassNamingConventions | The utility class name 'SpecialChar' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 3–44 |
| CommentRequired | Header comments are required | 3 | 3–44 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 26 |
| CommentRequired | Public method and constructor comments are required | 3 | 26–41 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 26 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 26 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 45–206 |
| ClassNamingConventions | The utility class name 'WikiCreoleToMarkdown' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 45–206 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 46–206 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| CommentRequired | Public method and constructor comments are required | 3 | 48–70 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'str' | 2 | 48 |
| CommentRequired | Public method and constructor comments are required | 3 | 72–204 |
| CyclomaticComplexity | The method 'wikiCreoleToMd(String, String, String, String)' has a cyclomatic complexity of 13. | 3 | 72–204 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 72 |
| NcssCount | The method 'wikiCreoleToMd(String, String, String, String)' has a NCSS line count of 88. | 3 | 72–204 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 72–204 |
| AvoidDuplicateLiterals | The String literal "$$span" appears 8 times in this file; the first occurrence is on line 85 | 3 | 85 |
| AvoidDuplicateLiterals | The String literal "class" appears 5 times in this file; the first occurrence is on line 97 | 3 | 97 |
| ShortVariable | Avoid variables with short names like p | 3 | 111 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 117–175 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'i' | 3 | 124 |
| StringToString | Avoid calling toString() on String objects; this is unnecessary. | 3 | 132 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 133 |
| ShortVariable | Avoid variables with short names like p | 3 | 134 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'i' | 3 | 137 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 142 |
| ShortVariable | Avoid variables with short names like p | 3 | 143 |
| StringToString | Avoid calling toString() on String objects; this is unnecessary. | 3 | 160 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 161 |
| ShortVariable | Avoid variables with short names like p | 3 | 162 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 170 |
| ShortVariable | Avoid variables with short names like p | 3 | 171 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'line' | 3 | 192 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'line' | 3 | 193 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 196 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 200 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 109 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UncommentedEmptyMethodBody | Document empty method body | 3 | 83–84 |
| ReturnEmptyArrayRatherThanNull | Return an empty array rather than null. | 1 | 108–111 |
| UncommentedEmptyMethodBody | Document empty method body | 3 | 309–310 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 45–84 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 39–88 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–1232 |
| GodClass | Possible God Class (WMC=52, ATFD=52, TCC=4.735%) | 3 | 95–1232 |
| ExcessiveClassLength | Avoid really long classes. | 3 | 95–1232 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 95–1232 |
| SingularField | Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. | 3 | 179 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 193 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 215 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 248 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 249 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 432 |
| UnusedLocalVariable | Avoid unused local variables such as 'nVersion'. | 3 | 432 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 450 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 463–472 |
| ShortVariable | Avoid variables with short names like sb | 3 | 510 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 513 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 535 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'page' | 3 | 645 |
| CyclomaticComplexity | The method 'doUploadImage(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 658–708 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 670 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 687–695 |
| EmptyStatementNotInLoop | An empty statement (semicolon) not part of a loop | 3 | 808 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 892–902 |
| CyclomaticComplexity | The method 'getPageExtendedPath(Topic, HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 997–1047 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'topic' | 2 | 997 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1025–1029 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1097–1105 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1161–1172 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'WikiDynamicInputs' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 58–145 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 59–145 |
| CommentRequired | Public method and constructor comments are required | 3 | 62–144 |
| ShortVariable | Avoid variables with short names like sb | 3 | 64 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 67 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 131 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 136 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 32–410 |
| ClassNamingConventions | The utility class name 'Utils' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 32–410 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 33–410 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 33–410 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'entities' | 3 | 34 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 34 |
| FieldNamingConventions | The static field name 'entities' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 34 |
| FieldNamingConventions | The constant name 'translitTable' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 35 |
| CommentRequired | Public method and constructor comments are required | 3 | 37–50 |
| ShortVariable | Avoid variables with short names like c | 3 | 37 |
| ShortVariable | Avoid variables with short names like c | 3 | 52 |
| CommentRequired | Public method and constructor comments are required | 3 | 52–55 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'text' | 2 | 64 |
| ShortVariable | Avoid variables with short names like sb | 3 | 77 |
| ShortVariable | Avoid variables with short names like c | 3 | 83 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 85 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'p' | 3 | 90 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 98 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 107 |
| CommentRequired | Public method and constructor comments are required | 3 | 127–172 |
| ShortVariable | Avoid variables with short names like s | 3 | 127 |
| ShortVariable | Avoid variables with short names like sb | 3 | 134 |
| ShortVariable | Avoid variables with short names like ch | 3 | 139 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 141 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 146 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 151 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 156 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 161 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 174 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 174–201 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 178 |
| CommentRequired | Public method and constructor comments are required | 3 | 203–268 |
| CyclomaticComplexity | The method 'unescapeHTML(String)' has a cyclomatic complexity of 11. | 3 | 203–268 |
| ShortVariable | Avoid variables with short names like sb | 3 | 216 |
| ShortVariable | Avoid variables with short names like c | 3 | 221 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 223 |
| ShortVariable | Avoid variables with short names like ce | 3 | 225 |
| ShortVariable | Avoid variables with short names like i1 | 3 | 226 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 230 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 232 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'i' | 3 | 254 |
| CommentRequired | Public method and constructor comments are required | 3 | 270–280 |
| ShortVariable | Avoid variables with short names like s | 3 | 270 |
| AvoidCatchingThrowable | A catch statement should never catch throwable since it includes errors. | 3 | 276 |
| CommentRequired | Public method and constructor comments are required | 3 | 282–292 |
| ShortVariable | Avoid variables with short names like s | 3 | 282 |
| AvoidCatchingThrowable | A catch statement should never catch throwable since it includes errors. | 3 | 288 |
| CommentRequired | Public method and constructor comments are required | 3 | 294–311 |
| ShortVariable | Avoid variables with short names like to | 3 | 294 |
| ShortVariable | Avoid variables with short names like i1 | 3 | 298 |
| ShortVariable | Avoid variables with short names like i2 | 3 | 299 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 301 |
| ShortVariable | Avoid variables with short names like s | 3 | 313 |
| CommentRequired | Public method and constructor comments are required | 3 | 313–336 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 321 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 325 |
| ShortVariable | Avoid variables with short names like a | 3 | 333 |
| CyclomaticComplexity | The method 'translit(String)' has a cyclomatic complexity of 17. | 3 | 346–389 |
| ShortVariable | Avoid variables with short names like s | 3 | 346 |
| ShortVariable | Avoid variables with short names like sb | 3 | 353 |
| ShortVariable | Avoid variables with short names like ch | 3 | 359 |
| ShortVariable | Avoid variables with short names like c | 3 | 365 |
| CommentRequired | Public method and constructor comments are required | 3 | 391–394 |
| ShortVariable | Avoid variables with short names like s | 3 | 391 |
| CommentRequired | Public method and constructor comments are required | 3 | 396–399 |
| ShortVariable | Avoid variables with short names like s | 3 | 396 |
| ShortVariable | Avoid variables with short names like s | 3 | 401 |
| CommentRequired | Public method and constructor comments are required | 3 | 401–404 |
| CommentRequired | Public method and constructor comments are required | 3 | 406–409 |
| ShortVariable | Avoid variables with short names like s | 3 | 406 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveClassLength | Avoid really long classes. | 3 | 40–1266 |
| CyclomaticComplexity | The class 'WikiParser' has a total cyclomatic complexity of 284 (highest 112). | 3 | 40–1266 |
| TooManyFields | Too many fields | 3 | 40–1266 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 41–1266 |
| AvoidDuplicateLiterals | The String literal "}}}" appears 4 times in this file; the first occurrence is on line 44 | 3 | 44 |
| FieldNamingConventions | The field name 'wikiLength' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 63 |
| FieldNamingConventions | The field name 'wikiChars' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 64 |
| AvoidStringBufferField | StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). | 3 | 65 |
| ShortVariable | Avoid variables with short names like sb | 3 | 65 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 150 characters appended. | 3 | 65 |
| FieldNamingConventions | The field name 'sb' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 65 |
| FieldNamingConventions | The field name 'toc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 66 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 46 characters appended. | 3 | 66 |
| AvoidStringBufferField | StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). | 3 | 66 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'tocLevel' | 3 | 67 |
| FieldNamingConventions | The field name 'tocLevel' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 67 |
| LooseCoupling | Avoid using implementation types like 'HashSet'; use the interface instead | 3 | 68 |
| FieldNamingConventions | The field name 'tocAnchorIds' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 68 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 68 |
| ImmutableField | Private field 'tocAnchorIds' could be made final; it is only initialized in the declaration or constructor. | 3 | 68 |
| FieldNamingConventions | The field name 'wikiText' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 69 |
| FieldNamingConventions | The field name 'pos' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 70 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'pos' | 3 | 70 |
| FieldNamingConventions | The field name 'listLevel' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 71 |
| FieldNamingConventions | The field name 'listLevels' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 72 |
| FieldNamingConventions | The field name 'blockquoteBR' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 73 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'blockquoteBR' | 3 | 73 |
| FieldNamingConventions | The field name 'inTable' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 74 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'inTable' | 3 | 74 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'mediawikiTableLevel' | 3 | 75 |
| FieldNamingConventions | The field name 'mediawikiTableLevel' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 75 |
| FieldNamingConventions | The field name 'HEADING_LEVEL_SHIFT' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 76 |
| FieldNamingConventions | The field name 'HEADING_ID_PREFIX' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 77 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'HEADING_ID_PREFIX' | 3 | 77 |
| CommentRequired | Protected method constructor comments are required | 3 | 82–86 |
| CommentRequired | Protected method constructor comments are required | 3 | 88–91 |
| ConstructorCallsOverridableMethod | Overridable method 'parse' called during object construction | 1 | 90 |
| CommentRequired | Protected method constructor comments are required | 3 | 93–96 |
| CommentRequired | Protected method constructor comments are required | 3 | 99–102 |
| CommentRequired | Protected method constructor comments are required | 3 | 105–108 |
| CommentRequired | Public method and constructor comments are required | 3 | 110–113 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'wikiText' | 2 | 115 |
| CommentRequired | Protected method constructor comments are required | 3 | 115–133 |
| ControlStatementBraces | This statement should have braces | 3 | 124–125 |
| EmptyWhileStmt | Avoid empty while statements | 3 | 125 |
| ControlStatementBraces | This statement should have braces | 3 | 129–130 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 129 |
| MissingOverride | The method 'toString()' is missing an @Override annotation. | 3 | 135–138 |
| CommentRequired | Public method and constructor comments are required | 3 | 135–138 |
| CyclomaticComplexity | The method 'parseBlock()' has a cyclomatic complexity of 81. | 3 | 155–457 |
| NPathComplexity | The method 'parseBlock()' has an NPath complexity of 6386688 | 3 | 155–457 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 155–457 |
| NcssCount | The method 'parseBlock()' has a NCSS line count of 146. | 3 | 155–457 |
| ControlStatementBraces | This statement should have braces | 3 | 157–158 |
| ShortVariable | Avoid variables with short names like c | 3 | 165 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 167 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 175 |
| ShortVariable | Avoid variables with short names like pp | 3 | 180 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 187 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 193–197 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 193 |
| ControlStatementBraces | This statement should have braces | 3 | 199–200 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 207–210 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 222 |
| ShortVariable | Avoid variables with short names like lc | 3 | 242 |
| ControlStatementBraces | This statement should have braces | 3 | 245–246 |
| ShortVariable | Avoid variables with short names like cc | 3 | 269 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 290 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 306 |
| ShortVariable | Avoid variables with short names like hc | 3 | 309 |
| ControlStatementBraces | This statement should have braces | 3 | 312–313 |
| ShortVariable | Avoid variables with short names like p | 3 | 320 |
| ControlStatementBraces | This statement should have braces | 3 | 322–323 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 341 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 353 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 358 |
| ShortVariable | Avoid variables with short names like pp | 3 | 377 |
| ControlStatementBraces | This statement should have braces | 3 | 379–380 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 384 |
| ShortVariable | Avoid variables with short names like p | 3 | 395 |
| ControlStatementBraces | This statement should have braces | 3 | 397–398 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 409 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 412–449 |
| ShortVariable | Avoid variables with short names like nc | 3 | 414 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 436 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 441 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 452 |
| ControlStatementBraces | This statement should have braces | 3 | 480–481 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 2 | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 2 | 531 |
| NPathComplexity | The method 'parseTableRow(int)' has an NPath complexity of 5762 | 3 | 531–606 |
| CyclomaticComplexity | The method 'parseTableRow(int)' has a cyclomatic complexity of 18. | 3 | 531–606 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 2 | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 2 | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 2 | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 2 | 531 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 2 | 531 |
| ControlStatementBraces | This statement should have braces | 3 | 546–547 |
| ShortVariable | Avoid variables with short names like th | 3 | 552 |
| ControlStatementBraces | This statement should have braces | 3 | 555–556 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 567 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 587 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'start' | 2 | 613 |
| ControlStatementBraces | This statement should have braces | 3 | 615–616 |
| NPathComplexity | The method 'checkURI(int, int, int)' has an NPath complexity of 290 | 3 | 647–691 |
| ReturnEmptyArrayRatherThanNull | Return an empty array rather than null. | 1 | 647–691 |
| ShortVariable | Avoid variables with short names like p | 3 | 647 |
| CyclomaticComplexity | The method 'checkURI(int, int, int)' has a cyclomatic complexity of 15. | 3 | 647–691 |
| ShortVariable | Avoid variables with short names like pb | 3 | 652 |
| ControlStatementBraces | This statement should have braces | 3 | 654–655 |
| ShortVariable | Avoid variables with short names like pe | 3 | 657 |
| ControlStatementBraces | This statement should have braces | 3 | 659–660 |
| ControlStatementBraces | This statement should have braces | 3 | 666–667 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'e' | 3 | 701 |
| NPathComplexity | The method 'parseItemThrow(int, String, ContextType)' has an NPath complexity of 18711842 | 3 | 705–1103 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 705–1103 |
| CyclomaticComplexity | The method 'parseItemThrow(int, String, ContextType)' has a cyclomatic complexity of 112. | 3 | 705–1103 |
| NcssCount | The method 'parseItemThrow(int, String, ContextType)' has a NCSS line count of 185. | 3 | 705–1103 |
| ShortVariable | Avoid variables with short names like tb | 3 | 707 |
| ShortVariable | Avoid variables with short names like p | 3 | 710 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 724–729 |
| ShortVariable | Avoid variables with short names like c | 3 | 732 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 736 |
| ControlStatementBraces | This statement should have braces | 3 | 750–751 |
| ShortVariable | Avoid variables with short names like pp | 3 | 764 |
| ControlStatementBraces | This statement should have braces | 3 | 766–767 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 799 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 804 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 809 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 812–823 |
| ShortVariable | Avoid variables with short names like pp | 3 | 814 |
| ControlStatementBraces | This statement should have braces | 3 | 816–817 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 832 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 834–838 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 843 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 859 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 864 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 910 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 929 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 942 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 946–960 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 964 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 971 |
| ShortVariable | Avoid variables with short names like pb | 3 | 978 |
| ShortVariable | Avoid variables with short names like pe | 3 | 979 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1013 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1020–1064 |
| ShortVariable | Avoid variables with short names like nc | 3 | 1022 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1048 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1053 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1079 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1082–1091 |
| CommentRequired | Protected method constructor comments are required | 3 | 1105–1117 |
| CommentRequired | Protected method constructor comments are required | 3 | 1119–1144 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1128–1130 |
| CommentRequired | Protected method constructor comments are required | 3 | 1146–1170 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 1155–1157 |
| CommentRequired | Protected method constructor comments are required | 3 | 1172–1175 |
| CommentRequired | Protected method constructor comments are required | 3 | 1177–1192 |
| ShortVariable | Avoid variables with short names like i | 3 | 1179 |
| ShortVariable | Avoid variables with short names like id | 3 | 1180 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1180 |
| CommentRequired | Protected method constructor comments are required | 3 | 1194–1216 |
| CommentRequired | Protected method constructor comments are required | 3 | 1218–1233 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 1229 |
| CommentRequired | Protected method constructor comments are required | 3 | 1235–1238 |
| CommentRequired | Header comments are required | 3 | 1240–1250 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the position field if you want a default access modifier | 3 | 1243 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 1243 |
| FieldNamingConventions | The field name 'position' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 1243 |
| CommentRequired | Public method and constructor comments are required | 3 | 1245–1249 |
| CommentRequired | Header comments are required | 3 | 1252–1260 |
| CommentRequired | Public method and constructor comments are required | 3 | 1256–1259 |
| CommentRequired | Enum comments are required | 3 | 1262–1265 |
| UnnecessaryModifier | Unnecessary modifier 'static' on enum 'ContextType': nested enums are implicitly static | 3 | 1262–1265 |
