5
93%
master: 93%

Ran 04 Jan 2015 03:59PM UTC

Files 93

Run time 47s

Badge

Embed ▾

Committed 04 Jan 2015 03:59PM UTC coverage: 74.349%. Remained the same

Job # 2284.5

Build Type

push

travis-ci

Committed by Dariusz Rumiński

Commit Message

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

This PR was merged into the 2.0-dev branch.

Discussion
----------

ParenthesisFixer - fix case with list call with trailing comma

Fix #930
Note: Should be merged into 1.4 branch as setted in milestone, not master as setted in PR.

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
-------

b02b097 ParenthesisFixer - fix case with list call with trailing comma

Coverage Stats

971 of 1306 relevant lines covered (74.35%)

20.7 hits per line

FriendsOfPHP / PHP-CS-Fixer / 2284 / 5
93%
master: 93%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2284.5

FriendsOfPHP / PHP-CS-Fixer / 2284 / 5 93% master: 93%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2284.5

FriendsOfPHP / PHP-CS-Fixer / 2284 / 5
93%
master: 93%

README BADGES
x