diff --git a/rest-api/src/main/java/lcsb/mapviewer/api/files/FileController.java b/rest-api/src/main/java/lcsb/mapviewer/api/files/FileController.java index 4b4a99b8a0bc4a668719139c94052cfab7bdefa3..5064344421b1f7996b2e52f6cf2811e9ae01bf73 100644 --- a/rest-api/src/main/java/lcsb/mapviewer/api/files/FileController.java +++ b/rest-api/src/main/java/lcsb/mapviewer/api/files/FileController.java @@ -42,7 +42,7 @@ public class FileController extends BaseController { @PreAuthorize("@fileService.getOwnerByFileId(#id)?.login == authentication.name") @PostMapping(value = "/{id}:uploadContent") - public Map<String, Object> uploadContent(@PathVariable(value = "id") String id, @RequestBody byte[] data) + public Map<String, Object> uploadContent(@PathVariable(value = "id") Integer id, @RequestBody byte[] data) throws ObjectNotFoundException { return fileRest.uploadContent(id, data); } diff --git a/rest-api/src/main/java/lcsb/mapviewer/api/files/FileRestImpl.java b/rest-api/src/main/java/lcsb/mapviewer/api/files/FileRestImpl.java index 636cf3c0052e0deb70260984a38a68d778c9292f..fe478a55e4905d425a9a387998047657d7d2cf58 100644 --- a/rest-api/src/main/java/lcsb/mapviewer/api/files/FileRestImpl.java +++ b/rest-api/src/main/java/lcsb/mapviewer/api/files/FileRestImpl.java @@ -60,7 +60,7 @@ public class FileRestImpl extends BaseRestImpl { return result; } - public Map<String, Object> uploadContent(String id, byte[] data) throws ObjectNotFoundException { + public Map<String, Object> uploadContent(Integer id, byte[] data) throws ObjectNotFoundException { int fileId = Integer.valueOf(id); UploadedFileEntry fileEntry = uploadedFileEntryDao.getById(fileId); if (fileEntry == null) { diff --git a/service/src/main/java/lcsb/mapviewer/services/impl/FileService.java b/service/src/main/java/lcsb/mapviewer/services/impl/FileService.java index 8690ac68f5c7d097055ccd1974c3d05ae795a043..0b94799a4caace195e693bbaebc813a767a6f7d8 100644 --- a/service/src/main/java/lcsb/mapviewer/services/impl/FileService.java +++ b/service/src/main/java/lcsb/mapviewer/services/impl/FileService.java @@ -21,20 +21,22 @@ public class FileService implements IFileService { } @Override - public UploadedFileEntry getById(int id) { + public UploadedFileEntry getById(Integer id) { return uploadedFileEntryDao.getById(id); } @Override - public User getOwnerByFileId(int id) { - UploadedFileEntry entry = uploadedFileEntryDao.getById(id); - if (entry != null && entry.getOwner() != null) { - //it's lazy initialized - entry.getOwner().getLogin(); - return uploadedFileEntryDao.getById(id).getOwner(); - } else { - return null; + public User getOwnerByFileId(Integer id) { + if (id != null) { + UploadedFileEntry entry = uploadedFileEntryDao.getById(id); + if (entry != null && entry.getOwner() != null) { + // it's lazy initialized + entry.getOwner().getLogin(); + return uploadedFileEntryDao.getById(id).getOwner(); + } } + return null; + } } diff --git a/service/src/main/java/lcsb/mapviewer/services/interfaces/IFileService.java b/service/src/main/java/lcsb/mapviewer/services/interfaces/IFileService.java index 4ea62980773b70380a0faea9547a976b76539469..5598fecf5f4a68941dfef47bcf243d87923f22f4 100644 --- a/service/src/main/java/lcsb/mapviewer/services/interfaces/IFileService.java +++ b/service/src/main/java/lcsb/mapviewer/services/interfaces/IFileService.java @@ -5,8 +5,8 @@ import lcsb.mapviewer.model.user.User; public interface IFileService { - UploadedFileEntry getById(int id); + UploadedFileEntry getById(Integer id); - User getOwnerByFileId(int id); + User getOwnerByFileId(Integer id); }