Side navigation
#9970 closed bug (fixed)
Opened August 04, 2011 01:27PM UTC
Closed August 23, 2011 01:44PM UTC
Last modified March 17, 2012 08:46PM UTC
Typo in ajax.js: status should be jzXHR.status?
| Reported by: | giova.chi | Owned by: | jaubourg |
|---|---|---|---|
| Priority: | low | Milestone: | 1.6.3 |
| Component: | ajax | Version: | 1.6.2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
jquery.js, line 7233
The "status" should be "jqXHR.status" I think
source timestamp: Date: Thu Jun 30 14:16:56 2011 -0400
maybe it passes your tests because that line is not included in the test coverage?
Attachments (0)
Change History (9)
Changed August 04, 2011 01:57PM UTC by comment:1
| component: | unfiled → ajax |
|---|---|
| owner: | → giova.chi |
| priority: | undecided → low |
| status: | new → pending |
Changed August 04, 2011 04:10PM UTC by comment:2
| status: | pending → new |
|---|
it's in the error handling code. I don't know how to make "transport.send(requestHeaders, done);" fail with jsFiddle.
Still, I think it's a bug.
Changed August 04, 2011 06:06PM UTC by comment:3
| status: | new → pending |
|---|
Is it perhaps reproducible on a private server?
Changed August 05, 2011 09:11AM UTC by comment:4
I think the easiest way is to take a look at the source code. That "status" variable is simply not defined anywhere in the scope. It's very hard to do a unit test for this error handling code, but whoever wrote the code should be able to see the problem very quickly.
I think he wanted to use "jqXHR.status" instead.
Changed August 20, 2011 07:57AM UTC by comment:5
| resolution: | → invalid |
|---|---|
| status: | pending → closed |
Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!
Changed August 20, 2011 06:43PM UTC by comment:6
| resolution: | invalid |
|---|---|
| status: | closed → reopened |
Reopening for jaubourg to get a chance to weigh in.
Changed August 20, 2011 06:43PM UTC by comment:7
| owner: | giova.chi → jaubourg |
|---|---|
| status: | reopened → assigned |
Changed August 22, 2011 02:10AM UTC by comment:8
| milestone: | None → 1.6.3 |
|---|---|
| summary: | jQuery 1.6.2 error → Typo in ajax.js: status should be jzXHR.status? |
Thanks for taking the time to contribute to the jQuery project! Please provide a reduced jsFiddle test case to help us assess your ticket. Additionally, be sure to test against the jQuery Edge version to ensure the issue still exists. To get you started, we've created this boilerplate: http://jsfiddle.net/rwaldron/da3nM/ Open the link and click to "Fork" in the top menu.