Ajax doesn't work with Tableselect with checkboxes

I can confirm the problem. Looking at form_process_tableselect() in form.inc shows that the line which passes the '#ajax' property on to the created child elements is only present in the 'radios' related else part of the if clause, but absent for the default checkbox creation part. Adding the line there fixes the problem on first sight - not sure if this has some negative effects and was left out on purpose, but I doubt that.

The attached patch does just that - add the '#ajax' property passing line from the 'radios' related part to the 'checkboxes' related one as well.

This patch works without a pager. However, I have a pager for this tableselect and after paging, the ajax handler is only called on the first checkbox clicked. Subsequent clicks do not fire the checkbox ajax.

UPDATE: This was due to my ajax form causing "An illegal choice has been detected. Please contact the site administrator." -- which I haven't solved yet.

UPDATE2: Hours later, I'm using my own custom ajax pager, which causes the same "illegal choice" error with my dynamically generated options. ['#validated']= TRUE; successfully suppresses the drupal form security feature that was balking at my changed form option values.

I have SimpleTests that are failing without this patch. What is the best practice for continuing to SimpleTest my modules until this patch is merged into core?

#1: drupal-tableselect-ajax-checkboxes-123456-1.patch queued for re-testing.

Just a follow up on the patch, it works fine if there is no pager.

If you do not have a paged display, it correctly attaches the AJAX callback and properties.

Rolled a patch for D8 to get it fixed there and marking for backport to D7.

I think the whole tableselect code could do with a review, but let's get the real problems like this fixed first.

This patch will make #ajax work for for checkboxes, but not for the "select all" checkbox that comes with them. This is probably a good thing, but something to be aware of.

@jbeuckm could you post your tests here?

Patch is using tabs.

Other than that it looks good but it needs tests.

Okay, my first attempt at tests.
One without fix which should fail and one with fix which should pass.

#9: drupal-tableselect-ajax-fixntest-1458824-9.patch queued for re-testing.

Re-rolled for file location changes.

This fixes the ajax callback if you select one of the options but if you select the "all" checkbox in the table header nothing happens like mentioned in #6. I know that this is an old bug report but it would be nice to get this fixed, please.

Updated the patch for the current 7.x core version

Hi,

I written a module with table select form element, but when I search a value with ajax, the tableselect is not showing results.

To resolve this issue, I utilize "form_process_tableselect", so the result is showing perfectly.

But the issue is, when I get search results, I don't perform any action on the results.

Procedure:

1. Form showing new post button ( to add the records )
2. table showing the records with tableselect element
3. Update button working to update the status

4. search a value with search field and button

No results are showing.

Modifications are added in the module like:
$form_html = drupal_render($form['list_form_items']);
changed this line with
$form_html = form_process_tableselect($form['list_form_items']);

4. results are showing, but when I select the check box for status update its not working.

Please check the code and let me know solutions

Example 1: ( no results are showing)

<?php

function advance_form_menu() {
    
    $items['advance-form'] = array(
        'title' => 'Advance form',
        'page callback' => 'user_advance_form_callback',
        //'page arguments' => array('advance_form_mod'),
        'access arguments' => array('access content'),
        'type' => MENU_CALLBACK,
    );
    $items['new-items/%'] = array(
        'title' => 'New form',
        'page callback' => 'advance_new_page',
        'page arguments' => array(1),
        'access callback' => TRUE,
        'type' => MENU_CALLBACK,
    );
    $items['change-service-status/%/%'] = array(
        'title' => 'Change service status',
        'page callback' => 'change_service_status',
        'page arguments' => array(1,2),
        'access callback' => TRUE,
        'type' => MENU_CALLBACK,
    );
    
    return $items;
}

function user_advance_form_callback(){


   $build = array();

    // Render your two forms here.
	  $build[] = drupal_get_form('advance_form_mod');
    $build[] = drupal_get_form('advance_user_search');
  

  return $build;
}
/**
 * Implements hook_form().
 */
