Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix IE6 $.ajaxTransport('+' + str, function(){}) case bug #1308

Closed
wants to merge 1 commit into from

Conversation

@harvey-woo
Copy link

commented Jul 13, 2013

IE6 do not support str[0] but str.charAt(0)

IE6 do not support str[0] but str.charAt(0)
@rwaldron

This comment has been minimized.

Copy link
Member

commented Jul 13, 2013

Thanks for the contribution!

Please file a bug here: http://bugs.jquery.com and include a reduced test case. This patch will also need to include a unit test that proves the fix.

@dmethvin

This comment has been minimized.

Copy link
Member

commented Aug 26, 2013

This looks like a valid issue since oldIE doesn't support indexing strings.

@jaubourg

This comment has been minimized.

Copy link
Member

commented Aug 26, 2013

Agreed. Weird that it never came up before.

@mgol

This comment has been minimized.

Copy link
Member

commented Sep 7, 2013

@alphmega Ping? Could you sign the CLA and introduce changes @rwaldron talked about?

@timmywil

This comment has been minimized.

Copy link
Member

commented Sep 12, 2013

This needs to be changed and there's more work than just switching to charAt. I'll go ahead and take this.

@timmywil timmywil closed this Sep 12, 2013
@lock lock bot locked as resolved and limited conversation to collaborators Jan 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
6 participants
You can’t perform that action at this time.