#899480: Limit search to same content type changed a bunch of Solr options that should be tested by someone using Solr...
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 1130596-uniqueness-solr-nids.patch | 783 bytes | dandaman |
#899480: Limit search to same content type changed a bunch of Solr options that should be tested by someone using Solr...
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 1130596-uniqueness-solr-nids.patch | 783 bytes | dandaman |
Comments
Comment #1
dandaman commentedSeems to work pretty well, except for the URLs. The array of results seems to be returned with generic numeric arrays while this code seems to think that the results should be keyed by nid. Thus, this is line 321 of uniqueness.module:
The URLs on my results were "node/0", "node/1", "node/2", "node/3" and so on.
Comment #2
dandaman commentedActually, looks like they are keyed correctly with the nid at some point, but the array_merge() function wipes them out here (line 439 of uniqueness.module):
Maybe we should use the "+" operator instead of "array_merge()" in both of these cases?
Comment #3
dandaman commentedYeah, that worked for me. Here's a patch:
Comment #4
bforchhammer commentedThanks for testing. Patch looks fine, commited.