Part of #2072251: [meta] Modernize forms to use FormBase

Updated: Comment #0

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(). Both ConfigExportForm and ConfigImportForm fit the bill.

Proposed resolution

Refactor ConfigExportForm and ConfigImportForm to extend FormBase

Remaining tasks

N/A

User interface changes

N/A

API changes

N/A

N/A

CommentFileSizeAuthor
#1 config-2071439-1.patch5.58 KBtim.plunkett

Comments

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Active » Needs review
StatusFileSize
new config-2071439-1.patch5.58 KB
jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Status: Needs review » Reviewed & tested by the community

This is simple swap for t to $this->t so I think it is safe to RTBC.

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Add link to meta issue