Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#15898 closed defect (fixed)

Tree: no error thrown setting invalid path

Reported by: bill Owned by: bill
Priority: undecided Milestone: 1.9
Component: Dijit Version: 1.8.0
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by bill)

Tree apparently doesn't throw an error when setting an invalid path, although it should.

I added tests for this in [29086] but forgot to "return d" so it was swallowing the errors.

Also, the test for setting a null path is incorrect.

Change History (3)

comment:1 Changed 7 years ago by bill

Description: modified (diff)
Milestone: tbd1.9
Summary: Tree: no error thrown setting null/invalid pathTree: no error thrown setting invalid path

comment:2 Changed 7 years ago by bill

Resolution: fixed
Status: newclosed

In [29568]:

Minimal fix to tree to throw an error when set("paths", ...) called with an invalid path. Also fixing incorrect test for setting null path. Fixes #15898 !strict.

comment:3 Changed 7 years ago by bill

In [29575]:

Fix other incorrect usages of DeferredList by switching to dojo/promise/all with a back-compat shim (to be removed in 2.0), refs #15898 !strict.

Note: See TracTickets for help on using tickets.