[wp-trac] [WordPress Trac] #34430: Improve the performance of wp_make_content_images_responsive()

WordPress Trac noreply at wordpress.org
Wed Oct 28 14:43:36 UTC 2015


#34430: Improve the performance of wp_make_content_images_responsive()
------------------------------------------------+------------------------
 Reporter:  joemcgill                           |       Owner:  joemcgill
     Type:  defect (bug)                        |      Status:  assigned
 Priority:  normal                              |   Milestone:  4.4
Component:  Media                               |     Version:  trunk
 Severity:  normal                              |  Resolution:
 Keywords:  has-patch needs-testing needs-docs  |     Focuses:
------------------------------------------------+------------------------

Comment (by joemcgill):

 Replying to [comment:17 swissspidy]:
 > I wonder if we should make `_wp_upload_dir_baseurl()` public, as
 something similar was requested in #33963.

 I'm wondering the same thing. This function addresses the main issue
 brought up in #34359 so we also need to make sure the concerns mentioned
 in that ticket have been addressed. The approach @azaozz took for using a
 static variable inside `_wp_upload_dir_baseurl()` should allow us to avoid
 any collisions with persistent cache. I'm not sure about situations where
 a site is conditionally filtering the uploads directory or if that is even
 a concert in our context here.

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


More information about the wp-trac mailing list