#1938600: Add a FormInterface replacement for confirm_form() adds a base class to allow FormInterface to mimic confirm_form(). We should convert all of them to it. This is the meta for that. Please see the change notice below for sample conversion.
Part of #1971384: [META] Convert page callbacks to controllers
Related issue #2011018: Reconcile entity forms and confirm forms
These are currently the only modules that use confirm_form().

Module Assigned Issue link
action module #1946318: Convert confirm_form() in action.admin.inc to the new form interface Assigned to: kim.pepper
aggregator module #1946324: Convert confirm_form() in aggregator.admin.inc to the new form interface Assigned to: ParisLiakos
ban module #1944910: Convert ban.module's page callbacks to Controllers Assigned to: tim.plunkett
block module #1946334: Convert confirm_form() in block.admin.inc to the new form interface Assigned to: mtift
book module #1938318: Convert book_remove_form to a new-style Form object
comment module #1946348: Convert all of confirm_form() in comment.admin.inc to the new form interface
config module #1945444: Convert confirm_form() in config_test.module to the new form interface and convert route Assigned to: xjm
contact module #1946366: Convert confirm_form() in contact.admin.inc to the new form interface Assigned to: kim.pepper
custom_block module #1997262: Convert custom_block_delete_form to new form interface Assigned to: naxoc
field_ui module #1946404: Convert forms in field_ui.admin.inc to the new form interface
filter module #1946410: Convert confirm_form() in filter.admin.inc to the new form interface Assigned to: tim.plunkett
forum module #1946414: Convert confirm_form() in forum.admin.inc to the new form interface
image module #1946416: Convert all of confirm_form() in image.admin.inc to the new form interface Assigned to: kim.pepper
language module #1946426: Convert all of confirm_form() in language.admin.inc to the new form interface
menu module #1946428: Convert all of confirm_form() in menu.admin.inc to the new form interface Assigned to: foopang
node module #1946434: Convert all of confirm_form() in node.admin.inc and node.pages.inc to the new form interface
openid module #1946438: Convert confirm_form() in openid.pages.inc to the new form interface Assigned to: tim.plunkett
path module #1946444: Convert confirm_form() in path.admin.inc to the new form interface Assigned to: kim.pepper
picture module #1946450: Convert confirm_form() in picture_mapping.admin.inc to the new form interface Assigned to: ayelet_Cr
search module #1946452: Convert confirm_form() in search.admin.inc to the new form interface
shortcut module #1945564: Convert confirm_form() in shortcut to the new form interface and convert route Assigned to: tim.plunkett
system module #1946454: Convert all confirm_form() in system.module and system.admin.inc to the new form interface and convert route
taxonomy module #1946456: Convert taxonomy_term_confirm_delete() and taxonomy_vocabulary_confirm_delete() to the new form interface
translation_entity module #1946462: Convert content_translation_translatable_form() to the new form interface Assigned to: andypost
user module #1946466: Convert all confirm_form() in user.module and user.pages.inc to the new form interface and convert route

Comments

tim.plunkett’s picture

Issue tags: +FormInterface

Tagging. Please tag any sub-issues you create with FormInterface as well.

tim.plunkett’s picture

Issue summary: View changes

I'll play.

tim.plunkett’s picture

Issue summary: View changes

Updated issue summary.

jibran’s picture

Issue summary: View changes

Updated issue summary.

jibran’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

kim.pepper’s picture

Issue summary: View changes

Added myself for contact module

tim.plunkett’s picture

Issue summary: View changes

used the fancy @ thing i didn't know about

kim.pepper’s picture

Issue summary: View changes

Assign kim.pepper to path issue

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

mtift’s picture

Issue summary: View changes

Updated issue summary.

kim.pepper’s picture

Issue summary: View changes

Added myself to action module

kim.pepper’s picture

Issue summary: View changes

Add myself to image

andypost’s picture

yched’s picture

People have been trying to help with #1946404: Convert forms in field_ui.admin.inc to the new form interface, but stumbled on the corresponding menu items currently being defined in a foreach, and thus the need to go through RouteSubscriber / RouteEnhancer (or something :-))

Any knowledgeable people to provide docs or code pointers on the correct way to leverage that over there ?

yched’s picture

Issue summary: View changes

Updated issue summary.

tim.plunkett’s picture

Issue summary: View changes

Updated issue summary.

jibran’s picture

jibran’s picture

Issue summary: View changes

added link to callback meta

jibran’s picture

Issue summary: View changes

Updated issue summary.

tim.plunkett’s picture

Issue summary: View changes

updated

webchick’s picture

Issue summary: View changes
Status: Active » Fixed

Just committed those two, so I think we can close this out.

tim.plunkett’s picture

I meant to link to #2149263: Remove confirm_form(), not to this issue.

Crell’s picture

jibran’s picture

Status: Fixed » Closed (fixed)

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