WordPress.org

Make WordPress Core

Opened 15 months ago

Closed 15 months ago

Last modified 13 months ago

#44812 closed defect (bug) (fixed)

Undefined variable unset

Reported by: Hitendra Chopda Owned by: SergeyBiryukov
Milestone: 5.1 Priority: normal
Severity: normal Version: 5.1
Component: Administration Keywords: has-patch 2nd-opinion
Focuses: Cc:
PR Number:

Description

I have checked out that the variable name $title_class is undefined and used it for unset.

Attachments (2)

admin-header.php.patch (775 bytes) - added by Hitendra Chopda 15 months ago.
This is patch file
44812.diff (442 bytes) - added by torontodigits 15 months ago.
Patch refreshed

Download all attachments as: .zip

Change History (7)

@Hitendra Chopda
15 months ago

This is patch file

#1 @torontodigits
15 months ago

  • Keywords 2nd-opinion added

Hello @hitendra-chopda,

Welcome to the Trac.

Thanks for reporting the bug.

Regarding the admin_body_class filter I don't think there should be inline documentation for $admin_body_classes parameter as this filter takes only on parameter and it is already documented.

I have refreshed the patch accordingly. Let's wait for second opinion as well.

Thanks

Last edited 15 months ago by torontodigits (previous) (diff)

@torontodigits
15 months ago

Patch refreshed

#2 @SergeyBiryukov
15 months ago

  • Component changed from General to Administration
  • Milestone changed from Awaiting Review to 5.0

#4 @SergeyBiryukov
15 months ago

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

In 43573:

Administration: Remove $title_class variable, unused since [18683].

Props hitendra-chopda, torontodigits.
Fixes #44812.

#5 @pento
13 months ago

  • Milestone changed from 5.0 to 5.1
Note: See TracTickets for help on using tickets.