Opened 11 years ago

Closed 3 years ago

#6346 closed defect (patchwelcome)

Tree DnD: drop twice the same object results in obscure error message.

Reported by: guest Owned by: bill
Priority: low Milestone: 1.13
Component: Dijit Version: 0.9
Keywords: Tree Dnd Cc:
Blocked By: Blocking:

Description

With Firefox, an exception occurs when you're trying to drop twice the same object in the same tree.

Exception: [Exception... "'Error: assertion failed in ItemFileWriteStore?' when calling method: [nsIDOMEventListener::handleEvent]" nsresult: "0x8057001c (NS_ERROR_XPC_JS_THREW_JS_OBJECT)" location: "<unknown>" data: no]

Attachments (1)

TreeDnd.JPG (98.7 KB) - added by guest 11 years ago.

Download all attachments as: .zip

Change History (10)

Changed 11 years ago by guest

Attachment: TreeDnd.JPG added

comment:1 Changed 11 years ago by Eugene Lazutkin

Owner: changed from Eugene Lazutkin to bill

comment:2 Changed 11 years ago by Eugene Lazutkin

Component: DnDDijit

comment:3 Changed 11 years ago by bill

Summary: Tree DnD: drop twice the same objectTree DnD: drop twice the same object results in obscure error message.

That's correct, an exception occurs since you can't add the same item to a store twice. Not sure how to fix this or even what the UI should be to inform the user of their error, but will look into it. Perhaps the drag icon would be red to begin with (maybe checkAcceptance() should check if the item already exists), although that doesn't really tell the user what the problem is.

comment:4 Changed 11 years ago by bill

Milestone: 1.3

comment:5 Changed 10 years ago by bill

Milestone: 1.31.5
Priority: highnormal
severity: majornormal
Version: 0.9

For now the application has to work around this... as I said above you can problem catch it w/a custom version of checkAcceptance().

comment:6 Changed 9 years ago by Adam Peller

Milestone: 1.51.6

comment:7 Changed 8 years ago by bill

Milestone: 1.6future

comment:8 Changed 6 years ago by bill

Priority: highlow

comment:9 Changed 3 years ago by dylan

Milestone: future1.12
Resolution: patchwelcome
Status: newclosed

Given that no one has shown interest in creating a patch in the past 7 years, I'm closing this as patchwelcome.

Note: See TracTickets for help on using tickets.