Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
On second thought, data alter callbacks could use the ability to have configuration forms, too.
Since this is most consistently achieved by making them objects too and proceeding as for services and processes, we should take that approach.
E.g., the "Fulltext field" callback could use some configuration as to which fields to accumulate, instead of just choosing by type. It could also use some configuration as to if and how to respect field boosts.
Comment | File | Size | Author |
---|---|---|---|
#4 | search_api.callback-modules.patch | 28.28 KB | drunken monkey |
#3 | search_api.callback-modules.patch | 28.29 KB | drunken monkey |
#1 | search_api.callback-modules.patch | 2.94 KB | drunken monkey |
Comments
Comment #1
drunken monkeyThe attached patch seems to work, gonna commit it about tomorrow.
Comment #2
fagoI think your patch misses the file containing SearchApiAlterCallbackInterface and the classes implementing it?
Comment #3
drunken monkeyOops, sorry! Must have clicked on just the .api.php file, instead of the whole project …
This should be the real thing.
Comment #4
drunken monkeyAnd here's one that should actually save/use the settings correctly …
Comment #5
drunken monkeyCommitted (plus a few small fixes).