Browse Source

Avoid saving empty data

jmontoyaa 8 years ago
parent
commit
02fdee3abb
2 changed files with 18 additions and 12 deletions
  1. 6 6
      main/cron/import_csv.php
  2. 12 6
      main/inc/lib/sessionmanager.lib.php

+ 6 - 6
main/cron/import_csv.php

@@ -2070,12 +2070,12 @@ class ImportCsv
                         ";
 
                 $result = Database::query($sql);
-                $userCourseData = Database::fetch_array($result, 'ASSOC');
-                $teacherBackup[$userId][$courseInfo['code']] = $userCourseData;
-                if (isset($userCourseData['user_course_cat'])) {
-                     $this->logger->addError(
-                         "Saving user course category: user #$userId course: ".$courseInfo['code']." course cat: ".$userCourseData['user_course_cat']
-                    );
+                $rows = Database::num_rows($result);
+                if ($rows > 0) {
+                    $userCourseData = Database::fetch_array($result, 'ASSOC');
+                    if (!empty($userCourseData) && !empty($userCourseData['user_course_cat'])) {
+                        $teacherBackup[$userId][$courseInfo['code']] = $userCourseData;
+                    }
                 }
 
                 $sql = "SELECT * FROM ".Database::get_course_table(TABLE_GROUP_USER)."

+ 12 - 6
main/inc/lib/sessionmanager.lib.php

@@ -4794,9 +4794,12 @@ class SessionManager
                                                     ";
 
                                             $result = Database::query($sql);
-                                            $userCourseData = Database::fetch_array($result, 'ASSOC');
-                                            if (!empty($userCourseData)) {
-                                                $teacherBackupList[$teacher['user_id']][$course_code] = $userCourseData;
+                                            $rows = Database::num_rows($result);
+                                            if ($rows > 0) {
+                                                $userCourseData = Database::fetch_array($result, 'ASSOC');
+                                                if (!empty($userCourseData) && !empty($userCourseData['user_course_cat'])) {
+                                                    $teacherBackupList[$teacher['user_id']][$course_code] = $userCourseData;
+                                                }
                                             }
 
                                             $sql = "SELECT * FROM ".Database::get_course_table(TABLE_GROUP_USER)."
@@ -4923,9 +4926,12 @@ class SessionManager
                                                         ";
 
                                                 $result = Database::query($sql);
-                                                $userCourseData = Database::fetch_array($result, 'ASSOC');
-                                                if (!empty($userCourseData)) {
-                                                    $teacherBackupList[$teacher['user_id']][$course_code] = $userCourseData;
+                                                $rows = Database::num_rows($result);
+                                                if ($rows > 0) {
+                                                    $userCourseData = Database::fetch_array($result, 'ASSOC');
+                                                    if (!empty($userCourseData) && !empty($userCourseData['user_course_cat'])) {
+                                                        $teacherBackupList[$teacher['user_id']][$course_code] = $userCourseData;
+                                                    }
                                                 }
 
                                                 $sql = "SELECT * FROM ".Database::get_course_table(TABLE_GROUP_USER)."