In particular I used modify_query to add a field to the $params['fl'] and was disappointed that the field wasn't automatically available to me in the search results. It was, in fact, available, but only through the badly named 'node' element of the array which contained the Apache_Solr_Document. I could have used the $result['node']->getField('blah'); method but imo this assumes too much knowledge of the phpclient library.

To make it easier to deal with this patch does two things.

1. renames 'node' to 'document'. Note that this creates some mismatch between the array and the theme layer. If it causes problems we have to address the theme layer.

2. iterates over the Apache_Solr_Document and makes an array of its fields called $result['fields'] for easy access.

Comments

robertdouglass’s picture

StatusFileSize
new879 bytes

Here's a version without renaming node->document. A little less disruptive.

robertdouglass’s picture

Version: 6.x-2.x-dev » 6.x-1.x-dev

Committed to DRUPAL-6--2. Marking to be ported.

robertdouglass’s picture

Status: Needs review » Patch (to be ported)
claudiu.cristea’s picture

Version: 6.x-1.x-dev » 5.x-2.x-dev
StatusFileSize
new2.3 KB

Here's a patch for 5.x-2.x-dev.

Note that I've also moved the comment count generation ($extra[] = format_plural($doc->comment_count, '1 comment', '@count comments');) above drupal_alter('apachesolr_search_result', $doc); (as is in 6.x-2.x-dev) so modules can alter also that value. I think this should also be fixed in 6.x-1.x-dev.

robertdouglass’s picture

Claudiu - better not to mix up two things in one patch. Can you start a new issue with the shuffling of the $extra and drupal_alter?

claudiu.cristea’s picture

StatusFileSize
new2.56 KB

Sure... Here's the patch for 5.x-2.x-dev

claudiu.cristea’s picture

Version: 5.x-2.x-dev » 6.x-1.x-dev

Committed in DRUPAL-5--2, #317518

Switched for backporting in 6.x-1.x-dev.

pwolanin’s picture

I don't think you actually have to use the getField() method.

pwolanin’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new1.12 KB
pwolanin’s picture

Status: Needs review » Fixed

committed to 6.x-1.x

Status: Fixed » Closed (fixed)

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