4
93%
master: 93%

Ran 04 Jan 2015 04:00PM UTC

Files 93

Run time 44s

Badge

Embed ▾

Committed 04 Jan 2015 03:59PM UTC coverage: 81.815% (+0.01%) from 81.804%

Job # 2284.4

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

2695 of 3294 relevant lines covered (81.82%)

51.39 hits per line

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2284.4

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2284.4

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

README BADGES
x