function advance_form_mod($form, &$form_state) {
    drupal_add_js(drupal_get_path('module','advance_form').'/js/advance_form.js');
    ctools_include('modal');
    ctools_modal_add_js();
    $form = array();
    $slogan_url='new-items/1';
    
    
    $form['url'] = array(
        '#type' => 'hidden',
        '#attributes' => array('class' => array('hs-ajax-button-url')),
        '#value' => url($slogan_url),
    );
    
    $form['ajax_button'] = array(
        '#type' => 'button',
        '#value' => t('New Post'),
        '#attributes' => array('class' => array('ctools-use-modal','ctools-modal-slogan-style')),
        '#id' => 'hs-ajax-button',
    );
    $form['change_status'] = array(
        '#type' => 'fieldset',
        '#title' => t('Change status'),
        '#attributes' => array('class' => array('container-inline')),
    );
    $form['change_status']['operations'] = array(
        '#type' => 'select',
        '#title' => t('Operation'),
//        '#title_display' => 'Change status options',
        '#options' => array('select_status'=>t('Select status'),t('Stop'),t('Start')),
        '#default_value' => 'select_status',
    );
    $form['change_status']['submit'] = array(
        '#type' => 'button',
        '#value' => t('Update'),
        '#ajax' => array(
            'callback' => 'changing_selected_item_status',
            'wrapper' => 'itemlist-div',
            'method' => 'replace',
            'effect' => 'fade',
        ),
    );
    
    $header = array(
        'name' => array('data' => t('Name')),
        'last_name' => array('data' => t('Last name')),
        'status' => array('data' => t('Status')),
        'enable_disable' => array('data' => t('Enable/Disable')),
    );
    
    $query = db_select('advance_form', 'af');
    
    $count_query = clone $query;
    $count_query->addExpression('COUNT(af.advance_form_id)');
    
    $query = $query->extend('PagerDefault')->extend('TableSort');
    $query
        ->fields('af', array('name', 'last_name','advance_form_id','fruite','status'))
        ->limit(5)
        ->orderByHeader($header)
        ->setCountQuery($count_query);
    $result = $query->execute();
    
    foreach ($result as $account) {
        $linkLabel = $account->status?'Stop':'Start';
        $changeStatusTo = $account->status?0:1;
        $options[$account->advance_form_id] = array(
            'name' => $account->name,
            'last_name' =>  $account->last_name,
            'status' =>  '<div id="current_status_'.$account->advance_form_id.'">'.($account->status?'active':'disabled'),
            'enable_disable' => array(
                  'data' => array(
                      '#markup' => "<div id='change_status_".$account->advance_form_id."'>".l($linkLabel,'change-service-status/nojs/'.$changeStatusTo.'-'.$account->advance_form_id,array('attributes' => array('class' => 'use-ajax')))."</div>",
                  ),
            ),
        );
    }
//    echo "<pre>";print_r($options);exit;
    
    
    $form['list_form_items'] = array(
        '#type' => 'tableselect',
        '#header' => $header,
        '#options' => (!empty($options))?$options:array(),
        '#prefix' => '<div id="itemlist-div">',
        '#suffix' => '</div>',
//        '#multiple' => FALSE,
//        '#js_select' => FALSE,
        '#empty' => t('No forms available.'),
      );
    
    $form['pager'] = array('#markup' => theme('pager'));
   
   
    //$form['#executes_submit_callback'] = FALSE;
    return $form;
}
function advance_user_search($form, &$form_state){
    $form['search_area'] = array(
        '#type' => 'fieldset',
        '#title' => t('Search'),
        '#attributes' => array('class' => array('search-container-inline')),
    );
    $form['search_area']['operations'] = array(
        '#type' => 'textfield',
        '#title' => t('Search'),
        '#size' => 40,
        '#default_value' => '',
    );
    $form['search_area']['submit'] = array(
        '#type' => 'button',
        '#value' => t('Search'),
        '#ajax' => array(
            'callback' => 'search_items',
            'wrapper' => 'itemlist-div',
            'method' => 'replace',
            'effect' => 'fade',
        ),
    );
    return $form;
}

function advance_new_form($form, &$form_state) {
    $form['name'] = array(
        '#type' => 'textfield',
        '#title' => t('your name'),
    );
    $form['last-name'] = array(
        '#type' => 'textfield',
        '#title' => t('last name'),
    );
    $form['selected-fruite'] = array(
        '#type' => 'hidden',
        '#value' => $form_state['input']['select-fruite'],
    );
    $form['some_text'] = array(
        '#type' => 'submit',
        '#value' => 'Submit'
    );

    return $form;
}

