Hello,

At the moment, the module only indexes the last file on a multiple files field.

Below patch fixes that problem.

Laurent_
Agence Web Coheractio

CommentFileSizeAuthor
#10 apachesolr_attachments-multiple-fields-1904534-10.patch732 bytesgabesullice
#3 apachesolr_attachments-multiple-fields-1904534-3.patch1.13 KBbburg
#2 apachesolr_attachments-multiple-fields-1904534-2.patch944 bytesagence web coheractio
#1 apachesolr_attachments-multiple-fields-1904534-1.patch463 bytesagence web coheractio

Comments

agence web coheractio’s picture

agence web coheractio commented
Status: Active » Needs review
StatusFileSize
new apachesolr_attachments-multiple-fields-1904534-1.patch463 bytes

Patch attached

agence web coheractio’s picture

agence web coheractio commented
Title: Module only indexin the last file » Module only indexing the last file
StatusFileSize
new apachesolr_attachments-multiple-fields-1904534-2.patch944 bytes

Sorry wrong patch

bburg’s picture

bburg
he/him
Location Washington, DC
commented
StatusFileSize
new apachesolr_attachments-multiple-fields-1904534-3.patch1.13 KB

Testing this out myself. Here is a 7.x-1.3 version of the patch. Will respond here on how this goes for me.

bburg’s picture

bburg
he/him
Location Washington, DC
commented

Seems to work for me!

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented

Looking good - If someone sets this to RTBC, I'd be happy to commit it

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented

However, I'd advice you to take a look at this issue :

https://drupal.org/node/1857240 and see if that solves your use case also. As I think that solution is more generic and would work equally well.

bburg’s picture

bburg
he/him
Location Washington, DC
commented
Status: Needs review » Reviewed & tested by the community

Done, I've already moved on to other tasks in my current project, but will definitely keep this in mind next time I come across this issue.

jamiehollern’s picture

jamiehollern
he/him
Primary language English
Location Alba/Scotland
commented
Status: Reviewed & tested by the community » Needs work

Patch in #3 doesn't work for me. It fails and says it does not apply.

gabesullice’s picture

gabesullice
Primary language English
commented
Version: 7.x-1.2 » 7.x-1.3

Having the same issue. The patches don't apply and making the changes manually doesn't work either.

gabesullice’s picture

gabesullice
Primary language English
commented
StatusFileSize
new apachesolr_attachments-multiple-fields-1904534-10.patch732 bytes

Patch for 7.x-1.3

jamiehollern’s picture

jamiehollern
he/him
Primary language English
Location Alba/Scotland
commented
Status: Needs work » Reviewed & tested by the community

Gabe, you're a legend! The patch in #10 works perfectly. Marking as RTBC.

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Reviewed & tested by the community » Fixed

This patch looks good. I do want to keep the efq_extra_fields module in sync with this one, but perhaps this patch is even better compared to the efq_extra_fields one. Committed this and I'll follow up later.

Status: Fixed » Closed (fixed)

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