Opened 6 years ago
Closed 6 years ago
#15210 closed bug (migrated)
2.1.2-pre, ajax.js: environments that don't have 'location' variable will break
| Reported by: | brianblocker | Owned by: | |
|---|---|---|---|
| Priority: | high | Milestone: | None |
| Component: | ajax | Version: | 2.1.1 |
| Keywords: | test | Cc: | |
| Blocked by: | Blocking: |
Description
It appears that the change which removes the "IE workaround" on ajax.js now breaks environments that do not have window.location. To be specific:
(from https://github.com/jquery/jquery/blob/master/src/ajax.js#L44) ajaxLocation = location.href,
When I try to run automated tests using jasmine-node, location is undefined (because there is no "browser" sorta), and therefore the test breaks. Using the previous version (2.1.1-pre) works fine, but it appears that the change introduced in 2.1.2-pre now breaks the environment.
Change History (2)
comment:1 Changed 6 years ago by
| Status: | new → open |
|---|
comment:2 Changed 6 years ago by
| Resolution: | → migrated |
|---|---|
| Status: | open → closed |
Migrated to https://github.com/jquery/jquery/issues/1788
Note: See
TracTickets for help on using
tickets.
Since this is a regression, I'd like to fix it. However, we don't unit test or support jasmine-node so to have any chance to do this we'll need your help. Can you propose a fix as a pull request?