Thought I should report a couple of bugs that I came across recently in dependecyGraph.js, again they seem to have been introduced during an attempt to optimize the code for performance.
There are a number of places where "vertices" should read "this.vertices", and also, since this property is an object rather than an array, "vertices.length" should not be used in for loops, but rather user for ... in. After fixing this, I could get rid of a call to model.rebuild() in some page scripts, and things work much smoother that way.
BTW, very silent in this forum nowadays... which is a bit weird, to me Formfaces still seems to be the best javascript implementation of XForms out there!
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Hi,
Thought I should report a couple of bugs that I came across recently in dependecyGraph.js, again they seem to have been introduced during an attempt to optimize the code for performance.
There are a number of places where "vertices" should read "this.vertices", and also, since this property is an object rather than an array, "vertices.length" should not be used in for loops, but rather user for ... in. After fixing this, I could get rid of a call to model.rebuild() in some page scripts, and things work much smoother that way.
BTW, very silent in this forum nowadays... which is a bit weird, to me Formfaces still seems to be the best javascript implementation of XForms out there!