[wp-trac] [WordPress Trac] #57375: Add move_dir() function

WordPress Trac noreply at wordpress.org
Tue Jan 24 22:15:57 UTC 2023


#57375: Add move_dir() function
----------------------------+--------------------------
 Reporter:  afragen         |       Owner:  (none)
     Type:  enhancement     |      Status:  new
 Priority:  normal          |   Milestone:  6.2
Component:  Filesystem API  |     Version:  trunk
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:  performance
----------------------------+--------------------------

Comment (by peterwilsoncc):

 As discussed on [https://github.com/WordPress/wordpress-develop/pull/3897
 PR#3897], the direct file system is the only class in which the `::move()`
 method does not handle both files and directories.

 I'm of the view that this is a bug and it's probably best to modify the
 direct filesystems method to support both files and directories if
 possible.

 Whether or not a global `move_dir()` function is created as an alias is
 probably worth discussing, as it would be consistent with the `copy_fir()`
 function.

 I see this was discussed in passing during the meeting a few hours ago, so
 it would be lovely if a few people could share their thoughts on this
 ticket.

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


More information about the wp-trac mailing list