Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#3210 closed defect (bug) (fixed)

Group activity permalinks are broken for private / hidden groups

Reported by: r-a-y's profile r-a-y Owned by:
Milestone: 1.5 Priority: normal
Severity: Version: 1.5
Component: Activity Keywords: has-patch
Cc:

Description

Group activity permalinks for private / hidden groups redirect to $bp->root_domain whether you're a member of the group or not.

The changes made to BP_Activity_Activity::get() to merge ::get_specific() from BP 1.2 breaks this.

A potential fix requires changing $show_hidden to true on the activity router, single activity permalink screen action and activity permalink template.

FYI, the single activity permalink screen action handles access control, so we don't have to worry about the usage of "show_hidden" on the permalink template.

Attachments (1)

3210.01.patch (1.7 KB) - added by r-a-y 13 years ago.

Download all attachments as: .zip

Change History (12)

@r-a-y
13 years ago

#1 @r-a-y
13 years ago

If the patch in #2579 is accepted, this ticket can be closed.

#2 @DJPaul
13 years ago

  • Milestone changed from Awaiting Review to 1.3

#3 @DJPaul
13 years ago

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

I like the approach of treating group permalink re-routed URLs the same as for the members component, so closing this in favour of #2579

#4 @DJPaul
13 years ago

  • Resolution duplicate deleted
  • Status changed from closed to reopened

#5 @djpaul
13 years ago

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

(In [4519]) Fix Group activity permalinks for private / hidden groups. Fixes #3210, props r-a-y

#6 @r-a-y
13 years ago

We should note this on the BPDevel blog and/or in the bp-default Theme Changelog on the BP codex.

This will be important as BP 1.3 will break group activity permalinks for those that have copied /bp-default/members/single/activity/permalink.php to their child theme.

#7 @sbrajesh
13 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Just to notice, the problem exists with BuddyPress 1.2 branch and It is causing problem in BuddyPress 1.2.9.
For private group activities, you can not see the single activity page. It always redirects to home page. Any chances of getting it in 1.2 branch ?

#8 @r-a-y
13 years ago

You're right, Brajesh. r4111 (which made its way into BP 1.2.9) added some functionality from BP 1.3-trunk.

Patch needs to be backported to 1.2-branch.

#9 @boonebgorges
13 years ago

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

(In [4626]) Backports r4519 to the 1.2 branch so that private/hidden group activity permalinks display their content correctly. Fixes #3210. Props r-a-y

#10 @boonebgorges
13 years ago

It's unlikely that there will be another 1.2.x release, but I've backported it anyway.

#11 @sbrajesh
13 years ago

Thank you Ray for confirming.

and Thanks a ton Boone for porting the patch. Highly appreciate it.

Note: See TracTickets for help on using tickets.