when I have a view linked to an index, if I revert the index, I get the message:
[warning] The following views were using the index es_node_index: search_es. You should disable or delete them.
This message should be displayed only if the index is deleted.
in fact, while reverting the index, the process execute the same code, so a test should be added to prevent this message
A revert flag is already set by the hook search_api_search_api_index_delete().
So my solution is to be sure that the hook search_api_views_search_api_index_delete() is called after the previous one.
Then this hook should test the revert flag before checking the index.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 2827717-3--index_revert_views_warning.patch | 693 bytes | drunken monkey |
| #2 | warning_revert_index-2827717-1.patch | 1.39 KB | Fabien.Godineau |
Comments
Comment #2
Fabien.Godineau commentedComment #3
drunken monkeyThe check is just a single method call, I don't think avoiding to do that twice is really worth mucking around with
hook_module_implements_alter().Comment #4
Fabien.Godineau commentedI agree with you. It is simpler.
Thank you
Comment #5
drunken monkeyGood to hear, thanks for the feedback!
Committed.