Opened 13 years ago
Closed 5 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)
Change History (10)
Changed 13 years ago by
Attachment: | TreeDnd.JPG added |
---|
comment:1 Changed 13 years ago by
Owner: | changed from Eugene Lazutkin to bill |
---|
comment:2 Changed 13 years ago by
Component: | DnD → Dijit |
---|
comment:3 Changed 13 years ago by
Summary: | Tree DnD: drop twice the same object → Tree DnD: drop twice the same object results in obscure error message. |
---|
comment:4 Changed 13 years ago by
Milestone: | → 1.3 |
---|
comment:5 Changed 12 years ago by
Milestone: | 1.3 → 1.5 |
---|---|
Priority: | high → normal |
severity: | major → normal |
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 11 years ago by
Milestone: | 1.5 → 1.6 |
---|
comment:7 Changed 10 years ago by
Milestone: | 1.6 → future |
---|
comment:8 Changed 8 years ago by
Priority: | high → low |
---|
comment:9 Changed 5 years ago by
Milestone: | future → 1.12 |
---|---|
Resolution: | → patchwelcome |
Status: | new → closed |
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.
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.