Skip to content

Commit 8b5a218

Browse files
committed
fix divide by zero
1 parent ccec6a9 commit 8b5a218

File tree

2 files changed

+11
-6
lines changed

2 files changed

+11
-6
lines changed

src/Riper/Dissect/Services/CoverageStatsService.php

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -190,15 +190,18 @@ protected function getClassesStats(\DOMDocument $dom)
190190
$classData['metrics']['coveredStatements'] = $metric->getAttribute('coveredstatements');
191191
$classData['metrics']['elements'] = $metric->getAttribute('elements');
192192
$classData['metrics']['coveredElements'] = $metric->getAttribute('coveredelements');
193+
193194
$classData['metrics']['coveredElementsPercent']
194-
=
195-
($classData['metrics']['coveredElements'] / $classData['metrics']['elements']) * 100;
195+
= $classData['metrics']['elements'] != 0 ?
196+
($classData['metrics']['coveredElements'] / $classData['metrics']['elements']) * 100 : 100;
197+
196198
$classData['metrics']['coveredMethodsPercent']
197-
=
198-
($classData['metrics']['coveredMethods'] / $classData['metrics']['methods']) * 100;
199+
= $classData['metrics']['methods'] != 0 ?
200+
($classData['metrics']['coveredMethods'] / $classData['metrics']['methods']) * 100 : 100;
201+
199202
$classData['metrics']['coveredStatementsPercent']
200-
=
201-
($classData['metrics']['coveredStatements'] / $classData['metrics']['statements']) * 100;
203+
= $classData['metrics']['statements'] != 0 ?
204+
($classData['metrics']['coveredStatements'] / $classData['metrics']['statements']) * 100 : 100;
202205
}
203206

204207
$methods = $xpath->query(".//line[@type='method']", $fileNode);

src/Riper/Dissect/bin/dissect

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,9 @@ $loaded = false;
66

77
foreach (array(
88
__DIR__ . '/../../../../autoload.php',
9+
__DIR__ . '/../../../../vendor/autoload.php',
910
__DIR__ . '../../../../../vendor/autoload.php',
11+
__DIR__ . '../../../../../autoload.php',
1012
__DIR__ . '/vendor/autoload.php',
1113
__DIR__ . '/../../../../../../autoload.php'
1214
) as $file) {

0 commit comments

Comments
 (0)