You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by ng...@apache.org on 2009/11/27 00:27:45 UTC

svn commit: r884720 - in /mina/sandbox/vysper/trunk/nbxml/src: main/java/org/apache/vysper/xmpp/xmlfragment/StackNamespaceResolver.java test/java/org/apache/vysper/xmpp/xmlfragment/RendererTestCase.java

Author: ngn
Date: Thu Nov 26 23:27:44 2009
New Revision: 884720

URL: http://svn.apache.org/viewvc?rev=884720&view=rev
Log:
Make namespace attribute order predictable

Modified:
    mina/sandbox/vysper/trunk/nbxml/src/main/java/org/apache/vysper/xmpp/xmlfragment/StackNamespaceResolver.java
    mina/sandbox/vysper/trunk/nbxml/src/test/java/org/apache/vysper/xmpp/xmlfragment/RendererTestCase.java

Modified: mina/sandbox/vysper/trunk/nbxml/src/main/java/org/apache/vysper/xmpp/xmlfragment/StackNamespaceResolver.java
URL: http://svn.apache.org/viewvc/mina/sandbox/vysper/trunk/nbxml/src/main/java/org/apache/vysper/xmpp/xmlfragment/StackNamespaceResolver.java?rev=884720&r1=884719&r2=884720&view=diff
==============================================================================
--- mina/sandbox/vysper/trunk/nbxml/src/main/java/org/apache/vysper/xmpp/xmlfragment/StackNamespaceResolver.java (original)
+++ mina/sandbox/vysper/trunk/nbxml/src/main/java/org/apache/vysper/xmpp/xmlfragment/StackNamespaceResolver.java Thu Nov 26 23:27:44 2009
@@ -20,7 +20,7 @@
 
 package org.apache.vysper.xmpp.xmlfragment;
 
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.Map;
 import java.util.Stack;
 
@@ -44,7 +44,7 @@
 	}
 
 	public Map<String, String> getNamespaceDeclarations() {
-		Map<String, String> ns = new HashMap<String, String>();
+		Map<String, String> ns = new LinkedHashMap<String, String>();
 		if(!elements.isEmpty()) {
 			XMLElement topElm = elements.peek();
 			for(Attribute attribute : topElm.getAttributes()) {

Modified: mina/sandbox/vysper/trunk/nbxml/src/test/java/org/apache/vysper/xmpp/xmlfragment/RendererTestCase.java
URL: http://svn.apache.org/viewvc/mina/sandbox/vysper/trunk/nbxml/src/test/java/org/apache/vysper/xmpp/xmlfragment/RendererTestCase.java?rev=884720&r1=884719&r2=884720&view=diff
==============================================================================
--- mina/sandbox/vysper/trunk/nbxml/src/test/java/org/apache/vysper/xmpp/xmlfragment/RendererTestCase.java (original)
+++ mina/sandbox/vysper/trunk/nbxml/src/test/java/org/apache/vysper/xmpp/xmlfragment/RendererTestCase.java Thu Nov 26 23:27:44 2009
@@ -125,7 +125,7 @@
 		XMLElementBuilder builder = new XMLElementBuilder("foo", "http://example.com");
 		builder.addNamespaceAttribute("pr", "http://other.com");
 		builder.startInnerElement("bar", "http://other.com");
-		assertRendering("<foo xmlns=\"http://example.com\" xmlns:pr=\"http://other.com\"><pr:bar></pr:bar></foo>", builder.build());
+		assertRendering("<foo xmlns:pr=\"http://other.com\" xmlns=\"http://example.com\"><pr:bar></pr:bar></foo>", builder.build());
 	}
 
 	public void testRenderInnerInheritedDefaultNamespaceElement() {