[wp-trac] [WordPress Trac] #56982: Empty ELSE statement

WordPress Trac noreply at wordpress.org
Fri Nov 4 05:37:02 UTC 2022


#56982: Empty ELSE statement
-----------------------------+-------------------------------
 Reporter:  krunal265        |       Owner:  (none)
     Type:  defect (bug)     |      Status:  new
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  Upgrade/Install  |     Version:  1.5
 Severity:  major            |  Resolution:
 Keywords:                   |     Focuses:  coding-standards
-----------------------------+-------------------------------

Comment (by brookedot):

 > Does anyone have a strong opinion on whether it should be kept or
 removed?

 My two cents either add an error/log message with something like
 `wp_error` or remove it. I don't find the comment particularly helpful and
 we could easily rework the code comments to not require the empty `else`

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


More information about the wp-trac mailing list