#4612 closed defect (fixed)
doc parser doesn't understand dojo.mixin()
Reported by: | dante | Owned by: | Neil Roberts |
---|---|---|---|
Priority: | high | Milestone: | 1.0 |
Component: | Doc parser | Version: | 0.9 |
Keywords: | Cc: | alex | |
Blocked By: | Blocking: |
Description (last modified by )
in trunk/dojo/_base/array.js, line 14:
{{{ dojo.mixin(dojo, {
indexOf: function( /*Array*/ array,
/*Object*/ value,
... etc
}}}
creates the various dojo.forEach .indexOf - doc parser is not finding these summaries (or at least not placing them into the 'dojo' object)
only base's complicated closure and excessive use of aliases (var d = dojo) affects this. should we document the dojo core API all externally? (on the premise it should not be changing anyway, so the whole "edit an extra file" issue is moot?)
Change History (6)
comment:1 Changed 12 years ago by
Description: | modified (diff) |
---|
comment:2 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:3 Changed 12 years ago by
Okay, so the parser basically handles this situation where you might have a function declaration inside of another function declaration, or inside of a mixin, etc.
We need to make sure that mixins that work on external variables can work just fine. Code was updated so that we can take the swallowed functions, and then remove any of them that are truly swallowed inside of a function in the format (function(){})();
should _not_ be changing.