[wp-trac] [WordPress Trac] #41889: Missing Break Statement In class-pclzip php file

WordPress Trac noreply at wordpress.org
Sat Sep 16 21:03:42 UTC 2017


#41889: Missing Break Statement In class-pclzip php file
-----------------------------------+------------------------------
 Reporter:  adnan.limdi            |       Owner:
     Type:  defect (bug)           |      Status:  new
 Priority:  low                    |   Milestone:  Awaiting Review
Component:  Filesystem API         |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch 2nd-opinion  |     Focuses:
-----------------------------------+------------------------------
Changes (by johnbillion):

 * keywords:  has-patch => has-patch 2nd-opinion
 * priority:  normal => low
 * focuses:  docs =>
 * component:  General => Filesystem API


Comment:

 A missing `break` statement does not automatically indicate an error in
 the code. An expression can match multiple `case` conditions and
 intentionally fall through (this happens in the large `switch` statement
 in the `map_meta_cap()` function).

 In this case, it does look like the `break` statement is missing, but I
 don't believe there is a reason to "fix" this unless a bug can be
 identified.

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


More information about the wp-trac mailing list