[wp-trac] [WordPress Trac] #63219: Minor Code Improvement: Adjust $mailserver_url Condition Order in wp-mail.php

WordPress Trac noreply at wordpress.org
Wed Apr 2 22:23:25 UTC 2025


#63219: Minor Code Improvement: Adjust $mailserver_url Condition Order in wp-
mail.php
--------------------------+-----------------------------
 Reporter:  dilipbheda    |       Owner:  SergeyBiryukov
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  6.9
Component:  Mail          |     Version:
 Severity:  normal        |  Resolution:  fixed
 Keywords:  has-patch     |     Focuses:
--------------------------+-----------------------------
Changes (by SergeyBiryukov):

 * owner:  (none) => SergeyBiryukov
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"60122" 60122]:
 {{{
 #!CommitTicketReference repository="" revision="60122"
 Coding Standards: Check for an empty mail server URL first in `wp-
 mail.php`.

 This follows a common best practice of checking for an empty value before
 doing a specific comparison.

 Follow-up to [39772].

 Props dilipbheda, mukesh27.
 Fixes #63219.
 }}}

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/63219#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list