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 #1971384: [META] Convert page callbacks to controllers
For instructions on how to convert a page callback into a controller, see the WSCCI Conversion Guide.
Comment | File | Size | Author |
---|---|---|---|
#7 | drupal-convert_config_test_add_page-1987662-7.patch | 1.69 KB | mparker17 |
#3 | drupal-convert_config_test_add_page-1987662-3.patch | 1.56 KB | h3rj4n |
#1 | drupal-convert_config_test_add_page-1987662-1.patch | 2.86 KB | h3rj4n |
Comments
Comment #1
h3rj4n CreditAttribution: h3rj4n commentedThe patch to convert this to a controller.
Comment #2
tim.plunkettYou should be able to just have
_entity_form: 'config_test'
, and then you can ditch the whole ConfigTestControllerComment #3
h3rj4n CreditAttribution: h3rj4n commentedChanged it and it seems to work fine.
Removed the controller, changed the routing file to _entity_form.
Comment #4
dawehnerHe, making progress by basically just removing code. Tests are running fine, so this feels great.
Comment #5
alexpottNeeds reroll...
Comment #6
mparker17I'll help!
Comment #7
mparker17Rerolled!
No interdiff because the old patch didn't apply. Also, it's literally just a re-roll.
Comment #8
tim.plunkettThanks!
Comment #9
jibran#1987668: Convert config_test module to routes I am sorry for doing this.