Skip to content

Commit f37ca23

Browse files
sebastianbergmannsebastianbergmann
committed
Fix CS/WS issues
1 parent 89e3d48 commit f37ca23

File tree

18 files changed

+148
-150
lines changed

18 files changed

+148
-150
lines changed

‎src/CodeCoverage.php‎

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -28,32 +28,32 @@ class PHP_CodeCoverage
2828
private $filter;
2929

3030
/**
31-
* @var boolean
31+
* @var bool
3232
*/
3333
private $cacheTokens = false;
3434

3535
/**
36-
* @var boolean
36+
* @var bool
3737
*/
3838
private $checkForUnintentionallyCoveredCode = false;
3939

4040
/**
41-
* @var boolean
41+
* @var bool
4242
*/
4343
private $forceCoversAnnotation = false;
4444

4545
/**
46-
* @var boolean
46+
* @var bool
4747
*/
4848
private $mapTestClassNameToCoveredClassName = false;
4949

5050
/**
51-
* @var boolean
51+
* @var bool
5252
*/
5353
private $addUncoveredFilesFromWhitelist = true;
5454

5555
/**
56-
* @var boolean
56+
* @var bool
5757
*/
5858
private $processUncoveredFilesFromWhitelist = false;
5959

@@ -84,8 +84,8 @@ class PHP_CodeCoverage
8484
/**
8585
* Constructor.
8686
*
87-
* @param PHP_CodeCoverage_Driver $driver
88-
* @param PHP_CodeCoverage_Filter $filter
87+
* @param PHP_CodeCoverage_Driver $driver
88+
* @param PHP_CodeCoverage_Filter $filter
8989
* @throws PHP_CodeCoverage_Exception
9090
*/
9191
public function __construct(PHP_CodeCoverage_Driver $driver = null, PHP_CodeCoverage_Filter $filter = null)
@@ -148,7 +148,7 @@ public function filter()
148148
* Returns the collected code coverage data.
149149
* Set $raw = true to bypass all filters.
150150
*
151-
* @param bool $raw
151+
* @param bool $raw
152152
* @return array
153153
* @since Method available since Release 1.1.0
154154
*/
@@ -204,7 +204,7 @@ public function setTests(array $tests)
204204
* Start collection of code coverage information.
205205
*
206206
* @param mixed $id
207-
* @param boolean $clear
207+
* @param bool $clear
208208
* @throws PHP_CodeCoverage_Exception
209209
*/
210210
public function start($id, $clear = false)
@@ -228,7 +228,7 @@ public function start($id, $clear = false)
228228
/**
229229
* Stop collection of code coverage information.
230230
*
231-
* @param boolean $append
231+
* @param bool $append
232232
* @param mixed $linesToBeCovered
233233
* @param array $linesToBeUsed
234234
* @return array
@@ -263,7 +263,7 @@ public function stop($append = true, $linesToBeCovered = array(), array $linesTo
263263
*
264264
* @param array $data
265265
* @param mixed $id
266-
* @param boolean $append
266+
* @param bool $append
267267
* @param mixed $linesToBeCovered
268268
* @param array $linesToBeUsed
269269
* @throws PHP_CodeCoverage_Exception
@@ -377,7 +377,7 @@ public function merge(PHP_CodeCoverage $that)
377377
}
378378

379379
/**
380-
* @param boolean $flag
380+
* @param bool $flag
381381
* @throws PHP_CodeCoverage_Exception
382382
* @since Method available since Release 1.1.0
383383
*/
@@ -402,7 +402,7 @@ public function getCacheTokens()
402402
}
403403

404404
/**
405-
* @param boolean $flag
405+
* @param bool $flag
406406
* @throws PHP_CodeCoverage_Exception
407407
* @since Method available since Release 2.0.0
408408
*/
@@ -419,7 +419,7 @@ public function setCheckForUnintentionallyCoveredCode($flag)
419419
}
420420

421421
/**
422-
* @param boolean $flag
422+
* @param bool $flag
423423
* @throws PHP_CodeCoverage_Exception
424424
*/
425425
public function setForceCoversAnnotation($flag)
@@ -435,7 +435,7 @@ public function setForceCoversAnnotation($flag)
435435
}
436436

437437
/**
438-
* @param boolean $flag
438+
* @param bool $flag
439439
* @throws PHP_CodeCoverage_Exception
440440
*/
441441
public function setMapTestClassNameToCoveredClassName($flag)
@@ -451,7 +451,7 @@ public function setMapTestClassNameToCoveredClassName($flag)
451451
}
452452

453453
/**
454-
* @param boolean $flag
454+
* @param bool $flag
455455
* @throws PHP_CodeCoverage_Exception
456456
*/
457457
public function setAddUncoveredFilesFromWhitelist($flag)
@@ -467,7 +467,7 @@ public function setAddUncoveredFilesFromWhitelist($flag)
467467
}
468468

