From: Aleksander Ɓukasz Date: Fri, 6 Dec 2013 16:30:30 +0000 (+0100) Subject: undo wip: fixes + smoke test X-Git-Url: https://git.mdrn.pl/fnpeditor.git/commitdiff_plain/7a4bca3706dc42043a773ece492592f6950814fa undo wip: fixes + smoke test --- diff --git a/src/smartxml/smartxml.js b/src/smartxml/smartxml.js index 47c527f..d722c20 100644 --- a/src/smartxml/smartxml.js +++ b/src/smartxml/smartxml.js @@ -431,7 +431,7 @@ $.extend(Document.prototype, Backbone.Events, { if(transformation) { this._transformationLevel++; toret = transformation.run(); - if(this._transformationLevel === 1) { + if(this._transformationLevel === 1 && !this._undoInProgress) { this.undoStack.push(transformation); } this._transformationLevel--; @@ -445,14 +445,18 @@ $.extend(Document.prototype, Backbone.Events, { undo: function() { var transformation = this.undoStack.pop(); if(transformation) { + this._undoInProgress = true; transformation.undo(); + this._undoInProgress = false; this.redoStack.push(transformation); } }, redo: function() { var transformation = this.redoStack.pop(); if(transformation) { + this._transformationLevel++; transformation.run(); + this._transformationLevel--; this.undoStack.push(transformation); } }, diff --git a/src/smartxml/smartxml.test.js b/src/smartxml/smartxml.test.js index 08598aa..3ee2793 100644 --- a/src/smartxml/smartxml.test.js +++ b/src/smartxml/smartxml.test.js @@ -994,63 +994,93 @@ describe('smartxml', function() { }); }); - // describe('Undo/redo', function() { + describe('Undo/redo', function() { - // it('does work', function() { - // var doc = getDocumentFromXML('
Alice
'), - // span = doc.root.contents()[0]; + it('smoke tests', function() { + var doc = getDocumentFromXML('
Alice
'), + textNode = doc.root.contents()[0], + result; + + expect(doc.undoStack).to.have.length(0); + + textNode.wrapWith({tagName: 'span', start:1, end:2}); + expect(doc.undoStack).to.have.length(1, '1'); + expect(doc.toXML()).to.equal('
Alice
'); - // span.transform('smartxml.detach'); + doc.undo(); + expect(doc.undoStack).to.have.length(0, '2'); + expect(doc.toXML()).to.equal('
Alice
'); + debugger; + doc.redo(); + expect(doc.undoStack).to.have.length(1, '3'); + expect(doc.toXML()).to.equal('
Alice
'); - // doc.undo(); + doc.undo(); + expect(doc.undoStack).to.have.length(0, '4'); + expect(doc.toXML()).to.equal('
Alice
'); + + doc.undo(); + expect(doc.undoStack).to.have.length(0, '5'); + expect(doc.toXML()).to.equal('
Alice
'); + + }); - // expect(doc.root.contents()).to.have.length(1); - // expect(doc.root.contents()[0].getTagName()).to.equal('span'); - // expect(doc.root.contents()[0].contents()[0].getText()).to.equal('Alice'); + // it('does work', function() { + // var doc = getDocumentFromXML('
Alice
'), + // span = doc.root.contents()[0]; - // doc.redo(); - // expect(doc.root.contents()).to.have.length(0); + // span.transform('smartxml.detach'); - // doc.undo(); - // expect(doc.root.contents()).to.have.length(1); - // expect(doc.root.contents()[0].getTagName()).to.equal('span'); - // expect(doc.root.contents()[0].contents()[0].getText()).to.equal('Alice'); - // }); - // it('does work - merged text nodes case', function() { - // var doc = getDocumentFromXML('
Alice has a cat.
'), - // span = doc.root.contents()[1]; + // doc.undo(); - // span.transform('smartxml.detach'); + // expect(doc.root.contents()).to.have.length(1); + // expect(doc.root.contents()[0].getTagName()).to.equal('span'); + // expect(doc.root.contents()[0].contents()[0].getText()).to.equal('Alice'); + // doc.redo(); + // expect(doc.root.contents()).to.have.length(0); - // doc.undo(); + // doc.undo(); + // expect(doc.root.contents()).to.have.length(1); + // expect(doc.root.contents()[0].getTagName()).to.equal('span'); + // expect(doc.root.contents()[0].contents()[0].getText()).to.equal('Alice'); - // expect(doc.root.contents().length).to.equal(3); - // //console.log(doc.toXML()); - // expect(doc.root.contents()[1].contents()[0].getText()).to.equal('has'); + // }); + // it('does work - merged text nodes case', function() { + // var doc = getDocumentFromXML('
Alice has a cat.
'), + // span = doc.root.contents()[1]; - // }); - // it('dbg - don not store nodes in tranformation state!', function() { - // var doc = getDocumentFromXML('
'), - // a = doc.root.contents()[0], - // b = doc.root.contents()[1]; + // span.transform('smartxml.detach'); - // a.transform('smartxml.detach'); - // b.transform('smartxml.detach'); - // doc.undo(); - // doc.undo(); - // expect(doc.root.contents().length).to.equal(2); - // expect(doc.root.contents()[0].getTagName()).to.equal('a'); - // expect(doc.root.contents()[1].getTagName()).to.equal('b'); - // doc.redo(); - // doc.redo(); - // expect(doc.root.contents().length).to.equal(0); + // doc.undo(); - // }); - // }); + // expect(doc.root.contents().length).to.equal(3); + // //console.log(doc.toXML()); + // expect(doc.root.contents()[1].contents()[0].getText()).to.equal('has'); + + // }); + // it('dbg - don not store nodes in tranformation state!', function() { + // var doc = getDocumentFromXML('
'), + // a = doc.root.contents()[0], + // b = doc.root.contents()[1]; + + // a.transform('smartxml.detach'); + // b.transform('smartxml.detach'); + // doc.undo(); + // doc.undo(); + // expect(doc.root.contents().length).to.equal(2); + // expect(doc.root.contents()[0].getTagName()).to.equal('a'); + // expect(doc.root.contents()[1].getTagName()).to.equal('b'); + + // doc.redo(); + // doc.redo(); + // expect(doc.root.contents().length).to.equal(0); + + // }); + }); });