Make WordPress Core

Opened 8 months ago

Closed 7 months ago

#60477 closed defect (bug) (wontfix)

Remove commented out code in wp-includes

Reported by: viralsampat's profile viralsampat Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: General Keywords:
Focuses: coding-standards Cc:

Description

Hello

I have reviewed core files and found that a few files still contain comments code that is not in use.

But as per the coding standard needs to be removed it from the files.

Files:

wp-includes/class-wp-query.php
wp-includes/rss.php
wp-includes/functions.wp-styles.php src/wp-includes/functions.wp-styles.php
wp-includes/sodium_compat/src/Compat.php

Attachments (1)

60477.patch (2.3 KB) - added by viralsampat 8 months ago.
Applied a patch

Download all attachments as: .zip

Change History (3)

@viralsampat
8 months ago

Applied a patch

#1 @swissspidy
8 months ago

  • Summary changed from Remove unused comments code from the wp-include files to Remove commented out code in wp-includes

Can you elaborate on the changes to src/wp-includes/class-wp-query.php and src/wp-includes/functions.wp-styles.php in your patch? Those a legit comments, not commented out code.

#2 @sabernhardt
7 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Yes, the class-wp-query.php and functions.wp-styles.php comments are legitimate and need to stay.

Note: See TracTickets for help on using tickets.