Skip to content

Commit 7104de0

Browse files
committed
Improve DisableMagicSerializeSniff error message
1 parent e77a9e3 commit 7104de0

File tree

1 file changed

+6
-9
lines changed

1 file changed

+6
-9
lines changed

Inpsyde/Sniffs/CodeQuality/DisableMagicSerializeSniff.php

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,10 @@
1111

1212
class DisableMagicSerializeSniff implements Sniff
1313
{
14-
/** @var list<string> */
14+
/** @var array<string, string> */
1515
public array $disabledFunctions = [
16-
'__sleep',
17-
'__wakeup',
16+
'__sleep' => '__serialize',
17+
'__wakeup' => '__unserialize',
1818
];
1919

2020
/**
@@ -40,13 +40,10 @@ public function process(File $phpcsFile, $stackPtr): void
4040
}
4141

4242
$name = FunctionDeclarations::getName($phpcsFile, $stackPtr);
43-
if (in_array($name, $this->disabledFunctions, true)) {
43+
$alternative = $this->disabledFunctions[$name] ?? null;
44+
if ($alternative !== null) {
4445
$phpcsFile->addError(
45-
sprintf(
46-
'The method "%s" is deprecated, '
47-
. 'please use __serialize and __unserialize instead.',
48-
$name
49-
),
46+
"The method '{$name}' is deprecated, please use '{$alternative}' instead.",
5047
$stackPtr,
5148
'Found'
5249
);

0 commit comments

Comments
 (0)