Add copy constructor and remove boolean parameter from toKeyValues

The boolean parameter toKeyValues didn't make sense anymore with all the
changes to KeyValueTree (due to AutoBuild Serialisation)and the value
was always false.

Author: jan

io.sloeber.autoBuild/src/io/sloeber/autoBuild/helpers/api/KeyValueTree.java

@@ -24,7 +24,7 @@ public class KeyValueTree {
 	private KeyValueTree myParent;
 
 	public static KeyValueTree createRoot() {
-		return new KeyValueTree(null, null);
+		return new KeyValueTree((String)null,(String) null);
 	}
 
 	private KeyValueTree(String newKey, String newValue) {
@@ -34,6 +34,33 @@ private KeyValueTree(String newKey, String newValue) {
 		myParent = null;
 	}
 
+	/**
+	 * Copy constructor from this location (parent of source is ignored
+	 * Note this copy constructor assumes the strings themselves are not changed.
+	 *
+	 * @param source
+	 */
+	public KeyValueTree(KeyValueTree source) {
+		this( source, null);
+	}
+
+	private KeyValueTree(KeyValueTree source, KeyValueTree parent) {
+		myChildren = new TreeMap<>();
+		myKey = source.myKey;
+		myValue = source.myValue;
+		myParent = parent;
+		copyChildren(source.myChildren);
+	}
+
+	private void copyChildren(Map<String, KeyValueTree> children) {
+		if(children==null) {
+			return;
+		}
+		for(Entry<String, KeyValueTree> curChild:children.entrySet()) {
+			myChildren.put(curChild.getKey(),new KeyValueTree(curChild.getValue(),this));
+		}
+	}
+
 	public KeyValueTree getParent() {
 		return this.myParent;
 	}
@@ -94,12 +121,8 @@ public Map<String, String> toKeyValues(boolean addParents) {
 		return toKeyValues(EMPTY_STRING, theRoot);
 	}
 
-	public Map<String, String> toKeyValues(String prefix, boolean addParents) {
-		KeyValueTree theRoot = this;
-		if (addParents) {
-			theRoot = null;
-		}
-		return toKeyValues(prefix, theRoot);
+	public Map<String, String> toKeyValues(String prefix) {
+		return toKeyValues(prefix, this);
 	}
 
 	public String dump() {