Opened 10 years ago

Closed 7 years ago

#10815 closed defect (fixed)

[patch][cla]dijit.Tree dndSource can never set pure dnd target

Reported by: youngho Owned by: bill
Priority: high Milestone: 1.8
Component: Dijit Version: 1.4.0
Keywords: Cc:
Blocked By: Blocking:

Description

Hello,

Because in tree postCreated method line 671
if(this[this.dndParams[i]]){ params[this.dndParams[i]] = this[this.dndParams[i]]; }

isSource variable can not set 'false' even if "isSource" add into dndParams array and set isSource: false into the tree.

I think isSource is dnd factor. so why don't you set this param at the dndController ?

I attach a patch.

Attachments (1)

tree.dndSource (703 bytes) - added by youngho 10 years ago.

Download all attachments as: .zip

Change History (3)

Changed 10 years ago by youngho

Attachment: tree.dndSource added

comment:1 Changed 7 years ago by bill

Milestone: tbd1.8
Owner: set to bill
Status: newassigned

Sorry that it took two years, but I'll check this in now.

comment:2 Changed 7 years ago by bill

Resolution: fixed
Status: assignedclosed

In [29289]:

Allow subclasses of dndSource to set isSource:false, fixes #10815 !strict, thanks youngho.

Note: See TracTickets for help on using tickets.