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.
Problem/Motivation
Currently there's no way to target all "confirm" forms, especially for theming confirm-form.html.twig
when the only variable to work with is {{ form }}
. This requires altering the form directly.
Proposed resolution
Implement BaseFormIdInterface
on ConfirmFormBase
and provide a default confirm_form
base_form_id. It can still be overridden.
Remaining tasks
- Patch
- Tests
User interface changes
None
API changes
Technically none if ConfirmFormBase
provided a default value.
Data model changes
None
Comment | File | Size | Author |
---|---|---|---|
#2 | confirmformbase_should-2800913-2.patch | 1.02 KB | markhalliwell |
Comments
Comment #2
markhalliwellComment #3
dawehnerGood idea!
Comment #16
smustgrave CreditAttribution: smustgrave at Mobomo commentedIf this is still a valid feature request it will need tests.