Changeset 46125 for trunk/src/wp-admin/includes/class-bulk-upgrader-skin.php

Timestamp:

09/15/2019 10:41:03 AM (7 years ago)

Author:

SergeyBiryukov

Message:

Code Modernisation: Introduce the spread operator in wp-admin/includes/class-*-upgrader-skin.php.

Rather than relying func_get_args() to retrieve arbitrary function arguments, we can now use the spread operator to assign them directly to a variable.

Props jrf.
See #47678.

File:

• trunk/src/wp-admin/includes/class-bulk-upgrader-skin.php (modified) (2 diffs)

trunk/src/wp-admin/includes/class-bulk-upgrader-skin.php

@@ -51 +51 @@
         /**
          * @param string $string
+         * @param mixed  ...$args Optional text replacements.
          */
-        public function feedback( $string ) {
+        public function feedback( $string, ...$args ) {
                 if ( isset( $this->upgrader->strings[ $string ] ) ) {
                         $string = $this->upgrader->strings[ $string ];
@@ -58 +59 @@
 
                 if ( strpos( $string, '%' ) !== false ) {
-                        $args = func_get_args();
-                        $args = array_splice( $args, 1 );
                         if ( $args ) {
                                 $args   = array_map( 'strip_tags', $args );