dojo.dom.hasParent() returns null instead of false

[[email protected]…]

I write this comment here, because bug I've spotted is very near to this improvement. It is IE bug (as far as I understand), but it could be fixed somehow in dojo.

Code:

var d = document.createElement("div");
d.innerHTML = 'Blah';
alert(dojo.dom.hasParent(d));

alerts true in IE7. Without .innerHTML it works fine.

P.S. Sorry about bad english.

[bill]

Well, we could change

	return node && node.parentNode && dojo.dom.isNode(node.parentNode);	//	boolean

to

	return Boolean(node && node.parentNode && dojo.dom.isNode(node.parentNode));	//	boolean

Not sure if it's worth it though. I imagine we have dozens of functions that don't return a true boolean value; not sure if we want to change them.

[alex]

null should exist but should be "falsey" in boolean situations. While it can't hurt to return a boolean, using the return of htis function today in a boolean scenario will yield the expected behavior.

This bug should not have been moved off of 0.4.1.

[alex]

(In [6531]) fixes #1669

[(none)]

Milestone 0.4.1 deleted