WordPress.org

Make WordPress Core

Opened 2 months ago

Closed 2 months 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:

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 2 months ago.
along with a svn cp site-icon.php class-wp-site-icon.php

Download all attachments as: .zip

Change History (4)

@kraftbj — 2 months ago

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

comment:1 @kraftbj — 2 months ago

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

See #16434 (main ticket)

Last edited 2 months ago by kraftbj (previous) (diff)

comment:2 @obenland — 2 months ago

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

comment:3 @obenland — 2 months 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.