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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drunken monkey’s picture

Status: Active » Needs review
FileSize
2.94 KB

The attached patch seems to work, gonna commit it about tomorrow.

fago’s picture

Status: Needs review » Needs work

I think your patch misses the file containing SearchApiAlterCallbackInterface and the classes implementing it?

drunken monkey’s picture

Status: Needs work » Needs review
FileSize
28.29 KB

Oops, sorry! Must have clicked on just the .api.php file, instead of the whole project …
This should be the real thing.

drunken monkey’s picture

And here's one that should actually save/use the settings correctly …

drunken monkey’s picture

Status: Needs review » Fixed

Committed (plus a few small fixes).

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.