Skip to:
Content

BuddyPress.org

Opened 10 years ago

Closed 10 months ago

#5358 closed defect (bug) (fixed)

Add $add_root as property of $bp

Reported by: needle's profile needle Owned by:
Milestone: 12.0.0 Priority: normal
Severity: normal Version: 1.9.1
Component: Core Keywords:
Cc:

Description

In bp-core-functions.php, line 504 in the bp_core_add_root_component() function throws "Indirect modification of overloaded property" notice, similarly to https://buddypress.trac.wordpress.org/ticket/5305

Change History (10)

#1 @r-a-y
10 years ago

  • Keywords dev-feedback added

It seems to me that we shouldn't even need $bp->add_root.

We should merge the bp_core_add_root_component() and bp_core_create_root_component_page() functions together since $bp->add_root doesn't do anything without the bp_core_create_root_component_page() function.

#2 @DJPaul
10 years ago

  • Milestone changed from Awaiting Review to 2.1

#3 @DJPaul
10 years ago

  • Keywords needs-patch added; dev-feedback removed
  • Milestone changed from 2.1 to 2.2

#4 @DJPaul
9 years ago

  • Milestone changed from 2.2 to Future Release

#5 @tw2113
7 years ago

Unless these two are dynamically called and named, I'm not seeing them used anywhere in core at the moment.

More than willing to be wrong.

#6 @espellcaste
2 years ago

  • Milestone changed from Awaiting Contributions to 11.0.0
  • Owner set to espellcaste
  • Status changed from new to assigned

#7 @espellcaste
18 months ago

  • Milestone changed from 11.0.0 to Up Next

Bumping this ticket to the next version so that I can look at this more closely. :)

#8 @imath
16 months ago

  • Milestone changed from Up Next to 12.0.0

#9 @espellcaste
14 months ago

  • Owner espellcaste deleted

#10 @imath
10 months ago

  • Keywords needs-patch removed
  • Resolution set to fixed
  • Status changed from assigned to closed

This function has been deprecated in 12.0 and is now only available from the BP Classic plugin. I’ll work on this issue from there: https://github.com/buddypress/bp-classic/issues/11

Note: See TracTickets for help on using tickets.