function advance_new_form_submit(&$form, &$form_state) {
    db_insert('advance_form')->fields(array(
        'name' => $form_state['values']['name'],
        'last_name' => $form_state['values']['last-name'],
        'fruite' => $form_state['input']['selected-fruite'],
        'status' => 1,
    ))->execute();
    // Tell the browser to close the modal.
    
//    echo "<pre>";print_r($form_html);
//    drupal_render($form_html);
//    $form['#redirect'] = '/advance-form';
//    drupal_render($form['list_form_items']);
      $form_state['ajax_commands'][] = ctools_modal_command_dismiss();
      $form_state['ajax_commands'][] = ctools_ajax_command_redirect('advance-form');
}

function advance_new_page($ajax) {
    if ($ajax) {
        ctools_include('ajax');
        ctools_include('modal');


        $form_state = array(
            'ajax' => TRUE,
            'title' => t('New Form'),
        );

        // Use ctools to generate ajax instructions for the browser to create
        // a form in a modal popup.
        $output = ctools_modal_form_wrapper('advance_new_form', $form_state);

        // If the form has been submitted, there may be additional instructions
        // such as dismissing the modal popup.
        if (!empty($form_state['ajax_commands'])) {
            $output = $form_state['ajax_commands'];
        }

        // Return the ajax instructions to the browser via ajax_render().
        print ajax_render($output);
        drupal_exit();
    } else {
        return drupal_get_form('advance_new_form');
    }
}

function changing_selected_item_status($form,&$form_state){
   // echo "<pre>";print_r($form_state['input']['list_form_items']);exit;
//    $form_state['input']['operations'];
    
    foreach($form_state['input']['list_form_items'] as $key => $value){
        if($value){
            if($form_state['input']['operations']==1){
                $num_updated = db_update('advance_form')
                    ->fields(array(
                      'status' => 1,
                    ))
                    ->condition('advance_form_id',$value , '=')
                    ->execute();
            }else{
                $num_updated = db_update('advance_form')
                    ->fields(array(
                      'status' => 0,
                    ))
                    ->condition('advance_form_id',$value , '=')
                    ->execute();
            }
            
        }
    }
    $form = drupal_build_form('advance_form_mod', $form_state);
    
    $form_html = drupal_render($form['list_form_items']);
    return $form_html;
}
//for search
function search_items($form,&$form_state){
    
    $header = array(
        'name' => array('data' => t('Name')),
        'last_name' => array('data' => t('Last name')),
        'status' => array('data' => t('Status')),
        'enable_disable' => array('data' => t('Enable/Disable')),
    );
    $query = db_select('advance_form', 'af');
    $count_query = clone $query;
    $count_query->addExpression('COUNT(af.advance_form_id)');
    $searchValue=$form_state['input']['operations'];
    //echo $searchValue;exit;
    $query = $query->extend('PagerDefault')->extend('TableSort');
    $query
        ->condition('af.name','%'.$searchValue.'%','like')
        ->fields('af', array('name', 'last_name','advance_form_id','fruite','status'))
        ->limit(5)
        ->orderByHeader($header)
        ->setCountQuery($count_query);
    $result = $query->execute();
    //echo "<pre>";print_r($result);exit;
    foreach ($result as $account) {
        $linkLabel = $account->status?'Stop':'Start';
        $changeStatusTo = $account->status?0:1;
        $options[$account->advance_form_id] = array(
            'name' => $account->name,
            'last_name' =>  $account->last_name,
            'status' =>  '<div id="current_status_'.$account->advance_form_id.'">'.($account->status?'active':'disabled'),
            'enable_disable' => array(
                  'data' => array(
                      '#markup' => "<div id='change_status_".$account->advance_form_id."'>".l($linkLabel,'change-service-status/nojs/'.$changeStatusTo.'-'.$account->advance_form_id,array('attributes' => array('class' => 'use-ajax')))."</div>",
                  ),
            ),
        );
    }
    $form['list_form_items'] = array(
        '#type' => 'tableselect',
        '#header' => $header,
        '#options' => (!empty($options))?$options:array(),
        '#prefix' => '<div id="itemlist-div">',
        '#suffix' => '</div>',
        '#empty' => t('No forms available.'),
        '#multiple' => TRUE,
      );
    /*echo "<pre>";
    print_r($form['list_form_items']);exit;*/
    $form_html = drupal_render($form['list_form_items']);

    return $form_html;

}

