Ignore filename case in crawl-local

The array ['banana', 'Rainbows!!', 'Abstract', 'kangaroo']..
..would previously be sorted as ['Abstract', 'Rainbows!!', 'banana',
'kangaroo']..
..but that was clearly wrong; it's now sorted as ['Abstract', 'banana',
'kangaroo', 'Rainbows!!'].

2 files changed, 10 insertions, 1 deletions

diff --git a/src/crawl-local.js b/src/crawl-local.js
index 1baebc3..91554af 100755
--- a/src/crawl-local.js
+++ b/src/crawl-local.js
@@ -11,6 +11,12 @@ const { promisify } = require('util')
 const readDir = promisify(fs.readdir)
 const stat = promisify(fs.stat)
 
+function sortIgnoreCase(sortFunction) {
+  return function(a, b) {
+    return sortFunction(a.toLowerCase(), b.toLowerCase())
+  }
+}
+
 function crawl(dirPath, extensions = [
   // This list isn't very extensive, and can be customized via the
   // --extensions (or --exts, -e) option.
@@ -18,7 +24,7 @@ function crawl(dirPath, extensions = [
   'wav', 'mp3', 'mp4', 'm4a', 'aac'
 ]) {
   return readDir(dirPath).then(items => {
-    items.sort(naturalSort())
+    items.sort(sortIgnoreCase(naturalSort()))
 
     return Promise.all(items.map(item => {
       const itemPath = path.join(dirPath, item)
diff --git a/todo.txt b/todo.txt
index 907c9e2..6395146 100644
--- a/todo.txt
+++ b/todo.txt
@@ -348,3 +348,6 @@ TODO: Rename pickers2.js to pickers.js, and get rid of the old pickers.js.
 TODO: Some way to control how verbose http-music is.. most people probably
       don't care about 'Indexing (flattening)', but it is handy for those on
       slow computers.
+
+TODO: Make the natural sort in crawl-local ignore capitalization case.
+      (Done!)