Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#4 | 1176546_make_user_field_adjustable_4.patch | 631 bytes | greggles |
#2 | 1176546_make_user_field_adjustable.patch | 631 bytes | greggles |
Comments
Comment #1
tsvenson CreditAttribution: tsvenson commentedUnfortunately 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.
Comment #2
gregglesWhat the heck.
Thanks to tsvenson for the pointer to that issue and to Dave Reid in irc for guidance.
Comment #3
Dave ReidJust 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.
Comment #4
gregglesGood point, thanks.
Comment #5
tsvenson CreditAttribution: tsvenson commentedThanks @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.
Comment #6
gregglesGreat, committed - http://drupalcode.org/project/comment_notify.git/commit/661f290
Thanks for the ideas and review tsvenson!
Comment #7
tsvenson CreditAttribution: tsvenson commentedMy 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.