Skip to content

Commit baad7e4

Browse files
committed
Code improvement
Signed-off-by: aalzate <[email protected]>
1 parent 56860a1 commit baad7e4

File tree

2 files changed

+7
-12
lines changed

2 files changed

+7
-12
lines changed

src/main/java/org/cyclonedx/util/DependencySerializer.java

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -32,11 +32,8 @@
3232

3333
public class DependencySerializer extends StdSerializer<List<Dependency>>
3434
{
35-
private final String NAMESPACE_PREFIX = "dg";
36-
private final String DEPENDENCY = "dependency";
37-
private final String DEPENDENCIES = "dependencies";
3835
private final String REF = "ref";
39-
private final String NAMESPACE_URI = "http://cyclonedx.org/schema/ext/dependency-graph/1.0";
36+
4037
private boolean useNamespace = false;
4138

4239
public DependencySerializer(final boolean useNamespace) {
@@ -91,7 +88,7 @@ private void writeXMLDependenciesWithGenerator(final ToXmlGenerator toXmlGenerat
9188
throws IOException, XMLStreamException
9289
{
9390
if (dependencies != null && !dependencies.isEmpty()) {
94-
processNamespace(toXmlGenerator, DEPENDENCIES);
91+
processNamespace(toXmlGenerator, "dependencies");
9592
toXmlGenerator.writeStartArray();
9693

9794
for (Dependency dependency : dependencies) {
@@ -106,7 +103,7 @@ private void writeXMLDependenciesWithGenerator(final ToXmlGenerator toXmlGenerat
106103
private void writeXMLDependency(final Dependency dependency, final ToXmlGenerator generator)
107104
throws IOException, XMLStreamException
108105
{
109-
processNamespace(generator, DEPENDENCY);
106+
processNamespace(generator, "dependency");
110107

111108
if (dependency.getDependencies() != null && !dependency.getDependencies().isEmpty()) {
112109
generator.writeStartArray();
@@ -131,16 +128,16 @@ private void writeXMLDependency(final Dependency dependency, final ToXmlGenerato
131128
generator.writeEndObject();
132129
}
133130

134-
private void processNamespace(final ToXmlGenerator toXmlGenerator, final String dependencies2)
131+
private void processNamespace(final ToXmlGenerator toXmlGenerator, final String dependencies)
135132
throws XMLStreamException, IOException
136133
{
137134
QName qName;
138135

139136
if (useNamespace) {
140-
qName = new QName(NAMESPACE_URI, dependencies2, NAMESPACE_PREFIX);
137+
qName = new QName("http://cyclonedx.org/schema/ext/dependency-graph/1.0", dependencies, "dg");
141138
toXmlGenerator.getStaxWriter().setPrefix(qName.getPrefix(), qName.getNamespaceURI());
142139
} else {
143-
qName = new QName(dependencies2);
140+
qName = new QName(dependencies);
144141
}
145142

146143
toXmlGenerator.setNextName(qName);

src/main/java/org/cyclonedx/util/ExtensibleTypesSerializer.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,6 @@ public class ExtensibleTypesSerializer extends StdSerializer<List<ExtensibleType
3535
{
3636
private final String XMLNS = "xmlns";
3737

38-
private final String DEFAULT_VALID_NAMESPACE = "http://www.w3.org/1999/xhtml";
39-
4038
public ExtensibleTypesSerializer() {
4139
this(null);
4240
}
@@ -70,7 +68,7 @@ public void serialize(
7068
}
7169
}
7270
} else {
73-
staxWriter.writeStartElement(ext.getNamespace(), ext.getName(), DEFAULT_VALID_NAMESPACE);
71+
staxWriter.writeStartElement(ext.getNamespace(), ext.getName(), "http://www.w3.org/1999/xhtml");
7472
}
7573

7674
if (ext.getExtensibleTypes() != null && !ext.getExtensibleTypes().isEmpty()) {

0 commit comments

Comments
 (0)