vscode-plantuml: Export Current Diagram fails

The command Export Current Diagram (plantuml.exportCurrent) causes following exception on my machine:


TypeError: Path must be a string. Received [Function]
	at assertPath (path.js:7:11)
	at Object.isAbsolute (path.js:439:5)
	at Includer._calcIncludes (C:\Users\weber\.vscode\extensions\jebbs.plantuml-2.3.0\out\src\plantuml\diagram\includer.js:24:23)
	at Includer.addIncludes (C:\Users\weber\.vscode\extensions\jebbs.plantuml-2.3.0\out\src\plantuml\diagram\includer.js:11:18)
	at DiagramAt (C:\Users\weber\.vscode\extensions\jebbs.plantuml-2.3.0\out\src\plantuml\diagram\diagram.js:89:48)
	at Diagram.GetCurrent (C:\Users\weber\.vscode\extensions\jebbs.plantuml-2.3.0\out\src\plantuml\diagram\diagram.js:50:18)
	at Object.<anonymous> (C:\Users\weber\.vscode\extensions\jebbs.plantuml-2.3.0\out\src\plantuml\exporter\exportDocument.js:49:50)
	at Generator.next (<anonymous>)
	at fulfilled (C:\Users\weber\.vscode\extensions\jebbs.plantuml-2.3.0\out\src\plantuml\exporter\exportDocument.js:4:58)
	at <anonymous>

the command plantuml.preview does not work either.

About this issue

Original URL
State: closed
Created 7 years ago
Comments: 15 (7 by maintainers)

Commits related to this issue

fix #68, #74 — committed to jkeys089/vscode-plantuml by qjebbs 7 years ago

Most upvoted comments

Basically, the fault is mine. Spellcheck plugin defines a function inArray in Array.prototype, this kind of defines take effects globally for all plugins, that I get a function among strings (use for...in, which I shouldn’t), and cause the problem.

Since every one may define inArray-like functions, it may cause conflicts & overrides. It’s not recommended to do so, in my opinion.

qjebbs on Nov 5, 2017

workaround found: I found that if I disabled the Spellcheck plugin this issue is fixed for me in both vscode and vscode-insider. 😃 Can the author perhaps install the plugin and debug the clash? It may be that the bug is introduced completely by the Spellcheck plugin and the issue belongs in their repo.

c835722 on Nov 4, 2017