Opened 13 years ago
Closed 8 years ago
#2536 closed defect (invalid)
dojo.requireIf fails in custom build with parentheses condition
| Reported by: | Owned by: | ||
|---|---|---|---|
| Priority: | high | Milestone: | future |
| Component: | BuildSystem | Version: | 0.4.2rc1 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description (last modified by )
In a source file, doing dojo.requireIf( (!(a && a.b)), "a.b") returns an error during the build "exception: SyntaxError?: missing ) after argument list". (rev 7517)
Refactoring the requireIf statement to look like: dojo.requireIf( !a || !a.b, "a.b") works fine. (ie. remove the parentheses)
I assume its a regexp somewhere which isn't being greedy enough, but I couldn't seem to find the correct one in buildUtil.js.
Change History (6)
comment:1 Changed 12 years ago by
| Milestone: | → 1.0 |
|---|
comment:2 Changed 12 years ago by
| Milestone: | 1.0 → 1.1 |
|---|
comment:3 Changed 12 years ago by
| Milestone: | 1.1 → 1.2 |
|---|
comment:4 Changed 11 years ago by
| Description: | modified (diff) |
|---|---|
| Milestone: | 1.2 → future |
comment:5 Changed 8 years ago by
| Owner: | changed from James Burke to robert.coup@… |
|---|---|
| Status: | new → pending |
is this issue still relevant?
comment:6 Changed 8 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | pending → closed |
Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!
See #4462: it might have a parens matching code change, and if so, it should solve this issue. Pushing to 1.1, with #4462.