Move trID under response #2637

app/views/epp/contacts/check.xml.builder

@@ -15,6 +15,6 @@ xml.epp_head do
       end
     end
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/contacts/create.xml.builder

@@ -11,6 +11,6 @@ xml.epp_head do
       end
     end
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/contacts/delete.xml.builder

@@ -4,6 +4,6 @@ xml.epp_head do
       xml.msg 'Command completed successfully'
     end
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/contacts/info.xml.builder

@@ -55,6 +55,6 @@ xml.epp_head do
       end
     end
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/contacts/update.xml.builder

@@ -11,6 +11,6 @@ xml.epp_head do
       end
     end
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/domains/check.xml.builder

@@ -15,6 +15,6 @@ xml.epp_head do
       end
     end
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/domains/create.xml.builder

@@ -11,7 +11,7 @@ xml.epp_head do
         xml.tag!('domain:exDate', @domain.valid_to)
       end
     end
-  end
 
-  xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
+  end
 end

app/views/epp/domains/info.xml.builder

@@ -59,26 +59,26 @@ xml.epp_head do
         end
       end
     end
-  end
 
-  xml.extension do
+    xml.extension do
-    xml.tag!('secDNS:infData', 'xmlns:secDNS' => 'urn:ietf:params:xml:ns:secDNS-1.1') do
+      xml.tag!('secDNS:infData', 'xmlns:secDNS' => 'urn:ietf:params:xml:ns:secDNS-1.1') do
-      @domain.dnskeys.sort.each do |key|
+        @domain.dnskeys.sort.each do |key|
-        xml.tag!('secDNS:dsData') do
+          xml.tag!('secDNS:dsData') do
-          xml.tag!('secDNS:keyTag', key.ds_key_tag)
+            xml.tag!('secDNS:keyTag', key.ds_key_tag)
-          xml.tag!('secDNS:alg', key.ds_alg)
+            xml.tag!('secDNS:alg', key.ds_alg)
-          xml.tag!('secDNS:digestType', key.ds_digest_type)
+            xml.tag!('secDNS:digestType', key.ds_digest_type)
-          xml.tag!('secDNS:digest', key.ds_digest)
+            xml.tag!('secDNS:digest', key.ds_digest)
-          xml.tag!('secDNS:keyData') do
+            xml.tag!('secDNS:keyData') do
-            xml.tag!('secDNS:flags', key.flags)
+              xml.tag!('secDNS:flags', key.flags)
-            xml.tag!('secDNS:protocol', key.protocol)
+              xml.tag!('secDNS:protocol', key.protocol)
-            xml.tag!('secDNS:alg', key.alg)
+              xml.tag!('secDNS:alg', key.alg)
-            xml.tag!('secDNS:pubKey', key.public_key)
+              xml.tag!('secDNS:pubKey', key.public_key)
+            end
           end
         end
       end
-    end
+    end if @domain.dnskeys.any?
-  end if @domain.dnskeys.any?
 
-  xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
+  end
 end

app/views/epp/domains/renew.xml.builder

@@ -11,6 +11,6 @@ xml.epp_head do
       end
     end
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/domains/success.xml.builder

@@ -3,7 +3,7 @@ xml.epp_head do
     xml.result('code' => '1000') do
       xml.msg 'Command completed successfully'
     end
-  end
 
-  xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
+  end
 end

app/views/epp/domains/success_pending.xml.builder

@@ -3,7 +3,6 @@ xml.epp_head do
     xml.result('code' => '1001') do
       xml.msg 'Command completed successfully; action pending'
     end
+    render('epp/shared/trID', builder: xml)
   end
-
-  xml << render('/epp/shared/trID')
 end

app/views/epp/domains/transfer.xml.builder

@@ -7,7 +7,7 @@ xml.epp_head do
     xml.resData do
       xml << render('epp/domains/partials/transfer', builder: xml, dt: @domain_transfer)
     end
-  end
 
-  xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
+  end
 end

app/views/epp/error.xml.builder

@@ -20,7 +20,6 @@ xml.epp_head do
       end
     end
 
+    render('epp/shared/trID', builder: xml)
   end
-
-  xml << render('/epp/shared/trID')
 end

app/views/epp/poll/poll_ack.xml.builder

@@ -6,6 +6,6 @@ xml.epp_head do
 
     xml.tag!('msgQ', 'count' => current_user.queued_messages.count, 'id' => @message.id)
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/poll/poll_keyrelay.xml.builder

@@ -44,6 +44,6 @@ xml.epp(
       end
     end
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/poll/poll_no_messages.xml.builder

@@ -4,6 +4,6 @@ xml.epp_head do
       xml.msg 'Command completed successfully; no messages'
     end
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/poll/poll_req.xml.builder

@@ -15,6 +15,6 @@ xml.epp_head do
       end
     end if @object
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/sessions/login_fail.xml.builder

@@ -3,7 +3,7 @@ xml.epp_head do
     xml.result('code' => '2501') do
       xml.msg(@msg || 'Authentication error; server closing connection')
     end
-  end
 
-  xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
+  end
 end

app/views/epp/sessions/login_success.xml.builder

@@ -4,6 +4,6 @@ xml.epp_head do
       xml.msg 'Command completed successfully'
     end
 
-    xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
   end
 end

app/views/epp/sessions/logout.xml.builder

@@ -3,7 +3,7 @@ xml.epp_head do
     xml.result('code' => '1500') do
       xml.msg 'Command completed successfully; ending session'
     end
-  end
 
-  xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
+  end
 end

app/views/epp/shared/_trID.xml.builder

@@ -1,4 +1,5 @@
-xml.trID do
+# builder ||= xml
-  xml.clTRID params[:clTRID] if params[:clTRID].present?
+builder.trID do
-  xml.svTRID @svTRID
+  builder.clTRID params[:clTRID] if params[:clTRID].present?
+  builder.svTRID @svTRID
 end

app/views/epp/shared/success.xml.builder

@@ -3,7 +3,7 @@ xml.epp_head do
     xml.result('code' => '1000') do
       xml.msg 'Command completed successfully'
     end
-  end
 
-  xml << render('/epp/shared/trID')
+    render('epp/shared/trID', builder: xml)
+  end
 end