Opened 12 years ago
Closed 4 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 12 years ago by
| Attachment: | TreeDnd.JPG added |
|---|
comment:1 Changed 12 years ago by
| Owner: | changed from Eugene Lazutkin to bill |
|---|
comment:2 Changed 12 years ago by
| Component: | DnD → Dijit |
|---|
comment:3 Changed 12 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 12 years ago by
| Milestone: | → 1.3 |
|---|
comment:5 Changed 11 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 10 years ago by
| Milestone: | 1.5 → 1.6 |
|---|
comment:7 Changed 9 years ago by
| Milestone: | 1.6 → future |
|---|
comment:8 Changed 7 years ago by
| Priority: | high → low |
|---|
comment:9 Changed 4 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.