function change_service_status($type,$status){
    if($type=='ajax'){
        drupal_add_library('system', 'drupal.ajax'); drupal_add_library('system', 'jquery.form');
        list($newStatus,$serviceID)=  explode('-', $status);
        $num_updated = db_update('advance_form')
                        ->fields(array(
                          'status' => $newStatus,
                        ))
                        ->condition('advance_form_id',$serviceID , '=')
                        ->execute();
        $showNewStatus = ($newStatus?'active':'disabled');
        $showLabel = ($newStatus?'Stop':'Start');
        $changeStatus=($newStatus?0:1);
        $commands[] = ajax_command_replace("#current_status_$serviceID", $showNewStatus); 
        $commands[] = ajax_command_replace("#change_status_$serviceID", "<div id='change_status_".$serviceID."'>".l($showLabel,'change-service-status/nojs/'.$changeStatus.'-'.$serviceID,array('attributes' => array('class' => 'use-ajax'))).'</div>'); 
        $page = array('#type' => 'ajax', '#commands' => $commands); 
        ajax_deliver($page);
    }
}

After modifications results are showing:

<?php

function advance_form_menu() {
    
    $items['advance-form'] = array(
        'title' => 'Advance form',
        'page callback' => 'user_advance_form_callback',
        //'page arguments' => array('advance_form_mod'),
        'access arguments' => array('access content'),
        'type' => MENU_CALLBACK,
    );
    $items['new-items/%'] = array(
        'title' => 'New form',
        'page callback' => 'advance_new_page',
        'page arguments' => array(1),
        'access callback' => TRUE,
        'type' => MENU_CALLBACK,
    );
    $items['change-service-status/%/%'] = array(
        'title' => 'Change service status',
        'page callback' => 'change_service_status',
        'page arguments' => array(1,2),
        'access callback' => TRUE,
        'type' => MENU_CALLBACK,
    );
    
    return $items;
}

function user_advance_form_callback(){


   $build = array();

    // Render your two forms here.
	  $build[] = drupal_get_form('advance_form_mod');
    $build[] = drupal_get_form('advance_user_search');
  

  return $build;
}
/**
 * Implements hook_form().
 */
