fix wci

Bump GTNHLib dep for CoordinatePacker

Bump deps

.gitignore

@@ -27,4 +27,3 @@ whitelist.json
 *.iws
 src/main/resources/mixins.*.json
 *.bat
-layout.json

dependencies.gradle

@@ -1,8 +1,8 @@
 // Add your dependencies here
 
 dependencies {
-    api("com.github.GTNewHorizons:GTNHLib:0.6.13:dev")
-    runtimeOnly("com.github.GTNewHorizons:NotEnoughItems:2.7.29-GTNH:dev")
+    api("com.github.GTNewHorizons:GTNHLib:0.3.2:dev")
+    runtimeOnly("com.github.GTNewHorizons:NotEnoughItems:2.6.14-GTNH:dev")
 
     testImplementation(platform('org.junit:junit-bom:5.9.2'))
     testImplementation('org.junit.jupiter:junit-jupiter')

gradle.properties

@@ -50,10 +50,10 @@ enableGenericInjection = true
 # Generate a class with a String field for the mod version named as defined below.
 # If generateGradleTokenClass is empty or not missing, no such class will be generated.
 # If gradleTokenVersion is empty or missing, the field will not be present in the class.
-generateGradleTokenClass = com.gtnewhorizon.structurelib.Tags
+generateGradleTokenClass =
 
 # Name of the token containing the project's current version to generate/replace.
-gradleTokenVersion = VERSION
+gradleTokenVersion = GRADLETOKEN_VERSION
 
 # [DEPRECATED] Mod ID replacement token.
 gradleTokenModId =
@@ -70,7 +70,7 @@ gradleTokenGroupName =
 # The string's content will be replaced with your mod's version when compiled. You should use this to specify your mod's
 # version in @Mod([...], version = VERSION, [...]).
 # Leave these properties empty to skip individual token replacements.
-replaceGradleTokenInFile =
+replaceGradleTokenInFile = StructureLib.java
 
 # In case your mod provides an API for other mods to implement you may declare its package here. Otherwise, you can
 # leave this property empty.
@@ -85,11 +85,6 @@ accessTransformersFile =
 # Provides setup for Mixins if enabled. If you don't know what mixins are: Keep it disabled!
 usesMixins = true
 
-# Set to a non-empty string to configure mixins in a separate source set under src/VALUE, instead of src/main.
-# This can speed up compile times thanks to not running the mixin annotation processor on all input sources.
-# Mixin classes will have access to "main" classes, but not the other way around.
-separateMixinSourceSet =
-
 # Adds some debug arguments like verbose output and class export.
 usesMixinDebug = false
 
@@ -122,15 +117,9 @@ minimizeShadowedDependencies = true
 # If disabled, won't rename the shadowed classes.
 relocateShadowedDependencies = true
 
-# Adds CurseMaven, Modrinth, and some more well-known 1.7.10 repositories.
+# Adds the GTNH maven, CurseMaven, Modrinth, and some more well-known 1.7.10 repositories.
 includeWellKnownRepositories = true
 
-# A list of repositories to exclude from the includeWellKnownRepositories setting. Should be a space separated
-# list of strings, with the acceptable keys being(case does not matter):
-# cursemaven
-# modrinth
-excludeWellKnownRepositories =
-
 # Change these to your Maven coordinates if you want to publish to a custom Maven repository instead of the default GTNH Maven.
 # Authenticate with the MAVEN_USER and MAVEN_PASSWORD environment variables.
 # If you need a more complex setup disable maven publishing here and add a publishing repository to addon.gradle.

gradle/wrapper/gradle-wrapper.properties

@@ -1,6 +1,6 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.13-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.8-bin.zip
 networkTimeout=10000
 validateDistributionUrl=true
 zipStoreBase=GRADLE_USER_HOME

gradlew

@@ -15,8 +15,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-# SPDX-License-Identifier: Apache-2.0
-#
 
 ##############################################################################
 #
@@ -86,7 +84,7 @@ done
 # shellcheck disable=SC2034
 APP_BASE_NAME=${0##*/}
 # Discard cd standard output in case $CDPATH is set (https://github.com/gradle/gradle/issues/25036)
-APP_HOME=$( cd -P "${APP_HOME:-./}" > /dev/null && printf '%s\n' "$PWD" ) || exit
+APP_HOME=$( cd "${APP_HOME:-./}" > /dev/null && pwd -P ) || exit
 
 # Use the maximum available, or set MAX_FD != -1 to use that value.
 MAX_FD=maximum
@@ -205,7 +203,7 @@ fi
 DEFAULT_JVM_OPTS='"-Xmx64m" "-Xms64m"'
 
 # Collect all arguments for the java command:
-#   * DEFAULT_JVM_OPTS, JAVA_OPTS, and optsEnvironmentVar are not allowed to contain shell fragments,
+#   * DEFAULT_JVM_OPTS, JAVA_OPTS, JAVA_OPTS, and optsEnvironmentVar are not allowed to contain shell fragments,
 #     and any embedded shellness will be escaped.
 #   * For example: A user cannot expect ${Hostname} to be expanded, as it is an environment variable and will be
 #     treated as '${Hostname}' itself on the command line.

gradlew.bat

@@ -13,8 +13,6 @@
 @rem See the License for the specific language governing permissions and
 @rem limitations under the License.
 @rem
-@rem SPDX-License-Identifier: Apache-2.0
-@rem
 
 @if "%DEBUG%"=="" @echo off
 @rem ##########################################################################

settings.gradle

@@ -17,7 +17,7 @@ pluginManagement {
 }
 
 plugins {
-    id 'com.gtnewhorizons.gtnhsettingsconvention' version '1.0.37'
+    id 'com.gtnewhorizons.gtnhsettingsconvention' version '1.0.23'
 }
 
 

src/main/java/com/gtnewhorizon/structurelib/StructureLib.java

@@ -42,7 +42,7 @@ import cpw.mods.fml.relauncher.SideOnly;
 @Mod(
         modid = StructureLibAPI.MOD_ID,
         name = "StructureLib",
-        version = Tags.VERSION,
+        version = "GRADLETOKEN_VERSION",
         acceptableRemoteVersions = "*",
         guiFactory = "com.gtnewhorizon.structurelib.GuiFactory")
 public class StructureLib {
@@ -100,8 +100,6 @@ public class StructureLib {
         }
     };
 
-    public static boolean isGTLoaded;
-
     @Mod.EventHandler
     public void preInit(FMLPreInitializationEvent e) {
         ConfigurationHandler.INSTANCE.init(e.getSuggestedConfigurationFile());
@@ -120,8 +118,6 @@ public class StructureLib {
         if (Loader.isModLoaded(STRUCTURECOMPAT_MODID)) {
             COMPAT = Loader.instance().getIndexedModList().get(STRUCTURECOMPAT_MODID).getMod();
         }
-
-        isGTLoaded = Loader.isModLoaded("gregtech");
     }
 
     @Mod.EventHandler

src/main/java/com/gtnewhorizon/structurelib/StructureLibAPI.java

@@ -72,7 +72,7 @@ public class StructureLibAPI {
      */
     public static void enableInstrument(Object identifier) {
         if (isInstrumentEnabled()) throw new IllegalStateException();
-        instrument.set(identifier);
+        instrument.set(instrument);
     }
 
     /**

src/main/java/com/gtnewhorizon/structurelib/gui/GuiScreenConfigureChannels.java

@@ -25,17 +25,6 @@ import com.gtnewhorizon.structurelib.gui.GuiScrollableList.IGuiScreen;
 
 public class GuiScreenConfigureChannels extends GuiScreen implements IGuiScreen {
 
-    private static final String I18N_PREFIX = "item.structurelib.constructableTrigger.gui.";
-
-    private static final int ADD_BTN = 0;
-    private static final int UNSET_BTN = 1;
-    private static final int WIPE_BTN = 2;
-    private static final int SHOW_ERROR_BTN = 3;
-    private static final int GT_NO_HATCH_BTN = 4;
-
-    private static final String SHOW_ERROR_CHANNEL = "show_error";
-    private static final String GT_NO_HATCH_CHANNEL = "gt_no_hatch";
-
     private static final int KEY_MAX_WIDTH = 50;
     private final ItemStack trigger;
     private final GuiChannelsList list;
@@ -139,7 +128,7 @@ public class GuiScreenConfigureChannels extends GuiScreen implements IGuiScreen
 
         addButton(
                 new GuiButton(
-                        ADD_BTN,
+                        0,
                         guiLeft + 12,
                         guiTop + 157,
                         47,
@@ -147,7 +136,7 @@ public class GuiScreenConfigureChannels extends GuiScreen implements IGuiScreen
                         I18n.format("item.structurelib.constructableTrigger.gui.add")));
         addButton(
                 new GuiButton(
-                        UNSET_BTN,
+                        1,
                         guiLeft + 65,
                         guiTop + 157,
                         47,
@@ -155,27 +144,13 @@ public class GuiScreenConfigureChannels extends GuiScreen implements IGuiScreen
                         I18n.format("item.structurelib.constructableTrigger.gui.unset")));
         addButton(
                 new GuiButton(
-                        WIPE_BTN,
+                        2,
                         guiLeft + 118,
                         guiTop + 157,
                         47,
                         20,
                         I18n.format("item.structurelib.constructableTrigger.gui.wipe")));
 
-        addButton(
-                new GuiButton(
-                        SHOW_ERROR_BTN,
-                        StructureLib.isGTLoaded ? guiLeft + 12 : guiLeft + 52,
-                        guiTop + 180,
-                        73,
-                        20,
-                        ""));
-
-        // only show GT hatch button if GT is loaded
-        if (StructureLib.isGTLoaded) {
-            addButton(new GuiButton(GT_NO_HATCH_BTN, guiLeft + 92, guiTop + 180, 73, 20, ""));
-        }
-
         updateButtons();
     }
 
@@ -196,7 +171,7 @@ public class GuiScreenConfigureChannels extends GuiScreen implements IGuiScreen
 
     @Override
     public int getYSize() {
-        return 211;
+        return 188;
     }
 
     @Override
@@ -224,29 +199,10 @@ public class GuiScreenConfigureChannels extends GuiScreen implements IGuiScreen
         return buttonList;
     }
 
-    private boolean isMouseOverValue() {
-        int mx = Mouse.getEventX() * this.width / this.mc.displayWidth;
-        int my = this.height - Mouse.getEventY() * this.height / this.mc.displayHeight - 1;
-        return mx >= value.xPosition && mx < value.xPosition + value.width
-                && my >= value.yPosition
-                && my < value.yPosition + value.height;
-
-    }
-
     @Override
     public void handleMouseInput() {
         int delta = Mouse.getEventDWheel();
-        if (delta != 0) {
-            if (isMouseOverValue()) {
-                if (delta > 0) {
-                    value.setText(String.valueOf(getValue() + 1));
-                } else {
-                    value.setText(String.valueOf(Math.max(getValue() - 1, 1)));
-                }
-            } else {
-                list.handleDWheel(delta);
-            }
-        }
+        if (delta != 0) list.handleDWheel(delta);
         super.handleMouseInput();
     }
 
@@ -294,26 +250,10 @@ public class GuiScreenConfigureChannels extends GuiScreen implements IGuiScreen
         // STACKOVERFLOW!
         String keyText = key.getText();
         boolean existing = !StringUtils.isEmpty(keyText) && ChannelDataAccessor.hasSubChannel(trigger, keyText);
-        getButtonList().get(ADD_BTN).displayString = existing ? I18n.format(I18N_PREFIX + "set")
-                : I18n.format(I18N_PREFIX + "add");
-        getButtonList().get(ADD_BTN).enabled = !StringUtils.isBlank(value.getText())
-                && Integer.parseInt(value.getText()) > 0;
-        getButtonList().get(UNSET_BTN).enabled = existing && !StringUtils.isBlank(value.getText());
-
-        if (ChannelDataAccessor.hasSubChannel(trigger, SHOW_ERROR_CHANNEL)) {
-            getButtonList().get(SHOW_ERROR_BTN).displayString = I18n.format(I18N_PREFIX + "error.hide");
-        } else {
-            getButtonList().get(SHOW_ERROR_BTN).displayString = I18n.format(I18N_PREFIX + "error.show");
-        }
-
-        // this button only exists if GT is loaded.
-        if (StructureLib.isGTLoaded) {
-            if (ChannelDataAccessor.hasSubChannel(trigger, GT_NO_HATCH_CHANNEL)) {
-                getButtonList().get(GT_NO_HATCH_BTN).displayString = I18n.format(I18N_PREFIX + "gt_no_hatch.disable");
-            } else {
-                getButtonList().get(GT_NO_HATCH_BTN).displayString = I18n.format(I18N_PREFIX + "gt_no_hatch.enable");
-            }
-        }
+        getButtonList().get(0).displayString = existing ? I18n.format("item.structurelib.constructableTrigger.gui.set")
+                : I18n.format("item.structurelib.constructableTrigger.gui.add");
+        getButtonList().get(0).enabled = !StringUtils.isBlank(value.getText());
+        getButtonList().get(1).enabled = existing && !StringUtils.isBlank(value.getText());
     }
 
     private int getValue() {
@@ -328,35 +268,18 @@ public class GuiScreenConfigureChannels extends GuiScreen implements IGuiScreen
     protected void actionPerformed(GuiButton btn) {
         if (btn == null) return;
         switch (btn.id) {
-            case ADD_BTN:
+            case 0:
                 int value = getValue();
                 if (value <= 0) return;
                 ChannelDataAccessor.setChannelData(trigger, key.getText(), value);
                 break;
-            case UNSET_BTN:
+            case 1:
                 ChannelDataAccessor.unsetChannelData(trigger, key.getText());
                 break;
-            case WIPE_BTN:
+            case 2:
                 ChannelDataAccessor.wipeChannelData(trigger);
                 break;
-            case SHOW_ERROR_BTN:
-                if (ChannelDataAccessor.hasSubChannel(trigger, SHOW_ERROR_CHANNEL)) {
-                    ChannelDataAccessor.unsetChannelData(trigger, SHOW_ERROR_CHANNEL);
-                } else {
-                    ChannelDataAccessor.setChannelData(trigger, SHOW_ERROR_CHANNEL, 1);
-                }
-                break;
-            case GT_NO_HATCH_BTN:
-                if (ChannelDataAccessor.hasSubChannel(trigger, GT_NO_HATCH_CHANNEL)) {
-                    ChannelDataAccessor.unsetChannelData(trigger, GT_NO_HATCH_CHANNEL);
-                } else {
-                    ChannelDataAccessor.setChannelData(trigger, GT_NO_HATCH_CHANNEL, 1);
-                }
-                break;
         }
-
-        updateButtons();
-
         super.actionPerformed(btn);
     }
 
@@ -427,7 +350,6 @@ public class GuiScreenConfigureChannels extends GuiScreen implements IGuiScreen
                 Entry<String, Integer> e = getElementAt(elementIndex);
                 if (e != null) {
                     ChannelDataAccessor.unsetChannelData(trigger, e.getKey());
-                    updateButtons();
                     return false;
                 }
             }

src/main/java/com/gtnewhorizon/structurelib/item/ItemConstructableTrigger.java

@@ -17,9 +17,6 @@ import org.lwjgl.input.Keyboard;
 import com.gtnewhorizon.structurelib.StructureLib;
 import com.gtnewhorizon.structurelib.alignment.constructable.ChannelDataAccessor;
 import com.gtnewhorizon.structurelib.alignment.constructable.ConstructableUtility;
-import com.gtnewhorizon.structurelib.gui.GuiScreenConfigureChannels;
-
-import cpw.mods.fml.common.FMLCommonHandler;
 
 public class ItemConstructableTrigger extends Item {
 
@@ -35,7 +32,7 @@ public class ItemConstructableTrigger extends Item {
             if (player.isSneaking()) {
                 StructureLib.instance().proxy().displayConfigGUI("registries");
             } else {
-                FMLCommonHandler.instance().showGuiScreen(new GuiScreenConfigureChannels(stack));
+                player.openGui(StructureLib.instance(), 0, world, player.inventory.currentItem, 0, 0);
             }
         }
         return super.onItemRightClick(stack, world, player);

src/main/java/com/gtnewhorizon/structurelib/structure/IStructureDefinition.java

@@ -12,7 +12,6 @@ import net.minecraft.world.World;
 
 import com.gtnewhorizon.structurelib.StructureLib;
 import com.gtnewhorizon.structurelib.StructureLibAPI;
-import com.gtnewhorizon.structurelib.alignment.constructable.ChannelDataAccessor;
 import com.gtnewhorizon.structurelib.alignment.constructable.ISurvivalConstructable;
 import com.gtnewhorizon.structurelib.alignment.enumerable.ExtendedFacing;
 
@@ -51,7 +50,7 @@ import com.gtnewhorizon.structurelib.alignment.enumerable.ExtendedFacing;
  * itself.</li>
  * <li>The "C" position would be the number of blocks between you and controller, not counting controller itself.</li>
  * </ul>
- *
+ * 
  * @param <T> Type of the context object. Usually this will be the multiblock controller.
  */
 public interface IStructureDefinition<T> {
@@ -73,7 +72,7 @@ public interface IStructureDefinition<T> {
 
     /**
      * Run a structure check.
-     *
+     * 
      * @param object              context object. usually multiblock controller.
      * @param piece               the structure piece's string identifier.
      * @param world               the world object this check takes place in.
@@ -108,7 +107,7 @@ public interface IStructureDefinition<T> {
 
     /**
      * Spawn hint particles. Should not be called on server side.
-     *
+     * 
      * @param object         context object. usually multiblock controller.
      * @param piece          the structure piece's string identifier.
      * @param world          the world object this check takes place in.
@@ -321,7 +320,7 @@ public interface IStructureDefinition<T> {
 
     /**
      * Low level utility.
-     *
+     * 
      * @param object                                  context object. usually multiblock controller.
      * @param trigger                                 The trigger item that contains channel data.
      * @param elements                                the structure piece
@@ -395,10 +394,6 @@ public interface IStructureDefinition<T> {
                         basePositionC,
                         ignoreBlockUnloaded((e, w, x, y, z, a, b, c) -> {
                             e.spawnHint(object, world, x, y, z, trigger);
-                            if (ChannelDataAccessor.hasSubChannel(trigger, "show_error")
-                                    && !e.couldBeValid(object, world, x, y, z, trigger)) {
-                                StructureLibAPI.markHintParticleError(StructureLib.getCurrentPlayer(), world, x, y, z);
-                            }
                             return true;
                         }),
                         "spawnHint");
@@ -425,7 +420,7 @@ public interface IStructureDefinition<T> {
 
     /**
      * Create a new instance of builder.
-     *
+     * 
      * @param <T> type of context object
      */
     static <T> StructureDefinition.Builder<T> builder() {

src/main/java/com/gtnewhorizon/structurelib/structure/IStructureElement.java

@@ -32,16 +32,6 @@ public interface IStructureElement<T> {
 
     boolean check(T t, World world, int x, int y, int z);
 
-    /**
-     * Pure (stateless and side effect free) function to test if current block could be valid. Used to give user hints
-     * about incorrectly placed blocks given a controller and trigger item. If couldBeValid(...) == false for a set of
-     * channels, with those same tiers, check(...) == false. Not required, defaults to true which is always safe but
-     * won't give the user error hints.
-     */
-    default boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-        return true;
-    }
-
     boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger);
 
     boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger);
@@ -195,11 +185,6 @@ public interface IStructureElement<T> {
                 return IStructureElement.this.check(t, world, x, y, z);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return IStructureElement.this.couldBeValid(t, world, x, y, z, trigger);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 return IStructureElement.this.spawnHint(t, world, x, y, z, trigger);

src/main/java/com/gtnewhorizon/structurelib/structure/IStructureElementChain.java

@@ -31,16 +31,6 @@ public interface IStructureElementChain<T> extends IStructureElement<T> {
         return false;
     }
 
-    @Override
-    default boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-        for (IStructureElement<T> fallback : fallbacks()) {
-            if (fallback.couldBeValid(t, world, x, y, z, trigger)) {
-                return true;
-            }
-        }
-        return false;
-    }
-
     @Override
     default boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
         for (IStructureElement<T> fallback : fallbacks()) {
@@ -72,8 +62,7 @@ public interface IStructureElementChain<T> extends IStructureElement<T> {
             if (e == null) continue;
             if (predicate == null) predicate = e.getPredicate();
             else predicate = predicate.or(e.getPredicate());
-            Iterable<ItemStack> stacks = e.getStacks();
-            if (stacks != null) is.add(stacks);
+            is.add(e.getStacks());
         }
         if (predicate == null) return null;
         return new BlocksToPlace(predicate, Iterables.concat(is));

src/main/java/com/gtnewhorizon/structurelib/structure/IStructureElementCheckOnly.java

@@ -14,11 +14,6 @@ public interface IStructureElementCheckOnly<T> extends IStructureElement<T> {
         return false;
     }
 
-    @Override
-    default boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-        return true;
-    }
-
     @Override
     default boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
         return false;

src/main/java/com/gtnewhorizon/structurelib/structure/IStructureNavigate.java

@@ -17,11 +17,6 @@ interface IStructureNavigate<T> extends IStructureElement<T> {
         return true;
     }
 
-    @Override
-    default boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-        return true;
-    }
-
     @Override
     default boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
         return true;

src/main/java/com/gtnewhorizon/structurelib/structure/LazyStructureElement.java

@@ -32,11 +32,6 @@ class LazyStructureElement<T> implements IStructureElementDeferred<T> {
         return get(t).check(t, world, x, y, z);
     }
 
-    @Override
-    public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-        return get(t).couldBeValid(t, world, x, y, z, trigger);
-    }
-
     @Override
     public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
         return get(t).placeBlock(t, world, x, y, z, trigger);

src/main/java/com/gtnewhorizon/structurelib/structure/StructureUtility.java

@@ -1,7 +1,5 @@
 package com.gtnewhorizon.structurelib.structure;
 
-import static com.gtnewhorizon.structurelib.StructureLib.LOGGER;
-import static com.gtnewhorizon.structurelib.StructureLib.PANIC_MODE;
 import static java.lang.Integer.MIN_VALUE;
 
 import java.util.Arrays;
@@ -157,11 +155,6 @@ public class StructureUtility {
             return world.isAirBlock(x, y, z);
         }
 
-        @Override
-        public boolean couldBeValid(Object t, World world, int x, int y, int z, ItemStack trigger) {
-            return check(t, world, x, y, z);
-        }
-
         @Override
         public boolean spawnHint(Object o, World world, int x, int y, int z, ItemStack trigger) {
             StructureLibAPI.hintParticle(world, x, y, z, StructureLibAPI.getBlockHint(), 13);
@@ -193,11 +186,6 @@ public class StructureUtility {
             return !world.isAirBlock(x, y, z);
         }
 
-        @Override
-        public boolean couldBeValid(Object t, World world, int x, int y, int z, ItemStack trigger) {
-            return check(t, world, x, y, z);
-        }
-
         @Override
         public boolean spawnHint(Object o, World world, int x, int y, int z, ItemStack trigger) {
             StructureLibAPI.hintParticle(world, x, y, z, StructureLibAPI.getBlockHint(), 14);
@@ -237,11 +225,6 @@ public class StructureUtility {
             return false;
         }
 
-        @Override
-        public boolean couldBeValid(Object t, World world, int x, int y, int z, ItemStack trigger) {
-            return check(t, world, x, y, z);
-        }
-
         @Override
         public boolean spawnHint(Object o, World world, int x, int y, int z, ItemStack trigger) {
             StructureLibAPI.hintParticle(world, x, y, z, StructureLibAPI.getBlockHint(), 15);
@@ -335,24 +318,18 @@ public class StructureUtility {
         int itemMeta = itemBlock instanceof ISpecialItemBlock
                 ? ((ISpecialItemBlock) itemBlock).getItemMetaFromBlockMeta(block, meta)
                 : meta;
-        ItemStack stack = new ItemStack(itemBlock, 1, itemMeta);
-        if (!s.takeOne(stack, true)) {
+        if (!s.takeOne(new ItemStack(itemBlock, 1, itemMeta), false)) {
             if (chatter != null) chatter.accept(
                     new ChatComponentTranslation(
                             "structurelib.autoplace.error.no_simple_block",
-                            stack.func_151000_E()));
+                            new ItemStack(itemBlock, 1, itemMeta).func_151000_E()));
             return PlaceResult.REJECT;
         }
         if (block instanceof ICustomBlockSetting) {
-            ICustomBlockSetting blockCustom = (ICustomBlockSetting) block;
-            blockCustom.setBlock(world, x, y, z, meta);
-        } else if (!stack.copy()
-                .tryPlaceItemIntoWorld(actor, world, x, y, z, ForgeDirection.UP.ordinal(), 0.5f, 0.5f, 0.5f)) {
-                    return PlaceResult.REJECT;
-                }
-        if (!s.takeOne(stack, false)) {
-            // rollback
-            world.setBlockToAir(x, y, z);
+            ICustomBlockSetting block2 = (ICustomBlockSetting) block;
+            block2.setBlock(world, x, y, z, meta);
+        } else {
+            world.setBlock(x, y, z, block, meta, 2);
         }
         return PlaceResult.ACCEPT;
     }
@@ -502,11 +479,6 @@ public class StructureUtility {
                 return true;
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return check(t, world, x, y, z);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 StructureLibAPI.hintParticle(world, x, y, z, StructureLibAPI.getBlockHint(), meta);
@@ -527,11 +499,6 @@ public class StructureUtility {
                 return true;
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return check(t, world, x, y, z);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 StructureLibAPI.hintParticle(world, x, y, z, icons.get());
@@ -552,11 +519,6 @@ public class StructureUtility {
                 return true;
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return check(t, world, x, y, z);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 StructureLibAPI.hintParticleTinted(world, x, y, z, icons.get(), RGBa);
@@ -594,7 +556,7 @@ public class StructureUtility {
             TIER current = getter.apply(t);
             if (Objects.equals(notSet, current)) {
                 if (Objects.equals(notSet, tier)) {
-                    if (PANIC_MODE) {
+                    if (StructureLib.PANIC_MODE) {
                         throw new AssertionError("tierExtractor should never return notSet: " + notSet);
                     } else {
                         StructureLib.LOGGER.error("#########################################");
@@ -705,23 +667,9 @@ public class StructureUtility {
             }
 
             private Pair<Block, Integer> getHint(ItemStack trigger) {
-                if (hints.isEmpty()) return null;
                 return hints.get(Math.min(Math.max(trigger.stackSize, 1), hints.size()) - 1);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                Pair<Block, Integer> hint = getHint(trigger);
-                if (hint == null) return true;
-                TIER hintTier = tierExtractor.convert(hint.getKey(), hint.getValue());
-                Block block = world.getBlock(x, y, z);
-                int meta = world.getBlockMetadata(x, y, z);
-                TIER worldTier = tierExtractor.convert(block, meta);
-
-                // if the block is in the same tier as the hint block, this could be valid
-                return Objects.equals(hintTier, worldTier);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 Pair<Block, Integer> hint = getHint(trigger);
@@ -819,11 +767,6 @@ public class StructureUtility {
                 return world.getBlock(x, y, z) == getBlock() && (wildcard || world.getBlockMetadata(x, y, z) == meta);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return check(t, world, x, y, z);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 if (getBlock() == null) return error().spawnHint(t, world, x, y, z, trigger);
@@ -898,16 +841,10 @@ public class StructureUtility {
 
             @Override
             public boolean check(T t, World world, int x, int y, int z) {
-                if (init()) return world.getBlock(x, y, z) == block && world.getBlockMetadata(x, y, z) == meta;
+                if (init()) return world.getBlock(x, y, z) != block && world.getBlockMetadata(x, y, z) == meta;
                 else return fallback.check(t, world, x, y, z);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                if (init()) return world.getBlock(x, y, z) == block && world.getBlockMetadata(x, y, z) == meta;
-                else return fallback.couldBeValid(t, world, x, y, z, trigger);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 if (init()) {
@@ -984,11 +921,6 @@ public class StructureUtility {
                 return blocsMap.getOrDefault(worldBlock, MIN_VALUE) == worldBlock.getDamageValue(world, x, y, z);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return check(t, world, x, y, z);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 StructureLibAPI.hintParticle(world, x, y, z, hintBlock, hintMeta);
@@ -1026,11 +958,6 @@ public class StructureUtility {
                         .contains(worldBlock.getDamageValue(world, x, y, z));
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return check(t, world, x, y, z);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 StructureLibAPI.hintParticle(world, x, y, z, hintBlock, hintMeta);
@@ -1055,11 +982,6 @@ public class StructureUtility {
                 return block == worldBlock && meta == worldBlock.getDamageValue(world, x, y, z);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return check(t, world, x, y, z);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 StructureLibAPI.hintParticle(world, x, y, z, hintBlock, hintMeta);
@@ -1094,13 +1016,6 @@ public class StructureUtility {
                 return iBlockAdder.apply(t, worldBlock, worldBlock.getDamageValue(world, x, y, z));
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                // calling iBlockAdder can potentially modify external state
-                // therefore we assume this can always be valid.
-                return true;
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 StructureLibAPI.hintParticle(world, x, y, z, hintBlock, hintMeta);
@@ -1134,11 +1049,6 @@ public class StructureUtility {
                     return blocsMap.getOrDefault(worldBlock, MIN_VALUE) == worldBlock.getDamageValue(world, x, y, z);
                 }
 
-                @Override
-                public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                    return check(t, world, x, y, z);
-                }
-
                 @Override
                 public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                     ((ICustomBlockSetting) defaultBlock).setBlock(world, x, y, z, defaultMeta);
@@ -1183,11 +1093,6 @@ public class StructureUtility {
                     return blocsMap.getOrDefault(worldBlock, MIN_VALUE) == worldBlock.getDamageValue(world, x, y, z);
                 }
 
-                @Override
-                public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                    return check(t, world, x, y, z);
-                }
-
                 @Override
                 public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                     world.setBlock(x, y, z, defaultBlock, defaultMeta, 2);
@@ -1253,11 +1158,6 @@ public class StructureUtility {
                             .contains(worldBlock.getDamageValue(world, x, y, z));
                 }
 
-                @Override
-                public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                    return check(t, world, x, y, z);
-                }
-
                 @Override
                 public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                     ((ICustomBlockSetting) defaultBlock).setBlock(world, x, y, z, defaultMeta);
@@ -1303,11 +1203,6 @@ public class StructureUtility {
                             .contains(worldBlock.getDamageValue(world, x, y, z));
                 }
 
-                @Override
-                public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                    return check(t, world, x, y, z);
-                }
-
                 @Override
                 public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                     world.setBlock(x, y, z, defaultBlock, defaultMeta, 2);
@@ -1356,14 +1251,6 @@ public class StructureUtility {
         if (block == null || defaultBlock == null) {
             throw new IllegalArgumentException();
         }
-        if (block == Blocks.air) {
-            if (PANIC_MODE) {
-                throw new IllegalArgumentException("ofBlock() does not accept air. use isAir() instead");
-            } else {
-                LOGGER.warn("ofBlock() does not accept air. use isAir() instead");
-                return isAir();
-            }
-        }
         if (block instanceof ICustomBlockSetting) {
             return new IStructureElement<T>() {
 
@@ -1373,11 +1260,6 @@ public class StructureUtility {
                     return block == worldBlock && meta == worldBlock.getDamageValue(world, x, y, z);
                 }
 
-                @Override
-                public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                    return check(t, world, x, y, z);
-                }
-
                 @Override
                 public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                     ((ICustomBlockSetting) defaultBlock).setBlock(world, x, y, z, defaultMeta);
@@ -1405,11 +1287,6 @@ public class StructureUtility {
                     return block == worldBlock && meta == worldBlock.getDamageValue(world, x, y, z);
                 }
 
-                @Override
-                public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                    return check(t, world, x, y, z);
-                }
-
                 @Override
                 public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                     world.setBlock(x, y, z, defaultBlock, defaultMeta, 2);
@@ -1446,11 +1323,6 @@ public class StructureUtility {
                     return block == world.getBlock(x, y, z);
                 }
 
-                @Override
-                public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                    return check(t, world, x, y, z);
-                }
-
                 @Override
                 public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                     ((ICustomBlockSetting) defaultBlock).setBlock(world, x, y, z, defaultMeta);
@@ -1478,11 +1350,6 @@ public class StructureUtility {
                     return block == world.getBlock(x, y, z);
                 }
 
-                @Override
-                public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                    return check(t, world, x, y, z);
-                }
-
                 @Override
                 public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                     world.setBlock(x, y, z, defaultBlock, defaultMeta, 2);
@@ -1549,13 +1416,6 @@ public class StructureUtility {
                     return iBlockAdder.apply(t, worldBlock, worldBlock.getDamageValue(world, x, y, z));
                 }
 
-                @Override
-                public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                    // calling iBlockAdder can potentially modify external state
-                    // therefore we assume this can always be valid.
-                    return true;
-                }
-
                 @Override
                 public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                     ((ICustomBlockSetting) defaultBlock).setBlock(world, x, y, z, defaultMeta);
@@ -1595,13 +1455,6 @@ public class StructureUtility {
                     return iBlockAdder.apply(t, worldBlock, worldBlock.getDamageValue(world, x, y, z));
                 }
 
-                @Override
-                public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                    // calling iBlockAdder can potentially modify external state
-                    // therefore we assume this can always be valid.
-                    return true;
-                }
-
                 @Override
                 public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                     world.setBlock(x, y, z, defaultBlock, defaultMeta, 2);
@@ -1657,13 +1510,6 @@ public class StructureUtility {
                 return iTileAdder.apply(t, tileEntity);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                // calling iTileAdder can potentially modify external state
-                // therefore we assume this can always be valid.
-                return true;
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 StructureLibAPI.hintParticle(world, x, y, z, hintBlock, hintMeta);
@@ -1690,13 +1536,6 @@ public class StructureUtility {
                 return tileClass.isInstance(tileEntity) && iTileAdder.test(t, tileClass.cast(tileEntity));
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                // calling iTileAdder can potentially modify external state
-                // therefore we assume this can always be valid.
-                return true;
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 StructureLibAPI.hintParticle(world, x, y, z, hintBlock, hintMeta);
@@ -1730,11 +1569,6 @@ public class StructureUtility {
                 return check;
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return element.couldBeValid(t, world, x, y, z, trigger);
-            }
-
             @Override
             public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                 return element.placeBlock(t, world, x, y, z, trigger);
@@ -1786,11 +1620,6 @@ public class StructureUtility {
                 return check;
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return element.couldBeValid(t, world, x, y, z, trigger);
-            }
-
             @Override
             public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                 return element.placeBlock(t, world, x, y, z, trigger);
@@ -1849,13 +1678,6 @@ public class StructureUtility {
                 return predicate.test(t) && downstream.check(t, world, x, y, z);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                // this COULD potentially impact external state
-                // but predicate really should not have side effects
-                return predicate.test(t) && downstream.couldBeValid(t, world, x, y, z, trigger);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 return predicate.test(t) && downstream.spawnHint(t, world, x, y, z, trigger);
@@ -1952,11 +1774,6 @@ public class StructureUtility {
                 return elem.check(t.getCurrentContext(), world, x, y, z);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return elem.couldBeValid(t.getCurrentContext(), world, x, y, z, trigger);
-            }
-
             @Override
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 return elem.spawnHint(t.getCurrentContext(), world, x, y, z, trigger);
@@ -2044,11 +1861,6 @@ public class StructureUtility {
                 return to.get().check(t, world, x, y, z);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return to.get().couldBeValid(t, world, x, y, z, trigger);
-            }
-
             @Override
             public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                 return to.get().placeBlock(t, world, x, y, z, trigger);
@@ -2102,11 +1914,6 @@ public class StructureUtility {
                 return to.apply(t).check(t, world, x, y, z);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return to.apply(t).couldBeValid(t, world, x, y, z, trigger);
-            }
-
             @Override
             public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                 return to.apply(t).placeBlock(t, world, x, y, z, trigger);
@@ -2319,11 +2126,6 @@ public class StructureUtility {
                 return to.apply(t, null).check(t, world, x, y, z);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return to.apply(t, trigger).couldBeValid(t, world, x, y, z, trigger);
-            }
-
             @Override
             public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                 return to.apply(t, trigger).placeBlock(t, world, x, y, z, trigger);
@@ -2516,11 +2318,6 @@ public class StructureUtility {
                 return toCheck.apply(t).check(t, world, x, y, z);
             }
 
-            @Override
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                return to.apply(t, trigger).couldBeValid(t, world, x, y, z, trigger);
-            }
-
             @Override
             public boolean placeBlock(T t, World world, int x, int y, int z, ItemStack trigger) {
                 return to.apply(t, trigger).placeBlock(t, world, x, y, z, trigger);
@@ -2765,11 +2562,6 @@ public class StructureUtility {
                 return backing.check(t, world, x, y, z);
             }
 
-            public boolean couldBeValid(T t, World world, int x, int y, int z, ItemStack trigger) {
-                ItemStack newTrigger = ChannelDataAccessor.withChannel(trigger, channel);
-                return backing.couldBeValid(t, world, x, y, z, newTrigger);
-            }
-
             public boolean spawnHint(T t, World world, int x, int y, int z, ItemStack trigger) {
                 ItemStack newTrigger = ChannelDataAccessor.withChannel(trigger, channel);
                 // We know spawnHint will only be called on client side, so the lack of player isn't quite an issue

src/main/java/com/gtnewhorizon/structurelib/util/SortedRegistry.java

@@ -1,64 +0,0 @@
-package com.gtnewhorizon.structurelib.util;
-
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.Iterator;
-import java.util.List;
-import java.util.NavigableMap;
-import java.util.Spliterator;
-import java.util.TreeMap;
-import java.util.function.Consumer;
-
-public class SortedRegistry<V> implements Iterable<V> {
-
-    private final NavigableMap<String, V> store = new TreeMap<>();
-    /**
-     * Store stuff in an array for faster iteration access than a red black tree
-     */
-    private List<V> baked = Collections.emptyList();
-
-    public void register(String key, V val) {
-        if (key == null || val == null) throw new NullPointerException();
-        V old = store.putIfAbsent(key, val);
-        if (old != null) {
-            throw new IllegalArgumentException("Duplicate key: " + key);
-        }
-        // TODO allow a user to edit the order with a configuration file
-        baked = new ArrayList<>(store.values());
-    }
-
-    public int size() {
-        return store.size();
-    }
-
-    public boolean isEmpty() {
-        return store.isEmpty();
-    }
-
-    public boolean containsKey(String key) {
-        return store.containsKey(key);
-    }
-
-    public boolean containsValue(V value) {
-        return store.containsValue(value);
-    }
-
-    public V get(String key) {
-        return store.get(key);
-    }
-
-    @Override
-    public Iterator<V> iterator() {
-        return baked.iterator();
-    }
-
-    @Override
-    public void forEach(Consumer<? super V> action) {
-        baked.forEach(action);
-    }
-
-    @Override
-    public Spliterator<V> spliterator() {
-        return baked.spliterator();
-    }
-}

src/main/resources/assets/structurelib/lang/en_US.lang

@@ -1,4 +1,3 @@
-tile.structurelib.blockhint.name=Hint
 tile.structurelib.blockhint.0.name=Hint 1 dot
 tile.structurelib.blockhint.1.name=Hint 2 dots
 tile.structurelib.blockhint.2.name=Hint 3 dots
@@ -36,10 +35,6 @@ item.structurelib.constructableTrigger.gui.add=Add
 item.structurelib.constructableTrigger.gui.set=Set
 item.structurelib.constructableTrigger.gui.unset=Unset
 item.structurelib.constructableTrigger.gui.wipe=§4Wipe
-item.structurelib.constructableTrigger.gui.error.show=Show Errors
-item.structurelib.constructableTrigger.gui.error.hide=Hide Errors
-item.structurelib.constructableTrigger.gui.gt_no_hatch.enable=No Hatch
-item.structurelib.constructableTrigger.gui.gt_no_hatch.disable=Hatches
 
 structurelib.autoplace.built_stat=Built %s blocks
 structurelib.autoplace.complete=§aThis multiblock has completed!

src/main/resources/assets/structurelib/lang/zh_CN.lang

@@ -1,4 +1,3 @@
-tile.structurelib.blockhint.name=提示方块
 tile.structurelib.blockhint.0.name=提示方块：1号
 tile.structurelib.blockhint.1.name=提示方块：2号
 tile.structurelib.blockhint.2.name=提示方块：3号
@@ -36,10 +35,6 @@ item.structurelib.constructableTrigger.gui.add=添加
 item.structurelib.constructableTrigger.gui.set=替换
 item.structurelib.constructableTrigger.gui.unset=删除
 item.structurelib.constructableTrigger.gui.wipe=§4全部清空
-item.structurelib.constructableTrigger.gui.error.show=显示结构错误
-item.structurelib.constructableTrigger.gui.error.hide=隐藏结构错误
-item.structurelib.constructableTrigger.gui.gt_no_hatch.enable=跳过舱室
-item.structurelib.constructableTrigger.gui.gt_no_hatch.disable=放置舱室
 
 structurelib.autoplace.built_stat=已放置 %s 个方块
 structurelib.autoplace.complete=§a这个多方块结构已经完成自动搭建！