WP_Ajax_Response
has one property only, $responses
. It was public until [28508], when it became private
in name only. Is it worth 4 magic methods to pretend that this property is private
? It is not.
Make it `public` and remove the magic methods. See #30891. git-svn-id: https://develop.svn.wordpress.org/trunk@31139 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
parent
ad81bf20db
commit
f323dc9b48
@ -11,9 +11,8 @@ class WP_Ajax_Response {
|
|||||||
*
|
*
|
||||||
* @since 2.1.0
|
* @since 2.1.0
|
||||||
* @var array
|
* @var array
|
||||||
* @access private
|
|
||||||
*/
|
*/
|
||||||
private $responses = array();
|
public $responses = array();
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Constructor - Passes args to {@link WP_Ajax_Response::add()}.
|
* Constructor - Passes args to {@link WP_Ajax_Response::add()}.
|
||||||
@ -28,58 +27,6 @@ class WP_Ajax_Response {
|
|||||||
$this->add($args);
|
$this->add($args);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* Make private properties readable for backwards compatibility.
|
|
||||||
*
|
|
||||||
* @since 4.0.0
|
|
||||||
* @access public
|
|
||||||
*
|
|
||||||
* @param string $name Property to get.
|
|
||||||
* @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 );
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Append to XML response based on given arguments.
|
* Append to XML response based on given arguments.
|
||||||
*
|
*
|
||||||
|
Loading…
Reference in New Issue
Block a user