Opened 9 months ago
Closed 9 months ago
#63219 closed defect (bug) (fixed)
Minor Code Improvement: Adjust $mailserver_url Condition Order in wp-mail.php
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 6.9 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Keywords: | has-patch | |
| Focuses: | Cc: |
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.
Thanks for ticket and PR.
Move to
6.9