Skip to:
Content

BuddyPress.org

Opened 5 years ago

Last modified 3 years ago

#5985 new defect (bug)

Registration screen doesn't use BP_REGISTER_SLUG

Reported by: slaFFik Owned by:
Milestone: Awaiting Contributions 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 (6)

This ticket was mentioned in Slack in #buddypress by slaffik. View the logs.


5 years ago

#2 @DJPaul
5 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
3 years ago

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

#4 @r-a-y
3 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. Or add some notes about this to a relevant codex page if applicable.

Last edited 3 years ago by r-a-y (previous) (diff)

#5 @boonebgorges
3 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
3 years ago

  • Milestone changed from 2.8 to Future Release
Note: See TracTickets for help on using tickets.