3
78%
1.11: 82%

Ran 04 Jan 2015 04:09PM UTC

Files 89

Run time 42s

Badge

Embed ▾

Committed 04 Jan 2015 04:08PM UTC coverage: 78.073% (+0.01%) from 78.059%

Job # 289.3

Build Type

push

travis-ci

Committed by Dariusz Rumiński

Commit Message

bug #955 ParenthesisFixer - fix case with list call with trailing comma (keradus)

This PR was merged into the 1.4 branch.

Discussion
----------

ParenthesisFixer - fix case with list call with trailing comma

Fix #930
Clone #952

Consider following code:
`<?php list($path, $mode,) = foo();`

If we run MethodArgumentSpaceFixer on it, it will add a space after comma:
`<?php list($path, $mode, ) = foo();`

Then if we run ParenthesisFixer it will remove that space:
`<?php list($path, $mode,) = foo();`

This PR change behavior of ParenthesisFixer - stop removing space if there is trailing comma.
Also added a tests for it in both fixers.

Commits
-------

0db8e07 ParenthesisFixer - fix case with list call with trailing comma

Run Details

2496 of 3197 relevant lines covered (78.07%)

42.76 hits per line

keradus / PHP-CS-Fixer / 289 / 3
78%
1.11: 82%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 289.3

keradus / PHP-CS-Fixer / 289 / 3 78% 1.11: 82%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 289.3

keradus / PHP-CS-Fixer / 289 / 3
78%
1.11: 82%

README BADGES
x