Skip to content

Commit 4748ab9

Browse files
authored
Merge pull request #3107 from TechnologyEnhancedLearning/Develop/Fixes/TD-5329-FixGetFrameworkAdminRolesBug
TD-5329 fix bug with showing error page after upload
2 parents bad2fe0 + 936e747 commit 4748ab9

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

DigitalLearningSolutions.Web/Controllers/FrameworksController/ImportCompetencies.cs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,10 @@ public IActionResult StartImport(ImportCompetenciesFormData model, int framework
5353
if (!workbook.Worksheets.Contains(ImportCompetenciesFromFileService.CompetenciesSheetName))
5454
{
5555
ModelState.AddModelError("ImportFile", CommonValidationErrorMessages.InvalidCompetenciesUploadExcelFile);
56-
return View("Developer/Import/Index", model);
56+
var adminId = GetAdminId();
57+
var framework = frameworkService.GetFrameworkDetailByFrameworkId(frameworkId, adminId);
58+
var viewModel = new ImportCompetenciesViewModel(framework, isNotBlank);
59+
return View("Developer/Import/Index", viewModel);
5760
}
5861
var competenciesFileName = FileHelper.UploadFile(webHostEnvironment, model.ImportFile);
5962
setupBulkUploadData(frameworkId, adminUserID, competenciesFileName, tabname, isNotBlank);
@@ -63,7 +66,10 @@ public IActionResult StartImport(ImportCompetenciesFormData model, int framework
6366
catch (DocumentFormat.OpenXml.Packaging.OpenXmlPackageException)
6467
{
6568
ModelState.AddModelError("DelegatesFile", "The Excel file has at least one cell containing an invalid hyperlink or email address.");
66-
return View("Index", model);
69+
var adminId = GetAdminId();
70+
var framework = frameworkService.GetFrameworkDetailByFrameworkId(frameworkId, adminId);
71+
var viewModel = new ImportCompetenciesViewModel(framework, isNotBlank);
72+
return View("Developer/Import/Index", viewModel);
6773
}
6874
catch (InvalidHeadersException)
6975
{

0 commit comments

Comments
 (0)