From 7a324b859ce0f00089e2d9be6ea473da0437f2d3 Mon Sep 17 00:00:00 2001
From: Sam Stephenson <sam@37signals.com>
Date: Tue, 24 Apr 2007 03:58:34 +0000
Subject: [PATCH] prototype: Add support for per-request onCreate callbacks to
 Ajax.Request.  Closes #8011.

---
 CHANGELOG           | 2 ++
 src/ajax.js         | 1 +
 test/unit/ajax.html | 8 ++++++++
 3 files changed, 11 insertions(+)

diff --git a/CHANGELOG b/CHANGELOG
index 0c3a831..76abecb 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,5 +1,7 @@
 *SVN*
 
+* Add support for per-request onCreate callbacks to Ajax.Request.  Closes #8011.  [Andrew Dupont]
+
 * Don't call evalResponse() when an Ajax response has no Content-type header.  Closes #7827.  [Tobie Langel]
 
 * Automatically strip security delimiter comments from JSON strings before evaling them.  The default delimiter is '/*-secure- ... */' or you can specify your own with the Prototype.JSONFilter regular expression.  If you wrap your JSON response bodies in this delimiter on the server side, rogue external sites can't hijack potentially sensitive data via <script> tags.  Closes #7910.  [Tobie Langel]
diff --git a/src/ajax.js b/src/ajax.js
index 5d4cfe4..179827f 100644
--- a/src/ajax.js
+++ b/src/ajax.js
@@ -101,6 +101,7 @@ Ajax.Request.prototype = Object.extend(new Ajax.Base(), {
     }
       
     try {
+      if (this.options.onCreate) this.options.onCreate(this.transport);
       Ajax.Responders.dispatch('onCreate', this, this.transport);
     
       this.transport.open(this.method.toUpperCase(), this.url, 
diff --git a/test/unit/ajax.html b/test/unit/ajax.html
index 095bca8..7bf443d 100644
--- a/test/unit/ajax.html
+++ b/test/unit/ajax.html
@@ -165,6 +165,14 @@
             assertEqual('application/bogus; charset=UTF-8', response.responseText);
         }
       });
+    }},
+    
+    testOnCreateCallback: function() {with(this) {
+      new Ajax.Request("fixtures/content.html", {
+        asynchronous: false,
+        onCreate: function(transport) { assertEqual(0, transport.readyState) },
+        onComplete: function(transport) { assertNotEqual(0, transport.readyState) }
+      });
     }}
     
   }, 'testlog');