Here's a patch that implements hook_field_extra_fields(). I wanted to make the fieldset that is added to the user account form organizable in the Manage Fields tab.

Members fund testing for the Drupal project. Drupal Association Learn more


Gábor Hojtsy’s picture

Wow, yeah. I've just been pushing for #967566: Several important Core User settings need to implement hook_field_extra_fields() a lot, so it only makes sense to do the same thing in my backyard :) Great suggestion. Will look into adding this in soon, thanks for the patch!

SebCorbin’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

ben.kyriakou’s picture

Issue summary: View changes
Status: Closed (fixed) » Needs work

There appears to be a bug in this implementation - the key used in the hook should be l10n_client, not l10nclient (see l10n_client_form_user_profile_form_alter()). Unfortunately the fix isn't as simple as changing the key, since the field element in the account interface has the id l10n-client, which is also used for the translation interface implemented in l10n_client_page_alter(). The styling for this breaks the element in the account interface.

If I have time I'll try and roll a working patch for this - the most straightforward fix seems to be changing the id used by the translation interface so they don't collide.

PascalAnimateur’s picture

Effectively, only changing the key doesn't solve the issue... my patch is useless!

PascalAnimateur’s picture

This is better.. I think this new patch works!

  • SebCorbin committed c742853 on 7.x-1.x authored by interX
    Issue #1277752 by interX: Implement hook_field_extra_fields().
SebCorbin’s picture

Status: Needs work » Needs review