Fixing query that resets the LP

14 years ago · ecbdc0376c
parent 8a4511a400
commit ecbdc0376c
1 changed files with 9 additions and 6 deletions
--- a/main/inc/lib/events.lib.inc.php
+++ b/main/inc/lib/events.lib.inc.php
@ -833,15 +833,18 @@ function delete_student_lp_events($user_id, $lp_id, $course, $session_id) {
    $session_id            = intval($session_id);
    
    //make sure we have the exact lp_view_id
-    $sqlview       = "SELECT id FROM $lp_view_table WHERE c_id = $course_id AND user_id = $user_id AND lp_id = $lp_id AND session_id = $session_id ";                    
+    $sqlview       = "SELECT id FROM $lp_view_table WHERE c_id = $course_id AND user_id = $user_id AND lp_id = $lp_id AND session_id = $session_id ";                        
    $resultview    = Database::query($sqlview);
-    $view          = Database::fetch_array($resultview, 'ASSOC');
-    $lp_view_id    = $view['id'];                  
    
-    $sql_delete = "DELETE FROM $lp_item_view_table  WHERE c_id = $course_id AND lp_view_id = $view_id ";
-    $result = Database::query($sql_delete);
+    if (Database::num_rows($sqlview)) {    
+        $view          = Database::fetch_array($resultview, 'ASSOC');
+        $lp_view_id    = $view['id'];        
+
+        $sql_delete = "DELETE FROM $lp_item_view_table WHERE c_id = $course_id AND lp_view_id = $lp_view_id ";        
+        $result = Database::query($sql_delete);
+    }
        
-    $sql_delete = "DELETE FROM $lp_view_table  WHERE c_id = $course_id AND user_id = $user_id AND lp_id= $lp_id AND session_id= $session_id ";
+    $sql_delete = "DELETE FROM $lp_view_table WHERE c_id = $course_id AND user_id = $user_id AND lp_id= $lp_id AND session_id= $session_id ";
    $result = Database::query($sql_delete);
    
    $select_all_attempts = "SELECT exe_id FROM $track_e_exercises WHERE exe_user_id = $user_id AND session_id= $session_id  AND exe_cours_id = '{$course['code']}' AND orig_lp_id = $lp_id";