This is an automated email from the ASF dual-hosted git repository.
reta pushed a commit to branch 3.6.x-fixes
in repository https://gitbox.apache.org/repos/asf/cxf.git
The following commit(s) were added to refs/heads/3.6.x-fixes by this push:
new ba66194e2a CXF-8928: Reduce code quality warnings in generated code
(#1435)
ba66194e2a is described below
commit ba66194e2a0e1ba9c12319081bd25a459516a441
Author: glysbaysb <[email protected]>
AuthorDate: Sat Sep 16 17:14:39 2023 +0200
CXF-8928: Reduce code quality warnings in generated code (#1435)
The cxf-codegen-plugin converts a WSDL into java classes, among them the
class for the Jax-WS Service.
In the template some class members have the modifiers "public final
static". The java standard recommends them to be in the order "public
static final"
> If two or more (distinct) field modifiers appear in a field
declaration, it is customary, though not required, that they appear in
the order consistent with that shown above in the production for
FieldModifier. –
https://docs.oracle.com/javase/specs/jls/se20/html/jls-8.html#jls-8.3.1
If the generated source code is checked in to source control it is
common that it will be analyzed using checkstyle, Teamscale and other
linters. To reduce linter complaints the order should be changed.
(cherry picked from commit 925e89910a77ca7b3d9542c271f90597578b6442)
---
.../org/apache/cxf/tools/wsdlto/frontend/jaxws/template/service.vm | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/template/service.vm
b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/template/service.vm
index 7362e77747..bba583a151 100644
---
a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/template/service.vm
+++
b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/template/service.vm
@@ -68,17 +68,17 @@ public class ${service.Name} extends ${serviceSuperclass} {
#if ($markGenerated == "true")
@Generated(value = "org.apache.cxf.tools.wsdlto.WSDLToJava", date =
"$currentdate")
#end
- public final static URL WSDL_LOCATION;
+ public static final URL WSDL_LOCATION;
#if ($markGenerated == "true")
@Generated(value = "org.apache.cxf.tools.wsdlto.WSDLToJava", date =
"$currentdate")
#end
- public final static QName SERVICE = new QName("$service.Namespace",
"$service.ServiceName");
+ public static final QName SERVICE = new QName("$service.Namespace",
"$service.ServiceName");
#foreach ($port in ${service.Ports})
#if ($markGenerated == "true")
@Generated(value = "org.apache.cxf.tools.wsdlto.WSDLToJava", date =
"$currentdate")
#end
- public final static QName $port.Name = new QName("$service.Namespace",
"$port.PortName");
+ public static final QName $port.Name = new QName("$service.Namespace",
"$port.PortName");
#end
static {
#if ($wsdlLocation == "")
