Skip to:
Content

BuddyPress.org

Changeset 2641


Ignore:
Timestamp:
02/10/2010 12:29:59 PM (14 years ago)
Author:
apeatling
Message:

Fixes #1789

Location:
trunk
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/bp-forums.php

    r2626 r2641  
    165165        break;
    166166
    167         case 'tag':
     167        case 'tags':
    168168            $query = new BB_Query( 'topic', array( 'forum_id' => $forum_id, 'topic_author_id' => $user_id, 'tag' => $filter, 'per_page' => $per_page, 'page' => $page, 'order_by' => 't.topic_time', 'sticky' => $show_stickies ) );
    169169            $topics =& $query->results;
  • trunk/bp-forums/bp-forums-templatetags.php

    r2626 r2641  
    4040                break;
    4141
    42             case 'tag':
    43                 $this->topics = bp_forums_get_forum_topics( array( 'user_id' => $user_id, 'type' => 'tag', 'filter' => $search_terms, 'forum_id' => $forum_id, 'page' => $this->pag_page, 'per_page' => $this->pag_num, 'show_stickies' => $no_stickies ) );
     42            case 'tags':
     43                $this->topics = bp_forums_get_forum_topics( array( 'user_id' => $user_id, 'type' => 'tags', 'filter' => $search_terms, 'forum_id' => $forum_id, 'page' => $this->pag_page, 'per_page' => $this->pag_num, 'show_stickies' => $no_stickies ) );
    4444                break;
    4545        }
     
    178178    }
    179179
    180     /* If we're viewing a tag in the directory, let's auto set the filter to the tag name */
    181     if ( 'tag' == $bp->current_action && !empty( $bp->action_variables[0] ) ) {
    182         $search_terms = $bp->action_variables[0];
    183         $type = 'tags';
    184     }
    185 
    186180    /* If $_GET['fs'] is set, let's auto populate the search_terms var */
    187181    if ( $bp->is_directory && !empty( $_GET['fs'] ) )
     
    206200    extract( $r );
    207201
     202    /* If we're viewing a tag URL in the directory, let's override the type and set it to tags and the filter to the tag name */
     203    if ( 'tag' == $bp->current_action && !empty( $bp->action_variables[0] ) ) {
     204        $search_terms = $bp->action_variables[0];
     205        $type = 'tags';
     206    }
     207
    208208    $forum_template = new BP_Forums_Template_Forum( $type, $forum_id, $user_id, $page, $per_page, $max, $no_stickies, $search_terms );
    209209    return apply_filters( 'bp_has_topics', $forum_template->has_topics(), &$forum_template );
  • trunk/bp-themes/bp-default/_inc/global.js

    r2604 r2641  
    597597    /* Clicking a forum tag */
    598598    j('div#forum-directory-tags a').click( function() {
    599         bp_filter_request( 'forums', 'tags', j.cookie('bp-forums-scope'), 'div.forums', j(this).html(), 1, j.cookie('bp-forums-extras') );
     599        bp_filter_request( 'forums', 'tags', j.cookie('bp-forums-scope'), 'div.forums', j(this).html().replace( / /g, '-' ), 1, j.cookie('bp-forums-extras') );
    600600        return false;
    601601    });
  • trunk/bp-themes/bp-default/functions.php

    r2635 r2641  
    5050        <div class="comment-avatar-box">
    5151            <div class="avb">
    52                 <a href="<?php echo get_comment_author_url() ?>">
     52                <a href="<?php echo get_comment_author_url() ?>" rel="nofollow">
    5353                    <?php if ( $comment->user_id ) : ?>
    5454                        <?php echo bp_core_fetch_avatar( array( 'item_id' => $comment->user_id, 'width' => 50, 'height' => 50, 'email' => $comment->comment_author_email ) ); ?>
     
    6363
    6464            <div class="comment-meta">
    65                 <a href="<?php echo get_comment_author_url() ?>"><?php echo get_comment_author(); ?></a> <?php _e( 'said:', 'buddypress' ) ?>
     65                <a href="<?php echo get_comment_author_url() ?>" rel="nofollow"><?php echo get_comment_author(); ?></a> <?php _e( 'said:', 'buddypress' ) ?>
    6666                <em><?php _e( 'On', 'buddypress' ) ?> <a href="#comment-<?php comment_ID() ?>" title=""><?php comment_date() ?></a></em>
    6767            </div>
Note: See TracChangeset for help on using the changeset viewer.