feat: allow earlier state update (#720)

Author: devnatan

inventory-framework-anvil-input/src/main/java/me/devnatan/inventoryframework/AnvilInputFeature.java

@@ -117,7 +117,7 @@ private void handleClick(PlatformView view) {
             final ItemStack ingredientItem = requireNonNull(clickedInventory.getItem(INGREDIENT_SLOT));
             final ItemMeta ingredientMeta = requireNonNull(ingredientItem.getItemMeta());
             ingredientMeta.setDisplayName(text);
-            context.updateState(anvilInput.internalId(), text);
+            context.updateState(anvilInput, text);
             ingredientItem.setItemMeta(ingredientMeta);
 
             if (config.closeOnSelect) {
@@ -189,7 +189,7 @@ private void handleClose(PlatformView view) {
             if (item == null || item.getType() == Material.AIR) return;
 
             final String input = requireNonNull(item.getItemMeta()).getDisplayName();
-            context.updateState(anvilInput.internalId(), input);
+            context.updateState(anvilInput, input);
         });
     }
 }

inventory-framework-api/src/main/java/me/devnatan/inventoryframework/state/StateValueHost.java

@@ -61,11 +61,11 @@ public interface StateValueHost {
      * <p><b><i>This is an internal inventory-framework API that should not be used from outside of
      * this library. No compatibility guarantees are provided.</i></b>
      *
-     * @param id    The state id.
+     * @param state The target state to update.
      * @param value The new state value.
      */
     @ApiStatus.Internal
-    void updateState(long id, Object value);
+    void updateState(State<?> state, Object value);
 
     @ApiStatus.Internal
     void watchState(long id, StateWatcher listener);

inventory-framework-core/src/main/java/me/devnatan/inventoryframework/component/PaginationImpl.java

@@ -438,7 +438,7 @@ public void updated(@NotNull IFSlotRenderContext context) {
      */
     private void simulateStateUpdate() {
         debug("[Pagination] State update simulation triggered on %d", getState().internalId());
-        host.updateState(getState().internalId(), this);
+        host.updateState(getState(), this);
     }
 
     @Override

inventory-framework-core/src/main/java/me/devnatan/inventoryframework/state/BaseMutableState.java

@@ -14,6 +14,6 @@ public BaseMutableState(long id, StateValueFactory valueFactory) {
 
     @Override
     public final void set(T value, @NotNull StateValueHost host) {
-        host.updateState(internalId(), value);
+        host.updateState(this, value);
     }
 }

inventory-framework-core/src/main/java/me/devnatan/inventoryframework/state/DefaultStateValueHost.java

@@ -60,13 +60,13 @@ public void initializeState(long id, @NotNull StateValue value) {
     }
 
     @Override
-    public void updateState(long id, Object value) {
-        final StateValue stateValue = getUninitializedStateValue(id);
+    public void updateState(State<?> state, Object value) {
+        final StateValue stateValue = getInternalStateValue(state);
         final Object oldValue = stateValue.get();
         stateValue.set(value);
 
         final Object newValue = stateValue.get();
-        IFDebug.debug("State %s updated (oldValue = %s, newValue = %s)", id, oldValue, newValue);
+        IFDebug.debug("State %s updated (oldValue = %s, newValue = %s)", state.internalId(), oldValue, newValue);
         callListeners(stateValue, listener -> listener.stateValueSet(this, stateValue, oldValue, newValue));
     }
 

inventory-framework-core/src/main/java/me/devnatan/inventoryframework/state/MutableGenericStateImpl.java

@@ -16,6 +16,6 @@ public MutableGenericStateImpl(long id, @NotNull StateValueFactory valueFactory)
 
     @Override
     public void set(T value, @NotNull StateValueHost host) {
-        host.updateState(internalId(), value);
+        host.updateState(this, value);
     }
 }

inventory-framework-platform-bukkit/src/main/java/me/devnatan/inventoryframework/context/CloseContext.java

@@ -132,8 +132,8 @@ public final StateValue getUninitializedStateValue(long stateId) {
     }
 
     @Override
-    public final void updateState(long id, Object value) {
-        getParent().updateState(id, value);
+    public final void updateState(State<?> state, Object value) {
+        getParent().updateState(state, value);
     }
 
     @Override

inventory-framework-platform-bukkit/src/main/java/me/devnatan/inventoryframework/context/SlotContext.java

@@ -128,8 +128,8 @@ public final void initializeState(long id, @NotNull StateValue value) {
     }
 
     @Override
-    public final void updateState(long id, Object value) {
-        getParent().updateState(id, value);
+    public final void updateState(State<?> state, Object value) {
+        getParent().updateState(state, value);
     }
 
     @Override

inventory-framework-platform-minestom/src/main/kotlin/me/devnatan/inventoryframework/context/CloseContext.kt

@@ -88,10 +88,10 @@ class CloseContext
         override fun getUninitializedStateValue(stateId: Long): StateValue = getParent().getUninitializedStateValue(stateId)
 
         override fun updateState(
-            id: Long,
+            state: State<*>,
             value: Any,
         ) {
-            getParent().updateState(id, value)
+            getParent().updateState(state, value)
         }
 
         override fun toString(): String =

inventory-framework-platform-minestom/src/main/kotlin/me/devnatan/inventoryframework/context/SlotContext.kt

@@ -98,10 +98,10 @@ abstract class SlotContext
         }
 
         override fun updateState(
-            id: Long,
+            state: State<*>,
             value: Any,
         ) {
-            getParent().updateState(id, value)
+            getParent().updateState(state, value)
         }
 
         override fun watchState(