Don't offer non-indexed fields as search options

pwolanin - July 2, 2009 - 16:02
Project:Apache Solr Search Integration
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

We added the teaser fields as a non-indexed 'text' field. This may have been a mistake - it probably should just be a string field. But in any case, in function apachesolr_search_settings_form() we are giving the use the option to search all text fields. We need to add a check to only offer indexed text fields.

#1

pwolanin - July 2, 2009 - 16:12
Status:active» needs review

most of the change in the patch is indentation - the main difference is an added if() {}

AttachmentSize
indexed-only-508364-1.patch 1.9 KB

#2

pwolanin - July 2, 2009 - 16:16

Minor changes to code comments.

AttachmentSize
indexed-only-508364-2.patch 2.67 KB

#3

pwolanin - July 2, 2009 - 19:13
Title:Don't offer non-indexed feilds as search options» Don't offer non-indexed fields as search options

#4

pwolanin - July 2, 2009 - 22:00
Status:needs review» fixed

committed

#5

System Message - July 16, 2009 - 22:00
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.