- Issue created by @berdir
- 🇨🇭Switzerland berdir Switzerland
Created a quick proof of concept.
All form callbacks go through FormState::getCallback(), which is nice in that it gives us a single place to adjust, but FormState isn't something we can inject, so we have to use Drupal::service() which I absolutely expect will break some unit tests.
Might want to deprecate this and add something else.
- 🇦🇺Australia kim.pepper 🏄♂️🇦🇺Sydney, Australia
kim.pepper → made their first commit to this issue’s fork.
- 🇦🇺Australia kim.pepper 🏄♂️🇦🇺Sydney, Australia
Added 📌 Move file form hooks to FileFormHooks class Active for the hooks cleanup mentioned above.
- 🇦🇺Australia kim.pepper 🏄♂️🇦🇺Sydney, Australia
Fixed the failing test by handling the exception. Not sure if this is the right place to handle it.
- 🇦🇺Australia kim.pepper 🏄♂️🇦🇺Sydney, Australia
📌 Move file form hooks to FileFormHooks class Active landed.
The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
