smartxml: removing node attributes by setting undefined as their value

Before this change calling Node.setAttr(undefined) was a noop.

diff --git a/src/smartxml/core.js b/src/smartxml/core.js
index b60bc2e..7326507 100644 (file)
--- a/src/smartxml/core.js
+++ b/src/smartxml/core.js
@@ -169,7 +169,11 @@ var elementNodeTransformations = {
 
     setAttr: function(name, value, silent) {
         var oldVal = this.getAttr(name);
 
     setAttr: function(name, value, silent) {
         var oldVal = this.getAttr(name);

-        this._$.attr(name, value);
+        if(_.isUndefined(value)) {
+            this._$.removeAttr(name);
+        } else {
+            this._$.attr(name, value);
+        }

         if(!silent) {
             this.triggerChangeEvent('nodeAttrChange', {attr: name, oldVal: oldVal, newVal: value});
         }
         if(!silent) {
             this.triggerChangeEvent('nodeAttrChange', {attr: name, oldVal: oldVal, newVal: value});
         }