search

adrai / flowchart.js

Draws simple SVG flow chart diagrams from textual representation of the diagram
http://flowchart.js.org/
MIT License
8.54k stars 1.21k forks source link

Fix params bug #175

Closed leo108 closed 5 years ago

adrai commented 5 years ago

Can you shortly explain what this is fixing?

leo108 commented 5 years ago

@adrai I thought the diff is clear enough.

The bug is that we cannot define more than one params like this

op1(param1=1,param2=2)=>operation: Node

the params property in symbol object will only have param1 property.

The code here is going to iterate the entries variable using for statement with i as index key, it obviously should access the array item using entries[i] instead of entries[0].

adrai commented 5 years ago

released in v1.12.1