Skip to:
Content

bbPress.org

Opened 15 months ago

Closed 8 months ago

Last modified 8 months ago

#3617 closed defect (bug) (fixed)

bbPress Integration

Reported by: davidbaumwaid's profile davidbaumwaid Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.6.14 Priority: normal
Severity: normal Version: trunk
Component: Extend - BuddyPress Keywords: reporter-feedback has-patch
Cc:

Description

If you install bbPress, so on the BuddyPress profile page, will appear automatically "Forums" tab. The tab has sub tab called "Favorites". Since the BP 12, if you click it, it will redirect to a 404 not found page because the slug automatically changed to "bp-member-activity-favorites" instead of "favorites"

Change History (8)

#2 @SirLouen
15 months ago

  • Component changed from General to Extend - BuddyPress
  • Keywords reporter-feedback added

Reproduction Report

❌ This report can't validate that the issue can be reproduced.

Environment

  • OS: Windows 11
  • Web Stack: WP-Env 10.8
  • WordPress: 6.8-alpha-59336
  • Browser: Chrome 130.0.6723.92
  • Theme: Twenty Ten
  • Active Plugins:
    • bbPress 2.7.0-alpha-2
    • BuddyPress 14.2.1
    • Query Monitor 3.16.4

Steps to Reproduce

  1. Create a forum and a post
  2. Make favorite such post
  3. Go into my user profile and navigate to Forums >> Favorites

Actual Results

  • ❌ Error condition don't occur (reproduced).

Conclusions

It's required that the reporter also provides the bbPress version that it's being used to trigger this issue

Additional Notes

There is a deprecation notice
Deprecated: Function bp_core_get_user_domain is deprecated since version 12.0.0! Use bp_members_get_user_url() instead. in /var/www/html/wp-includes/functions.php on line 6114

This is happening in get_profile_url inside src/includes/extend/buddypress/members.php

But this is not interfering with the real feature that user is reporting.

Supplemental Artifacts

Before the Deprecation notice screenshot

After the Deprecation notice patch screenshot

This ticket was mentioned in PR #29 on bbpress/bbPress by @SirLouen.


15 months ago
#3

  • Keywords has-patch added

#4 @johnjamesjacoby
8 months ago

  • Milestone changed from Awaiting Review to 2.6.14
  • Owner set to johnjamesjacoby
  • Status changed from new to assigned
  • Version set to trunk

#5 @johnjamesjacoby
8 months ago

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

In 7309:

Extend - BuddyPress - Members: correct a deprecated function usage.

This change adds a private method to encapsulate support for bp_core_get_user_domain() if it is still explicitly included, and using bp_members_get_user_url() otherwise.

Props davidbaumwaid, sirlouen.

In trunk, for 2.7.

Fixes #3617.

#6 @johnjamesjacoby
8 months ago

In 7310:

Extend - BuddyPress - Members: correct a deprecated function usage.

This change adds a private method to encapsulate support for bp_core_get_user_domain() if it is still explicitly included, and using bp_members_get_user_url() otherwise.

Props davidbaumwaid, sirlouen.

In branches/2.6, for 2.6.14.

Fixes #3617.

#7 @johnjamesjacoby
8 months ago

In 7311:

Extend - BuddyPress - Members: correct default value from r7309.

Change from string to array, to avoid a potential PHP warning.

See #3617.

@johnjamesjacoby commented on PR #29:


8 months ago
#8

This will be fixed in 2.6.14. Thank you @SirLouen!

Note: See TracTickets for help on using tickets.