#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 13 years ago by
Attachment: | selectmenu.patch added |
---|
comment:1 Changed 13 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.