Opened 9 years ago

Last modified 9 years ago

#11919 closed enhancement

AMD transform build tool problems — at Version 3

Reported by: ben hockey Owned by: James Burke
Priority: high Milestone: 1.7
Component: BuildTools Version: 1.5
Keywords: Cc: Rawld Gill, Kris Zyp
Blocked By: Blocking:

Description (last modified by ben hockey)

  • anonymous module definitions are not transformed by the build tool.
    define(["some/dep"], function (dep) {
    
        // define the module
    
    });
    

it seems that the transform expects a module name.

  • numbers not accepted in the identifier returned by the module
  • using relative paths in dependency list doesn't work
  • using a single-quote when passing a module name won't be recognized

eg define('something', ...);

Change History (3)

comment:1 Changed 9 years ago by ben hockey

in addition, it seems that you cannot have a return value that has a number in it's name. for example:

define("dojox/math/random/prng4", ["dojo"], function (dojo) {
    var prng4 = ... // define the module

    return prng4;
});

comment:2 Changed 9 years ago by ben hockey

(In [23098]) allow a number to be used in the identifier of the value returned by a module. refs #11919

comment:3 Changed 9 years ago by ben hockey

Description: modified (diff)
Summary: fileUtil.transformAsyncModule does not transform anonymous module definitionsAMD transform build tool problems

i updated the title to be more general - there are a few small things which shouldn't need individual tickets.

Note: See TracTickets for help on using tickets.