In apachesolr_fields_default_indexing_callback(...) we take a copy of the first value of the field and store it in a singular index. This works great for multi-value fields ($field_info['multiple'] == true), but results in us trying to index the same field twice with the same name if the field is already singular.

My patch add a check for $field_info['multiple'] to prevent this and makes the code a little easier to read by moving code that's designed to execute 0 or 1 times out of the for loop.

Comments

ianthomas_uk’s picture

Status: Active » Needs review
StatusFileSize
new1001 bytes

And here's the patch (my last for now)

nick_vh’s picture

StatusFileSize
new1.19 KB

I had some whitespace issues with your patch

nick_vh’s picture

Version: 7.x-1.x-dev » 6.x-3.x-dev
Status: Needs review » Patch (to be ported)

committed to 7.x-1.x

ianthomas_uk’s picture

Assigned: ianthomas_uk » Unassigned
nick_vh’s picture

Status: Patch (to be ported) » Fixed

could be cleany applied and works

Status: Fixed » Closed (fixed)

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