[wp-trac] [WordPress Trac] #55712: Remove _copy_dir() as originally intended

WordPress Trac noreply at wordpress.org
Thu May 12 03:54:08 UTC 2022


#55712: Remove _copy_dir() as originally intended
------------------------------+---------------------
 Reporter:  afragen           |       Owner:  (none)
     Type:  enhancement       |      Status:  new
 Priority:  normal            |   Milestone:  6.1
Component:  Upgrade/Install   |     Version:  3.2
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |     Focuses:
------------------------------+---------------------

Comment (by peterwilsoncc):

 Expanding on any back-compat concerns, to access the function does require
 a `require` or `include`.

 {{{
 wp> function_exists( 'copy_dir' );
 bool(true)

 wp> function_exists( '_copy_dir' );
 bool(false)
 }}}

 Additionally the function does not appear in the developer documentation
 due to an `@ignore` directive.

 There is still a part of me tempted to deprecate the function in place
 though. It's not a lot of effort to avoid fatal errors:

 {{{#!php
 <?php
 function _copy_dir( $from, $to, $skip_list = array() ) {
         _deprecated_function( __FUNCTION__, 'copy_dir' );
         return copy_dir( $from, $to, $skip_list ); /* optionally */
 }

 }}}

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


More information about the wp-trac mailing list