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 - 1560
Avoid unused method parameters such as 'filter'.822 - 885
Avoid unused method parameters such as 'sortCrit'.1074 - 1172
Avoid unused method parameters such as 'conn'.1509 - 1514
Avoid unused method parameters such as 'conn'.1516 - 1539

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

ViolationLine
Avoid unused method parameters such as 'lock'.285 - 311
Avoid unused method parameters such as 'lock'.313 - 335
Avoid unused local variables such as 'fid'.563

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 - 233

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 - 616
Avoid empty catch blocks484 - 486

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

ViolationLine
Too many static imports may lead to messy code36 - 602
Avoid empty catch blocks470 - 472

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

ViolationLine
Avoid unused method parameters such as 'blobMapping'.176 - 180

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 - 861
These nested if statements could be combined518 - 521
Avoid unused method parameters such as 'parentCTs'.582 - 592
Avoid unused method parameters such as 'parentEls'.594 - 656
Avoid unused method parameters such as 'parentCTs'.594 - 656
Avoid unused local variables such as 'dim'.635
Avoid unused local variables such as 'srid'.637
Avoid unused method parameters such as 'maxOccurs'.658 - 695
Avoid unused method parameters such as 'mc'.813 - 824
Avoid unused method parameters such as 'wildCard'.813 - 824
Avoid unused method parameters such as 'occurrence'.813 - 824
Avoid unused method parameters such as 'parentCTs'.813 - 824
Avoid unused private methods such as 'getQName(XSTypeDefinition)'.835

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