Bug fix: Removing item from DocumentNodeElement data caused removing all data
authorAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Fri, 2 Aug 2013 10:03:28 +0000 (12:03 +0200)
committerAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Fri, 2 Aug 2013 10:03:28 +0000 (12:03 +0200)
- DocumentNodeElement.append was removing wrong attribute but due
to bug in DocumentNodeElement.data it went undetected

modules/documentCanvas/canvas/documentElement.js

index c6c9f22..c38008b 100644 (file)
@@ -154,7 +154,7 @@ $.extend(DocumentNodeElement.prototype, {
         var dom = this.dom(),
             args = Array.prototype.slice.call(arguments, 0);
         if(args.length === 2 && args[1] === undefined)
         var dom = this.dom(),
             args = Array.prototype.slice.call(arguments, 0);
         if(args.length === 2 && args[1] === undefined)
-            return dom.removeData(args[1]);
+            return dom.removeData(args[0]);
         return dom.data.apply(dom, arguments);
     },
     toXML: function(level) {
         return dom.data.apply(dom, arguments);
     },
     toXML: function(level) {
@@ -227,7 +227,7 @@ $.extend(DocumentNodeElement.prototype, {
     },
     append: function(params) {
         if(params.tag !== 'span')
     },
     append: function(params) {
         if(params.tag !== 'span')
-            this.data('orig-ends', undefined);
+            this.data('orig-end', undefined);
         return manipulate(this, params, 'append');
     },
     before: function(params) {
         return manipulate(this, params, 'append');
     },
     before: function(params) {