fix(getTarget): do not return null; throw (#623)

TheSharpieOne · web-flow · commit e1d3b5f538bd · 2017-10-06T07:40:58.000-04:00
If it returns as it did does, it would be able to return null as it would immediately return and not go into the null check with throws.
diff --git a/src/utils.js b/src/utils.js
@@ -116,9 +116,9 @@ export function getTarget(target) {
   }
 
   if (typeof target === 'string' && document) {
-    const selection = document.querySelector(target);
+    let selection = document.querySelector(target);
     if (selection === null) {
-      return document.querySelector(`#${target}`);
+      selection = document.querySelector(`#${target}`);
     }
     if (selection === null) {
       throw new Error(`The target '${target}' could not be identified in the dom, tip: check spelling`);

Original file line number	Diff line number	Diff line change
`@@ -116,9 +116,9 @@ export function getTarget(target) {`
`116`	`116`	`}`
`117`	`117`
`118`	`118`	`if (typeof target === 'string' && document) {`
`119`		`- const selection = document.querySelector(target);`
	`119`	`+ let selection = document.querySelector(target);`
`120`	`120`	`if (selection === null) {`
`121`		- return document.querySelector(`#${target}`);
	`121`	+ selection = document.querySelector(`#${target}`);
`122`	`122`	`}`
`123`	`123`	`if (selection === null) {`
`124`	`124`	throw new Error(`The target '${target}' could not be identified in the dom, tip: check spelling`);