WordPress.org
Get WordPress

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#53005 closed enhancement (fixed)

Remove unused code fragment from wp-admin/themes.php

Reported by: rkradadiya's profile rkradadiya Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.8 Priority: normal
Severity: normal Version:
Component: Themes Keywords: has-screenshots has-patch
Focuses: docs, coding-standards Cc:

Description

Hello,

WordPress has a comment structure.
But developers have used a Single-line comment structure into Multi-line comment in the below file.

wp-admin/themes.php

I think we have used only one single or Multi
Right?

Thanks,
Rajesh

Attachments (2)

Screenshot_2.png​ (14.5 KB) - added by rkradadiya 5 years ago.
53005.patch​ (909 bytes) - added by mukesh27 5 years ago.
Initial patch

Download all attachments as: .zip

Change History (6)

@rkradadiya
5 years ago

#1 @mukesh27
5 years ago

  • Keywords has-patch added
  • Version 5.7 deleted

Hi there!

Thanks for the ticket.

I think we must remove that commented code from the file.

@mukesh27
5 years ago

Initial patch

#2 @rkradadiya
5 years ago

@mukesh27
Thanks for the quick opinion and patch.

#3 @SergeyBiryukov
5 years ago

  • Keywords dev-feedback removed
  • Milestone changed from Awaiting Review to 5.8
  • Summary changed from Comment Strucure Not followed in Core - Theme.php to Remove unused code fragment from wp-admin/themes.php
  • Type changed from feature request to enhancement

#4 @SergeyBiryukov
5 years ago

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

In 50694:

Themes: Remove unused code fragment from wp-admin/themes.php.

Follow-up to [25666], [26141].

Props mukesh27, rkradadiya.
Fixes #53005.

Note: See TracTickets for help on using tickets.