Skip to:
Content

bbPress.org

Opened 2 years ago

Closed 3 months ago

Last modified 3 months ago

#3589 closed defect (bug) (fixed)

Deprecated Warning

Reported by: vapvarun's profile vapvarun Owned by:
Milestone: Priority: low
Severity: trivial Version: 2.6.11
Component: Extend - BuddyPress Keywords: has-patch
Cc:

Description

Deprecated Creation of dynamic property BBP_Forums_Component::$members is deprecated
wp-content/plugins/bbpress/includes/extend/buddypress/loader.php:149

Deprecated Creation of dynamic property BBP_Forums_Component::$activity is deprecated
wp-content/plugins/bbpress/includes/extend/buddypress/loader.php:153

Active plugin set
bbPress
BuddyPress
Query Monitor

https://prnt.sc/GbzH3V6xy-Ac

Attachments (1)

deprecation-buddypress.patch (540 bytes) - added by SirLouen 13 months ago.
Patch for Sorting Deprecation Messages

Download all attachments as: .zip

Change History (6)

#1 @SirLouen
13 months ago

  • Component changed from General to Extend - BuddyPress
  • Keywords has-patch added
  • Priority changed from normal to low
  • Severity changed from normal to trivial
  • Version changed from 2.6.9 to 2.6.11

Basically, the problem is described here
https://wiki.php.net/rfc/deprecate_dynamic_properties
It will be deprecated and cause an error by PHP 9.0

According to the proposal, There is a simple solution. I attach a patch for this.

@SirLouen
13 months ago

Patch for Sorting Deprecation Messages

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


13 months ago
#2

#3 @emaralive
3 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

@johnjamesjacoby,

I may be wrong, but this ticket appears to have fallen through the cracks. Seems as though this should have been closed by changeset 7306.

Additionally, the attached Pull Request #22 appears to be open and, perhaps, should be closed, as well.

#4 @dd32
3 months ago

  • Resolution changed from invalid to fixed

Marking as resolved via [7305] + [7306]. Closing PR.

@dd32 commented on PR #22:


3 months ago
#5

Closing as a fix was committed in 1c2490bbed904fc69289c1ffce37df470a66103d

Note: See TracTickets for help on using tickets.