If multiple tables with the tablesorter class exist on the same page then the sorter will not work properly on any of them. This is because the jQuery references to the tables in tablesortervar.js are ambiguous.

CommentFileSizeAuthor
#5 multiple_tables_on_same-2716277-5.patch1.46 KBdinarcon
#2 tablesorter-multiple_tables-2716277-1.patch1.69 KBOliverColeman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

OliverColeman created an issue. See original summary.

OliverColeman’s picture

OliverColeman CreditAttribution: OliverColeman commented
FileSize
tablesorter-multiple_tables-2716277-1.patch1.69 KB
monymirza’s picture

monymirza
Primary language Urdu
Location Islamabad
CreditAttribution: monymirza as a volunteer commented
Status: Active » Patch (to be ported)
joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Status: Patch (to be ported) » Needs review

Patch (to be ported) is really to backport https://www.drupal.org/node/156119#patch-to-be-ported

dinarcon’s picture

dinarcon CreditAttribution: dinarcon at Agaric commented
FileSize
multiple_tables_on_same-2716277-5.patch1.46 KB

Thanks @OliverColeman, your patches worked! The patch uses the Drupal root directory as base for the path. Usually, patches are created from the module directory instead. See https://www.drupal.org/node/707484 I have added a new patch using the module directory as a base for the path.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Version: 7.x-1.2 » 7.x-1.x-dev
Status: Needs review » Reviewed & tested by the community

This seemed to fix the problem for me. Thanks @dinarcon and @OliverColeman

lolandese’s picture

lolandese CreditAttribution: lolandese at HCL Technologies Limited commented

RTBC also from me. Can this please be committed and merged? Thanks.

  • monymirza committed 0332cb1 on 7.x-1.x 
    Issue #2716277 by OliverColeman, dinarcon: Multiple tables on same page...
monymirza’s picture

monymirza
Primary language Urdu
Location Islamabad
CreditAttribution: monymirza as a volunteer commented
Status: Reviewed & tested by the community » Fixed
monymirza’s picture

monymirza
Primary language Urdu
Location Islamabad
CreditAttribution: monymirza as a volunteer commented
Status: Fixed » Closed (fixed)