Closed reinhardt closed 12 years ago
I added the patch for unrestrictedSearchResults in db7afe2ca64c091f6faf5e25b9b6be467896a8bb - and fixed it to fall back on the unrestrictedSearchResults method of the catalog instead of searchResults.
Dispatching reindexObject is actually "wrong". This is a ZCatalog API, and we cannot simply ignore updating ZCatalog. Moving all indexes over to Solr is also not supported. You need at least the UID index in the ZCatalog.
But in general you probably don't want to loose transactional integrity for most queries, except for full text search.
The methods
CatalogTool.unrestrictedSearchResults() CatalogTool.reindexObject()
are not patched by collective.solr and thus work on the standard ZCatalog. Some methods, e.g. reindexObjectSecurity(), rely on them to reindex objects. When clearing the standard Catalog and using solr for all indexes, this fails silently because said methods get no results from ZCatalog and don't reindex anything.
By patching the methods like this we were able to fix the issue: