Fix Bug Command Hub and Support Slimefun Experimental version#25
Merged
PranavVerma-droid merged 3 commits intoSlimefun-Addon-Community:devfrom Dec 22, 2025
Merged
Fix Bug Command Hub and Support Slimefun Experimental version#25PranavVerma-droid merged 3 commits intoSlimefun-Addon-Community:devfrom
PranavVerma-droid merged 3 commits intoSlimefun-Addon-Community:devfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the plugin to support Slimefun's experimental version (1.21) and Minecraft 1.21.4, while fixing a bug in the Command Hub inventory detection. The changes include dependency upgrades, API migrations, and code cleanup.
- Updated build configuration to use Java 21 and Paper API 1.21.4
- Migrated to Slimefun experimental-SNAPSHOT with corresponding API changes (CustomItemStack, SlimefunItems, BlockExplodeEvent)
- Fixed Command Hub bug where inventory checks used wrong Material constant (BLACK_STAINED_GLASS instead of BLACK_STAINED_GLASS_PANE)
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pom.xml | Updated Java version to 21, migrated to Paper API 1.21.4, updated Slimefun to experimental-SNAPSHOT, updated GuizhanLib to 2.3.0, removed deprecated repositories |
| .vscode/settings.json | Removed IDE-specific configuration file |
| SupportedPluginManager.java | Removed unused Bukkit import |
| framed_uranium.java | Removed unused and duplicate imports (PlayerRightClickEvent, BlockUseHandler, AdvancedTech, duplicate Radioactivity) |
| boosted_carbonado.java | Removed unused imports (PlayerRightClickEvent, Radioactive, Radioactivity, BlockUseHandler, AdvancedTech) |
| FireCake.java | Removed unused imports (AdvancedTech, duplicate Radioactivity) |
| AdvancedSolarGenLib.java | Removed unused imports (Material, EnergyNet, command_hub) |
| AdvancedSolarGen.java | Removed unused import (command_hub) |
| handheld_digger_3.java | Removed unused imports (Material, EventHandler, Listener, PlayerItemDamageEvent) |
| handheld_digger_2.java | Removed unused imports (Material, EventHandler, Listener, PlayerItemDamageEvent) |
| ExplosiveToolBreakBlocksEvent7x7.java | Migrated from org.apache.commons.lang to lang3, removed unnecessary self-package import |
| ExplosiveTool7x7.java | Added ExplosionResult import, updated BlockExplodeEvent constructor for 1.21.4 API compatibility |
| ExplosiveToolBreakBlocksEvent4x4.java | Migrated from org.apache.commons.lang to lang3 |
| ExplosiveTool4x4.java | Added ExplosionResult import, updated BlockExplodeEvent constructor for 1.21.4 API compatibility |
| ExplosiveToolBreakBlocksEvent5x5.java | Migrated from org.apache.commons.lang to lang3 |
| ExplosiveTool5x5.java | Added ExplosionResult import, updated BlockExplodeEvent constructor for 1.21.4 API compatibility |
| command_hub.java | Fixed bug by changing Material.BLACK_STAINED_GLASS to BLACK_STAINED_GLASS_PANE in inventory checks; refactored event handlers with improved null safety and early returns; removed unused imports |
| AdvancedTech.java | Migrated CustomItemStack constructor to static create method; updated all SlimefunItems references to use .item() method |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Collaborator
|
Nice! Good Fix! |
0ed2467
into
Slimefun-Addon-Community:dev
1 check passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
I made this pull request because I fixed the Command Hub bug and made this plugin support the experimental version of slimefun (1.21)
Proposed changes
I changed some code in some files to adjust the slimefun experimental (1.21) and minecraft 1.21.4 api, then I also changed the command_hub.java code to fix the bug
Related Issues (if applicable)
Resolve #24
Checklist
NonnullandNullableannotations to my methods to indicate their behaviour for null values