[wp-trac] [WordPress Trac] #47083: Site Info Update Not Adjusting "Last Updated" Value

WordPress Trac noreply at wordpress.org
Thu May 9 16:30:42 UTC 2019


#47083: Site Info Update Not Adjusting "Last Updated" Value
--------------------------------+------------------------------
 Reporter:  conner_bw           |       Owner:  (none)
     Type:  defect (bug)        |      Status:  new
 Priority:  normal              |   Milestone:  Awaiting Review
Component:  Networks and Sites  |     Version:  5.1.1
 Severity:  normal              |  Resolution:
 Keywords:                      |     Focuses:  multisite
--------------------------------+------------------------------

Comment (by conner_bw):

 > The lines of code you called out:
 > https://github.com/WordPress/wordpress-develop/blob/5.0/src/wp-includes
 /ms-blogs.php#L297
 > actually do still exist.

 I disagree. It does not.

 The linked code explicitly overrides:

 `$details['last_updated'] = current_time('mysql', true);`

 ~7 lines later, the code does:

 `$result = $wpdb->update( $wpdb->blogs, $update_details, array('blog_id'
 => $blog_id) );`

 I don't see any ambiguity between those 7 lines of code? In 5.1,
 last_updated was always set to the current time.

 > abstracted into the new wp_update_site

 Whoever refactored the code changed the behaviour. If it was intentended,
 then they need to fix bulk update to match.

 I don't think it was?

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


More information about the wp-trac mailing list