Merge pull request #1691 from griidc/epic/PELAGOS-5423-new-landing-pages-for-funding-cycle-and-person-page

Remove duplicates

Author: mickel1138

src/Entity/FundingCycle.php

@@ -277,7 +277,10 @@ public function getPeople(): Collection
 
         foreach ($this->getResearchGroups() as $researchGroup) {
             foreach ($researchGroup->getPersonResearchGroups() as $personResearchGroup) {
-                $people->add($personResearchGroup->getPerson());
+                $person = $personResearchGroup->getPerson();
+                if (!$people->contains($person)) {
+                    $people->add($person);
+                }
             }
         }
 

src/Entity/ResearchGroup.php

@@ -622,7 +622,10 @@ public function getPeople(): Collection
     {
         $people = new ArrayCollection();
         foreach ($this->getPersonResearchGroups() as $personResearchGroup) {
-            $people->add($personResearchGroup->getPerson());
+            $person = $personResearchGroup->getPerson();
+            if (!$people->contains($person)) {
+                $people->add($person);
+            }
         }
 
         return $people;