Skip to:
Content

BuddyPress.org

Opened 7 years ago

Last modified 3 years ago

#4996 new defect (bug)

Problematic custom activity slug

Reported by: henrywright Owned by:
Milestone: Awaiting Contributions Priority: normal
Severity: normal Version: 1.7
Component: Core Keywords:
Cc:

Description

define ( 'BP_ACTIVITY_SLUG', 'feed' );

http://buddypress.org/2009/05/customizable-slugs-in-buddypress/

Using "feed" as the activity slug results in members not being able to post an update to their activity stream.

Could using the slug "feed" be conflicting with some RSS functionality? Perhaps care should be taken when choosing to rename slugs? Renaming to the activity slug to words such as "wall" or "stream" seem to work fine.

Change History (6)

#1 @r-a-y
7 years ago

  • Keywords custom slugs removed
  • Milestone changed from Awaiting Review to Future Release

I can duplicate this when theme compat is on, but not with bp-default. It's probably a conflict with WP's feed rewrite rules.

This might be alleviated when #4954 is implemented.

#2 @slaFFik
3 years ago

Seems we should check the slugs against prohibited/reserved strings, like feed.

#3 @tw2113
3 years ago

I concur with @slaFFik in that this should probably be a blacklisted value

#4 @hnla
3 years ago

Do we need to build a blacklist for slugs, can't recall of top of head whether we do reserve any strings where this sort of renaming is concerned, we did work up changes quite recently for activity stream dir as that wasn't re-nameable before, ought to look up that ticket.

#5 @tw2113
3 years ago

I know there's a similar functioning list found in bp_core_get_illegal_names() and bp-members-functions.php

#6 @hnla
3 years ago

So we'll need to run the dir titles via that get_illegal_names function before or after our default slugs are set, we essentially rely on WP page slugs.

Just dropping this commit ref in as it reminds me how we handle these slugs:
https://buddypress.trac.wordpress.org/changeset/11080

This does strike me as something we ought to look into.

Note: See TracTickets for help on using tickets.