In the rules UI I have two different tabledrag tables on a page, but it sets the wrong parent id for elements of the second one - it sets the parent id of the first table. This is because the selector just selects for tr.draggable:first on the whole page - which is obviously wrong for the second tabledrag table.

Attached patch fixes that problem.

CommentFileSizeAuthor
#5 tabledrag_758514.patch685 bytesdrewish
tabledrag_fix.patch687 bytesfago

Comments

casey’s picture

casey commented
Status: Needs review » Reviewed & tested by the community

yep that's what it should be.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

drewish’s picture

drewish commented
Status: Closed (fixed) » Patch (to be ported)

Seems like this would apply to D6 as well.

drewish’s picture

drewish commented
Version: 7.x-dev » 6.x-dev
Status: Patch (to be ported) » Needs review
StatusFileSize
new tabledrag_758514.patch685 bytes

Here's a patch that applies with no fuzz.

Status: Needs review » Needs work

The last submitted patch, tabledrag_758514.patch, failed testing.

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.