Opened 17 years ago
Closed 14 years ago
#196 closed bug (fixed)
Please dont shaddow XMLHttpRequest
| Reported by: | Owned by: | ||
|---|---|---|---|
| Priority: | undecided | Milestone: | |
| Component: | ajax | Version: | |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description (last modified by )
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 17 years ago by
| Resolution: | → worksforme |
|---|---|
| Status: | new → closed |
comment:2 Changed 17 years ago by
| Resolution: | worksforme |
|---|---|
| Status: | closed → reopened |
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 14 years ago by
| Component: | → ajax |
|---|---|
| Description: | modified (diff) |
| need: | → Review |
| Priority: | → blocker |
| Resolution: | → fixed |
| Status: | reopened → closed |
| Type: | → bug |
Note: See
TracTickets for help on using
tickets.
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.