option to exclude certain nodetypes from search indexing

what would be the point of that?

if you use a node_access module like node_privacy_by_role then only the allowed results will turn up in searches.

I have content in nodes that is not restricted to any particular set of users, it's meant to be available to all. But I have created a few node types that support content in other nodes, they are "child" nodes to various "parent" nodes.

These nodes do not contain lots of specific content (text) but place graphics and such via JavaScript and misc URIs, there is also some stock mumbo jumbo about not having JavaScript on and Please turn it on... I thought the built in Search feature of Drupal was very cool until I noticed it was turning up all of those support nodes.

I still think it's cool. I just wished I knew how to tell the Search module interface to ignore certain node types. So for now I just empty the Search tables after I run cron and before backing up. There may be a not-too-hard way to override a function in search.module, but...I'm not at that point yet.

So from my standpoint too, I'd love to see this as a feature, a row/list of all one's nodes next to checkboxes at the bottom of Administer -> Search, or maybe it's easier to program (but harder to administer) Administer -> Content Types -> then set a radio button in each content type to "Index for Search" or "Ignore for Search".

I downloaded and read about "node_privacy_byrole.module" but it doesn't stop the "problem" where it happens, a huge amount of search results for nodes only meant to be accessed by their parent node. One can think of it in terms of efficiency as well. Why weigh the index down with stuff worth overlooking? :-)

btw I'm using the stock search.module with 4.7.3. So I bumped the version down to what I'm using. I guess that's how I'm supposed to do things...

-Drew

This a new feature and hence won't go in until Drupal 6 (as Drupal 5 is frozen).

However, you can hack this in by overriding the "types" selection. In node_search() in node.module, add the following two lines (change 'book,page,story' to your needs).

      $keys = search_query_insert($keys, 'type', 'book,page,story'); ///// Add this line
      if ($type = search_query_extract($keys, 'type')) {
        $types = array();
        foreach (explode(',', $type) as $t) {
          $types[] = "n.type = '%s'";
          $arguments1[] = $t;
        }
        $conditions1 .= ' AND ('. implode(' OR ', $types) .')';
        $keys = search_query_insert($keys, 'type');
      }
      $keys = search_query_insert($keys, 'type', ''); ///// Add this line

This will break the existing type selector though. You can edit node_form_alter() and hide it from the form by commenting out:

    $form['advanced']['type'] = array(
      '#type' => 'checkboxes',
      '#title' => t('Only of the type(s)'),
      '#prefix' => '<div class="criterion">',
      '#suffix' => '</div>',
      '#options' => $types,
    );

Steven, thanks. I'll see if I can incorporate your modification.

no new features go into 4.7

see D6 patch in http://drupal.org/node/111744