#1527 closed defect (wontfix)
PopupMenu2 for non-existent targetId becomes window's context menu.
| Reported by: | guest | Owned by: | liucougar |
|---|---|---|---|
| Priority: | low | Milestone: | |
| Component: | Widgets | Version: | 0.3 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
When a PopupMenu2 contextmenu is given a targetNodeIds value that contains an id that is not present in the document the contextmenu is bound as the window's context menu rather than just being ignored for that non-existent targetId.
Change History (6)
comment:1 Changed 13 years ago by
| Milestone: | → 0.5 |
|---|---|
| Owner: | changed from bill to liucougar |
comment:2 Changed 13 years ago by
| Milestone: | 0.5 → 0.4.1 |
|---|---|
| Status: | new → assigned |
comment:3 Changed 13 years ago by
| Cc: | wkeese@… added |
|---|
comment:4 Changed 13 years ago by
| Cc: | wkeese@… removed |
|---|
I think users should ensure validation of the id, because I don't like increasing dojo's download size just to detect programming errors, but having said that, I don't care much either way. In any case this doesn't seem like a critical bug (not sure why you scheduled it against 0.4.1; I guess because it's trivial to fix)
comment:5 Changed 13 years ago by
| Resolution: | → wontfix |
|---|---|
| Status: | assigned → closed |
I agree
so close this bug as wontfix
please check id before passing it to menu2 bind dom
Note: See
TracTickets for help on using
tickets.
Bill:
shall we add a checking in Menu2 ? or users shall ensure the validation of an id?