Merge pull request #879 from nordfjord/fix-linting

fix linting errors

docs/layout/tools/template-converter.js

@@ -1,5 +1,5 @@
 /* global m: false */
-// TODO: ensure this targets the current API.
+// NTBD: ensure this targets the current API.
 window.templateConverter = (function () {
 	"use strict"
 

mithril.js

@@ -44,8 +44,10 @@ void (function (global, factory) { // eslint-disable-line
 	}
 
 	function forOwn(obj, f) {
-		for (var prop in obj) if (hasOwn.call(obj, prop)) {
+		for (var prop in obj) {
-			if (f(obj[prop], prop)) break
+			if (hasOwn.call(obj, prop)) {
+				if (f(obj[prop], prop)) break
+			}
 		}
 	}
 
@@ -960,8 +962,10 @@ void (function (global, factory) { // eslint-disable-line
 				}
 			})
 
-			for (var rule in cachedAttr) if (hasOwn.call(cachedAttr, rule)) {
+			for (var rule in cachedAttr) {
-				if (!hasOwn.call(dataAttr, rule)) node.style[rule] = ""
+				if (hasOwn.call(cachedAttr, rule)) {
+					if (!hasOwn.call(dataAttr, rule)) node.style[rule] = ""
+				}
 			}
 		} else if (namespace != null) {
 			// handle SVG

test-deps/mock.js

@@ -33,8 +33,10 @@
 
 	Object.keys = Object.keys || function (obj) {
 		var keys = []
-		for (var i in obj) if ({}.hasOwnProperty.call(obj, i)) {
+		for (var i in obj) {
-			keys.push(i)
+			if ({}.hasOwnProperty.call(obj, i)) {
+				keys.push(i)
+			}
 		}
 		return keys
 	}
@@ -53,7 +55,7 @@ this.mock = (function (global) {
 	}
 
 	var document = window.document = {
-		// TODO: add document.createRange().createContextualFragment()
+		// NTBD: add document.createRange().createContextualFragment()
 
 		childNodes: [],
 
@@ -78,7 +80,7 @@ this.mock = (function (global) {
 				},
 
 				insertAdjacentHTML: function (position, html) {
-					// TODO: accept markup
+					// NTBD: accept markup
 					if (position === "beforebegin") {
 						this.parentNode.insertBefore(
 							document.createTextNode(html),