Browse files

Stop appending `&_=` to the parameters for non-GET Ajax requests in S…

…afari. We no longer support any version of Safari for which this is necessary. [#327 state:resolved] (John-David Dalton, Andrew Dupont)
  • Loading branch information...
1 parent ccb929d commit 704aa4033010059402336437f7fe6498ce780be0 @savetheclocktower savetheclocktower committed Oct 18, 2010
Showing with 4 additions and 5 deletions.
  1. +2 −0 CHANGELOG
  2. +2 −5 src/ajax/request.js
View
2 CHANGELOG
@@ -1,3 +1,5 @@
+* Stop appending `&_=` to the parameters for non-GET Ajax requests in Safari. We no longer support any version of Safari for which this is necessary. [#327 state:resolved] (John-David Dalton, Andrew Dupont)
+
* Ensure `Form.focusFirstElement` doesn't raise an exception on forms with no fields. [#341 state:resolved] (achernin, Andrew Dupont)
* Define a `relatedTarget` property on extended mouseenter/mouseleave events in IE's legacy event system. [#708 state:resolved] (Walter Smith, Tobie Langel, Andrew Dupont)
View
7 src/ajax/request.js
@@ -189,12 +189,9 @@ Ajax.Request = Class.create(Ajax.Base, {
this.method = 'post';
}
- if (params) {
+ if (params && this.method === 'get') {
// when GET, append parameters to URL
- if (this.method == 'get')
- this.url += (this.url.include('?') ? '&' : '?') + params;
- else if (/Konqueror|Safari|KHTML/.test(navigator.userAgent))
- params += '&_=';
+ this.url += (this.url.include('?') ? '&' : '?') + params;
}
this.parameters = params.toQueryParams();

0 comments on commit 704aa40

Please sign in to comment.