Set proper lti context id (needs latest tsugi as of 03/2024)

thexerteproject · Apr 1, 2024 · a4f4733 · a4f4733
1 parent 78b6741
commit a4f4733
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/lti_launch.php b/lti_launch.php
@@ -71,6 +71,7 @@
         $nrps = $LAUNCH->context->loadNamesAndRoles(false, $msg);
         //DONE: get all emails of current users add to array
         // https://gitlab.tor.nl/xerte-dashboard/dashboard/-/blob/master/index.php line 80
+	_debug('LTI 1.3 names and roles result: ' . print_r($nrps, true));
         $xerte_toolkits_site->lti_users = array();
         foreach ($nrps->members as $i => $member){
             if ($member->status == 'Active' && in_array('Learner', $member->roles))
@@ -135,18 +136,33 @@
     {
         $xerte_toolkits_site->module = $module;
     }
+    if (isset($LAUNCH->context->context_id))
+    {
+        $lticontextid = $LAUNCH->context->context_id;
+	_debug('Context id set from context->context_id');
+    }
+    else
+    {
     $lticontextid = $LAUNCH->ltiParameter('context_id');
+    if ($lticontextid !== false)
+    {
+	_debug('Context id set from parameter');
+    }
     if ($lticontextid === false)
     {
         $lticontextid = $LAUNCH->ltiCustomGet('context_id');
+	_debug('Context id set from custom parameter');
     }
     if ($lticontextid===false && isset($_REQUEST['context_id']))
     {
         $lticontextid = $_REQUEST['context_id'];
+	_debug('Context id set from request');
     }
     if ($lticontextid === false)
     {
         $lticontextid = $LAUNCH->context->id;
+	_debug('Context id set from context->id');
+    }
     }
     if ($lticontextid !== false)
     {