#16159 closed defect (duplicate)
Issues with loading some Node.js Modules
| Reported by: | Kitson Kelly | Owned by: | Colin Snover |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.9 |
| Component: | Core | Version: | 1.8.1 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
Sometimes when attempting to load certain Node.js modules that might relay on code that attempts to detect if it is running in an AMD environment with dojo/node, it can cause issues where it thinks it should be dealing with define() when in fact it shouldn't and instead return values as part of the module export.
When requiring a module, the scope should be limited via a closure and define "descoped" in order for the code to execute properly.
I have attached a patch which resolves this issue.
Attachments (1)
Change History (5)
comment:1 Changed 8 years ago by
| Owner: | set to Colin Snover |
|---|---|
| Status: | new → assigned |
Changed 8 years ago by
| Attachment: | node.js.patch added |
|---|
comment:2 Changed 8 years ago by
| Milestone: | tbd → 1.9 |
|---|---|
| Priority: | undecided → blocker |
ugh, this did not get fixed? thought it did. fixing now.
comment:3 Changed 8 years ago by
| Resolution: | → duplicate |
|---|---|
| Status: | assigned → closed |
Duplicate of #16414.
Excuse you kitsonk, creating duplicate tickets, sheesh! :)
Note: See
TracTickets for help on using
tickets.
Patches dojo/node to scope out define