class ajax-sample-table don't assigned to the table which is why ajax_command_restripe("table.ajax-sample-table") don't work correctly.

CommentFileSizeAuthor
#4 1941652-3.patch684 bytesdmitrii
ctools_ajax_sample.module.diff681 bytesdmitrii

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-1.2 » 7.x-1.x-dev
Issue summary: View changes

DamienMcKenna queued ctools_ajax_sample.module.diff for re-testing.

chris matthews’s picture

chris matthews commented
Status: Needs review » Needs work
Issue tags: +Needs reroll

The 5 year old patch to ctools_ajax_sample.module does not apply to the latest 7.x-1.x-dev.

Checking patch ctools_ajax_sample/ctools_ajax_sample.module...
error: while searching for:
    );
  }

  $output .= theme('table', array('header' => $header, 'rows' => $rows, array('class' => array('ajax-sample-table'))));

  // Show examples of ctools javascript widgets
  $output .= '<h2>'. t('CTools Javascript Widgets') .'</h2>';

error: patch failed: ctools_ajax_sample/ctools_ajax_sample.module:187
error: ctools_ajax_sample/ctools_ajax_sample.module: patch does not apply
dmitrii’s picture

dmitrii commented
Status: Needs work » Needs review
StatusFileSize
new 1941652-3.patch684 bytes

Updated.

chris matthews’s picture

chris matthews commented
Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs reroll

Thanks dmitrii, the updated patch in #4 now applies to the latest 7.x-1.x-dev and fixes this issue for me.

  • joelpittet committed 9c61238 on 7.x-1.x authored by dmitrii 
    Issue #1941652 by dmitrii, Chris Matthews, DamienMcKenna:...
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Reviewed & tested by the community » Fixed

Thanks I've committed this to the latest dev release

Status: Fixed » Closed (fixed)

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