Plugin: Positioning: Fix condition if tests selected in base course but using them in sessions - refs BT#19725

plugin/positioning/src/Positioning.php

@@ -70,6 +70,7 @@ class Positioning extends Plugin
     public function getPositionData($exerciseId, $courseId, $sessionId)
     {
         $table = $this->table;
+        $exerciseId = (int) $exerciseId;
         $courseId = (int) $courseId;
         $sessionId = (int) $sessionId;
 
@@ -83,6 +84,18 @@ class Positioning extends Plugin
 
         if (Database::num_rows($result) > 0) {
             return Database::fetch_array($result, 'ASSOC');
+        } elseif (0 !== $sessionId) {
+            // if no exercise was set in the session, also look for it in the base course
+            $sql = "SELECT * FROM $table
+                WHERE
+                    exercise_id = $exerciseId AND
+                    c_id = $courseId AND
+                    session_id = 0
+                    ";
+            $result = Database::query($sql);
+            if (Database::num_rows($result) > 0) {
+                return Database::fetch_array($result, 'ASSOC');
+            }
         }
 
         return false;
@@ -212,23 +225,35 @@ class Positioning extends Plugin
                     c_id = $courseId AND
                     session_id = $sessionId
                     ";
+        $sqlEnd = '';
 
         if ($isInitial) {
-            $sql .= ' AND is_initial = 1 ';
+            $sqlEnd .= ' AND is_initial = 1 ';
         } else {
-            $sql .= ' AND is_initial = 0 ';
+            $sqlEnd .= ' AND is_initial = 0 ';
         }
 
         if ($isFinal) {
-            $sql .= ' AND is_final = 1 ';
+            $sqlEnd .= ' AND is_final = 1 ';
         } else {
-            $sql .= ' AND is_final = 0 ';
+            $sqlEnd .= ' AND is_final = 0 ';
         }
-
-        $result = Database::query($sql);
+        $result = Database::query($sql.$sqlEnd);
 
         if (Database::num_rows($result) > 0) {
+
             return Database::fetch_array($result, 'ASSOC');
+        } elseif (0 !== $sessionId) {
+            $sql = "SELECT * FROM $table
+                WHERE
+                    c_id = $courseId AND
+                    session_id = 0
+                    ";
+            $result = Database::query($sql.$sqlEnd);
+            if (Database::num_rows($result) > 0) {
+
+                return Database::fetch_array($result, 'ASSOC');
+            }
         }
 
         return false;

plugin/positioning/start_student.php

@@ -80,21 +80,23 @@ if ($finalData) {
 }
 // Set the initial results as second series to make sure it appears on top
 $lpUrlAndProgress = $studentAverage.'%';
+$radars = '';
+
 if (!empty($initialExercise)) {
     $exercisesToRadar[] = $initialExercise;
     $exercisesToRadarLabel[] = $plugin->get_lang('InitialTest');
     if (!empty($initialResults)) {
         $lpUrlAndProgress = '<a href="'.api_get_path(WEB_CODE_PATH).'lp/lp_controller.php?'.api_get_cidreq().'">'.$studentAverage.'%</a>';
     }
+    $radars = $initialExercise->getRadarsFromUsers(
+        [$currentUserId],
+        $exercisesToRadar,
+        $exercisesToRadarLabel,
+        $courseId,
+        $sessionId
+    );
 }
 
-$radars = $initialExercise->getRadarsFromUsers(
-    [$currentUserId],
-    $exercisesToRadar,
-    $exercisesToRadarLabel,
-    $courseId,
-    $sessionId
-);
 $nameTools = $plugin->get_lang('Positioning');
 
 $template = new Template($nameTools);