smartxml: transaction calls error handler instead of throwing error
authorAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Mon, 24 Mar 2014 12:18:45 +0000 (13:18 +0100)
committerAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Fri, 28 Mar 2014 11:25:31 +0000 (12:25 +0100)
src/smartxml/smartxml.js
src/smartxml/smartxml.test.js

index 663b332..35c5299 100644 (file)
@@ -583,14 +583,18 @@ $.extend(Document.prototype, Backbone.Events, {
         this._currentTransaction = null;
     },
 
         this._currentTransaction = null;
     },
 
-    transaction: function(callback, context, metadata) {
+    transaction: function(callback, params) {
         var toret;
         var toret;
-        this.startTransaction(metadata);
+        params = params || {};
+        this.startTransaction(params.metadata);
         try {
         try {
-            toret = callback.call(context);
+            toret = callback.call(params.context || this);
         } catch(e) {
         } catch(e) {
+            if(params.error) {
+                params.error(e);
+            }
             this.rollbackTransaction();
             this.rollbackTransaction();
-            throw e;
+            return;
         }
         this.endTransaction();
         return toret;
         }
         this.endTransaction();
         return toret;
index d1eaf22..fb420db 100644 (file)
@@ -1661,17 +1661,17 @@ describe('smartxml', function() {
                 expect(doc.root.contents().length).to.equal(0);
             });
 
                 expect(doc.root.contents().length).to.equal(0);
             });
 
-            it('rollbacks and rethrow if error gets thrown', function() {
+            it('rollbacks and calls error handleor if error gets thrown', function() {
                 var doc = getDocumentFromXML('<root></root>'),
                 var doc = getDocumentFromXML('<root></root>'),
-                    err = new Error();
+                    err = new Error(),
+                    spy = sinon.spy();
                 
                 
-                expect(function() {
-                    doc.transaction(function() {
-                        doc.root.append({tagName: 'div'});
-                        throw err;
-                    });
-                }).to.throw(err);
+                doc.transaction(function() {
+                    doc.root.append({tagName: 'div'});
+                    throw err;
+                }, {error: spy});
 
 
+                expect(spy.args[0][0]).to.equal(err);
                 expect(doc.root.contents().length).to.equal(0);
                 expect(doc.undoStack.length).to.equal(0);
             });
                 expect(doc.root.contents().length).to.equal(0);
                 expect(doc.undoStack.length).to.equal(0);
             });