Skip to content

Commit 97856cf

Browse files
Remove self dependencies from graph
1 parent b317a91 commit 97856cf

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

app/Infrastructure/Graph/Adapters/Cytoscape/CytoscapeNetworkBuilder.php

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
use App\Presenter\Analyze\Shared\Network\Network;
66
use App\Presenter\Analyze\Shared\Network\NetworkBuilder;
77
use App\Infrastructure\Graph\Adapters\Cytoscape\CytoscapeNetwork;
8+
use App\Presenter\Analyze\Shared\Network\NetworkAttribute;
89

910
class CytoscapeNetworkBuilder implements NetworkBuilder
1011
{
@@ -33,6 +34,10 @@ private function mapEdges(array $metrics): void
3334

3435
foreach ($item->dependencies() as $dependency) {
3536

37+
if ($this->isSelfDependency($dependency, $item)) {
38+
continue;
39+
}
40+
3641
if ($this->network->missingNode($dependency)) {
3742
$this->network->addNode($dependency);
3843
}
@@ -41,4 +46,12 @@ private function mapEdges(array $metrics): void
4146
}
4247
}
4348
}
49+
50+
/**
51+
* We remove self dependency from graph for readability reasons
52+
*/
53+
private function isSelfDependency(string $dependency, NetworkAttribute $item): bool
54+
{
55+
return $dependency === $item->name();
56+
}
4457
}

0 commit comments

Comments
 (0)