Fix VariableWrong false positives for partial doc blocks

[dereuromark]

Summary

The VariableWrong check in DocBlockParamSniff was producing false positives when a doc block only documented some parameters (e.g., only those needing complex type hints like array<string>).

Problem

Given this code:

/**
 * @param array<string> $lines
 */
protected function parseBlocks(Node $parent, array $lines, int $indent): void

The sniff would report:

Doc Block param variable `$lines` should be `$parent`

This happened because params were compared by position (1st doc block param vs 1st method param) rather than by name.

Solution

Match doc block params to method params by variable name instead of position. This allows:

• Partial documentation (only documenting params that need complex type hints)
• Correct error reporting when a doc block param doesn't exist in the method signature at all

Test case

/**
 * @param array<string> $lines  <- This is the 2nd method param, but 1st in docblock
 */
protected function parseBlocks(Node $parent, array $lines, int $indent): void

Previously: Error (false positive)
Now: No error (correct - $lines exists in method signature)

🤖 Generated with Claude Code