469469
/**
470-
* @param boolean $flag
470+
* @param bool $flag
471471
* @throws PHP_CodeCoverage_Exception
472472
*/
473473
public function setProcessUncoveredFilesFromWhitelist($flag)
@@ -693,7 +693,7 @@ private function getLinesToBeIgnored($filename)
693693

694694
if (!$ignore) {
695695
$start = $token->getLine();
696-
$end = $start + substr_count($token, "\n");
696+
$end = $start + substr_count($token, "\n");
697697

698698
// Do not ignore the first line when there is a token
699699
// before the comment

‎src/CodeCoverage/Driver/Xdebug.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ private function cleanup(array $data)
6666
unset($data[$file][0]);
6767
}
6868

69-
if ($file != "xdebug://debug-eval" && file_exists($file)) {
69+
if ($file != 'xdebug://debug-eval' && file_exists($file)) {
7070
$numLines = $this->getNumberOfLinesInFile($file);
7171

7272
foreach (array_keys($data[$file]) as $line) {
@@ -82,7 +82,7 @@ private function cleanup(array $data)
8282

8383
/**
8484
* @param string $file
85-
* @return integer
85+
* @return int
8686
* @since Method available since Release 2.0.0
8787
*/
8888
private function getNumberOfLinesInFile($file)

‎src/CodeCoverage/Filter.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ public function isFile($filename)
197197
* When the whitelist is not empty, whitelisting is used.
198198
*
199199
* @param string $filename
200-
* @return boolean
200+
* @return bool
201201
* @throws PHP_CodeCoverage_Exception
202202
*/
203203
public function isFiltered($filename)
@@ -238,7 +238,7 @@ public function getWhitelist()
238238
/**
239239
* Returns whether this filter has a whitelist.
240240
*
241-
* @return boolean
241+
* @return bool
242242
* @since Method available since Release 1.1.0
243243
*/
244244
public function hasWhitelist()

‎src/CodeCoverage/Report/Clover.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ class PHP_CodeCoverage_Report_Clover
2323
*/
2424
public function process(PHP_CodeCoverage $coverage, $target = null, $name = null)
2525
{
26-
$xmlDocument = new DOMDocument('1.0', 'UTF-8');
26+
$xmlDocument = new DOMDocument('1.0', 'UTF-8');
2727
$xmlDocument->formatOutput = true;
2828

2929
$xmlCoverage = $xmlDocument->createElement('coverage');

‎src/CodeCoverage/Report/Crap4j.php‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ public function __construct($threshold = 30)
4141
*/
4242
public function process(PHP_CodeCoverage $coverage, $target = null, $name = null)
4343
{
44-
$document = new DOMDocument('1.0', 'UTF-8');
44+
$document = new DOMDocument('1.0', 'UTF-8');
4545
$document->formatOutput = true;
4646

4747
$root = $document->createElement('crap_result');
@@ -136,9 +136,9 @@ public function process(PHP_CodeCoverage $coverage, $target = null, $name = null
136136
}
137137

138138
/**
139-
* @param float $crapValue
140-
* @param integer $cyclomaticComplexity
141-
* @param float $coveragePercent
139+
* @param float $crapValue
140+
* @param int $cyclomaticComplexity
141+
* @param float $coveragePercent
142142
* @return float
143143
*/
144144
private function getCrapLoad($crapValue, $cyclomaticComplexity, $coveragePercent)

‎src/CodeCoverage/Report/Factory.php‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
class PHP_CodeCoverage_Report_Factory
1717
{
1818
/**
19-
* @param PHP_CodeCoverage $coverage
19+
* @param PHP_CodeCoverage $coverage
2020
* @return PHP_CodeCoverage_Report_Node_Directory
2121
*/
2222
public function create(PHP_CodeCoverage $coverage)
@@ -42,7 +42,7 @@ public function create(PHP_CodeCoverage $coverage)
4242
* @param PHP_CodeCoverage_Report_Node_Directory $root
4343
* @param array $items
4444
* @param array $tests
45-
* @param boolean $cacheTokens
45+
* @param bool $cacheTokens
4646
*/
4747
private function addItems(PHP_CodeCoverage_Report_Node_Directory $root, array $items, array $tests, $cacheTokens)
4848
{
@@ -231,7 +231,7 @@ private function reducePaths(&$files)
231231
$max = count($original);
232232

233233
for ($i = 0; $i < $max; $i++) {
234-
$files[join('/', $paths[$i])] = $files[$original[$i]];
234+
$files[implode('/', $paths[$i])] = $files[$original[$i]];
235235
unset($files[$original[$i]]);
236236
}
237237

‎src/CodeCoverage/Report/HTML.php‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -26,21 +26,21 @@ class PHP_CodeCoverage_Report_HTML
2626
private $generator;
2727

2828
/**
29-
* @var integer
29+
* @var int
3030
*/
3131
private $lowUpperBound;
3232

3333
/**
34-
* @var integer
34+
* @var int
3535
*/
3636
private $highLowerBound;
3737

3838
/**
3939
* Constructor.
4040
*
41-
* @param integer $lowUpperBound
42-
* @param integer $highLowerBound
43-
* @param string $generator
41+
* @param int $lowUpperBound
42+
* @param int $highLowerBound
43+
* @param string $generator
4444
*/
4545
public function __construct($lowUpperBound = 50, $highLowerBound = 90, $generator = '')
4646
{

‎src/CodeCoverage/Report/HTML/Renderer.php‎

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,12 @@ abstract class PHP_CodeCoverage_Report_HTML_Renderer
3333
protected $date;
3434

3535
/**
36-
* @var integer
36+
* @var int
3737
*/
3838
protected $lowUpperBound;
3939

4040
/**
41-
* @var integer
41+
* @var int
4242
*/
4343
protected $highLowerBound;
4444

@@ -50,11 +50,11 @@ abstract class PHP_CodeCoverage_Report_HTML_Renderer
5050
/**
5151
* Constructor.
5252
*
53-
* @param string $templatePath
54-
* @param string $generator
55-
* @param string $date
56-
* @param integer $lowUpperBound
57-
* @param integer $highLowerBound
53+
* @param string $templatePath
54+
* @param string $generator
55+
* @param string $date
56+
* @param int $lowUpperBound
57+
* @param int $highLowerBound
5858
*/
5959
public function __construct($templatePath, $generator, $date, $lowUpperBound, $highLowerBound)
6060
{
@@ -123,21 +123,21 @@ protected function renderItemTemplate(Text_Template $template, array $data)
123123

124124
$template->setVar(
125125
array(
126-
'icon' => isset($data['icon']) ? $data['icon'] : '',
127-
'crap' => isset($data['crap']) ? $data['crap'] : '',
128-
'name' => $data['name'],
129-
'lines_bar' => $linesBar,
126+
'icon' => isset($data['icon']) ? $data['icon'] : '',
127+
'crap' => isset($data['crap']) ? $data['crap'] : '',
128+
'name' => $data['name'],
129+
'lines_bar' => $linesBar,
130130
'lines_executed_percent' => $data['linesExecutedPercentAsString'],
131-
'lines_level' => $linesLevel,
132-
'lines_number' => $linesNumber,
133-
'methods_bar' => $methodsBar,
131+
'lines_level' => $linesLevel,
132+
'lines_number' => $linesNumber,
133+
'methods_bar' => $methodsBar,
134134
'methods_tested_percent' => $data['testedMethodsPercentAsString'],
135-
'methods_level' => $methodsLevel,
136-
'methods_number' => $methodsNumber,
137-
'classes_bar' => $classesBar,
135+
'methods_level' => $methodsLevel,
136+
'methods_number' => $methodsNumber,
137+
'classes_bar' => $classesBar,
138138
'classes_tested_percent' => isset($data['testedClassesPercentAsString']) ? $data['testedClassesPercentAsString'] : '',
139-
'classes_level' => $classesLevel,
140-
'classes_number' => $classesNumber
139+
'classes_level' => $classesLevel,
140+
'classes_number' => $classesNumber
141141
)
142142
);
143143

@@ -245,13 +245,13 @@ protected function getCoverageBar($percent)
245245
'}}'
246246
);
247247

248-
$template->setVar(array('level' => $level, 'percent' => sprintf("%.2F", $percent)));
248+
$template->setVar(array('level' => $level, 'percent' => sprintf('%.2F', $percent)));
249249

250250
return $template->render();
251251
}
252252

253253
/**
254-
* @param integer $percent
254+
* @param int $percent
255255
* @return string
256256
*/
257257
protected function getColorLevel($percent)

‎src/CodeCoverage/Report/HTML/Renderer/Dashboard.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ protected function complexity(array $classes, $baseLink)
9292
}
9393

9494
return array(
95-
'class' => json_encode($result['class']),
95+
'class' => json_encode($result['class']),
9696
'method' => json_encode($result['method'])
9797
);
9898
}
@@ -161,7 +161,7 @@ protected function coverageDistribution(array $classes)
161161
}
162162

163163
return array(
164-
'class' => json_encode(array_values($result['class'])),
164+
'class' => json_encode(array_values($result['class'])),
165165
'method' => json_encode(array_values($result['method']))
166166
);
167167
}

‎src/CodeCoverage/Report/HTML/Renderer/Directory.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ public function render(PHP_CodeCoverage_Report_Node_Directory $node, $file)
4747

4848
/**
4949
* @param PHP_CodeCoverage_Report_Node $item
50-
* @param boolean $total
50+
* @param bool $total
5151
* @return string
5252
*/
5353
protected function renderItem(PHP_CodeCoverage_Report_Node $item, $total = false)

0 commit comments

Comments
 (0)