Refactor delete methods in ParcelContentManager, ItemStorageManager, and DeliveryManager to return boolean indicating success

Author: Jakubk15

src/main/java/com/eternalcode/parcellockers/content/ParcelContentManager.java

@@ -51,10 +51,10 @@ public ParcelContent create(UUID parcel, List<ItemStack> items) {
         return content;
     }
 
-    public CompletableFuture<Void> delete(UUID parcel) {
+    public CompletableFuture<Boolean> delete(UUID parcel) {
         return this.contentRepository.delete(parcel).thenApply(i -> {
             this.cache.invalidate(parcel);
-            return null;
+            return i > 0;
         });
     }
 

src/main/java/com/eternalcode/parcellockers/delivery/DeliveryManager.java

@@ -39,10 +39,10 @@ public Delivery create(UUID parcel, Instant deliveryTimestamp) {
         return delivery;
     }
 
-    public CompletableFuture<Void> delete(UUID parcel) {
+    public CompletableFuture<Boolean> delete(UUID parcel) {
         return this.deliveryRepository.delete(parcel).thenApply(i -> {
             this.deliveryCache.invalidate(parcel);
-            return null;
+            return i > 0;
         });
     }
 

src/main/java/com/eternalcode/parcellockers/itemstorage/ItemStorageManager.java

@@ -61,9 +61,9 @@ private void cacheAll() {
                 itemStorage))));
     }
 
-    public CompletableFuture<Void> delete(UUID parcel) {
+    public CompletableFuture<Boolean> delete(UUID parcel) {
         this.cache.invalidate(parcel);
-        return this.itemStorageRepository.delete(parcel).thenApply(i -> null);
+        return this.itemStorageRepository.delete(parcel).thenApply(i -> i > 0);
     }
 
     public CompletableFuture<Void> deleteAll(CommandSender sender, NoticeService noticeService) {