Now that #1974210: Convert admin/structure/forum to a Controller is in, we can use the getCancelRoute() method as described in the DeleteForm todo.
Patch follows.

CommentFileSizeAuthor
#2 2202151-forum-deleteform-2.patch964 byteslongwave
#1 2202151-use-getCancelRoute-for-forum.patch807 bytesBarisW
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BarisW’s picture

BarisW CreditAttribution: BarisW commented
FileSize
2202151-use-getCancelRoute-for-forum.patch807 bytes
longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave commented
FileSize
2202151-forum-deleteform-2.patch964 bytes

We can simplify buildForm() a bit further.

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan commented
Status: Needs review » Reviewed & tested by the community

Thanks

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed e378459 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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