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 Owned by: SergeyBiryukov
Priority: normal Milestone: 5.8
Component: Themes Version:
Severity: normal Keywords: has-screenshots has-patch
Cc: Focuses: docs, coding-standards

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

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 Awaiting Review → 5.8
  • Summary Comment Strucure Not followed in Core - Theme.php → Remove unused code fragment from wp-admin/themes.php
  • Type feature request → enhancement

#4 @SergeyBiryukov
5 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → 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.