fix: only run sortArray on string[] (#136)

keithamus · Jan 9, 2020 · 1ce4d77 · 1ce4d77
1 parent 3b523ff
commit 1ce4d77
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/index.js b/index.js
@@ -7,8 +7,10 @@ const hasOwnProperty = (object, property) =>
   Object.prototype.hasOwnProperty.call(object, property)
 const pipe = fns => x => fns.reduce((result, fn) => fn(result), x)
 const onArray = fn => x => (Array.isArray(x) ? fn(x) : x)
-const uniq = onArray(xs => xs.filter((x, i) => i === xs.indexOf(x)))
-const sortArray = onArray(array => [...array].sort())
+const onStringArray = fn => x =>
+  Array.isArray(x) && x.every(item => typeof item === 'string') ? fn(x) : x
+const uniq = onStringArray(xs => xs.filter((x, i) => i === xs.indexOf(x)))
+const sortArray = onStringArray(array => [...array].sort())
 const uniqAndSortArray = pipe([uniq, sortArray])
 const isPlainObject = x =>
   x && Object.prototype.toString.call(x) === '[object Object]'

diff --git a/package-lock.json b/package-lock.json