Checkstyle Results
The following document contains the results of
Checkstyle
.
Summary
| Files | Infos | Warnings | Errors |
| 23 | 0 | 0 | 338 |
Files
org/astrogrid/portal/workflow/WKF.java
| Error | Line |
| Class WKF should be declared as final. |
24
|
| Missing a Javadoc comment. |
26
|
| Missing a Javadoc comment. |
31
|
| Line contains a tab character. |
34
|
| Line contains a tab character. |
35
|
| Missing a Javadoc comment. |
35
|
| Line contains a tab character. |
36
|
| Line contains a tab character. |
37
|
| Line contains a tab character. |
38
|
| Missing a Javadoc comment. |
45
|
| Missing a Javadoc comment. |
55
|
| Missing a Javadoc comment. |
61
|
| Missing a Javadoc comment. |
69
|
| Missing a Javadoc comment. |
87
|
| Missing a Javadoc comment. |
91
|
| Missing a Javadoc comment. |
96
|
| Parameter key should be final. |
113
|
| Parameter category should be final. |
113
|
| Missing a Javadoc comment. |
117
|
| Missing a Javadoc comment. |
118
|
| Line contains a tab character. |
130
|
org/astrogrid/portal/workflow/WorkflowException.java
| Error | Line |
| Missing a Javadoc comment. |
33
|
| Parameter message should be final. |
33
|
| Missing a Javadoc comment. |
37
|
| Parameter message should be final. |
37
|
| Parameter exception should be final. |
37
|
org/astrogrid/portal/workflow/impl/BasicWorkflowBuilder.java
| Error | Line |
| Line is longer than 132 characters. |
31
|
| Method 'createWorkflow' is not designed for extension - needs to be abstract, final or empty. |
33
|
| Parameter creds should be final. |
33
|
| Parameter name should be final. |
33
|
| Parameter description should be final. |
33
|
org/astrogrid/portal/workflow/impl/FileApplicationRegistry.java
| Error | Line |
| Missing a Javadoc comment. |
39
|
| Parameter regFile should be final. |
43
|
| Expected @param tag for 'regFile'. |
43
|
| Expected @throws tag for 'IOException'. |
43
|
| Expected @throws tag for 'CastorException'. |
43
|
| Missing a Javadoc comment. |
55
|
| Variable 'map' must be private and have accessor methods. |
55
|
| Method 'listApplications' is not designed for extension - needs to be abstract, final or empty. |
60
|
| Method 'getDescriptionFor' is not designed for extension - needs to be abstract, final or empty. |
72
|
| Parameter applicationName should be final. |
72
|
| Method 'listUIApplications' is not designed for extension - needs to be abstract, final or empty. |
82
|
org/astrogrid/portal/workflow/impl/FileWorkflowStore.java
| Error | Line |
| Unused import - org.astrogrid.portal.workflow.intf.WorkflowStore. |
16
|
| Missing a Javadoc comment. |
41
|
| Parameter baseDir should be final. |
45
|
| Expected @param tag for 'baseDir'. |
45
|
| Missing a Javadoc comment. |
52
|
| Variable 'baseDir' must be private and have accessor methods. |
52
|
| Missing a Javadoc comment. |
53
|
| Parameter acc should be final. |
53
|
| Parameter name should be final. |
53
|
| Missing a Javadoc comment. |
63
|
| Missing a Javadoc comment. |
64
|
| Missing a Javadoc comment. |
66
|
| Parameter acc should be final. |
66
|
| Parameter name should be final. |
66
|
| Missing a Javadoc comment. |
69
|
| Parameter acc should be final. |
69
|
| Parameter name should be final. |
69
|
| Line is longer than 132 characters. |
73
|
| Method 'deleteWorkflow' is not designed for extension - needs to be abstract, final or empty. |
75
|
| Parameter acc should be final. |
75
|
| Parameter name should be final. |
75
|
| Method 'readQuery' is not designed for extension - needs to be abstract, final or empty. |
82
|
| Parameter acc should be final. |
82
|
| Parameter name should be final. |
82
|
| Method 'listQueries' is not designed for extension - needs to be abstract, final or empty. |
98
|
| Parameter acc should be final. |
98
|
| Parameter dir should be final. |
103
|
| Parameter name should be final. |
103
|
| Line is longer than 132 characters. |
114
|
| Method 'readWorkflow' is not designed for extension - needs to be abstract, final or empty. |
116
|
| Parameter acc should be final. |
116
|
| Parameter name should be final. |
116
|
| Method 'listWorkflows' is not designed for extension - needs to be abstract, final or empty. |
133
|
| Parameter acc should be final. |
133
|
| Parameter dir should be final. |
138
|
| Parameter name should be final. |
138
|
| Line is longer than 132 characters. |
149
|
| Method 'saveWorkflow' is not designed for extension - needs to be abstract, final or empty. |
151
|
| Parameter acc should be final. |
151
|
| Parameter workflow should be final. |
151
|
org/astrogrid/portal/workflow/impl/JesJobExecutionService.java
| Error | Line |
| Unused import - java.net.URL. |
24
|
| Parameter endpoint should be final. |
44
|
| Expected @param tag for 'endpoint'. |
44
|
| Missing a Javadoc comment. |
47
|
| Variable 'controller' must be private and have accessor methods. |
47
|
| Method 'submitWorkflow' is not designed for extension - needs to be abstract, final or empty. |
51
|
| Parameter workflow should be final. |
51
|
| Method 'deleteJob' is not designed for extension - needs to be abstract, final or empty. |
61
|
| Parameter jobURN should be final. |
61
|
| Method 'cancelJob' is not designed for extension - needs to be abstract, final or empty. |
71
|
| Parameter jobURN should be final. |
71
|
| Method 'readJob' is not designed for extension - needs to be abstract, final or empty. |
81
|
| Parameter jobURN should be final. |
81
|
| Method 'readJobList' is not designed for extension - needs to be abstract, final or empty. |
91
|
| Parameter account should be final. |
91
|
| Method 'listJobs' is not designed for extension - needs to be abstract, final or empty. |
103
|
| Parameter account should be final. |
103
|
org/astrogrid/portal/workflow/impl/RegistryApplicationRegistry.java
| Error | Line |
| Unused import - org.astrogrid.applications.beans.v1.InterfacesType. |
14
|
| Unused import - org.astrogrid.registry.beans.resource.ResourceType. |
24
|
| Unused import - org.astrogrid.registry.beans.resource.VODescription. |
25
|
| Unused import - org.apache.axis.utils.XMLUtils. |
30
|
| Unused import - org.w3c.dom.Node. |
37
|
| Unused import - java.io.File. |
40
|
| Unused import - java.io.FileWriter. |
41
|
| Line is longer than 132 characters. |
47
|
| Missing a Javadoc comment. |
53
|
| Parameter endpoint should be final. |
68
|
| Missing a Javadoc comment. |
74
|
| Variable 'service' must be private and have accessor methods. |
74
|
| Missing a Javadoc comment. |
91
|
| 'static' modifier out of order with the JLS suggestions. |
91
|
| Method 'listApplications' is not designed for extension - needs to be abstract, final or empty. |
96
|
| Missing a Javadoc comment. |
96
|
| Method 'listUIApplications' is not designed for extension - needs to be abstract, final or empty. |
128
|
| Method 'getDescriptionFor' is not designed for extension - needs to be abstract, final or empty. |
178
|
| Parameter applicationName should be final. |
178
|
| Line is longer than 132 characters. |
185
|
org/astrogrid/portal/workflow/impl/RegistryV10ApplicationRegistry.java
| Error | Line |
| Missing a Javadoc comment. |
60
|
| Parameter endpoint should be final. |
60
|
| Missing a Javadoc comment. |
68
|
| Variable 'service' must be private and have accessor methods. |
68
|
| Missing a Javadoc comment. |
70
|
| 'static' modifier out of order with the JLS suggestions. |
70
|
| Method 'listApplications' is not designed for extension - needs to be abstract, final or empty. |
78
|
| Method 'getDescriptionFor' is not designed for extension - needs to be abstract, final or empty. |
104
|
| Parameter applicationName should be final. |
104
|
| Line is longer than 132 characters. |
117
|
| Method 'listUIApplications' is not designed for extension - needs to be abstract, final or empty. |
159
|
| Missing a Javadoc comment. |
159
|
org/astrogrid/portal/workflow/impl/VoSpaceClientWorkflowStore.java
| Error | Line |
| Unused import - org.astrogrid.community.common.exception.CommunityException. |
14
|
| Unused import - org.astrogrid.registry.RegistryException. |
21
|
| Unused import - org.exolab.castor.xml.CastorException. |
25
|
| Unused import - java.net.URISyntaxException. |
32
|
| Missing a Javadoc comment. |
46
|
| Variable 'factory' must be private and have accessor methods. |
46
|
| Method 'readWorkflow' is not designed for extension - needs to be abstract, final or empty. |
57
|
| Parameter user should be final. |
57
|
| Parameter locationToReadFrom should be final. |
57
|
| Line is longer than 132 characters. |
71
|
| Method 'readWorkflow' is not designed for extension - needs to be abstract, final or empty. |
73
|
| Parameter userId should be final. |
73
|
| Parameter password should be final. |
73
|
| Parameter locationToReadFrom should be final. |
73
|
| Line is longer than 132 characters. |
85
|
| Method 'readWorkflow' is not designed for extension - needs to be abstract, final or empty. |
87
|
| Parameter fmc should be final. |
87
|
| Parameter locationToReadFrom should be final. |
87
|
| Line is longer than 132 characters. |
98
|
| Method 'saveWorkflow' is not designed for extension - needs to be abstract, final or empty. |
100
|
| Parameter user should be final. |
100
|
| Parameter locationToSaveTo should be final. |
100
|
| Parameter workflow should be final. |
100
|
| Must have at least one statement. |
119
|
| Line is longer than 132 characters. |
128
|
| Line is longer than 132 characters. |
130
|
| Method 'saveWorkflow' is not designed for extension - needs to be abstract, final or empty. |
130
|
| Parameter userId should be final. |
130
|
| Parameter password should be final. |
130
|
| Parameter locationToSaveTo should be final. |
130
|
| Parameter workflow should be final. |
130
|
| Must have at least one statement. |
148
|
| Line is longer than 132 characters. |
158
|
| Line is longer than 132 characters. |
160
|
| Method 'saveWorkflow' is not designed for extension - needs to be abstract, final or empty. |
160
|
| Parameter fmc should be final. |
160
|
| Parameter locationToSaveTo should be final. |
160
|
| Parameter workflow should be final. |
160
|
| Must have at least one statement. |
178
|
org/astrogrid/portal/workflow/intf/ActivityKey.java
| Error | Line |
| Class ActivityKey should be declared as final. |
19
|
| Parameter xpath should be final. |
23
|
| Expected @param tag for 'xpath'. |
23
|
| Missing a Javadoc comment. |
26
|
| Line is longer than 132 characters. |
33
|
| Parameter root should be final. |
36
|
| Parameter current should be final. |
36
|
| Redundant throws: 'IllegalArgumentException' is unchecked exception. |
36
|
| Parameter root should be final. |
55
|
| Missing a Javadoc comment. |
60
|
| Parameter o should be final. |
73
|
| Expected @param tag for 'o'. |
73
|
| Avoid inline conditionals. |
84
|
| '31' is a magic number. |
93
|
| Avoid inline conditionals. |
93
|
org/astrogrid/portal/workflow/intf/ApplicationDescription.java
| Error | Line |
| Line is longer than 132 characters. |
35
|
| Line is longer than 132 characters. |
43
|
| Parameter app should be final. |
68
|
| Expected @param tag for 'app'. |
68
|
| Parameter app should be final. |
78
|
| Parameter voDesc should be final. |
78
|
| Missing a Javadoc comment. |
84
|
| Missing a Javadoc comment. |
85
|
| Missing a Javadoc comment. |
86
|
| Missing a Javadoc comment. |
88
|
| Redundant 'final' modifier. |
88
|
| Parameter app should be final. |
88
|
| Line is longer than 132 characters. |
98
|
| Method 'getDefinitionForReference' is not designed for extension - needs to be abstract, final or empty. |
104
|
| Parameter parameterRef should be final. |
104
|
| Redundant throws: 'IllegalArgumentException' is unchecked exception. |
104
|
| Line is longer than 132 characters. |
107
|
| Expected an @return tag. |
113
|
| Method 'getReferenceForValue' is not designed for extension - needs to be abstract, final or empty. |
113
|
| Parameter paramVal should be final. |
113
|
| Expected @param tag for 'paramVal'. |
113
|
| Parameter intf should be final. |
113
|
| Expected @param tag for 'intf'. |
113
|
| Expected @throws tag for 'IllegalArgumentException'. |
113
|
| Redundant throws: 'IllegalArgumentException' is unchecked exception. |
113
|
| Expected an @return tag. |
143
|
| Method 'getDefinitionForValue' is not designed for extension - needs to be abstract, final or empty. |
143
|
| Parameter paramVal should be final. |
143
|
| Expected @param tag for 'paramVal'. |
143
|
| Parameter intf should be final. |
143
|
| Expected @param tag for 'intf'. |
143
|
| Expected @throws tag for 'IllegalArgumentException'. |
143
|
| Redundant throws: 'IllegalArgumentException' is unchecked exception. |
143
|
| Line is longer than 132 characters. |
151
|
| Method 'createValueFromDefinition' is not designed for extension - needs to be abstract, final or empty. |
154
|
| Parameter defn should be final. |
154
|
| Expected an @return tag. |
173
|
| Method 'createToolFromDefaultInterface' is not designed for extension - needs to be abstract, final or empty. |
173
|
| Method 'createToolFromInterface' is not designed for extension - needs to be abstract, final or empty. |
182
|
| Parameter intf should be final. |
182
|
| Redundant throws: 'IllegalArgumentException' is unchecked exception. |
182
|
| Line is longer than 132 characters. |
212
|
| Method 'validate' is not designed for extension - needs to be abstract, final or empty. |
214
|
| Parameter t should be final. |
214
|
| Expected @param tag for 't'. |
214
|
| Expected @throws tag for 'ToolValidationException'. |
214
|
| Missing a Javadoc comment. |
246
|
| Parameter reference should be final. |
246
|
| Parameter searchRoot should be final. |
246
|
| Avoid inline conditionals. |
259
|
| Method 'getInterfaces' is not designed for extension - needs to be abstract, final or empty. |
273
|
| Method 'getName' is not designed for extension - needs to be abstract, final or empty. |
280
|
| Method 'getParameters' is not designed for extension - needs to be abstract, final or empty. |
287
|
| Method 'getOriginalVODescription' is not designed for extension - needs to be abstract, final or empty. |
297
|
org/astrogrid/portal/workflow/intf/ApplicationDescriptionSummary.java
| Error | Line |
| Parameter name should be final. |
24
|
| Expected @param tag for 'name'. |
24
|
| Parameter uiName should be final. |
24
|
| Expected @param tag for 'uiName'. |
24
|
| Parameter interfaceNames should be final. |
24
|
| Expected @param tag for 'interfaceNames'. |
24
|
| Missing a Javadoc comment. |
30
|
| Variable 'name' must be private and have accessor methods. |
30
|
| Missing a Javadoc comment. |
31
|
| Variable 'uiName' must be private and have accessor methods. |
31
|
| Missing a Javadoc comment. |
32
|
| Variable 'interfaceNames' must be private and have accessor methods. |
32
|
| Expected an @return tag. |
35
|
| Method 'getName' is not designed for extension - needs to be abstract, final or empty. |
35
|
| Expected an @return tag. |
40
|
| Method 'getUIName' is not designed for extension - needs to be abstract, final or empty. |
40
|
| Expected an @return tag. |
44
|
| Method 'getInterfaceNames' is not designed for extension - needs to be abstract, final or empty. |
44
|
org/astrogrid/portal/workflow/intf/ApplicationRegistry.java
| Error | Line |
| Expected an @return tag. |
23
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
23
|
| Unused @throws tag for 'WorkflowtInterfaceException'. |
29
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
31
|
| Expected an @return tag. |
34
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
34
|
org/astrogrid/portal/workflow/intf/JobExecutionService.java
| Error | Line |
| Expected @throws tag for 'WorkflowInterfaceException'. |
37
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
43
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
58
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
66
|
org/astrogrid/portal/workflow/intf/ToolValidationException.java
| Error | Line |
| Parameter message should be final. |
23
|
| Expected @param tag for 'message'. |
23
|
| Parameter cause should be final. |
29
|
| Expected @param tag for 'cause'. |
29
|
| Parameter message should be final. |
36
|
| Expected @param tag for 'message'. |
36
|
| Parameter cause should be final. |
36
|
| Expected @param tag for 'cause'. |
36
|
org/astrogrid/portal/workflow/intf/WorkflowBuilder.java
| Error | Line |
| Line is longer than 132 characters. |
24
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
29
|
org/astrogrid/portal/workflow/intf/WorkflowInterfaceException.java
| Error | Line |
| Parameter message should be final. |
27
|
| Expected @param tag for 'message'. |
27
|
| Parameter cause should be final. |
33
|
| Expected @param tag for 'cause'. |
33
|
| Parameter message should be final. |
40
|
| Expected @param tag for 'message'. |
40
|
| Parameter cause should be final. |
40
|
| Expected @param tag for 'cause'. |
40
|
org/astrogrid/portal/workflow/intf/WorkflowManager.java
| Error | Line |
| Line is longer than 132 characters. |
16
|
| Missing a Javadoc comment. |
23
|
| Parameter builder should be final. |
23
|
| Parameter store should be final. |
23
|
| Parameter reg should be final. |
23
|
| Parameter jes should be final. |
23
|
| Missing a Javadoc comment. |
31
|
| Missing a Javadoc comment. |
32
|
| Missing a Javadoc comment. |
33
|
| Missing a Javadoc comment. |
34
|
| Expected an @return tag. |
37
|
| Method 'getWorkflowBuilder' is not designed for extension - needs to be abstract, final or empty. |
37
|
| Expected an @return tag. |
41
|
| Method 'getWorkflowStore' is not designed for extension - needs to be abstract, final or empty. |
41
|
| Expected an @return tag. |
45
|
| Method 'getToolRegistry' is not designed for extension - needs to be abstract, final or empty. |
45
|
| Expected an @return tag. |
49
|
| Method 'getJobExecutionService' is not designed for extension - needs to be abstract, final or empty. |
49
|
org/astrogrid/portal/workflow/intf/WorkflowManagerFactory.java
| Error | Line |
| Line is longer than 132 characters. |
31
|
| Missing a Javadoc comment. |
52
|
| Parameter conf should be final. |
61
|
| Expected @param tag for 'conf'. |
61
|
| Missing a Javadoc comment. |
64
|
| Expected an @return tag. |
67
|
| Method 'getManager' is not designed for extension - needs to be abstract, final or empty. |
67
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
67
|
| Missing a Javadoc comment. |
75
|
| Line is longer than 132 characters. |
88
|
| Missing a Javadoc comment. |
88
|
| Expected an @return tag. |
95
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
95
|
| Expected an @return tag. |
111
|
| Expected an @return tag. |
125
|
| Expected @throws tag for 'Exception'. |
125
|
| Redundant throws: 'Exception' is unchecked exception. |
125
|
| Expected an @return tag. |
147
|
| Expected an @return tag. |
173
|
| Expected @throws tag for 'Exception'. |
173
|
| Redundant throws: 'Exception' is unchecked exception. |
173
|
org/astrogrid/portal/workflow/intf/WorkflowStore.java
| Error | Line |
| Unused import - org.astrogrid.community.beans.v1.Account. |
14
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
73
|
| Unused @param tag for 'authenticated'. |
78
|
| Expected @param tag for 'fileManagerClient'. |
82
|
| Expected @param tag for 'locationToReadFrom'. |
82
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
82
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
103
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
114
|
| Unused @param tag for 'authenticated'. |
118
|
| Expected an @return tag. |
123
|
| Line is longer than 132 characters. |
123
|
| Expected @param tag for 'fileManagerClient'. |
123
|
| Expected @throws tag for 'WorkflowInterfaceException'. |
123
|
org/astrogrid/portal/workflow/package.html
| Error | Line |
| Missing package documentation file. | 0 |