Bug Tracker

Opened 13 years ago

Closed 10 years ago

#196 closed bug (fixed)

Please dont shaddow XMLHttpRequest

Reported by: daniel.doubleday@… Owned by:
Priority: undecided Milestone:
Component: ajax Version:
Keywords: Cc:
Blocked by: Blocking:

Description (last modified by dmethvin)

This breaks other libraries which distinguish between ie and other via "is defined" checks such as DWR and others.

Please use a factory method instead.

I know that this will break backward compatability but I think the "I know that I'm not alone" approach is worth it.

Change History (3)

comment:1 Changed 13 years ago by joern

Resolution: worksforme
Status: newclosed

If we remove the shadowed XMLHttpRequest, I'd have to remove the fixed event object etc. pp. That can't be the right solution either, therefore I close this report.

comment:2 Changed 13 years ago by john

Resolution: worksforme
Status: closedreopened

I still think we should consider this - this is one area in which we do introduce something into the global namespace, when we really shouldn't.

comment:3 Changed 10 years ago by dmethvin

Component: ajax
Description: modified (diff)
need: Review
Priority: blocker
Resolution: fixed
Status: reopenedclosed
Type: bug
Note: See TracTickets for help on using tickets.