Update:Replace library scan toast with task manager #1279

server/managers/PodcastManager.js

@@ -18,7 +18,6 @@ const LibraryFile = require('../objects/files/LibraryFile')
 const PodcastEpisodeDownload = require('../objects/PodcastEpisodeDownload')
 const PodcastEpisode = require('../objects/entities/PodcastEpisode')
 const AudioFile = require('../objects/files/AudioFile')
-const Task = require("../objects/Task")
 
 class PodcastManager {
   constructor(watcher, notificationManager) {
@@ -70,14 +69,12 @@ class PodcastManager {
       return
     }
 
-    const task = new Task()
     const taskDescription = `Downloading episode "${podcastEpisodeDownload.podcastEpisode.title}".`
     const taskData = {
       libraryId: podcastEpisodeDownload.libraryId,
       libraryItemId: podcastEpisodeDownload.libraryItemId,
     }
-    task.setData('download-podcast-episode', 'Downloading Episode', taskDescription, false, taskData)
-    TaskManager.addTask(task)
+    const task = TaskManager.createAndAddTask('download-podcast-episode', 'Downloading Episode', taskDescription, false, taskData)
 
     SocketAuthority.emitter('episode_download_started', podcastEpisodeDownload.toJSONForClient())
     this.currentDownload = podcastEpisodeDownload

server/managers/TaskManager.js

@@ -28,5 +28,21 @@ class TaskManager {
       SocketAuthority.emitter('task_finished', task.toJSON())
     }
   }
+
+  /**
+   * Create new task and add
+   * 
+   * @param {string} action 
+   * @param {string} title 
+   * @param {string} description 
+   * @param {boolean} showSuccess 
+   * @param {Object} [data] 
+   */
+  createAndAddTask(action, title, description, showSuccess, data = {}) {
+    const task = new Task()
+    task.setData(action, title, description, showSuccess, data)
+    this.addTask(task)
+    return task
+  }
 }
 module.exports = new TaskManager()

server/scanner/LibraryScan.js

@@ -6,7 +6,7 @@ const date = require('../libs/dateAndTime')
 const Logger = require('../Logger')
 const Library = require('../objects/Library')
 const { LogLevel } = require('../utils/constants')
-const { secondsToTimestamp } = require('../utils/index')
+const { secondsToTimestamp, elapsedPretty } = require('../utils/index')
 
 class LibraryScan {
   constructor() {
@@ -67,6 +67,15 @@ class LibraryScan {
   get logFilename() {
     return date.format(new Date(), 'YYYY-MM-DD') + '_' + this.id + '.txt'
   }
+  get scanResultsString() {
+    if (this.error) return this.error
+    const strs = []
+    if (this.resultsAdded) strs.push(`${this.resultsAdded} added`)
+    if (this.resultsUpdated) strs.push(`${this.resultsUpdated} updated`)
+    if (this.resultsMissing) strs.push(`${this.resultsMissing} missing`)
+    if (!strs.length) return `Everything was up to date (${elapsedPretty(this.elapsed / 1000)})`
+    return strs.join(', ') + ` (${elapsedPretty(this.elapsed / 1000)})`
+  }
 
   toJSON() {
     return {

server/scanner/LibraryScanner.js

@@ -9,6 +9,7 @@ const fileUtils = require('../utils/fileUtils')
 const scanUtils = require('../utils/scandir')
 const { LogLevel, ScanResult } = require('../utils/constants')
 const libraryFilters = require('../utils/queries/libraryFilters')
+const TaskManager = require('../managers/TaskManager')
 const LibraryItemScanner = require('./LibraryItemScanner')
 const LibraryScan = require('./LibraryScan')
 const LibraryItemScanData = require('./LibraryItemScanData')
@@ -68,7 +69,12 @@ class LibraryScanner {
     libraryScan.verbose = true
     this.librariesScanning.push(libraryScan.getScanEmitData)
 
-    SocketAuthority.emitter('scan_start', libraryScan.getScanEmitData)
+    const taskData = {
+      libraryId: library.id,
+      libraryName: library.name,
+      libraryMediaType: library.mediaType
+    }
+    const task = TaskManager.createAndAddTask('library-scan', `Scanning "${library.name}" library`, null, true, taskData)
 
     Logger.info(`[LibraryScanner] Starting${forceRescan ? ' (forced)' : ''} library scan ${libraryScan.id} for ${libraryScan.libraryName}`)
 
@@ -85,9 +91,11 @@ class LibraryScanner {
     this.librariesScanning = this.librariesScanning.filter(ls => ls.id !== library.id)
 
     if (canceled && !libraryScan.totalResults) {
+      task.setFinished('Scan canceled')
+      TaskManager.taskFinished(task)
+
       const emitData = libraryScan.getScanEmitData
       emitData.results = null
-      SocketAuthority.emitter('scan_complete', emitData)
       return
     }
 
@@ -98,7 +106,8 @@ class LibraryScanner {
     }
     await Database.libraryModel.updateFromOld(library)
 
-    SocketAuthority.emitter('scan_complete', libraryScan.getScanEmitData)
+    task.setFinished(libraryScan.scanResultsString)
+    TaskManager.taskFinished(task)
 
     if (libraryScan.totalResults) {
       libraryScan.saveLog()

server/utils/index.js

@@ -65,6 +65,9 @@ module.exports.getId = (prepend = '') => {
 }
 
 function elapsedPretty(seconds) {
+  if (seconds > 0 && seconds < 1) {
+    return `${Math.floor(seconds * 1000)} ms`
+  }
   if (seconds < 60) {
     return `${Math.floor(seconds)} sec`
   }