[wp-trac] [WordPress Trac] #58206: Replace usage of strpos with str_contains

WordPress Trac noreply at wordpress.org
Wed Jun 21 22:59:29 UTC 2023


#58206: Replace usage of strpos with str_contains
-------------------------------------+-------------------------------------
 Reporter:  spacedmonkey             |       Owner:  SergeyBiryukov
     Type:  enhancement              |      Status:  assigned
 Priority:  normal                   |   Milestone:  6.3
Component:  General                  |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  good-first-bug has-      |     Focuses:  performance, coding-
  patch 2nd-opinion commit           |  standards
-------------------------------------+-------------------------------------

Comment (by peterwilsoncc):

 I think this can go in as a way to encourage modern practices and making
 the code easier to read for developers.

 I agree with Felix, this can be decoupled from a performance discussion.
 Any changes appear to be a rounding error. I suspect people are seeing
 different results because the difference is environmental and effected by
 other processes.

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


More information about the wp-trac mailing list