From 5ee3a54cfd21c35be073ee7ec4274e8b448a0d99 Mon Sep 17 00:00:00 2001
From: Julio Montoya <gugli100@gmail.com>
Date: Wed, 24 Jul 2019 08:02:12 +0200
Subject: [PATCH] Fix function name, add comments see BT#2957

---
 main/webservices/user_import/import.lib.php | 23 ++++++++++++++-------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/main/webservices/user_import/import.lib.php b/main/webservices/user_import/import.lib.php
index 5a7053df42..066bd5b78d 100755
--- a/main/webservices/user_import/import.lib.php
+++ b/main/webservices/user_import/import.lib.php
@@ -1,4 +1,6 @@
 <?php
+/* For licensing terms, see /license.txt*/
+
 /**
  * Validates imported data.
  */
@@ -87,6 +89,8 @@ function complete_missing_data($user)
 
 /**
  * Save the imported data.
+ *
+ * @param array
  */
 function save_data($users)
 {
@@ -107,13 +111,16 @@ function save_data($users)
                 '',
                 $user['AuthSource']
             );
-            foreach ($user['Courses'] as $index => $course) {
-                if (CourseManager:: course_exists($course)) {
-                    CourseManager:: subscribe_user(
-                        $user_id,
-                        $course,
-                        $user['Status']
-                    );
+
+            if (!empty($user['Courses'])) {
+                foreach ($user['Courses'] as $course) {
+                    if (CourseManager::course_exists($course)) {
+                        CourseManager::subscribeUser(
+                            $user_id,
+                            $course,
+                            $user['Status']
+                        );
+                    }
                 }
             }
 
@@ -151,7 +158,7 @@ function save_data($users)
  */
 function parse_csv_data($file)
 {
-    $users = Import :: csvToArray($file);
+    $users = Import::csvToArray($file);
     foreach ($users as $index => $user) {
         if (isset($user['Courses'])) {
             $user['Courses'] = explode('|', trim($user['Courses']));