Ajax rewrite #45

Closed
wants to merge 298 commits into
from

Conversation

Projects
None yet
3 participants
@jaubourg
Member

jaubourg commented Oct 11, 2010

Revamping of the internals of jQuery.ajax() as a pluggable architecture with more control over data types, late callback binding and complete xhr emulation.

See http://oksoclap.com/6Y26bm1ZsB for more detailed information.

I'm not looking for inclusion right now but rather for code review and general remarks. It's been hell to keep the rewrite up-to-date with all the activity as of late ;)

jaubourg added some commits Dec 6, 2009

jaubourg
Reworked callback lists code and changed method call & naming
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Better handling of dataTypes option (also added default in ajaxSettin…
…gs comment)

Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
When data converter is not found try srcType => text & text => destTy…
…pe (unit test changed accordingly)

Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
transport/jsonp no longer put text inbetween jsonp & json dataTypes (…
…no needed anymore)

Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
More compact (but less readable) status filtering: switch was overkill.
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Fixed the condition on setting X-Requested-With and set its key as lo…
…wercase

Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Revised headers regexp
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Added newline at end of files
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Merge branch 'master' of git://github.com/jquery/jquery
Conflicts:
	Rakefile

Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Fire complete AFTER success or error (as of previous implementation)
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Removed cache handling & fixed completely off status system (john's n…
…ot the only one to commit while tired it seems)

Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Revised status handling... need to concentrate
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Fixed error in jquery/jquery
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Add preliminary support for s.jsonpCallback
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Remove test of s.jsonCallback, awaiting type selection refactoring
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Preliminary prefilters support + various comments & spacing thingies
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Merge branch 'master' of git://github.com/jquery/jquery
Conflicts:
	Rakefile
	src/manipulation.js
jaubourg
Transport system V2
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Fixed silly merge I made on build.xml
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
- Callbacks can now be specified by array or with multiple arguments
- ajax.bindTransport now uses a function pointer to append instead of a test each loop

Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Better cleanup of jsonp callback using complete callback
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Removed handleCallback from dereferences (not needed anymore)
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Test for async rather than force it
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Added jsonp over iframe transport
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Revised unit tests now including jsonp over iframe tests
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Avoided unnecessary loop and dereferences fire when firing is skipped…
… on empty

Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Moved all variables up
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Shorter support.crossDomainRequest values : "xhr" & "xdr".
Signed-off-by: jaubourg <aubourg.julian@gmail.com>

jaubourg added some commits Oct 11, 2010

jaubourg
Merge branch 'master' of git://github.com/jquery/jquery
Conflicts:
	build/google-compiler-20091218.jar
	src/ajax.js
	test/unit/ajax.js
jaubourg
Removed plugins.
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Merge branch 'master' of git://github.com/jquery/jquery
Conflicts:
	src/ajax.js
	test/unit/ajax.js

Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Added success data to ajaxSuccess event + minor rewriting.
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Some rewriting for better minification.
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Fixed prefilters addition, removed recursion from transport selection…
… and made it more robust + other minor rewritings.

Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Removed xdr transport per Paul Irish's recommendation: was introduced…
… in IE8 and IE9 should support CORS.

Signed-off-by: jaubourg <aubourg.julian@gmail.com>
jaubourg
Revamped event binding for jQuery XHR objects.
Signed-off-by: jaubourg <aubourg.julian@gmail.com>
@csnover

This comment has been minimized.

Show comment
Hide comment
@csnover

csnover Nov 21, 2010

Member

This is for ticket #7195.

Member

csnover commented Nov 21, 2010

This is for ticket #7195.

@jitter

This comment has been minimized.

Show comment
Hide comment
@jitter

jitter Dec 13, 2010

Contributor

jaubourg you may close this one now :D

Contributor

jitter commented Dec 13, 2010

jaubourg you may close this one now :D

@jaubourg

This comment has been minimized.

Show comment
Hide comment
@jaubourg

jaubourg Dec 13, 2010

Member

It looks closed to me Oo

Member

jaubourg commented Dec 13, 2010

It looks closed to me Oo

@jitter

This comment has been minimized.

Show comment
Hide comment
@jitter

jitter Dec 13, 2010

Contributor

When I was commenting it still showed me the "comment and close" button. But snover closed it shortly thereafter :P

Contributor

jitter commented Dec 13, 2010

When I was commenting it still showed me the "comment and close" button. But snover closed it shortly thereafter :P

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment