For instructions on how to convert a page callback into a controller, see the WSCCI Conversion Guide.

Part of #1971384: [META] Convert page callbacks to controllers

CommentFileSizeAuthor
#1 drupal-node-type-form-2005618-1.patch24.4 KBbdone
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bdone’s picture

bdone CreditAttribution: bdone commented
Status: Active » Needs review
FileSize
drupal-node-type-form-2005618-1.patch24.4 KB

here's a first pass at a WSCCI conversion for review

bdone’s picture

bdone CreditAttribution: bdone commented

after reading #1981644: Figure out how to deal with 'title/title callback', i'm unsure how to handle node type titles for other local tasks such as manage fields (admin/structure/types/manage/%/comment/fields). thus, the patch in #1 defaults to hook_menu title.

swentel’s picture

swentel CreditAttribution: swentel commented

Note that this is done in #111715: Convert node/content types into configuration as well .. so I think we can close this one.

Status: Needs review » Needs work

The last submitted patch, drupal-node-type-form-2005618-1.patch, failed testing.

bdone’s picture

bdone CreditAttribution: bdone commented

@swentel: didn't realize that, thanks. still nice to have learned a good bit about wscii in the process.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs work » Fixed

SO this patch is done.

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