MyBatis
PMD Results
The following document contains the results of PMD 7.14.0.
Violations By Priority
Priority 3
org/mybatis/guice/AbstractMyBatisModule.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 16–181 |
org/mybatis/guice/MyBatisModule.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty finally clause | 109–111 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'bindBoolean': private methods cannot be overridden | 266 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'bindTypeHandler': an anonymous class cannot be extended | 552 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'bindProvidedTypeHandler': an anonymous class cannot be extended | 556 |
org/mybatis/guice/XMLMyBatisModule.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 126–128 |
org/mybatis/guice/configuration/settings/ConfigurationSetting.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'applyConfigurationSetting': the method is declared in an interface type | 21 |
org/mybatis/guice/session/SqlSessionFactoryProvider.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'configuration'. | 51 |
org/mybatis/guice/transactional/Isolation.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Isolation(TransactionIsolationLevel)': enum constructors are implicitly private | 44 |
org/mybatis/guice/transactional/Transactional.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on enum 'TxType': the enum is declared in an annotation type | 118 |
org/mybatis/guice/transactional/XASqlSessionManager.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 217–219 |
| EmptyCatchBlock | Avoid empty catch blocks | 517–518 |
Priority 4
org/mybatis/guice/AbstractMyBatisModule.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AbstractMyBatisModule': 'getClasses' is already in scope | 60 |
org/mybatis/guice/datasource/dbcp/SharedPoolDataSourceProvider.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 111 |
org/mybatis/guice/transactional/TxTransactionalMethodInterceptor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'String': 'format' is already in scope | 97 |
| UselessParentheses | Useless parentheses. | 154 |
| UselessParentheses | Useless parentheses. | 155 |
| UselessParentheses | Useless parentheses. | 160 |
org/mybatis/guice/transactional/XASqlSessionManager.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMENDRSCAN' is already in scope because it is inherited by an enclosing type | 78 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMFAIL' is already in scope because it is inherited by an enclosing type | 80 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMJOIN' is already in scope because it is inherited by an enclosing type | 82 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMNOFLAGS' is already in scope because it is inherited by an enclosing type | 84 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMONEPHASE' is already in scope because it is inherited by an enclosing type | 86 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMRESUME' is already in scope because it is inherited by an enclosing type | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMSTARTRSCAN' is already in scope because it is inherited by an enclosing type | 90 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMSUCCESS' is already in scope because it is inherited by an enclosing type | 92 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMSUSPEND' is already in scope because it is inherited by an enclosing type | 94 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMNOFLAGS' is already in scope because it is inherited by an enclosing type | 133 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMJOIN' is already in scope because it is inherited by an enclosing type | 133 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMJOIN' is already in scope because it is inherited by an enclosing type | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMNOFLAGS' is already in scope because it is inherited by an enclosing type | 159 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMSUCCESS' is already in scope because it is inherited by an enclosing type | 187 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMFAIL' is already in scope because it is inherited by an enclosing type | 187 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMFAIL' is already in scope because it is inherited by an enclosing type | 215 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'XA_OK' is already in scope because it is inherited by an enclosing type | 254 |
Files
org/mybatis/guice/AbstractMyBatisModule.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 16–181 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AbstractMyBatisModule': 'getClasses' is already in scope | 4 | 60 |
org/mybatis/guice/MyBatisModule.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty finally clause | 3 | 109–111 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'bindBoolean': private methods cannot be overridden | 3 | 266 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'bindTypeHandler': an anonymous class cannot be extended | 3 | 552 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'bindProvidedTypeHandler': an anonymous class cannot be extended | 3 | 556 |
org/mybatis/guice/XMLMyBatisModule.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 126–128 |
org/mybatis/guice/configuration/settings/ConfigurationSetting.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'applyConfigurationSetting': the method is declared in an interface type | 3 | 21 |
org/mybatis/guice/datasource/dbcp/SharedPoolDataSourceProvider.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 111 |
org/mybatis/guice/session/SqlSessionFactoryProvider.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'configuration'. | 3 | 51 |
org/mybatis/guice/transactional/Isolation.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Isolation(TransactionIsolationLevel)': enum constructors are implicitly private | 3 | 44 |
org/mybatis/guice/transactional/Transactional.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on enum 'TxType': the enum is declared in an annotation type | 3 | 118 |
org/mybatis/guice/transactional/TxTransactionalMethodInterceptor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'String': 'format' is already in scope | 4 | 97 |
| UselessParentheses | Useless parentheses. | 4 | 154 |
| UselessParentheses | Useless parentheses. | 4 | 155 |
| UselessParentheses | Useless parentheses. | 4 | 160 |
org/mybatis/guice/transactional/XASqlSessionManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMENDRSCAN' is already in scope because it is inherited by an enclosing type | 4 | 78 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMFAIL' is already in scope because it is inherited by an enclosing type | 4 | 80 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMJOIN' is already in scope because it is inherited by an enclosing type | 4 | 82 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMNOFLAGS' is already in scope because it is inherited by an enclosing type | 4 | 84 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMONEPHASE' is already in scope because it is inherited by an enclosing type | 4 | 86 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMRESUME' is already in scope because it is inherited by an enclosing type | 4 | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMSTARTRSCAN' is already in scope because it is inherited by an enclosing type | 4 | 90 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMSUCCESS' is already in scope because it is inherited by an enclosing type | 4 | 92 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMSUSPEND' is already in scope because it is inherited by an enclosing type | 4 | 94 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMNOFLAGS' is already in scope because it is inherited by an enclosing type | 4 | 133 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMJOIN' is already in scope because it is inherited by an enclosing type | 4 | 133 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMJOIN' is already in scope because it is inherited by an enclosing type | 4 | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMNOFLAGS' is already in scope because it is inherited by an enclosing type | 4 | 159 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMSUCCESS' is already in scope because it is inherited by an enclosing type | 4 | 187 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMFAIL' is already in scope because it is inherited by an enclosing type | 4 | 187 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'TMFAIL' is already in scope because it is inherited by an enclosing type | 4 | 215 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 217–219 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XAResource': 'XA_OK' is already in scope because it is inherited by an enclosing type | 4 | 254 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 517–518 |