Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Follow-up from #2078867: Convert _form_test_* functions to classes.
Part of #1971384: [META] Convert page callbacks to controllers
Convert below form callback to new controller (form/callback).
ajax_forms_test_ajax_commands_form
ajax_forms_test_lazy_load_form
ajax_forms_test_simple_form
ajax_forms_test_validation_form
For instructions on how to convert a page callback into a controller, see the WSCCI Conversion Guide.
Comment | File | Size | Author |
---|---|---|---|
#5 | ajax-test-2147009-5.patch | 32.18 KB | tim.plunkett |
#3 | system.ajax_forms_test.forms-2147009-3.patch | 23.97 KB | RoSk0 |
Comments
Comment #1
RoSk0Comment #2
RoSk0Patch is on his way.
Comment #3
RoSk0Converted ajax_forms_test_lazy_load_form, ajax_forms_test_simple_form, ajax_forms_test_validation_form forms and incorporated patch from #2132465: Convert ajax_forms_test_ajax_commands_form to new controller, this changes must go together not to leave garbage like AjaxFormsTestForm(see diff) in place.
Also restoring original title and description
Comment #5
tim.plunkettYour comment in #3 said you merged the two issues, but you were missing the actual form class.
My only other changes were changing t() to $this->t(), all other work was yours, so I'm going to RTBC (passed locally).
Comment #6
webchickLooks good.
Committed and pushed to 8.x. Thanks!