function advance_form_mod($form, &$form_state) {
    drupal_add_js(drupal_get_path('module','advance_form').'/js/advance_form.js');
    ctools_include('modal');
    ctools_modal_add_js();
    $form = array();
    $slogan_url='new-items/1';
    
    
    $form['url'] = array(
        '#type' => 'hidden',
        '#attributes' => array('class' => array('hs-ajax-button-url')),
        '#value' => url($slogan_url),
    );
    
    $form['ajax_button'] = array(
        '#type' => 'button',
        '#value' => t('New Post'),
        '#attributes' => array('class' => array('ctools-use-modal','ctools-modal-slogan-style')),
        '#id' => 'hs-ajax-button',
    );
    $form['change_status'] = array(
        '#type' => 'fieldset',
        '#title' => t('Change status'),
        '#attributes' => array('class' => array('container-inline')),
    );
    $form['change_status']['operations'] = array(
        '#type' => 'select',
        '#title' => t('Operation'),
//        '#title_display' => 'Change status options',
        '#options' => array('select_status'=>t('Select status'),t('Stop'),t('Start')),
        '#default_value' => 'select_status',
    );
    $form['change_status']['submit'] = array(
        '#type' => 'button',
        '#value' => t('Update'),
        '#ajax' => array(
            'callback' => 'changing_selected_item_status',
            'wrapper' => 'itemlist-div',
            'method' => 'replace',
            'effect' => 'fade',
        ),
    );
    
    $header = array(
        'name' => array('data' => t('Name')),
        'last_name' => array('data' => t('Last name')),
        'status' => array('data' => t('Status')),
        'enable_disable' => array('data' => t('Enable/Disable')),
    );
    
    $query = db_select('advance_form', 'af');
    
    $count_query = clone $query;
    $count_query->addExpression('COUNT(af.advance_form_id)');
    
    $query = $query->extend('PagerDefault')->extend('TableSort');
    $query
        ->fields('af', array('name', 'last_name','advance_form_id','fruite','status'))
        ->limit(5)
        ->orderByHeader($header)
        ->setCountQuery($count_query);
    $result = $query->execute();
    
    foreach ($result as $account) {
        $linkLabel = $account->status?'Stop':'Start';
        $changeStatusTo = $account->status?0:1;
        $options[$account->advance_form_id] = array(
            'name' => $account->name,
            'last_name' =>  $account->last_name,
            'status' =>  '<div id="current_status_'.$account->advance_form_id.'">'.($account->status?'active':'disabled'),
            'enable_disable' => array(
                  'data' => array(
                      '#markup' => "<div id='change_status_".$account->advance_form_id."'>".l($linkLabel,'change-service-status/nojs/'.$changeStatusTo.'-'.$account->advance_form_id,array('attributes' => array('class' => 'use-ajax')))."</div>",
                  ),
            ),
        );
    }
//    echo "<pre>";print_r($options);exit;
    
    
    $form['list_form_items'] = array(
        '#type' => 'tableselect',
        '#header' => $header,
        '#options' => (!empty($options))?$options:array(),
        '#prefix' => '<div id="itemlist-div">',
        '#suffix' => '</div>',
//        '#multiple' => FALSE,
//        '#js_select' => FALSE,
        '#empty' => t('No forms available.'),
      );
    
    $form['pager'] = array('#markup' => theme('pager'));
   
   
    //$form['#executes_submit_callback'] = FALSE;
    return $form;
}
function advance_user_search($form, &$form_state){
    $form['search_area'] = array(
        '#type' => 'fieldset',
        '#title' => t('Search'),
        '#attributes' => array('class' => array('search-container-inline')),
    );
    $form['search_area']['operations'] = array(
        '#type' => 'textfield',
        '#title' => t('Search'),
        '#size' => 40,
        '#default_value' => '',
    );
    $form['search_area']['submit'] = array(
        '#type' => 'button',
        '#value' => t('Search'),
        '#ajax' => array(
            'callback' => 'search_items',
            'wrapper' => 'itemlist-div',
            'method' => 'replace',
            'effect' => 'fade',
        ),
    );
    return $form;
}

function advance_new_form($form, &$form_state) {
    $form['name'] = array(
        '#type' => 'textfield',
        '#title' => t('your name'),
    );
    $form['last-name'] = array(
        '#type' => 'textfield',
        '#title' => t('last name'),
    );
    $form['selected-fruite'] = array(
        '#type' => 'hidden',
        '#value' => $form_state['input']['select-fruite'],
    );
    $form['some_text'] = array(
        '#type' => 'submit',
        '#value' => 'Submit'
    );

    return $form;
}

function advance_new_form_submit(&$form, &$form_state) {
    db_insert('advance_form')->fields(array(
        'name' => $form_state['values']['name'],
        'last_name' => $form_state['values']['last-name'],
        'fruite' => $form_state['input']['selected-fruite'],
        'status' => 1,
    ))->execute();
    // Tell the browser to close the modal.
    
//    echo "<pre>";print_r($form_html);
//    drupal_render($form_html);
//    $form['#redirect'] = '/advance-form';
//    drupal_render($form['list_form_items']);
      $form_state['ajax_commands'][] = ctools_modal_command_dismiss();
      $form_state['ajax_commands'][] = ctools_ajax_command_redirect('advance-form');
}

function advance_new_page($ajax) {
    if ($ajax) {
        ctools_include('ajax');
        ctools_include('modal');


        $form_state = array(
            'ajax' => TRUE,
            'title' => t('New Form'),
        );

        // Use ctools to generate ajax instructions for the browser to create
        // a form in a modal popup.
        $output = ctools_modal_form_wrapper('advance_new_form', $form_state);

        // If the form has been submitted, there may be additional instructions
        // such as dismissing the modal popup.
        if (!empty($form_state['ajax_commands'])) {
            $output = $form_state['ajax_commands'];
        }

        // Return the ajax instructions to the browser via ajax_render().
        print ajax_render($output);
        drupal_exit();
    } else {
        return drupal_get_form('advance_new_form');
    }
}

