Opened 9 years ago
Closed 9 years ago
#14990 closed defect (fixed)
dojo AMD loader(require), plugin(dojo/text) issues
Reported by: | Eric Wang | Owned by: | Rawld Gill |
---|---|---|---|
Priority: | undecided | Milestone: | 1.8 |
Component: | Loader | Version: | 1.7.2 |
Keywords: | Cc: | cjolif, Patrick Ruzand | |
Blocked By: | Blocking: |
Description
- dojo AMD loader(require) sync load not trigger require.on("error") handler
- require cannot catch loader plugin(dojo/text) error
- perform is too low when loading an noexist module several times
see the attachment for the details.
Please put the test file under dojo/tests directory to test.
Attachments (1)
Change History (6)
Changed 9 years ago by
Attachment: | require.html added |
---|
comment:1 Changed 9 years ago by
Cc: | cjolif added |
---|
comment:2 Changed 9 years ago by
Cc: | Patrick Ruzand added |
---|
comment:3 Changed 9 years ago by
Milestone: | tbd → 1.8 |
---|---|
Status: | new → assigned |
comment:4 Changed 9 years ago by
comment:5 Changed 9 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Hi EricWang?! thanks for the great test.
Unfortunately most of the issues it points out are undefined in the AMD spec. These issues are being discussed here http://groups.google.com/group/amd-implement/browse_thread/thread/bb47f5eb576b7490.
The third test did point out that the loader was not timing out in case a plugin resource-id failed to load. I fixed this in [28359]. As this is the only real defect, I am closing this ticket. I also opened a new ticket at #15194 to track error recovery as the spec evolves.
Note: See
TracTickets for help on using
tickets.
In [28359]: