[wp-trac] [WordPress Trac] #35593: Metadata from wrong file in media_handle_upload

[WordPress Trac]

#35593: Metadata from wrong file in media_handle_upload
-------------------------------------------------+-------------------------
 Reporter:  normanrz                             |       Owner:  joemcgill
     Type:  defect (bug)                         |      Status:  reviewing
 Priority:  normal                               |   Milestone:  Awaiting
Component:  Media                                |  Review
 Severity:  normal                               |     Version:
 Keywords:  has-patch needs-testing reporter-    |  Resolution:
  feedback                                       |     Focuses:
                                                 |  administration
-------------------------------------------------+-------------------------
Changes (by joemcgill):

 * keywords:  has-patch needs-testing => has-patch needs-testing reporter-
     feedback
 * owner:   => joemcgill
 * status:  new => reviewing


Comment:

 Hi @normanrz,

 Thanks for the submission. I can see how this would indeed cause
 unexpected behavior.

 However, it seems like a plugin that is firing on an action hook to change
 the filename should also take responsibility to update the metadata using
 the `wp_update_attachment_metadata` hook rather than adding an extra DB
 hit for everyone. Any reason why that wouldn't be a workable solution?

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