#58053 closed defect (bug) (fixed)
Fixing Multi-line PHP Snippet Embedding in wp-admin/network/user-new.php
| Reported by: | faisalahammad | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.3 |
| Component: | Administration | Version: | 6.2 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | coding-standards |
Description
I've noticed that the file wp-admin/network/user-new.php on line 103 has a multi-line PHP snippet that violates the recommended coding standard of WordPress. Specifically, the closing PHP tag is not on a new line, making the code harder to read and understand.
To adhere to the WordPress coding standard and make the code easier to maintain, I propose to fix this issue by moving the closing PHP tag to a new line.
Change History (6)
This ticket was mentioned in PR #4286 on WordPress/wordpress-develop by @faisalahammad.
3 years ago
#1
- Keywords has-patch added; needs-patch removed
@audrasjb commented on PR #4286:
3 years ago
#2
@faisalahammad thanks for the PR! However, it looks like it doesn't contain the change described in the ticket, and that it adds an unrelated change instead :)
@faisalahammad commented on PR #4286:
3 years ago
#3
Thank you @audrasjb
I've pushed down the ?> to a new line. Could you please recheck and let me know if I've missed anything?
@SergeyBiryukov commented on PR #4286:
3 years ago
#6
Thanks for the PR! Merged in r55618.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
I've noticed that the file wp-admin/network/user-new.php on line 103 has a multi-line PHP snippet that violates the recommended coding standard of WordPress. Specifically, the closing PHP tag is not on a new line, making the code harder to read and understand.
To adhere to the WordPress coding standard and make the code easier to maintain, I propose to fix this issue by moving the closing PHP tag to a new line. I have created a pull request that implements this change.
Please review and merge the pull request if you find it acceptable. Thank you for your attention to this matter.
Trac ticket: https://core.trac.wordpress.org/ticket/58053