Consensys / vscode-solidity-auditor

Solidity language support and visual security auditor for Visual Studio Code
https://marketplace.visualstudio.com/items?itemName=tintinweb.solidity-visual-auditor
GNU General Public License v3.0
578 stars 60 forks source link

graph(this) feature does not work #20

Closed 3q-coder closed 5 years ago

3q-coder commented 5 years ago

It returns only text file. 3 I have installed all needed extensions and surya v2.0.11. I use Ubuntu 18.04.

tintinweb commented 5 years ago

Hi @DryginAlexander,

Looks like something went wrong with the installation of the dot preview extension I created for solidity-visual-auditor. How it basically works is that when you install solidity-visual-auditor vscode is actually installing two extension, one is the auditor plugin itself and the other one is an interactive dot preview extension I created. If you click "graph (this)" it is running surya to generate the graph, then creates a new document and previews it in the extension - if it is installed. if not you'll get the plain graphviz dot source.

Normally vscode should take care of installing the preview extension for you as instructed in the vscode-auditor metadata.

let me know if that solves the problem.

cheers, tin

3q-coder commented 5 years ago

Hi @tintinweb,

Vscode shows that the dot preview extension has been already installed. 1

tintinweb commented 5 years ago

@DryginAlexander

Oh, wait. It appears that the default for the auto preview is off by default (because this used to be an experimental feature). https://github.com/tintinweb/vscode-solidity-auditor/blob/master/package.json#L217-L220

Please verify that it works by switching on solidity-va.preview.dot in the settings. I'll release an updated version if that fixes the problem.

3q-coder commented 5 years ago

@tintinweb

I have switched on this option and the graph features works correctly now.

Thanks for your time!

tintinweb commented 5 years ago

👍 - should be fixed in v0.0.16