fix: avoid NPE found in the logs

ursjoss · Nov 15, 2024 · daaf3cd · daaf3cd
1 parent 6b3143f
commit daaf3cd
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/core/core-web/src/main/java/ch/difty/scipamato/core/web/paper/entry/EditablePaperPanel.java b/core/core-web/src/main/java/ch/difty/scipamato/core/web/paper/entry/EditablePaperPanel.java
@@ -644,12 +644,14 @@ protected void onSubmit(@NotNull final AjaxRequestTarget target) {
                 super.onSubmit(target);
                 Paper p = null;
                 final List<FileUpload> fileUploads = model.getObject();
-                for (final FileUpload upload : fileUploads) {
-                    try {
-                        p = paperService.saveAttachment(convertToPaperAttachment(upload));
-                    } catch (final Exception ex) {
-                        log.error("Unexpected error when uploading file {}: {}", upload.getClientFileName(), ex.getMessage());
-                        error("Unexpected error saving file " + upload.getClientFileName() + ": " + ex.getMessage());
+                if (fileUploads != null) {
+                    for (final FileUpload upload : fileUploads) {
+                        try {
+                            p = paperService.saveAttachment(convertToPaperAttachment(upload));
+                        } catch (final Exception ex) {
+                            log.error("Unexpected error when uploading file {}: {}", upload.getClientFileName(), ex.getMessage());
+                            error("Unexpected error saving file " + upload.getClientFileName() + ": " + ex.getMessage());
+                        }
                     }
                 }
                 if (p != null) {