[wp-trac] [WordPress Trac] #61860: PHPCS: Empty FOREACH statement detected

WordPress Trac noreply at wordpress.org
Tue Aug 13 07:15:07 UTC 2024


#61860: PHPCS: Empty FOREACH statement detected
-----------------------------------------+------------------------------
 Reporter:  dhruvang21                   |       Owner:  (none)
     Type:  enhancement                  |      Status:  new
 Priority:  normal                       |   Milestone:  Awaiting Review
Component:  Formatting                   |     Version:  4.2.3
 Severity:  normal                       |  Resolution:
 Keywords:  changes-requested has-patch  |     Focuses:  docs
-----------------------------------------+------------------------------

Comment (by TobiasBg):

 I like @jrf's suggestion as it's concise and easily readable. The
 previously/currently used `foreach` approach might be efficient, but its
 intention is overlooked easily. To improve readability, changing this
 makes sense. @dhruvang21 or @mi5t4n, would you maybe like to prepare this
 as an updated patch?

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


More information about the wp-trac mailing list