Skip to main content

Search and Top Navigation

#3567 closed bug (fixed)

Opened November 13, 2008 10:25AM UTC

Closed November 24, 2008 01:25PM UTC

toArray not working

Reported by: PJ Owned by: paul
Priority: blocker Milestone: 1.7
Component: ui.sortable Version: 1.6rc2
Keywords: sortable toArray Cc:
Blocked by: Blocking:
Description

When there is a call to sortable('toArray'), we get the following error :

o is undefined

items.each(function() { ret.push($(this).attr(o.attr || 'id')); });

Attachments (0)
Change History (5)

Changed November 13, 2008 10:31AM UTC by PJ comment:1

I've found the bug :

just add the o parameter to the function, like this:

items.each(function(o) { ret.push($(this).attr(o.attr('id')); });

Changed November 13, 2008 10:39AM UTC by PJ comment:2

Sorry, here is the exact fix :

items.each(function(o){ret.push($(this).attr(o.attr||'id'));});

At line 118 in ui.sortable.js

Changed November 16, 2008 05:15AM UTC by scottgonzalez comment:3

priority: minorblocker

Changed November 23, 2008 01:23PM UTC by scottgonzalez comment:4

milestone: TBD1.6

Changed November 24, 2008 01:25PM UTC by paul comment:5

resolution: → fixed
status: newclosed

This has recently been fixed.