CommentFileSizeAuthor
#7 system_config_form-1924990-7.patch2.37 KBtim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

Issue summary: View changes

assigning user to me

tim.plunkett’s picture

Issue summary: View changes

removed stray *

tim.plunkett’s picture

Issue tags: +FormInterface

Okay, I did user and aggregator as examples.

All conversions should use routes and be defined as a service ID, so that config.factory can be injected in.
Aggregator has the additional example of how to inject other dependencies.

tim.plunkett’s picture

Issue summary: View changes

added aggregator issue

tim.plunkett’s picture

Issue summary: View changes

added book

larowlan’s picture

larowlan’s picture

Issue summary: View changes

Updated issue summary.

tim.plunkett’s picture

Issue summary: View changes

Updated issue summary.

tim.plunkett’s picture

Issue summary: View changes

Updated issue summary.

catch’s picture

I don't want to convert any more routes as services until #1915774: Decide whether core route controllers should generally/always be DIC services or not is resolved. This is exactly the sort of service bloat I'm concerned about in the other issue.

catch’s picture

Issue summary: View changes

Updated issue summary.

ACF’s picture

Issue summary: View changes

added search

YesCT’s picture

Issue tags: +Novice

ok. 1915774 was committed. it's waiting on a change notice, so that's why it's open. We can proceed creating the rest of the conversion issues. Since there are examples already, I'm tagging with novice.

When creating the issues, look at the examples already in the issue summary and be sure and update this issue summary and link the issues together.

YesCT’s picture

Issue summary: View changes

Updated

ACF’s picture

Issue summary: View changes

Added simpletest issue.

ACF’s picture

Issue summary: View changes

Added simpletest issue properly

ACF’s picture

Issue summary: View changes

Add statistics issue.

ACF’s picture

Issue summary: View changes

Add update issue

floretan’s picture

Issue summary: View changes

Add link to the menu module issue.

floretan’s picture

Issue summary: View changes

Indicate the menu module issue is assigned to me.

amateescu’s picture

I've rerolled aggregator, update and user (#1925048-6: Convert aggregator's system_config_form() to SystemConfigFormBase, #1937998-2: Convert update's system_config_form() to SystemConfigFormBase and #1924992-9: Convert user's system_config_form() to SystemConfigFormBase) to account for #1934832: Provide a dedicated approach for using forms in routes which got committed in the meantime.

All the other patches need to follow the new pattern for converting forms, the main difference being that you no longer have to create a service for each form class. More details and examples are in #1934832: Provide a dedicated approach for using forms in routes.

amateescu’s picture

Issue tags: +SprintWeekend2013

Add sprintweekend tag.

amateescu’s picture

Issue summary: View changes

Add locale issue.

tim.plunkett’s picture

Status: Active » Needs review
FileSize
2.37 KB

We can now kill off system_config_form()! Yay

jibran’s picture

Status: Needs review » Reviewed & tested by the community

Yay!!

alexpott’s picture

Title: [meta] Convert all of system_config_form() to SystemConfigFormBase » Change notice: [meta] Convert all of system_config_form() to SystemConfigFormBase
Priority: Major » Critical
Status: Reviewed & tested by the community » Active
Issue tags: +Needs change record
tim.plunkett’s picture

tstoeckler’s picture

Title: Change notice: [meta] Convert all of system_config_form() to SystemConfigFormBase » [meta] Convert all of system_config_form() to SystemConfigFormBase
Priority: Critical » Major
Status: Active » Fixed
Issue tags: -Needs change record

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