Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#12486 closed defect (fixed)

[regression] Tree: exception removing selected node

Reported by: Mewel Owned by: bill
Priority: high Milestone: 1.6.1
Component: Dijit Version: 1.6.0
Keywords: Cc:
Blocked By: Blocking:

Description

Removing a node from a tree and then selecting another node throws an exception.

Sample file is attached.

Attachments (1)

tree_bug.html (1.7 KB) - added by Mewel 9 years ago.
testcase

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by Mewel

Attachment: tree_bug.html added

testcase

comment:1 Changed 9 years ago by bill

Summary: dijit.Tree - removing tree nodesTree: removing tree nodes

So, are these the steps to reproduce?

1) select a tree node

2) press delete button

3) select another tree node

comment:2 Changed 9 years ago by Mewel

Exactly, these are the steps to get the exception.

comment:3 Changed 9 years ago by bill

Milestone: tbd1.6.1
Summary: Tree: removing tree nodes[regression] Tree: exception removing focused node

Kgf tells me this is a regression.

comment:4 Changed 9 years ago by bill

Owner: set to bill
Status: newassigned

Thanks for pointing it out (and for the test case), I'll fix.

comment:5 Changed 9 years ago by bill

Resolution: fixed
Status: assignedclosed

(In [24104]) Remove deleted tree nodes from selection (if they are in the selection). Avoids exceptions later on when changing the selection. Also cleaned up Tree.html test case, in addition to adding a test for this bug.

Fixes #12486 on trunk, !strict.

comment:6 Changed 9 years ago by bill

(In [24105]) Remove deleted tree nodes from selection (if they are in the selection). Avoids exceptions later on when changing the selection. Also cleaned up Tree.html test case, in addition to adding a test for this bug.

Fixes #12486 on 1.6/ branch, !strict.

comment:7 Changed 9 years ago by bill

Summary: [regression] Tree: exception removing focused node[regression] Tree: exception removing selected node
Note: See TracTickets for help on using tickets.