Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Part of #1945406: [meta] Convert all of confirm_form() to ConfirmFormBase broken out of #1946462: Convert content_translation_translatable_form() to the new form interface where this was originally considered to be included but then removed.
Comment | File | Size | Author |
---|---|---|---|
#2 | ct-2086479-2.patch | 7.02 KB | tim.plunkett |
Comments
Comment #1
disasm CreditAttribution: disasm commentedduplicate of #1987882: Convert content_translation routes to a new style controller.
Comment #2
tim.plunkettI really want confirm_form() dead (it's one of the last places using $_GET!)
Comment #3
dawehnerEven i kind of hate to force other people to reroll patches for the sake of own patches.
This is fine given that this is just temporary anyway.
Comment #4
tim.plunkettYeah I was halfway done injecting it when I realized it was a @todo and would be easier to fix later if left as-is.
Comment #5
tim.plunkettComment #6
webchickCommitted and pushed to 8.x. Thanks!