Skip to content

Allow disabling items & change research disabling logic#153

Merged
LordIdra merged 13 commits into
masterfrom
allow-disabling-items
Jun 30, 2025
Merged

Allow disabling items & change research disabling logic#153
LordIdra merged 13 commits into
masterfrom
allow-disabling-items

Conversation

@LordIdra

@LordIdra LordIdra commented Jun 22, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Adds a disabled-items config option
  • Disabled items can be crafted, but cannot be used
  • Removes the inventory ejection logic for unresearched items
  • Unresearched items now cannot be crafted or used
  • Added PrePylonCraftEvent and PylonCraftEvent to facilitate this - must be called manually by blocks with recipes. Not called by vanilla crafting recipes.

closes #128
closes #85

Seggan
Seggan requested changes Jun 27, 2025
View reviewed changes
Comment thread pylon-core/src/main/kotlin/io/github/pylonmc/pylon/core/block/BlockListener.kt Outdated
Comment thread pylon-core/src/main/kotlin/io/github/pylonmc/pylon/core/command/PylonCommand.kt Outdated
Comment thread pylon-core/src/main/kotlin/io/github/pylonmc/pylon/core/item/PylonItem.kt Outdated
Comment thread pylon-core/src/main/kotlin/io/github/pylonmc/pylon/core/item/research/Research.kt Outdated
Comment thread pylon-core/src/main/kotlin/io/github/pylonmc/pylon/core/recipe/RecipeListener.kt Outdated
Comment thread pylon-core/src/main/kotlin/io/github/pylonmc/pylon/core/recipe/RecipeListener.kt Outdated
@LordIdra
LordIdra requested a review from Seggan June 28, 2025 11:44
Seggan
Seggan reviewed Jun 29, 2025
View reviewed changes
Comment thread pylon-core/src/main/kotlin/io/github/pylonmc/pylon/core/item/research/Research.kt Outdated
@LordIdra
LordIdra requested a review from Seggan June 29, 2025 14:56
Base automatically changed from add-pylon-guide to master June 30, 2025 20:30
@LordIdra
LordIdra merged commit 3d38a7e into master Jun 30, 2025
1 check passed
@LordIdra
LordIdra deleted the allow-disabling-items branch June 30, 2025 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Seggan Seggan Seggan approved these changes
+1 more reviewer
@ihateblueb ihateblueb ihateblueb left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

/pylon key Allow enabling/disabling of items

3 participants

@LordIdra @Seggan @ihateblueb