JimLiu/angular-ui-tree — pull request #261 by NickPadilla (2014-10-17 17:23:30+00:00)
Title: Bug fixes in Master 

There were some problems with master.

In file, uiTreeNode.js.

Line 710 - problem when the list is null or undefined - cannot call .length. So i updated it to use the protected method to find the childNode count without throwing up. :)

Line 725 - this i am unsure about, and would like some dialog. Problem was that when callbacks.accept is called, because move == true, we have undefined $childNodesScope and childNodes(). So what I am doing is going up the heriarchy so that we can get the parent that contains the children. This has stopped causing JS exceptions from being thrown but not the right fix i don't think.

Line 870, 936, 941, and 977(line #'s all reflect the update to 725) all have been modified so that we use the standard $document to bind to. This is really just to ensure we remain consistent in binding usage.

NOTE: we will need to merge PR #256 so that the build happens successfully.

Please advise as to how to proceed with modifications on line 725. Also, as of now the resorting functionality is completely broken - so I am not going to use it as of now. The accept() callback does return true but we never get a drop() callback to happen and the placeholders never present themselves properly. I had to use master because of a memory leak that was causing problems that has been fixed but not released.

Thanks for the cool project and please let me know what i can do to get this all working again!
Nick