[wp-trac] [WordPress Trac] #57386: Add filter to WP_Upgrader::install_package for copy_dir()

WordPress Trac noreply at wordpress.org
Tue Jan 10 16:45:55 UTC 2023


#57386: Add filter to WP_Upgrader::install_package for copy_dir()
------------------------------------+--------------------------
 Reporter:  afragen                 |       Owner:  (none)
     Type:  feature request         |      Status:  reopened
 Priority:  normal                  |   Milestone:  6.2
Component:  Upgrade/Install         |     Version:  trunk
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |     Focuses:  performance
------------------------------------+--------------------------

Comment (by SergeyBiryukov):

 Replying to [comment:27 afragen]:
 > I like @costdev's above plan for testing VirtualBox without #56057. This
 should mean that #56057 can be reverted with the understanding that after
 testing it would simply be a switch to `move_dir()` or we would need the
 PR presented here.

 Same here, the plan outlined in comment:26 sounds great to me.

 > I still think it's appropriate to commit #57375 as `move_dir()` will be
 a great addition to Core and is needed regardless of which path we end up
 taking.

 I agree, the PR on #57375 looks good to me. That said, I believe we
 generally only introduce new core functions once they are actually used in
 core, so perhaps it can be committed as soon as the approach here is
 finalized.

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


More information about the wp-trac mailing list