Skip to:
Content

BuddyPress.org

Opened 10 years ago

Closed 9 months ago

#5985 closed defect (bug) (wontfix)

Registration screen doesn't use BP_REGISTER_SLUG

Reported by: slaffik's profile slaFFik Owned by:
Milestone: Priority: lowest
Severity: normal Version:
Component: Registration Keywords: close 2nd-opinion
Cc:

Description

Test flow:

  • remove connection on BuddyPress - Pages page between Registration page and any WP page
  • define BP_REGISTER_SLUG with as sample-page (for example, first default page after clean WP instal)
  • open site.com/sample-page

I expect there to be registration page content, but I see ordinary WP page content so far

Filtering bp_get_signup_slug() gives no result as well.

Change History (7)

This ticket was mentioned in ​Slack in #buddypress by slaffik. ​View the logs.
10 years ago

#2 @DJPaul
10 years ago

  • Milestone changed from Awaiting Review to Future Release
  • Summary changed from BP_REGISTER_SLUG seems doesn't work to Registration screen doesn't use BP_REGISTER_SLUG

Similar to your report in #5991, this is likely a routing bug somewhere. For legacy reasons, if this slug is set, it should override whatever it is set to in the Page Mapping.

#3 @slaFFik
8 years ago

  • Component changed from Core to Registration
  • Milestone changed from Future Release to 2.8

#4 @r-a-y
8 years ago

  • Keywords close 2nd-opinion added
  • Priority changed from normal to lowest

I would say this is the lowest of all priorities.

Part of the problem was moving to WordPress pages in BP 1.5. Need to use sample-page as the registration page? Set it in the BP admin area.

The only way I see this being useful is if you are trying to automate the BP setup process for a new site.

We should probably deprecate the usage of the BP_REGISTER_SLUG constant if we haven't already.

Version 2, edited 8 years ago by r-a-y (previous) (next) (diff)

#5 @boonebgorges
8 years ago

Yeah, I agree with @r-a-y that there's probably no way for BP_REGISTER_SLUG to work since 1.5. (The case is different for BP_MEMBERS_SLUG etc, since in the case of component directories, the post_name determines the "root slug".) I suggest we add documentation to this effect.

#6 @DJPaul
8 years ago

  • Milestone changed from 2.8 to Future Release

#7 @imath
9 months ago

  • Milestone Awaiting Contributions deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Slug constants were deprecated in 12.0.0

Note: See TracTickets for help on using tickets.