3
78%
master: 79%

Ran 12 Mar 2025 10:43PM UTC

Files 262

Run time 11s

Badge

Embed ▾

Committed 12 Mar 2025 10:35PM UTC coverage: 2.057%. Remained the same

Job # 13822886692.3

Build Type

push

github

Committed by

jrfnl

Commit Message

Ruleset: bug fix - correctly handle empty array property setting

While already handled correctly for inline properties set via `phpcs:set`, setting a property to an empty array from a ruleset file was not handled correctly until now.

As things were, when setting an array property from the ruleset to an empty array like so:
```xml
    <rule ref="Standard.Category.SniffName">
        <properties>
            <property name="arrayProperty" type="array"/>
        </properties>
    </rule>
```

 the property value would be set to:
```php
public $arrayProperty = [
    0 => '',
];
```
... while the expected behaviour would be:
```php
public $arrayProperty = [];
```

While it is not expected that this type of property setting is encountered a lot in the wild, it should still work correctly.

Fixed now.

Includes test safeguarding the fix.

Run Details

627 of 30477 relevant lines covered (2.06%)

0.23 hits per line

PHPCSStandards / PHP_CodeSniffer / 13822886692 / 3
78%
master: 79%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job os-windows-latest-php-7.2-custom-ini-false - 13822886692.3

PHPCSStandards / PHP_CodeSniffer / 13822886692 / 3 78% master: 79%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job os-windows-latest-php-7.2-custom-ini-false - 13822886692.3

PHPCSStandards / PHP_CodeSniffer / 13822886692 / 3
78%
master: 79%

README BADGES
x