gnodet commented on code in PR #2029:
URL: https://github.com/apache/maven/pull/2029#discussion_r1904987719
##########
api/maven-api-xml/src/main/java/org/apache/maven/api/xml/XmlNode.java:
##########
@@ -18,121 +18,244 @@
*/
package org.apache.maven.api.xml;
+import javax.xml.stream.XMLStreamException;
+
+import java.io.Serializable;
+import java.io.StringWriter;
import java.util.List;
+import java.util.ListIterator;
import java.util.Map;
-
-import org.apache.maven.api.annotations.Experimental;
-import org.apache.maven.api.annotations.Immutable;
-import org.apache.maven.api.annotations.Nonnull;
-import org.apache.maven.api.annotations.Nullable;
-import org.apache.maven.api.annotations.ThreadSafe;
+import java.util.Objects;
+import java.util.function.Function;
/**
- * An immutable xml node.
- *
- * @since 4.0.0
+ * NOTE: remove all the util code in here when separated, this class should
be pure data.
*/
-@Experimental
-@ThreadSafe
-@Immutable
-public interface XmlNode {
+public class XmlNode implements Serializable {
+ private static final long serialVersionUID = 2567894443061173996L;
- String CHILDREN_COMBINATION_MODE_ATTRIBUTE = "combine.children";
+ public static final String CHILDREN_COMBINATION_MODE_ATTRIBUTE =
"combine.children";
- String CHILDREN_COMBINATION_MERGE = "merge";
+ public static final String CHILDREN_COMBINATION_MERGE = "merge";
- String CHILDREN_COMBINATION_APPEND = "append";
+ public static final String CHILDREN_COMBINATION_APPEND = "append";
/**
* This default mode for combining children DOMs during merge means that
where element names match, the process will
* try to merge the element data, rather than putting the dominant and
recessive elements (which share the same
* element name) as siblings in the resulting DOM.
*/
- String DEFAULT_CHILDREN_COMBINATION_MODE = CHILDREN_COMBINATION_MERGE;
+ public static final String DEFAULT_CHILDREN_COMBINATION_MODE =
CHILDREN_COMBINATION_MERGE;
- String SELF_COMBINATION_MODE_ATTRIBUTE = "combine.self";
+ public static final String SELF_COMBINATION_MODE_ATTRIBUTE =
"combine.self";
- String SELF_COMBINATION_OVERRIDE = "override";
+ public static final String SELF_COMBINATION_OVERRIDE = "override";
- String SELF_COMBINATION_MERGE = "merge";
+ public static final String SELF_COMBINATION_MERGE = "merge";
- String SELF_COMBINATION_REMOVE = "remove";
+ public static final String SELF_COMBINATION_REMOVE = "remove";
/**
* In case of complex XML structures, combining can be done based on id.
*/
- String ID_COMBINATION_MODE_ATTRIBUTE = "combine.id";
+ public static final String ID_COMBINATION_MODE_ATTRIBUTE = "combine.id";
/**
* In case of complex XML structures, combining can be done based on keys.
* This is a comma separated list of attribute names.
*/
- String KEYS_COMBINATION_MODE_ATTRIBUTE = "combine.keys";
+ public static final String KEYS_COMBINATION_MODE_ATTRIBUTE =
"combine.keys";
/**
* This default mode for combining a DOM node during merge means that
where element names match, the process will
* try to merge the element attributes and values, rather than overriding
the recessive element completely with the
* dominant one. This means that wherever the dominant element doesn't
provide the value or a particular attribute,
* that value or attribute will be set from the recessive DOM node.
*/
- String DEFAULT_SELF_COMBINATION_MODE = SELF_COMBINATION_MERGE;
+ public static final String DEFAULT_SELF_COMBINATION_MODE =
SELF_COMBINATION_MERGE;
+
+ protected final String prefix;
Review Comment:
This isn't necessary if using an interface + builder and non public class.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
