Skip to main content

Search and Top Navigation

#5034 closed bug (notabug)

Opened December 31, 2009 10:47PM UTC

Closed January 01, 2010 04:10AM UTC

Last modified October 11, 2012 09:15PM UTC

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 (2)

Changed January 01, 2010 04:10AM UTC by scottgonzalez comment:1

resolution: → invalid
status: newclosed

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

Changed October 11, 2012 09:15PM UTC by scottgonzalez comment:2

milestone: TBD

Milestone TBD deleted