Opened 17 months ago
Closed 17 months ago
#63219 closed defect (bug) (fixed)
Minor Code Improvement: Adjust $mailserver_url Condition Order in wp-mail.php
| Reported by: | dilipbheda | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.9 |
| Component: | Version: | ||
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
Description
Reordered the condition in the $mailserver_url check to prioritize the empty() check.
Reason for Change:
- Improves readability and logic consistency by checking for an empty value first.
- Prevents unnecessary string comparison if
$mailserver_urlis empty.
It follows a common best practice of checking for an empty value before doing a specific comparison.
Change History (3)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Thanks for ticket and PR.
Move to
6.9