Do not allow attribute names of '0'

While working with dynamically created elements and attributes, it's
surprisingly easy to wind up with an attribute name of '0'.

When this happens, nomplate throws an arcane, disconnnected, async and
hard to debug exception because DOM elements do not allow attributes
with this name.

Could consider doing something more helpful too, but for now let's not
blow up.

Author: lukebayes

package.json

@@ -1,6 +1,6 @@
 {
   "name": "nomplate",
-  "version": "1.3.33",
+  "version": "1.3.34",
   "description": "Nomplate: Node template engine",
   "main": "index.js",
   "engines": {

src/operations.js

@@ -23,6 +23,9 @@ function setId(value) {
 function setAttribute(name, value) {
   return function _setAttribute(domElement, stack, document) {
     const updatedName = name === 'key' ? constants.NOM_ATTR_KEY : name;
+    if (updatedName === '0') {
+      return domElement;
+    }
 
     if (typeof value === 'boolean' && !value) {
       domElement.removeAttribute(updatedName);

test/render_element_test.js

@@ -62,6 +62,11 @@ describe('renderElement', () => {
       assert.equal(domElement.outerHTML, '<option value="0"></option>');
     });
 
+    it('does not apply attribute values if the key is "0"', () => {
+      const domElement = renderElement(dom.div({0: 'abcd', className: 'foo'}), doc);
+      assert.equal(domElement.outerHTML, '<div class="foo"></div>');
+    });
+
     it('creates children', () => {
       const domElement = renderElement(dom.div(() => {
         dom.ul(() => {