[wp-trac] [WordPress Trac] #56689: Use WP_Query in get_page_by_path

[WordPress Trac]

#56689: Use WP_Query in get_page_by_path
-------------------------------------------------+-------------------------
 Reporter:  spacedmonkey                         |       Owner:
                                                 |  spacedmonkey
     Type:  defect (bug)                         |      Status:  closed
 Priority:  normal                               |   Milestone:
Component:  Query                                |     Version:  trunk
 Severity:  normal                               |  Resolution:  wontfix
 Keywords:  needs-testing has-patch has-unit-    |     Focuses:
  tests has-testing-info 2nd-opinion             |  performance
-------------------------------------------------+-------------------------

Comment (by SergeyBiryukov):

 Replying to [comment:20 spacedmonkey]:
 > We can reopen in the future if can work out another way around this
 issue.

 What if we replace the direct SQL query, but also keep the function's own
 caching? Then the infinite loop should not be triggered if my
 understanding is correct.

 [attachment:56689.diff] illustrates the idea. The tests still pass with
 this patch.

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