PMD Results

The following document contains the results of PMD 4.2.2.

Files

KS Contract Documentation Generator - org/kuali/student/contract/model/SearchRow.java

ViolationLine
Overriding method merely calls super355 - 358

KS Contract Documentation Generator - org/kuali/student/contract/model/TypeStateCaseConstraint.java

ViolationLine
No need to import a type that lives in the same package18

KS Contract Documentation Generator - org/kuali/student/contract/model/TypeStateConstraint.java

ViolationLine
No need to import a type that lives in the same package18

KS Contract Documentation Generator - org/kuali/student/contract/model/impl/ServiceContractModelPescXsdLoader.java

ViolationLine
Avoid unused local variables such as 'schemas'.147
Avoid unused method parameters such as 'ct'.181 - 186
Avoid unused local variables such as 'found'.244
Avoid unused method parameters such as 'element'.336 - 353
Avoid unused private methods such as 'getFacetValue(XSElementDecl,String)'.395

KS Contract Documentation Generator - org/kuali/student/contract/model/impl/ServiceContractModelQDoxLoader.java

ViolationLine
Avoid unused private methods such as 'dump(DocletTag)'.125
These nested if statements could be combined297 - 299
These nested if statements could be combined309 - 311
These nested if statements could be combined365 - 367
These nested if statements could be combined544 - 548
These nested if statements could be combined606 - 612
These nested if statements could be combined607 - 611
These nested if statements could be combined608 - 610
Avoid unnecessary return statements625
These nested if statements could be combined724 - 726
Avoid unused method parameters such as 'beanField'.742 - 755
Avoid unused method parameters such as 'setterMethod'.742 - 755
Avoid unused method parameters such as 'beanField'.757 - 766
Avoid unused method parameters such as 'setterMethod'.757 - 766
Avoid unused method parameters such as 'beanField'.800 - 809
Avoid unused method parameters such as 'setterMethod'.800 - 809
These nested if statements could be combined1211 - 1213
Avoid unused method parameters such as 'mainClass'.1271 - 1281
Avoid unused method parameters such as 'shortName'.1271 - 1281
Avoid unused method parameters such as 'setterMethod'.1271 - 1281
Avoid unused method parameters such as 'mainClass'.1310 - 1321
Avoid unused method parameters such as 'shortName'.1310 - 1321
Avoid unused method parameters such as 'setterMethod'.1310 - 1321
Avoid unused private methods such as 'calcTypeOfSetterMethodFirstParam(JavaMethod)'.1328
Avoid unused private methods such as 'calcRealJavaClassOfSetterFirstParam(JavaMethod)'.1401

KS Contract Documentation Generator - org/kuali/student/contract/model/util/DateUtility.java

ViolationLine
Avoid unused imports such as 'org.kuali.student.contract.exception.DictionaryExecutionException'23

KS Contract Documentation Generator - org/kuali/student/contract/model/util/DictionaryMainTypeExpander.java

ViolationLine
No need to import a type that lives in the same package20
Avoid unnecessary return statements59

KS Contract Documentation Generator - org/kuali/student/contract/model/util/DictionaryParentSetter.java

ViolationLine
These nested if statements could be combined83 - 86

KS Contract Documentation Generator - org/kuali/student/contract/model/util/GroupTypeStatePatternMatcher.java

ViolationLine
These nested if statements could be combined37 - 40

KS Contract Documentation Generator - org/kuali/student/contract/model/util/HtmlContractServiceWriter.java

ViolationLine
These nested if statements could be combined162 - 164
These nested if statements could be combined168 - 170
These nested if statements could be combined176 - 178

KS Contract Documentation Generator - org/kuali/student/contract/model/util/HtmlContractWriter.java

ViolationLine
Avoid unused private fields such as 'SERVICE_NAME_COMPARATOR'.76

KS Contract Documentation Generator - org/kuali/student/contract/model/util/MessageStructureHierarchyDumper.java

ViolationLine
Avoid unused private fields such as 'model'.20
These nested if statements could be combined119 - 125

KS Contract Documentation Generator - org/kuali/student/contract/model/util/ModelFinder.java

ViolationLine
These nested if statements could be combined86 - 88
These nested if statements could be combined109 - 111
These nested if statements could be combined238 - 245
These nested if statements could be combined278 - 285
These nested if statements could be combined311 - 316
These nested if statements could be combined312 - 315
These nested if statements could be combined340 - 342
These nested if statements could be combined348 - 350
These nested if statements could be combined428 - 430
These nested if statements could be combined459 - 461

KS Contract Documentation Generator - org/kuali/student/contract/model/validation/ConstraintValidator.java

ViolationLine
Avoid unused imports such as 'java.util.List'22

KS Contract Documentation Generator - org/kuali/student/contract/model/validation/DictionaryModelValidator.java

ViolationLine
Avoid unused imports such as 'org.kuali.student.contract.model.XmlType'22

KS Contract Documentation Generator - org/kuali/student/contract/model/validation/DictionaryValidator.java

ViolationLine
These nested if statements could be combined176 - 178

KS Contract Documentation Generator - org/kuali/student/contract/model/validation/OrchObjValidator.java

ViolationLine
Avoid unused imports such as 'java.util.List'22

KS Contract Documentation Generator - org/kuali/student/contract/model/validation/OrchestrationModelValidator.java

ViolationLine
Avoid unused private fields such as 'finder'.33

KS Contract Documentation Generator - org/kuali/student/contract/model/validation/SearchTypeValidator.java

ViolationLine
These nested if statements could be combined81 - 84

