Backport #233 fix, update release notes

Author: cowtowncoder

release-notes/VERSION

@@ -10,6 +10,8 @@ Project: jackson-dataformat-xml
 
 #228: `XmlReadContext` should hold current value
  (suggested by kfypmqqw@github)
+#233: XmlMapper.copy() doesn't properly copy internal configurations
+ (reported by gtrog@github)
 
 2.8.8 (05-Apr-2017)
 2.8.7 (21-Feb-2017)

src/main/java/com/fasterxml/jackson/dataformat/xml/XmlMapper.java

@@ -86,11 +86,19 @@ public XmlMapper(XmlFactory xmlFactory, JacksonXmlModule module)
         _serializationConfig = _serializationConfig.withDefaultPrettyPrinter(DEFAULT_XML_PRETTY_PRINTER);
     }
 
+    /**
+     * @since 2.8.9
+     */
+    protected XmlMapper(XmlMapper mapper) {
+        super(mapper);
+        this._xmlModule = mapper._xmlModule;
+    }
+
     @Override
     public XmlMapper copy()
     {
         _checkInvalidCopy(XmlMapper.class);
-        return new XmlMapper((XmlFactory) _jsonFactory.copy(), _xmlModule);
+        return new XmlMapper(this);
     }
 
     @Override

src/main/java/com/fasterxml/jackson/dataformat/xml/ser/XmlSerializerProvider.java

@@ -49,12 +49,25 @@ public XmlSerializerProvider(XmlSerializerProvider src,
         _rootNameLookup  = src._rootNameLookup;
     }
 
+    /**
+     * @since 2.8.9
+     */
+    protected XmlSerializerProvider(XmlSerializerProvider toCopy) {
+        super(toCopy);
+        this._rootNameLookup = toCopy._rootNameLookup;
+    }
+
     /*
     /**********************************************************************
     /* Overridden methods
     /**********************************************************************
      */
 
+    @Override
+    public DefaultSerializerProvider copy() {
+        return new XmlSerializerProvider(this);
+    }
+
     @Override
     public DefaultSerializerProvider createInstance(SerializationConfig config,
             SerializerFactory jsf) {