Updated: Comment #N

Problem/Motivation

When SystemConfigFormBase was added in #1921996: Convert system_config_form() to implement FormInterface as a base class., we named it directly after system_config_form() and left it in system.module, because there was only \Drupal\Core\Form\FormInterface in that directory.

Now it can live alongside ConfirmFormBase and FormBase

Proposed resolution

Move \Drupal\system\SystemConfigFormBase to \Drupal\Core\Form\ConfigFormBase

Remaining tasks

N/A

User interface changes

N/A

API changes

If you extend SystemConfigFormBase, you'll have to fix your use statements.

N/A

Files: 
CommentFileSizeAuthor
#1 config-form-base-2089627-1.patch22.19 KBtim.plunkett
PASSED: [[SimpleTest]]: [MySQL] 58,826 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new22.19 KB
PASSED: [[SimpleTest]]: [MySQL] 58,826 pass(es).
[ View ]

Issue tags:+pants

Tagging.

Status:Needs review» Reviewed & tested by the community
Issue tags:-pants

Now it can live alongside ConfirmFormBase and FormBase

happily ever after. :D

Assigned:Unassigned» webchick
Status:Reviewed & tested by the community» Fixed

<3 <3 <3

Committed and pushed to 8.x. Thanks!

I'll take care of the change notice.

Assigned:webchick» Unassigned
Issue tags:+pants

Hooray, done. :)

Assigned:Unassigned» webchick

Tagging.

Assigned:webchick» Unassigned

x-post.

Seems not pushed yet?

Priority:Normal» Critical
Status:Fixed» Needs work

@webchick sorry I do not know how else to raise this, seems like this is not pushed to HEAD... or I am missing sth

Status:Needs work» Reviewed & tested by the community

Still applies.

Priority:Critical» Normal
Status:Reviewed & tested by the community» Fixed

Oops! So sorry. :(

Really committed and pushed to 8.x this time. :) Thanks!

Although this is a form, I don't see how this fundamentally belongs to the forms system. I think \Drupal\Core\Config\Form would be a much more natural fit. Thoughts?

That's not a namespace that exists. And we're not going to have \Drupal\Core\Confirm\Form\ConfirmFormBase or anything...

Well, OK I was a bit terse with my comment. The "Form" namespace below \Drupal\Core\Config does not exist. But the "Config" system is actually a thing, and to me it seems a thing like "ConfigFormBase" belongs to it. That was my suggestion.

There's no such thing as a "Confirm" system. That's just silly.

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