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.
Part of #2072251: [meta] Modernize forms to use FormBase
Updated: Comment #N
Problem/Motivation
Now that #2059245: Add a FormBase class containing useful methods is in, I was looking at old forms that have empty validateForm() methods, or still use Drupal::service() or t().
Proposed resolution
Convert existing FormInterface forms to extend FormBase
Remaining tasks
User interface changes
API changes
Related Issues
Comment | File | Size | Author |
---|---|---|---|
#5 | modernize_file_module-2077977-5.patch | 4.54 KB | nielsonm |
#4 | drupal8-file-form-2077977-4.patch | 4.04 KB | er.pushpinderrana |
Comments
Comment #1
plopescFirst round...
Comment #4
er.pushpinderrana CreditAttribution: er.pushpinderrana commentedUnable to apply above patch. Please review updated patch.
Comment #5
nielsonm CreditAttribution: nielsonm at Phase2 commentedRerolled patch
Comment #6
xjmComment #7
alexpottNeeds a reroll.