Learnpath: Add sanity checks in the lp-specific group subscription co…

…nfiguration page (ItemPropertyRepository) to avoid null errors - refs BT#17726
chamilo · Sep 14, 2020 · 9229284 · 9229284
1 parent 800c0de
commit 9229284
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/Chamilo/CoreBundle/Entity/Repository/ItemPropertyRepository.php b/src/Chamilo/CoreBundle/Entity/Repository/ItemPropertyRepository.php
@@ -100,7 +100,10 @@ public function subscribeGroupsToItem(
         if ($groupsSubscribedToItem) {
             /** @var CItemProperty $itemProperty */
             foreach ($groupsSubscribedToItem as $itemProperty) {
-                $alreadyAdded[] = $itemProperty->getGroup()->getId();
+                $getGroup = $itemProperty->getGroup();
+                if (!empty($getGroup)) {
+                    $alreadyAdded[] = $getGroup->getId();
+                }
             }
         }