From 894144405a3619c16793d7cfa17ff43994a0f70b Mon Sep 17 00:00:00 2001
From: Toby Zerner <toby.zerner@gmail.com>
Date: Sat, 6 Feb 2016 18:58:34 +1030
Subject: [PATCH] Update Mithril

---
 framework/core/js/admin/dist/app.js | 2667 +++++++++++++++++----------
 framework/core/js/forum/dist/app.js | 2659 ++++++++++++++++----------
 2 files changed, 3387 insertions(+), 1939 deletions(-)

diff --git a/framework/core/js/admin/dist/app.js b/framework/core/js/admin/dist/app.js
index 06c9c20c3..2d229046d 100644
--- a/framework/core/js/admin/dist/app.js
+++ b/framework/core/js/admin/dist/app.js
@@ -607,42 +607,68 @@ exports.System = System;
 
 })(window);
 ;
-var m = (function app(window, undefined) {
-	"use strict";
-  	var VERSION = "v0.2.2-rc.1";
+;(function (global, factory) { // eslint-disable-line
+	"use strict"
+	/* eslint-disable no-undef */
+	var m = factory(global)
+	if (typeof module === "object" && module != null && module.exports) {
+		module.exports = m
+	} else if (typeof define === "function" && define.amd) {
+		define(function () { return m })
+	} else {
+		global.m = m
+	}
+	/* eslint-enable no-undef */
+})(typeof window !== "undefined" ? window : {}, function (global, undefined) { // eslint-disable-line
+	"use strict"
+
+	m.version = function () {
+		return "v0.2.2-rc.1"
+	}
+
+	var hasOwn = {}.hasOwnProperty
+	var type = {}.toString
+
 	function isFunction(object) {
-		return typeof object === "function";
+		return typeof object === "function"
 	}
+
 	function isObject(object) {
-		return type.call(object) === "[object Object]";
+		return type.call(object) === "[object Object]"
 	}
+
 	function isString(object) {
-		return type.call(object) === "[object String]";
+		return type.call(object) === "[object String]"
 	}
+
 	var isArray = Array.isArray || function (object) {
-		return type.call(object) === "[object Array]";
-	};
-	var type = {}.toString;
-	var parser = /(?:(^|#|\.)([^#\.\[\]]+))|(\[.+?\])/g, attrParser = /\[(.+?)(?:=("|'|)(.*?)\2)?\]/;
-	var voidElements = /^(AREA|BASE|BR|COL|COMMAND|EMBED|HR|IMG|INPUT|KEYGEN|LINK|META|PARAM|SOURCE|TRACK|WBR)$/;
-	var noop = function () {};
+		return type.call(object) === "[object Array]"
+	}
+
+	function noop() {}
+
+	/* eslint-disable max-len */
+	var voidElements = /^(AREA|BASE|BR|COL|COMMAND|EMBED|HR|IMG|INPUT|KEYGEN|LINK|META|PARAM|SOURCE|TRACK|WBR)$/
+	/* eslint-enable max-len */
 
 	// caching commonly used variables
-	var $document, $location, $requestAnimationFrame, $cancelAnimationFrame;
+	var $document, $location, $requestAnimationFrame, $cancelAnimationFrame
 
 	// self invoking function needed because of the way mocks work
-	function initialize(window) {
-		$document = window.document;
-		$location = window.location;
-		$cancelAnimationFrame = window.cancelAnimationFrame || window.clearTimeout;
-		$requestAnimationFrame = window.requestAnimationFrame || window.setTimeout;
+	function initialize(mock) {
+		$document = mock.document
+		$location = mock.location
+		$cancelAnimationFrame = mock.cancelAnimationFrame || mock.clearTimeout
+		$requestAnimationFrame = mock.requestAnimationFrame || mock.setTimeout
 	}
 
-	initialize(window);
+	// testing API
+	m.deps = function (mock) {
+		initialize(global = mock || window)
+		return global
+	}
 
-	m.version = function() {
-		return VERSION;
-	};
+	m.deps(global)
 
 	/**
 	 * @typedef {String} Tag
@@ -650,98 +676,148 @@ var m = (function app(window, undefined) {
 	 * Which describes a DOM node
 	 */
 
+	function parseTagAttrs(cell, tag) {
+		var classes = []
+		var parser = /(?:(^|#|\.)([^#\.\[\]]+))|(\[.+?\])/g
+		var match
+
+		while ((match = parser.exec(tag))) {
+			if (match[1] === "" && match[2]) {
+				cell.tag = match[2]
+			} else if (match[1] === "#") {
+				cell.attrs.id = match[2]
+			} else if (match[1] === ".") {
+				classes.push(match[2])
+			} else if (match[3][0] === "[") {
+				var pair = /\[(.+?)(?:=("|'|)(.*?)\2)?\]/.exec(match[3])
+				cell.attrs[pair[1]] = pair[3] || (pair[2] ? "" : true)
+			}
+		}
+
+		return classes
+	}
+
+	function getVirtualChildren(args, hasAttrs) {
+		var children = hasAttrs ? args.slice(1) : args
+
+		if (children.length === 1 && isArray(children[0])) {
+			return children[0]
+		} else {
+			return children
+		}
+	}
+
+	function assignAttrs(target, attrs, classes) {
+		var classAttr = "class" in attrs ? "class" : "className"
+
+		for (var attrName in attrs) {
+			if (hasOwn.call(attrs, attrName)) {
+				if (attrName === classAttr &&
+						attrs[attrName] != null &&
+						attrs[attrName] !== "") {
+					classes.push(attrs[attrName])
+					// create key in correct iteration order
+					target[attrName] = ""
+				} else {
+					target[attrName] = attrs[attrName]
+				}
+			}
+		}
+
+		if (classes.length) target[classAttr] = classes.join(" ")
+	}
+
 	/**
 	 *
 	 * @param {Tag} The DOM node tag
 	 * @param {Object=[]} optional key-value pairs to be mapped to DOM attrs
-	 * @param {...mNode=[]} Zero or more Mithril child nodes. Can be an array, or splat (optional)
-	 *
+	 * @param {...mNode=[]} Zero or more Mithril child nodes. Can be an array,
+	 *                      or splat (optional)
 	 */
 	function m(tag, pairs) {
 		for (var args = [], i = 1; i < arguments.length; i++) {
-			args[i - 1] = arguments[i];
-		}
-		if (isObject(tag)) return parameterize(tag, args);
-		var hasAttrs = pairs != null && isObject(pairs) && !("tag" in pairs || "view" in pairs || "subtree" in pairs);
-		var attrs = hasAttrs ? pairs : {};
-		var classAttrName = "class" in attrs ? "class" : "className";
-		var cell = {tag: "div", attrs: {}};
-		var match, classes = [];
-		if (!isString(tag)) throw new Error("selector in m(selector, attrs, children) should be a string");
-		while ((match = parser.exec(tag)) != null) {
-			if (match[1] === "" && match[2]) cell.tag = match[2];
-			else if (match[1] === "#") cell.attrs.id = match[2];
-			else if (match[1] === ".") classes.push(match[2]);
-			else if (match[3][0] === "[") {
-				var pair = attrParser.exec(match[3]);
-				cell.attrs[pair[1]] = pair[3] || (pair[2] ? "" :true);
-			}
+			args[i - 1] = arguments[i]
 		}
 
-		var children = hasAttrs ? args.slice(1) : args;
-		if (children.length === 1 && isArray(children[0])) {
-			cell.children = children[0];
-		}
-		else {
-			cell.children = children;
+		if (isObject(tag)) return parameterize(tag, args)
+
+		if (!isString(tag)) {
+			throw new Error("selector in m(selector, attrs, children) should " +
+				"be a string")
 		}
 
-		for (var attrName in attrs) {
-			if (attrs.hasOwnProperty(attrName)) {
-				if (attrName === classAttrName && attrs[attrName] != null && attrs[attrName] !== "") {
-					classes.push(attrs[attrName]);
-					cell.attrs[attrName] = ""; //create key in correct iteration order
-				}
-				else cell.attrs[attrName] = attrs[attrName];
-			}
-		}
-		if (classes.length) cell.attrs[classAttrName] = classes.join(" ");
+		var hasAttrs = pairs != null && isObject(pairs) &&
+			!("tag" in pairs || "view" in pairs || "subtree" in pairs)
 
-		return cell;
+		var attrs = hasAttrs ? pairs : {}
+		var cell = {
+			tag: "div",
+			attrs: {},
+			children: getVirtualChildren(args, hasAttrs)
+		}
+
+		assignAttrs(cell.attrs, attrs, parseTagAttrs(cell, tag))
+		return cell
 	}
+
 	function forEach(list, f) {
-		for (var i = 0; i < list.length && !f(list[i], i++);) {}
+		for (var i = 0; i < list.length && !f(list[i], i++);) {
+			// function called in condition
+		}
 	}
+
 	function forKeys(list, f) {
 		forEach(list, function (attrs, i) {
-			return (attrs = attrs && attrs.attrs) && attrs.key != null && f(attrs, i);
-		});
+			return (attrs = attrs && attrs.attrs) &&
+				attrs.key != null &&
+				f(attrs, i)
+		})
 	}
 	// This function was causing deopts in Chrome.
 	function dataToString(data) {
-		//data.toString() might throw or return null if data is the return value of Console.log in Firefox (behavior depends on version)
+		// data.toString() might throw or return null if data is the return
+		// value of Console.log in some versions of Firefox (behavior depends on
+		// version)
 		try {
-			if (data == null || data.toString() == null) return "";
+			if (data != null && data.toString() != null) return data
 		} catch (e) {
-			return "";
+			// silently ignore errors
 		}
-		return data;
+		return ""
 	}
+
 	// This function was causing deopts in Chrome.
 	function injectTextNode(parentElement, first, index, data) {
 		try {
-			insertNode(parentElement, first, index);
-			first.nodeValue = data;
-		} catch (e) {} //IE erroneously throws error when appending an empty text node after a null
+			insertNode(parentElement, first, index)
+			first.nodeValue = data
+		} catch (e) {
+			// IE erroneously throws error when appending an empty text node
+			// after a null
+		}
 	}
 
 	function flatten(list) {
-		//recursively flatten array
+		// recursively flatten array
 		for (var i = 0; i < list.length; i++) {
 			if (isArray(list[i])) {
-				list = list.concat.apply([], list);
-				//check current index again and flatten until there are no more nested arrays at that index
-				i--;
+				list = list.concat.apply([], list)
+				// check current index again and flatten until there are no more
+				// nested arrays at that index
+				i--
 			}
 		}
-		return list;
+		return list
 	}
 
 	function insertNode(parentElement, node, index) {
-		parentElement.insertBefore(node, parentElement.childNodes[index] || null);
+		parentElement.insertBefore(node,
+			parentElement.childNodes[index] || null)
 	}
 
-	var DELETION = 1, INSERTION = 2, MOVE = 3;
+	var DELETION = 1
+	var INSERTION = 2
+	var MOVE = 3
 
 	function handleKeysDiffer(data, existing, cached, parentElement) {
 		forKeys(data, function (key, i) {
@@ -749,167 +825,274 @@ var m = (function app(window, undefined) {
 				action: MOVE,
 				index: i,
 				from: existing[key].index,
-				element: cached.nodes[existing[key].index] || $document.createElement("div")
-			} : {action: INSERTION, index: i};
-		});
-		var actions = [];
-		for (var prop in existing) actions.push(existing[prop]);
-		var changes = actions.sort(sortChanges), newCached = new Array(cached.length);
-		newCached.nodes = cached.nodes.slice();
+				element: cached.nodes[existing[key].index] ||
+					$document.createElement("div")
+			} : {action: INSERTION, index: i}
+		})
+
+		var actions = []
+		for (var prop in existing) if (hasOwn.call(existing, prop)) {
+			actions.push(existing[prop])
+		}
+
+		var changes = actions.sort(sortChanges)
+		var newCached = new Array(cached.length)
+
+		newCached.nodes = cached.nodes.slice()
 
 		forEach(changes, function (change) {
-			var index = change.index;
+			var index = change.index
 			if (change.action === DELETION) {
-				clear(cached[index].nodes, cached[index]);
-				newCached.splice(index, 1);
+				clear(cached[index].nodes, cached[index])
+				newCached.splice(index, 1)
 			}
 			if (change.action === INSERTION) {
-				var dummy = $document.createElement("div");
-				dummy.key = data[index].attrs.key;
-				insertNode(parentElement, dummy, index);
+				var dummy = $document.createElement("div")
+				dummy.key = data[index].attrs.key
+				insertNode(parentElement, dummy, index)
 				newCached.splice(index, 0, {
 					attrs: {key: data[index].attrs.key},
 					nodes: [dummy]
-				});
-				newCached.nodes[index] = dummy;
+				})
+				newCached.nodes[index] = dummy
 			}
 
 			if (change.action === MOVE) {
-				var changeElement = change.element;
-				var maybeChanged = parentElement.childNodes[index];
+				var changeElement = change.element
+				var maybeChanged = parentElement.childNodes[index]
 				if (maybeChanged !== changeElement && changeElement !== null) {
-					parentElement.insertBefore(changeElement, maybeChanged || null);
+					parentElement.insertBefore(changeElement,
+						maybeChanged || null)
 				}
-				newCached[index] = cached[change.from];
-				newCached.nodes[index] = changeElement;
+				newCached[index] = cached[change.from]
+				newCached.nodes[index] = changeElement
 			}
-		});
+		})
 
-		return newCached;
+		return newCached
 	}
 
 	function diffKeys(data, cached, existing, parentElement) {
-		var keysDiffer = data.length !== cached.length;
+		var keysDiffer = data.length !== cached.length
+
 		if (!keysDiffer) {
 			forKeys(data, function (attrs, i) {
-				var cachedCell = cached[i];
-				return keysDiffer = cachedCell && cachedCell.attrs && cachedCell.attrs.key !== attrs.key;
-			});
+				var cachedCell = cached[i]
+				return keysDiffer = cachedCell &&
+					cachedCell.attrs &&
+					cachedCell.attrs.key !== attrs.key
+			})
 		}
 
-		return keysDiffer ? handleKeysDiffer(data, existing, cached, parentElement) : cached;
+		if (keysDiffer) {
+			return handleKeysDiffer(data, existing, cached, parentElement)
+		} else {
+			return cached
+		}
 	}
 
 	function diffArray(data, cached, nodes) {
-		//diff the array itself
+		// diff the array itself
 
-		//update the list of DOM nodes by collecting the nodes from each item
+		// update the list of DOM nodes by collecting the nodes from each item
 		forEach(data, function (_, i) {
-			if (cached[i] != null) nodes.push.apply(nodes, cached[i].nodes);
+			if (cached[i] != null) nodes.push.apply(nodes, cached[i].nodes)
 		})
-		//remove items from the end of the array if the new array is shorter than the old one. if errors ever happen here, the issue is most likely
-		//a bug in the construction of the `cached` data structure somewhere earlier in the program
+		// remove items from the end of the array if the new array is shorter
+		// than the old one. if errors ever happen here, the issue is most
+		// likely a bug in the construction of the `cached` data structure
+		// somewhere earlier in the program
 		forEach(cached.nodes, function (node, i) {
-			if (node.parentNode != null && nodes.indexOf(node) < 0) clear([node], [cached[i]]);
+			if (node.parentNode != null && nodes.indexOf(node) < 0) {
+				clear([node], [cached[i]])
+			}
 		})
-		if (data.length < cached.length) cached.length = data.length;
-		cached.nodes = nodes;
+
+		if (data.length < cached.length) cached.length = data.length
+		cached.nodes = nodes
 	}
 
 	function buildArrayKeys(data) {
-		var guid = 0;
+		var guid = 0
 		forKeys(data, function () {
 			forEach(data, function (attrs) {
-				if ((attrs = attrs && attrs.attrs) && attrs.key == null) attrs.key = "__mithril__" + guid++;
+				if ((attrs = attrs && attrs.attrs) && attrs.key == null) {
+					attrs.key = "__mithril__" + guid++
+				}
 			})
-			return 1;
-		});
+			return 1
+		})
+	}
+
+	function isDifferentEnough(data, cached, dataAttrKeys) {
+		if (data.tag !== cached.tag) return true
+
+		if (dataAttrKeys.sort().join() !==
+				Object.keys(cached.attrs).sort().join()) {
+			return true
+		}
+
+		if (data.attrs.id !== cached.attrs.id) {
+			return true
+		}
+
+		if (data.attrs.key !== cached.attrs.key) {
+			return true
+		}
+
+		if (m.redraw.strategy() === "all") {
+			return !cached.configContext || cached.configContext.retain !== true
+		}
+
+		if (m.redraw.strategy() === "diff") {
+			return cached.configContext && cached.configContext.retain === false
+		}
+
+		return false
 	}
 
 	function maybeRecreateObject(data, cached, dataAttrKeys) {
-		//if an element is different enough from the one in cache, recreate it
-		if (data.tag !== cached.tag ||
-				dataAttrKeys.sort().join() !== Object.keys(cached.attrs).sort().join() ||
-				data.attrs.id !== cached.attrs.id ||
-				data.attrs.key !== cached.attrs.key ||
-				(m.redraw.strategy() === "all" && (!cached.configContext || cached.configContext.retain !== true)) ||
-				(m.redraw.strategy() === "diff" && cached.configContext && cached.configContext.retain === false)) {
-			if (cached.nodes.length) clear(cached.nodes);
-			if (cached.configContext && isFunction(cached.configContext.onunload)) cached.configContext.onunload();
+		// if an element is different enough from the one in cache, recreate it
+		if (isDifferentEnough(data, cached, dataAttrKeys)) {
+			if (cached.nodes.length) clear(cached.nodes)
+
+			if (cached.configContext &&
+					isFunction(cached.configContext.onunload)) {
+				cached.configContext.onunload()
+			}
+
 			if (cached.controllers) {
 				forEach(cached.controllers, function (controller) {
-					if (controller.unload) controller.onunload({preventDefault: noop});
+					if (controller.onunload) controller.onunload({preventDefault: noop});
 				});
 			}
 		}
 	}
 
 	function getObjectNamespace(data, namespace) {
-		return data.attrs.xmlns ? data.attrs.xmlns :
-			data.tag === "svg" ? "http://www.w3.org/2000/svg" :
-			data.tag === "math" ? "http://www.w3.org/1998/Math/MathML" :
-			namespace;
+		if (data.attrs.xmlns) return data.attrs.xmlns
+		if (data.tag === "svg") return "http://www.w3.org/2000/svg"
+		if (data.tag === "math") return "http://www.w3.org/1998/Math/MathML"
+		return namespace
+	}
+
+	var pendingRequests = 0
+	m.startComputation = function () { pendingRequests++ }
+	m.endComputation = function () {
+		if (pendingRequests > 1) {
+			pendingRequests--
+		} else {
+			pendingRequests = 0
+			m.redraw()
+		}
 	}
 
 	function unloadCachedControllers(cached, views, controllers) {
 		if (controllers.length) {
-			cached.views = views;
-			cached.controllers = controllers;
+			cached.views = views
+			cached.controllers = controllers
 			forEach(controllers, function (controller) {
-				if (controller.onunload && controller.onunload.$old) controller.onunload = controller.onunload.$old;
-				if (pendingRequests && controller.onunload) {
-					var onunload = controller.onunload;
-					controller.onunload = noop;
-					controller.onunload.$old = onunload;
+				if (controller.onunload && controller.onunload.$old) {
+					controller.onunload = controller.onunload.$old
 				}
-			});
+
+				if (pendingRequests && controller.onunload) {
+					var onunload = controller.onunload
+					controller.onunload = noop
+					controller.onunload.$old = onunload
+				}
+			})
 		}
 	}
 
 	function scheduleConfigsToBeCalled(configs, data, node, isNew, cached) {
-		//schedule configs to be called. They are called after `build`
-		//finishes running
+		// schedule configs to be called. They are called after `build` finishes
+		// running
 		if (isFunction(data.attrs.config)) {
-			var context = cached.configContext = cached.configContext || {};
+			var context = cached.configContext = cached.configContext || {}
 
-			//bind
-			configs.push(function() {
-				return data.attrs.config.call(data, node, !isNew, context, cached);
-			});
+			// bind
+			configs.push(function () {
+				return data.attrs.config.call(data, node, !isNew, context,
+					cached)
+			})
 		}
 	}
 
