diff --git a/server/scanner/BookScanner.js b/server/scanner/BookScanner.js
index 48e8529ad..6c93dddfd 100644
--- a/server/scanner/BookScanner.js
+++ b/server/scanner/BookScanner.js
@@ -217,7 +217,8 @@ class BookScanner {
} else if (key === 'series') {
// Check for series added
for (const seriesObj of bookMetadata.series) {
- if (!media.series.some(se => se.name === seriesObj.name)) {
+ const existingBookSeries = media.series.find(se => se.name === seriesObj.name)
+ if (!existingBookSeries) {
const existingSeries = Database.libraryFilterData[libraryItemData.libraryId].series.find(se => se.name === seriesObj.name)
if (existingSeries) {
await Database.bookSeriesModel.create({
@@ -238,6 +239,11 @@ class BookScanner {
libraryScan.addLog(LogLevel.DEBUG, `Updating book "${bookMetadata.title}" added new series "${seriesObj.name}"${seriesObj.sequence ? ` with sequence "${seriesObj.sequence}"` : ''}`)
seriesUpdated = true
}
+ } else if (seriesObj.sequence && existingBookSeries.bookSeries.sequence !== seriesObj.sequence) {
+ libraryScan.addLog(LogLevel.DEBUG, `Updating book "${bookMetadata.title}" series "${seriesObj.name}" sequence "${existingBookSeries.bookSeries.sequence || ''}" => "${seriesObj.sequence}"`)
+ seriesUpdated = true
+ existingBookSeries.bookSeries.sequence = seriesObj.sequence
+ await existingBookSeries.bookSeries.save()
}
}
// Check for series removed
@@ -657,7 +663,7 @@ class BookScanner {
if (!this.libraryItemData.metadataNfoLibraryFile) return
await NfoFileScanner.scanBookNfoFile(this.libraryItemData.metadataNfoLibraryFile, this.bookMetadata)
}
-
+
/**
* Description from desc.txt and narrator from reader.txt
*/
diff --git a/server/scanner/OpfFileScanner.js b/server/scanner/OpfFileScanner.js
index abc2540ac..87c4f5659 100644
--- a/server/scanner/OpfFileScanner.js
+++ b/server/scanner/OpfFileScanner.js
@@ -32,11 +32,8 @@ class OpfFileScanner {
bookMetadata.narrators = opfMetadata.narrators
}
} else if (key === 'series') {
- if (opfMetadata.series) {
- bookMetadata.series = [{
- name: opfMetadata.series,
- sequence: opfMetadata.sequence || null
- }]
+ if (opfMetadata.series?.length) {
+ bookMetadata.series = opfMetadata.series
}
} else if (opfMetadata[key] && key !== 'sequence') {
bookMetadata[key] = opfMetadata[key]
diff --git a/server/utils/parsers/parseOpfMetadata.js b/server/utils/parsers/parseOpfMetadata.js
index d5fb4651c..b51ceea5f 100644
--- a/server/utils/parsers/parseOpfMetadata.js
+++ b/server/utils/parsers/parseOpfMetadata.js
@@ -100,13 +100,19 @@ function fetchLanguage(metadata) {
}
function fetchSeries(metadataMeta) {
- if (!metadataMeta) return null
- return fetchTagString(metadataMeta, "calibre:series")
-}
-
-function fetchVolumeNumber(metadataMeta) {
- if (!metadataMeta) return null
- return fetchTagString(metadataMeta, "calibre:series_index")
+ if (!metadataMeta) return []
+ const result = []
+ for (let i = 0; i < metadataMeta.length; i++) {
+ if (metadataMeta[i].$?.name === "calibre:series" && metadataMeta[i].$.content?.trim()) {
+ const name = metadataMeta[i].$.content.trim()
+ let sequence = null
+ if (metadataMeta[i + 1]?.$?.name === "calibre:series_index" && metadataMeta[i + 1].$?.content?.trim()) {
+ sequence = metadataMeta[i + 1].$.content.trim()
+ }
+ result.push({ name, sequence })
+ }
+ }
+ return result
}
function fetchNarrators(creators, metadata) {
@@ -173,8 +179,7 @@ module.exports.parseOpfMetadataXML = async (xml) => {
description: fetchDescription(metadata),
genres: fetchGenres(metadata),
language: fetchLanguage(metadata),
- series: fetchSeries(metadata.meta),
- sequence: fetchVolumeNumber(metadata.meta),
+ series: fetchSeries(metadataMeta),
tags: fetchTags(metadata)
}
return data
diff --git a/test/server/utils/parsers/parseOpfMetadata.test.js b/test/server/utils/parsers/parseOpfMetadata.test.js
new file mode 100644
index 000000000..f1d5ce89e
--- /dev/null
+++ b/test/server/utils/parsers/parseOpfMetadata.test.js
@@ -0,0 +1,113 @@
+const chai = require('chai')
+const expect = chai.expect
+const { parseOpfMetadataXML } = require('../../../../server/utils/parsers/parseOpfMetadata')
+
+describe('parseOpfMetadata - test series', async () => {
+ it('test one series', async () => {
+ const opf = `
+
+
+
+
+
+
+
+ `
+ const parsedOpf = await parseOpfMetadataXML(opf)
+ expect(parsedOpf.series).to.deep.equal([{ "name": "Serie", "sequence": "1" }])
+ })
+
+ it('test more then 1 series - in correct order', async () => {
+ const opf = `
+
+
+
+
+
+
+
+
+
+
+
+ `
+ const parsedOpf = await parseOpfMetadataXML(opf)
+ expect(parsedOpf.series).to.deep.equal([
+ { "name": "Serie 1", "sequence": "1" },
+ { "name": "Serie 2", "sequence": "2" },
+ { "name": "Serie 3", "sequence": "3" },
+ ])
+ })
+
+ it('test messed order of series content and index', async () => {
+ const opf = `
+
+
+
+
+
+
+
+
+
+
+ `
+ const parsedOpf = await parseOpfMetadataXML(opf)
+ expect(parsedOpf.series).to.deep.equal([
+ { "name": "Serie 1", "sequence": "1" },
+ { "name": "Serie 3", "sequence": null },
+ ])
+ })
+
+ it('test different values of series content and index', async () => {
+ const opf = `
+
+
+
+
+
+
+
+
+
+
+
+ `
+ const parsedOpf = await parseOpfMetadataXML(opf)
+ expect(parsedOpf.series).to.deep.equal([
+ { "name": "Serie 1", "sequence": null },
+ { "name": "Serie 2", "sequence": "abc" },
+ { "name": "Serie 3", "sequence": null },
+ ])
+ })
+
+ it('test empty series content', async () => {
+ const opf = `
+
+
+
+
+
+
+
+ `
+ const parsedOpf = await parseOpfMetadataXML(opf)
+ expect(parsedOpf.series).to.deep.equal([])
+ })
+
+ it('test series and index using an xml namespace', async () => {
+ const opf = `
+
+
+
+
+
+
+
+ `
+ const parsedOpf = await parseOpfMetadataXML(opf)
+ expect(parsedOpf.series).to.deep.equal([
+ { "name": "Serie 1", "sequence": null }
+ ])
+ })
+})