prevent toc from appearing in teaser lists

The site I'm currently working on is using the except module so this doesn't really come up. But, this should probably be dealt with. I don't know if it's possible to change your input filter based on if its a teaser or not, anyone have any thoughts?

--Andrew

It's pretty simple though:

function tableofcontents_nodeapi(&$node, $op, $a3 = NULL, $a4 = NULL) {
  switch($op) {
    case 'submit':
      $arr = array_keys(filter_list_format($node->format));
      if (in_array('tableofcontents/0', $arr)) {
        if ($node->teaser) {
          // Remove toc from teasers
          $node->teaser = preg_replace('!<\!-- ?tableofcontents(.*)-->!', '', $node->teaser);
        }
      }
      break;
  }
}

submit event is fired when a node is stored in the database.

This looks like a reasonable solution. Please roll a patch and I'll be glad to test / apply. The regex should probably be defined somewhere so it can be easily changed.

--Andrew

Here is a solution from casey's zip file:

function tableofcontents_nodeapi(&$node, $op, $a3 = NULL, $a4 = NULL) {
  switch($op) {
    case 'submit':
      if (variable_get("tableofcontents_noteaser_$format", 1)) {
        $arr = filter_list_format($node->format);
        if (isset($arr['tableofcontents/0'])) {
          if ($node->teaser) {
            // Remove toc from teasers
            $node->teaser = preg_replace('/<!-- ?tableofcontents(.*)-->/', '', $node->teaser);
          }
        }
      }
      break;
  }
}

This needs to be written up as a proper patch. I'd like to postpone this until 296171: Change Defaults? lands, as it involves a new setting. I'm going to suggest that this setting not be offered on a per-node basis, as it's almost guaranteed that you don't want the table of contents showing up in teasers.

Here are patches for Drupal 5 and Drupal 6. The Drupal 5 version works as expected. However, the Drupal 6 version has some additional code to handle the D6 split teaser feature. I'm pretty happy with the D5 version, but I'd appreciate some additional testing for D6 before committing.

Thanks!
--Andrew

Drupal 5 version works well for me, thanks.

Thanks for the testing. I've applied this patch to both versions in commits #158739 and #158740.