Opened 8 years ago
Closed 8 years ago
#16593 closed defect (invalid)
Bug in lang.mixin()
Reported by: | evogli | Owned by: | evogli |
---|---|---|---|
Priority: | undecided | Milestone: | tbd |
Component: | Parser | Version: | 1.8.3 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
dojo/_base/lang.js line 79
(dest[name] !== s && (!(name in empty) | empty[name] !== s))){ |
dest[name] = copyFunc ? copyFunc(s) : s;
}
I have a widget that is failing to parse because when it gets to this line "dest" is a string.
There should be a check if dest is an object before assuming so.
Change History (6)
comment:1 Changed 8 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
Summary: | Bug in → Bug in lang.mixin() |
comment:2 Changed 8 years ago by
If that's the case then the parser somewhere doesn't do a check before calling this method.
I have a data-dojo-type on a div in a widget template which fails on this line. If the parser is mistakenly calling this method on a string type then it should be checked there.
comment:3 Changed 8 years ago by
Please attach a test case using the "attach file" button. It should be as small as possible to still reproduce the problem, almost always a single HTML file that we can load in the browser (i.e. not PHP, JSP, etc.)
Then, give exact instructions on how to reproduce the problem using your attached test file.
The test case is necessary both to confirm that there's a bug, and for us to be able to debug the problem.
Thanks!
comment:4 Changed 8 years ago by
Resolution: | invalid |
---|---|
Status: | closed → reopened |
comment:5 Changed 8 years ago by
Component: | General → Parser |
---|---|
Owner: | set to evogli |
Status: | reopened → pending |
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!
It's not a bug, it's a requirement that the the first argument to lang.mixin() must be an object rather than null. Passing null doesn't make any sense.
We typically don't bloat our code with checks about valid parameters.