Opened 3 years ago
Closed 6 months ago
#58337 closed defect (bug) (wontfix)
Indentation and Spacing Issue in rss.php File
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | low | |
| Severity: | normal | Version: | |
| Component: | Feeds | Keywords: | has-patch close |
| Focuses: | coding-standards | Cc: |
Description
I have noticed indentation and spacing issue within the rss.php file in WordPress. The problem lies in inconsistent indentation and spacing throughout the code, which hinders readability and comprehension.
Indentation and spacing issue found in the following lines of code within a wp-includes/rss.php
https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-includes/rss.php#L77
https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-includes/rss.php#L109
https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-includes/rss.php#L125
https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-includes/rss.php#L146
https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-includes/rss.php#L176
Change History (6)
This ticket was mentioned in PR #4465 on WordPress/wordpress-develop by madhusudan-hub.
3 years ago
#1
- Keywords has-patch added
#2
@
3 years ago
- Component changed from General to Feeds
Hello and thanks for the ticket,
Worth noting that MagpieRSS is an external library and its use in WordPress is deprecated since version 3.0, that's why it doesn't respect WordPress Coding Standards :)
This ticket was mentioned in Slack in #core-test by ironprogrammer. View the logs.
3 years ago
#5
@
3 years ago
- Keywords close added
This ticket was reviewed during test triage.
Because the class in question has been proposed to deprecate in #57107, we've added close to the ticket. Why? Rather than marking for dev-feedback, this will raise the ticket with core committers anyways, who can then evaluate moving it forward or not.
#6
@
6 months ago
- Milestone Awaiting Review deleted
- Resolution set to wontfix
- Status changed from new to closed
A committers meeting identified the desire to no-op or remove old deprecated classes as a short-term task. #57107 would be the direction to take now.
Indentation and Spacing Issue fixed.
Trac ticket: https://core.trac.wordpress.org/ticket/58337