Bug in lang.mixin()

[evogli]

dojo/_base/lang.js line 79

if(!(name in dest)

(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.

[bill]

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.

[evogli]

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.

[bill]

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!

[trac-o-bot]

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!