-
Notifications
You must be signed in to change notification settings - Fork 177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
loadData bug, not giving the node back #18
Comments
This can be a problem if the node_id parameter is a string and the node ids are integers. You can fix this by using parseInt:
Anyway, I do agree that using '==' would be better in this case. I can't change that at this moment because jqTree is written in CofeeScript, and CoffeeScript does not allow '==' (see http://coffeescript.org/#operators). |
Oke, sounds logic, i'll do that. |
Changed in the dev branch: the getNodeById comparison is now less strict. It uses the javascript '=='. |
thanks, i'm using 1.3.1 now, works fine :) |
I think i found a bug, ik wanted to use loadData, it didn't work because the node i got back from getNodeById was null. After a bit of searching i saw that the node_id & node.id in the getNodeById function are beeing compared with ===, instead of ==.
jqTree is awesome btw, i love it :)
The text was updated successfully, but these errors were encountered: