From f8294b84009a05add0b71e32c69f3bdf4b2aa8db Mon Sep 17 00:00:00 2001
From: Sam Georges <sam@daftspunk.com>
Date: Thu, 15 May 2014 14:16:11 +1000
Subject: [PATCH] Add extra hooks for before ajax updates

---
 modules/system/assets/js/framework.js | 30 ++++++++++++++++++++++-----
 1 file changed, 25 insertions(+), 5 deletions(-)

diff --git a/modules/system/assets/js/framework.js b/modules/system/assets/js/framework.js
index d4f98f3b3..efed58959 100644
--- a/modules/system/assets/js/framework.js
+++ b/modules/system/assets/js/framework.js
@@ -63,8 +63,23 @@ if (window.jQuery === undefined)
                 'X-OCTOBER-REQUEST-PARTIALS': this.extractPartials(options.update)
             },
             success: function(data, textStatus, jqXHR) {
-                this.options.beforeUpdate.apply(this, [data, textStatus, jqXHR])
 
+                /*
+                 * Halt here if beforeUpdate() or data-request-before-update returns false
+                 */
+                if (this.options.beforeUpdate.apply(this, [context, data, textStatus, jqXHR]) === false) return
+                if (options.evalBeforeUpdate && eval('(function($el, context, data, textStatus, jqXHR) {'+options.evalBeforeUpdate+'}($el, context, textStatus, jqXHR))') === false) return
+
+                /*
+                 * Trigger 'ajaxBeforeUpdate' on the form, halt if event.preventDefault() is called
+                 */
+                var _event = jQuery.Event('ajaxBeforeUpdate')
+                form.trigger(_event, [context, data, textStatus, jqXHR])
+                if (_event.isDefaultPrevented()) return
+
+                /*
+                 * Proceed with the update process
+                 */
                 var updatePromise = requestOptions.handleUpdateResponse(data, textStatus, jqXHR)
 
                 updatePromise.done(function(){
@@ -99,10 +114,13 @@ if (window.jQuery === undefined)
 
                 updatePromise.done(function(){
                     $el.data('error-message', errorMsg)
-                    var e = jQuery.Event( "ajaxError" )
-                    form.trigger(e, [context, textStatus, jqXHR])
-                    if (e.isDefaultPrevented())
-                        return
+
+                    /*
+                     * Trigger 'ajaxError' on the form, halt if event.preventDefault() is called
+                     */
+                    var _event = jQuery.Event('ajaxError')
+                    form.trigger(_event, [context, textStatus, jqXHR])
+                    if (_event.isDefaultPrevented()) return
 
                     /*
                      * Halt here if the data-request-error attribute returns false
@@ -224,6 +242,7 @@ if (window.jQuery === undefined)
         update: {},
         type : 'POST',
         beforeUpdate: function(data, textStatus, jqXHR) {},
+        evalBeforeUpdate: null,
         evalSuccess: null,
         evalError: null
     }
@@ -250,6 +269,7 @@ if (window.jQuery === undefined)
 
         var $this = $(this).first()
         var data  = {
+            evalBeforeUpdate: $this.data('request-before-update'),
             evalSuccess: $this.data('request-success'),
             evalError: $this.data('request-error'),
             confirm: $this.data('request-confirm'),