Currently we use $index->getFields() in SearchApiSolrService::search() to get the indexed fields, unfortunately this doesn't know / return search_api_id and search_api_relevance.
Currently we already handle this for field names, see $index->getFieldNames().
The missing entries in $index_fields causes errors e.g. when grouping by one of the missing fields - the group handling does this $index_fields[$collapse_field]['type'] - which will throw a notice.
Manually add the fields to the array $index_fields, or add a getFields() method which works similar to getFieldNames() and encapsulates this special knowledge.
Reviews needed.
None.
None.
| Comment | File | Size | Author |
|---|---|---|---|
| search_api_solr-add-search_api_id-and-relvane-to-known-fields.patch | 696 bytes | das-peter |
Comments
Comment #1
drunken monkeyHm, while using these special fields for grouping isn't explicitly supported, if it works with just these few lines I guess we can just as well support it.
The patch also looks good, thanks a lot! I don't think an extra method is needed, we only get the index's fields this one time in the whole class anyways.
So, committed. Thanks again!