Opened 3 years ago
Closed 2 years ago
#60477 closed defect (bug) (wontfix)
Remove commented out code in wp-includes
| Reported by: | viralsampat | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | General | Version: | |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: | coding-standards |
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)
Change History (3)
#1
@
3 years ago
- Summary Remove unused comments code from the wp-include files → 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
@
2 years ago
- Milestone Awaiting Review
- Resolution → wontfix
- Status new → closed
Yes, the class-wp-query.php and functions.wp-styles.php comments are legitimate and need to stay.
- Since [4956], WP query has had a pair of comments to indicate where the
is_*correction begins and ends. - [4105] added the short note about fixing the call from
wp_head(ticket:2701#comment:19). - The
rss.phpfile is the deprecated MagpieRSS external library, and #57107 might remove the library's code. If not, #59548 already suggested similar edits. - To suggest any revisions to the sodium_compat external library, you could visit https://github.com/paragonie/sodium_compat
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Applied a patch