From 5cb3c43b5e10c3c751ffc68938be5827c14b1f25 Mon Sep 17 00:00:00 2001
From: Stephen Paul Weber <singpolyma@singpolyma.net>
Date: Sat, 17 Jun 2017 10:28:06 -0500
Subject: [PATCH] Put <error/> child first on error

Every error is of type error, and put the <error/> child first for
broken parsers that only get the first child.
---
 sgx-catapult.rb | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/sgx-catapult.rb b/sgx-catapult.rb
index fb6e6db490b712379f356aa919221d70c985f1d8..e50514054fa0e106a9e0ba22ff5447825c684c68 100755
--- a/sgx-catapult.rb
+++ b/sgx-catapult.rb
@@ -135,10 +135,7 @@ module SGXcatapult
 	end
 
 	def self.error_msg(orig, query_node, type, name, text=nil)
-		if not query_node.nil?
-			orig.add_child(query_node)
-			orig.type = :error
-		end
+		orig.type = :error
 
 		error = Nokogiri::XML::Node.new 'error', orig.document
 		error['type'] = type
@@ -148,6 +145,8 @@ module SGXcatapult
 		suberr['xmlns'] = 'urn:ietf:params:xml:ns:xmpp-stanzas'
 		error.add_child(suberr)
 
+		orig.add_child(query_node) if query_node
+
 		# TODO: add some explanatory xml:lang='en' text (see text param)
 		puts "RESPONSE3: #{orig.inspect}"
 		return orig