smartxml: normalizing document in loadXML instead of constructor

[fnpeditor.git] / src / smartxml / smartxml.js

diff --git a/src/smartxml/smartxml.js b/src/smartxml/smartxml.js
index 1876076..f4aa9fe 100644 (file)
--- a/src/smartxml/smartxml.js
+++ b/src/smartxml/smartxml.js
@@ -16,11 +16,16 @@ var DocumentNode = function(nativeNode, document) {
         throw new Error('undefined document for a node');
     }
     this.document = document;
-    this.nativeNode = nativeNode;
-    this._$ = $(nativeNode);
+    this._setNativeNode(nativeNode);
+
 };
 
 $.extend(DocumentNode.prototype, {
+    _setNativeNode: function(nativeNode) {
+        this.nativeNode = nativeNode;
+        this._$ = $(nativeNode);
+    },
+
     detach: function() { this._$.detach(); },
 
     sameNode: function(otherNode) {
@@ -93,14 +98,35 @@ $.extend(ElementNode.prototype, DocumentNode.prototype, {
         return this._$.contents().index(node._$);
     },
 
+    setTag: function(tagName) {
+        var node = this.document.createElementNode({tagName: tagName}),
+            oldTagName = this.getTagName(),
+            myContents = this._$.contents();
+
+        this.getAttrs().forEach(function(attribute) {
+            node.setAttr(attribute.name, attribute.value, true);
+        });
+        node.setData(this.getData());
+
+        if(this.sameNode(this.document.root)) {
+            defineDocumentProperties(this.document, node._$);
+        }
+        this._$.replaceWith(node._$);
+        this._setNativeNode(node._$[0]);
+        this._$.append(myContents);
+        this.triggerChangeEvent('nodeTagChange', {oldTagName: oldTagName, newTagName: this.getTagName()});
+    },
+
     getAttr: function(name) {
         return this._$.attr(name);
     },
 
-    setAttr: function(name, value) {
+    setAttr: function(name, value, silent) {
         var oldVal = this.getAttr(name);
         this._$.attr(name, value);
-        this.triggerChangeEvent('nodeAttrChange', {attr: name, oldVal: oldVal, newVal: value});
+        if(!silent) {
+            this.triggerChangeEvent('nodeAttrChange', {attr: name, oldVal: oldVal, newVal: value});
+        }
     },
 
     getAttrs: function() {
@@ -163,8 +189,12 @@ $.extend(ElementNode.prototype, DocumentNode.prototype, {
 
     toXML: function() {
         var wrapper = $('<div>');
-        wrapper.append(this._$);
+        wrapper.append(this._getXMLDOMToDump());
         return wrapper.html();
+    },
+    
+    _getXMLDOMToDump: function() {
+        return this._$;
     }
 });
 
@@ -194,33 +224,43 @@ var parseXML = function(xml) {
 };
 
 var Document = function(xml) {
-    var $document = $(parseXML(xml));
-
-    var doc = this;
-    Object.defineProperty(this, 'root', {get: function() {
-        return doc.createElementNode($document[0]);
-    }});
-    Object.defineProperty(this, 'dom', {get: function() {
-        return $document[0];
-    }});
+    this.loadXML(xml);
 };
+
 $.extend(Document.prototype, Backbone.Events, {
     ElementNodeFactory: ElementNode,
     TextNodeFactory: TextNode,
 
-    createElementNode: function(nativeNode) {
-        return new this.ElementNodeFactory(nativeNode, this);
+    createElementNode: function(from) {
+        if(!(from instanceof HTMLElement)) {
+            from = $('<' + from.tagName + '>');
+        }
+        return new this.ElementNodeFactory(from, this);
     },
 
     createTextNode: function(nativeNode) {
         return new this.TextNodeFactory(nativeNode, this);
     },
 
+    loadXML: function(xml) {
+        defineDocumentProperties(this, $(parseXML(xml)));
+        
+        this.trigger('contentSet');
+    },
+
     toXML: function() {
         return this.root.toXML();
     }
 });
 
+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) {