self-hosted/joplin
1
0
Fork 0
mirror of https://github.com/laurent22/joplin.git synced 2025-10-31 00:07:48 +02:00
Code Issues Releases Activity

Desktop: Fixes #7782: Markdown + Front Matter export fails when tag(s) lost (#7820)

Browse Source
This commit is contained in:
pedr
2023-02-21 14:57:04 -03:00
committed by GitHub
parent 9e73d3590b
commit 6a9848ebe7
2 changed files with 17 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
packages/lib/services/interop/InteropService_Exporter_Md_frontmatter.test.ts
View File

@@ -131,4 +131,15 @@ describe('interop/InteropService_Exporter_Md_frontmatter', () => {
expect(content).not.toContain('longitude');
expect(content).not.toContain('altitude');
}));
test('should export note without tag keyword if the tag has been deleted', (async () => {
const folder1 = await Folder.save({ title: 'folder1' });
const note = await Note.save({ title: 'NoTag', body: '**ma note**', parent_id: folder1.id });
const tag = await Tag.save({ title: 'tag' });
await Tag.setNoteTagsByIds(note.id, [tag.id]);
await Tag.delete(tag.id);
const content = await exportAndLoad(`${exportDir()}/folder1/NoTag.md`);
expect(content).not.toContain('tag');
}));
});

8
packages/lib/services/interop/InteropService_Exporter_Md_frontmatter.ts
View File

@@ -127,8 +127,12 @@ export default class InteropService_Exporter_Md_frontmatter extends InteropServi
const context: FrontMatterContext = this.context();
if (context.noteTags[note.id]) {
const tagIds = context.noteTags[note.id];
const tags = tagIds.map((id: string) => context.tagTitles[id]).sort();
md['tags'] = tags;
// In some cases a NoteTag can still exist, while the Tag does not. In this case, tagTitles
// for that tagId will return undefined, which can't be handled by the yaml library (issue #7782)
const tags = tagIds.map((id: string) => context.tagTitles[id]).filter(e => !!e).sort();
if (tags.length > 0) {
md['tags'] = tags;
}
}
// This guarentees that fields will always be ordered the same way