[wp-trac] [WordPress Trac] #11334: Add caching to get_page_of_comment()

WordPress Trac noreply at wordpress.org
Tue Feb 17 03:43:40 UTC 2015


#11334: Add caching to get_page_of_comment()
--------------------------+--------------------------
 Reporter:  Viper007Bond  |       Owner:
     Type:  enhancement   |      Status:  new
 Priority:  high          |   Milestone:  4.2
Component:  Comments      |     Version:  2.9
 Severity:  major         |  Resolution:
 Keywords:  has-patch     |     Focuses:  performance
--------------------------+--------------------------

Comment (by boonebgorges):

 [11334.4.diff] updates 3.diff as follows:

 - fixes 'type' regression noted by DrewAPicture above
 - improves function documentation
 - improves cache invalidation. Looking the patch over, I realized that
 changing changing the status of a comment has the potential to change the
 pagination values for any other comment on the post. So the cache-clearing
 callback now loops through all comments on the post in question.

 I'm leaving the patch to simmer here for a few days before I circle back
 around to recheck my work :)

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


More information about the wp-trac mailing list