function changing_selected_item_status($form,&$form_state){
   // echo "<pre>";print_r($form_state['input']['list_form_items']);exit;
//    $form_state['input']['operations'];
    
    foreach($form_state['input']['list_form_items'] as $key => $value){
        if($value){
            if($form_state['input']['operations']==1){
                $num_updated = db_update('advance_form')
                    ->fields(array(
                      'status' => 1,
                    ))
                    ->condition('advance_form_id',$value , '=')
                    ->execute();
            }else{
                $num_updated = db_update('advance_form')
                    ->fields(array(
                      'status' => 0,
                    ))
                    ->condition('advance_form_id',$value , '=')
                    ->execute();
            }
            
        }
    }
    $form = drupal_build_form('advance_form_mod', $form_state);
    
    $form_html = drupal_render($form['list_form_items']);
    return $form_html;
}
//for search
function search_items($form,&$form_state){
    
    $header = array(
        'name' => array('data' => t('Name')),
        'last_name' => array('data' => t('Last name')),
        'status' => array('data' => t('Status')),
        'enable_disable' => array('data' => t('Enable/Disable')),
    );
    $query = db_select('advance_form', 'af');
    $count_query = clone $query;
    $count_query->addExpression('COUNT(af.advance_form_id)');
    $searchValue=$form_state['input']['operations'];
    //echo $searchValue;exit;
    $query = $query->extend('PagerDefault')->extend('TableSort');
    $query
        ->condition('af.name','%'.$searchValue.'%','like')
        ->fields('af', array('name', 'last_name','advance_form_id','fruite','status'))
        ->limit(5)
        ->orderByHeader($header)
        ->setCountQuery($count_query);
    $result = $query->execute();
    //echo "<pre>";print_r($result);exit;
    foreach ($result as $account) {
        $linkLabel = $account->status?'Stop':'Start';
        $changeStatusTo = $account->status?0:1;
        $options[$account->advance_form_id] = array(
            'name' => $account->name,
            'last_name' =>  $account->last_name,
            'status' =>  '<div id="current_status_'.$account->advance_form_id.'">'.($account->status?'active':'disabled'),
            'enable_disable' => array(
                  'data' => array(
                      '#markup' => "<div id='change_status_".$account->advance_form_id."'>".l($linkLabel,'change-service-status/nojs/'.$changeStatusTo.'-'.$account->advance_form_id,array('attributes' => array('class' => 'use-ajax')))."</div>",
                  ),
            ),
        );
    }
    $form['list_form_items'] = array(
        '#type' => 'tableselect',
        '#header' => $header,
        '#options' => (!empty($options))?$options:array(),
        '#prefix' => '<div id="itemlist-div">',
        '#suffix' => '</div>',
        '#empty' => t('No forms available.'),
        '#multiple' => TRUE,
      );
    /*echo "<pre>";
    print_r($form['list_form_items']);exit;*/
    $form_html = form_process_tableselect($form['list_form_items']);

    return $form_html;

}

function change_service_status($type,$status){
    if($type=='ajax'){
        drupal_add_library('system', 'drupal.ajax'); drupal_add_library('system', 'jquery.form');
        list($newStatus,$serviceID)=  explode('-', $status);
        $num_updated = db_update('advance_form')
                        ->fields(array(
                          'status' => $newStatus,
                        ))
                        ->condition('advance_form_id',$serviceID , '=')
                        ->execute();
        $showNewStatus = ($newStatus?'active':'disabled');
        $showLabel = ($newStatus?'Stop':'Start');
        $changeStatus=($newStatus?0:1);
        $commands[] = ajax_command_replace("#current_status_$serviceID", $showNewStatus); 
        $commands[] = ajax_command_replace("#change_status_$serviceID", "<div id='change_status_".$serviceID."'>".l($showLabel,'change-service-status/nojs/'.$changeStatus.'-'.$serviceID,array('attributes' => array('class' => 'use-ajax'))).'</div>'); 
        $page = array('#type' => 'ajax', '#commands' => $commands); 
        ajax_deliver($page);
    }
}

