Opened 10 years ago

Closed 9 years ago

#13643 closed defect (fixed)

loader pulling in xhr but not using it

Reported by: bill Owned by: Rawld Gill
Priority: high Milestone: 1.7
Component: Loader Version: 1.7.0b1
Keywords: Cc:
Blocked By: Blocking:

Description

In loader.js the variable xhr here is never used: 

define(["./kernel", "../has", "require", "module", "./json", "./lang", "./array", "../has!host-browser?./xhr"], 
function(dojo, has, require, thisModule, json, lang, array, xhr) {

I know you need XHR for legacy loading but perhaps that's done in a separate module?

Change History (3)

comment:1 Changed 9 years ago by Rawld Gill

Status: new → assigned

comment:2 Changed 9 years ago by Rawld Gill

Resolution: → fixed
Status: assigned → closed

In [26616]:

removed unused code paths; fixed dojo.require to consume second arg; fixes #13643; fixes #13644; !strict

comment:3 Changed 9 years ago by bill

Milestone: tbd → 1.7
Note: See TracTickets for help on using tickets.