From c1499a2371ed34add45e6e486af9f4fc821cb226 Mon Sep 17 00:00:00 2001
From: 90 <90@national.shitposting.agency>
Date: Mon, 16 Jan 2023 18:42:41 +0000
Subject: [PATCH] Fix incorrect reversibility check

---
 .../item/cell/CompressionHandler.java         | 30 ++++++++++++-------
 1 file changed, 20 insertions(+), 10 deletions(-)

diff --git a/common/src/main/java/gripe/_90/megacells/item/cell/CompressionHandler.java b/common/src/main/java/gripe/_90/megacells/item/cell/CompressionHandler.java
index 734291f1..f887e327 100644
--- a/common/src/main/java/gripe/_90/megacells/item/cell/CompressionHandler.java
+++ b/common/src/main/java/gripe/_90/megacells/item/cell/CompressionHandler.java
@@ -1,6 +1,5 @@
 package gripe._90.megacells.item.cell;
 
-import java.util.Collections;
 import java.util.List;
 import java.util.stream.Stream;
 
@@ -9,7 +8,6 @@
 import it.unimi.dsi.fastutil.objects.Object2IntMap;
 import it.unimi.dsi.fastutil.objects.ObjectArrayList;
 
-import net.minecraft.server.MinecraftServer;
 import net.minecraft.world.item.Item;
 import net.minecraft.world.item.crafting.CraftingRecipe;
 import net.minecraft.world.item.crafting.RecipeType;
@@ -40,30 +38,34 @@ private boolean isDecompressionRecipe(CraftingRecipe recipe) {
     }
 
     private List<CraftingRecipe> getCompressionRecipes(List<CraftingRecipe> recipes) {
-        List<CraftingRecipe> compressionRecipes = new ObjectArrayList<>();
+        var compressionRecipes = new ObjectArrayList<CraftingRecipe>();
+
         for (var recipe : recipes) {
             if (isCompressionRecipe(recipe)) {
                 compressionRecipes.add(recipe);
             }
         }
+
         return compressionRecipes;
     }
 
     private List<CraftingRecipe> getDecompressionRecipes(List<CraftingRecipe> recipes) {
-        List<CraftingRecipe> decompressionRecipes = new ObjectArrayList<>();
+        var decompressionRecipes = new ObjectArrayList<CraftingRecipe>();
+
         for (var recipe : recipes) {
             if (isDecompressionRecipe(recipe)) {
                 decompressionRecipes.add(recipe);
             }
         }
+
         return decompressionRecipes;
     }
 
     private List<CraftingRecipe> getCandidateRecipes() {
-        MinecraftServer currentServer = AppEng.instance().getCurrentServer();
-        List<CraftingRecipe> allRecipes = currentServer != null
+        var currentServer = AppEng.instance().getCurrentServer();
+        var allRecipes = currentServer != null
                 ? currentServer.getRecipeManager().getAllRecipesFor(RecipeType.CRAFTING)
-                : Collections.emptyList();
+                : new ObjectArrayList<CraftingRecipe>();
 
         return Stream.concat(
                 allRecipes.stream().filter(this::isCompressionRecipe),
@@ -71,22 +73,27 @@ private List<CraftingRecipe> getCandidateRecipes() {
     }
 
     private boolean isReversibleRecipe(CraftingRecipe recipe, List<CraftingRecipe> candidates) {
+        var compressible = false;
+        var decompressible = false;
+
         var input = recipe.getIngredients().get(0);
         var output = recipe.getResultItem();
 
         for (var candidate : candidates) {
             for (var item : candidate.getIngredients().get(0).getItems()) {
                 if (item.getItem().equals(output.getItem())) {
-                    return true;
+                    compressible = true;
                 }
             }
+
             for (var item : input.getItems()) {
                 if (item.getItem().equals(candidate.getResultItem().getItem())) {
-                    return true;
+                    decompressible = true;
                 }
             }
         }
-        return false;
+
+        return compressible && decompressible;
     }
 
     public void load() {
@@ -104,6 +111,7 @@ private Pair<Item, Integer> getSubsequentVariant(Item item, List<CraftingRecipe>
                 }
             }
         }
+
         return null;
     }
 
@@ -111,9 +119,11 @@ private int getMultiplier(CraftingRecipe recipe) {
         if (isCompressionRecipe(recipe)) {
             return recipe.getIngredients().size();
         }
+
         if (isDecompressionRecipe(recipe)) {
             return recipe.getResultItem().getCount();
         }
+
         return 1;
     }