fire should only check for null and undefined.

[mbullington]

In the fire function, in file utils.js at line 153:

var detail = detail || {};

This restricts 0 or empty Strings from going through the fire function, when they are perfectly fine objects. So, fire should only check the detail variable for undefined or null, things that would cause a nasty error on the other side.

[sjmiles]

I made this commit (googlearchive/polymer-dev@43fe8b9) to prevent 0 from being squelched.

Normally, it would be good practice to have a test as well, but I'm making an exception because this is part of a larger issue.

Namely, this.fire has evolved two responsibilities (event sugaring and dom-based RPC), and I believe this method needs to be factored or at least revisited (in particular, why is detail not allowed to be null? the reason may be vestigial).

I'm going to leave this ticket open until I can link it to a more specific ticket for the larger question.

[mbullington]

Thanks!

[sjmiles]

https://github.com/Polymer/ToDoIssues/issues/45