Skip to content

Commit

Permalink
fixes scoping issue with closures by explicitly passing variable into…
Browse files Browse the repository at this point in the history
… it.
  • Loading branch information
stopfstedt committed Jan 22, 2025
1 parent 1976184 commit 2690dcc
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions edit_form.php
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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.
Expand Down

0 comments on commit 2690dcc

Please sign in to comment.