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.
See http://drupal.org/node/967566.

CommentFileSizeAuthor
#6 l10n_client-rename_extra_field_key-1277752-6.patch1.33 KBPascalAnimateur
#5 l10n_client-fix_field_extra_field.patch564 bytesPascalAnimateur
l10n_client-hook_field_extra_fields.patch585 bytesinterx

Comments

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

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

SebCorbin commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

ben.kyriakou’s picture

ben.kyriakou commented
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

PascalAnimateur commented
StatusFileSize
new l10n_client-fix_field_extra_field.patch564 bytes

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

PascalAnimateur’s picture

PascalAnimateur commented
StatusFileSize
new l10n_client-rename_extra_field_key-1277752-6.patch1.33 KB

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

SebCorbin commented
Status: Needs work » Needs review