When one goes to admin/user/user_terms there is a "Group terms" configuration section in which one can customize the text to use as a heading when the user terms appear grouped as a fieldset.

However, they always appear grouped under the title "User terms" on the registration form. Shouldn't they use our custom "Grouped terms heading" setting there too?

CommentFileSizeAuthor
#3 956856-3.user_terms.group-label-form.patch822 bytesjoachim
#1 user_terms-956856.patch809 bytesAlexander N

Comments

Alexander N’s picture

Alexander N commented
StatusFileSize
new user_terms-956856.patch809 bytes

In case this is accepted as a bug here's the patch.

Alexander N’s picture

Alexander N commented
Version: 6.x-1.3 » 6.x-1.x-dev
joachim’s picture

joachim commented
Category: bug » feature
Status: Active » Needs review
StatusFileSize
new 956856-3.user_terms.group-label-form.patch822 bytes

Seems like a good idea.

Output needs to be sanitized though. Try this patch.

Alexander N’s picture

Alexander N commented
Status: Needs review » Reviewed & tested by the community

Ah even better. :)

Works fine here.

joachim’s picture

joachim commented
Category: feature » bug
Status: Reviewed & tested by the community » Fixed

#956856 by Alexander N, joachim: Fixed grouped terms heading not shown on registration.

Committed. Thanks for the patch and testing!

Michsk’s picture

Michsk commented
Status: Fixed » Needs work

This doesn't seem fixed at all. They still are always grouped, even when i not have checked the checkbox to group them.

joachim’s picture

joachim commented
Status: Needs work » Postponed (maintainer needs more info)

Are you using the dev version?

mrtoner’s picture

mrtoner commented
Status: Postponed (maintainer needs more info) » Closed (fixed)

No additional info provided by OP, no activity in 6+ months, and patch now committed to 6.x-1.4.

joachim’s picture

joachim commented

Yeah, sorry for leaving it so long to make a stable release. I was figuring more patches would get reviewed, and this module fell off my radar.