Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 years ago

#2058 closed defect (bug) (fixed)

'All Groups' link has wrong url

Reported by: dwenaus's profile dwenaus Owned by: dwenaus's profile dwenaus
Milestone: 1.2.4 Priority: minor
Severity: Version:
Component: Groups Keywords: has-patch
Cc:

Description

in the file bp-themes/bp-default/groups/index.php
on the line
<li class="selected" id="groups-all"><a href="<?php bp_root_domain() ?>"><?php printf( __( 'All Groups (%s)', 'buddypress' ), bp_get_total_group_count() ) ?></a></li>

bp_root_domain()

should be:

bp_get_root_domain() . '/' . BP_GROUPS_SLUG

this is not a big deal because all the action is happening with javascript which ignores the url. however the link is incorrect - and if someone were to copy the url for use as a direct link, it wouldn't work.

Attachments (1)

2058_group_home_link.diff (1012 bytes) - added by dwenaus 12 years ago.

Download all attachments as: .zip

Change History (7)

#1 @dwenaus
12 years ago

  • Priority changed from trivial to minor

#2 @dwenaus
12 years ago

  • Keywords has-patch added

patch is tested and works

#3 @dwenaus
12 years ago

  • Owner set to dwenaus
  • Status changed from new to accepted

#4 @dwenaus
12 years ago

  • Status changed from accepted to assigned

#5 @dwenaus
12 years ago

  • Component set to Groups
  • Milestone changed from 1.3 to 1.2.4

#6 @johnjamesjacoby
12 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [2898]) Fixes #2058 props dwenaus

Note: See TracTickets for help on using tickets.