The DrupalApacheSolrService class is already overriding many of the base class methods, and adds a bunch more. For simplicity sake, let's take advantage of the BSD license and subsume it so we odn't have an external library dependency.

CommentFileSizeAuthor
#7 1107502-merge-7.patch53.21 KBpwolanin
#6 1107502-merge-6.patch48.06 KBpwolanin
#5 1107502-merge-5.patch43.15 KBpwolanin
#4 1107502-merge-4.patch39.62 KBpwolanin
#3 1107502-merge-3.patch38.94 KBpwolanin
#2 1107502-merge-2.patch35.39 KBpwolanin
#1 1107502-merge-1.patch39.58 KBpwolanin

Comments

pwolanin’s picture

pwolanin commented
Status: Active » Needs work
StatusFileSize
new 1107502-merge-1.patch39.58 KB

This is not working yet, but a good ways along into the merge.

pwolanin’s picture

pwolanin commented
StatusFileSize
new 1107502-merge-2.patch35.39 KB

oops - that had a funny diff due to tab to space changes. This is better.

pwolanin’s picture

pwolanin commented
Status: Needs work » Needs review
StatusFileSize
new 1107502-merge-3.patch38.94 KB

This seems to mostly be working now.

pwolanin’s picture

pwolanin commented
StatusFileSize
new 1107502-merge-4.patch39.62 KB

some more fixes.

pwolanin’s picture

pwolanin commented
StatusFileSize
new 1107502-merge-5.patch43.15 KB

More complete, makes more use of auto-loading.

Does all query string encoding with rawurlencode().

pwolanin’s picture

pwolanin commented
StatusFileSize
new 1107502-merge-6.patch48.06 KB

simplify to just using URL in the constructor.

not fully tested yet.

pwolanin’s picture

pwolanin commented
StatusFileSize
new 1107502-merge-7.patch53.21 KB

Test and fix the settings form for servers. Trying to keep all the url parts separate seemed an unneeded hassle given how rarely this will be visited.

pwolanin’s picture

pwolanin commented
Status: Needs review » Fixed

oops - committed and pushed together with #899590: CCK facet definitions needs to expand its role and have a 'facets' param. instead of on its own.

Status: Fixed » Closed (fixed)

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