- Issue created by @nicxvan
- π¨πSwitzerland berdir Switzerland
FWIW, per my comment in the previous issue, I'm not sure we should do this at all, despite being the person who originally suggested this.
My idea was that we could/would use hierarchy for those attribute classes, so it would be FormAlter extends Alter extends Hook. However, this would complicate parsing of attribute classes by api.module, so we might not want to do that.
And Alter on its own is not a hook, so that might be a bit confusing. I think we should probably only do it as an abstract class that you can't use directly *if* we want to support hierarchy like that.
Otherwise we end up one more pattern that you have to look for. Right now, it's just
#[Hook('form_FOO_alter')]
, the linked issue also means you have to look for#[FormAlter('FOO')]
, adding this means it could also be#[Alter('form_FOO')]
. - π¨π¦Canada Charlie ChX Negyesi πCanada
About api module, that's my bad, originally I said I do not know how it will fall out but api already has a "class hierarchy" option which we can reuse.