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
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

Status: In Review

Development

Successfully merging this pull request may close these issues.

2 participants