To make sure MB knows I enabled my module, I modified "theme_multiblock_general" to show the enabled status. This might be useful information for other developers.

function theme_multiblock_general($add_block_form, $multiblocks) {
  $output = '';
  $noyes = array('No', 'Yes');
  
  $output .= '<p><h3>'. t('Add Instance') .'</h3>'. $add_block_form .'</p>';
  
  $header = array(t('Title'), t('Original Block Title'), t('Original Module'), t('Enabled'), t('Original Delta'), t('Action'));
  
  foreach ($multiblocks as $row) {
    $delete_link = l(t('Delete'), 'admin/build/block/instances/delete/'. $row->delta);
    $title = multiblock_get_block_title($row->module, $row->orig_delta);
    $mb_enabled = $noyes[$row->multi_settings];
    $rows[] = array(check_plain($row->title), $title, $row->module, $mb_enabled, $row->orig_delta, $delete_link);
  }
  
  $output .= '<p><h3>'. t('Manage Instances') .'</h3>'. theme('table', $header, $rows) .'</p>';
  
  return $output;
}

Comments

nancydru’s picture

Fix committed to DRUPAL-5 and DRUPAL-6--1 branches.

nancydru’s picture

Status: Needs review » Fixed
andrewlevine’s picture

Status: Fixed » Active

I think the title "Enabled" might be a little confusing, maybe "MultiBlock Enabled" is a little better? That still is even a little confusing because my guess is 80% of users won't know what that means. Let me know what you think. Useful feature anyways though, thanks for adding it.

nancydru’s picture

How about "Module enabled"?

andrewlevine’s picture

I'm not sure that would be accurate because it doesn't have to do with whether the module is enabled or not. How about "Saves Per-block Settings"?

nancydru’s picture

Actually, I've come back to your original suggestion because it pretty much says what the README and handbook page says. It will go in with the title editing fix tomorrow.

nancydru’s picture

Status: Active » Fixed

Fix committed on both branches. Plus the "yes/no" is changed to check mark or red X.

andrewlevine’s picture

Status: Fixed » Active

I think we need to run the check/X images through theme_image so it corrects the path in the case that the Drupal site isn't at the root.

nancydru’s picture

Status: Active » Fixed

Committed on both branches.

Status: Fixed » Closed (fixed)

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