Fixes #3315: Slightly more permissive checks for autocrafting (#3319)

* Fixes #3315: Slightly more permissive checks for autocrafting

Also fixes a bug in the fallback for looking up patterns to only use
ItemStack.EMPTY.

src/main/java/appeng/me/cache/CraftingGridCache.java

@@ -489,7 +489,8 @@ public class CraftingGridCache implements ICraftingGrid, ICraftingProviderHelper
 					if( ais.getItem() == whatToCraft.getItem() && ( !ais.getItem().getHasSubtypes() || ais.getItemDamage() == whatToCraft.getItemDamage() ) )
 					{
 						// TODO: check if OK
-						if( details.isValidItemForSlot( slotIndex, ais.createItemStack(), world ) )
+						// TODO: this is slightly hacky, but fine as long as we only deal with itemstacks
+						if( details.isValidItemForSlot( slotIndex, ais.asItemStackRepresentation(), world ) )
 						{
 							return this.craftableItems.get( ais );
 						}

src/main/java/appeng/util/helpers/ItemComparisonHelper.java

@@ -180,10 +180,25 @@ public class ItemComparisonHelper
 		{
 			return true;
 		}
+
+		final boolean isLeftEmpty = left == null || left.hasNoTags();
+		final boolean isRightEmpty = right == null || right.hasNoTags();
+
+		if( isLeftEmpty && isRightEmpty )
+		{
+			return true;
+		}
+
+		if( isLeftEmpty != isRightEmpty )
+		{
+			return false;
+		}
+
 		if( left != null )
 		{
 			return left.equals( right );
 		}
+
 		return false;
 	}
 }