Simplify Xapian search configuration
| Project: | xapian |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
The wording and terms used in the Xapian search configuration page are different from those used by Drupal's core search functionality, and as a result it's confusing when first using the module.
The Xapian search module is also confusing because it considers all indexing "reindexing", which is not always the case.
This patch attempts to cleanup the search configuration page, adding descriptions and making it much more similar to the Drupal core search configuration page. It also removes an extra cron-related option -- cron now simply checks if there are items queued, and if there are it indexes them.
The word "reindex" is being replaced with "index" to simplify the language.
The configuration options were reorderd, moving the performance settings higher on the settings page on the theory that it's a more important configuration option.
The reindex option now has a confirmation, explaining the impact of reindexing the site.
All wording is being reworked with the plan to support more than just nodes.
| Attachment | Size |
|---|---|
| xapian.module.patch | 8.67 KB |
#1
Shortening issue title...
#2
Patch updated so that immediate indexing is actually immediate, and queued indexing is actually queued. Also, replaced TRUE/FALSE switch with simpler if/else.
#3
I hadn't noticed my first read through that the # of items waiting to be indexed disappears depending on the configuration -- I removed this logic so it is always displayed. (Even if indexing immediately, when you re-index the site it is queued up for cron).
I also reworded the re-index confirmation page to make it more clear. (It's more complex than standard Drupal search due to the ability to index immediately.)
#4
#5
Automatically closed -- issue fixed for two weeks with no activity.