Update Status feature is not working

find the screen shots ::

Patch in 14 has been working for me for over a year now.

I concur, that patch is also working perfectly for me and has been used on half a dozen forms with tableselect AJAX inputs.

Re-setting the version back to Drupal 8. Issues must be fixed for the current development version first, then backported.

It looks like there is an 8.x patch in #12. I'm guessing it won't apply anymore, but I'll re-test it next.

#12 needs to be rerolled.

I replaced the call to drupalPostAJAX() with the new drupalPostAjaxForm()

Did the empty ajax callback ever pass the testbot? Anyway, I added a valid one.

I applied the patch of mayaz17 (#14) and it worked for me.

Patch no longer applies.

I rerolled the patch 1458824-31.patch (#31)

Tested 1458824-36.patch and it doesn't seem to work. Also it is not possible to add radios / checkboxes within FormStateInterface on #ajax callback.

According to the provided test, the AJAX works now with the patch. We use table select with checkboxes in multiple places in Core, but not combined with AJAX. I can't find anything wrong. Possibly the test could do something more than just verifying that the AJAX was executed. The array() can be updated to []. But, I don't see a specific reason mentioned in #37 why this should be "Needs work".

I updated the patch to work with the current version 7.43

For a better assert.

On second thought, the test is ok, this isn't about testing AJAX, just that AJAX is working. So RTBC #36.

Committed #36 / f6bfa62 and pushed to 8.0.x, 8.1.x and 8.2.x. Thanks!

WOOO!

Patch + Tests For 7.x

The patch should have the tests.

Patch with tests.

I am not sure why the test fails.
It works on simplytest and even in local environment.

This has got to be good for D7; It's the same fix. Can any of the original commenters confirm that this fixes their issue?

Today I was facing the same issue. I have a custom button which does some database stuff and finally adding a new element via Ajax to a tableselect list. Works great so far. Just the the checkboxes are now correctly enabled, though they are enabled. When reloading it is correctly. So abviously a refreshing bug.

I tested so many stuff. And finally I found this one here. So it's a real issue anyway. At least it was not my fault in defining the form. I just tested unset($form_state['input']['my_tableselect']); and it instantly worked.

So yeah give it a try. It will just be fixed in Drupal 8. Is this correct? Or will there also be a fix in Drupal 7?

Hope I could help someone here with Drupal 7.

Sorry @kwoxer but overzealous Mollom unpublished your posts.

Really sad. Because there was always the message, post successfully. Finally it's published. It's ok.

Maybe it was because I had a link to Stackoverflow at the very first. But now it's just the code. I think that's ok, too. :)

It will just be fixed in Drupal 8. Is this correct? Or will there also be a fix in Drupal 7?

It is committed to Drupal 8. Someone needs to test the Drupal 7 patch in comment #50 and move the issue to RTBC. Then, it may be committed to Drupal 7.

@kwoxer: Thank you for adding that solution. But, what you describe in #60 doesn't seem to be the exact same problem that this issue is about. Actually, it doesn't even seem like a bug; Seems like you just needed to add that code to unset the $form_state for your specific problem. This issue is specifically about the #ajax not even triggering.

Actually my issue was this one here: http://drupal.stackexchange.com/questions/25131/need-to-force-default-va...

But I reread the description here, that says: "When a tableselect is created with multiple = false, which means radios buttons on the left column, the #ajax attribute works fine. However with multiple = true, the #ajax attribute doesn't do anything.".

This sounds like exactly my issue. :)

As I said, the form_state did not set the checkbox states for me.

Sorry when it's the wrong issue here. But I landed here and maybe someone else with my issue.

Tested this patch and it works.
I have:

$table = array(
  '#type' => 'tableselect',
  '#header' => $header,
  '#options' => $options,
  '#ajax' => array(
    'callback' => 'my_ajax_callback',
    'wrapper' => 'my-wrapper',
  )
);

Without patch nothing happens, with patch ajax callback get's triggered.

We also use this patch on 7 and it works for us. My vote for rtbc status.

RTBC + 1, looks good to me, straightforward bug fix and if this really conflicted somewhere it can still be work-around'ed.

Marked for commit.

Committed and pushed to 7.x! Thanks!