If session id empty dont insert

main/cron/import_csv.php

@@ -1168,7 +1168,7 @@ class ImportCsv
                             $courseInfo['code']
                         );
                         break;
-                   case 'drh':
+                    case 'drh':
                         $userInfo = api_get_user_info($userId);
                         SessionManager::suscribe_sessions_to_hr_manager(
                             $userInfo,
@@ -1289,7 +1289,6 @@ class ImportCsv
                     $this->logger->addError('User does not exists: '.$chamiloUserName);
                     continue;
                 }
-                $status = null;
                 switch ($type) {
                     case 'student':
                         SessionManager::subscribe_users_to_session_course(
@@ -1624,7 +1623,7 @@ class ImportCsv
                     $this->logger->addError('User does not exists: '.$chamiloUserName);
                     continue;
                 }
-                $status = null;
+
                 switch ($type) {
                     case 'student':
                         SessionManager::subscribe_users_to_session_course(

main/inc/lib/course.lib.php

@@ -5164,7 +5164,11 @@ class CourseManager
                     // Remove old and add new
                     if ($deleteSessionTeacherNotInList) {
                         foreach ($teachers as $userId) {
-                            SessionManager::set_coach_to_course_session($userId, $session['id'], $courseId);
+                            SessionManager::set_coach_to_course_session(
+                                $userId,
+                                $session['id'],
+                                $courseId
+                            );
                         }
 
                         $teachersToDelete = array();
@@ -5174,14 +5178,22 @@ class CourseManager
 
                         if (!empty($teachersToDelete)) {
                             foreach ($teachersToDelete as $userId) {
-                                SessionManager::set_coach_to_course_session($userId, $session['id'], $courseId,
-                                    true);
+                                SessionManager::set_coach_to_course_session(
+                                    $userId,
+                                    $session['id'],
+                                    $courseId,
+                                    true
+                                );
                             }
                         }
                     } else {
                         // Add new teachers only
                         foreach ($teachers as $userId) {
-                            SessionManager::set_coach_to_course_session($userId, $session['id'], $courseId);
+                            SessionManager::set_coach_to_course_session(
+                                $userId,
+                                $session['id'],
+                                $courseId
+                            );
                         }
                     }
                 }

main/inc/lib/sessionmanager.lib.php

@@ -2776,6 +2776,11 @@ class SessionManager
             $session_id = api_get_session_id();
         }
 
+        if (empty($session_id)) {
+            return false;
+        }
+
+
         if (!empty($courseId)) {
             $courseId = intval($courseId);
         } else {
@@ -2828,10 +2833,11 @@ class SessionManager
                                 c_id = $courseId AND
                                 user_id = $user_id ";
                     $result = Database::query($sql);
-                    if (Database::affected_rows($result) > 0)
+                    if (Database::affected_rows($result) > 0) {
                         return true;
-                    else
+                    } else {
                         return false;
+                    }
                 }
             } else {
                 // Assign user as a coach to course