danvics/soulsync
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

#916: align missing-track title match with the Reorganize matcher

Browse source 
Reporter's image 3 shows Reorganize maps all 62 multi-disc tracks correctly ('62 unchanged') —
it matches by title, proving the titles DO align on this album. My first normalizer DELETED
bracket content, so 'X - Main Theme' (file) vs 'X (Main Theme)' (canonical) would mismatch.
Reorganize treats brackets as separators (keeps the words); now _normTitleForMatch does the
same — drop only the (feat. Y) credit, turn every other separator into whitespace.

Verified: dash<->bracket, curly<->straight apostrophe, special<->regular hyphen, and feat all
normalize equal; distinct titles stay distinct.
This commit is contained in:
BoulderBadgeDad 2026-06-23 15:58:36 -07:00
parent 16cb29c9ee
commit 2934903874
1 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
webui/static/library.js
View file

@ -3964,13 +3964,19 @@ async function ensureEnhancedAlbumCanonicalTracks(album) {
}
}
// Loose title key for owned<->canonical matching. Lowercase, drop bracketed
// annotations ((feat. X), [Explicit]) and punctuation so editions line up.
// Loose title key for owned<->canonical matching. Mirrors the Reorganize
// matcher (core.library_reorganize._normalize_title), which already maps these
// same multi-disc tracks correctly: drop only the featured-artist credit, then
// treat every other separator (brackets, dashes, slashes, punctuation) as
// whitespace — so "X (Main Theme)" and "X - Main Theme" collapse to the same key
// while "(feat. Y)" is removed. Keeping bracket CONTENT (not deleting it) is what
// makes editions line up.
function _normTitleForMatch(value) {
return String(value || '')
.toLowerCase()
.replace(/\([^)]*\)|\[[^\]]*\]/g, ' ')
.replace(/[^a-z0-9]+/g, ' ')
.replace(/[([]\s*(?:feat|ft|featuring)\b[^)\]]*[)\]]/g, ' ') // (feat. Y) / [ft Y]
.replace(/\s+(?:feat|ft|featuring)\b\.?\s.*$/g, ' ') // trailing feat. Y …
.replace(/[^a-z0-9]+/g, ' ') // all other separators -> space (KEEP content)
.trim();
}