Opened 8 years ago

Last modified 3 years ago

#14029 assigned defect

[patch][cla]dojox.mdnd Moveable throwning error

Reported by: jbest Owned by:
Priority: high Milestone: 1.15
Component: Dojox Version: 1.6.1
Keywords: Cc:
Blocked By: Blocking:

Description

Moving DnD items when the onDrop event is fired requires you to first un-register them and then re-register them after moved. In this scenario dojox.mdnd.Moveable was throwning an error onMouseUp that 'this.node' is null (destroy was called on unregister).

Attachments (1)

fix_dojox_mdnd.patch​ (6.9 KB) - added by jbest 8 years ago.

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by jbest

Attachment: fix_dojox_mdnd.patch​ added

comment:1 Changed 8 years ago by Adam Peller

Owner: changed from Adam Peller to jfcunat

comment:2 Changed 8 years ago by Adam Peller

Owner: changed from jfcunat to Eugene Lazutkin
Summary: dojox.mdnd Moveable throwning error → [patch][cla]dojox.mdnd Moveable throwning error

comment:3 Changed 7 years ago by Eugene Lazutkin

Owner: changed from Eugene Lazutkin to jbest
Status: new → pending

I am not an owner of dojox.mdnd. Please assign to one of these guys:

  • Erwan Morvillez (emorvillez),
  • Jean-Jacques Patard (jjpatard),
  • Jeff Cunat (jfcunat)

comment:4 Changed 7 years ago by Adam Peller

Owner: changed from jbest to jfcunat
Status: pending → assigned

comment:5 Changed 4 years ago by dylan

Milestone: tbd → 1.12
Owner: jfcunat deleted

But is rather stale but probably still valid. Will consider for 1.12.

comment:6 Changed 3 years ago by dylan

Milestone: 1.13 → 1.15

Ticket planning... move current 1.13 tickets out to 1.15 to make it easier to move tickets into the 1.13 milestone.

Note: See TracTickets for help on using tickets.