[wp-trac] [WordPress Trac] #36711: Add caching to get_page_by_path

WordPress Trac noreply at wordpress.org
Sat May 21 17:21:20 UTC 2016


#36711: Add caching to get_page_by_path
-------------------------------------------------+-------------------------
 Reporter:  spacedmonkey                         |       Owner:
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  4.6
Component:  Posts, Post Types                    |     Version:  2.1
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests dev-        |     Focuses:
  feedback                                       |  performance
-------------------------------------------------+-------------------------

Comment (by boonebgorges):

 > I believe they should look something like this.

 Ah yes, thanks. I've made the change and added a test that describes it.

 > One thing I have never understand about this function, why doesn't it
 return anything if nothing is found? It could at least return null or
 false.

 The answer, if there is one, lies in the log. Go to the log, my son. Go to
 the log. (And open a new ticket if you want to change it, and have
 supporting info from the log.)

 Let's put this in and see what breaks.

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


More information about the wp-trac mailing list