From df4686bc966bd16a18352a07e8a752c0d28a9b2f Mon Sep 17 00:00:00 2001 From: Ivan Nieto Date: Tue, 2 Feb 2021 14:23:36 -0600 Subject: [PATCH] Added a clause that lets the function that updates the department to fail gracefully if no dept. name is provided in the CSV passed to the Importer. (#9078) --- app/Importer/UserImporter.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/app/Importer/UserImporter.php b/app/Importer/UserImporter.php index 4fcf221cba..be476aef97 100644 --- a/app/Importer/UserImporter.php +++ b/app/Importer/UserImporter.php @@ -125,7 +125,10 @@ class UserImporter extends ItemImporter if ($department) { $this->log('A matching department ' . $department_name . ' already exists'); return $department->id; + } else { + return null; } + $department = new department(); $department->name = $department_name; $department->user_id = $this->user_id; @@ -134,7 +137,8 @@ class UserImporter extends ItemImporter $this->log('department ' . $department_name . ' was created'); return $department->id; } - $this->logError($department, 'Company'); + + $this->logError($department, 'Department'); return null; }