Follow-up from #1945406: [meta] Convert all of confirm_form() to ConfirmFormBase.
There are two blockers for this left.
So I don't lose it.
Yes, let's do this. Since this is not just a dumb wrapper like t() or whatever keeping this means we need to support it and we really don't want to do that.
Sorry, this is not major.
Ok. I can't seem to envision a way to make this just a dumb wrapper function, so despite this being an API change, it should probably go in. confirm_form() has been marked deprecated for quite awhile, so hopefully this won't impact too many contribs. Adding the tag.
Committed and pushed to 8.x. Thanks!
Looks like we need to update https://drupal.org/node/1945416 accordingly, now.
working on the change record.
The change record https://drupal.org/node/1945416 has been updated with the following changes:
Marking as fixed.
Automatically closed - issue fixed for 2 weeks with no activity.
Drupal is a registered trademark of Dries Buytaert.