Problem/Motivation

If we use the route enhancer for _form on RoleDelegationSettingsForm then we won't need the controller anymore, less code :)

Proposed resolution

Replace _controller with _form in the routing file, delete the controller.

Remaining tasks

Code.

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 2693409-2.patch1.73 KBbenjy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

benjy created an issue. See original summary.

benjy’s picture

benjy CreditAttribution: benjy at PreviousNext commented
Status: Active » Needs review
FileSize
2693409-2.patch1.73 KB
Sam152’s picture

Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented
Status: Needs review » Reviewed & tested by the community

  • benjy committed 5eb99a2 on 8.x-1.x 
    Issue #2693409 by benjy, Sam152: RoleDelegationSettingsForm should use...
benjy’s picture

benjy CreditAttribution: benjy at PreviousNext commented
Status: Reviewed & tested by the community » Fixed

Thanks.

Status: Fixed » Closed (fixed)

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