Opened 11 years ago

Closed 11 years ago

#7295 closed enhancement (fixed)

Make _mapIntoDojo accessible outside of _base/NodeList.js

Reported by: dante Owned by: alex
Priority: high Milestone: 1.3
Component: Core Version: 1.1.1
Keywords: Cc: alex, Eugene Lazutkin
Blocked By: Blocking:

Description

the "private" _mapIntoDojo function is currently hidden within an anonymous scope in Nodelist.js -- I would like to see this useful function extracted, or at least accessible outside of that scope.

patch attached.

This only allows dojo.* namespaced functions to be mapped in, but is a caveat I am prepared to workaround.

Attachments (1)

mapin.patch (1.1 KB) - added by dante 11 years ago.

Download all attachments as: .zip

Change History (5)

Changed 11 years ago by dante

Attachment: mapin.patch added

comment:1 Changed 11 years ago by dante

Cc: alex added; dante removed
Component: GeneralCore
Milestone: tbd1.3
Owner: changed from alex to dante
Status: newassigned

after talking w/ Alex, we should probably rename mapIntoDojo to mapIn, and allow it to accept a scope (matching api with dojo.partial), and expose it as proposed above. Not super-pressing, but I'd like to see it done. Taking myself, and setting milestone to give adequate testing time.

comment:2 Changed 11 years ago by dante

Owner: changed from dante to alex
Status: assignednew

comment:3 Changed 11 years ago by Eugene Lazutkin

Cc: Eugene Lazutkin added

comment:4 Changed 11 years ago by alex

Resolution: fixed
Status: newclosed

(In [16615]) exposing _mapIntoDojo, but renaming to _mapIn. Fixes #7295. !strict

Note: See TracTickets for help on using tickets.