From 2690dccd5e44bdf1bfd258a0d424f3dc5d23cad2 Mon Sep 17 00:00:00 2001
From: Stefan Topfstedt <Stefan.Topfstedt@ucsf.edu>
Date: Tue, 21 Jan 2025 17:10:02 -0800
Subject: [PATCH] fixes scoping issue with closures by explicitly passing
 variable into it.

---
 edit_form.php | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/edit_form.php b/edit_form.php
index 31e8b88..9c2f955 100644
--- a/edit_form.php
+++ b/edit_form.php
@@ -143,7 +143,8 @@ protected function definition(): void {
                             &$processparents,
                             &$learnergroupoptions,
                             &$grouptitle,
-                            &$instance
+                            &$instance,
+                            $ilios
                         ) {
                             $parentgroup = $ilios->get_learner_group($child->parent);
                             $instance->learnergroupid = $parentgroup->id;
@@ -444,7 +445,7 @@ public function definition_after_data(): void {
                                                                    ' ('. count($subgroup->children) .')'.
                                                                    ' ('. count($subgroup->users) .')';
                 if (!empty($subgroup->children)) {
-                    $processchildren = function ($parent) use (&$processchildren, &$subgroupoptions) {
+                    $processchildren = function ($parent) use (&$processchildren, &$subgroupoptions, $ilios) {
                         $subgrps = $ilios->get_learner_groups([ 'parent' => $parent->id], [ 'title' => "ASC"]);
                         foreach ($subgrps as $subgrp) {
                             $subgroupoptions["$subgrp->id:$parent->title / $subgrp->title"] = $parent->title.' / '.$subgrp->title.