[wp-trac] [WordPress Trac] #14662: Posts and Terms APIs allow loops in hierarchical data
WordPress Trac
wp-trac at lists.automattic.com
Mon Oct 11 21:14:38 UTC 2010
#14662: Posts and Terms APIs allow loops in hierarchical data
--------------------------+-------------------------------------------------
Reporter: mdawaffe | Owner: westi
Type: defect (bug) | Status: accepted
Priority: normal | Milestone: Awaiting Review
Component: General | Version: 3.0.1
Severity: normal | Keywords:
--------------------------+-------------------------------------------------
Comment(by ryan):
I started on a simple-minded patch before noticing this ticket.
Basically, it adds simple loop detection to _get_post_ancestors(), makes
sure ->ancestors is always populated after calling get_post(), and moves
places doing their own loops to use ->ancestors. The loop detection just
looks to see if an ID is repeated anywhere in the ancestry. It doesn't try
to do full loop detection like Mike's patch. Mike's better detection
triggered via filters could be merged in with my ancestors cleanup.
The ancestors cleanup also fixes issues like #10381 and #14329.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/14662#comment:7>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list