Index: lams_admin/src/java/org/lamsfoundation/lams/admin/web/ImportExcelSaveAction.java =================================================================== diff -u -rf8ae5239ffcb6ac62937458e9750b3780528dc9f -r221823dcaabe5ed0b06b51b873aafdfb0ea7c000 --- lams_admin/src/java/org/lamsfoundation/lams/admin/web/ImportExcelSaveAction.java (.../ImportExcelSaveAction.java) (revision f8ae5239ffcb6ac62937458e9750b3780528dc9f) +++ lams_admin/src/java/org/lamsfoundation/lams/admin/web/ImportExcelSaveAction.java (.../ImportExcelSaveAction.java) (revision 221823dcaabe5ed0b06b51b873aafdfb0ea7c000) @@ -49,6 +49,7 @@ * * @struts:action-forward name="importresult" path=".importresult" * @struts:action-forward name="sysadmin" path="/sysadminstart.do" + * @struts:action-forward name="import" path="/importexcel.do" */ public class ImportExcelSaveAction extends Action { @@ -67,6 +68,11 @@ ImportExcelForm importExcelForm = (ImportExcelForm)form; FormFile file = importExcelForm.getFile(); + // validation + if (file==null || file.getFileSize()<=0) { + return mapping.findForward("import"); + } + List results = importService.parseSpreadsheet(file); String successMessageKey = (importService.isUserSpreadsheet(file) ? "msg.users.created" : "msg.users.added"); Index: lams_admin/src/java/org/lamsfoundation/lams/admin/web/action/ImportV1SaveAction.java =================================================================== diff -u -rf8ae5239ffcb6ac62937458e9750b3780528dc9f -r221823dcaabe5ed0b06b51b873aafdfb0ea7c000 --- lams_admin/src/java/org/lamsfoundation/lams/admin/web/action/ImportV1SaveAction.java (.../ImportV1SaveAction.java) (revision f8ae5239ffcb6ac62937458e9750b3780528dc9f) +++ lams_admin/src/java/org/lamsfoundation/lams/admin/web/action/ImportV1SaveAction.java (.../ImportV1SaveAction.java) (revision 221823dcaabe5ed0b06b51b873aafdfb0ea7c000) @@ -51,6 +51,7 @@ * * @struts:action-forward name="importv1contents" path="/importv1contents.do" * @struts:action-forward name="sysadmin" path="/sysadminstart.do" + * @struts:action-forward name="importv1" path="/importv1.do" */ public class ImportV1SaveAction extends Action { @@ -72,6 +73,11 @@ FormFile file = importV1Form.getFile(); boolean includeIntegrated = importV1Form.getIntegrated(); + // validation + if (file==null || file.getFileSize()<=0) { + return mapping.findForward("importv1"); + } + List results = importService.parseV1UsersFile(file, includeIntegrated); List users = results.get(0); List orgs = results.get(1);