Applied code styles

martinyde · martinyde · commit 167d9d08831e · 2026-04-22T09:10:24.000+02:00
diff --git a/src/Entity/Advisory.php b/src/Entity/Advisory.php
@@ -195,18 +195,16 @@ public function removePackageVersion(PackageVersion $packageVersion): self
      */
     public function getSites(): Collection
     {
-        $sites = new ArrayCollection();
+        $sites = [];
         foreach ($this->packageVersions as $packageVersion) {
             foreach ($packageVersion->getInstallations() as $installation) {
                 foreach ($installation->getSites() as $site) {
-                    if (!$sites->contains($site)) {
-                        $sites->add($site);
-                    }
+                    $sites[$site->getId()->toRfc4122()] = $site;
                 }
             }
         }
 
-        return $sites;
+        return new ArrayCollection(array_values($sites));
     }
 
     public function getPackage(): ?Package

Original file line number	Diff line number	Diff line change
`@@ -195,18 +195,16 @@ public function removePackageVersion(PackageVersion $packageVersion): self`
`195`	`195`	`*/`
`196`	`196`	`public function getSites(): Collection`
`197`	`197`	`{`
`198`		`- $sites = new ArrayCollection();`
	`198`	`+ $sites = [];`
`199`	`199`	`foreach ($this->packageVersions as $packageVersion) {`
`200`	`200`	`foreach ($packageVersion->getInstallations() as $installation) {`
`201`	`201`	`foreach ($installation->getSites() as $site) {`
`202`		`- if (!$sites->contains($site)) {`
`203`		`- $sites->add($site);`
`204`		`- }`
	`202`	`+ $sites[$site->getId()->toRfc4122()] = $site;`
`205`	`203`	`}`
`206`	`204`	`}`
`207`	`205`	`}`
`208`	`206`
`209`		`- return $sites;`
	`207`	`+ return new ArrayCollection(array_values($sites));`
`210`	`208`	`}`
`211`	`209`
`212`	`210`	`public function getPackage(): ?Package`