Currently, we have two hook implementations (hook_mail() and hook_search_api_saved_search_presave()) in our module which exist not for the module itself, but just for the purpose of the "E-mail" notification plugin. (We have the same situation in the Search API regarding the "Content entity" datasource, actually.)
I think it would be cleaner to clearly separate them from the "normal" module code – either by having them in a separate include file (e.g., search_api_saved_searches.email_hooks.inc), or by moving just their function bodies to new static methods on the Email plugin class.
Input (or patches) welcome.
Fixed
1.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Issue #2955519 by drunken monkey: Fixed indistinct separation of module...
Automatically closed - issue fixed for 2 weeks with no activity.