Bug Tracker

Modify

Ticket #6346 (closed bug: invalid)

Opened 3 years ago

Last modified 15 months ago

conflict between jQuery 1.4 and Sarissa

Reported by: doug Owned by: doug
Priority: low Milestone: 1.4.3
Component: ajax Version: 1.4.2
Keywords: sarissa Cc:
Blocking: Blocked by:

Description

Change History

comment:1 Changed 3 years ago by doug

comment:2 Changed 3 years ago by doug

This change to Sarissa seems to work.

_SARISSA_XMLHTTP_PROGID = Sarissa.pickRecentProgID(["Msxml2.XMLHTTP.6.0", "MSXML2.XMLHTTP.3.0", "MSXML2.XMLHTTP", "Microsoft.XMLHTTP"]); _SARISSA_XMLHTTP_PROGID = Sarissa.pickRecentProgID(["MSXML2.XMLHTTP", "Microsoft.XMLHTTP"]);

Any idea why Msxml2.XMLHTTP.6.0 would error?

comment:3 Changed 3 years ago by dmethvin

So should I close this ticket?

comment:4 Changed 3 years ago by doug

What is the difference between Msxml2.XMLHTTP.6.0 (which errors) and Microsoft.XMLHTTP?

The problem was introduced in jQuery 1.4 because the order of creating xhr was reversed.

What was the reason to reverse the order?

1.4

xhr: window.XMLHttpRequest && (window.location.protocol !== "file:"
!window.ActiveXObject) ?

function() {

return new window.XMLHttpRequest();

} : function() {

try {

return new window.ActiveXObject("Microsoft.XMLHTTP");

} catch(e) {}

},

1.3

xhr: function() {

return window.ActiveXObject ? new ActiveXObject("Microsoft.XMLHTTP") : new XMLHttpRequest();

},

comment:6 Changed 3 years ago by dmethvin

More info:

 http://code.google.com/p/jstree/issues/detail?id=430#c6

This implies the reference to xhr.abort (not intended as a call) may actually cause a call for only *some* versions of the MS XMLHttpRequest ActiveX control. The try/catch around the replacement of the abort method may be masking the problem.

Do we have a specific version of IE where this is known to happen? Comment 2 says it happened on IE8 but standard $.ajax requests work fine on my test system. Seems like this will be impossible to fix without more info.

comment:7 Changed 3 years ago by dmethvin

  • Owner set to doug
  • Status changed from new to pending

We need to know the specific versions of the Microsoft XHR where this occurs in order to reproduce it.

comment:8 Changed 3 years ago by dmethvin

  • Keywords needsreview removed

comment:9 Changed 3 years ago by trac-o-bot

  • Status changed from pending to closed
  • Resolution set to invalid

Automatically closed due to 14 days of inactivity.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.