Skip to content

Conversation

@Absolutionism
Copy link
Contributor

Problem

No proper support for setting and getting attributes of copper golems and copper blocks.

Solution

Removes 'copper golem' from SimpleEntityData and adds CopperGolemData allowing users to specify the copper state of the copper golem and whether they're waxed.
Adds a condition to check if a copper golem or a copper block is waxed.
Adds an effect to wax or unwax a copper golem or a copper block.
Adds an expression to get/set the time until the next oxidation for a copper golem.
Adds an expression to get/set the pose of a copper golem statue.
Adds an expression to get/set the copper state of a copper golem or a copper block.

Adds 'CopperState' that mimics 'WeatheringCopperState' for the versions that don't have it. Allowing easy use for getting and setting the copper states.

Registers ClassInfo for copper golem statue poses
Registers ClassInfo for 'CopperState' or 'WeatheringCopperState' if it exists.

Adds 'ReflectUtils' that is a utility class for common reflection and caches where possible. Reflection was needed for 'CopperState', 'ExprCopperState', and 'ExprCopperGolemOxidationTime' which led to the creation of ReflectUtils.

Testing Completed

EntityData.sk
EffWax.sk
ExprCopperGolemOxidationTime.sk
ExprCopperGolemPose.sk
ExprCopperState.sk
EffSecSpawn.sk

Supporting Information

N/A


Completes: none
Related: none

@Absolutionism Absolutionism requested a review from a team as a code owner October 18, 2025 00:57
@Absolutionism Absolutionism requested review from Burbulinis and TheMug06 and removed request for a team October 18, 2025 00:57
@skriptlang-automation skriptlang-automation bot added the needs reviews A PR that needs additional reviews label Oct 18, 2025
@sovdeeth sovdeeth added the feature Pull request adding a new feature. label Oct 18, 2025
@sovdeeth sovdeeth moved this to In Review in 2.14 Releases Oct 18, 2025
@sovdeeth
Copy link
Member

Forgot to write this in the last review, but this should all be in the skriptlang package.

@Absolutionism Absolutionism requested a review from a team as a code owner October 23, 2025 20:35
@Absolutionism Absolutionism requested review from APickledWalrus and removed request for a team October 23, 2025 20:35
@Absolutionism Absolutionism added the 2.14 Targeting a 2.14.X version release. label Nov 2, 2025
@github-project-automation github-project-automation bot moved this from In Review to Awaiting Merge in 2.14 Releases Nov 3, 2025
@skriptlang-automation skriptlang-automation bot added feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. and removed needs reviews A PR that needs additional reviews labels Nov 3, 2025
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.

Overall this looks good. I have some suggestions about the organization/modularization.

@github-project-automation github-project-automation bot moved this from Awaiting Merge to In Review in 2.14 Releases Nov 3, 2025
@skriptlang-automation skriptlang-automation bot added feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. and removed feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. labels Nov 3, 2025
# Conflicts:
#	src/main/java/org/skriptlang/skript/bukkit/misc/expressions/ExprTextOf.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.14 Targeting a 2.14.X version release. feature Pull request adding a new feature. feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version.

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

4 participants