Method params incorrectly detected when default value uses short array syntax

[joshdavis11]

*When code sniffing for doc blocks, PHPCS broke when the PHP 5.4 short array syntax was introduced.

*To fix this, I added a check for a bracket_opener and then skipped to the bracket_closer similar to the check for a parenthesis_opener/closer.

[aik099]

Can you please add code example where it fails and used code standard?

[joshdavis11]

This is what was failing:

/**
 * Complete a step
 *
 * @param string $status status of step to complete
 * @param array  $array  array
 * @param string $note   optional note
 *
 * @return void
 */
public function completeStep($status, array $array = [Class1::class, 'test'], $note = '') {
...
}

FILE: /Users/jdavis/repos/main/test.php
----------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
----------------------------------------------------------------------
 8 | ERROR | Doc comment for parameter $note does not match actual
   |       | variable name $array
----------------------------------------------------------------------

The coding standards used are:
PEAR, PHPCS, PSR1, PSR2, Squiz and Zend

[aik099]

Now I clearly can see the problem. Thanks.

[joshdavis11]

One other note: it only seemed to break when $array was not the last parameter. If it was the last parameter, it was fine (because it wasn't trying to parse the next parameter).

[gsherwood]

Thanks a lot for fixing this.