groove/audiobookshelf
1
0
Fork 0
mirror of https://github.com/advplyr/audiobookshelf.git synced 2026-03-01 05:29:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

Merge c15cb48def into 1d0b7e383a

Browse source
This commit is contained in:
kctdfh 2026-02-25 02:55:17 +01:00 • committed by GitHub
commit 713d06fc8a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 22 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

8
server/finders/BookFinder.js
View file

@ -227,7 +227,7 @@ class BookFinder {
title = this.#removeAuthorFromTitle(title) title = this.#removeAuthorFromTitle(title)
const titleTransformers = [ const titleTransformers = [
[/([,:;_]| by ).*/g, ''], // Remove subtitle [/(: |[,;_]| by ).*/g, ''], // Remove subtitle
[/(^| )\d+k(bps)?( |$)/, ' '], // Remove bitrate [/(^| )\d+k(bps)?( |$)/, ' '], // Remove bitrate
[/ (2nd|3rd|\d+th)\s+ed(\.|ition)?/g, ''], // Remove edition [/ (2nd|3rd|\d+th)\s+ed(\.|ition)?/g, ''], // Remove edition
[/(^| |\.)(m4b|m4a|mp3)( |$)/g, ''], // Remove file-type [/(^| |\.)(m4b|m4a|mp3)( |$)/g, ''], // Remove file-type
@ -646,11 +646,11 @@ class BookFinder {
module.exports = new BookFinder() module.exports = new BookFinder()
function hasSubtitle(title) { function hasSubtitle(title) {
return title.includes(':') || title.includes(' - ') return title.includes(': ') || title.includes(' - ')
} }
function stripSubtitle(title) { function stripSubtitle(title) {
if (title.includes(':')) { if (title.includes(': ')) {
return title.split(':')[0].trim() return title.split(': ')[0].trim()
} else if (title.includes(' - ')) { } else if (title.includes(' - ')) {
return title.split(' - ')[0].trim() return title.split(' - ')[0].trim()
} }

2
server/utils/parsers/parseNfoMetadata.js
View file

@ -22,7 +22,7 @@ function parseNfoMetadata(nfoText) {
switch (key) { switch (key) {
case 'title': case 'title':
{ {
const titleMatch = value.match(/^(.*?):(.*)$/) const titleMatch = value.match(/^(.*?): (.*)$/)
if (titleMatch) { if (titleMatch) {
metadata.title = titleMatch[1].trim() metadata.title = titleMatch[1].trim()
metadata.subtitle = titleMatch[2].trim() metadata.subtitle = titleMatch[2].trim()

3
test/server/finders/BookFinder.test.js
View file

@ -35,7 +35,10 @@ describe('TitleCandidates', () => {
['adds candidate, removing author', `anna karenina by ${cleanAuthor}`, ['anna karenina']], ['adds candidate, removing author', `anna karenina by ${cleanAuthor}`, ['anna karenina']],
['does not add empty candidate after removing author', cleanAuthor, []], ['does not add empty candidate after removing author', cleanAuthor, []],
['adds candidate, removing subtitle', 'anna karenina: subtitle', ['anna karenina']], ['adds candidate, removing subtitle', 'anna karenina: subtitle', ['anna karenina']],
['adds candidate, not stripping subtitle for bare colon in title', '10:04', ['10:04']],
['adds candidate, not stripping subtitle for colon between words without space', 'making the mission:impossible movies', ['making the mission:impossible movies']],
['adds candidate + variant, removing "by ..."', 'anna karenina by arnold schwarzenegger', ['anna karenina', 'anna karenina by arnold schwarzenegger']], ['adds candidate + variant, removing "by ..."', 'anna karenina by arnold schwarzenegger', ['anna karenina', 'anna karenina by arnold schwarzenegger']],
['adds candidate + variant, removing "by ..." when title has bare colon', '10:04 by ben lerner', ['10:04', '10:04 by ben lerner']],
['adds candidate + variant, removing bitrate', 'anna karenina 64kbps', ['anna karenina', 'anna karenina 64kbps']], ['adds candidate + variant, removing bitrate', 'anna karenina 64kbps', ['anna karenina', 'anna karenina 64kbps']],
['adds candidate + variant, removing edition 1', 'anna karenina 2nd edition', ['anna karenina', 'anna karenina 2nd edition']], ['adds candidate + variant, removing edition 1', 'anna karenina 2nd edition', ['anna karenina', 'anna karenina 2nd edition']],
['adds candidate + variant, removing edition 2', 'anna karenina 4th ed.', ['anna karenina', 'anna karenina 4th ed.']], ['adds candidate + variant, removing edition 2', 'anna karenina 4th ed.', ['anna karenina', 'anna karenina 4th ed.']],

14
test/server/utils/parsers/parseNfoMetadata.test.js
View file

@ -21,6 +21,20 @@ describe('parseNfoMetadata', () => {
expect(result.subtitle).to.equal('A Novel') expect(result.subtitle).to.equal('A Novel')
}) })
it('does not split title on bare colon without space', () => {
const nfoText = 'Title: 10:04'
const result = parseNfoMetadata(nfoText)
expect(result.title).to.equal('10:04')
expect(result.subtitle).to.be.undefined
})
it('does not split title on colon between words without space', () => {
const nfoText = 'Title: Making the Mission:Impossible Movies'
const result = parseNfoMetadata(nfoText)
expect(result.title).to.equal('Making the Mission:Impossible Movies')
expect(result.subtitle).to.be.undefined
})
it('parses authors', () => { it('parses authors', () => {
const nfoText = 'Author: F. Scott Fitzgerald' const nfoText = 'Author: F. Scott Fitzgerald'
const result = parseNfoMetadata(nfoText) const result = parseNfoMetadata(nfoText)