deblan/php-censor
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fixed TechnicalDebt plugin. Issue #82.

Browse source
This commit is contained in:
Dmitry Khomutov 2017-12-07 20:34:08 +07:00
commit 2b57886db0
No known key found for this signature in database
GPG key ID: EC19426474B37AAC
1 changed files with 19 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

32
src/PHPCensor/Plugin/TechnicalDebt.php
View file

@ -133,9 +133,9 @@ class TechnicalDebt extends Plugin implements ZeroConfigPluginInterface
/**
* Gets the number and list of errors returned from the search
*
* @return array
* @return integer
*/
public function getErrorList()
protected function getErrorList()
{
$dirIterator = new \RecursiveDirectoryIterator($this->directory);
$iterator = new \RecursiveIteratorIterator($dirIterator, \RecursiveIteratorIterator::SELF_FIRST);
@ -166,34 +166,38 @@ class TechnicalDebt extends Plugin implements ZeroConfigPluginInterface
}
}
$files = array_filter(array_unique($files));
$files = array_filter(array_unique($files));
$errorCount = 0;
foreach ($files as $file) {
foreach ($this->searches as $search) {
$fileContent = file_get_contents($file);
$allLines = explode(PHP_EOL, $fileContent);
$beforeString = strstr($fileContent, $search, true);
$handle = fopen($file, "r");
$lineNumber = 1;
while (false === feof($handle)) {
$line = fgets($handle);
if (false !== $beforeString) {
$lines = explode(PHP_EOL, $beforeString);
$lineNumber = count($lines);
$content = trim($allLines[$lineNumber - 1]);
$errorCount++;
$technicalDeptLine = false;
foreach ($this->searches as $search) {
if ($technicalDeptLine = trim(strstr($line, $search))) {
break;
}
}
if ($technicalDeptLine) {
$fileName = str_replace($this->directory, '', $file);
$this->build->reportError(
$this->builder,
'technical_debt',
$content,
$technicalDeptLine,
PHPCensor\Model\BuildError::SEVERITY_LOW,
$fileName,
$lineNumber
);
}
$lineNumber++;
}
fclose ($handle);
}
return $errorCount;