1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
|
<?php
/**
* Trait for issuing warnings on deprecated access.
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation; either version 2 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License along
* with this program; if not, write to the Free Software Foundation, Inc.,
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
* http://www.gnu.org/copyleft/gpl.html
*
* @file
*/
/**
* Use this trait in classes which have properties for which public access
* is deprecated or implementation has been moved to another class.
* Set the list of properties in $deprecatedPublicProperties
* and make the properties non-public. The trait will preserve public access
* but issue deprecation warnings when it is needed.
*
* Example usage:
* class Foo {
* use DeprecationHelper;
* protected $bar;
* public function __construct() {
* $this->deprecatePublicProperty( 'bar', '1.21', __CLASS__ );
* $this->deprecatePublicPropertyFallback(
* 'movedValue',
* '1.35',
* function () {
* return MediaWikiServices()::getInstance()
* ->getNewImplementationService()->getValue();
* },
* function ( $value ) {
* MediaWikiServices()::getInstance()
* ->getNewImplementationService()->setValue( $value );
* }
* );
* }
* }
*
* $foo = new Foo;
* $foo->bar; // works but logs a warning
* $foo->movedValue = 10; // works but logs a warning
* $movedValue = $foo->movedValue; // also works
*
* Cannot be used with classes that have their own __get/__set methods.
*
* @since 1.32
*/
trait DeprecationHelper {
/**
* List of deprecated properties, in <property name> => [<version>, <class>,
* <component>, <getter>, <setter> ] format where <version> is the MediaWiki version
* where the property got deprecated, <class> is the
* the name of the class defining the property, <component> is the MediaWiki component
* (extension, skin etc.) for use in the deprecation warning) or null if it is MediaWiki.
* E.g. [ 'mNewRev' => [ '1.32', 'DifferenceEngine', null ]
* @var string[][]
*/
protected $deprecatedPublicProperties = [];
/**
* Whether to emit a deprecation warning when unknown properties are accessed.
*
* @var bool|array
*/
private $dynamicPropertiesAccessDeprecated = false;
/**
* Mark a property as deprecated. Only use this for properties that used to be public and only
* call it in the constructor.
*
* @note Providing callbacks makes it not serializable
*
* @param string $property The name of the property.
* @param string $version MediaWiki version where the property became deprecated.
* @param string|null $class The class which has the deprecated property. This can usually be
* guessed, but PHP can get confused when both the parent class and the subclass use the
* trait, so it should be specified in classes meant for subclassing.
* @param string|null $component
* @see wfDeprecated()
*/
protected function deprecatePublicProperty(
$property,
$version,
$class = null,
$component = null
) {
$this->deprecatedPublicProperties[$property] = [
$version,
$class ?: __CLASS__,
$component,
null, null
];
}
/**
* Mark a removed public property as deprecated and provide fallback getter and setter callables.
* Only use this for properties that used to be public and only
* call it in the constructor.
*
* @param string $property The name of the property.
* @param string $version MediaWiki version where the property became deprecated.
* @param callable|string $getter A user provided getter that implements a `get` logic
* for the property. If a string is given, it is called as a method on $this.
* @param callable|string|null $setter A user provided setter that implements a `set` logic
* for the property. If a string is given, it is called as a method on $this.
* @param string|null $class The class which has the deprecated property.
* @param string|null $component
*
* @since 1.36
* @see wfDeprecated()
*/
protected function deprecatePublicPropertyFallback(
string $property,
string $version,
$getter,
$setter = null,
$class = null,
$component = null
) {
$this->deprecatedPublicProperties[$property] = [
$version,
$class ?: __CLASS__,
null,
$getter,
$setter,
$component
];
}
/**
* Emit deprecation warnings when dynamic and unknown properties
* are accessed.
*
* @param string $version MediaWiki version where the property became deprecated.
* @param string|null $class The class which has the deprecated property.
* @param string|null $component
*/
protected function deprecateDynamicPropertiesAccess(
string $version,
string $class = null,
string $component = null
) {
$this->dynamicPropertiesAccessDeprecated = [ $version, $class ?: __CLASS__, $component ];
}
public function __isset( $name ) {
// Overriding magic __isset is required not only for isset() and empty(),
// but to correctly support null coalescing for dynamic properties,
// e.g. $foo->bar ?? 'default'
if ( isset( $this->deprecatedPublicProperties[$name] ) ) {
list( $version, $class, $component, $getter ) = $this->deprecatedPublicProperties[$name];
$qualifiedName = $class . '::$' . $name;
wfDeprecated( $qualifiedName, $version, $component, 2 );
if ( $getter ) {
return $this->deprecationHelperCallGetter( $getter );
}
return true;
}
$ownerClass = $this->deprecationHelperGetPropertyOwner( $name );
if ( $ownerClass ) {
// Someone tried to access a normal non-public property. Try to behave like PHP would.
return false;
} else {
if ( $this->dynamicPropertiesAccessDeprecated ) {
[ $version, $class, $component ] = $this->dynamicPropertiesAccessDeprecated;
$qualifiedName = $class . '::$' . $name;
wfDeprecated( $qualifiedName, $version, $component, 2 );
}
return false;
}
}
public function __get( $name ) {
if ( isset( $this->deprecatedPublicProperties[$name] ) ) {
list( $version, $class, $component, $getter ) = $this->deprecatedPublicProperties[$name];
$qualifiedName = $class . '::$' . $name;
wfDeprecated( $qualifiedName, $version, $component, 2 );
if ( $getter ) {
return $this->deprecationHelperCallGetter( $getter );
}
return $this->$name;
}
$ownerClass = $this->deprecationHelperGetPropertyOwner( $name );
$qualifiedName = ( $ownerClass ?: get_class( $this ) ) . '::$' . $name;
if ( $ownerClass ) {
// Someone tried to access a normal non-public property. Try to behave like PHP would.
trigger_error( "Cannot access non-public property $qualifiedName", E_USER_ERROR );
} elseif ( property_exists( $this, $name ) ) {
// Normally __get method will not be even called if the property exists,
// but in tests if we mock an object that uses DeprecationHelper,
// __get and __set magic methods will be mocked as well, and called
// regardless of the property existence. Support that use-case.
return $this->$name;
} else {
// Non-existing property. Try to behave like PHP would.
trigger_error( "Undefined property: $qualifiedName", E_USER_NOTICE );
}
return null;
}
public function __set( $name, $value ) {
if ( isset( $this->deprecatedPublicProperties[$name] ) ) {
list( $version, $class, $component, , $setter ) = $this->deprecatedPublicProperties[$name];
$qualifiedName = $class . '::$' . $name;
wfDeprecated( $qualifiedName, $version, $component, 2 );
if ( $setter ) {
$this->deprecationHelperCallSetter( $setter, $value );
} elseif ( property_exists( $this, $name ) ) {
$this->$name = $value;
} else {
trigger_error( "Cannot access non-public property $qualifiedName", E_USER_ERROR );
}
return;
}
$ownerClass = $this->deprecationHelperGetPropertyOwner( $name );
$qualifiedName = ( $ownerClass ?: get_class( $this ) ) . '::$' . $name;
if ( $ownerClass ) {
// Someone tried to access a normal non-public property. Try to behave like PHP would.
trigger_error( "Cannot access non-public property $qualifiedName", E_USER_ERROR );
} else {
if ( $this->dynamicPropertiesAccessDeprecated ) {
[ $version, $class, $component ] = $this->dynamicPropertiesAccessDeprecated;
$qualifiedName = $class . '::$' . $name;
wfDeprecated( $qualifiedName, $version, $component, 2 );
}
// Non-existing property. Try to behave like PHP would.
$this->$name = $value;
}
}
/**
* Like property_exists but also check for non-visible private properties and returns which
* class in the inheritance chain declared the property.
* @param string $property
* @return string|bool Best guess for the class in which the property is defined. False if
* the object does not have such a property.
*/
private function deprecationHelperGetPropertyOwner( $property ) {
// Returning false is a non-error path and should avoid slow checks like reflection.
// Use array cast hack instead.
$obfuscatedProps = array_keys( (array)$this );
$obfuscatedPropTail = "\0$property";
foreach ( $obfuscatedProps as $obfuscatedProp ) {
// private props are in the form \0<classname>\0<propname>
if ( strpos( $obfuscatedProp, $obfuscatedPropTail, 1 ) !== false ) {
$classname = substr( $obfuscatedProp, 1, -strlen( $obfuscatedPropTail ) );
if ( $classname === '*' ) {
// protected property; we didn't get the name, but we are on an error path
// now so it's fine to use reflection
return ( new ReflectionProperty( $this, $property ) )->getDeclaringClass()->getName();
}
return $classname;
}
}
return false;
}
private function deprecationHelperCallGetter( $getter ) {
if ( is_string( $getter ) ) {
$getter = [ $this, $getter ];
}
return $getter();
}
private function deprecationHelperCallSetter( $setter, $value ) {
if ( is_string( $setter ) ) {
$setter = [ $this, $setter ];
}
$setter( $value );
}
}
|