WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#32834 closed defect (bug) (fixed)

Site Icon: File name doesn't match nomenclature pattern

Reported by: kraftbj Owned by: obenland
Milestone: 4.3 Priority: normal
Severity: trivial Version: 4.3
Component: Administration Keywords: has-patch
Focuses: Cc:
PR Number:

Description

src/wp-admin/includes/site-icon.php is for the WP_Site_Icon class. The rest of the /wp-admin/includes/ that are classes are named class-[class-name].php.

Attachments (1)

32834.patch (550 bytes) - added by kraftbj 4 years ago.
along with a svn cp site-icon.php class-wp-site-icon.php

Download all attachments as: .zip

Change History (4)

@kraftbj
4 years ago

along with a svn cp site-icon.php class-wp-site-icon.php

#1 @kraftbj
4 years ago

  • Keywords has-patch added
  • Severity changed from normal to trivial

See #16434 (main ticket)

Last edited 4 years ago by kraftbj (previous) (diff)

#2 @obenland
4 years ago

  • Milestone changed from Awaiting Review to 4.3
  • Owner set to obenland
  • Status changed from new to accepted

#3 @obenland
4 years ago

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

In 33001:

Use existing nomenclature pattern for Site Icon class.

Props kraftbj.
Fixes #32834.

Note: See TracTickets for help on using tickets.