From e1fef8a5849274a64f2c75ea4c7a261d67cea1a3 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Aleksander=20=C5=81ukasz?=
 <aleksander.lukasz@nowoczesnapolska.org.pl>
Date: Wed, 23 Oct 2013 13:05:05 +0200
Subject: [PATCH 1/1] smartxml: fixing changing tag of a root node

---
 src/smartxml/smartxml.js      | 21 ++++++++++++---------
 src/smartxml/smartxml.test.js |  6 ++++++
 2 files changed, 18 insertions(+), 9 deletions(-)

diff --git a/src/smartxml/smartxml.js b/src/smartxml/smartxml.js
index 9a3ba26..aa04fd7 100644
--- a/src/smartxml/smartxml.js
+++ b/src/smartxml/smartxml.js
@@ -107,6 +107,9 @@ $.extend(ElementNode.prototype, DocumentNode.prototype, {
         });
         node.setData(this.getData());
 
+        if(this.sameNode(this.document.root)) {
+            defineDocumentProperties(this.document, node._$);
+        }
         this._$.replaceWith(node._$);
         this._setNativeNode(node._$[0]);
         this.triggerChangeEvent('nodeTagChange', {oldTagName: oldTagName, newTagName: this.getTagName()});
@@ -234,15 +237,7 @@ $.extend(Document.prototype, Backbone.Events, {
     },
 
     loadXML: function(xml) {
-        var $document = $(parseXML(xml));
-
-        var doc = this;
-        Object.defineProperty(this, 'root', {get: function() {
-            return doc.createElementNode($document[0]);
-        }, configurable: true});
-        Object.defineProperty(this, 'dom', {get: function() {
-            return $document[0];
-        }, configurable: true});
+        defineDocumentProperties(this, $(parseXML(xml)));
         
         this.trigger('contentSet');
     },
@@ -252,6 +247,14 @@ $.extend(Document.prototype, Backbone.Events, {
     }
 });
 
+var defineDocumentProperties = function(doc, $document) {
+    Object.defineProperty(doc, 'root', {get: function() {
+        return doc.createElementNode($document[0]);
+    }, configurable: true});
+    Object.defineProperty(doc, 'dom', {get: function() {
+        return $document[0];
+    }, configurable: true});
+};
 
 return {
     documentFromXML: function(xml) {
diff --git a/src/smartxml/smartxml.test.js b/src/smartxml/smartxml.test.js
index 938e5f7..e58e754 100644
--- a/src/smartxml/smartxml.test.js
+++ b/src/smartxml/smartxml.test.js
@@ -105,6 +105,12 @@ describe('smartxml', function() {
                     expect(node.getTagName()).to.equal('header');
                     expect(node.getData()).to.eql({key: 'value'});
                 });
+
+                it('can change document root tag name', function() {
+                    var doc = getDocumentFromXML('<div></div>');
+                    doc.root.setTag('span');
+                    expect(doc.root.getTagName()).to.equal('span');
+                });
             });
 
 
-- 
2.20.1