smartxml: Automatically rollback transaction on error thrown
authorAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Wed, 19 Mar 2014 12:16:53 +0000 (13:16 +0100)
committerAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Fri, 28 Mar 2014 11:25:31 +0000 (12:25 +0100)
This works for:
- implicit transactions created for transformations run when there is
no transaction in progress
- transactions started by Document.transaction call

It won't work for transactions started by Document.transactionStart.

src/smartxml/smartxml.js
src/smartxml/smartxml.test.js

index 31faef0..663b332 100644 (file)
@@ -586,7 +586,12 @@ $.extend(Document.prototype, Backbone.Events, {
     transaction: function(callback, context, metadata) {
         var toret;
         this.startTransaction(metadata);
     transaction: function(callback, context, metadata) {
         var toret;
         this.startTransaction(metadata);
-        toret = callback.call(context);
+        try {
+            toret = callback.call(context);
+        } catch(e) {
+            this.rollbackTransaction();
+            throw e;
+        }
         this.endTransaction();
         return toret;
     },
         this.endTransaction();
         return toret;
     },
index c792cf9..d1eaf22 100644 (file)
@@ -1660,6 +1660,21 @@ describe('smartxml', function() {
                 expect(doc.undoStack.length).to.equal(0, 'nothing to undo');
                 expect(doc.root.contents().length).to.equal(0);
             });
                 expect(doc.undoStack.length).to.equal(0, 'nothing to undo');
                 expect(doc.root.contents().length).to.equal(0);
             });
+
+            it('rollbacks and rethrow if error gets thrown', function() {
+                var doc = getDocumentFromXML('<root></root>'),
+                    err = new Error();
+                
+                expect(function() {
+                    doc.transaction(function() {
+                        doc.root.append({tagName: 'div'});
+                        throw err;
+                    });
+                }).to.throw(err);
+
+                expect(doc.root.contents().length).to.equal(0);
+                expect(doc.undoStack.length).to.equal(0);
+            });
         });
 
         describe('Regression tests', function() {
         });
 
         describe('Regression tests', function() {