Skip to:
Content

bbPress.org

Opened 2 years ago

Closed 5 months ago

#3508 closed defect (bug) (fixed)

BBP_Converter_DB does not have a method “__destruct

Reported by: robin-w's profile Robin W Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.6.10 Priority: high
Severity: normal Version:
Component: API - Importers Keywords: has-patch
Cc:

Description (last modified by dd32)

https://bbpress.org/forums/topic/bbp_converter_db-does-not-have-a-method-__destruct/#post-233566

line 33 of

says

register_shutdown_function( array( $this, '__destruct' ) );

but there is no function for this in bbpress

Attachments (1)

3508.patch (555 bytes) - added by GDragoN 10 months ago.
Removed register_shutdown_function call.

Download all attachments as: .zip

Change History (5)

#2 @johnjamesjacoby
23 months ago

  • Component changed from General to API - Importers
  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 2.6.10
  • Owner set to johnjamesjacoby
  • Priority changed from normal to high
  • Status changed from new to assigned

@GDragoN
10 months ago

Removed register_shutdown_function call.

#3 @GDragoN
10 months ago

  • Keywords has-patch added; needs-patch removed

#4 @johnjamesjacoby
5 months ago

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

In 7256:

Converter: remove __destruct() registration

This change removes the need for wpdb extensions (like in the Converter) to register their own shutdown methods.

This is necessary because the __destruct() method was removed in the parent wpdb class, so attempting to add it in the subclass will generate PHP errors.

See #WP21402.

Fixes #3508.

Props GDragoN, dd32.

In branches/2.6, for 2.6.10.

Note: See TracTickets for help on using tickets.