Opened 10 years ago

Closed 10 years ago

Last modified 7 years ago

#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)

selectmenu.patch (179 bytes) - added by smith 10 years ago.

Download all attachments as: .zip

Change History (3)

Changed 10 years ago by smith

Attachment: selectmenu.patch added

comment:1 Changed 10 years ago by Scott González

Resolution: invalid
Status: newclosed

We don't accept tickets for labs plugins. Feel free to mention this on the planning page.

comment:2 Changed 7 years ago by Scott González

Milestone: TBD

Milestone TBD deleted

Note: See TracTickets for help on using tickets.