The form element for user account has no default #type set in user_account_form of user.module. This means drupal_process_form always sets this element to #processed=false since there is no default #process to run on it. This probably has no major implications except that the field_group module only allows elements with #processed=true inside field groups and therefore user account information cannot be placed inside a field group (see http://drupal.org/node/1182198). The obvious patch is to set $form['account']['#type'] = 'container'.

Files: 
CommentFileSizeAuthor
#1 1184830-user-account-container.patch427 byteslinclark
PASSED: [[SimpleTest]]: [MySQL] 33,437 pass(es).
[ View ]

Comments

linclark’s picture

Version:7.2» 8.x-dev
Status:Active» Needs review
StatusFileSize
new427 bytes
PASSED: [[SimpleTest]]: [MySQL] 33,437 pass(es).
[ View ]

This makes sense, and there is a precedent for doing it in all of the other forms which are in user.admin.inc, so I've attached a patch.

linclark’s picture

Priority:Normal» Minor

Although it looks like the issue that caused this is closed... I would be fine with either a commit or a CWF on this.

tstoeckler’s picture

Status:Needs review» Reviewed & tested by the community
webchick’s picture

Status:Reviewed & tested by the community» Fixed

I couldn't remember seeing this pattern before, but a quick grep shows that it's fairly common, esp. in *.admin.inc files.

This looks pretty harmless for D7 as well, so committed and pushed to both 8.x and 7.x. Thanks!

neoglez’s picture

Issue summary:View changes

Updated issue summary.

Status:Fixed» Closed (fixed)

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

Anonymous’s picture

Issue summary:View changes

Updated issue summary.