More Granular cck_field_mappings, Please

whoops, looks like my patch didn't attach...here it is...

A couple of additions:

I've added a callback to override search parameters that were previously hard-coded. If no search params hook is called the search will use the module defaults you provided.

The CCK mapping also now includes an additional dimension (facet_display) that allows modules to provide custom display settings on a per-facet basis to override the default settings.

With field-level CCK mappings, per-facet display settings and search parameter overrides, I'm just about to Solr nirvana.

The only other modification that I'm contemplating right now is to decouple the Solr block item links from the core search module implementation.

I very much appreciate that you've tied ApacheSolr to core search and allow for a default solr search UI there. But I'd also like to build up multiple search UIs on a per URL basis (ex URLS that don't include module implementation details like "apachesolr_search")...that are not tied directly to core search. In order to do that, I'd need links w/in ApacheSolr-provided blocks to be configurable as well.

But, with the current additions, the callbacks look like:

<?php
/*
* Implementation of hook_cck_field_mappings()
* See apacesolr module
*/
function apachesolr_cite_apachesolr_cck_field_mappings() {
  $mapping = array();
  $mapping['field_creator'] = array(
    'index_type' => 'text',
    'facet' => TRUE,    
    );
  $mapping['field_subject'] = array(
    'index_type' => 'text',
    'facet' => TRUE,
    'callback' => 'explode_cck_field',
    'multiple' => TRUE,
    'facet_display' => array(
      'facet.limit' => 2,
      'facet.mincount' => 1,
      'facet.sort' => 'false',
      ),
    );
  }
      
  return $mapping;
}

function apachesolr_cite_apachesolr_search_params($params) {
  $params['rows'] = 10;
  return $params;
}
?>

Thanks.

Hi Robert,

Thanks for considering this patch. Sorry for the delay in my response, I just now saw your note. I downloaded and tested the patch against alpha three and the fields were indexed as expected.

I should have specified that I only tested my patch against 5.x-1.0-alpha3. I'll test 6.x-1.0-alpha3 in the next day or so.

Ok, no problem - I'll take a look at both D5 & D6 tomorrow.

The indexing hook failed on DRUPAL-5/dev version - $key was not being properly set in ApacheSolrUpdate. The attached mapping patch now includes a fix for the related $key variable issue. This patch removes the hard-coding of $document variables:

  $document->nid = $node->nid;
  $document->uid = $node->uid;
  $document->title = $node->title;
  $document->body  = $node->body;
  $document->type  = $node->type;
  $document->changed = $node->changed;
  $document->comment_count = $node->comment_count;
  $document->name = $node->name;
  $document->language = $node->language;

in favor of the dynamic approach taken in alpha3:

$fields = array('title', 'body', 'type', 'uid', 'changed', 'nid', 'comment_count', 'name');
foreach ((array)$node as $key => $value) {
  if (in_array($key, $fields)) {
    $document->$key = $value;
  }
...