Skip to content

Commit ff42244

Browse files
committed
[TASK] Deprecate method forwarding OutputFormat to OutputFormatter
Instead, the corresponding method on the formatter should be called directly. This increases type safety and helps static code analysis. Also, it makes the code easier to understand.
1 parent 5c440f3 commit ff42244

File tree

2 files changed

+3
-0
lines changed

2 files changed

+3
-0
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ Please also have a look at our
2929

3030
### Deprecated
3131

32+
- Deprecate magic method forwarding from `OutputFormat` to `OutputFormatter`
33+
(#894)
3234
- `OutputFormat` properties for space around list separators as an array (#880)
3335

3436
### Removed

src/OutputFormat.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -292,6 +292,7 @@ public function set($aNames, $mValue)
292292
public function __call(string $sMethodName, array $aArguments)
293293
{
294294
if (\method_exists(OutputFormatter::class, $sMethodName)) {
295+
// @deprecated since 8.8.0, will be removed in 9.0.0. Call the method on the formatter directly instead.
295296
return \call_user_func_array([$this->getFormatter(), $sMethodName], $aArguments);
296297
} else {
297298
throw new \Exception('Unknown OutputFormat method called: ' . $sMethodName);

0 commit comments

Comments
 (0)