#5034 closed bug (notabug)
for..in loop conflict in select menu
| Reported by: | smith | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | |
| Component: | ui.core | Version: | |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
In the ui.selectmenu, a for..in loop is used on an Array, which creates a bunch of undefined values on the menu if the Array prototype has been extended (usually w/ Prototype or Mootools loaded.)
I changed this to a regular for loop in an attached patch.
(note: This is in labs ui.selectmenu, so the Component selection is wrong on the ticket.)
Attachments (1)
Change History (3)
Changed 10 years ago by
| Attachment: | selectmenu.patch added |
|---|
comment:1 Changed 10 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
We don't accept tickets for labs plugins. Feel free to mention this on the planning page.