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.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 1107502-merge-7.patch | 53.21 KB | pwolanin |
| #6 | 1107502-merge-6.patch | 48.06 KB | pwolanin |
| #5 | 1107502-merge-5.patch | 43.15 KB | pwolanin |
| #4 | 1107502-merge-4.patch | 39.62 KB | pwolanin |
| #3 | 1107502-merge-3.patch | 38.94 KB | pwolanin |
Comments
Comment #1
pwolanin commentedThis is not working yet, but a good ways along into the merge.
Comment #2
pwolanin commentedoops - that had a funny diff due to tab to space changes. This is better.
Comment #3
pwolanin commentedThis seems to mostly be working now.
Comment #4
pwolanin commentedsome more fixes.
Comment #5
pwolanin commentedMore complete, makes more use of auto-loading.
Does all query string encoding with rawurlencode().
Comment #6
pwolanin commentedsimplify to just using URL in the constructor.
not fully tested yet.
Comment #7
pwolanin commentedTest 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.
Comment #8
pwolanin commentedoops - committed and pushed together with #899590: CCK facet definitions needs to expand its role and have a 'facets' param. instead of on its own.