Improve track order detection, allow for excluding audio files from tracklist

server/ApiController.js

@@ -182,9 +182,9 @@ class ApiController {
   async updateAudiobookTracks(req, res) {
     var audiobook = this.db.audiobooks.find(a => a.id === req.params.id)
     if (!audiobook) return res.sendStatus(404)
-    var files = req.body.files
+    var orderedFileData = req.body.orderedFileData
     Logger.info(`Updating audiobook tracks called ${audiobook.id}`)
-    audiobook.updateAudioTracks(files)
+    audiobook.updateAudioTracks(orderedFileData)
     await this.db.updateAudiobook(audiobook)
     this.emitter('audiobook_updated', audiobook.toJSONMinified())
     res.json(audiobook.toJSON())

server/Watcher.js

@@ -25,7 +25,8 @@ class FolderWatcher extends EventEmitter {
         .on('add', (path) => {
           this.onNewFile(path)
         }).on('change', (path) => {
-          this.onFileUpdated(path)
+          // This is triggered from metadata changes, not what we want
+          // this.onFileUpdated(path)
         }).on('unlink', path => {
           this.onFileRemoved(path)
         }).on('rename', (path, pathNext) => {

server/objects/AudioFile.js

@@ -29,6 +29,7 @@ class AudioFile {
 
     this.manuallyVerified = false
     this.invalid = false
+    this.exclude = false
     this.error = null
 
     if (data) {
@@ -49,6 +50,7 @@ class AudioFile {
       trackNumFromFilename: this.trackNumFromFilename,
       manuallyVerified: !!this.manuallyVerified,
       invalid: !!this.invalid,
+      exclude: !!this.exclude,
       error: this.error || null,
       format: this.format,
       duration: this.duration,
@@ -76,6 +78,7 @@ class AudioFile {
     this.addedAt = data.addedAt
     this.manuallyVerified = !!data.manuallyVerified
     this.invalid = !!data.invalid
+    this.exclude = !!data.exclude
     this.error = data.error || null
 
     this.trackNumFromMeta = data.trackNumFromMeta || null
@@ -112,6 +115,7 @@ class AudioFile {
 
     this.manuallyVerified = !!data.manuallyVerified
     this.invalid = !!data.invalid
+    this.exclude = !!data.exclude
     this.error = data.error || null
 
     this.format = data.format
@@ -131,6 +135,10 @@ class AudioFile {
     this.tagTrack = data.file_tag_track || null
   }
 
+  clone() {
+    return new AudioFile(this.toJSON())
+  }
+
   syncFile(newFile) {
     var hasUpdates = false
     var keysToSync = ['path', 'fullPath', 'ext', 'filename']

server/objects/Audiobook.js

@@ -57,14 +57,14 @@ class Audiobook {
     return this.book ? this.book.title : 'No Title'
   }
 
-  get cover() {
-    return this.book ? this.book.cover : ''
-  }
-
   get author() {
     return this.book ? this.book.author : 'Unknown'
   }
 
+  get cover() {
+    return this.book ? this.book.cover : ''
+  }
+
   get authorLF() {
     return this.book ? this.book.authorLF : null
   }
@@ -273,19 +273,32 @@ class Audiobook {
     return hasUpdates
   }
 
-  updateAudioTracks(files) {
+  updateAudioTracks(orderedFileData) {
     var index = 1
-    this.audioFiles = files.map((file) => {
-      file.manuallyVerified = true
-      file.invalid = false
-      file.error = null
-      file.index = index++
-      return new AudioFile(file)
+    this.audioFiles = orderedFileData.map((fileData) => {
+      var audioFile = this.audioFiles.find(af => af.ino === fileData.ino)
+      audioFile.manuallyVerified = true
+      audioFile.invalid = false
+      audioFile.error = null
+      if (fileData.exclude !== undefined) {
+        audioFile.exclude = !!fileData.exclude
+      }
+      if (audioFile.exclude) {
+        audioFile.index = -1
+      } else {
+        audioFile.index = index++
+      }
+      return audioFile
     })
+
+    this.audioFiles.sort((a, b) => a.index - b.index)
+
     this.tracks = []
     this.missingParts = []
     this.audioFiles.forEach((file) => {
-      this.addTrack(file)
+      if (!file.exclude) {
+        this.addTrack(file)
+      }
     })
     this.lastUpdate = Date.now()
   }

server/utils/audioFileScanner.js

@@ -66,8 +66,15 @@ function getTrackNumberFromMeta(scanData) {
   return !isNaN(scanData.trackNumber) && scanData.trackNumber !== null ? Number(scanData.trackNumber) : null
 }
 
-function getTrackNumberFromFilename(filename) {
+function getTrackNumberFromFilename(title, author, series, publishYear, filename) {
   var partbasename = Path.basename(filename, Path.extname(filename))
+
+  // Remove title, author, series, and publishYear from filename if there
+  if (title) partbasename = partbasename.replace(title, '')
+  if (author) partbasename = partbasename.replace(author, '')
+  if (series) partbasename = partbasename.replace(series, '')
+  if (publishYear) partbasename = partbasename.replace(publishYear)
+
   var numbersinpath = partbasename.match(/\d+/g)
   if (!numbersinpath) return null
 
@@ -92,7 +99,8 @@ async function scanAudioFiles(audiobook, newAudioFiles) {
     }
 
     var trackNumFromMeta = getTrackNumberFromMeta(scanData)
-    var trackNumFromFilename = getTrackNumberFromFilename(audioFile.filename)
+    var book = audiobook.book || {}
+    var trackNumFromFilename = getTrackNumberFromFilename(book.title, book.author, book.series, book.publishYear, audioFile.filename)
 
     var audioFileObj = {
       ino: audioFile.ino,