OF-839 Forwarded extension should not overwrite extension namespaces of the forwarded message.

fd821670 · Christian Schudt · 2743c92c · fd821670 · fd821670
Commit fd821670 authored Aug 25, 2014 by Christian Schudt
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 3 deletions

Forwarded.java src/java/org/jivesoftware/openfire/forward/Forwarded.java +4 -1

ForwardTest.java ...t/java/org/jivesoftware/openfire/forward/ForwardTest.java +4 -2

No files found.
--- a/src/java/org/jivesoftware/openfire/forward/Forwarded.java
+++ b/src/java/org/jivesoftware/openfire/forward/Forwarded.java
@@ -19,7 +19,10 @@ public class Forwarded extends PacketExtension {
        for (Object element : copy.getElement().elements()) {
            if (element instanceof Element) {
                Element el = (Element) element;
-                el.setQName(QName.get(el.getName(), "jabber:client"));
+                // Only set the "jabber:client" namespace if the namespace is empty (otherwise the resulting xml would look like <body xmlns=""/>)
+                if ("".equals(el.getNamespace().getStringValue())) {
+                    el.setQName(QName.get(el.getName(), "jabber:client"));
+                }
            }
        }
        element.add(copy.getElement());

--- a/src/test/java/org/jivesoftware/openfire/forward/ForwardTest.java
+++ b/src/test/java/org/jivesoftware/openfire/forward/ForwardTest.java
 package org.jivesoftware.openfire.forward;

 import org.junit.Test;
+import org.xmpp.forms.DataForm;
 import org.xmpp.packet.Message;

 import static org.junit.Assert.assertEquals;
@@ -15,12 +16,13 @@ public class ForwardTest {
        Message message = new Message();
        message.setType(Message.Type.chat);
        message.setBody("Tests");
+        message.addExtension(new DataForm(DataForm.Type.submit));

        Forwarded forwarded = new Forwarded(message);
        Forwarded forwarded2 = new Forwarded(message);
        String xml1 = forwarded.getElement().asXML();
        String xml2 = forwarded2.getElement().asXML();
-        assertEquals("<forwarded xmlns=\"urn:xmpp:forward:0\"><message xmlns=\"jabber:client\" type=\"chat\"><body>Tests</body></message></forwarded>", xml1);
-        assertEquals("<forwarded xmlns=\"urn:xmpp:forward:0\"><message xmlns=\"jabber:client\" type=\"chat\"><body>Tests</body></message></forwarded>", xml2);
+        assertEquals("<forwarded xmlns=\"urn:xmpp:forward:0\"><message xmlns=\"jabber:client\" type=\"chat\"><body>Tests</body><x xmlns=\"jabber:x:data\" type=\"submit\"/></message></forwarded>", xml1);
+        assertEquals("<forwarded xmlns=\"urn:xmpp:forward:0\"><message xmlns=\"jabber:client\" type=\"chat\"><body>Tests</body><x xmlns=\"jabber:x:data\" type=\"submit\"/></message></forwarded>", xml2);
    }
 }