Skip to content

Fix Markdown TOC Provider for Invalid References#22553

Merged
mjbvz merged 1 commit intomicrosoft:masterfrom
mjbvz:fix-markdown-headers-symbols-for-invalid-references
Mar 13, 2017
Merged

Fix Markdown TOC Provider for Invalid References#22553
mjbvz merged 1 commit intomicrosoft:masterfrom
mjbvz:fix-markdown-headers-symbols-for-invalid-references

Conversation

@mjbvz
Copy link
Collaborator

@mjbvz mjbvz commented Mar 13, 2017

Fixes #22494

Bug
References without a definition can cause the markdown table of contents provider to break

Fix
Pass in an empty environment to markdown-it parse to prevent the null dereference on invalid links. Also picks up new markdown it version and fixes a potential case where currentDocument is undefined

Fixes microsoft#22494

**Bug**
References without a definition can cause the markdown table of contents provider to break

**Fix**
Pass in an empty environment to markdown-it `parse` to prevent the null dereference on invalid links.
@mjbvz mjbvz self-assigned this Mar 13, 2017
@mjbvz mjbvz merged commit 31d0799 into microsoft:master Mar 13, 2017
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Markdown Header is not displayed (empty) in "Go to Symbol" of Markdown mode.

2 participants