WordPress.org

Make WordPress Core

Opened 18 months ago

Closed 15 months ago

Last modified 15 months ago

#22246 closed defect (bug) (fixed)

PHP Notice On _remove_theme_support() function

Reported by: alex-ye Owned by: ryan
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.4.2
Component: Warnings/Notices Keywords: has-patch
Focuses: Cc:

Description

Hi Guys , I just found this PHP Notice on _remove_theme_support() function , Tested On :

1 - WordPress 3.5 Beta 2

2 - PHP 5.4

To get this notice on just add child theme to TwentyTwelve and add this simple code on functions.php :

add_action ( 'after_setup_theme', 'Nash_setup' , 11 );

function Nash_setup() {

    remove_theme_support ( 'custom-background' );   
    remove_theme_support ( 'custom-header' );    

} // end Nash_setup()

Attachments (5)

theme.php.patch (1.4 KB) - added by alex-ye 18 months ago.
Using isset() to fix the bug
22246.patch (1.4 KB) - added by SergeyBiryukov 18 months ago.
With proper formatting
22246.diff (889 bytes) - added by nacin 18 months ago.
22246.2.diff (1.6 KB) - added by nacin 18 months ago.
Made against head.
theme.php.2.patch (1.2 KB) - added by alex-ye 15 months ago.
Using isset() after nacin patch

Download all attachments as: .zip

Change History (18)

alex-ye18 months ago

Using isset() to fix the bug

comment:1 alex-ye18 months ago

  • Keywords has-patch added; needs-patch removed

SergeyBiryukov18 months ago

With proper formatting

comment:2 SergeyBiryukov18 months ago

  • Component changed from Themes to Warnings/Notices
  • Milestone changed from Awaiting Review to 3.5

comment:3 ryan18 months ago

  • Owner set to ryan
  • Resolution set to fixed
  • Status changed from new to closed

In [22274]:

Avoid notices in _remove_theme_support(). Props alex-ye, SergeyBiryukov. fixes #22246

nacin18 months ago

comment:4 nacin18 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Rather than papering over the notices in [22274], we can bail in the particular situation where this occurs: When the "just in time" initialization hasn't occurred yet, we don't need to do anything beyond unsetting the item from the global array. See 22246.diff (made against pre-[22274]).

nacin18 months ago

Made against head.

comment:5 obenland18 months ago

#21905 was marked as a duplicate.

comment:6 ryan18 months ago

Fine with 22246.2.diff.

comment:7 nacin18 months ago

In [22312]:

Revert [22274]. see #22246.

comment:8 nacin18 months ago

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

In [22313]:

If we have not yet initialized the custom header/background, do not do extra removal in _remove_theme_support(). fixes #22246.

comment:9 alex-ye15 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Version 3.4.2 deleted

As I test the PHP notice is still exists even after Nacin patch .
I just add isset() function to check the array :)

comment:10 alex-ye15 months ago

  • Version set to 3.5

alex-ye15 months ago

Using isset() after nacin patch

comment:11 SergeyBiryukov15 months ago

  • Resolution set to fixed
  • Status changed from reopened to closed
  • Version changed from 3.5 to 3.4.2

This ticket was closed on a completed milestone. Please open a new one.

comment:13 SergeyBiryukov15 months ago

In 23367:

Properly check if the custom header/background has been initialized. fixes #23170. see #22246.

Note: See TracTickets for help on using tickets.