@@ -833,25 +833,26 @@ Ajax.Request.prototype = Object.extend(new Ajax.Base(), { request: function(url) { this.url = url; - var params = this.options.parameters, method = this.options.method; + this.method = this.options.method; + var params = this.options.parameters; - if (!['get', 'post'].include(method)) { + if (!['get', 'post'].include(this.method)) { // simulate other verbs over post - params['_method'] = method; - method = 'post'; + params['_method'] = this.method; + this.method = 'post'; } params = Hash.toQueryString(params); if (params && /Konqueror|Safari|KHTML/.test(navigator.userAgent)) params += '&_=' // when GET, append parameters to URL - if (method == 'get' && params) + if (this.method == 'get' && params) this.url += (this.url.indexOf('?') > -1 ? '&' : '?') + params; try { Ajax.Responders.dispatch('onCreate', this, this.transport); - this.transport.open(method.toUpperCase(), this.url, + this.transport.open(this.method.toUpperCase(), this.url, this.options.asynchronous); if (this.options.asynchronous) @@ -860,7 +861,7 @@ Ajax.Request.prototype = Object.extend(new Ajax.Base(), { this.transport.onreadystatechange = this.onStateChange.bind(this); this.setRequestHeaders(); - var body = method == 'post' ? (this.options.postBody || params) : null; + var body = this.method == 'post' ? (this.options.postBody || params) : null; this.transport.send(body); @@ -887,7 +888,7 @@ Ajax.Request.prototype = Object.extend(new Ajax.Base(), { 'Accept': 'text/javascript, text/html, application/xml, text/xml, */*' }; - if (this.options.method == 'post') { + if (this.method == 'post') { headers['Content-type'] = this.options.contentType + (this.options.encoding ? '; charset=' + this.options.encoding : ''); actionpack/lib/action_view/helpers/javascripts/prototype.js @@ -833,25 +833,26 @@ Ajax.Request.prototype = Object.extend(new Ajax.Base(), { request: function(url) { this.url = url; - var params = this.options.parameters, method = this.options.method; + this.method = this.options.method; + var params = this.options.parameters; - if (!['get', 'post'].include(method)) { + if (!['get', 'post'].include(this.method)) { // simulate other verbs over post - params['_method'] = method; - method = 'post'; + params['_method'] = this.method; + this.method = 'post'; } params = Hash.toQueryString(params); if (params && /Konqueror|Safari|KHTML/.test(navigator.userAgent)) params += '&_=' // when GET, append parameters to URL - if (method == 'get' && params) + if (this.method == 'get' && params) this.url += (this.url.indexOf('?') > -1 ? '&' : '?') + params; try { Ajax.Responders.dispatch('onCreate', this, this.transport); - this.transport.open(method.toUpperCase(), this.url, + this.transport.open(this.method.toUpperCase(), this.url, this.options.asynchronous); if (this.options.asynchronous) @@ -860,7 +861,7 @@ Ajax.Request.prototype = Object.extend(new Ajax.Base(), { this.transport.onreadystatechange = this.onStateChange.bind(this); this.setRequestHeaders(); - var body = method == 'post' ? (this.options.postBody || params) : null; + var body = this.method == 'post' ? (this.options.postBody || params) : null; this.transport.send(body); @@ -887,7 +888,7 @@ Ajax.Request.prototype = Object.extend(new Ajax.Base(), { 'Accept': 'text/javascript, text/html, application/xml, text/xml, */*' }; - if (this.options.method == 'post') { + if (this.method == 'post') { headers['Content-type'] = this.options.contentType + (this.options.encoding ? '; charset=' + this.options.encoding : ''); railties/html/javascripts/prototype.js 3ec808eadb8f754798c485add6f83a9739212195 Thomas Fuchs thomas@fesch.at http://github.com/rubyruy/rails/commit/03e763ffc823531ea5a5a3be7134318f85370139 03e763ffc823531ea5a5a3be7134318f85370139 2007-01-18T12:27:17-08:00 2007-01-18T12:27:17-08:00 Update prototype.js to [5985], fixes content-type issue with simulated HTTP methods git-svn-id: http://svn-commit.rubyonrails.org/rails/branches/1-2-pre-release@5987 5ecf4fe2-1ee6-0310-87b1-e25e094e27de 42253245d544a0d3c0bded36cb13a78da38a563e Thomas Fuchs thomas@fesch.at