refactor: improve log messages by prefix them with image id.

Author: php-coder

src/main/java/ru/mystamps/web/feature/image/FilesystemImagePersistenceStrategy.java

@@ -83,7 +83,7 @@ public void save(MultipartFile file, ImageInfoDto image) {
 			Path dest = generateFilePath(storageDir, image);
 			writeToFile(file, dest);
 
-			log.info("Image data has been written into file {}", dest);
+			log.info("Image #{}: data has been written into file {}", image.getId(), dest);
 
 		} catch (IOException ex) {
 			throw new ImagePersistenceException(ex);
@@ -96,7 +96,7 @@ public void savePreview(byte[] data, ImageInfoDto image) {
 			Path dest = generateFilePath(previewDir, image);
 			writeToFile(data, dest);
 
-			log.info("Image preview data has been written into file {}", dest);
+			log.info("Image #{}: preview has been written into file {}", image.getId(), dest);
 
 		} catch (IOException ex) {
 			throw new ImagePersistenceException(ex);
@@ -160,8 +160,14 @@ public void removeIfPossible(ImageInfoDto image) {
 		Path dest = generateFilePath(storageDir, image);
 		try {
 			Files.deleteIfExists(dest);
+			log.debug("Image #{}: data ({}) has been removed", image.getId(), dest);
+			
 		} catch (Exception ex) { // NOPMD: AvoidCatchingGenericException
-			log.warn("Couldn't delete file {}: {}", dest, ex.getMessage());
+			log.warn(
+				"Couldn't delete file {}: {}. You have to remove it manually",
+				dest,
+				ex.getMessage()
+			);
 		}
 	}
 

src/main/java/ru/mystamps/web/feature/image/ImageServiceImpl.java

@@ -73,7 +73,7 @@ public ImageInfoDto save(MultipartFile file) {
 		}
 
 		ImageInfoDto imageInfo = new ImageInfoDto(imageId, imageType);
-		log.info("Image info has been saved to database ({})", imageInfo);
+		log.info("Image #{}: meta data has been saved ({})", imageId, imageInfo);
 
 		imagePersistenceStrategy.save(file, imageInfo);