Skip to content

Conversation

@Absolutionism
Copy link
Contributor

@Absolutionism Absolutionism commented Nov 4, 2024

Description

This PR aims to add support for the upcoming Equippable Components to Skript. Allowing users to modify the components of an item and create new equippable components with features such as: Dispensable, Damageable, Swappable, allowed entities, equip sound, and many more.

This will be marked as draft due to Equippable Components being ApiStatus.Experimental at this time.
Open to any syntax changes.


Target Minecraft Versions: any
Requirements: none
Related Issues: none

@Absolutionism Absolutionism marked this pull request as draft November 4, 2024 23:19
@Efnilite Efnilite added the feature Pull request adding a new feature. label Nov 5, 2024
@Absolutionism Absolutionism marked this pull request as ready for review November 10, 2024 18:32
…leComponents

# Conflicts:
#	src/main/java/ch/njol/skript/Skript.java
#	src/main/resources/lang/default.lang
Copy link
Member

@Moderocky Moderocky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@sovdeeth
Copy link
Member

sovdeeth commented Dec 8, 2024

Can you explain the reasoning behind making the equippable component a new type? Why not just have all the syntaxes act on items, like you already have them doing?

@Absolutionism
Copy link
Contributor Author

Can you explain the reasoning behind making the equippable component a new type? Why not just have all the syntaxes act on items, like you already have them doing?

I believe having this new type will be beneficial for storing template equippable components. I personally feel it to be more intuitive

…leComponents

# Conflicts:
#	src/main/java/ch/njol/skript/util/slot/EquipmentSlot.java
…leComponents

# Conflicts:
#	src/main/java/ch/njol/skript/Skript.java
#	src/main/java/ch/njol/skript/bukkitutil/SoundUtils.java
#	src/main/resources/lang/default.lang
@Burbulinis Burbulinis mentioned this pull request Jan 19, 2025
…leComponents

# Conflicts:
#	src/main/java/ch/njol/skript/Skript.java
#	src/main/java/ch/njol/skript/util/slot/EquipmentSlot.java
#	src/main/resources/lang/default.lang
@Absolutionism Absolutionism requested review from a team as code owners May 11, 2025 07:01
@Absolutionism Absolutionism requested review from UnderscoreTud and sovdeeth and removed request for a team May 11, 2025 07:01
Copy link
Member

@APickledWalrus APickledWalrus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good. Perhaps I could persuade you to use Registration API for syntax? 🙂

Copy link
Member

@APickledWalrus APickledWalrus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Almost ready

@github-project-automation github-project-automation bot moved this from In Review to Awaiting Merge in 2.13 Releases Sep 30, 2025
@Absolutionism Absolutionism merged commit 8f657a8 into SkriptLang:dev/feature Sep 30, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from Awaiting Merge to Done - Awaiting Release in 2.13 Releases Sep 30, 2025
@sovdeeth sovdeeth moved this from Done - Awaiting Release to Done - Released in 2.13 Releases Oct 15, 2025
erenkarakal pushed a commit to erenkarakal/Skript that referenced this pull request Nov 26, 2025
* Starter Commit

* PR ready

* remove "runtime"

* Changes

* Update

* Update

* Overhaul

* Update CondEquipCompInteract.java

* Requested Changes

* Partial Changes

* Update EquippableComponents.sk

* revert

* Record Usage

* Partial Changes

* More Changes

* Changes

* Requested Changes

* Update EquippableComponents.sk

* Update EquippableComponents.sk

* Adooblus Changes

* Update EquippableExperiment.java

* Update Experiment Class  Name

* Requested Changes

* Cloneable Changes

* Update EquippableModule.java

* Update ItemComponentModule.java

* Partial Update

Experiencing some issues, need to see if incorrect merge

* Docs Update

* Missing Import

* More Elements

* Docs Fix

* Fix ItemType

* Revert Misc Changed Files

* Source Retainability

* Update EquippableComponents.sk

* Update ExprEquippableComponent.java

* Requested Changes

* Update Wrapper

* ComponentUtils

* Docs Update

* Update ExprSecBlankEquipComp.java

* Merge dev/feature

* Requested Changes

* Update src/main/java/org/skriptlang/skript/bukkit/itemcomponents/equippable/elements/ExprEquipCompModel.java

Co-authored-by: sovdee <10354869+sovdeeth@users.noreply.github.com>

* Pickles Changes - Part 20

* Update CondEquipCompShearable.java
erenkarakal pushed a commit to erenkarakal/Skript that referenced this pull request Nov 26, 2025
* Starter Commit

* PR ready

* remove "runtime"

* Changes

* Update

* Update

* Overhaul

* Update CondEquipCompInteract.java

* Requested Changes

* Partial Changes

* Update EquippableComponents.sk

* revert

* Record Usage

* Partial Changes

* More Changes

* Changes

* Requested Changes

* Update EquippableComponents.sk

* Update EquippableComponents.sk

* Adooblus Changes

* Update EquippableExperiment.java

* Update Experiment Class  Name

* Requested Changes

* Cloneable Changes

* Update EquippableModule.java

* Update ItemComponentModule.java

* Partial Update

Experiencing some issues, need to see if incorrect merge

* Docs Update

* Missing Import

* More Elements

* Docs Fix

* Fix ItemType

* Revert Misc Changed Files

* Source Retainability

* Update EquippableComponents.sk

* Update ExprEquippableComponent.java

* Requested Changes

* Update Wrapper

* ComponentUtils

* Docs Update

* Update ExprSecBlankEquipComp.java

* Merge dev/feature

* Requested Changes

* Update src/main/java/org/skriptlang/skript/bukkit/itemcomponents/equippable/elements/ExprEquipCompModel.java

Co-authored-by: sovdee <10354869+sovdeeth@users.noreply.github.com>

* Pickles Changes - Part 20

* Update CondEquipCompShearable.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Pull request adding a new feature. needs reviews A PR that needs additional reviews

Projects

No open projects
Status: Done - Released

Development

Successfully merging this pull request may close these issues.

8 participants