KS Contract Documentation Generator - org/kuali/student/contract/model/validation/ServiceMethodValidator.java

ViolationLine
Avoid unused imports such as 'org.kuali.student.contract.model.DictionaryModel'18

KS Contract Documentation Generator - org/kuali/student/contract/model/validation/XmlTypesValidator.java

ViolationLine
Avoid unused private fields such as 'model'.31

KS Contract Documentation Generator - org/kuali/student/contract/writer/HtmlWriter.java

ViolationLine
These nested if statements could be combined74 - 77

KS Contract Documentation Generator - org/kuali/student/contract/writer/JavaClassWriter.java

ViolationLine
These nested if statements could be combined119 - 121
These nested if statements could be combined134 - 137

KS Contract Documentation Generator - org/kuali/student/contract/writer/search/SearchModelWriter.java

ViolationLine
Avoid unused local variables such as 'keys'.300

KS Contract Documentation Generator - org/kuali/student/contract/writer/service/GetterSetterNameCalculator.java

ViolationLine
These nested if statements could be combined52 - 54

KS Contract Documentation Generator - org/kuali/student/contract/writer/service/PureJavaInfcBeanWriter.java

ViolationLine
Avoid unused private fields such as 'directory'.35

KS Contract Documentation Generator - org/kuali/student/contract/writer/service/PureJavaInfcInfcWriter.java

ViolationLine
Avoid unused private fields such as 'directory'.35
Avoid unused private fields such as 'service'.37

KS Contract Documentation Generator - org/kuali/student/contract/writer/service/PureJavaInfcServiceWriter.java

ViolationLine
Avoid unused private fields such as 'directory'.36

KS Contract Documentation Generator - org/kuali/student/contract/writer/service/PureJavaInfcWriter.java

ViolationLine
These nested if statements could be combined90 - 93
Avoid unused private methods such as 'getServiceMethodErrors()'.99

KS Contract Documentation Generator - org/kuali/student/contract/writer/service/PureJavaInfcWriterForOneService.java

ViolationLine
These nested if statements could be combined88 - 90
Avoid unused private methods such as 'getXmlTypesUsedByService(List)'.96
These nested if statements could be combined128 - 130

KS Contract Documentation Generator - org/kuali/student/contract/writer/service/ServiceExceptionWriter.java

ViolationLine
Avoid unused private fields such as 'model'.30
Avoid unused private fields such as 'directory'.31
Avoid unused private fields such as 'rootPackage'.32

KS Contract Documentation Generator - org/kuali/student/datadictionary/mojo/KSDictionaryDocMojo.java

ViolationLine
These nested if statements could be combined126 - 130
Avoid unused local variables such as 'outputFileNames'.146
Avoid unused local variables such as 'endUL'.161

KS Contract Documentation Generator - org/kuali/student/datadictionary/util/Bean2DictionaryConverter.java

ViolationLine
Avoid empty if statements104 - 105
Avoid empty catch blocks204 - 206

KS Contract Documentation Generator - org/kuali/student/datadictionary/util/ComplexSubstructuresHelper.java

ViolationLine
Avoid unused imports such as 'java.lang.reflect.TypeVariable'25

KS Contract Documentation Generator - org/kuali/student/datadictionary/util/Dictionary2BeanComparer.java

ViolationLine
These nested if statements could be combined138 - 140
These nested if statements could be combined143 - 145

KS Contract Documentation Generator - org/kuali/student/datadictionary/util/DictionaryFormatter.java

ViolationLine
Avoid unnecessary return statements276
Avoid unused private methods such as 'getSortedFields()'.508
Avoid unused private methods such as 'formatAsString(List)'.522
Avoid unused method parameters such as 'ad'.554 - 559
Avoid unused method parameters such as 'ad'.579 - 585
Avoid unused private methods such as 'calcComplexSubStructureName(AttributeDefinition)'.609
Avoid unused method parameters such as 'ad'.609 - 616
Avoid unused private methods such as 'calcNotSoSimpleName(String)'.625
These nested if statements could be combined638 - 640
These nested if statements could be combined660 - 662
These nested if statements could be combined878 - 880
Avoid unused method parameters such as 'ad'.907 - 929
Avoid unused method parameters such as 'ad'.965 - 987
Avoid unused method parameters such as 'cons'.965 - 987
Avoid unused method parameters such as 'attribute'.989 - 1001
Avoid unused method parameters such as 'attribute'.1003 - 1015
Avoid unused private methods such as 'calcOverride(String,Object,Object)'.1040
Avoid unused private methods such as 'writeLink(PrintStream,String,String)'.1128

KS Contract Documentation Generator - org/kuali/student/datadictionary/util/DictionaryTesterHelper.java

ViolationLine
These nested if statements could be combined69 - 72

KS Contract Documentation Generator - org/kuali/student/datadictionary/util/DictionaryValidator.java

ViolationLine
Avoid unused private fields such as 'alreadyValidated'.39
These nested if statements could be combined95 - 97
Avoid unused private methods such as 'errorIfNotNull(List,AttributeDefinition,String,Object)'.140 - 142
Avoid unused local variables such as 'pattern'.260

KS Contract Documentation Generator - org/kuali/student/datadictionary/util/KradDictionaryCreator.java

ViolationLine
These nested if statements could be combined99 - 102
Avoid unused local variables such as 'beanName'.452
Avoid unused method parameters such as 'currentClassName'.483 - 555
Avoid unused local variables such as 'beanName'.677
Avoid unused method parameters such as 'currentClass'.699 - 711
Avoid unused private methods such as 'getMessageStructure(String)'.890