-	function buildUpdatedNode(cached, data, editable, hasKeys, namespace, views, configs, controllers) {
-		var node = cached.nodes[0];
-		if (hasKeys) setAttributes(node, data.tag, data.attrs, cached.attrs, namespace);
-		cached.children = build(node, data.tag, undefined, undefined, data.children, cached.children, false, 0, data.attrs.contenteditable ? node : editable, namespace, configs);
-		cached.nodes.intact = true;
+	function buildUpdatedNode(
+		cached,
+		data,
+		editable,
+		hasKeys,
+		namespace,
+		views,
+		configs,
+		controllers
+	) {
+		var node = cached.nodes[0]
 
-		if (controllers.length) {
-			cached.views = views;
-			cached.controllers = controllers;
+		if (hasKeys) {
+			setAttributes(node, data.tag, data.attrs, cached.attrs, namespace)
 		}
 
-		return node;
+		cached.children = build(
+			node,
+			data.tag,
+			undefined,
+			undefined,
+			data.children,
+			cached.children,
+			false,
+			0,
+			data.attrs.contenteditable ? node : editable,
+			namespace,
+			configs
+		)
+
+		cached.nodes.intact = true
+
+		if (controllers.length) {
+			cached.views = views
+			cached.controllers = controllers
+		}
+
+		return node
 	}
 
 	function handleNonexistentNodes(data, parentElement, index) {
-		var nodes;
+		var nodes
 		if (data.$trusted) {
-			nodes = injectHTML(parentElement, index, data);
-		}
-		else {
-			nodes = [$document.createTextNode(data)];
-			if (!parentElement.nodeName.match(voidElements)) insertNode(parentElement, nodes[0], index);
+			nodes = injectHTML(parentElement, index, data)
+		} else {
+			nodes = [$document.createTextNode(data)]
+			if (!parentElement.nodeName.match(voidElements)) {
+				insertNode(parentElement, nodes[0], index)
+			}
 		}
 
-		var cached = typeof data === "string" || typeof data === "number" || typeof data === "boolean" ? new data.constructor(data) : data;
-		cached.nodes = nodes;
-		return cached;
+		var cached
+
+		if (typeof data === "string" ||
+				typeof data === "number" ||
+				typeof data === "boolean") {
+			cached = new data.constructor(data)
+		} else {
+			cached = data
+		}
+
+		cached.nodes = nodes
+		return cached
 	}
 
-	function reattachNodes(data, cached, parentElement, editable, index, parentTag) {
-		var nodes = cached.nodes;
+	function reattachNodes(
+		data,
+		cached,
+		parentElement,
+		editable,
+		index,
+		parentTag
+	) {
+		var nodes = cached.nodes
 		if (!editable || editable !== $document.activeElement) {
 			if (data.$trusted) {
 				clear(nodes, cached)
@@ -922,864 +1105,1370 @@ var m = (function app(window, undefined) {
 				editable.innerHTML = data
 			} else {
 				// was a trusted string
-				if (nodes[0].nodeType === 1 || nodes.length > 1 || (nodes[0].nodeValue.trim && !nodes[0].nodeValue.trim())) {
+				if (nodes[0].nodeType === 1 || nodes.length > 1 ||
+						(nodes[0].nodeValue.trim &&
+							!nodes[0].nodeValue.trim())) {
 					clear(cached.nodes, cached)
 					nodes = [$document.createTextNode(data)]
 				}
-				injectTextNode(parentElement, nodes[0], index, data);
+
+				injectTextNode(parentElement, nodes[0], index, data)
 			}
 		}
-		cached = new data.constructor(data);
-		cached.nodes = nodes;
-		return cached;
+		cached = new data.constructor(data)
+		cached.nodes = nodes
+		return cached
 	}
 
-	function handleText(cached, data, index, parentElement, shouldReattach, editable, parentTag) {
-		//handle text nodes
-		return cached.nodes.length === 0 ? handleNonexistentNodes(data, parentElement, index) :
-			cached.valueOf() !== data.valueOf() || shouldReattach === true ?
-				reattachNodes(data, cached, parentElement, editable, index, parentTag) :
-			(cached.nodes.intact = true, cached);
+	function handleTextNode(
+		cached,
+		data,
+		index,
+		parentElement,
+		shouldReattach,
+		editable,
+		parentTag
+	) {
+		if (!cached.nodes.length) {
+			return handleNonexistentNodes(data, parentElement, index)
+		} else if (cached.valueOf() !== data.valueOf() || shouldReattach) {
+			return reattachNodes(data, cached, parentElement, editable, index,
+				parentTag)
+		} else {
+			return (cached.nodes.intact = true, cached)
+		}
 	}
 
 	function getSubArrayCount(item) {
 		if (item.$trusted) {
-			//fix offset of next element if item was a trusted string w/ more than one html element
-			//the first clause in the regexp matches elements
-			//the second clause (after the pipe) matches text nodes
-			var match = item.match(/<[^\/]|\>\s*[^<]/g);
-			if (match != null) return match.length;
+			// fix offset of next element if item was a trusted string w/ more
+			// than one html element
+			// the first clause in the regexp matches elements
+			// the second clause (after the pipe) matches text nodes
+			var match = item.match(/<[^\/]|\>\s*[^<]/g)
+			if (match != null) return match.length
+		} else if (isArray(item)) {
+			return item.length
 		}
-		else if (isArray(item)) {
-			return item.length;
-		}
-		return 1;
+		return 1
 	}
 
-	function buildArray(data, cached, parentElement, index, parentTag, shouldReattach, editable, namespace, configs) {
-		data = flatten(data);
-		var nodes = [], intact = cached.length === data.length, subArrayCount = 0;
+	function buildArray(
+		data,
+		cached,
+		parentElement,
+		index,
+		parentTag,
+		shouldReattach,
+		editable,
+		namespace,
+		configs
+	) {
+		data = flatten(data)
+		var nodes = []
+		var intact = cached.length === data.length
+		var subArrayCount = 0
+
+		// keys algorithm: sort elements without recreating them if keys are
+		// present
+		//
+		// 1) create a map of all existing keys, and mark all for deletion
+		// 2) add new keys to map and mark them for addition
+		// 3) if key exists in new list, change action from deletion to a move
+		// 4) for each key, handle its corresponding action as marked in
+		//    previous steps
+
+		var existing = {}
+		var shouldMaintainIdentities = false
 
-		//keys algorithm: sort elements without recreating them if keys are present
-		//1) create a map of all existing keys, and mark all for deletion
-		//2) add new keys to map and mark them for addition
-		//3) if key exists in new list, change action from deletion to a move
-		//4) for each key, handle its corresponding action as marked in previous steps
-		var existing = {}, shouldMaintainIdentities = false;
 		forKeys(cached, function (attrs, i) {
-			shouldMaintainIdentities = true;
-			existing[cached[i].attrs.key] = {action: DELETION, index: i};
-		});
+			shouldMaintainIdentities = true
+			existing[cached[i].attrs.key] = {action: DELETION, index: i}
+		})
 
-		buildArrayKeys(data);
-		if (shouldMaintainIdentities) cached = diffKeys(data, cached, existing, parentElement);
-		//end key algorithm
+		buildArrayKeys(data)
+		if (shouldMaintainIdentities) {
+			cached = diffKeys(data, cached, existing, parentElement)
+		}
+		// end key algorithm
 
-		var cacheCount = 0;
-		//faster explicitly written
+		var cacheCount = 0
+		// faster explicitly written
 		for (var i = 0, len = data.length; i < len; i++) {
-			//diff each item in the array
-			var item = build(parentElement, parentTag, cached, index, data[i], cached[cacheCount], shouldReattach, index + subArrayCount || subArrayCount, editable, namespace, configs);
+			// diff each item in the array
+			var item = build(
+				parentElement,
+				parentTag,
+				cached,
+				index,
+				data[i],
+				cached[cacheCount],
+				shouldReattach,
+				index + subArrayCount || subArrayCount,
+				editable,
+				namespace,
+				configs)
 
 			if (item !== undefined) {
-				intact = intact && item.nodes.intact;
-				subArrayCount += getSubArrayCount(item);
-				cached[cacheCount++] = item;
+				intact = intact && item.nodes.intact
+				subArrayCount += getSubArrayCount(item)
+				cached[cacheCount++] = item
 			}
 		}
 
-		if (!intact) diffArray(data, cached, nodes);
+		if (!intact) diffArray(data, cached, nodes)
 		return cached
 	}
 
 	function makeCache(data, cached, index, parentIndex, parentCache) {
 		if (cached != null) {
-			if (type.call(cached) === type.call(data)) return cached;
+			if (type.call(cached) === type.call(data)) return cached
 
 			if (parentCache && parentCache.nodes) {
-				var offset = index - parentIndex, end = offset + (isArray(data) ? data : cached.nodes).length;
-				clear(parentCache.nodes.slice(offset, end), parentCache.slice(offset, end));
+				var offset = index - parentIndex
+				var end = offset + (isArray(data) ? data : cached.nodes).length
+				clear(
+					parentCache.nodes.slice(offset, end),
+					parentCache.slice(offset, end))
 			} else if (cached.nodes) {
-				clear(cached.nodes, cached);
+				clear(cached.nodes, cached)
 			}
 		}
 
-		cached = new data.constructor();
-		//if constructor creates a virtual dom element, use a blank object
-		//as the base cached node instead of copying the virtual el (#277)
-		if (cached.tag) cached = {};
-		cached.nodes = [];
-		return cached;
+		cached = new data.constructor()
+		// if constructor creates a virtual dom element, use a blank object as
+		// the base cached node instead of copying the virtual el (#277)
+		if (cached.tag) cached = {}
+		cached.nodes = []
+		return cached
 	}
 
 	function constructNode(data, namespace) {
-		return namespace === undefined ?
-			data.attrs.is ? $document.createElement(data.tag, data.attrs.is) : $document.createElement(data.tag) :
-			data.attrs.is ? $document.createElementNS(namespace, data.tag, data.attrs.is) : $document.createElementNS(namespace, data.tag);
+		if (data.attrs.is) {
+			if (namespace == null) {
+				return $document.createElement(data.tag, data.attrs.is)
+			} else {
+				return $document.createElementNS(namespace, data.tag,
+					data.attrs.is)
+			}
+		} else if (namespace == null) {
+			return $document.createElement(data.tag)
+		} else {
+			return $document.createElementNS(namespace, data.tag)
+		}
 	}
 
 	function constructAttrs(data, node, namespace, hasKeys) {
-		return hasKeys ? setAttributes(node, data.tag, data.attrs, {}, namespace) : data.attrs;
+		if (hasKeys) {
+			return setAttributes(node, data.tag, data.attrs, {}, namespace)
+		} else {
+			return data.attrs
+		}
 	}
 
-	function constructChildren(data, node, cached, editable, namespace, configs) {
-		return data.children != null && data.children.length > 0 ?
-			build(node, data.tag, undefined, undefined, data.children, cached.children, true, 0, data.attrs.contenteditable ? node : editable, namespace, configs) :
-			data.children;
+	function constructChildren(
+		data,
+		node,
+		cached,
+		editable,
+		namespace,
+		configs
+	) {
+		if (data.children != null && data.children.length > 0) {
+			return build(
+				node,
+				data.tag,
+				undefined,
+				undefined,
+				data.children,
+				cached.children,
+				true,
+				0,
+				data.attrs.contenteditable ? node : editable,
+				namespace,
+				configs)
+		} else {
+			return data.children
+		}
 	}
 
-	function reconstructCached(data, attrs, children, node, namespace, views, controllers) {
-		var cached = {tag: data.tag, attrs: attrs, children: children, nodes: [node]};
-		unloadCachedControllers(cached, views, controllers);
-		if (cached.children && !cached.children.nodes) cached.children.nodes = [];
-		//edge case: setting value on <select> doesn't work before children exist, so set it again after children have been created
-		if (data.tag === "select" && "value" in data.attrs) setAttributes(node, data.tag, {value: data.attrs.value}, {}, namespace);
+	function reconstructCached(
+		data,
+		attrs,
+		children,
+		node,
+		namespace,
+		views,
+		controllers
+	) {
+		var cached = {
+			tag: data.tag,
+			attrs: attrs,
+			children: children,
+			nodes: [node]
+		}
+
+		unloadCachedControllers(cached, views, controllers)
+
+		if (cached.children && !cached.children.nodes) {
+			cached.children.nodes = []
+		}
+
+		// edge case: setting value on <select> doesn't work before children
+		// exist, so set it again after children have been created
+		if (data.tag === "select" && "value" in data.attrs) {
+			setAttributes(node, data.tag, {value: data.attrs.value}, {},
+				namespace)
+		}
+
 		return cached
 	}
 
 	function getController(views, view, cachedControllers, controller) {
-		var controllerIndex = m.redraw.strategy() === "diff" && views ? views.indexOf(view) : -1;
-		return controllerIndex > -1 ? cachedControllers[controllerIndex] :
-			typeof controller === "function" ? new controller() : {};
+		var controllerIndex
+
+		if (m.redraw.strategy() === "diff" && views) {
+			controllerIndex = views.indexOf(view)
+		} else {
+			controllerIndex = -1
+		}
+
+		if (controllerIndex > -1) {
+			return cachedControllers[controllerIndex]
+		} else if (isFunction(controller)) {
+			return new controller()
+		} else {
+			return {}
+		}
 	}
 
+	var unloaders = []
+
 	function updateLists(views, controllers, view, controller) {
-		if (controller.onunload != null) unloaders.push({controller: controller, handler: controller.onunload});
-		views.push(view);
-		controllers.push(controller);
+		if (controller.onunload != null) {
+			unloaders.push({
+				controller: controller,
+				handler: controller.onunload
+			})
+		}
+
+		views.push(view)
+		controllers.push(controller)
 	}
 
+	var forcing = false
 	function checkView(data, view, cached, cachedControllers, controllers, views) {
-		var controller = getController(cached.views, view, cachedControllers, data.controller);
-		var key = (data && data.attrs && data.attrs.key);
-		data = pendingRequests === 0 || forcing || cachedControllers && cachedControllers.indexOf(controller) > -1 ? data.view(controller) : {tag: "placeholder"};
+		var controller = getController(cached.views, view, cachedControllers, data.controller)
+		var key = data && data.attrs && data.attrs.key
+		data = pendingRequests === 0 || forcing || cachedControllers && cachedControllers.indexOf(controller) > -1 ? data.view(controller) : {tag: "placeholder"}
 		if (data.subtree === "retain") return data;
-		(data.attrs = data.attrs || {}).key = key;
-		updateLists(views, controllers, view, controller);
-		return data;
+		data.attrs = data.attrs || {}
+		data.attrs.key = key
+		updateLists(views, controllers, view, controller)
+		return data
 	}
 
 	function markViews(data, cached, views, controllers) {
-		var cachedControllers = cached && cached.controllers;
-		while (data.view != null) data = checkView(data, data.view.$original || data.view, cached, cachedControllers, controllers, views);
-		return data;
+		var cachedControllers = cached && cached.controllers
+
+		while (data.view != null) {
+			data = checkView(
+				data,
+				data.view.$original || data.view,
+				cached,
+				cachedControllers,
+				controllers,
+				views)
+		}
+
+		return data
 	}
 
-	function buildObject(data, cached, editable, parentElement, index, shouldReattach, namespace, configs) {
-		var views = [], controllers = [];
-		data = markViews(data, cached, views, controllers);
-		if (data.subtree === "retain") return cached;
-		if (!data.tag && controllers.length) throw new Error("Component template must return a virtual element, not an array, string, etc.");
-		data.attrs = data.attrs || {};
-		cached.attrs = cached.attrs || {};
-		var dataAttrKeys = Object.keys(data.attrs);
-		var hasKeys = dataAttrKeys.length > ("key" in data.attrs ? 1 : 0);
-		maybeRecreateObject(data, cached, dataAttrKeys);
-		if (!isString(data.tag)) return;
-		var isNew = cached.nodes.length === 0;
-		namespace = getObjectNamespace(data, namespace);
-		var node;
+	function buildObject( // eslint-disable-line max-statements
+		data,
+		cached,
+		editable,
+		parentElement,
+		index,
+		shouldReattach,
+		namespace,
+		configs
+	) {
+		var views = []
+		var controllers = []
+
+		data = markViews(data, cached, views, controllers)
+
+		if (data.subtree === "retain") return cached
+
+		if (!data.tag && controllers.length) {
+			throw new Error("Component template must return a virtual " +
+				"element, not an array, string, etc.")
+		}
+
+		data.attrs = data.attrs || {}
+		cached.attrs = cached.attrs || {}
+
+		var dataAttrKeys = Object.keys(data.attrs)
+		var hasKeys = dataAttrKeys.length > ("key" in data.attrs ? 1 : 0)
+
+		maybeRecreateObject(data, cached, dataAttrKeys)
+
+		if (!isString(data.tag)) return
+
+		var isNew = cached.nodes.length === 0
+
+		namespace = getObjectNamespace(data, namespace)
+
+		var node
 		if (isNew) {
-			node = constructNode(data, namespace);
-			//set attributes first, then create children
+			node = constructNode(data, namespace)
+			// set attributes first, then create children
 			var attrs = constructAttrs(data, node, namespace, hasKeys)
-			var children = constructChildren(data, node, cached, editable, namespace, configs);
-			cached = reconstructCached(data, attrs, children, node, namespace, views, controllers);
+
+			var children = constructChildren(data, node, cached, editable,
+				namespace, configs)
+
+			cached = reconstructCached(
+				data,
+				attrs,
+				children,
+				node,
+				namespace,
+				views,
+				controllers)
+		} else {
+			node = buildUpdatedNode(
+				cached,
+				data,
+				editable,
+				hasKeys,
+				namespace,
+				views,
+				configs,
+				controllers)
 		}
-		else {
-			node = buildUpdatedNode(cached, data, editable, hasKeys, namespace, views, configs, controllers);
+
+		if (isNew || shouldReattach === true && node != null) {
+			insertNode(parentElement, node, index)
 		}
-		if (isNew || shouldReattach === true && node != null) insertNode(parentElement, node, index);
-		//schedule configs to be called. They are called after `build`
-		//finishes running
-		scheduleConfigsToBeCalled(configs, data, node, isNew, cached);
+
+		// The configs are called after `build` finishes running
+		scheduleConfigsToBeCalled(configs, data, node, isNew, cached)
+
 		return cached
 	}
 
-	function build(parentElement, parentTag, parentCache, parentIndex, data, cached, shouldReattach, index, editable, namespace, configs) {
-		//`build` is a recursive function that manages creation/diffing/removal
-		//of DOM elements based on comparison between `data` and `cached`
-		//the diff algorithm can be summarized as this:
-		//1 - compare `data` and `cached`
-		//2 - if they are different, copy `data` to `cached` and update the DOM
-		//    based on what the difference is
-		//3 - recursively apply this algorithm for every array and for the
-		//    children of every virtual element
+	function build(
+		parentElement,
+		parentTag,
+		parentCache,
+		parentIndex,
+		data,
+		cached,
+		shouldReattach,
+		index,
+		editable,
+		namespace,
+		configs
+	) {
+		/*
+		 * `build` is a recursive function that manages creation/diffing/removal
+		 * of DOM elements based on comparison between `data` and `cached` the
+		 * diff algorithm can be summarized as this:
+		 *
+		 * 1 - compare `data` and `cached`
+		 * 2 - if they are different, copy `data` to `cached` and update the DOM
+		 *     based on what the difference is
+		 * 3 - recursively apply this algorithm for every array and for the
+		 *     children of every virtual element
+		 *
+		 * The `cached` data structure is essentially the same as the previous
+		 * redraw's `data` data structure, with a few additions:
+		 * - `cached` always has a property called `nodes`, which is a list of
+		 *    DOM elements that correspond to the data represented by the
+		 *    respective virtual element
+		 * - in order to support attaching `nodes` as a property of `cached`,
+		 *    `cached` is *always* a non-primitive object, i.e. if the data was
+		 *    a string, then cached is a String instance. If data was `null` or
+		 *    `undefined`, cached is `new String("")`
+		 * - `cached also has a `configContext` property, which is the state
+		 *    storage object exposed by config(element, isInitialized, context)
+		 * - when `cached` is an Object, it represents a virtual element; when
+		 *    it's an Array, it represents a list of elements; when it's a
+		 *    String, Number or Boolean, it represents a text node
+		 *
+		 * `parentElement` is a DOM element used for W3C DOM API calls
+		 * `parentTag` is only used for handling a corner case for textarea
+		 * values
+		 * `parentCache` is used to remove nodes in some multi-node cases
+		 * `parentIndex` and `index` are used to figure out the offset of nodes.
+		 * They're artifacts from before arrays started being flattened and are
+		 * likely refactorable
+		 * `data` and `cached` are, respectively, the new and old nodes being
+		 * diffed
+		 * `shouldReattach` is a flag indicating whether a parent node was
+		 * recreated (if so, and if this node is reused, then this node must
+		 * reattach itself to the new parent)
+		 * `editable` is a flag that indicates whether an ancestor is
+		 * contenteditable
+		 * `namespace` indicates the closest HTML namespace as it cascades down
+		 * from an ancestor
+		 * `configs` is a list of config functions to run after the topmost
+		 * `build` call finishes running
+		 *
+		 * there's logic that relies on the assumption that null and undefined
+		 * data are equivalent to empty strings
+		 * - this prevents lifecycle surprises from procedural helpers that mix
+		 *   implicit and explicit return statements (e.g.
+		 *   function foo() {if (cond) return m("div")}
+		 * - it simplifies diffing code
+		 */
+		data = dataToString(data)
+		if (data.subtree === "retain") return cached
+		cached = makeCache(data, cached, index, parentIndex, parentCache)
 
-		//the `cached` data structure is essentially the same as the previous
-		//redraw's `data` data structure, with a few additions:
-		//- `cached` always has a property called `nodes`, which is a list of
-		//   DOM elements that correspond to the data represented by the
-		//   respective virtual element
-		//- in order to support attaching `nodes` as a property of `cached`,
-		//   `cached` is *always* a non-primitive object, i.e. if the data was
-		//   a string, then cached is a String instance. If data was `null` or
-		//   `undefined`, cached is `new String("")`
-		//- `cached also has a `configContext` property, which is the state
-		//   storage object exposed by config(element, isInitialized, context)
-		//- when `cached` is an Object, it represents a virtual element; when
-		//   it's an Array, it represents a list of elements; when it's a
-		//   String, Number or Boolean, it represents a text node
-
-		//`parentElement` is a DOM element used for W3C DOM API calls
-		//`parentTag` is only used for handling a corner case for textarea
-		//values
-		//`parentCache` is used to remove nodes in some multi-node cases
-		//`parentIndex` and `index` are used to figure out the offset of nodes.
-		//They're artifacts from before arrays started being flattened and are
-		//likely refactorable
-		//`data` and `cached` are, respectively, the new and old nodes being
-		//diffed
-		//`shouldReattach` is a flag indicating whether a parent node was
-		//recreated (if so, and if this node is reused, then this node must
-		//reattach itself to the new parent)
-		//`editable` is a flag that indicates whether an ancestor is
-		//contenteditable
-		//`namespace` indicates the closest HTML namespace as it cascades down
-		//from an ancestor
-		//`configs` is a list of config functions to run after the topmost
-		//`build` call finishes running
-
-		//there's logic that relies on the assumption that null and undefined
-		//data are equivalent to empty strings
-		//- this prevents lifecycle surprises from procedural helpers that mix
-		//  implicit and explicit return statements (e.g.
-		//  function foo() {if (cond) return m("div")}
-		//- it simplifies diffing code
-		data = dataToString(data);
-		if (data.subtree === "retain") return cached;
-		cached = makeCache(data, cached, index, parentIndex, parentCache);
-		return isArray(data) ? buildArray(data, cached, parentElement, index, parentTag, shouldReattach, editable, namespace, configs) :
-			data != null && isObject(data) ? buildObject(data, cached, editable, parentElement, index, shouldReattach, namespace, configs) :
-			!isFunction(data) ? handleText(cached, data, index, parentElement, shouldReattach, editable, parentTag) :
-			cached;
+		if (isArray(data)) {
+			return buildArray(
+				data,
+				cached,
+				parentElement,
+				index,
+				parentTag,
+				shouldReattach,
+				editable,
+				namespace,
+				configs)
+		} else if (data != null && isObject(data)) {
+			return buildObject(
+				data,
+				cached,
+				editable,
+				parentElement,
+				index,
+				shouldReattach,
+				namespace,
+				configs)
+		} else if (!isFunction(data)) {
+			return handleTextNode(
+				cached,
+				data,
+				index,
+				parentElement,
+				shouldReattach,
+				editable,
+				parentTag)
+		} else {
+			return cached
+		}
 	}
-	function sortChanges(a, b) { return a.action - b.action || a.index - b.index; }
-	function setAttributes(node, tag, dataAttrs, cachedAttrs, namespace) {
-		for (var attrName in dataAttrs) {
-			var dataAttr = dataAttrs[attrName];
-			var cachedAttr = cachedAttrs[attrName];
-			if (!(attrName in cachedAttrs) || (cachedAttr !== dataAttr)) {
-				cachedAttrs[attrName] = dataAttr;
-				try {
-					//`config` isn't a real attributes, so ignore it
-					if (attrName === "config" || attrName === "key") continue;
-					//hook event handlers to the auto-redrawing system
-					else if (isFunction(dataAttr) && attrName.slice(0, 2) === "on") {
-						node[attrName] = autoredraw(dataAttr, node);
-					}
-					//handle `style: {...}`
-					else if (attrName === "style" && dataAttr != null && isObject(dataAttr)) {
-						for (var rule in dataAttr) {
-							if (cachedAttr == null || cachedAttr[rule] !== dataAttr[rule]) node.style[rule] = dataAttr[rule];
-						}
-						for (var rule in cachedAttr) {
-							if (!(rule in dataAttr)) node.style[rule] = "";
-						}
-					}
-					//handle SVG
-					else if (namespace != null) {
-						if (attrName === "href") node.setAttributeNS("http://www.w3.org/1999/xlink", "href", dataAttr);
-						else node.setAttribute(attrName === "className" ? "class" : attrName, dataAttr);
-					}
-					//handle cases that are properties (but ignore cases where we should use setAttribute instead)
-					//- list and form are typically used as strings, but are DOM element references in js
-					//- when using CSS selectors (e.g. `m("[style='']")`), style is used as a string, but it's an object in js
-					else if (attrName in node && attrName !== "list" && attrName !== "style" && attrName !== "form" && attrName !== "type" && attrName !== "width" && attrName !== "height") {
-						//#348 don't set the value if not needed otherwise cursor placement breaks in Chrome
-						try {
-							if (tag !== "input" || node[attrName] !== dataAttr) node[attrName] = dataAttr;
-						} catch (e) {
-							node.setAttribute(attrName, dataAttr);
-						}
-					}
-					else node.setAttribute(attrName, dataAttr);
-				}
-				catch (e) {
-					//swallow IE's invalid argument errors to mimic HTML's fallback-to-doing-nothing-on-invalid-attributes behavior
-					if (e.message.indexOf("Invalid argument") < 0) throw e;
-				}
-			}
-			//#348 dataAttr may not be a string, so use loose comparison (double equal) instead of strict (triple equal)
-			else if (attrName === "value" && tag === "input" && node.value != dataAttr) {
-				node.value = dataAttr;
+
+	function sortChanges(a, b) {
+		return a.action - b.action || a.index - b.index
+	}
+
+	function copyStyleAttrs(node, dataAttr, cachedAttr) {
+		for (var rule in dataAttr) if (hasOwn.call(dataAttr, rule)) {
+			if (cachedAttr == null || cachedAttr[rule] !== dataAttr[rule]) {
+				node.style[rule] = dataAttr[rule]
 			}
 		}
-		return cachedAttrs;
+
+		for (rule in cachedAttr) if (hasOwn.call(cachedAttr, rule)) {
+			if (!hasOwn.call(dataAttr, rule)) node.style[rule] = ""
+		}
 	}
+
+	function shouldUseSetAttribute(attrName) {
+		return attrName !== "list" &&
+			attrName !== "style" &&
+			attrName !== "form" &&
+			attrName !== "type" &&
+			attrName !== "width" &&
+			attrName !== "height"
+	}
+
+	function setSingleAttr(
+		node,
+		attrName,
+		dataAttr,
+		cachedAttr,
+		tag,
+		namespace
+	) {
+		if (attrName === "config" || attrName === "key") {
+			// `config` isn't a real attribute, so ignore it
+			return true
+		} else if (isFunction(dataAttr) && attrName.slice(0, 2) === "on") {
+			// hook event handlers to the auto-redrawing system
+			node[attrName] = autoredraw(dataAttr, node)
+		} else if (attrName === "style" && dataAttr != null &&
+				isObject(dataAttr)) {
+			// handle `style: {...}`
+			copyStyleAttrs(node, dataAttr, cachedAttr)
+		} else if (namespace != null) {
+			// handle SVG
+			if (attrName === "href") {
+				node.setAttributeNS("http://www.w3.org/1999/xlink",
+					"href", dataAttr)
+			} else {
+				node.setAttribute(
+					attrName === "className" ? "class" : attrName,
+					dataAttr)
+			}
+		} else if (attrName in node && shouldUseSetAttribute(attrName)) {
+			// handle cases that are properties (but ignore cases where we
+			// should use setAttribute instead)
+			//
+			// - list and form are typically used as strings, but are DOM
+			//   element references in js
+			//
+			// - when using CSS selectors (e.g. `m("[style='']")`), style is
+			//   used as a string, but it's an object in js
+			//
+			// #348 don't set the value if not needed - otherwise, cursor
+			// placement breaks in Chrome
+			try {
+				if (tag !== "input" || node[attrName] !== dataAttr) {
+					node[attrName] = dataAttr
+				}
+			} catch (e) {
+				node.setAttribute(attrName, dataAttr)
+			}
+		}
+		else node.setAttribute(attrName, dataAttr)
+	}
+
+	function trySetAttr(
+		node,
+		attrName,
+		dataAttr,
+		cachedAttr,
+		cachedAttrs,
+		tag,
+		namespace
+	) {
+		if (!(attrName in cachedAttrs) || (cachedAttr !== dataAttr)) {
+			cachedAttrs[attrName] = dataAttr
+			try {
+				return setSingleAttr(
+					node,
+					attrName,
+					dataAttr,
+					cachedAttr,
+					tag,
+					namespace)
+			} catch (e) {
+				// swallow IE's invalid argument errors to mimic HTML's
+				// fallback-to-doing-nothing-on-invalid-attributes behavior
+				if (e.message.indexOf("Invalid argument") < 0) throw e
+			}
+		} else if (attrName === "value" && tag === "input" &&
+				node.value !== dataAttr) {
+			// #348 dataAttr may not be a string, so use loose comparison
+			node.value = dataAttr
+		}
+	}
+
+	function setAttributes(node, tag, dataAttrs, cachedAttrs, namespace) {
+		for (var attrName in dataAttrs) if (hasOwn.call(dataAttrs, attrName)) {
+			if (trySetAttr(
+					node,
+					attrName,
+					dataAttrs[attrName],
+					cachedAttrs[attrName],
+					cachedAttrs,
+					tag,
+					namespace)) {
+				continue
+			}
+		}
+		return cachedAttrs
+	}
+
 	function clear(nodes, cached) {
 		for (var i = nodes.length - 1; i > -1; i--) {
 			if (nodes[i] && nodes[i].parentNode) {
-				try { nodes[i].parentNode.removeChild(nodes[i]); }
-				catch (e) {} //ignore if this fails due to order of events (see http://stackoverflow.com/questions/21926083/failed-to-execute-removechild-on-node)
-				cached = [].concat(cached);
-				if (cached[i]) unload(cached[i]);
+				try {
+					nodes[i].parentNode.removeChild(nodes[i])
+				} catch (e) {
+					/* eslint-disable max-len */
+					// ignore if this fails due to order of events (see
+					// http://stackoverflow.com/questions/21926083/failed-to-execute-removechild-on-node)
+					/* eslint-enable max-len */
+				}
+				cached = [].concat(cached)
+				if (cached[i]) unload(cached[i])
 			}
 		}
-		//release memory if nodes is an array. This check should fail if nodes is a NodeList (see loop above)
-		if (nodes.length) nodes.length = 0;
+		// release memory if nodes is an array. This check should fail if nodes
+		// is a NodeList (see loop above)
+		if (nodes.length) {
+			nodes.length = 0
+		}
 	}
+
 	function unload(cached) {
 		if (cached.configContext && isFunction(cached.configContext.onunload)) {
-			cached.configContext.onunload();
-			cached.configContext.onunload = null;
+			cached.configContext.onunload()
+			cached.configContext.onunload = null
 		}
 		if (cached.controllers) {
 			forEach(cached.controllers, function (controller) {
-				if (isFunction(controller.onunload)) controller.onunload({preventDefault: noop});
-			});
+				if (isFunction(controller.onunload)) {
+					controller.onunload({preventDefault: noop})
+				}
+			})
 		}
 		if (cached.children) {
-			if (isArray(cached.children)) forEach(cached.children, unload);
-			else if (cached.children.tag) unload(cached.children);
+			if (isArray(cached.children)) forEach(cached.children, unload)
+			else if (cached.children.tag) unload(cached.children)
 		}
 	}
+
 	function appendTextFragment(parentElement, data) {
-	  try {
-	    parentElement.appendChild($document.createRange().createContextualFragment(data));
-	  } catch (e) {
-	    parentElement.insertAdjacentHTML("beforeend", data);
-	  }
+		try {
+			parentElement.appendChild(
+				$document.createRange().createContextualFragment(data))
+		} catch (e) {
+			parentElement.insertAdjacentHTML("beforeend", data)
+		}
 	}
+
 	function injectHTML(parentElement, index, data) {
-		var nextSibling = parentElement.childNodes[index];
+		var nextSibling = parentElement.childNodes[index]
 		if (nextSibling) {
-			var isElement = nextSibling.nodeType !== 1;
-			var placeholder = $document.createElement("span");
+			var isElement = nextSibling.nodeType !== 1
+			var placeholder = $document.createElement("span")
 			if (isElement) {
-				parentElement.insertBefore(placeholder, nextSibling || null);
-				placeholder.insertAdjacentHTML("beforebegin", data);
-				parentElement.removeChild(placeholder);
+				parentElement.insertBefore(placeholder, nextSibling || null)
+				placeholder.insertAdjacentHTML("beforebegin", data)
+				parentElement.removeChild(placeholder)
+			} else {
+				nextSibling.insertAdjacentHTML("beforebegin", data)
 			}
-			else nextSibling.insertAdjacentHTML("beforebegin", data);
+		} else {
+			appendTextFragment(parentElement, data)
 		}
-		else {
-			appendTextFragment(parentElement, data);
-		}
-		var nodes = [];
+
+		var nodes = []
+
 		while (parentElement.childNodes[index] !== nextSibling) {
-			nodes.push(parentElement.childNodes[index]);
-			index++;
+			nodes.push(parentElement.childNodes[index])
+			index++
 		}
-		return nodes;
+
+		return nodes
 	}
+
 	function autoredraw(callback, object) {
-		return function(e) {
-			e = e || event;
-			m.redraw.strategy("diff");
-			m.startComputation();
-			try { return callback.call(object, e); }
-			finally {
-				endFirstComputation();
+		return function (e) {
+			e = e || event
+			m.redraw.strategy("diff")
+			m.startComputation()
+			try {
+				return callback.call(object, e)
+			} finally {
+				endFirstComputation()
 			}
-		};
+		}
 	}
 
-	var html;
+	var html
 	var documentNode = {
-		appendChild: function(node) {
-			if (html === undefined) html = $document.createElement("html");
-			if ($document.documentElement && $document.documentElement !== node) {
-				$document.replaceChild(node, $document.documentElement);
+		appendChild: function (node) {
+			if (html === undefined) html = $document.createElement("html")
+			if ($document.documentElement &&
+					$document.documentElement !== node) {
+				$document.replaceChild(node, $document.documentElement)
+			} else {
+				$document.appendChild(node)
 			}
-			else $document.appendChild(node);
-			this.childNodes = $document.childNodes;
+
+			this.childNodes = $document.childNodes
 		},
-		insertBefore: function(node) {
-			this.appendChild(node);
+
+		insertBefore: function (node) {
+			this.appendChild(node)
 		},
+
 		childNodes: []
-	};
-	var nodeCache = [], cellCache = {};
-	m.render = function(root, cell, forceRecreation) {
-		var configs = [];
-		if (!root) throw new Error("Ensure the DOM element being passed to m.route/m.mount/m.render is not undefined.");
-		var id = getCellCacheKey(root);
-		var isDocumentRoot = root === $document;
-		var node = isDocumentRoot || root === $document.documentElement ? documentNode : root;
-		if (isDocumentRoot && cell.tag !== "html") cell = {tag: "html", attrs: {}, children: cell};
-		if (cellCache[id] === undefined) clear(node.childNodes);
-		if (forceRecreation === true) reset(root);
-		cellCache[id] = build(node, null, undefined, undefined, cell, cellCache[id], false, 0, null, undefined, configs);
-		forEach(configs, function (config) { config(); });
-	};
-	function getCellCacheKey(element) {
-		var index = nodeCache.indexOf(element);
-		return index < 0 ? nodeCache.push(element) - 1 : index;
 	}
 
-	m.trust = function(value) {
-		value = new String(value);
-		value.$trusted = true;
-		return value;
-	};
+	var nodeCache = []
+	var cellCache = {}
+
+	m.render = function (root, cell, forceRecreation) {
+		if (!root) {
+			throw new Error("Ensure the DOM element being passed to " +
+				"m.route/m.mount/m.render is not undefined.")
+		}
+		var configs = []
+		var id = getCellCacheKey(root)
+		var isDocumentRoot = root === $document
+		var node
+
+		if (isDocumentRoot || root === $document.documentElement) {
+			node = documentNode
+		} else {
+			node = root
+		}
+
+		if (isDocumentRoot && cell.tag !== "html") {
+			cell = {tag: "html", attrs: {}, children: cell}
+		}
+
+		if (cellCache[id] === undefined) clear(node.childNodes)
+		if (forceRecreation === true) reset(root)
+
+		cellCache[id] = build(
+			node,
+			null,
+			undefined,
+			undefined,
+			cell,
+			cellCache[id],
+			false,
+			0,
+			null,
+			undefined,
+			configs)
+
+		forEach(configs, function (config) { config() })
+	}
+
+	function getCellCacheKey(element) {
+		var index = nodeCache.indexOf(element)
+		return index < 0 ? nodeCache.push(element) - 1 : index
+	}
+
+	m.trust = function (value) {
+		value = new String(value) // eslint-disable-line no-new-wrappers
+		value.$trusted = true
+		return value
+	}
 
 	function gettersetter(store) {
-		var prop = function() {
-			if (arguments.length) store = arguments[0];
-			return store;
-		};
+		function prop() {
+			if (arguments.length) store = arguments[0]
+			return store
+		}
 
-		prop.toJSON = function() {
-			return store;
-		};
+		prop.toJSON = function () {
+			return store
+		}
 
-		return prop;
+		return prop
 	}
 
 	m.prop = function (store) {
-		//note: using non-strict equality check here because we're checking if store is null OR undefined
-		if ((store != null && isObject(store) || isFunction(store)) && isFunction(store.then)) {
-			return propify(store);
+		if ((store != null && isObject(store) || isFunction(store)) &&
+				isFunction(store.then)) {
+			return propify(store)
 		}
 
-		return gettersetter(store);
-	};
-
-	var roots = [], components = [], controllers = [], lastRedrawId = null, lastRedrawCallTime = 0, computePreRedrawHook = null, computePostRedrawHook = null, topComponent, unloaders = [];
-	var FRAME_BUDGET = 16; //60 frames per second = 1 call per 16 ms
-	function parameterize(component, args) {
-		var controller = function() {
-			return (component.controller || noop).apply(this, args) || this;
-		};
-		if (component.controller) controller.prototype = component.controller.prototype;
-		var view = function(ctrl) {
-			var currentArgs = arguments.length > 1 ? args.concat([].slice.call(arguments, 1)) : args;
-			return component.view.apply(component, currentArgs ? [ctrl].concat(currentArgs) : [ctrl]);
-		};
-		view.$original = component.view;
-		var output = {controller: controller, view: view};
-		if (args[0] && args[0].key != null) output.attrs = {key: args[0].key};
-		return output;
+		return gettersetter(store)
 	}
-	m.component = function(component) {
-		for (var args = [], i = 1; i < arguments.length; i++) args.push(arguments[i]);
-		return parameterize(component, args);
-	};
-	m.mount = m.module = function(root, component) {
-		if (!root) throw new Error("Please ensure the DOM element exists before rendering a template into it.");
-		var index = roots.indexOf(root);
-		if (index < 0) index = roots.length;
 
-		var isPrevented = false;
-		var event = {preventDefault: function() {
-			isPrevented = true;
-			computePreRedrawHook = computePostRedrawHook = null;
-		}};
+	var roots = []
+	var components = []
+	var controllers = []
+	var lastRedrawId = null
+	var lastRedrawCallTime = 0
+	var computePreRedrawHook = null
+	var computePostRedrawHook = null
+	var topComponent
+	var FRAME_BUDGET = 16 // 60 frames per second = 1 call per 16 ms
+
+	function parameterize(component, args) {
+		function controller() {
+			/* eslint-disable no-invalid-this */
+			return (component.controller || noop).apply(this, args) || this
+			/* eslint-enable no-invalid-this */
+		}
+
+		if (component.controller) {
+			controller.prototype = component.controller.prototype
+		}
+
+		function view(ctrl) {
+			var currentArgs = [ctrl].concat(args)
+			for (var i = 1; i < arguments.length; i++) {
+				currentArgs.push(arguments[i])
+			}
+
+			return component.view.apply(component, currentArgs)
+		}
+
+		view.$original = component.view
+		var output = {controller: controller, view: view}
+		if (args[0] && args[0].key != null) output.attrs = {key: args[0].key}
+		return output
+	}
+
+	m.component = function (component) {
+		for (var args = [], i = 1; i < arguments.length; i++) {
+			args.push(arguments[i])
+		}
+
+		return parameterize(component, args)
+	}
+
+	function checkPrevented(component, root, index, isPrevented) {
+		if (!isPrevented) {
+			m.redraw.strategy("all")
+			m.startComputation()
+			roots[index] = root
+			var currentComponent
+
+			if (component) {
+				currentComponent = topComponent = component
+			} else {
+				currentComponent = topComponent = component = {controller: noop}
+			}
+
+			var controller = new (component.controller || noop)()
+
+			// controllers may call m.mount recursively (via m.route redirects,
+			// for example)
+			// this conditional ensures only the last recursive m.mount call is
+			// applied
+			if (currentComponent === topComponent) {
+				controllers[index] = controller
+				components[index] = component
+			}
+			endFirstComputation()
+			if (component === null) {
+				removeRootElement(root, index)
+			}
+			return controllers[index]
+		} else if (component == null) {
+			removeRootElement(root, index)
+		}
+	}
+
+	m.mount = m.module = function (root, component) {
+		if (!root) {
+			throw new Error("Please ensure the DOM element exists before " +
+				"rendering a template into it.")
+		}
+
+		var index = roots.indexOf(root)
+		if (index < 0) index = roots.length
+
+		var isPrevented = false
+		var event = {
+			preventDefault: function () {
+				isPrevented = true
+				computePreRedrawHook = computePostRedrawHook = null
+			}
+		}
 
 		forEach(unloaders, function (unloader) {
-			unloader.handler.call(unloader.controller, event);
-			unloader.controller.onunload = null;
-		});
+			unloader.handler.call(unloader.controller, event)
+			unloader.controller.onunload = null
+		})
 
 		if (isPrevented) {
 			forEach(unloaders, function (unloader) {
-				unloader.controller.onunload = unloader.handler;
-			});
+				unloader.controller.onunload = unloader.handler
+			})
+		} else {
+			unloaders = []
 		}
-		else unloaders = [];
 
 		if (controllers[index] && isFunction(controllers[index].onunload)) {
-			controllers[index].onunload(event);
+			controllers[index].onunload(event)
 		}
 
-		var isNullComponent = component === null;
-
-		if (!isPrevented) {
-			m.redraw.strategy("all");
-			m.startComputation();
-			roots[index] = root;
-			var currentComponent = component ? (topComponent = component) : (topComponent = component = {controller: noop});
-			var controller = new (component.controller || noop)();
-			//controllers may call m.mount recursively (via m.route redirects, for example)
-			//this conditional ensures only the last recursive m.mount call is applied
-			if (currentComponent === topComponent) {
-				controllers[index] = controller;
-				components[index] = component;
-			}
-			endFirstComputation();
-			if (isNullComponent) {
-				removeRootElement(root, index);
-			}
-			return controllers[index];
-		}
-		if (isNullComponent) {
-			removeRootElement(root, index);
-		}
-	};
+		return checkPrevented(component, root, index, isPrevented)
+	}
 
 	function removeRootElement(root, index) {
-		roots.splice(index, 1);
-		controllers.splice(index, 1);
-		components.splice(index, 1);
-		reset(root);
-		nodeCache.splice(getCellCacheKey(root), 1);
+		roots.splice(index, 1)
+		controllers.splice(index, 1)
+		components.splice(index, 1)
+		reset(root)
+		nodeCache.splice(getCellCacheKey(root), 1)
 	}
 
-	var redrawing = false, forcing = false;
-	m.redraw = function(force) {
-		if (redrawing) return;
-		redrawing = true;
-		if (force) forcing = true;
+	var redrawing = false
+	m.redraw = function (force) {
+		if (redrawing) return
+		redrawing = true
+		if (force) forcing = true
+
 		try {
-			//lastRedrawId is a positive number if a second redraw is requested before the next animation frame
-			//lastRedrawID is null if it's the first redraw and not an event handler
+			// lastRedrawId is a positive number if a second redraw is requested
+			// before the next animation frame
+			// lastRedrawID is null if it's the first redraw and not an event
+			// handler
 			if (lastRedrawId && !force) {
-				//when setTimeout: only reschedule redraw if time between now and previous redraw is bigger than a frame, otherwise keep currently scheduled timeout
-				//when rAF: always reschedule redraw
-				if ($requestAnimationFrame === window.requestAnimationFrame || new Date - lastRedrawCallTime > FRAME_BUDGET) {
-					if (lastRedrawId > 0) $cancelAnimationFrame(lastRedrawId);
-					lastRedrawId = $requestAnimationFrame(redraw, FRAME_BUDGET);
+				// when setTimeout: only reschedule redraw if time between now
+				// and previous redraw is bigger than a frame, otherwise keep
+				// currently scheduled timeout
+				// when rAF: always reschedule redraw
+				if ($requestAnimationFrame === global.requestAnimationFrame ||
+						new Date() - lastRedrawCallTime > FRAME_BUDGET) {
+					if (lastRedrawId > 0) $cancelAnimationFrame(lastRedrawId)
+					lastRedrawId = $requestAnimationFrame(redraw, FRAME_BUDGET)
 				}
+			} else {
+				redraw()
+				lastRedrawId = $requestAnimationFrame(function () {
+					lastRedrawId = null
+				}, FRAME_BUDGET)
 			}
-			else {
-				redraw();
-				lastRedrawId = $requestAnimationFrame(function() { lastRedrawId = null; }, FRAME_BUDGET);
-			}
+		} finally {
+			redrawing = forcing = false
 		}
-		finally {
-			redrawing = forcing = false;
-		}
-	};
-	m.redraw.strategy = m.prop();
+	}
+
+	m.redraw.strategy = m.prop()
 	function redraw() {
 		if (computePreRedrawHook) {
-			computePreRedrawHook();
-			computePreRedrawHook = null;
+			computePreRedrawHook()
+			computePreRedrawHook = null
 		}
 		forEach(roots, function (root, i) {
-			var component = components[i];
+			var component = components[i]
 			if (controllers[i]) {
-				var args = [controllers[i]];
-				m.render(root, component.view ? component.view(controllers[i], args) : "");
+				var args = [controllers[i]]
+				m.render(root,
+					component.view ? component.view(controllers[i], args) : "")
 			}
-		});
-		//after rendering within a routed context, we need to scroll back to the top, and fetch the document title for history.pushState
+		})
+		// after rendering within a routed context, we need to scroll back to
+		// the top, and fetch the document title for history.pushState
 		if (computePostRedrawHook) {
-			computePostRedrawHook();
-			computePostRedrawHook = null;
-		}
-		lastRedrawId = null;
-		lastRedrawCallTime = new Date;
-		m.redraw.strategy("diff");
-	}
-
-	var pendingRequests = 0;
-	m.startComputation = function() { pendingRequests++; };
-	m.endComputation = function() {
-		if (pendingRequests > 1) pendingRequests--;
-		else {
-			pendingRequests = 0;
-			m.redraw();
+			computePostRedrawHook()
+			computePostRedrawHook = null
 		}
+		lastRedrawId = null
+		lastRedrawCallTime = new Date()
+		m.redraw.strategy("diff")
 	}
 
 	function endFirstComputation() {
 		if (m.redraw.strategy() === "none") {
-			pendingRequests--;
-			m.redraw.strategy("diff");
+			pendingRequests--
+			m.redraw.strategy("diff")
+		} else {
+			m.endComputation()
 		}
-		else m.endComputation();
 	}
 
-	m.withAttr = function(prop, withAttrCallback, callbackThis) {
-		return function(e) {
-			e = e || event;
-			var currentTarget = e.currentTarget || this;
-			var _this = callbackThis || this;
-			withAttrCallback.call(_this, prop in currentTarget ? currentTarget[prop] : currentTarget.getAttribute(prop));
-		};
-	};
+	m.withAttr = function (prop, withAttrCallback, callbackThis) {
+		return function (e) {
+			e = e || event
+			/* eslint-disable no-invalid-this */
+			var currentTarget = e.currentTarget || this
+			var _this = callbackThis || this
+			/* eslint-enable no-invalid-this */
+			var target = prop in currentTarget ?
+				currentTarget[prop] :
+				currentTarget.getAttribute(prop)
+			withAttrCallback.call(_this, target)
+		}
+	}
 
-	//routing
-	var modes = {pathname: "", hash: "#", search: "?"};
-	var redirect = noop, routeParams, currentRoute, isDefaultRoute = false;
-	m.route = function(root, arg1, arg2, vdom) {
-		//m.route()
-		if (arguments.length === 0) return currentRoute;
-		//m.route(el, defaultRoute, routes)
-		else if (arguments.length === 3 && isString(arg1)) {
-			redirect = function(source) {
-				var path = currentRoute = normalizeRoute(source);
+	// routing
+	var modes = {pathname: "", hash: "#", search: "?"}
+	var redirect = noop
+	var isDefaultRoute = false
+	var routeParams, currentRoute
+
+	m.route = function (root, arg1, arg2, vdom) { // eslint-disable-line
+		// m.route()
+		if (arguments.length === 0) return currentRoute
+		// m.route(el, defaultRoute, routes)
+		if (arguments.length === 3 && isString(arg1)) {
+			redirect = function (source) {
+				var path = currentRoute = normalizeRoute(source)
 				if (!routeByValue(root, arg2, path)) {
-					if (isDefaultRoute) throw new Error("Ensure the default route matches one of the routes defined in m.route");
-					isDefaultRoute = true;
-					m.route(arg1, true);
-					isDefaultRoute = false;
+					if (isDefaultRoute) {
+						throw new Error("Ensure the default route matches " +
+							"one of the routes defined in m.route")
+					}
+
+					isDefaultRoute = true
+					m.route(arg1, true)
+					isDefaultRoute = false
 				}
-			};
-			var listener = m.route.mode === "hash" ? "onhashchange" : "onpopstate";
-			window[listener] = function() {
-				var path = $location[m.route.mode];
-				if (m.route.mode === "pathname") path += $location.search;
-				if (currentRoute !== normalizeRoute(path)) redirect(path);
-			};
+			}
 
-			computePreRedrawHook = setScroll;
-			window[listener]();
+			var listener = m.route.mode === "hash" ?
+				"onhashchange" :
+				"onpopstate"
+
+			global[listener] = function () {
+				var path = $location[m.route.mode]
+				if (m.route.mode === "pathname") path += $location.search
+				if (currentRoute !== normalizeRoute(path)) redirect(path)
+			}
+
+			computePreRedrawHook = setScroll
+			global[listener]()
+
+			return
 		}
-		//config: m.route
-		else if (root.addEventListener || root.attachEvent) {
-			root.href = (m.route.mode !== 'pathname' ? $location.pathname : '') + modes[m.route.mode] + vdom.attrs.href;
+
+		// config: m.route
+		if (root.addEventListener || root.attachEvent) {
+			var base = m.route.mode !== "pathname" ? $location.pathname : ""
+			root.href = base + modes[m.route.mode] + vdom.attrs.href
 			if (root.addEventListener) {
-				root.removeEventListener("click", routeUnobtrusive);
-				root.addEventListener("click", routeUnobtrusive);
+				root.removeEventListener("click", routeUnobtrusive)
+				root.addEventListener("click", routeUnobtrusive)
+			} else {
+				root.detachEvent("onclick", routeUnobtrusive)
+				root.attachEvent("onclick", routeUnobtrusive)
 			}
-			else {
-				root.detachEvent("onclick", routeUnobtrusive);
-				root.attachEvent("onclick", routeUnobtrusive);
-			}
-		}
-		//m.route(route, params, shouldReplaceHistoryEntry)
-		else if (isString(root)) {
-			var oldRoute = currentRoute;
-			currentRoute = root;
-			var args = arg1 || {};
-			var queryIndex = currentRoute.indexOf("?");
-			var params = queryIndex > -1 ? parseQueryString(currentRoute.slice(queryIndex + 1)) : {};
-			for (var i in args) params[i] = args[i];
-			var querystring = buildQueryString(params);
-			var currentPath = queryIndex > -1 ? currentRoute.slice(0, queryIndex) : currentRoute;
-			if (querystring) currentRoute = currentPath + (currentPath.indexOf("?") === -1 ? "?" : "&") + querystring;
 
-			var shouldReplaceHistoryEntry = (arguments.length === 3 ? arg2 : arg1) === true || oldRoute === root;
+			return
+		}
+		// m.route(route, params, shouldReplaceHistoryEntry)
+		if (isString(root)) {
+			var oldRoute = currentRoute
+			currentRoute = root
 
-			if (window.history.pushState) {
-				computePreRedrawHook = setScroll;
-				computePostRedrawHook = function() {
-					window.history[shouldReplaceHistoryEntry ? "replaceState" : "pushState"](null, $document.title, modes[m.route.mode] + currentRoute);
-				};
-				redirect(modes[m.route.mode] + currentRoute);
+			var args = arg1 || {}
+			var queryIndex = currentRoute.indexOf("?")
+			var params
+
+			if (queryIndex > -1) {
+				params = parseQueryString(currentRoute.slice(queryIndex + 1))
+			} else {
+				params = {}
 			}
-			else {
-				$location[m.route.mode] = currentRoute;
-				redirect(modes[m.route.mode] + currentRoute);
+
+			for (var i in args) if (hasOwn.call(args, i)) {
+				params[i] = args[i]
+			}
+
+			var querystring = buildQueryString(params)
+			var currentPath
+
+			if (queryIndex > -1) {
+				currentPath = currentRoute.slice(0, queryIndex)
+			} else {
+				currentPath = currentRoute
+			}
+
+			if (querystring) {
+				currentRoute = currentPath +
+					(currentPath.indexOf("?") === -1 ? "?" : "&") +
+					querystring
+			}
+
+			var replaceHistory =
+				(arguments.length === 3 ? arg2 : arg1) === true ||
+				oldRoute === root
+
+			if (global.history.pushState) {
+				var method = replaceHistory ? "replaceState" : "pushState"
+				computePreRedrawHook = setScroll
+				computePostRedrawHook = function () {
+					global.history[method](null, $document.title,
+						modes[m.route.mode] + currentRoute)
+				}
+				redirect(modes[m.route.mode] + currentRoute)
+			} else {
+				$location[m.route.mode] = currentRoute
+				redirect(modes[m.route.mode] + currentRoute)
 			}
 		}
-	};
-	m.route.param = function(key) {
-		if (!routeParams) throw new Error("You must call m.route(element, defaultRoute, routes) before calling m.route.param()");
-		if( !key ){
-			return routeParams;
-		}
-		return routeParams[key];
-	};
-	m.route.mode = "search";
-	function normalizeRoute(route) {
-		return route.slice(modes[m.route.mode].length);
 	}
-	function routeByValue(root, router, path) {
-		routeParams = {};
 
-		var queryStart = path.indexOf("?");
+	m.route.param = function (key) {
+		if (!routeParams) {
+			throw new Error("You must call m.route(element, defaultRoute, " +
+				"routes) before calling m.route.param()")
+		}
+
+		if (!key) {
+			return routeParams
+		}
+
+		return routeParams[key]
+	}
+
+	m.route.mode = "search"
+
+	function normalizeRoute(route) {
+		return route.slice(modes[m.route.mode].length)
+	}
+
+	function routeByValue(root, router, path) {
+		routeParams = {}
+
+		var queryStart = path.indexOf("?")
 		if (queryStart !== -1) {
-			routeParams = parseQueryString(path.substr(queryStart + 1, path.length));
-			path = path.substr(0, queryStart);
+			routeParams = parseQueryString(
+				path.substr(queryStart + 1, path.length))
+			path = path.substr(0, queryStart)
 		}
 
 		// Get all routes and check if there's
 		// an exact match for the current path
-		var keys = Object.keys(router);
-		var index = keys.indexOf(path);
-		if(index !== -1){
-			m.mount(root, router[keys [index]]);
-			return true;
+		var keys = Object.keys(router)
+		var index = keys.indexOf(path)
+
+		if (index !== -1){
+			m.mount(root, router[keys [index]])
+			return true
 		}
 
-		for (var route in router) {
+		for (var route in router) if (hasOwn.call(router, route)) {
 			if (route === path) {
-				m.mount(root, router[route]);
-				return true;
+				m.mount(root, router[route])
+				return true
 			}
 
-			var matcher = new RegExp("^" + route.replace(/:[^\/]+?\.{3}/g, "(.*?)").replace(/:[^\/]+/g, "([^\\/]+)") + "\/?$");
+			var matcher = new RegExp("^" + route
+				.replace(/:[^\/]+?\.{3}/g, "(.*?)")
+				.replace(/:[^\/]+/g, "([^\\/]+)") + "\/?$")
 
 			if (matcher.test(path)) {
-				path.replace(matcher, function() {
-					var keys = route.match(/:[^\/]+/g) || [];
-					var values = [].slice.call(arguments, 1, -2);
+				/* eslint-disable no-loop-func */
+				path.replace(matcher, function () {
+					var keys = route.match(/:[^\/]+/g) || []
+					var values = [].slice.call(arguments, 1, -2)
 					forEach(keys, function (key, i) {
-						routeParams[key.replace(/:|\./g, "")] = decodeURIComponent(values[i]);
+						routeParams[key.replace(/:|\./g, "")] =
+							decodeURIComponent(values[i])
 					})
-					m.mount(root, router[route]);
-				});
-				return true;
+					m.mount(root, router[route])
+				})
+				/* eslint-enable no-loop-func */
+				return true
 			}
 		}
 	}
+
 	function routeUnobtrusive(e) {
-		e = e || event;
-		if (e.ctrlKey || e.metaKey || e.shiftKey || e.which === 2) return;
+		e = e || event
+		if (e.ctrlKey || e.metaKey || e.shiftKey || e.which === 2) return
 
-		if (e.preventDefault) e.preventDefault();
-		else e.returnValue = false;
+		if (e.preventDefault) {
+			e.preventDefault()
+		} else {
+			e.returnValue = false
+		}
+
+		var currentTarget = e.currentTarget || e.srcElement
+		var args
+
+		if (m.route.mode === "pathname" && currentTarget.search) {
+			args = parseQueryString(currentTarget.search.slice(1))
+		} else {
+			args = {}
+		}
+
+		while (currentTarget && !/a/i.test(currentTarget.nodeName)) {
+			currentTarget = currentTarget.parentNode
+		}
 
-		var currentTarget = e.currentTarget || e.srcElement;
-		var args = m.route.mode === "pathname" && currentTarget.search ? parseQueryString(currentTarget.search.slice(1)) : {};
-		while (currentTarget && currentTarget.nodeName.toUpperCase() !== "A") currentTarget = currentTarget.parentNode;
 		// clear pendingRequests because we want an immediate route change
-		pendingRequests = 0;
-		m.route(currentTarget[m.route.mode].slice(modes[m.route.mode].length), args);
+		pendingRequests = 0
+		m.route(currentTarget[m.route.mode]
+			.slice(modes[m.route.mode].length), args)
 	}
+
 	function setScroll() {
-		if (m.route.mode !== "hash" && $location.hash) $location.hash = $location.hash;
-		else window.scrollTo(0, 0);
+		if (m.route.mode !== "hash" && $location.hash) {
+			$location.hash = $location.hash
+		} else {
+			global.scrollTo(0, 0)
+		}
 	}
+
 	function buildQueryString(object, prefix) {
-		var duplicates = {};
-		var str = [];
-		for (var prop in object) {
-			var key = prefix ? prefix + "[" + prop + "]" : prop;
-			var value = object[prop];
+		var duplicates = {}
+		var str = []
+
+		for (var prop in object) if (hasOwn.call(object, prop)) {
+			var key = prefix ? prefix + "[" + prop + "]" : prop
+			var value = object[prop]
 
 			if (value === null) {
-				str.push(encodeURIComponent(key));
+				str.push(encodeURIComponent(key))
 			} else if (isObject(value)) {
-				str.push(buildQueryString(value, key));
+				str.push(buildQueryString(value, key))
 			} else if (isArray(value)) {
-				var keys = [];
-				duplicates[key] = duplicates[key] || {};
+				var keys = []
+				duplicates[key] = duplicates[key] || {}
+				/* eslint-disable no-loop-func */
 				forEach(value, function (item) {
+					/* eslint-enable no-loop-func */
 					if (!duplicates[key][item]) {
-						duplicates[key][item] = true;
-						keys.push(encodeURIComponent(key) + "=" + encodeURIComponent(item));
+						duplicates[key][item] = true
+						keys.push(encodeURIComponent(key) + "=" +
+							encodeURIComponent(item))
 					}
-				});
-				str.push(keys.join("&"));
+				})
+				str.push(keys.join("&"))
 			} else if (value !== undefined) {
-				str.push(encodeURIComponent(key) + "=" + encodeURIComponent(value));
+				str.push(encodeURIComponent(key) + "=" +
+					encodeURIComponent(value))
 			}
 		}
-		return str.join("&");
+		return str.join("&")
 	}
+
 	function parseQueryString(str) {
-		if (str === "" || str == null) return {};
-		if (str.charAt(0) === "?") str = str.slice(1);
+		if (str === "" || str == null) return {}
+		if (str.charAt(0) === "?") str = str.slice(1)
+
+		var pairs = str.split("&")
+		var params = {}
 
-		var pairs = str.split("&"), params = {};
 		forEach(pairs, function (string) {
-			var pair = string.split("=");
-			var key = decodeURIComponent(pair[0]);
-			var value = pair.length === 2 ? decodeURIComponent(pair[1]) : null;
+			var pair = string.split("=")
+			var key = decodeURIComponent(pair[0])
+			var value = pair.length === 2 ? decodeURIComponent(pair[1]) : null
 			if (params[key] != null) {
-				if (!isArray(params[key])) params[key] = [params[key]];
-				params[key].push(value);
+				if (!isArray(params[key])) params[key] = [params[key]]
+				params[key].push(value)
 			}
-			else params[key] = value;
-		});
+			else params[key] = value
+		})
 
-		return params;
+		return params
 	}
-	m.route.buildQueryString = buildQueryString;
-	m.route.parseQueryString = parseQueryString;
+
+	m.route.buildQueryString = buildQueryString
+	m.route.parseQueryString = parseQueryString
 
 	function reset(root) {
-		var cacheKey = getCellCacheKey(root);
-		clear(root.childNodes, cellCache[cacheKey]);
-		cellCache[cacheKey] = undefined;
+		var cacheKey = getCellCacheKey(root)
+		clear(root.childNodes, cellCache[cacheKey])
+		cellCache[cacheKey] = undefined
 	}
 
 	m.deferred = function () {
-		var deferred = new Deferred();
-		deferred.promise = propify(deferred.promise);
-		return deferred;
-	};
-	function propify(promise, initialValue) {
-		var prop = m.prop(initialValue);
-		promise.then(prop);
-		prop.then = function(resolve, reject) {
-			return propify(promise.then(resolve, reject), initialValue);
-		};
-		prop["catch"] = prop.then.bind(null, null);
-		return prop;
+		var deferred = new Deferred()
+		deferred.promise = propify(deferred.promise)
+		return deferred
 	}
-	//Promiz.mithril.js | Zolmeister | MIT
-	//a modified version of Promiz.js, which does not conform to Promises/A+ for two reasons:
-	//1) `then` callbacks are called synchronously (because setTimeout is too slow, and the setImmediate polyfill is too big
-	//2) throwing subclasses of Error cause the error to be bubbled up instead of triggering rejection (because the spec does not account for the important use case of default browser error handling, i.e. message w/ line number)
-	function Deferred(successCallback, failureCallback) {
-		var RESOLVING = 1, REJECTING = 2, RESOLVED = 3, REJECTED = 4;
-		var self = this, state = 0, promiseValue = 0, next = [];
 
-		self.promise = {};
-
-		self.resolve = function(value) {
-			if (!state) {
-				promiseValue = value;
-				state = RESOLVING;
-
-				fire();
-			}
-			return this;
-		};
-
-		self.reject = function(value) {
-			if (!state) {
-				promiseValue = value;
-				state = REJECTING;
-
-				fire();
-			}
-			return this;
-		};
-
-		self.promise.then = function(successCallback, failureCallback) {
-			var deferred = new Deferred(successCallback, failureCallback)
-			if (state === RESOLVED) {
-				deferred.resolve(promiseValue);
-			}
-			else if (state === REJECTED) {
-				deferred.reject(promiseValue);
-			}
-			else {
-				next.push(deferred);
-			}
-			return deferred.promise
-		};
-
-		function finish(type) {
-			state = type || REJECTED;
-			next.map(function(deferred) {
-				state === RESOLVED ? deferred.resolve(promiseValue) : deferred.reject(promiseValue);
-			});
+	function propify(promise, initialValue) {
+		var prop = m.prop(initialValue)
+		promise.then(prop)
+		prop.then = function (resolve, reject) {
+			return propify(promise.then(resolve, reject), initialValue)
 		}
 
-		function thennable(then, successCallback, failureCallback, notThennableCallback) {
-			if (((promiseValue != null && isObject(promiseValue)) || isFunction(promiseValue)) && isFunction(then)) {
+		prop.catch = prop.then.bind(null, null)
+		return prop
+	}
+	// Promiz.mithril.js | Zolmeister | MIT
+	// a modified version of Promiz.js, which does not conform to Promises/A+
+	// for two reasons:
+	//
+	// 1) `then` callbacks are called synchronously (because setTimeout is too
+	//    slow, and the setImmediate polyfill is too big
+	//
+	// 2) throwing subclasses of Error cause the error to be bubbled up instead
+	//    of triggering rejection (because the spec does not account for the
+	//    important use case of default browser error handling, i.e. message w/
+	//    line number)
+
+	var RESOLVING = 1
+	var REJECTING = 2
+	var RESOLVED = 3
+	var REJECTED = 4
+
+	function Deferred(onSuccess, onFailure) {
+		var self = this
+		var state = 0
+		var promiseValue = 0
+		var next = []
+
+		self.promise = {}
+
+		self.resolve = function (value) {
+			if (!state) {
+				promiseValue = value
+				state = RESOLVING
+
+				fire()
+			}
+
+			return self
+		}
+
+		self.reject = function (value) {
+			if (!state) {
+				promiseValue = value
+				state = REJECTING
+
+				fire()
+			}
+
+			return self
+		}
+
+		self.promise.then = function (onSuccess, onFailure) {
+			var deferred = new Deferred(onSuccess, onFailure)
+
+			if (state === RESOLVED) {
+				deferred.resolve(promiseValue)
+			} else if (state === REJECTED) {
+				deferred.reject(promiseValue)
+			} else {
+				next.push(deferred)
+			}
+
+			return deferred.promise
+		}
+
+		function finish(type) {
+			state = type || REJECTED
+			next.map(function (deferred) {
+				if (state === RESOLVED) {
+					deferred.resolve(promiseValue)
+				} else {
+					deferred.reject(promiseValue)
+				}
+			})
+		}
+
+		function thennable(then, success, failure, notThennable) {
+			if (((promiseValue != null && isObject(promiseValue)) ||
+					isFunction(promiseValue)) && isFunction(then)) {
 				try {
 					// count protects against abuse calls from spec checker
-					var count = 0;
-					then.call(promiseValue, function(value) {
-						if (count++) return;
-						promiseValue = value;
-						successCallback();
+					var count = 0
+					then.call(promiseValue, function (value) {
+						if (count++) return
+						promiseValue = value
+						success()
 					}, function (value) {
-						if (count++) return;
-						promiseValue = value;
-						failureCallback();
-					});
-				}
-				catch (e) {
-					m.deferred.onerror(e);
-					promiseValue = e;
-					failureCallback();
+						if (count++) return
+						promiseValue = value
+						failure()
+					})
+				} catch (e) {
+					m.deferred.onerror(e)
+					promiseValue = e
+					failure()
 				}
 			} else {
-				notThennableCallback();
+				notThennable()
 			}
 		}
 
 		function fire() {
 			// check if it's a thenable
-			var then;
+			var then
 			try {
-				then = promiseValue && promiseValue.then;
-			}
-			catch (e) {
-				m.deferred.onerror(e);
-				promiseValue = e;
-				state = REJECTING;
-				return fire();
+				then = promiseValue && promiseValue.then
+			} catch (e) {
+				m.deferred.onerror(e)
+				promiseValue = e
+				state = REJECTING
+				return fire()
 			}
 
 			if (state === REJECTING) {
@@ -1794,226 +2483,262 @@ var m = (function app(window, undefined) {
 				fire()
 			}, function () {
 				try {
-					if (state === RESOLVING && isFunction(successCallback)) {
-						promiseValue = successCallback(promiseValue);
+					if (state === RESOLVING && isFunction(onSuccess)) {
+						promiseValue = onSuccess(promiseValue)
+					} else if (state === REJECTING && isFunction(onFailure)) {
+						promiseValue = onFailure(promiseValue)
+						state = RESOLVING
 					}
-					else if (state === REJECTING && isFunction(failureCallback)) {
-						promiseValue = failureCallback(promiseValue);
-						state = RESOLVING;
-					}
-				}
-				catch (e) {
-					m.deferred.onerror(e);
-					promiseValue = e;
-					return finish();
+				} catch (e) {
+					m.deferred.onerror(e)
+					promiseValue = e
+					return finish()
 				}
 
 				if (promiseValue === self) {
-					promiseValue = TypeError();
-					finish();
+					promiseValue = TypeError()
+					finish()
 				} else {
 					thennable(then, function () {
-						finish(RESOLVED);
+						finish(RESOLVED)
 					}, finish, function () {
-						finish(state === RESOLVING && RESOLVED);
-					});
+						finish(state === RESOLVING && RESOLVED)
+					})
 				}
-			});
+			})
 		}
 	}
-	m.deferred.onerror = function(e) {
-		if (type.call(e) === "[object Error]" && !e.constructor.toString().match(/ Error/)) {
-			pendingRequests = 0;
-			throw e;
-		}
-	};
 
-	m.sync = function(args) {
-		var method = "resolve";
+	m.deferred.onerror = function (e) {
+		if (type.call(e) === "[object Error]" &&
+				!e.constructor.toString().match(/ Error/)) {
+			pendingRequests = 0
+			throw e
+		}
+	}
+
+	m.sync = function (args) {
+		var deferred = m.deferred()
+		var outstanding = args.length
+		var results = new Array(outstanding)
+		var method = "resolve"
 
 		function synchronizer(pos, resolved) {
-			return function(value) {
-				results[pos] = value;
-				if (!resolved) method = "reject";
+			return function (value) {
+				results[pos] = value
+				if (!resolved) method = "reject"
 				if (--outstanding === 0) {
-					deferred.promise(results);
-					deferred[method](results);
+					deferred.promise(results)
+					deferred[method](results)
 				}
-				return value;
-			};
+				return value
+			}
 		}
 
-		var deferred = m.deferred();
-		var outstanding = args.length;
-		var results = new Array(outstanding);
 		if (args.length > 0) {
 			forEach(args, function (arg, i) {
-				arg.then(synchronizer(i, true), synchronizer(i, false));
-			});
+				arg.then(synchronizer(i, true), synchronizer(i, false))
+			})
+		} else {
+			deferred.resolve([])
 		}
-		else deferred.resolve([]);
 
-		return deferred.promise;
-	};
-	function identity(value) { return value; }
+		return deferred.promise
+	}
+
+	function identity(value) { return value }
+
+	function handleJsonp(options) {
+		var callbackKey = "mithril_callback_" +
+			new Date().getTime() + "_" +
+			(Math.round(Math.random() * 1e16)).toString(36)
+
+		var script = $document.createElement("script")
+
+		global[callbackKey] = function (resp) {
+			script.parentNode.removeChild(script)
+			options.onload({
+				type: "load",
+				target: {
+					responseText: resp
+				}
+			})
+			global[callbackKey] = undefined
+		}
+
+		script.onerror = function () {
+			script.parentNode.removeChild(script)
+
+			options.onerror({
+				type: "error",
+				target: {
+					status: 500,
+					responseText: JSON.stringify({
+						error: "Error making jsonp request"
+					})
+				}
+			})
+			global[callbackKey] = undefined
+
+			return false
+		}
+
+		script.onload = function () {
+			return false
+		}
+
+		script.src = options.url +
+			(options.url.indexOf("?") > 0 ? "&" : "?") +
+			(options.callbackKey ? options.callbackKey : "callback") +
+			"=" + callbackKey +
+			"&" + buildQueryString(options.data || {})
+
+		$document.body.appendChild(script)
+	}
+
+	function createXhr(options) {
+		var xhr = new global.XMLHttpRequest()
+		xhr.open(options.method, options.url, true, options.user,
+			options.password)
+
+		xhr.onreadystatechange = function () {
+			if (xhr.readyState === 4) {
+				if (xhr.status >= 200 && xhr.status < 300) {
+					options.onload({type: "load", target: xhr})
+				} else {
+					options.onerror({type: "error", target: xhr})
+				}
+			}
+		}
+
+		if (options.serialize === JSON.stringify &&
+				options.data &&
+				options.method !== "GET") {
+			xhr.setRequestHeader("Content-Type",
+				"application/json; charset=utf-8")
+		}
+
+		if (options.deserialize === JSON.parse) {
+			xhr.setRequestHeader("Accept", "application/json, text/*")
+		}
+
+		if (isFunction(options.config)) {
+			var maybeXhr = options.config(xhr, options)
+			if (maybeXhr != null) xhr = maybeXhr
+		}
+
+		var data = options.method === "GET" || !options.data ? "" : options.data
+
+		if (data && !isString(data) && data.constructor !== global.FormData) {
+			throw new Error("Request data should be either be a string or " +
+				"FormData. Check the `serialize` option in `m.request`")
+		}
+
+		xhr.send(data)
+		return xhr
+	}
 
 	function ajax(options) {
 		if (options.dataType && options.dataType.toLowerCase() === "jsonp") {
-			var callbackKey = "mithril_callback_" + new Date().getTime() + "_" + (Math.round(Math.random() * 1e16)).toString(36)
-			var script = $document.createElement("script");
-
-			window[callbackKey] = function(resp) {
-				script.parentNode.removeChild(script);
-				options.onload({
-					type: "load",
-					target: {
-						responseText: resp
-					}
-				});
-				window[callbackKey] = undefined;
-			};
-
-			script.onerror = function() {
-				script.parentNode.removeChild(script);
-
-				options.onerror({
-					type: "error",
-					target: {
-						status: 500,
-						responseText: JSON.stringify({
-							error: "Error making jsonp request"
-						})
-					}
-				});
-				window[callbackKey] = undefined;
-
-				return false;
-			}
-
-			script.onload = function() {
-				return false;
-			};
-
-			script.src = options.url
-				+ (options.url.indexOf("?") > 0 ? "&" : "?")
-				+ (options.callbackKey ? options.callbackKey : "callback")
-				+ "=" + callbackKey
-				+ "&" + buildQueryString(options.data || {});
-			$document.body.appendChild(script);
-		}
-		else {
-			var xhr = new window.XMLHttpRequest();
-			xhr.open(options.method, options.url, true, options.user, options.password);
-			xhr.onreadystatechange = function() {
-				if (xhr.readyState === 4) {
-					if (xhr.status >= 200 && xhr.status < 300) options.onload({type: "load", target: xhr});
-					else options.onerror({type: "error", target: xhr});
-				}
-			};
-			if (options.serialize === JSON.stringify && options.data && options.method !== "GET") {
-				xhr.setRequestHeader("Content-Type", "application/json; charset=utf-8");
-			}
-			if (options.deserialize === JSON.parse) {
-				xhr.setRequestHeader("Accept", "application/json, text/*");
-			}
-			if (isFunction(options.config)) {
-				var maybeXhr = options.config(xhr, options);
-				if (maybeXhr != null) xhr = maybeXhr;
-			}
-
-			var data = options.method === "GET" || !options.data ? "" : options.data;
-			if (data && (!isString(data) && data.constructor !== window.FormData)) {
-				throw new Error("Request data should be either be a string or FormData. Check the `serialize` option in `m.request`");
-			}
-			xhr.send(data);
-			return xhr;
+			return handleJsonp(options)
+		} else {
+			return createXhr(options)
 		}
 	}
 
-	function bindData(xhrOptions, data, serialize) {
-		if (xhrOptions.method === "GET" && xhrOptions.dataType !== "jsonp") {
-			var prefix = xhrOptions.url.indexOf("?") < 0 ? "?" : "&";
-			var querystring = buildQueryString(data);
-			xhrOptions.url = xhrOptions.url + (querystring ? prefix + querystring : "");
+	function bindData(options, data, serialize) {
+		if (options.method === "GET" && options.dataType !== "jsonp") {
+			var prefix = options.url.indexOf("?") < 0 ? "?" : "&"
+			var querystring = buildQueryString(data)
+			options.url += (querystring ? prefix + querystring : "")
+		} else {
+			options.data = serialize(data)
 		}
-		else xhrOptions.data = serialize(data);
-		return xhrOptions;
 	}
 
 	function parameterizeUrl(url, data) {
-		var tokens = url.match(/:[a-z]\w+/gi);
+		var tokens = url.match(/:[a-z]\w+/gi)
+
 		if (tokens && data) {
 			forEach(tokens, function (token) {
-				var key = token.slice(1);
-				url = url.replace(token, data[key]);
-				delete data[key];
-			});
+				var key = token.slice(1)
+				url = url.replace(token, data[key])
+				delete data[key]
+			})
 		}
-		return url;
+
+		return url
 	}
 
-	m.request = function(xhrOptions) {
-		if (xhrOptions.background !== true) m.startComputation();
-		var deferred = new Deferred();
-		var isJSONP = xhrOptions.dataType && xhrOptions.dataType.toLowerCase() === "jsonp"
-		var serialize = xhrOptions.serialize = isJSONP ? identity : xhrOptions.serialize || JSON.stringify;
-		var deserialize = xhrOptions.deserialize = isJSONP ? identity : xhrOptions.deserialize || JSON.parse;
-		var extract = isJSONP ? function(jsonp) { return jsonp.responseText } : xhrOptions.extract || function(xhr) {
-			if (xhr.responseText.length === 0 && deserialize === JSON.parse) {
-				return null
-			} else {
-				return xhr.responseText
-			}
-		};
-		xhrOptions.method = (xhrOptions.method || "GET").toUpperCase();
-		xhrOptions.url = parameterizeUrl(xhrOptions.url, xhrOptions.data);
-		xhrOptions = bindData(xhrOptions, xhrOptions.data, serialize);
-		xhrOptions.onload = xhrOptions.onerror = function(e) {
-			try {
-				e = e || event;
-				var unwrap = (e.type === "load" ? xhrOptions.unwrapSuccess : xhrOptions.unwrapError) || identity;
-				var response = unwrap(deserialize(extract(e.target, xhrOptions)), e.target);
-				if (e.type === "load") {
-					if (isArray(response) && xhrOptions.type) {
-						forEach(response, function (res, i) {
-							response[i] = new xhrOptions.type(res);
-						});
-					} else if (xhrOptions.type) {
-						response = new xhrOptions.type(response);
-					}
-					deferred.resolve(response)
-				} else {
-					deferred.reject(response)
-				}
+	m.request = function (options) {
+		if (options.background !== true) m.startComputation()
+		var deferred = new Deferred()
+		var isJSONP = options.dataType &&
+			options.dataType.toLowerCase() === "jsonp"
 
-				deferred[e.type === "load" ? "resolve" : "reject"](response);
-			}
-			catch (e) {
-				deferred.reject(e);
-			}
-			finally {
-				if (xhrOptions.background !== true) m.endComputation()
+		var serialize, deserialize, extract
+
+		if (isJSONP) {
+			serialize = options.serialize =
+			deserialize = options.deserialize = identity
+
+			extract = function (jsonp) { return jsonp.responseText }
+		} else {
+			serialize = options.serialize = options.serialize || JSON.stringify
+
+			deserialize = options.deserialize =
+				options.deserialize || JSON.parse
+			extract = options.extract || function (xhr) {
+				if (xhr.responseText.length || deserialize !== JSON.parse) {
+					return xhr.responseText
+				} else {
+					return null
+				}
 			}
 		}
 
-		ajax(xhrOptions);
-		deferred.promise = propify(deferred.promise, xhrOptions.initialValue);
-		return deferred.promise;
-	};
+		options.method = (options.method || "GET").toUpperCase()
+		options.url = parameterizeUrl(options.url, options.data)
+		bindData(options, options.data, serialize)
+		options.onload = options.onerror = function (ev) {
+			try {
+				ev = ev || event
+				var response = deserialize(extract(ev.target, options))
+				if (ev.type === "load") {
+					if (options.unwrapSuccess) {
+						response = options.unwrapSuccess(response, ev.target)
+					}
 
-	//testing API
-	m.deps = function(mock) {
-		initialize(window = mock || window);
-		return window;
-	};
-	//for internal testing only, do not use `m.deps.factory`
-	m.deps.factory = app;
+					if (isArray(response) && options.type) {
+						forEach(response, function (res, i) {
+							response[i] = new options.type(res)
+						})
+					} else if (options.type) {
+						response = new options.type(response)
+					}
 
-	return m;
-})(typeof window !== "undefined" ? window : {});
+					deferred.resolve(response)
+				} else {
+					if (options.unwrapError) {
+						response = options.unwrapError(response, ev.target)
+					}
 
-if (typeof module === "object" && module != null && module.exports) module.exports = m;
-else if (typeof define === "function" && define.amd) define(function() { return m });
+					deferred.reject(response)
+				}
+			} catch (e) {
+				deferred.reject(e)
+			} finally {
+				if (options.background !== true) m.endComputation()
+			}
+		}
+
+		ajax(options)
+		deferred.promise = propify(deferred.promise, options.initialValue)
+		return deferred.promise
+	}
+
+	return m
+})
 ;
 ( function _package( factory ){
 	if( typeof define === 'function' && define.amd ){
@@ -21389,10 +22114,10 @@ System.register('flarum/Model', [], function (_export) {
     }
   };
 });;
-System.register('flarum/models/Discussion', ['flarum/Model', 'flarum/utils/mixin', 'flarum/utils/computed', 'flarum/utils/ItemList', 'flarum/utils/string', 'flarum/components/Badge'], function (_export) {
+System.register('flarum/models/Discussion', ['flarum/Model', 'flarum/utils/mixin', 'flarum/utils/computed', 'flarum/utils/ItemList', 'flarum/components/Badge'], function (_export) {
   'use strict';
 
-  var Model, mixin, computed, ItemList, slug, Badge, Discussion;
+  var Model, mixin, computed, ItemList, Badge, Discussion;
   return {
     setters: [function (_flarumModel) {
       Model = _flarumModel['default'];
@@ -21402,8 +22127,6 @@ System.register('flarum/models/Discussion', ['flarum/Model', 'flarum/utils/mixin
       computed = _flarumUtilsComputed['default'];
     }, function (_flarumUtilsItemList) {
       ItemList = _flarumUtilsItemList['default'];
-    }, function (_flarumUtilsString) {
-      slug = _flarumUtilsString.slug;
     }, function (_flarumComponentsBadge) {
       Badge = _flarumComponentsBadge['default'];
     }],
@@ -21423,7 +22146,7 @@ System.register('flarum/models/Discussion', ['flarum/Model', 'flarum/utils/mixin
 
       babelHelpers._extends(Discussion.prototype, {
         title: Model.attribute('title'),
-        slug: computed('title', slug),
+        slug: Model.attribute('slug'),
 
         startTime: Model.attribute('startTime', Model.transformDate),
         startUser: Model.hasOne('startUser'),
diff --git a/framework/core/js/forum/dist/app.js b/framework/core/js/forum/dist/app.js
index 6e1ed2b5b..b3c19aa10 100644
--- a/framework/core/js/forum/dist/app.js
+++ b/framework/core/js/forum/dist/app.js
@@ -607,42 +607,68 @@ exports.System = System;
 
 })(window);
 ;
-var m = (function app(window, undefined) {
-	"use strict";
-  	var VERSION = "v0.2.2-rc.1";
+;(function (global, factory) { // eslint-disable-line
+	"use strict"
+	/* eslint-disable no-undef */
+	var m = factory(global)
+	if (typeof module === "object" && module != null && module.exports) {
+		module.exports = m
+	} else if (typeof define === "function" && define.amd) {
+		define(function () { return m })
+	} else {
+		global.m = m
+	}
+	/* eslint-enable no-undef */
+})(typeof window !== "undefined" ? window : {}, function (global, undefined) { // eslint-disable-line
+	"use strict"
+
+	m.version = function () {
+		return "v0.2.2-rc.1"
+	}
+
+	var hasOwn = {}.hasOwnProperty
+	var type = {}.toString
+
 	function isFunction(object) {
-		return typeof object === "function";
+		return typeof object === "function"
 	}
+
 	function isObject(object) {
-		return type.call(object) === "[object Object]";
+		return type.call(object) === "[object Object]"
 	}
+
 	function isString(object) {
-		return type.call(object) === "[object String]";
+		return type.call(object) === "[object String]"
 	}
+
 	var isArray = Array.isArray || function (object) {
-		return type.call(object) === "[object Array]";
-	};
-	var type = {}.toString;
-	var parser = /(?:(^|#|\.)([^#\.\[\]]+))|(\[.+?\])/g, attrParser = /\[(.+?)(?:=("|'|)(.*?)\2)?\]/;
-	var voidElements = /^(AREA|BASE|BR|COL|COMMAND|EMBED|HR|IMG|INPUT|KEYGEN|LINK|META|PARAM|SOURCE|TRACK|WBR)$/;
-	var noop = function () {};
+		return type.call(object) === "[object Array]"
+	}
+
+	function noop() {}
+
+	/* eslint-disable max-len */
+	var voidElements = /^(AREA|BASE|BR|COL|COMMAND|EMBED|HR|IMG|INPUT|KEYGEN|LINK|META|PARAM|SOURCE|TRACK|WBR)$/
+	/* eslint-enable max-len */
 
 	// caching commonly used variables
-	var $document, $location, $requestAnimationFrame, $cancelAnimationFrame;
+	var $document, $location, $requestAnimationFrame, $cancelAnimationFrame
 
 	// self invoking function needed because of the way mocks work
-	function initialize(window) {
-		$document = window.document;
-		$location = window.location;
-		$cancelAnimationFrame = window.cancelAnimationFrame || window.clearTimeout;
-		$requestAnimationFrame = window.requestAnimationFrame || window.setTimeout;
+	function initialize(mock) {
+		$document = mock.document
+		$location = mock.location
+		$cancelAnimationFrame = mock.cancelAnimationFrame || mock.clearTimeout
+		$requestAnimationFrame = mock.requestAnimationFrame || mock.setTimeout
 	}
 
-	initialize(window);
+	// testing API
+	m.deps = function (mock) {
+		initialize(global = mock || window)
+		return global
+	}
 
-	m.version = function() {
-		return VERSION;
-	};
+	m.deps(global)
 
 	/**
 	 * @typedef {String} Tag
@@ -650,98 +676,148 @@ var m = (function app(window, undefined) {
 	 * Which describes a DOM node
 	 */
 
+	function parseTagAttrs(cell, tag) {
+		var classes = []
+		var parser = /(?:(^|#|\.)([^#\.\[\]]+))|(\[.+?\])/g
+		var match
+
+		while ((match = parser.exec(tag))) {
+			if (match[1] === "" && match[2]) {
+				cell.tag = match[2]
+			} else if (match[1] === "#") {
+				cell.attrs.id = match[2]
+			} else if (match[1] === ".") {
+				classes.push(match[2])
+			} else if (match[3][0] === "[") {
+				var pair = /\[(.+?)(?:=("|'|)(.*?)\2)?\]/.exec(match[3])
+				cell.attrs[pair[1]] = pair[3] || (pair[2] ? "" : true)
+			}
+		}
+
+		return classes
+	}
+
+	function getVirtualChildren(args, hasAttrs) {
+		var children = hasAttrs ? args.slice(1) : args
+
+		if (children.length === 1 && isArray(children[0])) {
+			return children[0]
+		} else {
+			return children
+		}
+	}
+
+	function assignAttrs(target, attrs, classes) {
+		var classAttr = "class" in attrs ? "class" : "className"
+
+		for (var attrName in attrs) {
+			if (hasOwn.call(attrs, attrName)) {
+				if (attrName === classAttr &&
+						attrs[attrName] != null &&
+						attrs[attrName] !== "") {
+					classes.push(attrs[attrName])
+					// create key in correct iteration order
+					target[attrName] = ""
+				} else {
+					target[attrName] = attrs[attrName]
+				}
+			}
+		}
+
+		if (classes.length) target[classAttr] = classes.join(" ")
+	}
+
 	/**
 	 *
 	 * @param {Tag} The DOM node tag
 	 * @param {Object=[]} optional key-value pairs to be mapped to DOM attrs
-	 * @param {...mNode=[]} Zero or more Mithril child nodes. Can be an array, or splat (optional)
-	 *
+	 * @param {...mNode=[]} Zero or more Mithril child nodes. Can be an array,
+	 *                      or splat (optional)
 	 */
 	function m(tag, pairs) {
 		for (var args = [], i = 1; i < arguments.length; i++) {
-			args[i - 1] = arguments[i];
-		}
-		if (isObject(tag)) return parameterize(tag, args);
-		var hasAttrs = pairs != null && isObject(pairs) && !("tag" in pairs || "view" in pairs || "subtree" in pairs);
-		var attrs = hasAttrs ? pairs : {};
-		var classAttrName = "class" in attrs ? "class" : "className";
-		var cell = {tag: "div", attrs: {}};
-		var match, classes = [];
-		if (!isString(tag)) throw new Error("selector in m(selector, attrs, children) should be a string");
-		while ((match = parser.exec(tag)) != null) {
-			if (match[1] === "" && match[2]) cell.tag = match[2];
-			else if (match[1] === "#") cell.attrs.id = match[2];
-			else if (match[1] === ".") classes.push(match[2]);
-			else if (match[3][0] === "[") {
-				var pair = attrParser.exec(match[3]);
-				cell.attrs[pair[1]] = pair[3] || (pair[2] ? "" :true);
-			}
+			args[i - 1] = arguments[i]
 		}
 
-		var children = hasAttrs ? args.slice(1) : args;
-		if (children.length === 1 && isArray(children[0])) {
-			cell.children = children[0];
-		}
-		else {
-			cell.children = children;
+		if (isObject(tag)) return parameterize(tag, args)
+
+		if (!isString(tag)) {
+			throw new Error("selector in m(selector, attrs, children) should " +
+				"be a string")
 		}
 
-		for (var attrName in attrs) {
-			if (attrs.hasOwnProperty(attrName)) {
-				if (attrName === classAttrName && attrs[attrName] != null && attrs[attrName] !== "") {
-					classes.push(attrs[attrName]);
-					cell.attrs[attrName] = ""; //create key in correct iteration order
-				}
-				else cell.attrs[attrName] = attrs[attrName];
-			}
-		}
-		if (classes.length) cell.attrs[classAttrName] = classes.join(" ");
+		var hasAttrs = pairs != null && isObject(pairs) &&
+			!("tag" in pairs || "view" in pairs || "subtree" in pairs)
 
-		return cell;
+		var attrs = hasAttrs ? pairs : {}
+		var cell = {
+			tag: "div",
+			attrs: {},
+			children: getVirtualChildren(args, hasAttrs)
+		}
+
+		assignAttrs(cell.attrs, attrs, parseTagAttrs(cell, tag))
+		return cell
 	}
+
 	function forEach(list, f) {
-		for (var i = 0; i < list.length && !f(list[i], i++);) {}
+		for (var i = 0; i < list.length && !f(list[i], i++);) {
+			// function called in condition
+		}
 	}
+
 	function forKeys(list, f) {
 		forEach(list, function (attrs, i) {
-			return (attrs = attrs && attrs.attrs) && attrs.key != null && f(attrs, i);
-		});
+			return (attrs = attrs && attrs.attrs) &&
+				attrs.key != null &&
+				f(attrs, i)
+		})
 	}
 	// This function was causing deopts in Chrome.
 	function dataToString(data) {
-		//data.toString() might throw or return null if data is the return value of Console.log in Firefox (behavior depends on version)
+		// data.toString() might throw or return null if data is the return
+		// value of Console.log in some versions of Firefox (behavior depends on
+		// version)
 		try {
-			if (data == null || data.toString() == null) return "";
+			if (data != null && data.toString() != null) return data
 		} catch (e) {
-			return "";
+			// silently ignore errors
 		}
-		return data;
+		return ""
 	}
+
 	// This function was causing deopts in Chrome.
 	function injectTextNode(parentElement, first, index, data) {
 		try {
-			insertNode(parentElement, first, index);
-			first.nodeValue = data;
-		} catch (e) {} //IE erroneously throws error when appending an empty text node after a null
+			insertNode(parentElement, first, index)
+			first.nodeValue = data
+		} catch (e) {
+			// IE erroneously throws error when appending an empty text node
+			// after a null
+		}
 	}
 
 	function flatten(list) {
-		//recursively flatten array
+		// recursively flatten array
 		for (var i = 0; i < list.length; i++) {
 			if (isArray(list[i])) {
-				list = list.concat.apply([], list);
-				//check current index again and flatten until there are no more nested arrays at that index
-				i--;
+				list = list.concat.apply([], list)
+				// check current index again and flatten until there are no more
+				// nested arrays at that index
+				i--
 			}
 		}
-		return list;
+		return list
 	}
 
 	function insertNode(parentElement, node, index) {
-		parentElement.insertBefore(node, parentElement.childNodes[index] || null);
+		parentElement.insertBefore(node,
+			parentElement.childNodes[index] || null)
 	}
 
-	var DELETION = 1, INSERTION = 2, MOVE = 3;
+	var DELETION = 1
+	var INSERTION = 2
+	var MOVE = 3
 
 	function handleKeysDiffer(data, existing, cached, parentElement) {
 		forKeys(data, function (key, i) {
@@ -749,167 +825,274 @@ var m = (function app(window, undefined) {
 				action: MOVE,
 				index: i,
 				from: existing[key].index,
-				element: cached.nodes[existing[key].index] || $document.createElement("div")
-			} : {action: INSERTION, index: i};
-		});
-		var actions = [];
-		for (var prop in existing) actions.push(existing[prop]);
-		var changes = actions.sort(sortChanges), newCached = new Array(cached.length);
-		newCached.nodes = cached.nodes.slice();
+				element: cached.nodes[existing[key].index] ||
+					$document.createElement("div")
+			} : {action: INSERTION, index: i}
+		})
+
+		var actions = []
+		for (var prop in existing) if (hasOwn.call(existing, prop)) {
+			actions.push(existing[prop])
+		}
+
+		var changes = actions.sort(sortChanges)
+		var newCached = new Array(cached.length)
+
+		newCached.nodes = cached.nodes.slice()
 
 		forEach(changes, function (change) {
-			var index = change.index;
+			var index = change.index
 			if (change.action === DELETION) {
-				clear(cached[index].nodes, cached[index]);
-				newCached.splice(index, 1);
+				clear(cached[index].nodes, cached[index])
+				newCached.splice(index, 1)
 			}
 			if (change.action === INSERTION) {
-				var dummy = $document.createElement("div");
-				dummy.key = data[index].attrs.key;
-				insertNode(parentElement, dummy, index);
+				var dummy = $document.createElement("div")
+				dummy.key = data[index].attrs.key
+				insertNode(parentElement, dummy, index)
 				newCached.splice(index, 0, {
 					attrs: {key: data[index].attrs.key},
 					nodes: [dummy]
-				});
-				newCached.nodes[index] = dummy;
+				})
+				newCached.nodes[index] = dummy
 			}
 
 			if (change.action === MOVE) {
-				var changeElement = change.element;
-				var maybeChanged = parentElement.childNodes[index];
+				var changeElement = change.element
+				var maybeChanged = parentElement.childNodes[index]
 				if (maybeChanged !== changeElement && changeElement !== null) {
-					parentElement.insertBefore(changeElement, maybeChanged || null);
+					parentElement.insertBefore(changeElement,
+						maybeChanged || null)
 				}
-				newCached[index] = cached[change.from];
-				newCached.nodes[index] = changeElement;
+				newCached[index] = cached[change.from]
+				newCached.nodes[index] = changeElement
 			}
-		});
+		})
 
-		return newCached;
+		return newCached
 	}
 
 	function diffKeys(data, cached, existing, parentElement) {
-		var keysDiffer = data.length !== cached.length;
+		var keysDiffer = data.length !== cached.length
+
 		if (!keysDiffer) {
 			forKeys(data, function (attrs, i) {
-				var cachedCell = cached[i];
-				return keysDiffer = cachedCell && cachedCell.attrs && cachedCell.attrs.key !== attrs.key;
-			});
+				var cachedCell = cached[i]
+				return keysDiffer = cachedCell &&
+					cachedCell.attrs &&
+					cachedCell.attrs.key !== attrs.key
+			})
 		}
 
-		return keysDiffer ? handleKeysDiffer(data, existing, cached, parentElement) : cached;
+		if (keysDiffer) {
+			return handleKeysDiffer(data, existing, cached, parentElement)
+		} else {
+			return cached
+		}
 	}
 
 	function diffArray(data, cached, nodes) {
-		//diff the array itself
+		// diff the array itself
 
-		//update the list of DOM nodes by collecting the nodes from each item
+		// update the list of DOM nodes by collecting the nodes from each item
 		forEach(data, function (_, i) {
-			if (cached[i] != null) nodes.push.apply(nodes, cached[i].nodes);
+			if (cached[i] != null) nodes.push.apply(nodes, cached[i].nodes)
 		})
-		//remove items from the end of the array if the new array is shorter than the old one. if errors ever happen here, the issue is most likely
-		//a bug in the construction of the `cached` data structure somewhere earlier in the program
+		// remove items from the end of the array if the new array is shorter
+		// than the old one. if errors ever happen here, the issue is most
+		// likely a bug in the construction of the `cached` data structure
+		// somewhere earlier in the program
 		forEach(cached.nodes, function (node, i) {
-			if (node.parentNode != null && nodes.indexOf(node) < 0) clear([node], [cached[i]]);
+			if (node.parentNode != null && nodes.indexOf(node) < 0) {
+				clear([node], [cached[i]])
+			}
 		})
-		if (data.length < cached.length) cached.length = data.length;
-		cached.nodes = nodes;
+
+		if (data.length < cached.length) cached.length = data.length
+		cached.nodes = nodes
 	}
 
 	function buildArrayKeys(data) {
-		var guid = 0;
+		var guid = 0
 		forKeys(data, function () {
 			forEach(data, function (attrs) {
-				if ((attrs = attrs && attrs.attrs) && attrs.key == null) attrs.key = "__mithril__" + guid++;
+				if ((attrs = attrs && attrs.attrs) && attrs.key == null) {
+					attrs.key = "__mithril__" + guid++
+				}
 			})
-			return 1;
-		});
+			return 1
+		})
+	}
+
+	function isDifferentEnough(data, cached, dataAttrKeys) {
+		if (data.tag !== cached.tag) return true
+
+		if (dataAttrKeys.sort().join() !==
+				Object.keys(cached.attrs).sort().join()) {
+			return true
+		}
+
+		if (data.attrs.id !== cached.attrs.id) {
+			return true
+		}
+
+		if (data.attrs.key !== cached.attrs.key) {
+			return true
+		}
+
+		if (m.redraw.strategy() === "all") {
+			return !cached.configContext || cached.configContext.retain !== true
+		}
+
+		if (m.redraw.strategy() === "diff") {
+			return cached.configContext && cached.configContext.retain === false
+		}
+
+		return false
 	}
 
 	function maybeRecreateObject(data, cached, dataAttrKeys) {
-		//if an element is different enough from the one in cache, recreate it
-		if (data.tag !== cached.tag ||
-				dataAttrKeys.sort().join() !== Object.keys(cached.attrs).sort().join() ||
-				data.attrs.id !== cached.attrs.id ||
-				data.attrs.key !== cached.attrs.key ||
-				(m.redraw.strategy() === "all" && (!cached.configContext || cached.configContext.retain !== true)) ||
-				(m.redraw.strategy() === "diff" && cached.configContext && cached.configContext.retain === false)) {
-			if (cached.nodes.length) clear(cached.nodes);
-			if (cached.configContext && isFunction(cached.configContext.onunload)) cached.configContext.onunload();
+		// if an element is different enough from the one in cache, recreate it
+		if (isDifferentEnough(data, cached, dataAttrKeys)) {
+			if (cached.nodes.length) clear(cached.nodes)
+
+			if (cached.configContext &&
+					isFunction(cached.configContext.onunload)) {
+				cached.configContext.onunload()
+			}
+
 			if (cached.controllers) {
 				forEach(cached.controllers, function (controller) {
-					if (controller.unload) controller.onunload({preventDefault: noop});
+					if (controller.onunload) controller.onunload({preventDefault: noop});
 				});
 			}
 		}
 	}
 
 	function getObjectNamespace(data, namespace) {
-		return data.attrs.xmlns ? data.attrs.xmlns :
-			data.tag === "svg" ? "http://www.w3.org/2000/svg" :
-			data.tag === "math" ? "http://www.w3.org/1998/Math/MathML" :
-			namespace;
+		if (data.attrs.xmlns) return data.attrs.xmlns
+		if (data.tag === "svg") return "http://www.w3.org/2000/svg"
+		if (data.tag === "math") return "http://www.w3.org/1998/Math/MathML"
+		return namespace
+	}
+
+	var pendingRequests = 0
+	m.startComputation = function () { pendingRequests++ }
+	m.endComputation = function () {
+		if (pendingRequests > 1) {
+			pendingRequests--
+		} else {
+			pendingRequests = 0
+			m.redraw()
+		}
 	}
 
 	function unloadCachedControllers(cached, views, controllers) {
 		if (controllers.length) {
-			cached.views = views;
-			cached.controllers = controllers;
+			cached.views = views
+			cached.controllers = controllers
 			forEach(controllers, function (controller) {
-				if (controller.onunload && controller.onunload.$old) controller.onunload = controller.onunload.$old;
-				if (pendingRequests && controller.onunload) {
-					var onunload = controller.onunload;
-					controller.onunload = noop;
-					controller.onunload.$old = onunload;
+				if (controller.onunload && controller.onunload.$old) {
+					controller.onunload = controller.onunload.$old
 				}
-			});
+
+				if (pendingRequests && controller.onunload) {
+					var onunload = controller.onunload
+					controller.onunload = noop
+					controller.onunload.$old = onunload
+				}
+			})
 		}
 	}
 
 	function scheduleConfigsToBeCalled(configs, data, node, isNew, cached) {
-		//schedule configs to be called. They are called after `build`
-		//finishes running
+		// schedule configs to be called. They are called after `build` finishes
+		// running
 		if (isFunction(data.attrs.config)) {
-			var context = cached.configContext = cached.configContext || {};
+			var context = cached.configContext = cached.configContext || {}
 
-			//bind
-			configs.push(function() {
-				return data.attrs.config.call(data, node, !isNew, context, cached);
-			});
+			// bind
+			configs.push(function () {
+				return data.attrs.config.call(data, node, !isNew, context,
+					cached)
+			})
 		}
 	}
 
-	function buildUpdatedNode(cached, data, editable, hasKeys, namespace, views, configs, controllers) {
-		var node = cached.nodes[0];
-		if (hasKeys) setAttributes(node, data.tag, data.attrs, cached.attrs, namespace);
-		cached.children = build(node, data.tag, undefined, undefined, data.children, cached.children, false, 0, data.attrs.contenteditable ? node : editable, namespace, configs);
-		cached.nodes.intact = true;
+	function buildUpdatedNode(
+		cached,
+		data,
+		editable,
+		hasKeys,
+		namespace,
+		views,
+		configs,
+		controllers
+	) {
+		var node = cached.nodes[0]
 
-		if (controllers.length) {
-			cached.views = views;
-			cached.controllers = controllers;
+		if (hasKeys) {
+			setAttributes(node, data.tag, data.attrs, cached.attrs, namespace)
 		}
 
-		return node;
+		cached.children = build(
+			node,
+			data.tag,
+			undefined,
+			undefined,
+			data.children,
+			cached.children,
+			false,
+			0,
+			data.attrs.contenteditable ? node : editable,
+			namespace,
+			configs
+		)
+
+		cached.nodes.intact = true
+
+		if (controllers.length) {
+			cached.views = views
+			cached.controllers = controllers
+		}
+
+		return node
 	}
 
 	function handleNonexistentNodes(data, parentElement, index) {
-		var nodes;
+		var nodes
 		if (data.$trusted) {
-			nodes = injectHTML(parentElement, index, data);
-		}
-		else {
-			nodes = [$document.createTextNode(data)];
-			if (!parentElement.nodeName.match(voidElements)) insertNode(parentElement, nodes[0], index);
+			nodes = injectHTML(parentElement, index, data)
+		} else {
+			nodes = [$document.createTextNode(data)]
+			if (!parentElement.nodeName.match(voidElements)) {
+				insertNode(parentElement, nodes[0], index)
+			}
 		}
 
-		var cached = typeof data === "string" || typeof data === "number" || typeof data === "boolean" ? new data.constructor(data) : data;
-		cached.nodes = nodes;
-		return cached;
+		var cached
+
+		if (typeof data === "string" ||
+				typeof data === "number" ||
+				typeof data === "boolean") {
+			cached = new data.constructor(data)
+		} else {
+			cached = data
+		}
+
+		cached.nodes = nodes
+		return cached
 	}
 
-	function reattachNodes(data, cached, parentElement, editable, index, parentTag) {
-		var nodes = cached.nodes;
+	function reattachNodes(
+		data,
+		cached,
+		parentElement,
+		editable,
+		index,
+		parentTag
+	) {
+		var nodes = cached.nodes
 		if (!editable || editable !== $document.activeElement) {
 			if (data.$trusted) {
 				clear(nodes, cached)
@@ -922,864 +1105,1370 @@ var m = (function app(window, undefined) {
 				editable.innerHTML = data
 			} else {
 				// was a trusted string
-				if (nodes[0].nodeType === 1 || nodes.length > 1 || (nodes[0].nodeValue.trim && !nodes[0].nodeValue.trim())) {
+				if (nodes[0].nodeType === 1 || nodes.length > 1 ||
+						(nodes[0].nodeValue.trim &&
+							!nodes[0].nodeValue.trim())) {
 					clear(cached.nodes, cached)
 					nodes = [$document.createTextNode(data)]
 				}
-				injectTextNode(parentElement, nodes[0], index, data);
+
+				injectTextNode(parentElement, nodes[0], index, data)
 			}
 		}
-		cached = new data.constructor(data);
-		cached.nodes = nodes;
-		return cached;
+		cached = new data.constructor(data)
+		cached.nodes = nodes
+		return cached
 	}
 
-	function handleText(cached, data, index, parentElement, shouldReattach, editable, parentTag) {
-		//handle text nodes
-		return cached.nodes.length === 0 ? handleNonexistentNodes(data, parentElement, index) :
-			cached.valueOf() !== data.valueOf() || shouldReattach === true ?
-				reattachNodes(data, cached, parentElement, editable, index, parentTag) :
-			(cached.nodes.intact = true, cached);
+	function handleTextNode(
+		cached,
+		data,
+		index,
+		parentElement,
+		shouldReattach,
+		editable,
+		parentTag
+	) {
+		if (!cached.nodes.length) {
+			return handleNonexistentNodes(data, parentElement, index)
+		} else if (cached.valueOf() !== data.valueOf() || shouldReattach) {
+			return reattachNodes(data, cached, parentElement, editable, index,
+				parentTag)
+		} else {
+			return (cached.nodes.intact = true, cached)
+		}
 	}
 
 	function getSubArrayCount(item) {
 		if (item.$trusted) {
-			//fix offset of next element if item was a trusted string w/ more than one html element
-			//the first clause in the regexp matches elements
-			//the second clause (after the pipe) matches text nodes
-			var match = item.match(/<[^\/]|\>\s*[^<]/g);
-			if (match != null) return match.length;
+			// fix offset of next element if item was a trusted string w/ more
+			// than one html element
+			// the first clause in the regexp matches elements
+			// the second clause (after the pipe) matches text nodes
+			var match = item.match(/<[^\/]|\>\s*[^<]/g)
+			if (match != null) return match.length
+		} else if (isArray(item)) {
+			return item.length
 		}
-		else if (isArray(item)) {
-			return item.length;
-		}
-		return 1;
+		return 1
 	}
 
-	function buildArray(data, cached, parentElement, index, parentTag, shouldReattach, editable, namespace, configs) {
-		data = flatten(data);
-		var nodes = [], intact = cached.length === data.length, subArrayCount = 0;
+	function buildArray(
+		data,
+		cached,
+		parentElement,
+		index,
+		parentTag,
+		shouldReattach,
+		editable,
+		namespace,
+		configs
+	) {
+		data = flatten(data)
+		var nodes = []
+		var intact = cached.length === data.length
+		var subArrayCount = 0
+
+		// keys algorithm: sort elements without recreating them if keys are
+		// present
+		//
+		// 1) create a map of all existing keys, and mark all for deletion
+		// 2) add new keys to map and mark them for addition
+		// 3) if key exists in new list, change action from deletion to a move
+		// 4) for each key, handle its corresponding action as marked in
+		//    previous steps
+
+		var existing = {}
+		var shouldMaintainIdentities = false
 
-		//keys algorithm: sort elements without recreating them if keys are present
-		//1) create a map of all existing keys, and mark all for deletion
-		//2) add new keys to map and mark them for addition
-		//3) if key exists in new list, change action from deletion to a move
-		//4) for each key, handle its corresponding action as marked in previous steps
-		var existing = {}, shouldMaintainIdentities = false;
 		forKeys(cached, function (attrs, i) {
-			shouldMaintainIdentities = true;
-			existing[cached[i].attrs.key] = {action: DELETION, index: i};
-		});
+			shouldMaintainIdentities = true
+			existing[cached[i].attrs.key] = {action: DELETION, index: i}
+		})
 
-		buildArrayKeys(data);
-		if (shouldMaintainIdentities) cached = diffKeys(data, cached, existing, parentElement);
-		//end key algorithm
+		buildArrayKeys(data)
+		if (shouldMaintainIdentities) {
+			cached = diffKeys(data, cached, existing, parentElement)
+		}
+		// end key algorithm
 
-		var cacheCount = 0;
-		//faster explicitly written
+		var cacheCount = 0
+		// faster explicitly written
 		for (var i = 0, len = data.length; i < len; i++) {
-			//diff each item in the array
-			var item = build(parentElement, parentTag, cached, index, data[i], cached[cacheCount], shouldReattach, index + subArrayCount || subArrayCount, editable, namespace, configs);
+			// diff each item in the array
+			var item = build(
+				parentElement,
+				parentTag,
+				cached,
+				index,
+				data[i],
+				cached[cacheCount],
+				shouldReattach,
+				index + subArrayCount || subArrayCount,
+				editable,
+				namespace,
+				configs)
 
 			if (item !== undefined) {
-				intact = intact && item.nodes.intact;
-				subArrayCount += getSubArrayCount(item);
-				cached[cacheCount++] = item;
+				intact = intact && item.nodes.intact
+				subArrayCount += getSubArrayCount(item)
+				cached[cacheCount++] = item
 			}
 		}
 
-		if (!intact) diffArray(data, cached, nodes);
+		if (!intact) diffArray(data, cached, nodes)
 		return cached
 	}
 
 	function makeCache(data, cached, index, parentIndex, parentCache) {
 		if (cached != null) {
-			if (type.call(cached) === type.call(data)) return cached;
+			if (type.call(cached) === type.call(data)) return cached
 
 			if (parentCache && parentCache.nodes) {
-				var offset = index - parentIndex, end = offset + (isArray(data) ? data : cached.nodes).length;
-				clear(parentCache.nodes.slice(offset, end), parentCache.slice(offset, end));
+				var offset = index - parentIndex
+				var end = offset + (isArray(data) ? data : cached.nodes).length
+				clear(
+					parentCache.nodes.slice(offset, end),
+					parentCache.slice(offset, end))
 			} else if (cached.nodes) {
-				clear(cached.nodes, cached);
+				clear(cached.nodes, cached)
 			}
 		}
 
-		cached = new data.constructor();
-		//if constructor creates a virtual dom element, use a blank object
-		//as the base cached node instead of copying the virtual el (#277)
-		if (cached.tag) cached = {};
-		cached.nodes = [];
-		return cached;
+		cached = new data.constructor()
+		// if constructor creates a virtual dom element, use a blank object as
+		// the base cached node instead of copying the virtual el (#277)
+		if (cached.tag) cached = {}
+		cached.nodes = []
+		return cached
 	}
 
 	function constructNode(data, namespace) {
-		return namespace === undefined ?
-			data.attrs.is ? $document.createElement(data.tag, data.attrs.is) : $document.createElement(data.tag) :
-			data.attrs.is ? $document.createElementNS(namespace, data.tag, data.attrs.is) : $document.createElementNS(namespace, data.tag);
+		if (data.attrs.is) {
+			if (namespace == null) {
+				return $document.createElement(data.tag, data.attrs.is)
+			} else {
+				return $document.createElementNS(namespace, data.tag,
+					data.attrs.is)
+			}
+		} else if (namespace == null) {
+			return $document.createElement(data.tag)
+		} else {
+			return $document.createElementNS(namespace, data.tag)
+		}
 	}
 
 	function constructAttrs(data, node, namespace, hasKeys) {
-		return hasKeys ? setAttributes(node, data.tag, data.attrs, {}, namespace) : data.attrs;
+		if (hasKeys) {
+			return setAttributes(node, data.tag, data.attrs, {}, namespace)
+		} else {
+			return data.attrs
+		}
 	}
 
-	function constructChildren(data, node, cached, editable, namespace, configs) {
-		return data.children != null && data.children.length > 0 ?
-			build(node, data.tag, undefined, undefined, data.children, cached.children, true, 0, data.attrs.contenteditable ? node : editable, namespace, configs) :
-			data.children;
+	function constructChildren(
+		data,
+		node,
+		cached,
+		editable,
+		namespace,
+		configs
+	) {
+		if (data.children != null && data.children.length > 0) {
+			return build(
+				node,
+				data.tag,
+				undefined,
+				undefined,
+				data.children,
+				cached.children,
+				true,
+				0,
+				data.attrs.contenteditable ? node : editable,
+				namespace,
+				configs)
+		} else {
+			return data.children
+		}
 	}
 
-	function reconstructCached(data, attrs, children, node, namespace, views, controllers) {
-		var cached = {tag: data.tag, attrs: attrs, children: children, nodes: [node]};
-		unloadCachedControllers(cached, views, controllers);
-		if (cached.children && !cached.children.nodes) cached.children.nodes = [];
-		//edge case: setting value on <select> doesn't work before children exist, so set it again after children have been created
-		if (data.tag === "select" && "value" in data.attrs) setAttributes(node, data.tag, {value: data.attrs.value}, {}, namespace);
+	function reconstructCached(
+		data,
+		attrs,
+		children,
+		node,
+		namespace,
+		views,
+		controllers
+	) {
+		var cached = {
+			tag: data.tag,
+			attrs: attrs,
+			children: children,
+			nodes: [node]
+		}
+
+		unloadCachedControllers(cached, views, controllers)
+
+		if (cached.children && !cached.children.nodes) {
+			cached.children.nodes = []
+		}
+
+		// edge case: setting value on <select> doesn't work before children
+		// exist, so set it again after children have been created
+		if (data.tag === "select" && "value" in data.attrs) {
+			setAttributes(node, data.tag, {value: data.attrs.value}, {},
+				namespace)
+		}
+
 		return cached
 	}
 
 	function getController(views, view, cachedControllers, controller) {
-		var controllerIndex = m.redraw.strategy() === "diff" && views ? views.indexOf(view) : -1;
-		return controllerIndex > -1 ? cachedControllers[controllerIndex] :
-			typeof controller === "function" ? new controller() : {};
+		var controllerIndex
+
+		if (m.redraw.strategy() === "diff" && views) {
+			controllerIndex = views.indexOf(view)
+		} else {
+			controllerIndex = -1
+		}
+
+		if (controllerIndex > -1) {
+			return cachedControllers[controllerIndex]
+		} else if (isFunction(controller)) {
+			return new controller()
+		} else {
+			return {}
+		}
 	}
 
+	var unloaders = []
+
 	function updateLists(views, controllers, view, controller) {
-		if (controller.onunload != null) unloaders.push({controller: controller, handler: controller.onunload});
-		views.push(view);
-		controllers.push(controller);
+		if (controller.onunload != null) {
+			unloaders.push({
+				controller: controller,
+				handler: controller.onunload
+			})
+		}
+
+		views.push(view)
+		controllers.push(controller)
 	}
 
+	var forcing = false
 	function checkView(data, view, cached, cachedControllers, controllers, views) {
-		var controller = getController(cached.views, view, cachedControllers, data.controller);
-		var key = (data && data.attrs && data.attrs.key);
-		data = pendingRequests === 0 || forcing || cachedControllers && cachedControllers.indexOf(controller) > -1 ? data.view(controller) : {tag: "placeholder"};
+		var controller = getController(cached.views, view, cachedControllers, data.controller)
+		var key = data && data.attrs && data.attrs.key
+		data = pendingRequests === 0 || forcing || cachedControllers && cachedControllers.indexOf(controller) > -1 ? data.view(controller) : {tag: "placeholder"}
 		if (data.subtree === "retain") return data;
-		(data.attrs = data.attrs || {}).key = key;
-		updateLists(views, controllers, view, controller);
-		return data;
+		data.attrs = data.attrs || {}
+		data.attrs.key = key
+		updateLists(views, controllers, view, controller)
+		return data
 	}
 
 	function markViews(data, cached, views, controllers) {
-		var cachedControllers = cached && cached.controllers;
-		while (data.view != null) data = checkView(data, data.view.$original || data.view, cached, cachedControllers, controllers, views);
-		return data;
+		var cachedControllers = cached && cached.controllers
+
+		while (data.view != null) {
+			data = checkView(
+				data,
+				data.view.$original || data.view,
+				cached,
+				cachedControllers,
+				controllers,
+				views)
+		}
+
+		return data
 	}
 
-	function buildObject(data, cached, editable, parentElement, index, shouldReattach, namespace, configs) {
-		var views = [], controllers = [];
-		data = markViews(data, cached, views, controllers);
-		if (data.subtree === "retain") return cached;
-		if (!data.tag && controllers.length) throw new Error("Component template must return a virtual element, not an array, string, etc.");
-		data.attrs = data.attrs || {};
-		cached.attrs = cached.attrs || {};
-		var dataAttrKeys = Object.keys(data.attrs);
-		var hasKeys = dataAttrKeys.length > ("key" in data.attrs ? 1 : 0);
-		maybeRecreateObject(data, cached, dataAttrKeys);
-		if (!isString(data.tag)) return;
-		var isNew = cached.nodes.length === 0;
-		namespace = getObjectNamespace(data, namespace);
-		var node;
+	function buildObject( // eslint-disable-line max-statements
+		data,
+		cached,
+		editable,
+		parentElement,
+		index,
+		shouldReattach,
+		namespace,
+		configs
+	) {
+		var views = []
+		var controllers = []
+
+		data = markViews(data, cached, views, controllers)
+
+		if (data.subtree === "retain") return cached
+
+		if (!data.tag && controllers.length) {
+			throw new Error("Component template must return a virtual " +
+				"element, not an array, string, etc.")
+		}
+
+		data.attrs = data.attrs || {}
+		cached.attrs = cached.attrs || {}
+
+		var dataAttrKeys = Object.keys(data.attrs)
+		var hasKeys = dataAttrKeys.length > ("key" in data.attrs ? 1 : 0)
+
+		maybeRecreateObject(data, cached, dataAttrKeys)
+
+		if (!isString(data.tag)) return
+
+		var isNew = cached.nodes.length === 0
+
+		namespace = getObjectNamespace(data, namespace)
+
+		var node
 		if (isNew) {
-			node = constructNode(data, namespace);
-			//set attributes first, then create children
+			node = constructNode(data, namespace)
+			// set attributes first, then create children
 			var attrs = constructAttrs(data, node, namespace, hasKeys)
-			var children = constructChildren(data, node, cached, editable, namespace, configs);
-			cached = reconstructCached(data, attrs, children, node, namespace, views, controllers);
+
+			var children = constructChildren(data, node, cached, editable,
+				namespace, configs)
+
+			cached = reconstructCached(
+				data,
+				attrs,
+				children,
+				node,
+				namespace,
+				views,
+				controllers)
+		} else {
+			node = buildUpdatedNode(
+				cached,
+				data,
+				editable,
+				hasKeys,
+				namespace,
+				views,
+				configs,
+				controllers)
 		}
-		else {
-			node = buildUpdatedNode(cached, data, editable, hasKeys, namespace, views, configs, controllers);
+
+		if (isNew || shouldReattach === true && node != null) {
+			insertNode(parentElement, node, index)
 		}
-		if (isNew || shouldReattach === true && node != null) insertNode(parentElement, node, index);
-		//schedule configs to be called. They are called after `build`
-		//finishes running
-		scheduleConfigsToBeCalled(configs, data, node, isNew, cached);
+
+		// The configs are called after `build` finishes running
+		scheduleConfigsToBeCalled(configs, data, node, isNew, cached)
+
 		return cached
 	}
 
-	function build(parentElement, parentTag, parentCache, parentIndex, data, cached, shouldReattach, index, editable, namespace, configs) {
-		//`build` is a recursive function that manages creation/diffing/removal
-		//of DOM elements based on comparison between `data` and `cached`
-		//the diff algorithm can be summarized as this:
-		//1 - compare `data` and `cached`
-		//2 - if they are different, copy `data` to `cached` and update the DOM
-		//    based on what the difference is
-		//3 - recursively apply this algorithm for every array and for the
-		//    children of every virtual element
+	function build(
+		parentElement,
+		parentTag,
+		parentCache,
+		parentIndex,
+		data,
+		cached,
+		shouldReattach,
+		index,
+		editable,
+		namespace,
+		configs
+	) {
+		/*
+		 * `build` is a recursive function that manages creation/diffing/removal
+		 * of DOM elements based on comparison between `data` and `cached` the
+		 * diff algorithm can be summarized as this:
+		 *
+		 * 1 - compare `data` and `cached`
+		 * 2 - if they are different, copy `data` to `cached` and update the DOM
+		 *     based on what the difference is
+		 * 3 - recursively apply this algorithm for every array and for the
+		 *     children of every virtual element
+		 *
+		 * The `cached` data structure is essentially the same as the previous
+		 * redraw's `data` data structure, with a few additions:
+		 * - `cached` always has a property called `nodes`, which is a list of
+		 *    DOM elements that correspond to the data represented by the
+		 *    respective virtual element
+		 * - in order to support attaching `nodes` as a property of `cached`,
+		 *    `cached` is *always* a non-primitive object, i.e. if the data was
+		 *    a string, then cached is a String instance. If data was `null` or
+		 *    `undefined`, cached is `new String("")`
+		 * - `cached also has a `configContext` property, which is the state
+		 *    storage object exposed by config(element, isInitialized, context)
+		 * - when `cached` is an Object, it represents a virtual element; when
+		 *    it's an Array, it represents a list of elements; when it's a
+		 *    String, Number or Boolean, it represents a text node
+		 *
+		 * `parentElement` is a DOM element used for W3C DOM API calls
+		 * `parentTag` is only used for handling a corner case for textarea
+		 * values
+		 * `parentCache` is used to remove nodes in some multi-node cases
+		 * `parentIndex` and `index` are used to figure out the offset of nodes.
+		 * They're artifacts from before arrays started being flattened and are
+		 * likely refactorable
+		 * `data` and `cached` are, respectively, the new and old nodes being
+		 * diffed
+		 * `shouldReattach` is a flag indicating whether a parent node was
+		 * recreated (if so, and if this node is reused, then this node must
+		 * reattach itself to the new parent)
+		 * `editable` is a flag that indicates whether an ancestor is
+		 * contenteditable
+		 * `namespace` indicates the closest HTML namespace as it cascades down
+		 * from an ancestor
+		 * `configs` is a list of config functions to run after the topmost
+		 * `build` call finishes running
+		 *
+		 * there's logic that relies on the assumption that null and undefined
+		 * data are equivalent to empty strings
+		 * - this prevents lifecycle surprises from procedural helpers that mix
+		 *   implicit and explicit return statements (e.g.
+		 *   function foo() {if (cond) return m("div")}
+		 * - it simplifies diffing code
+		 */
+		data = dataToString(data)
+		if (data.subtree === "retain") return cached
+		cached = makeCache(data, cached, index, parentIndex, parentCache)
 
-		//the `cached` data structure is essentially the same as the previous
-		//redraw's `data` data structure, with a few additions:
-		//- `cached` always has a property called `nodes`, which is a list of
-		//   DOM elements that correspond to the data represented by the
-		//   respective virtual element
-		//- in order to support attaching `nodes` as a property of `cached`,
-		//   `cached` is *always* a non-primitive object, i.e. if the data was
-		//   a string, then cached is a String instance. If data was `null` or
-		//   `undefined`, cached is `new String("")`
-		//- `cached also has a `configContext` property, which is the state
-		//   storage object exposed by config(element, isInitialized, context)
-		//- when `cached` is an Object, it represents a virtual element; when
-		//   it's an Array, it represents a list of elements; when it's a
-		//   String, Number or Boolean, it represents a text node
-
-		//`parentElement` is a DOM element used for W3C DOM API calls
-		//`parentTag` is only used for handling a corner case for textarea
-		//values
-		//`parentCache` is used to remove nodes in some multi-node cases
-		//`parentIndex` and `index` are used to figure out the offset of nodes.
-		//They're artifacts from before arrays started being flattened and are
-		//likely refactorable
-		//`data` and `cached` are, respectively, the new and old nodes being
-		//diffed
-		//`shouldReattach` is a flag indicating whether a parent node was
-		//recreated (if so, and if this node is reused, then this node must
-		//reattach itself to the new parent)
-		//`editable` is a flag that indicates whether an ancestor is
-		//contenteditable
-		//`namespace` indicates the closest HTML namespace as it cascades down
-		//from an ancestor
-		//`configs` is a list of config functions to run after the topmost
-		//`build` call finishes running
-
-		//there's logic that relies on the assumption that null and undefined
-		//data are equivalent to empty strings
-		//- this prevents lifecycle surprises from procedural helpers that mix
-		//  implicit and explicit return statements (e.g.
-		//  function foo() {if (cond) return m("div")}
-		//- it simplifies diffing code
-		data = dataToString(data);
-		if (data.subtree === "retain") return cached;
-		cached = makeCache(data, cached, index, parentIndex, parentCache);
-		return isArray(data) ? buildArray(data, cached, parentElement, index, parentTag, shouldReattach, editable, namespace, configs) :
-			data != null && isObject(data) ? buildObject(data, cached, editable, parentElement, index, shouldReattach, namespace, configs) :
-			!isFunction(data) ? handleText(cached, data, index, parentElement, shouldReattach, editable, parentTag) :
-			cached;
+		if (isArray(data)) {
+			return buildArray(
+				data,
+				cached,
+				parentElement,
+				index,
+				parentTag,
+				shouldReattach,
+				editable,
+				namespace,
+				configs)
+		} else if (data != null && isObject(data)) {
+			return buildObject(
+				data,
+				cached,
+				editable,
+				parentElement,
+				index,
+				shouldReattach,
+				namespace,
+				configs)
+		} else if (!isFunction(data)) {
+			return handleTextNode(
+				cached,
+				data,
+				index,
+				parentElement,
+				shouldReattach,
+				editable,
+				parentTag)
+		} else {
+			return cached
+		}
 	}
-	function sortChanges(a, b) { return a.action - b.action || a.index - b.index; }
-	function setAttributes(node, tag, dataAttrs, cachedAttrs, namespace) {
-		for (var attrName in dataAttrs) {
-			var dataAttr = dataAttrs[attrName];
-			var cachedAttr = cachedAttrs[attrName];
-			if (!(attrName in cachedAttrs) || (cachedAttr !== dataAttr)) {
-				cachedAttrs[attrName] = dataAttr;
-				try {
-					//`config` isn't a real attributes, so ignore it
-					if (attrName === "config" || attrName === "key") continue;
-					//hook event handlers to the auto-redrawing system
-					else if (isFunction(dataAttr) && attrName.slice(0, 2) === "on") {
-						node[attrName] = autoredraw(dataAttr, node);
-					}
-					//handle `style: {...}`
-					else if (attrName === "style" && dataAttr != null && isObject(dataAttr)) {
-						for (var rule in dataAttr) {
-							if (cachedAttr == null || cachedAttr[rule] !== dataAttr[rule]) node.style[rule] = dataAttr[rule];
-						}
-						for (var rule in cachedAttr) {
-							if (!(rule in dataAttr)) node.style[rule] = "";
-						}
-					}
-					//handle SVG
-					else if (namespace != null) {
-						if (attrName === "href") node.setAttributeNS("http://www.w3.org/1999/xlink", "href", dataAttr);
-						else node.setAttribute(attrName === "className" ? "class" : attrName, dataAttr);
-					}
-					//handle cases that are properties (but ignore cases where we should use setAttribute instead)
-					//- list and form are typically used as strings, but are DOM element references in js
-					//- when using CSS selectors (e.g. `m("[style='']")`), style is used as a string, but it's an object in js
-					else if (attrName in node && attrName !== "list" && attrName !== "style" && attrName !== "form" && attrName !== "type" && attrName !== "width" && attrName !== "height") {
-						//#348 don't set the value if not needed otherwise cursor placement breaks in Chrome
-						try {
-							if (tag !== "input" || node[attrName] !== dataAttr) node[attrName] = dataAttr;
-						} catch (e) {
-							node.setAttribute(attrName, dataAttr);
-						}
-					}
-					else node.setAttribute(attrName, dataAttr);
-				}
-				catch (e) {
-					//swallow IE's invalid argument errors to mimic HTML's fallback-to-doing-nothing-on-invalid-attributes behavior
-					if (e.message.indexOf("Invalid argument") < 0) throw e;
-				}
-			}
-			//#348 dataAttr may not be a string, so use loose comparison (double equal) instead of strict (triple equal)
-			else if (attrName === "value" && tag === "input" && node.value != dataAttr) {
-				node.value = dataAttr;
+
+	function sortChanges(a, b) {
+		return a.action - b.action || a.index - b.index
+	}
+
+	function copyStyleAttrs(node, dataAttr, cachedAttr) {
+		for (var rule in dataAttr) if (hasOwn.call(dataAttr, rule)) {
+			if (cachedAttr == null || cachedAttr[rule] !== dataAttr[rule]) {
+				node.style[rule] = dataAttr[rule]
 			}
 		}
-		return cachedAttrs;
+
+		for (rule in cachedAttr) if (hasOwn.call(cachedAttr, rule)) {
+			if (!hasOwn.call(dataAttr, rule)) node.style[rule] = ""
+		}
 	}
+
+	function shouldUseSetAttribute(attrName) {
+		return attrName !== "list" &&
+			attrName !== "style" &&
+			attrName !== "form" &&
+			attrName !== "type" &&
+			attrName !== "width" &&
+			attrName !== "height"
+	}
+
+	function setSingleAttr(
+		node,
+		attrName,
+		dataAttr,
+		cachedAttr,
+		tag,
+		namespace
+	) {
+		if (attrName === "config" || attrName === "key") {
+			// `config` isn't a real attribute, so ignore it
+			return true
+		} else if (isFunction(dataAttr) && attrName.slice(0, 2) === "on") {
+			// hook event handlers to the auto-redrawing system
+			node[attrName] = autoredraw(dataAttr, node)
+		} else if (attrName === "style" && dataAttr != null &&
+				isObject(dataAttr)) {
+			// handle `style: {...}`
+			copyStyleAttrs(node, dataAttr, cachedAttr)
+		} else if (namespace != null) {
+			// handle SVG
+			if (attrName === "href") {
+				node.setAttributeNS("http://www.w3.org/1999/xlink",
+					"href", dataAttr)
+			} else {
+				node.setAttribute(
+					attrName === "className" ? "class" : attrName,
+					dataAttr)
+			}
+		} else if (attrName in node && shouldUseSetAttribute(attrName)) {
+			// handle cases that are properties (but ignore cases where we
+			// should use setAttribute instead)
+			//
+			// - list and form are typically used as strings, but are DOM
+			//   element references in js
+			//
+			// - when using CSS selectors (e.g. `m("[style='']")`), style is
+			//   used as a string, but it's an object in js
+			//
+			// #348 don't set the value if not needed - otherwise, cursor
+			// placement breaks in Chrome
+			try {
+				if (tag !== "input" || node[attrName] !== dataAttr) {
+					node[attrName] = dataAttr
+				}
+			} catch (e) {
+				node.setAttribute(attrName, dataAttr)
+			}
+		}
+		else node.setAttribute(attrName, dataAttr)
+	}
+
+	function trySetAttr(
+		node,
+		attrName,
+		dataAttr,
+		cachedAttr,
+		cachedAttrs,
+		tag,
+		namespace
+	) {
+		if (!(attrName in cachedAttrs) || (cachedAttr !== dataAttr)) {
+			cachedAttrs[attrName] = dataAttr
+			try {
+				return setSingleAttr(
+					node,
+					attrName,
+					dataAttr,
+					cachedAttr,
+					tag,
+					namespace)
+			} catch (e) {
+				// swallow IE's invalid argument errors to mimic HTML's
+				// fallback-to-doing-nothing-on-invalid-attributes behavior
+				if (e.message.indexOf("Invalid argument") < 0) throw e
+			}
+		} else if (attrName === "value" && tag === "input" &&
+				node.value !== dataAttr) {
+			// #348 dataAttr may not be a string, so use loose comparison
+			node.value = dataAttr
+		}
+	}
+
+	function setAttributes(node, tag, dataAttrs, cachedAttrs, namespace) {
+		for (var attrName in dataAttrs) if (hasOwn.call(dataAttrs, attrName)) {
+			if (trySetAttr(
+					node,
+					attrName,
+					dataAttrs[attrName],
+					cachedAttrs[attrName],
+					cachedAttrs,
+					tag,
+					namespace)) {
+				continue
+			}
+		}
+		return cachedAttrs
+	}
+
 	function clear(nodes, cached) {
 		for (var i = nodes.length - 1; i > -1; i--) {
 			if (nodes[i] && nodes[i].parentNode) {
-				try { nodes[i].parentNode.removeChild(nodes[i]); }
-				catch (e) {} //ignore if this fails due to order of events (see http://stackoverflow.com/questions/21926083/failed-to-execute-removechild-on-node)
-				cached = [].concat(cached);
-				if (cached[i]) unload(cached[i]);
+				try {
+					nodes[i].parentNode.removeChild(nodes[i])
+				} catch (e) {
+					/* eslint-disable max-len */
+					// ignore if this fails due to order of events (see
+					// http://stackoverflow.com/questions/21926083/failed-to-execute-removechild-on-node)
+					/* eslint-enable max-len */
+				}
+				cached = [].concat(cached)
+				if (cached[i]) unload(cached[i])
 			}
 		}
-		//release memory if nodes is an array. This check should fail if nodes is a NodeList (see loop above)
-		if (nodes.length) nodes.length = 0;
+		// release memory if nodes is an array. This check should fail if nodes
+		// is a NodeList (see loop above)
+		if (nodes.length) {
+			nodes.length = 0
+		}
 	}
+
 	function unload(cached) {
 		if (cached.configContext && isFunction(cached.configContext.onunload)) {
-			cached.configContext.onunload();
-			cached.configContext.onunload = null;
+			cached.configContext.onunload()
+			cached.configContext.onunload = null
 		}
 		if (cached.controllers) {
 			forEach(cached.controllers, function (controller) {
-				if (isFunction(controller.onunload)) controller.onunload({preventDefault: noop});
-			});
+				if (isFunction(controller.onunload)) {
+					controller.onunload({preventDefault: noop})
+				}
+			})
 		}
 		if (cached.children) {
-			if (isArray(cached.children)) forEach(cached.children, unload);
-			else if (cached.children.tag) unload(cached.children);
+			if (isArray(cached.children)) forEach(cached.children, unload)
+			else if (cached.children.tag) unload(cached.children)
 		}
 	}
+
 	function appendTextFragment(parentElement, data) {
-	  try {
-	    parentElement.appendChild($document.createRange().createContextualFragment(data));
-	  } catch (e) {
-	    parentElement.insertAdjacentHTML("beforeend", data);
-	  }
+		try {
+			parentElement.appendChild(
+				$document.createRange().createContextualFragment(data))
+		} catch (e) {
+			parentElement.insertAdjacentHTML("beforeend", data)
+		}
 	}
+
 	function injectHTML(parentElement, index, data) {
-		var nextSibling = parentElement.childNodes[index];
+		var nextSibling = parentElement.childNodes[index]
 		if (nextSibling) {
-			var isElement = nextSibling.nodeType !== 1;
-			var placeholder = $document.createElement("span");
+			var isElement = nextSibling.nodeType !== 1
+			var placeholder = $document.createElement("span")
 			if (isElement) {
-				parentElement.insertBefore(placeholder, nextSibling || null);
-				placeholder.insertAdjacentHTML("beforebegin", data);
-				parentElement.removeChild(placeholder);
+				parentElement.insertBefore(placeholder, nextSibling || null)
+				placeholder.insertAdjacentHTML("beforebegin", data)
+				parentElement.removeChild(placeholder)
+			} else {
+				nextSibling.insertAdjacentHTML("beforebegin", data)
 			}
-			else nextSibling.insertAdjacentHTML("beforebegin", data);
+		} else {
+			appendTextFragment(parentElement, data)
 		}
-		else {
-			appendTextFragment(parentElement, data);
-		}
-		var nodes = [];
+
+		var nodes = []
+
 		while (parentElement.childNodes[index] !== nextSibling) {
-			nodes.push(parentElement.childNodes[index]);
-			index++;
+			nodes.push(parentElement.childNodes[index])
+			index++
 		}
-		return nodes;
+
+		return nodes
 	}
+
 	function autoredraw(callback, object) {
-		return function(e) {
-			e = e || event;
-			m.redraw.strategy("diff");
-			m.startComputation();
-			try { return callback.call(object, e); }
-			finally {
-				endFirstComputation();
+		return function (e) {
+			e = e || event
+			m.redraw.strategy("diff")
+			m.startComputation()
+			try {
+				return callback.call(object, e)
+			} finally {
+				endFirstComputation()
 			}
-		};
+		}
 	}
 
-	var html;
+	var html
 	var documentNode = {
-		appendChild: function(node) {
-			if (html === undefined) html = $document.createElement("html");
-			if ($document.documentElement && $document.documentElement !== node) {
-				$document.replaceChild(node, $document.documentElement);
+		appendChild: function (node) {
+			if (html === undefined) html = $document.createElement("html")
+			if ($document.documentElement &&
+					$document.documentElement !== node) {
+				$document.replaceChild(node, $document.documentElement)
+			} else {
+				$document.appendChild(node)
 			}
-			else $document.appendChild(node);
-			this.childNodes = $document.childNodes;
+
+			this.childNodes = $document.childNodes
 		},
-		insertBefore: function(node) {
-			this.appendChild(node);
+
+		insertBefore: function (node) {
+			this.appendChild(node)
 		},
+
 		childNodes: []
-	};
-	var nodeCache = [], cellCache = {};
-	m.render = function(root, cell, forceRecreation) {
-		var configs = [];
-		if (!root) throw new Error("Ensure the DOM element being passed to m.route/m.mount/m.render is not undefined.");
-		var id = getCellCacheKey(root);
-		var isDocumentRoot = root === $document;
-		var node = isDocumentRoot || root === $document.documentElement ? documentNode : root;
-		if (isDocumentRoot && cell.tag !== "html") cell = {tag: "html", attrs: {}, children: cell};
-		if (cellCache[id] === undefined) clear(node.childNodes);
-		if (forceRecreation === true) reset(root);
-		cellCache[id] = build(node, null, undefined, undefined, cell, cellCache[id], false, 0, null, undefined, configs);
-		forEach(configs, function (config) { config(); });
-	};
-	function getCellCacheKey(element) {
-		var index = nodeCache.indexOf(element);
-		return index < 0 ? nodeCache.push(element) - 1 : index;
 	}
 
-	m.trust = function(value) {
-		value = new String(value);
-		value.$trusted = true;
-		return value;
-	};
+	var nodeCache = []
+	var cellCache = {}
+
+	m.render = function (root, cell, forceRecreation) {
+		if (!root) {
+			throw new Error("Ensure the DOM element being passed to " +
+				"m.route/m.mount/m.render is not undefined.")
+		}
+		var configs = []
+		var id = getCellCacheKey(root)
+		var isDocumentRoot = root === $document
+		var node
+
+		if (isDocumentRoot || root === $document.documentElement) {
+			node = documentNode
+		} else {
+			node = root
+		}
+
+		if (isDocumentRoot && cell.tag !== "html") {
+			cell = {tag: "html", attrs: {}, children: cell}
+		}
+
+		if (cellCache[id] === undefined) clear(node.childNodes)
+		if (forceRecreation === true) reset(root)
+
+		cellCache[id] = build(
+			node,
+			null,
+			undefined,
+			undefined,
+			cell,
+			cellCache[id],
+			false,
+			0,
+			null,
+			undefined,
+			configs)
+
+		forEach(configs, function (config) { config() })
+	}
+
+	function getCellCacheKey(element) {
+		var index = nodeCache.indexOf(element)
+		return index < 0 ? nodeCache.push(element) - 1 : index
+	}
+
+	m.trust = function (value) {
+		value = new String(value) // eslint-disable-line no-new-wrappers
+		value.$trusted = true
+		return value
+	}
 
 	function gettersetter(store) {
-		var prop = function() {
-			if (arguments.length) store = arguments[0];
-			return store;
-		};
+		function prop() {
+			if (arguments.length) store = arguments[0]
+			return store
+		}
 
-		prop.toJSON = function() {
-			return store;
-		};
+		prop.toJSON = function () {
+			return store
+		}
 
-		return prop;
+		return prop
 	}
 
 	m.prop = function (store) {
-		//note: using non-strict equality check here because we're checking if store is null OR undefined
-		if ((store != null && isObject(store) || isFunction(store)) && isFunction(store.then)) {
-			return propify(store);
+		if ((store != null && isObject(store) || isFunction(store)) &&
+				isFunction(store.then)) {
+			return propify(store)
 		}
 
-		return gettersetter(store);
-	};
-
-	var roots = [], components = [], controllers = [], lastRedrawId = null, lastRedrawCallTime = 0, computePreRedrawHook = null, computePostRedrawHook = null, topComponent, unloaders = [];
-	var FRAME_BUDGET = 16; //60 frames per second = 1 call per 16 ms
-	function parameterize(component, args) {
-		var controller = function() {
-			return (component.controller || noop).apply(this, args) || this;
-		};
-		if (component.controller) controller.prototype = component.controller.prototype;
-		var view = function(ctrl) {
-			var currentArgs = arguments.length > 1 ? args.concat([].slice.call(arguments, 1)) : args;
-			return component.view.apply(component, currentArgs ? [ctrl].concat(currentArgs) : [ctrl]);
-		};
-		view.$original = component.view;
-		var output = {controller: controller, view: view};
-		if (args[0] && args[0].key != null) output.attrs = {key: args[0].key};
-		return output;
+		return gettersetter(store)
 	}
-	m.component = function(component) {
-		for (var args = [], i = 1; i < arguments.length; i++) args.push(arguments[i]);
-		return parameterize(component, args);
-	};
-	m.mount = m.module = function(root, component) {
-		if (!root) throw new Error("Please ensure the DOM element exists before rendering a template into it.");
-		var index = roots.indexOf(root);
-		if (index < 0) index = roots.length;
 
-		var isPrevented = false;
-		var event = {preventDefault: function() {
-			isPrevented = true;
-			computePreRedrawHook = computePostRedrawHook = null;
-		}};
+	var roots = []
+	var components = []
+	var controllers = []
+	var lastRedrawId = null
+	var lastRedrawCallTime = 0
+	var computePreRedrawHook = null
+	var computePostRedrawHook = null
+	var topComponent
+	var FRAME_BUDGET = 16 // 60 frames per second = 1 call per 16 ms
+
+	function parameterize(component, args) {
+		function controller() {
+			/* eslint-disable no-invalid-this */
+			return (component.controller || noop).apply(this, args) || this
+			/* eslint-enable no-invalid-this */
+		}
+
+		if (component.controller) {
+			controller.prototype = component.controller.prototype
+		}
+
+		function view(ctrl) {
+			var currentArgs = [ctrl].concat(args)
+			for (var i = 1; i < arguments.length; i++) {
+				currentArgs.push(arguments[i])
+			}
+
+			return component.view.apply(component, currentArgs)
+		}
+
+		view.$original = component.view
+		var output = {controller: controller, view: view}
+		if (args[0] && args[0].key != null) output.attrs = {key: args[0].key}
+		return output
+	}
+
+	m.component = function (component) {
+		for (var args = [], i = 1; i < arguments.length; i++) {
+			args.push(arguments[i])
+		}
+
+		return parameterize(component, args)
+	}
+
+	function checkPrevented(component, root, index, isPrevented) {
+		if (!isPrevented) {
+			m.redraw.strategy("all")
+			m.startComputation()
+			roots[index] = root
+			var currentComponent
+
+			if (component) {
+				currentComponent = topComponent = component
+			} else {
+				currentComponent = topComponent = component = {controller: noop}
+			}
+
+			var controller = new (component.controller || noop)()
+
+			// controllers may call m.mount recursively (via m.route redirects,
+			// for example)
+			// this conditional ensures only the last recursive m.mount call is
+			// applied
+			if (currentComponent === topComponent) {
+				controllers[index] = controller
+				components[index] = component
+			}
+			endFirstComputation()
+			if (component === null) {
+				removeRootElement(root, index)
+			}
+			return controllers[index]
+		} else if (component == null) {
+			removeRootElement(root, index)
+		}
+	}
+
+	m.mount = m.module = function (root, component) {
+		if (!root) {
+			throw new Error("Please ensure the DOM element exists before " +
+				"rendering a template into it.")
+		}
+
+		var index = roots.indexOf(root)
+		if (index < 0) index = roots.length
+
+		var isPrevented = false
+		var event = {
+			preventDefault: function () {
+				isPrevented = true
+				computePreRedrawHook = computePostRedrawHook = null
+			}
+		}
 
 		forEach(unloaders, function (unloader) {
-			unloader.handler.call(unloader.controller, event);
-			unloader.controller.onunload = null;
-		});
+			unloader.handler.call(unloader.controller, event)
+			unloader.controller.onunload = null
+		})
 
 		if (isPrevented) {
 			forEach(unloaders, function (unloader) {
-				unloader.controller.onunload = unloader.handler;
-			});
+				unloader.controller.onunload = unloader.handler
+			})
+		} else {
+			unloaders = []
 		}
-		else unloaders = [];
 
 		if (controllers[index] && isFunction(controllers[index].onunload)) {
-			controllers[index].onunload(event);
+			controllers[index].onunload(event)
 		}
 
-		var isNullComponent = component === null;
-
-		if (!isPrevented) {
-			m.redraw.strategy("all");
-			m.startComputation();
-			roots[index] = root;
-			var currentComponent = component ? (topComponent = component) : (topComponent = component = {controller: noop});
-			var controller = new (component.controller || noop)();
-			//controllers may call m.mount recursively (via m.route redirects, for example)
-			//this conditional ensures only the last recursive m.mount call is applied
-			if (currentComponent === topComponent) {
-				controllers[index] = controller;
-				components[index] = component;
-			}
-			endFirstComputation();
-			if (isNullComponent) {
-				removeRootElement(root, index);
-			}
-			return controllers[index];
-		}
-		if (isNullComponent) {
-			removeRootElement(root, index);
-		}
-	};
+		return checkPrevented(component, root, index, isPrevented)
+	}
 
 	function removeRootElement(root, index) {
-		roots.splice(index, 1);
-		controllers.splice(index, 1);
-		components.splice(index, 1);
-		reset(root);
-		nodeCache.splice(getCellCacheKey(root), 1);
+		roots.splice(index, 1)
+		controllers.splice(index, 1)
+		components.splice(index, 1)
+		reset(root)
+		nodeCache.splice(getCellCacheKey(root), 1)
 	}
 
-	var redrawing = false, forcing = false;
-	m.redraw = function(force) {
-		if (redrawing) return;
-		redrawing = true;
-		if (force) forcing = true;
+	var redrawing = false
+	m.redraw = function (force) {
+		if (redrawing) return
+		redrawing = true
+		if (force) forcing = true
+
 		try {
-			//lastRedrawId is a positive number if a second redraw is requested before the next animation frame
-			//lastRedrawID is null if it's the first redraw and not an event handler
+			// lastRedrawId is a positive number if a second redraw is requested
+			// before the next animation frame
+			// lastRedrawID is null if it's the first redraw and not an event
+			// handler
 			if (lastRedrawId && !force) {
-				//when setTimeout: only reschedule redraw if time between now and previous redraw is bigger than a frame, otherwise keep currently scheduled timeout
-				//when rAF: always reschedule redraw
-				if ($requestAnimationFrame === window.requestAnimationFrame || new Date - lastRedrawCallTime > FRAME_BUDGET) {
-					if (lastRedrawId > 0) $cancelAnimationFrame(lastRedrawId);
-					lastRedrawId = $requestAnimationFrame(redraw, FRAME_BUDGET);
+				// when setTimeout: only reschedule redraw if time between now
+				// and previous redraw is bigger than a frame, otherwise keep
+				// currently scheduled timeout
+				// when rAF: always reschedule redraw
+				if ($requestAnimationFrame === global.requestAnimationFrame ||
+						new Date() - lastRedrawCallTime > FRAME_BUDGET) {
+					if (lastRedrawId > 0) $cancelAnimationFrame(lastRedrawId)
+					lastRedrawId = $requestAnimationFrame(redraw, FRAME_BUDGET)
 				}
+			} else {
+				redraw()
+				lastRedrawId = $requestAnimationFrame(function () {
+					lastRedrawId = null
+				}, FRAME_BUDGET)
 			}
-			else {
-				redraw();
-				lastRedrawId = $requestAnimationFrame(function() { lastRedrawId = null; }, FRAME_BUDGET);
-			}
+		} finally {
+			redrawing = forcing = false
 		}
-		finally {
-			redrawing = forcing = false;
-		}
-	};
-	m.redraw.strategy = m.prop();
+	}
+
+	m.redraw.strategy = m.prop()
 	function redraw() {
 		if (computePreRedrawHook) {
-			computePreRedrawHook();
-			computePreRedrawHook = null;
+			computePreRedrawHook()
+			computePreRedrawHook = null
 		}
 		forEach(roots, function (root, i) {
-			var component = components[i];
+			var component = components[i]
 			if (controllers[i]) {
-				var args = [controllers[i]];
-				m.render(root, component.view ? component.view(controllers[i], args) : "");
+				var args = [controllers[i]]
+				m.render(root,
+					component.view ? component.view(controllers[i], args) : "")
 			}
-		});
-		//after rendering within a routed context, we need to scroll back to the top, and fetch the document title for history.pushState
+		})
+		// after rendering within a routed context, we need to scroll back to
+		// the top, and fetch the document title for history.pushState
 		if (computePostRedrawHook) {
-			computePostRedrawHook();
-			computePostRedrawHook = null;
-		}
-		lastRedrawId = null;
-		lastRedrawCallTime = new Date;
-		m.redraw.strategy("diff");
-	}
-
-	var pendingRequests = 0;
-	m.startComputation = function() { pendingRequests++; };
-	m.endComputation = function() {
-		if (pendingRequests > 1) pendingRequests--;
-		else {
-			pendingRequests = 0;
-			m.redraw();
+			computePostRedrawHook()
+			computePostRedrawHook = null
 		}
+		lastRedrawId = null
+		lastRedrawCallTime = new Date()
+		m.redraw.strategy("diff")
 	}
 
 	function endFirstComputation() {
 		if (m.redraw.strategy() === "none") {
-			pendingRequests--;
-			m.redraw.strategy("diff");
+			pendingRequests--
+			m.redraw.strategy("diff")
+		} else {
+			m.endComputation()
 		}
-		else m.endComputation();
 	}
 
-	m.withAttr = function(prop, withAttrCallback, callbackThis) {
-		return function(e) {
-			e = e || event;
-			var currentTarget = e.currentTarget || this;
-			var _this = callbackThis || this;
-			withAttrCallback.call(_this, prop in currentTarget ? currentTarget[prop] : currentTarget.getAttribute(prop));
-		};
-	};
+	m.withAttr = function (prop, withAttrCallback, callbackThis) {
+		return function (e) {
+			e = e || event
+			/* eslint-disable no-invalid-this */
+			var currentTarget = e.currentTarget || this
+			var _this = callbackThis || this
+			/* eslint-enable no-invalid-this */
+			var target = prop in currentTarget ?
+				currentTarget[prop] :
+				currentTarget.getAttribute(prop)
+			withAttrCallback.call(_this, target)
+		}
+	}
 
-	//routing
-	var modes = {pathname: "", hash: "#", search: "?"};
-	var redirect = noop, routeParams, currentRoute, isDefaultRoute = false;
-	m.route = function(root, arg1, arg2, vdom) {
-		//m.route()
-		if (arguments.length === 0) return currentRoute;
-		//m.route(el, defaultRoute, routes)
-		else if (arguments.length === 3 && isString(arg1)) {
-			redirect = function(source) {
-				var path = currentRoute = normalizeRoute(source);
+	// routing
+	var modes = {pathname: "", hash: "#", search: "?"}
+	var redirect = noop
+	var isDefaultRoute = false
+	var routeParams, currentRoute
+
+	m.route = function (root, arg1, arg2, vdom) { // eslint-disable-line
+		// m.route()
+		if (arguments.length === 0) return currentRoute
+		// m.route(el, defaultRoute, routes)
+		if (arguments.length === 3 && isString(arg1)) {
+			redirect = function (source) {
+				var path = currentRoute = normalizeRoute(source)
 				if (!routeByValue(root, arg2, path)) {
-					if (isDefaultRoute) throw new Error("Ensure the default route matches one of the routes defined in m.route");
-					isDefaultRoute = true;
-					m.route(arg1, true);
-					isDefaultRoute = false;
+					if (isDefaultRoute) {
+						throw new Error("Ensure the default route matches " +
+							"one of the routes defined in m.route")
+					}
+
+					isDefaultRoute = true
+					m.route(arg1, true)
+					isDefaultRoute = false
 				}
-			};
-			var listener = m.route.mode === "hash" ? "onhashchange" : "onpopstate";
-			window[listener] = function() {
-				var path = $location[m.route.mode];
-				if (m.route.mode === "pathname") path += $location.search;
-				if (currentRoute !== normalizeRoute(path)) redirect(path);
-			};
+			}
 
-			computePreRedrawHook = setScroll;
-			window[listener]();
+			var listener = m.route.mode === "hash" ?
+				"onhashchange" :
+				"onpopstate"
+
+			global[listener] = function () {
+				var path = $location[m.route.mode]
+				if (m.route.mode === "pathname") path += $location.search
+				if (currentRoute !== normalizeRoute(path)) redirect(path)
+			}
+
+			computePreRedrawHook = setScroll
+			global[listener]()
+
+			return
 		}
-		//config: m.route
-		else if (root.addEventListener || root.attachEvent) {
-			root.href = (m.route.mode !== 'pathname' ? $location.pathname : '') + modes[m.route.mode] + vdom.attrs.href;
+
+		// config: m.route
+		if (root.addEventListener || root.attachEvent) {
+			var base = m.route.mode !== "pathname" ? $location.pathname : ""
+			root.href = base + modes[m.route.mode] + vdom.attrs.href
 			if (root.addEventListener) {
-				root.removeEventListener("click", routeUnobtrusive);
-				root.addEventListener("click", routeUnobtrusive);
+				root.removeEventListener("click", routeUnobtrusive)
+				root.addEventListener("click", routeUnobtrusive)
+			} else {
+				root.detachEvent("onclick", routeUnobtrusive)
+				root.attachEvent("onclick", routeUnobtrusive)
 			}
-			else {
-				root.detachEvent("onclick", routeUnobtrusive);
-				root.attachEvent("onclick", routeUnobtrusive);
-			}
-		}
-		//m.route(route, params, shouldReplaceHistoryEntry)
-		else if (isString(root)) {
-			var oldRoute = currentRoute;
-			currentRoute = root;
-			var args = arg1 || {};
-			var queryIndex = currentRoute.indexOf("?");
-			var params = queryIndex > -1 ? parseQueryString(currentRoute.slice(queryIndex + 1)) : {};
-			for (var i in args) params[i] = args[i];
-			var querystring = buildQueryString(params);
-			var currentPath = queryIndex > -1 ? currentRoute.slice(0, queryIndex) : currentRoute;
-			if (querystring) currentRoute = currentPath + (currentPath.indexOf("?") === -1 ? "?" : "&") + querystring;
 
-			var shouldReplaceHistoryEntry = (arguments.length === 3 ? arg2 : arg1) === true || oldRoute === root;
+			return
+		}
+		// m.route(route, params, shouldReplaceHistoryEntry)
+		if (isString(root)) {
+			var oldRoute = currentRoute
+			currentRoute = root
 
-			if (window.history.pushState) {
-				computePreRedrawHook = setScroll;
-				computePostRedrawHook = function() {
-					window.history[shouldReplaceHistoryEntry ? "replaceState" : "pushState"](null, $document.title, modes[m.route.mode] + currentRoute);
-				};
-				redirect(modes[m.route.mode] + currentRoute);
+			var args = arg1 || {}
+			var queryIndex = currentRoute.indexOf("?")
+			var params
+
+			if (queryIndex > -1) {
+				params = parseQueryString(currentRoute.slice(queryIndex + 1))
+			} else {
+				params = {}
 			}
-			else {
-				$location[m.route.mode] = currentRoute;
-				redirect(modes[m.route.mode] + currentRoute);
+
+			for (var i in args) if (hasOwn.call(args, i)) {
+				params[i] = args[i]
+			}
+
+			var querystring = buildQueryString(params)
+			var currentPath
+
+			if (queryIndex > -1) {
+				currentPath = currentRoute.slice(0, queryIndex)
+			} else {
+				currentPath = currentRoute
+			}
+
+			if (querystring) {
+				currentRoute = currentPath +
+					(currentPath.indexOf("?") === -1 ? "?" : "&") +
+					querystring
+			}
+
+			var replaceHistory =
+				(arguments.length === 3 ? arg2 : arg1) === true ||
+				oldRoute === root
+
+			if (global.history.pushState) {
+				var method = replaceHistory ? "replaceState" : "pushState"
+				computePreRedrawHook = setScroll
+				computePostRedrawHook = function () {
+					global.history[method](null, $document.title,
+						modes[m.route.mode] + currentRoute)
+				}
+				redirect(modes[m.route.mode] + currentRoute)
+			} else {
+				$location[m.route.mode] = currentRoute
+				redirect(modes[m.route.mode] + currentRoute)
 			}
 		}
-	};
-	m.route.param = function(key) {
-		if (!routeParams) throw new Error("You must call m.route(element, defaultRoute, routes) before calling m.route.param()");
-		if( !key ){
-			return routeParams;
-		}
-		return routeParams[key];
-	};
-	m.route.mode = "search";
-	function normalizeRoute(route) {
-		return route.slice(modes[m.route.mode].length);
 	}
-	function routeByValue(root, router, path) {
-		routeParams = {};
 
-		var queryStart = path.indexOf("?");
+	m.route.param = function (key) {
+		if (!routeParams) {
+			throw new Error("You must call m.route(element, defaultRoute, " +
+				"routes) before calling m.route.param()")
+		}
+
+		if (!key) {
+			return routeParams
+		}
+
+		return routeParams[key]
+	}
+
+	m.route.mode = "search"
+
+	function normalizeRoute(route) {
+		return route.slice(modes[m.route.mode].length)
+	}
+
+	function routeByValue(root, router, path) {
+		routeParams = {}
+
+		var queryStart = path.indexOf("?")
 		if (queryStart !== -1) {
-			routeParams = parseQueryString(path.substr(queryStart + 1, path.length));
-			path = path.substr(0, queryStart);
+			routeParams = parseQueryString(
+				path.substr(queryStart + 1, path.length))
+			path = path.substr(0, queryStart)
 		}
 
 		// Get all routes and check if there's
 		// an exact match for the current path
-		var keys = Object.keys(router);
-		var index = keys.indexOf(path);
-		if(index !== -1){
-			m.mount(root, router[keys [index]]);
-			return true;
+		var keys = Object.keys(router)
+		var index = keys.indexOf(path)
+
+		if (index !== -1){
+			m.mount(root, router[keys [index]])
+			return true
 		}
 
-		for (var route in router) {
+		for (var route in router) if (hasOwn.call(router, route)) {
 			if (route === path) {
-				m.mount(root, router[route]);
-				return true;
+				m.mount(root, router[route])
+				return true
 			}
 
-			var matcher = new RegExp("^" + route.replace(/:[^\/]+?\.{3}/g, "(.*?)").replace(/:[^\/]+/g, "([^\\/]+)") + "\/?$");
+			var matcher = new RegExp("^" + route
+				.replace(/:[^\/]+?\.{3}/g, "(.*?)")
+				.replace(/:[^\/]+/g, "([^\\/]+)") + "\/?$")
 
 			if (matcher.test(path)) {
-				path.replace(matcher, function() {
-					var keys = route.match(/:[^\/]+/g) || [];
-					var values = [].slice.call(arguments, 1, -2);
+				/* eslint-disable no-loop-func */
+				path.replace(matcher, function () {
+					var keys = route.match(/:[^\/]+/g) || []
+					var values = [].slice.call(arguments, 1, -2)
 					forEach(keys, function (key, i) {
-						routeParams[key.replace(/:|\./g, "")] = decodeURIComponent(values[i]);
+						routeParams[key.replace(/:|\./g, "")] =
+							decodeURIComponent(values[i])
 					})
-					m.mount(root, router[route]);
-				});
-				return true;
+					m.mount(root, router[route])
+				})
+				/* eslint-enable no-loop-func */
+				return true
 			}
 		}
 	}
+
 	function routeUnobtrusive(e) {
-		e = e || event;
-		if (e.ctrlKey || e.metaKey || e.shiftKey || e.which === 2) return;
+		e = e || event
+		if (e.ctrlKey || e.metaKey || e.shiftKey || e.which === 2) return
 
-		if (e.preventDefault) e.preventDefault();
-		else e.returnValue = false;
+		if (e.preventDefault) {
+			e.preventDefault()
+		} else {
+			e.returnValue = false
+		}
+
+		var currentTarget = e.currentTarget || e.srcElement
+		var args
+
+		if (m.route.mode === "pathname" && currentTarget.search) {
+			args = parseQueryString(currentTarget.search.slice(1))
+		} else {
+			args = {}
+		}
+
+		while (currentTarget && !/a/i.test(currentTarget.nodeName)) {
+			currentTarget = currentTarget.parentNode
+		}
 
-		var currentTarget = e.currentTarget || e.srcElement;
-		var args = m.route.mode === "pathname" && currentTarget.search ? parseQueryString(currentTarget.search.slice(1)) : {};
-		while (currentTarget && currentTarget.nodeName.toUpperCase() !== "A") currentTarget = currentTarget.parentNode;
 		// clear pendingRequests because we want an immediate route change
-		pendingRequests = 0;
-		m.route(currentTarget[m.route.mode].slice(modes[m.route.mode].length), args);
+		pendingRequests = 0
+		m.route(currentTarget[m.route.mode]
+			.slice(modes[m.route.mode].length), args)
 	}
+
 	function setScroll() {
-		if (m.route.mode !== "hash" && $location.hash) $location.hash = $location.hash;
-		else window.scrollTo(0, 0);
+		if (m.route.mode !== "hash" && $location.hash) {
+			$location.hash = $location.hash
+		} else {
+			global.scrollTo(0, 0)
+		}
 	}
+
 	function buildQueryString(object, prefix) {
-		var duplicates = {};
-		var str = [];
-		for (var prop in object) {
-			var key = prefix ? prefix + "[" + prop + "]" : prop;
-			var value = object[prop];
+		var duplicates = {}
+		var str = []
+
+		for (var prop in object) if (hasOwn.call(object, prop)) {
+			var key = prefix ? prefix + "[" + prop + "]" : prop
+			var value = object[prop]
 
 			if (value === null) {
-				str.push(encodeURIComponent(key));
+				str.push(encodeURIComponent(key))
 			} else if (isObject(value)) {
-				str.push(buildQueryString(value, key));
+				str.push(buildQueryString(value, key))
 			} else if (isArray(value)) {
-				var keys = [];
-				duplicates[key] = duplicates[key] || {};
+				var keys = []
+				duplicates[key] = duplicates[key] || {}
+				/* eslint-disable no-loop-func */
 				forEach(value, function (item) {
+					/* eslint-enable no-loop-func */
 					if (!duplicates[key][item]) {
-						duplicates[key][item] = true;
-						keys.push(encodeURIComponent(key) + "=" + encodeURIComponent(item));
+						duplicates[key][item] = true
+						keys.push(encodeURIComponent(key) + "=" +
+							encodeURIComponent(item))
 					}
-				});
-				str.push(keys.join("&"));
+				})
+				str.push(keys.join("&"))
 			} else if (value !== undefined) {
-				str.push(encodeURIComponent(key) + "=" + encodeURIComponent(value));
+				str.push(encodeURIComponent(key) + "=" +
+					encodeURIComponent(value))
 			}
 		}
-		return str.join("&");
+		return str.join("&")
 	}
+
 	function parseQueryString(str) {
-		if (str === "" || str == null) return {};
-		if (str.charAt(0) === "?") str = str.slice(1);
+		if (str === "" || str == null) return {}
+		if (str.charAt(0) === "?") str = str.slice(1)
+
+		var pairs = str.split("&")
+		var params = {}
 
-		var pairs = str.split("&"), params = {};
 		forEach(pairs, function (string) {
-			var pair = string.split("=");
-			var key = decodeURIComponent(pair[0]);
-			var value = pair.length === 2 ? decodeURIComponent(pair[1]) : null;
+			var pair = string.split("=")
+			var key = decodeURIComponent(pair[0])
+			var value = pair.length === 2 ? decodeURIComponent(pair[1]) : null
 			if (params[key] != null) {
-				if (!isArray(params[key])) params[key] = [params[key]];
-				params[key].push(value);
+				if (!isArray(params[key])) params[key] = [params[key]]
+				params[key].push(value)
 			}
-			else params[key] = value;
-		});
+			else params[key] = value
+		})
 
-		return params;
+		return params
 	}
-	m.route.buildQueryString = buildQueryString;
-	m.route.parseQueryString = parseQueryString;
+
+	m.route.buildQueryString = buildQueryString
+	m.route.parseQueryString = parseQueryString
 
 	function reset(root) {
-		var cacheKey = getCellCacheKey(root);
-		clear(root.childNodes, cellCache[cacheKey]);
-		cellCache[cacheKey] = undefined;
+		var cacheKey = getCellCacheKey(root)
+		clear(root.childNodes, cellCache[cacheKey])
+		cellCache[cacheKey] = undefined
 	}
 
 	m.deferred = function () {
-		var deferred = new Deferred();
-		deferred.promise = propify(deferred.promise);
-		return deferred;
-	};
-	function propify(promise, initialValue) {
-		var prop = m.prop(initialValue);
-		promise.then(prop);
-		prop.then = function(resolve, reject) {
-			return propify(promise.then(resolve, reject), initialValue);
-		};
-		prop["catch"] = prop.then.bind(null, null);
-		return prop;
+		var deferred = new Deferred()
+		deferred.promise = propify(deferred.promise)
+		return deferred
 	}
-	//Promiz.mithril.js | Zolmeister | MIT
-	//a modified version of Promiz.js, which does not conform to Promises/A+ for two reasons:
-	//1) `then` callbacks are called synchronously (because setTimeout is too slow, and the setImmediate polyfill is too big
-	//2) throwing subclasses of Error cause the error to be bubbled up instead of triggering rejection (because the spec does not account for the important use case of default browser error handling, i.e. message w/ line number)
-	function Deferred(successCallback, failureCallback) {
-		var RESOLVING = 1, REJECTING = 2, RESOLVED = 3, REJECTED = 4;
-		var self = this, state = 0, promiseValue = 0, next = [];
 
-		self.promise = {};
-
-		self.resolve = function(value) {
-			if (!state) {
-				promiseValue = value;
-				state = RESOLVING;
-
-				fire();
-			}
-			return this;
-		};
-
-		self.reject = function(value) {
-			if (!state) {
-				promiseValue = value;
-				state = REJECTING;
-
-				fire();
-			}
-			return this;
-		};
-
-		self.promise.then = function(successCallback, failureCallback) {
-			var deferred = new Deferred(successCallback, failureCallback)
-			if (state === RESOLVED) {
-				deferred.resolve(promiseValue);
-			}
-			else if (state === REJECTED) {
-				deferred.reject(promiseValue);
-			}
-			else {
-				next.push(deferred);
-			}
-			return deferred.promise
-		};
-
-		function finish(type) {
-			state = type || REJECTED;
-			next.map(function(deferred) {
-				state === RESOLVED ? deferred.resolve(promiseValue) : deferred.reject(promiseValue);
-			});
+	function propify(promise, initialValue) {
+		var prop = m.prop(initialValue)
+		promise.then(prop)
+		prop.then = function (resolve, reject) {
+			return propify(promise.then(resolve, reject), initialValue)
 		}
 
-		function thennable(then, successCallback, failureCallback, notThennableCallback) {
-			if (((promiseValue != null && isObject(promiseValue)) || isFunction(promiseValue)) && isFunction(then)) {
+		prop.catch = prop.then.bind(null, null)
+		return prop
+	}
+	// Promiz.mithril.js | Zolmeister | MIT
+	// a modified version of Promiz.js, which does not conform to Promises/A+
+	// for two reasons:
+	//
+	// 1) `then` callbacks are called synchronously (because setTimeout is too
+	//    slow, and the setImmediate polyfill is too big
+	//
+	// 2) throwing subclasses of Error cause the error to be bubbled up instead
+	//    of triggering rejection (because the spec does not account for the
+	//    important use case of default browser error handling, i.e. message w/
+	//    line number)
+
+	var RESOLVING = 1
+	var REJECTING = 2
+	var RESOLVED = 3
+	var REJECTED = 4
+
+	function Deferred(onSuccess, onFailure) {
+		var self = this
+		var state = 0
+		var promiseValue = 0
+		var next = []
+
+		self.promise = {}
+
+		self.resolve = function (value) {
+			if (!state) {
+				promiseValue = value
+				state = RESOLVING
+
+				fire()
+			}
+
+			return self
+		}
+
+		self.reject = function (value) {
+			if (!state) {
+				promiseValue = value
+				state = REJECTING
+
+				fire()
+			}
+
+			return self
+		}
+
+		self.promise.then = function (onSuccess, onFailure) {
+			var deferred = new Deferred(onSuccess, onFailure)
+
+			if (state === RESOLVED) {
+				deferred.resolve(promiseValue)
+			} else if (state === REJECTED) {
+				deferred.reject(promiseValue)
+			} else {
+				next.push(deferred)
+			}
+
+			return deferred.promise
+		}
+
+		function finish(type) {
+			state = type || REJECTED
+			next.map(function (deferred) {
+				if (state === RESOLVED) {
+					deferred.resolve(promiseValue)
+				} else {
+					deferred.reject(promiseValue)
+				}
+			})
+		}
+
+		function thennable(then, success, failure, notThennable) {
+			if (((promiseValue != null && isObject(promiseValue)) ||
+					isFunction(promiseValue)) && isFunction(then)) {
 				try {
 					// count protects against abuse calls from spec checker
-					var count = 0;
-					then.call(promiseValue, function(value) {
-						if (count++) return;
-						promiseValue = value;
-						successCallback();
+					var count = 0
+					then.call(promiseValue, function (value) {
+						if (count++) return
+						promiseValue = value
+						success()
 					}, function (value) {
-						if (count++) return;
-						promiseValue = value;
-						failureCallback();
-					});
-				}
-				catch (e) {
-					m.deferred.onerror(e);
-					promiseValue = e;
-					failureCallback();
+						if (count++) return
+						promiseValue = value
+						failure()
+					})
+				} catch (e) {
+					m.deferred.onerror(e)
+					promiseValue = e
+					failure()
 				}
 			} else {
-				notThennableCallback();
+				notThennable()
 			}
 		}
 
 		function fire() {
 			// check if it's a thenable
-			var then;
+			var then
 			try {
-				then = promiseValue && promiseValue.then;
-			}
-			catch (e) {
-				m.deferred.onerror(e);
-				promiseValue = e;
-				state = REJECTING;
-				return fire();
+				then = promiseValue && promiseValue.then
+			} catch (e) {
+				m.deferred.onerror(e)
+				promiseValue = e
+				state = REJECTING
+				return fire()
 			}
 
 			if (state === REJECTING) {
@@ -1794,226 +2483,262 @@ var m = (function app(window, undefined) {
 				fire()
 			}, function () {
 				try {
-					if (state === RESOLVING && isFunction(successCallback)) {
-						promiseValue = successCallback(promiseValue);
+					if (state === RESOLVING && isFunction(onSuccess)) {
+						promiseValue = onSuccess(promiseValue)
+					} else if (state === REJECTING && isFunction(onFailure)) {
+						promiseValue = onFailure(promiseValue)
+						state = RESOLVING
 					}
-					else if (state === REJECTING && isFunction(failureCallback)) {
-						promiseValue = failureCallback(promiseValue);
-						state = RESOLVING;
-					}
-				}
-				catch (e) {
-					m.deferred.onerror(e);
-					promiseValue = e;
-					return finish();
+				} catch (e) {
+					m.deferred.onerror(e)
+					promiseValue = e
+					return finish()
 				}
 
 				if (promiseValue === self) {
-					promiseValue = TypeError();
-					finish();
+					promiseValue = TypeError()
+					finish()
 				} else {
 					thennable(then, function () {
-						finish(RESOLVED);
+						finish(RESOLVED)
 					}, finish, function () {
-						finish(state === RESOLVING && RESOLVED);
-					});
+						finish(state === RESOLVING && RESOLVED)
+					})
 				}
-			});
+			})
 		}
 	}
-	m.deferred.onerror = function(e) {
-		if (type.call(e) === "[object Error]" && !e.constructor.toString().match(/ Error/)) {
-			pendingRequests = 0;
-			throw e;
-		}
-	};
 
-	m.sync = function(args) {
-		var method = "resolve";
+	m.deferred.onerror = function (e) {
+		if (type.call(e) === "[object Error]" &&
+				!e.constructor.toString().match(/ Error/)) {
+			pendingRequests = 0
+			throw e
+		}
+	}
+
+	m.sync = function (args) {
+		var deferred = m.deferred()
+		var outstanding = args.length
+		var results = new Array(outstanding)
+		var method = "resolve"
 
 		function synchronizer(pos, resolved) {
-			return function(value) {
-				results[pos] = value;
-				if (!resolved) method = "reject";
+			return function (value) {
+				results[pos] = value
+				if (!resolved) method = "reject"
 				if (--outstanding === 0) {
-					deferred.promise(results);
-					deferred[method](results);
+					deferred.promise(results)
+					deferred[method](results)
 				}
-				return value;
-			};
+				return value
+			}
 		}
 
-		var deferred = m.deferred();
-		var outstanding = args.length;
-		var results = new Array(outstanding);
 		if (args.length > 0) {
 			forEach(args, function (arg, i) {
-				arg.then(synchronizer(i, true), synchronizer(i, false));
-			});
+				arg.then(synchronizer(i, true), synchronizer(i, false))
+			})
+		} else {
+			deferred.resolve([])
 		}
-		else deferred.resolve([]);
 
-		return deferred.promise;
-	};
-	function identity(value) { return value; }
+		return deferred.promise
+	}
+
+	function identity(value) { return value }
+
+	function handleJsonp(options) {
+		var callbackKey = "mithril_callback_" +
+			new Date().getTime() + "_" +
+			(Math.round(Math.random() * 1e16)).toString(36)
+
+		var script = $document.createElement("script")
+
+		global[callbackKey] = function (resp) {
+			script.parentNode.removeChild(script)
+			options.onload({
+				type: "load",
+				target: {
+					responseText: resp
+				}
+			})
+			global[callbackKey] = undefined
+		}
+
+		script.onerror = function () {
+			script.parentNode.removeChild(script)
+
+			options.onerror({
+				type: "error",
+				target: {
+					status: 500,
+					responseText: JSON.stringify({
+						error: "Error making jsonp request"
+					})
+				}
+			})
+			global[callbackKey] = undefined
+
+			return false
+		}
+
+		script.onload = function () {
+			return false
+		}
+
+		script.src = options.url +
+			(options.url.indexOf("?") > 0 ? "&" : "?") +
+			(options.callbackKey ? options.callbackKey : "callback") +
+			"=" + callbackKey +
+			"&" + buildQueryString(options.data || {})
+
+		$document.body.appendChild(script)
+	}
+
+	function createXhr(options) {
+		var xhr = new global.XMLHttpRequest()
+		xhr.open(options.method, options.url, true, options.user,
+			options.password)
+
+		xhr.onreadystatechange = function () {
+			if (xhr.readyState === 4) {
+				if (xhr.status >= 200 && xhr.status < 300) {
+					options.onload({type: "load", target: xhr})
+				} else {
+					options.onerror({type: "error", target: xhr})
+				}
+			}
+		}
+
+		if (options.serialize === JSON.stringify &&
+				options.data &&
+				options.method !== "GET") {
+			xhr.setRequestHeader("Content-Type",
+				"application/json; charset=utf-8")
+		}
+
+		if (options.deserialize === JSON.parse) {
+			xhr.setRequestHeader("Accept", "application/json, text/*")
+		}
+
+		if (isFunction(options.config)) {
+			var maybeXhr = options.config(xhr, options)
+			if (maybeXhr != null) xhr = maybeXhr
+		}
+
+		var data = options.method === "GET" || !options.data ? "" : options.data
+
+		if (data && !isString(data) && data.constructor !== global.FormData) {
+			throw new Error("Request data should be either be a string or " +
+				"FormData. Check the `serialize` option in `m.request`")
+		}
+
+		xhr.send(data)
+		return xhr
+	}
 
 	function ajax(options) {
 		if (options.dataType && options.dataType.toLowerCase() === "jsonp") {
-			var callbackKey = "mithril_callback_" + new Date().getTime() + "_" + (Math.round(Math.random() * 1e16)).toString(36)
-			var script = $document.createElement("script");
-
-			window[callbackKey] = function(resp) {
-				script.parentNode.removeChild(script);
-				options.onload({
-					type: "load",
-					target: {
-						responseText: resp
-					}
-				});
-				window[callbackKey] = undefined;
-			};
-
-			script.onerror = function() {
-				script.parentNode.removeChild(script);
-
-				options.onerror({
-					type: "error",
-					target: {
-						status: 500,
-						responseText: JSON.stringify({
-							error: "Error making jsonp request"
-						})
-					}
-				});
-				window[callbackKey] = undefined;
-
-				return false;
-			}
-
-			script.onload = function() {
-				return false;
-			};
-
-			script.src = options.url
-				+ (options.url.indexOf("?") > 0 ? "&" : "?")
-				+ (options.callbackKey ? options.callbackKey : "callback")
-				+ "=" + callbackKey
-				+ "&" + buildQueryString(options.data || {});
-			$document.body.appendChild(script);
-		}
-		else {
-			var xhr = new window.XMLHttpRequest();
-			xhr.open(options.method, options.url, true, options.user, options.password);
-			xhr.onreadystatechange = function() {
-				if (xhr.readyState === 4) {
-					if (xhr.status >= 200 && xhr.status < 300) options.onload({type: "load", target: xhr});
-					else options.onerror({type: "error", target: xhr});
-				}
-			};
-			if (options.serialize === JSON.stringify && options.data && options.method !== "GET") {
-				xhr.setRequestHeader("Content-Type", "application/json; charset=utf-8");
-			}
-			if (options.deserialize === JSON.parse) {
-				xhr.setRequestHeader("Accept", "application/json, text/*");
-			}
-			if (isFunction(options.config)) {
-				var maybeXhr = options.config(xhr, options);
-				if (maybeXhr != null) xhr = maybeXhr;
-			}
-
-			var data = options.method === "GET" || !options.data ? "" : options.data;
-			if (data && (!isString(data) && data.constructor !== window.FormData)) {
-				throw new Error("Request data should be either be a string or FormData. Check the `serialize` option in `m.request`");
-			}
-			xhr.send(data);
-			return xhr;
+			return handleJsonp(options)
+		} else {
+			return createXhr(options)
 		}
 	}
 
-	function bindData(xhrOptions, data, serialize) {
-		if (xhrOptions.method === "GET" && xhrOptions.dataType !== "jsonp") {
-			var prefix = xhrOptions.url.indexOf("?") < 0 ? "?" : "&";
-			var querystring = buildQueryString(data);
-			xhrOptions.url = xhrOptions.url + (querystring ? prefix + querystring : "");
+	function bindData(options, data, serialize) {
+		if (options.method === "GET" && options.dataType !== "jsonp") {
+			var prefix = options.url.indexOf("?") < 0 ? "?" : "&"
+			var querystring = buildQueryString(data)
+			options.url += (querystring ? prefix + querystring : "")
+		} else {
+			options.data = serialize(data)
 		}
-		else xhrOptions.data = serialize(data);
-		return xhrOptions;
 	}
 
 	function parameterizeUrl(url, data) {
-		var tokens = url.match(/:[a-z]\w+/gi);
+		var tokens = url.match(/:[a-z]\w+/gi)
+
 		if (tokens && data) {
 			forEach(tokens, function (token) {
-				var key = token.slice(1);
-				url = url.replace(token, data[key]);
-				delete data[key];
-			});
+				var key = token.slice(1)
+				url = url.replace(token, data[key])
+				delete data[key]
+			})
 		}
-		return url;
+
+		return url
 	}
 
-	m.request = function(xhrOptions) {
-		if (xhrOptions.background !== true) m.startComputation();
-		var deferred = new Deferred();
-		var isJSONP = xhrOptions.dataType && xhrOptions.dataType.toLowerCase() === "jsonp"
-		var serialize = xhrOptions.serialize = isJSONP ? identity : xhrOptions.serialize || JSON.stringify;
-		var deserialize = xhrOptions.deserialize = isJSONP ? identity : xhrOptions.deserialize || JSON.parse;
-		var extract = isJSONP ? function(jsonp) { return jsonp.responseText } : xhrOptions.extract || function(xhr) {
-			if (xhr.responseText.length === 0 && deserialize === JSON.parse) {
-				return null
-			} else {
-				return xhr.responseText
-			}
-		};
-		xhrOptions.method = (xhrOptions.method || "GET").toUpperCase();
-		xhrOptions.url = parameterizeUrl(xhrOptions.url, xhrOptions.data);
-		xhrOptions = bindData(xhrOptions, xhrOptions.data, serialize);
-		xhrOptions.onload = xhrOptions.onerror = function(e) {
-			try {
-				e = e || event;
-				var unwrap = (e.type === "load" ? xhrOptions.unwrapSuccess : xhrOptions.unwrapError) || identity;
-				var response = unwrap(deserialize(extract(e.target, xhrOptions)), e.target);
-				if (e.type === "load") {
-					if (isArray(response) && xhrOptions.type) {
-						forEach(response, function (res, i) {
-							response[i] = new xhrOptions.type(res);
-						});
-					} else if (xhrOptions.type) {
-						response = new xhrOptions.type(response);
-					}
-					deferred.resolve(response)
-				} else {
-					deferred.reject(response)
-				}
+	m.request = function (options) {
+		if (options.background !== true) m.startComputation()
+		var deferred = new Deferred()
+		var isJSONP = options.dataType &&
+			options.dataType.toLowerCase() === "jsonp"
 
-				deferred[e.type === "load" ? "resolve" : "reject"](response);
-			}
-			catch (e) {
-				deferred.reject(e);
-			}
-			finally {
-				if (xhrOptions.background !== true) m.endComputation()
+		var serialize, deserialize, extract
+
+		if (isJSONP) {
+			serialize = options.serialize =
+			deserialize = options.deserialize = identity
+
+			extract = function (jsonp) { return jsonp.responseText }
+		} else {
+			serialize = options.serialize = options.serialize || JSON.stringify
+
+			deserialize = options.deserialize =
+				options.deserialize || JSON.parse
+			extract = options.extract || function (xhr) {
+				if (xhr.responseText.length || deserialize !== JSON.parse) {
+					return xhr.responseText
+				} else {
+					return null
+				}
 			}
 		}
 
-		ajax(xhrOptions);
-		deferred.promise = propify(deferred.promise, xhrOptions.initialValue);
-		return deferred.promise;
-	};
+		options.method = (options.method || "GET").toUpperCase()
+		options.url = parameterizeUrl(options.url, options.data)
+		bindData(options, options.data, serialize)
+		options.onload = options.onerror = function (ev) {
+			try {
+				ev = ev || event
+				var response = deserialize(extract(ev.target, options))
+				if (ev.type === "load") {
+					if (options.unwrapSuccess) {
+						response = options.unwrapSuccess(response, ev.target)
+					}
 
-	//testing API
-	m.deps = function(mock) {
-		initialize(window = mock || window);
-		return window;
-	};
-	//for internal testing only, do not use `m.deps.factory`
-	m.deps.factory = app;
+					if (isArray(response) && options.type) {
+						forEach(response, function (res, i) {
+							response[i] = new options.type(res)
+						})
+					} else if (options.type) {
+						response = new options.type(response)
+					}
 
-	return m;
-})(typeof window !== "undefined" ? window : {});
+					deferred.resolve(response)
+				} else {
+					if (options.unwrapError) {
+						response = options.unwrapError(response, ev.target)
+					}
 
-if (typeof module === "object" && module != null && module.exports) module.exports = m;
-else if (typeof define === "function" && define.amd) define(function() { return m });
+					deferred.reject(response)
+				}
+			} catch (e) {
+				deferred.reject(e)
+			} finally {
+				if (options.background !== true) m.endComputation()
+			}
+		}
+
+		ajax(options)
+		deferred.promise = propify(deferred.promise, options.initialValue)
+		return deferred.promise
+	}
+
+	return m
+})
 ;
 ( function _package( factory ){
 	if( typeof define === 'function' && define.amd ){