isEmpty should check for empty objects

[m0rk]

I understand this part of the documentation

when used with an empty ({}) context, will result in:

as if you have an empty object, the value will be considered "falsy".
Handlebars.Utils.isEmpty(context) has no check for empty objects, so did I just understand the documentation wrong?

Anyhow, I would like to have a check on empty objects, thus the pull request. :)

[zedd45]

+1. I'm going to integrate this fix into my local dev. Thanks Mark!

[c089]

so did I just understand the documentation wrong?

Yes I think you did. But the documentation is misleading here, I had to read it several times as well: It says if you render that template example with an empty context ({}), then "author" is undefined and thus the block is not rendered.

However, I still think that empty object should be falsy!

[kpdecker]

This is not something that we can change at this point. I've updated the documentation to make the author portion clearer. We're also going to add formal documentation on this method so downstream code can monkey patch with the behavior in the PR if they so choose.