[wp-trac] [WordPress Trac] #19225: WordPress-Importer : Perform less database queries when backfilling attachment urls

WordPress Trac wp-trac at lists.automattic.com
Thu Nov 10 22:51:08 UTC 2011


#19225: WordPress-Importer : Perform less database queries when backfilling
attachment urls
-------------------------------------+----------------------------
 Reporter:  tott                     |       Owner:  duck_
     Type:  enhancement              |      Status:  assigned
 Priority:  normal                   |   Milestone:  WordPress.org
Component:  Import                   |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |
-------------------------------------+----------------------------

Comment (by duck_):

 [attachment:better-attachment-url-backfilling.2.diff] includes:

 {{{
 count( $post_ids ) > 0
 }}}

 but $post_ids isn't being declared in this version of the patch. Should be
 $posts?

 Could you explain what you mean by "invalidation of a lot of data" and
 "data that's being invalidated"? Just want to make sure I fully understand
 the why :)

 Another minor thing, but I think the uksort() before the first loop is
 unnecessary now.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/19225#comment:6>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list