Update:Remove oldSeries model

server/Database.js

@@ -442,11 +442,6 @@ class Database {
     await this.models.feed.removeById(feedId)
   }
 
-  updateSeries(oldSeries) {
-    if (!this.sequelize) return false
-    return this.models.series.updateFromOld(oldSeries)
-  }
-
   async createBulkBookAuthors(bookAuthors) {
     if (!this.sequelize) return false
     await this.models.bookAuthor.bulkCreate(bookAuthors)

server/controllers/SeriesController.js

@@ -59,19 +59,28 @@ class SeriesController {
   }
 
   /**
-   * TODO: Update to use new model
+   * TODO: Currently unused in the client, should check for duplicate name
    *
    * @param {SeriesControllerRequest} req
    * @param {Response} res
    */
   async update(req, res) {
-    const oldSeries = req.series.getOldSeries()
-    const hasUpdated = oldSeries.update(req.body)
-    if (hasUpdated) {
-      await Database.updateSeries(oldSeries)
-      SocketAuthority.emitter('series_updated', oldSeries.toJSON())
+    const keysToUpdate = ['name', 'description']
+    const payload = {}
+    for (const key of keysToUpdate) {
+      if (req.body[key] !== undefined && typeof req.body[key] === 'string') {
+        payload[key] = req.body[key]
+      }
+    }
+    if (!Object.keys(payload).length) {
+      return res.status(400).send('No valid fields to update')
+    }
+    req.series.set(payload)
+    if (req.series.changed()) {
+      await req.series.save()
+      SocketAuthority.emitter('series_updated', req.series.toOldJSON())
     }
-    res.json(oldSeries.toJSON())
+    res.json(req.series.toOldJSON())
   }
 
   /**

server/models/Series.js

@@ -1,6 +1,5 @@
 const { DataTypes, Model, where, fn, col } = require('sequelize')
 
-const oldSeries = require('../objects/entities/Series')
 const { getTitlePrefixAtEnd } = require('../utils/index')
 
 class Series extends Model {
@@ -23,36 +22,6 @@ class Series extends Model {
     this.updatedAt
   }
 
-  getOldSeries() {
-    return new oldSeries({
-      id: this.id,
-      name: this.name,
-      description: this.description,
-      libraryId: this.libraryId,
-      addedAt: this.createdAt.valueOf(),
-      updatedAt: this.updatedAt.valueOf()
-    })
-  }
-
-  static updateFromOld(oldSeries) {
-    const series = this.getFromOld(oldSeries)
-    return this.update(series, {
-      where: {
-        id: series.id
-      }
-    })
-  }
-
-  static getFromOld(oldSeries) {
-    return {
-      id: oldSeries.id,
-      name: oldSeries.name,
-      nameIgnorePrefix: oldSeries.nameIgnorePrefix,
-      description: oldSeries.description,
-      libraryId: oldSeries.libraryId
-    }
-  }
-
   /**
    * Check if series exists
    * @param {string} seriesId

server/utils/queries/libraryFilters.js

@@ -196,7 +196,7 @@ module.exports = {
    * @param {import('../../models/User')} user
    * @param {string[]} include
    * @param {number} limit
-   * @returns {{ series:import('../../objects/entities/Series')[], count:number}}
+   * @returns {{ series:any[], count:number}}
    */
   async getSeriesMostRecentlyAdded(library, user, include, limit) {
     if (!library.isBook) return { series: [], count: 0 }