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.

Comments

pwolanin’s picture

Status: Active » Needs work
StatusFileSize
new39.58 KB

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

pwolanin’s picture

StatusFileSize
new35.39 KB

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

pwolanin’s picture

Status: Needs work » Needs review
StatusFileSize
new38.94 KB

This seems to mostly be working now.

pwolanin’s picture

StatusFileSize
new39.62 KB

some more fixes.

pwolanin’s picture

StatusFileSize
new43.15 KB

More complete, makes more use of auto-loading.

Does all query string encoding with rawurlencode().

pwolanin’s picture

StatusFileSize
new48.06 KB

simplify to just using URL in the constructor.

not fully tested yet.

pwolanin’s picture

StatusFileSize
new53.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

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.