10374975250
79%

Ran 13 Aug 2024 06:16PM UTC

Jobs 0

Files 0

Run time –

Badge

Embed ▾

pending completion

Build # 10374975250

Build Type

push

github

Committed by

jrfnl

Commit Message

Squiz/EmbeddedPhp: fix false positive when handling short open tags

The content of a PHP long open tag token is `<?php ` (note
the space after the tag). The content of a PHP short open tag is
`<?` (no space after the tag). The sniff did not account correctly for
this difference when checking the expected number of spaces after a
short open tag, resulting in false positives and incorrect fixes.

For example, the code below:

```
<? echo 'one space after short open tag'; ?>
```

Resulted in the error (there is just one space after the opening tag and
not two, as stated in the error):

```
Expected 1 space after opening PHP tag; 2 found (Squiz.PHP.EmbeddedPhp.SpacingAfterOpen)
```

And the incorrect fix:

```
<?echo 'without space after short open tag'; ?>
```

This commit fixes this problem by changing the sniff code to consider
that only long open tags contain a space after the tag in the `content`
key of the token array.

Run Details

Source Files on build 10374975250

Detailed source file information is not available for this build.

PHPCSStandards / PHP_CodeSniffer / 10374975250
79%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 10374975250

PHPCSStandards / PHP_CodeSniffer / 10374975250 79%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 10374975250

PHPCSStandards / PHP_CodeSniffer / 10374975250
79%

README BADGES
x