Bug Tracker

Opened 5 years ago

Closed 5 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 5 years ago by dmethvin

Status: newopen

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?

comment:2 Changed 5 years ago by m_gol

Resolution: migrated
Status: openclosed
Note: See TracTickets for help on using tickets.