Hello

I want to propose add opportunity to use fields from taxonomy term on the form of forum. Sometimes useful to have additional fields on the form of creation/editing forum to allow user specify some options.
For example, on my current project part of forums are not available for anonymous users and customer wants to specify is this forum protected or not protected just in few clicks.

And it's very simple - we just create field for taxonomy vocabulary via field API and can use it later on the form of creation/editing forum.

CommentFileSizeAuthor
#8 advanced_forum-use-fields-from-term-in-forum-2599500-8-D7.patch586 byteseugene.ilyin
#5 advanced_forum-use-fields-from-term-in-forum-2599500-5-D7.patch4.53 KBeugene.ilyin
#2 advanced_forum-use-fields-from-term-in-forum-2599500-2-D7.patch1.61 KBeugene.ilyin

Comments

eugene.ilyin created an issue. See original summary.

eugene.ilyin’s picture

eugene.ilyin commented
StatusFileSize
new advanced_forum-use-fields-from-term-in-forum-2599500-2-D7.patch1.61 KB

I've prepared simple patch to implement this functionality.

It provides additional option on the forum settings form to switch on this new opportunity and logic to add fields from taxonomy term on the forum of creation/editing of the forum.

eugene.ilyin’s picture

eugene.ilyin commented
Status: Active » Needs review
podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Status: Needs review » Needs work
Issue tags: +Needs tests, +Needs documentation

Can you please add a bit of documentation to README file about this feature.
Also, we definitely should have a test for this sunctionality.

Code looks good from my point of view.

Please add test and docs and it would be ready to go.

eugene.ilyin’s picture

eugene.ilyin commented
Issue summary: View changes
Status: Needs work » Needs review
StatusFileSize
new advanced_forum-use-fields-from-term-in-forum-2599500-5-D7.patch4.53 KB

Thank you for quick reaction!

I've added info into README.txt and tests for creation of the forum with active option 'Use fields from taxonomy term in forum'.

Also I've put new setting into another form. I think it's more correct, because this setting has been provided by module advanced forum.

  • podarok committed 25dc7b8 on 7.x-2.x authored by eugene.ilyin 
    Issue #2599500 by eugene.ilyin, podarok: Opportunity to use fields from...
podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Status: Needs review » Fixed

#5 pushed to 7.x-2.x.
Thank you

Will be tagged into next upcoming release

eugene.ilyin’s picture

eugene.ilyin commented
Priority: Normal » Major
Status: Fixed » Needs review
StatusFileSize
new advanced_forum-use-fields-from-term-in-forum-2599500-8-D7.patch586 bytes

Seems I had a mistake in previous patch. Can you apply this little correction please?

  • podarok committed f214a7d on 7.x-2.x authored by eugene.ilyin 
    Issue #2599500 by eugene.ilyin, podarok: Opportunity to use fields from...
podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Category: Feature request » Bug report
Status: Needs review » Fixed
Issue tags: -Needs tests, -Needs documentation

Status: Fixed » Closed (fixed)

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