Ticket #39593: 39593.2.diff

src/wp-admin/includes/class-automatic-upgrader-skin.php

@@ -19 +19 @@
  * @see Bulk_Upgrader_Skin
  */
 class Automatic_Upgrader_Skin extends WP_Upgrader_Skin {
+
+        /**
+         * Variable that holds internal messages of array in this class.
+         *
+         * @access protected
+         * @var array
+         */
         protected $messages = array();
 
         /**
@@ -42 +49 @@
                         $this->options['context'] = $context;
                 }
                 // TODO: fix up request_filesystem_credentials(), or split it, to allow us to request a no-output version
-                // This will output a credentials form in event of failure, We don't want that, so just hide with a buffer
+                // This will output a credentials form in event of failure, We don't want that, so just hide with a buffer.
                 ob_start();
                 $result = parent::request_filesystem_credentials( $error, $context, $allow_relaxed_file_ownership );
                 ob_end_clean();
@@ -50 +57 @@
         }
 
         /**
+         * Returns upgrade messages.
+         *
          * @access public
          *
          * @return array
@@ -59 +68 @@
         }
 
         /**
-         * @param string|array|WP_Error $data
+         * Displays success or error message on the update process.
+         *
+         * @param string $data Upgrade function that needs feedback to show if success or fail.
          */
         public function feedback( $data ) {
                 if ( is_wp_error( $data ) ) {
@@ -98 +109 @@
         }
 
         /**
+         * Buffers the output.
+         *
          * @access public
          */
         public function header() {
@@ -105 +118 @@
         }
 
         /**
+         * Cleans the output buffer.
+         *
          * @access public
          */
         public function footer() {