There is a no user_update_index() in the user module, so this shouldn't be a default for updating on cron. This is cleanup, and would confuse anyone just trying to understand what this module is doing.

CommentFileSizeAuthor
#1 search-bad-default-2068001-1.patch539 bytesrobbiethegeek

Comments

robbiethegeek’s picture

robbiethegeek commented
Status: Needs review » Active
StatusFileSize
new search-bad-default-2068001-1.patch539 bytes

Tested on simplytest.me after generating nodes. Search still works for nodes.

steven merrill’s picture

steven merrill commented
Status: Active » Needs review

Looks good to me - user_update_index() is no longer present in core. Let's let the bot check it out.

steven merrill’s picture

steven merrill commented
Status: Active » Reviewed & tested by the community

I tried this on simplytest.me with no ill effects.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Closed (works as designed)

This is not necessary, and besides which we would need to do it in Drupal 8.x first if we were to accept this patch. It's only causing a very minor bit of overhead, if any, and is only setting up a default for the variable_get(), which will almost certainly never get used because that variable should always be set.