PMD Results

The following document contains the results of PMD 4.3.

Files

org/deegree/feature/persistence/sql/SQLFeatureStore.java

ViolationLine
Too many static imports may lead to messy code36 - 1559
Avoid unused method parameters such as 'filter'.823 - 886
Avoid unused method parameters such as 'sortCrit'.1075 - 1172
Avoid unused method parameters such as 'conn'.1508 - 1513
Avoid unused method parameters such as 'conn'.1515 - 1538

org/deegree/feature/persistence/sql/SQLFeatureStoreTransaction.java

ViolationLine
Avoid unused method parameters such as 'lock'.285 - 309
Avoid unused method parameters such as 'lock'.311 - 333
Avoid unused local variables such as 'fid'.561

org/deegree/feature/persistence/sql/blob/BlobCodec.java

ViolationLine
Avoid unused method parameters such as 'nsContext'.225 - 244

org/deegree/feature/persistence/sql/config/AbstractMappedSchemaBuilder.java

ViolationLine
Too many static imports may lead to messy code36 - 230

org/deegree/feature/persistence/sql/config/MappedSchemaBuilderGML.java

ViolationLine
Too many static imports may lead to messy code36 - 466
These nested if statements could be combined355 - 358

org/deegree/feature/persistence/sql/config/MappedSchemaBuilderTable.java

ViolationLine
Too many static imports may lead to messy code36 - 614
Avoid empty catch blocks482 - 484

org/deegree/feature/persistence/sql/config/MappedSchemaBuilderTableOld.java

ViolationLine
Too many static imports may lead to messy code36 - 600
Avoid empty catch blocks468 - 470

org/deegree/feature/persistence/sql/config/SQLFeatureStoreConfigWriter.java

ViolationLine
Avoid unused method parameters such as 'blobMapping'.177 - 181

org/deegree/feature/persistence/sql/converter/FeatureParticleConverter.java

ViolationLine
Avoid unused private fields such as 'valueFt'.67
Avoid unused private fields such as 'schema'.69

org/deegree/feature/persistence/sql/ddl/DDLCreator.java

ViolationLine
Avoid empty if statements219 - 221

org/deegree/feature/persistence/sql/ddl/MSSQLDDLCreator.java

ViolationLine
Avoid unused method parameters such as 'dbField'.82 - 96
Avoid unused method parameters such as 'mapping'.82 - 96
Avoid unused method parameters such as 'table'.82 - 96

org/deegree/feature/persistence/sql/ddl/OracleDDLCreator.java

ViolationLine
Avoid unused local variables such as 'schema'.128
Avoid unused local variables such as 'srid'.130

org/deegree/feature/persistence/sql/insert/InsertRow.java

ViolationLine
These nested if statements could be combined259 - 261

org/deegree/feature/persistence/sql/insert/InsertRowManager.java

ViolationLine
These nested if statements could be combined306 - 308
Avoid empty if statements402 - 404

org/deegree/feature/persistence/sql/mapper/AppSchemaMapper.java

ViolationLine
Too many static imports may lead to messy code36 - 862
These nested if statements could be combined519 - 522
Avoid unused method parameters such as 'parentCTs'.583 - 593
Avoid unused method parameters such as 'parentEls'.595 - 657
Avoid unused method parameters such as 'parentCTs'.595 - 657
Avoid unused local variables such as 'dim'.636
Avoid unused local variables such as 'srid'.638
Avoid unused method parameters such as 'maxOccurs'.659 - 696
Avoid unused method parameters such as 'mc'.814 - 825
Avoid unused method parameters such as 'wildCard'.814 - 825
Avoid unused method parameters such as 'occurrence'.814 - 825
Avoid unused method parameters such as 'parentCTs'.814 - 825
Avoid unused private methods such as 'getQName(XSTypeDefinition)'.836

org/deegree/feature/persistence/sql/mapper/XPathSchemaWalker.java

ViolationLine
Avoid empty if statements121 - 123
Avoid empty if statements207 - 209
Avoid empty if statements209 - 211

org/deegree/feature/persistence/sql/rules/FeatureBuilderRelational.java

ViolationLine
Avoid empty if statements229 - 231
Avoid unused local variables such as 'fm'.379
Avoid unused private methods such as 'getNilledAttributes(XSElementDeclaration,Map)'.592 - 593

org/deegree/feature/persistence/sql/xpath/AttrStep.java

ViolationLine
Ensure you override both equals() and hashCode()55

org/deegree/feature/persistence/sql/xpath/ElementStep.java

ViolationLine
Ensure you override both equals() and hashCode()66

org/deegree/feature/persistence/sql/xpath/MappableStep.java

ViolationLine
Avoid empty if statements82 - 84

org/deegree/feature/persistence/sql/xpath/MappedXPath.java

ViolationLine
Avoid unused method parameters such as 'steps'.232 - 234
Avoid unused method parameters such as 'remaining'.236 - 249
Avoid unused method parameters such as 'remaining'.251 - 264

org/deegree/feature/persistence/sql/xpath/TextStep.java

ViolationLine
Ensure you override both equals() and hashCode()49

org/deegree/feature/persistence/mapping/antlr/FMLLexer.java

ViolationLine
Avoid unused imports such as 'java.util.Stack'7
Avoid unused imports such as 'java.util.List'8
Avoid unused imports such as 'java.util.ArrayList'9

org/deegree/feature/persistence/mapping/antlr/FMLParser.java

ViolationLine
Avoid unused imports such as 'java.util.Collections'4
Avoid unused imports such as 'java.util.Stack'12
Avoid unused imports such as 'java.util.List'13
Avoid unused imports such as 'java.util.ArrayList'14