Skip to:
Content

Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#1129 closed enhancement (duplicate)

Forum Topic Tags not visible/editable

Reported by: Detective Owned by:
Milestone: 1.5 Priority: minor
Severity: Version:
Component: Forums Keywords:
Cc: Detective

Description

Currently, forum topic tags only appear on the forum directory. This is good but the tags don't appear anywhere else:

a) The topic page doesn't display the current topic tags. It is possible to show tags using a template tag like this (just a proof of concept):

function bp_the_topic_tags() {
	$tags = bb_get_topic_tags( bp_get_the_topic_id() );
	if ( !empty( $tags ) ) { 
		_e( 'Tagged as:', 'buddypress' ) . ' '; 
		$plain_tags = array();
		foreach ( $tags as &$tag ) 
			$plain_tags[] = sprintf( '<a href="%s" rel="tag">%s</a>', get_option( 'home' ) . '/' . BP_FORUMS_SLUG . '/' . 'tag' . '/' . $tag->slug, $tag->name );
		echo implode( ', ', $plain_tags ) . '.';
	} else { 
		_e('No tags.', 'buddypress');
	} 
}

I'm pretty sure that bb_get_tag_link could be used because of the bp_forums_filter_tag filter.

b) The edit topic page doesn't allow the topic author to edit topic tags.

c) The group members are unable to add tags to a topic. Maybe this can be left out, but since in bbPress it's possible for each user to add tags, I guess it could be added easily.

d) The current forum directory displays the most popular forums on topics matching a tag. This is confusing because if the user clicks on a tag link, and gets the "popular forums" list, he/she can understand that those forums are tagged. There could be a template tag like

function bp_forums_is_tag() {
	$tag = bp_get_forums_tag_name();
	return !empty( $tag );
}

And then we could wrap the forum list with this conditional.

Change History (4)

#1 @DJPaul
9 years ago

  • Milestone changed from 1.2 to 1.2.1

These issues need confirming in BP 1.2-RC2.

#2 @DJPaul
9 years ago

Ticket #1988 seems to be a duplicate of some of the issue described above. Might need individual tickets for the points above. Bump for 1.3?

#3 @boonebgorges
8 years ago

  • Component set to Core
  • Resolution set to duplicate
  • Status changed from new to closed

See #2281 for further discussion

#4 @boonebgorges
8 years ago

  • Component changed from Core to Forums
Note: See TracTickets for help on using tickets.