My favorites | Sign in
Project Home Downloads
READ-ONLY: This project has been archived. For more information see this post.
Search
for
  Advanced search   Search tips   Subscriptions
Issue 326: Sorting while bEnableUpdate is false causes error
1 person starred this issue and may be notified of changes. Back to list
Status:  WontFix
Owner:  ----
Closed:  May 2014


 
Reported by slubow...@gmail.com, Jul 10, 2012
What steps will reproduce the problem?
Actually have very complicated use case but suspect following:

1. Call enableUpdate(false) on tree
2. Add a node with children
3. Call sortChildren on new node (deep)
4. Call enableUpdate(true) on tree

What is the expected output? What do you see instead?
Expect child node with children to be shown in tree. Get error instead.

What version of the dynatree and jQuery are you using?
dynatree 1.2.1 - jQuery 1.7.2

On what operating system and browser?
Windows 7 - FireFox nightly

Please provide any additional information below.
During sort render is called (line 546). render fails as parent.li is null (line 317).
Fix is to change line 545 from
if( deep !== "$norender$" )
to
if( this.tree.bEnableUpdate && deep !== "$norender$" )
Jul 13, 2012
#1 slubow...@gmail.com
See also  issue 330  which while different use case (remove) exposes same issue - render being called while tree.bEnableUpdate is false.
Jul 13, 2012
Project Member #2 moo...@wwwendt.de
(No comment was entered for this change.)
Status: Accepted
Labels: Milestone-Release1.2.2
Oct 6, 2012
Project Member #3 moo...@wwwendt.de
(No comment was entered for this change.)
Labels: -Milestone-Release1.2.2 Milestone-Release1.2.x
May 1, 2014
Project Member #4 moo...@wwwendt.de
As of 2014 Dynatree is feature frozen.
Please have a look at Fancytree (sequel of DynaTree 1.x): chances are good that the problem was resolved / the requested featuer is already implemented.
Please open a new issue there otherwise:

https://github.com/mar10/fancytree
Status: WontFix
May 3, 2014
Project Member #5 moo...@wwwendt.de
(No comment was entered for this change.)
Labels: -Milestone-Release1.2.x Milestone-Release2.0

Powered by Google Project Hosting