This commit is contained in:
+8
-1
@@ -499,7 +499,14 @@ export function legacySubtitleMigration(existingMeta: Record<string, unknown> |
|
|||||||
const langs = legacyCsvMembers(existingMeta, 'subtitleLanguages');
|
const langs = legacyCsvMembers(existingMeta, 'subtitleLanguages');
|
||||||
if (langs) {
|
if (langs) {
|
||||||
deletes.push('subtitleLanguages');
|
deletes.push('subtitleLanguages');
|
||||||
for (const l of langs.map(toLang3)) if (l !== 'und') sets[`subtitleLanguages/${l}`] = 'true';
|
for (const l of langs.map(toLang3)) {
|
||||||
|
if (l === 'und') continue;
|
||||||
|
sets[`subtitleLanguages/${l}`] = 'true';
|
||||||
|
// …and the union (§9 rule #7). The live write path below already
|
||||||
|
// does this; the migration path did not, so records that only ever
|
||||||
|
// went through it were left half-written.
|
||||||
|
sets[`languages/${l}`] = 'true';
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
const subs = legacyCsvMembers(existingMeta, 'subtitles');
|
const subs = legacyCsvMembers(existingMeta, 'subtitles');
|
||||||
|
|||||||
@@ -310,7 +310,13 @@ describe('legacySubtitleMigration', () => {
|
|||||||
subtitles: 'cidS',
|
subtitles: 'cidS',
|
||||||
})).toEqual({
|
})).toEqual({
|
||||||
deletes: ['extractedSubtitles', 'subtitleLanguages', 'subtitles'],
|
deletes: ['extractedSubtitles', 'subtitleLanguages', 'subtitles'],
|
||||||
sets: { 'subtitleLanguages/eng': 'true', 'subtitleLanguages/fre': 'true', 'subtitles/und/cidS': 'true' },
|
sets: {
|
||||||
|
'subtitleLanguages/eng': 'true',
|
||||||
|
'languages/eng': 'true',
|
||||||
|
'subtitleLanguages/fre': 'true',
|
||||||
|
'languages/fre': 'true',
|
||||||
|
'subtitles/und/cidS': 'true',
|
||||||
|
},
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user