It would be nice if this module could be made arrangeable in the Manage Fields settings for user profiles. Right now it is not and thus can end up almost anywhere when you start moving the fields that are available.

It also prevents the possibility of for example putting it in a tab using the excellent Field Group module.

CommentFileSizeAuthor
#4 1176546_make_user_field_adjustable_4.patch631 bytesgreggles
#2 1176546_make_user_field_adjustable.patch631 bytesgreggles
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tsvenson’s picture

tsvenson CreditAttribution: tsvenson commented

Unfortunately I'm no coder so I can't provide a patch. However, I believe this is quite easy to accomplish in Drupal 7 using the hook_field_extra_fields() function. At least that is what Dave Reid said in a similar issue I filed for the XML Sitemap module #1176526: Make XML Sitemap setting visible on Manage Fields/Display.

I fact, less than 3h after I filed the issue Dave had committed a fix, see comment #8 in the above issue.

Hopefully this info can help someone with better coding skills to be able to implement this.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Title: Make field available in Manage Fields » Make user profile setting available in Manage Fields
Status: Active » Needs review
FileSize
1176546_make_user_field_adjustable.patch631 bytes

What the heck.

Thanks to tsvenson for the pointer to that issue and to Dave Reid in irc for guidance.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

Just be sure to match #weight from what you're currently using - look like in comment_notify_form_alter() #weight is 4, but here its defined as 5. As long as they're the same.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
FileSize
1176546_make_user_field_adjustable_4.patch631 bytes

Good point, thanks.

tsvenson’s picture

tsvenson CreditAttribution: tsvenson commented
Status: Needs review » Reviewed & tested by the community

Thanks @greggles. Even though I am no coder (yet), I at least can try and save some time for those who are.

Tried and tested, works as it says on the tin.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Reviewed & tested by the community » Fixed

Great, committed - http://drupalcode.org/project/comment_notify.git/commit/661f290

Thanks for the ideas and review tsvenson!

tsvenson’s picture

tsvenson CreditAttribution: tsvenson commented

My pleasure. Will actually use your patch for this and see if I can't contribute some patches to other contribs, and Core, features that need this too. Seems simple enough for me to start with.

Status: Fixed » Closed (fixed)

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