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

WordPress Trac noreply at wordpress.org
Mon May 29 19:01:03 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 spacedmonkey):

 Maybe the performance lead, @flixos90 could take a look into this.

 To be clear, if they’re a slight performance loss, I think this change is
 worth for the code quality improvement.

 Worth noting, the performance team is currently working on docs on how we
 run our benchmarks. That way we could run the same benchmarks and we could
 see similar results.

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


More information about the wp-trac mailing list