Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.The core Drupal search module indexes comments along with the node title and body. It also allows other modules to alter the node before it indexes it.
This patch updates node_xapian_index() to first allow other modules to change the node before it is indexed. It also invokes the nodeapi's 'update index' operator, allowing other modules to attach node content to the search indexes. In particular, this causes comments to get indexed along with the node contents.
It also makes xapian_reindex_queued_nodes a little more consistent with the core search code using an object instead of an array when indexing nodes.
It also implements the _comment hook, so nodes are reindexed as comments are added/updated/deleted/published/unpublished.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | xapian.module.patch | 2.95 KB | jeremy |
| xapian.module.patch | 2.54 KB | jeremy |
Comments
Comment #1
jeremy commentedDoing test searches, I'm seeing some blank results come back ever since this patch. I need to review closer what has broken -- updating the issue status to note that this patch does not appear to be ready for prime time yet.
Comment #2
jeremy commentedThe blank results I was getting was because I wiped my Drupal database, but neglected to wipe my Xapian database too. Once I did that, everything was working perfectly again - so it was my fault, not a bug in the code.
Here's an updated version of the patch. In addition to the above, it also prevents the same node from being added to the index queue twice -- that's just wasted effort.
Comment #3
singularoComment #4
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.