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.
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 |
---|---|---|---|
#20 | drupal8.system-module.1987656-20.patch | 3.27 KB | mparker17 |
#20 | interdiff.txt | 777 bytes | mparker17 |
#18 | drupal8.system-module.1987656-18.patch | 3.22 KB | mparker17 |
#12 | drupal8.system-module.1987656-11.patch | 3.29 KB | mparker17 |
#12 | interdiff.txt | 670 bytes | mparker17 |
Comments
Comment #1
vijaycs85Need to rewrite the whole module to make test sync with current test implementation. For more details, please refer: #1988802: [META] Rewrite test modules in system to provide better unit testing.
Comment #2
ayelet_Cr CreditAttribution: ayelet_Cr commentedComment #3
mparker17I'll help!
Comment #4
mparker17Try this...
Comment #5
dawehnerdrupal_set_title can be replaced by return array('#title' => 'Test querystring'); If we do that, just put a @return statement in there as well.
Comment #6
dawehnerdrupal_set_title can be replaced by return array('#title' => 'Test querystring'); If we do that, just put a @return statement in there as well.
Comment #7
mparker17Try this...
Comment #8
dawehnerSorry, pattern should start with "/"
Comment #9
mparker17K
Comment #10
xjmThanks for your work on this issue! Please see #1971384-43: [META] Convert page callbacks to controllers for an update on the routing system conversion process.
Comment #11
mparker17Try this...
Comment #12
mparker17Try this...
Comment #14
mparker17#12: drupal8.system-module.1987656-11.patch queued for re-testing.
Comment #16
mparker17#12: drupal8.system-module.1987656-11.patch queued for re-testing.
Comment #18
mparker17Straight re-roll of the patch; no interdiff.
Comment #19
dawehnerCan we have just a simple @return statement, please?
Comment #20
mparker17OK...
Comment #21
dawehnerThank you very much!!
Comment #22
webchickCommitted and pushed to 8.x. Thanks!