Ticket #2468 (closed defect: wontfix)

Opened 21 months ago

Last modified 14 months ago

Change build an loader behavior for dojo.requireIf() vs. dojo.requireAfterIf()

Reported by: jburke Owned by: jburke
Priority: normal Milestone: 1.0
Component: PackageSystem Version: 0.4.1
Severity: normal Keywords:
Cc:

Description

Technically, dojo.requireIf includes should put the module resource from the requireIf directly in the place of the dojo.requireIf() call, where dojo.requireAfterIf() should load the module resource after the current module resource.

Currently in the build system, loader_xd.js and in browser_debug.js, requireIf is treated like requireAfterIf(). It might be difficult to solve this for browser_debug.js, but look at possibilities for the build system and loader_xd.js.

It would mean in the xdomain case, splitting the top resource into two different calls to the dojo.hostenv.packageLoaded(). This might be weird though because that means two different scopes, where in the source file, it would have assumed one scope, so make sure to set the scope correctly.

Change History

Changed 16 months ago by jburke

  • milestone changed from 0.9 to 1.0

This is a very technical edge case I'm trying to patch here. In practice, treating dojo.requireIf calls like dojo.requireAfterIf calls seem to be fine for the normal uses of requireIf. And I don't like how much possible code change this could be Pushing to 1.0, but may even punt it for that release.

Changed 14 months ago by jburke

  • status changed from new to closed
  • resolution set to wontfix

I'm closing this as as a wontfix. The xdomain loader is already chunky, and this seems to be a practically non-existent use case in the real world.

Note: See TracTickets for help on using tickets.