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'.

CommentFileSizeAuthor
#1 1184830-user-account-container.patch427 byteslinclark
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Version: 7.2 » 8.x-dev
Status: Active » Needs review
FileSize
427 bytes

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.

Anonymous’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.