Changeset 31133

Timestamp:

01/10/2015 09:58:19 PM (12 years ago)

Author:

wonderboymusic

Message:

In Custom_Background:

• In [28481], $admin_header_callback and $admin_image_div_callback were set to private based on their erroneous @param value
• $admin_header_callback and $admin_image_div_callback are used as hook callbacks - as such, they must be public
• In [28521] and [28524], magic methods were added for back-compat
• Currently, there are 2 properties marked private, $page and $updated - $page is never used and $updated was added by me in [30186] during 4.1

Set $admin_header_callback and $admin_image_div_callback to public.
Remove the $page property - it duplicated the $page local var and is referenced/used nowhere.
Remove the magic methods - they were beyond overkill and rendered moot by the above changes.

See #30891.

File:

• trunk/src/wp-admin/custom-background.php (modified) (4 diffs)

trunk/src/wp-admin/custom-background.php

@@ -21 +21 @@
          * @var callback
          * @since 3.0.0
-         * @access private
-         */
-        private $admin_header_callback;
+         */
+        public $admin_header_callback;
 
         /**
@@ -30 +29 @@
          * @var callback
          * @since 3.0.0
-         * @access private
-         */
-        private $admin_image_div_callback;
-
-        /**
-         * Holds the page menu hook.
-         *
-         * @var string
-         * @since 3.0.0
-         * @access private
-         */
-        private $page = '';
+         */
+        public $admin_image_div_callback;
 
         /**
@@ -65 +54 @@
                 // Unused since 3.5.0.
                 add_action( 'wp_ajax_set-background-image', array( $this, 'wp_set_background_image' ) );
-        }
-
-        /**
-         * Make private properties readable for backwards compatibility.
-         *
-         * @since 4.0.0
-         * @access public
-         *
-         * @param string $name Property name.
-         * @return mixed Property.
-         */
-        public function __get( $name ) {
-                return $this->$name;
-        }
-
-        /**
-         * Make private properties settable for backwards compatibility.
-         *
-         * @since 4.0.0
-         * @access public
-         *
-         * @param string $name  Property to set.
-         * @param mixed  $value Property value.
-         * @return mixed Newly-set property.
-         */
-        public function __set( $name, $value ) {
-                return $this->$name = $value;
-        }
-
-        /**
-         * Make private properties checkable for backwards compatibility.
-         *
-         * @since 4.0.0
-         * @access public
-         *
-         * @param string $name Property to check if set.
-         * @return bool Whether the property is set.
-         */
-        public function __isset( $name ) {
-                return isset( $this->$name );
-        }
-
-        /**
-         * Make private properties un-settable for backwards compatibility.
-         *
-         * @since 4.0.0
-         * @access public
-         *
-         * @param string $name Property to unset.
-         */
-        public function __unset( $name ) {
-                unset( $this->$name );
         }
 
@@ -129 +66 @@
                         return;
                 }
-
-                $this->page = $page;
 
                 add_action( "load-$page", array( $this, 'admin_load' ) );