Callback handlers have changed in 6-x-2-dev solr attachments does not support this currently. I realise that this might be postponed for now as a feature, but patch attached incase anyone else is having the issue I did. This patch add a the new hook hook_apachesolr_document_handlers and corrects the signature of apachesolr_attachments_add_documents to the new format.

CommentFileSizeAuthor
solr-attach-solr-2x.patch7.49 KBmarcingy

Comments

pwolanin’s picture

pwolanin commented
Version: 6.x-2.0-alpha1 » 6.x-2.x-dev

I just switch the active development here to 6.x-1.x to reduce possible confusion.

I'm not really eager to maintain two branches at this point- I'll see if Robert wants the challenge, unless you want to do it.

marcingy’s picture

marcingy commented

I fully understand that and hence the reason for marking this as a feature request. I think I could find time to support a branch that works with/tracks the 6-2-x-dev version of solr as any solr development I do use that version and document searching is becoming a more and more common request for clients.

robertdouglass’s picture

robertdouglass commented

I'm willing to help maintain a 6.2 branch, but if I have my way the 6.2 branch will now start to diverge more from the 6.1 apis, so it might be a challenge. A good challenge, though. =) @marcingy - can you take the lead with the 6.2 branch here? The patch is a good start.

marcingy’s picture

marcingy commented

I'm willing to take the lead with 6.2 branch. A challenge is always interesting ;)

pwolanin’s picture

pwolanin commented

ok, well I'll make sure you are both on the CVS access list. I think HEAD is in sync with the 6--1 branch, so you can start with that for future 6.x-2.x code. Let's leave that marked as unsupported a while longer to nudge people to switch over.

marcingy’s picture

marcingy commented
Assigned: Unassigned » marcingy
marcingy’s picture

marcingy commented

@pwolanin do you want me to commit to head and then start 6-x-2-dev or create a 6-x-2-dev branch and go from there? I suppose what I'm trying to say is do we need to keep head in sync with 6-x-1 or can it now diverge?

marcingy’s picture

marcingy commented
Status: Needs review » Fixed

Committed to head.

Status: Fixed » Closed (fixed)

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